| name | pre-commit-check |
| description | Pre-commit validation checklist before staging and committing code. Use BEFORE every git commit to ensure code quality, tests pass, and documentation is updated. Triggers on "commit", "stage changes", "ready to commit", or when completing a feature or fix. |
Pre-Commit Checklist
Run through this checklist before every commit.
Required Checks
1. Build Verification
npm run build
- Build completes without errors
- No new warnings introduced
2. Test Suite
npm test
- All tests pass
- No skipped tests that should run
- Coverage not decreased (if tracked)
3. Linting (if configured)
npm run lint
- No linting errors
- No new warnings
Context Updates
4. ConPort Documentation
- Log any decisions made during this work (
mcp__conport__log_decision) - Update progress entries to DONE (
mcp__conport__update_progress) - Log any new patterns discovered (
mcp__conport__log_system_pattern) - Link related items (
mcp__conport__link_conport_items)
5. Serena Memory
- Document architectural learnings if applicable (
write_memory)
Code Review
6. Self-Review
- Changes match the task requirements
- No debug code left behind (console.log, debugger statements)
- No commented-out code without explanation
- No hardcoded values that should be configurable
7. Automated Reviews (for significant changes)
Launch these sub-agents in parallel for comprehensive review:
Code Quality Review:
Task(subagent_type="pr-review-toolkit:code-reviewer", prompt="Review unstaged changes for code quality, bugs, and adherence to project conventions")
Security Review (when changes involve):
- User input handling, file uploads, URL processing
- API keys, authentication, authorization
- Data sanitization or validation
- External API calls or network requests
Task(subagent_type="security-pro:security-auditor", prompt="Review changes for security vulnerabilities, OWASP compliance, and secure coding practices")
Frontend Security (for UI changes):
Task(subagent_type="frontend-mobile-security:frontend-security-coder", prompt="Review frontend changes for XSS prevention, output sanitization, and client-side security")
Backend Security (for API/service changes):
Task(subagent_type="backend-api-security:backend-security-coder", prompt="Review backend changes for input validation, authentication security, and API security")
Architecture Review (for structural changes):
Task(subagent_type="code-review-ai:architect-review", prompt="Review changes for architectural integrity, patterns adherence, and maintainability")
- Address all high-priority findings before committing
Commit
8. Stage and Commit
git add -A
git status # Review staged files
git commit -m "type: descriptive message"
Commit message format:
feat:- New featurefix:- Bug fixrefactor:- Code refactoringdocs:- Documentation onlytest:- Adding/updating testschore:- Maintenance tasks