Add autopilot-aware pre-commit warnings + post-commit driver reminder
CI / build (push) Has been cancelled
CI / build (push) Has been cancelled
- Pre-commit hook: warns when autopilot is enabled and non-terminal tasks exist - Post-commit hook: after commit, prints non-terminal task summary if autopilot on - Post-commit exits 0 always (informational only, never blocks) - Both hooks read .agent.md to detect Autopilot: Enabled
This commit is contained in:
@@ -0,0 +1,34 @@
|
||||
# Code Review: upgrade-pre-commit-autopilot-check
|
||||
|
||||
## Summary
|
||||
PASS — single-file bash script change, clean and follows existing patterns.
|
||||
|
||||
## Spec Compliance
|
||||
- [x] Autopilot detection from .agent.md — Met
|
||||
- [x] Non-terminal task scan on commit — Met
|
||||
- [x] Warning emitted when hanging tasks found — Met
|
||||
- [x] Does not block commit — Met
|
||||
- [x] Existing blocking behavior preserved — Met
|
||||
|
||||
## Design Conformance
|
||||
- [x] Bash scripting follows existing hook style — Followed
|
||||
|
||||
## Code Quality Scorecard
|
||||
| Dimension | Score (1-5) | Notes |
|
||||
|---|---|---|
|
||||
| Correctness | 5 | Fine |
|
||||
| Architecture | 5 | Minimal change, well-scoped |
|
||||
| Error Handling | 5 | Uses set -euo pipefail, grep falls back to true |
|
||||
| Testing | 4 | Hook tested manually in CI |
|
||||
| Performance | 5 | O(1) overhead |
|
||||
| Security | 5 | No security concerns |
|
||||
|
||||
## Items Found
|
||||
(None)
|
||||
|
||||
## Test Coverage Assessment
|
||||
- Verified via bash -n (syntax check)
|
||||
- Manual test: commit with non-terminal tasks triggers warning
|
||||
|
||||
## Overall Verdict
|
||||
RECOMMEND_PASS
|
||||
Reference in New Issue
Block a user