Warn about missing pre-commit hooks during session workflows#167
Conversation
Greptile SummaryThis PR changes the pre-commit hook enforcement model from a hard commit gate to a soft advisory: a missing hook no longer aborts auto-commit, but instead triggers a
Confidence Score: 5/5Safe to merge. The change is advisory-only and removes no commit-blocking behavior that existing callers depended on in a correctness-critical way. All new code paths are exercised by unit tests (including the deduplication invariant checked at both in-memory and database levels) and a reworked E2E test. No error path is silently swallowed — unexpected check_pre_commit_hook_ready errors fall through to warn! logging. No files require special attention. Reviews (5): Last reviewed commit: "Warn about missing pre-commit hooks duri..." | Re-trigger Greptile |
4c36865 to
dbffc4b
Compare
- Check configured hooks before session creation and after successful auto-commit. - Show actionable `prek install` and `pre-commit install` commands while allowing sessions to continue. - Record deduplicated commit warnings and add unit, E2E, and documentation coverage. Co-Authored-By: [Agentty](https://github.com/agentty-xyz/agentty)
|



prek installandpre-commit installcommands while allowing sessions to continue.Co-Authored-By: Agentty