fix: add top-level exception handler to main() #169

Merged

fuzzy merged 1 commit from fix/141-cli-exception-handling into main

2026-07-06 09:07:23 +00:00

automation commented

2026-07-06 08:55:03 +00:00

Owner

Wraps main() in _entry_point() with a top-level try/except that catches all exceptions, logs them via logger.exception(), and prints a user-friendly message to stderr instead of a raw traceback.

Closes #141

Wraps `main()` in `_entry_point()` with a top-level try/except that catches all exceptions, logs them via `logger.exception()`, and prints a user-friendly message to stderr instead of a raw traceback. Closes #141

automation added 1 commit

2026-07-06 08:55:03 +00:00

fix(cli): add top-level exception handler to main()

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

Details

8a616a9fd4

Wrap main() in _entry_point() that catches all exceptions,
logs them, and prints a user-friendly message instead of
a raw traceback.

Closes #141

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

2026-07-06 09:06:52 +00:00

the.auditor reviewed

2026-07-06 09:07:19 +00:00

the.auditor left a comment

Audit Review — PR #169

Clean fix for #141. _entry_point() wraps main() with top-level exception handler. Full traceback logged via logger.exception(), user gets friendly message + hint to use --verbose. Pragmatic approach for a CLI tool.

No suggestions.

Summary

Result: COMMENT — no issues found.

## Audit Review — PR #169 Clean fix for #141. `_entry_point()` wraps `main()` with top-level exception handler. Full traceback logged via `logger.exception()`, user gets friendly message + hint to use `--verbose`. Pragmatic approach for a CLI tool. No suggestions. ## Summary **Result:** COMMENT — no issues found.

the.auditor approved these changes

2026-07-06 09:07:21 +00:00

the.auditor left a comment

APPROVED. No more raw tracebacks to users, closes #141. Merge when ready.