feat(polecat): add explicit lint/test step to decommission checklist (#934)

- Add quality gates (lint, format, tests) as step 1 before committing
- Support both npm and Go project types
- Add explicit warning: "DO NOT commit if lint or tests fail"
- Explain why manual checks are needed (worktrees may not trigger hooks)

Fixes hq-lint1

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
Mad Dan
2026-01-24 23:48:38 -06:00
committed by GitHub
parent db60489d0f
commit 9dcddaf13d

View File

@@ -181,15 +181,24 @@ Question: <what you need>"
## Completion Protocol (MANDATORY)
When your work is done, follow this EXACT checklist - **step 4 is REQUIRED**:
When your work is done, follow this EXACT checklist - **step 5 is REQUIRED**:
⚠️ **DO NOT commit if lint or tests fail. Fix issues first.**
```
[ ] 1. Tests pass: go test ./...
[ ] 2. Commit changes: git add <files> && git commit -m "msg (issue-id)"
[ ] 3. Sync beads: bd sync
[ ] 4. Self-clean: gt done ← MANDATORY FINAL STEP
[ ] 1. Run quality gates (ALL must pass before committing):
- For npm projects: npm run lint && npm run format && npm test
- For Go projects: go test ./... && go vet ./...
[ ] 2. Stage changes: git add <files>
[ ] 3. Commit changes: git commit -m "msg (issue-id)"
[ ] 4. Sync beads: bd sync
[ ] 5. Self-clean: gt done ← MANDATORY FINAL STEP
```
**Quality gates are not optional.** Git worktrees may not trigger pre-commit hooks
reliably, so you MUST run lint/format/tests manually before every commit. CI will
catch failures, but by then you've wasted the merge queue's time.
⚠️ **CRITICAL: You MUST run `gt done` after completing work. This is not optional.**
The `gt done` command (self-cleaning):