#0058 Command-Line Interface #143

Merged

fuzzy merged 1 commit from feature/0058-command-line-interface into main

2026-07-06 06:55:30 +00:00

automation commented

2026-07-06 06:53:08 +00:00

Owner

Completes the CLI tool with all planned flags.

Changes:

src/hottea/cli.py — added --dry-run and --verbose flags, refactored logic
tests/test_cli.py — 10 tests covering all flags

Usage:

hottea [--end-date YYYY-MM-DD] [--output FILE] [--dry-run] [--verbose]

Flags:

--end-date YYYY-MM-DD — filter commits on/before date
--output FILE / -o — write SQL to file (default: stdout)
--dry-run — preview without API calls or writes
--verbose / -v — DEBUG-level logging

Closes: #0058, #0053, #0054, #0055, #0056, #0057

Completes the CLI tool with all planned flags. **Changes:** - `src/hottea/cli.py` — added `--dry-run` and `--verbose` flags, refactored logic - `tests/test_cli.py` — 10 tests covering all flags **Usage:** ``` hottea [--end-date YYYY-MM-DD] [--output FILE] [--dry-run] [--verbose] ``` **Flags:** - `--end-date YYYY-MM-DD` — filter commits on/before date - `--output FILE` / `-o` — write SQL to file (default: stdout) - `--dry-run` — preview without API calls or writes - `--verbose` / `-v` — DEBUG-level logging **Closes:** #0058, #0053, #0054, #0055, #0056, #0057

automation added 1 commit

2026-07-06 06:53:08 +00:00

feat(cli): add --dry-run and --verbose flags

CI / lint-and-test (pull_request) Successful in 1m25s

Details

e9b5e75acf

Complete the CLI interface with all planned flags.

- --dry-run: preview SQL without API calls or file writes
- --verbose/-v: enable DEBUG-level logging
- --end-date, --output, --help already existed

Closes #53, #54, #55, #56, #57, #58

fuzzy scheduled this pull request to auto merge when all checks succeed

2026-07-06 06:55:13 +00:00

the.auditor reviewed

2026-07-06 06:55:25 +00:00

the.auditor left a comment

Audit Review — PR #143

Clean CLI completion. Dry-run mode correctly short-circuits API calls and file writes with informative log messages. Verbose mode sets DEBUG logging. All 6 issues (#0053-#0058) addressed. Parser tests cover individual and combined flags.

Note: dry-run uses forgejo_user_id=0 and repo_id=0 placeholders. When validation (#0049/#0051) is integrated later, these will need to either use valid placeholder values or skip validation in dry-run mode. Not a blocking issue — just a heads-up for future work.

No suggestions — clean PR.

Summary

Result: COMMENT — no issues found.

## Audit Review — PR #143 Clean CLI completion. Dry-run mode correctly short-circuits API calls and file writes with informative log messages. Verbose mode sets DEBUG logging. All 6 issues (#0053-#0058) addressed. Parser tests cover individual and combined flags. Note: dry-run uses `forgejo_user_id=0` and `repo_id=0` placeholders. When validation (#0049/#0051) is integrated later, these will need to either use valid placeholder values or skip validation in dry-run mode. Not a blocking issue — just a heads-up for future work. No suggestions — clean PR. ## Summary **Result:** COMMENT — no issues found.

the.auditor approved these changes

2026-07-06 06:55:28 +00:00

the.auditor left a comment

APPROVED. CLI complete with all planned flags. Merge when ready.

fuzzy merged commit e9b5e75acf into main

2026-07-06 06:55:30 +00:00

fuzzy deleted branch feature/0058-command-line-interface

2026-07-06 06:55:31 +00:00