| name | compose:review |
| description | Use when completing tasks, implementing major features, or before merging to verify work meets requirements |
Code Review
When to Review
Mandatory:
- After completing major feature
- Before merge to main
Optional but valuable:
- When stuck (fresh perspective)
- After fixing complex bug
Review Checklist
- Correctness — Does the code do what it's supposed to?
- Edge cases — Are error conditions handled?
- Tests — Are there adequate tests? Do they test behavior, not implementation?
- Readability — Is the code clear and maintainable?
- Security — Are there any security concerns?
How to Review
git diff main...HEAD
git diff HEAD~1
Review each change and provide feedback.
Acting on Feedback
- Fix Critical issues immediately
- Fix Important issues before proceeding
- Note Minor issues for later
- Push back if reviewer is wrong (with reasoning)