Review Skill
When to use this skill
- before delivering final code changes
- after implementing a planned set of steps
- before merging or shipping
Severity levels
- Blocker: wrong behavior, security issue, data loss risk, broken tests/build
- Major: likely bug, missing edge cases, poor reliability
- Minor: style, clarity, small maintainability issues
- Nit: optional polish
Checklist
- Correctness vs requirements
- Edge cases & error handling
- Tests (adequate coverage, meaningful assertions)
- Security (secrets, auth, injection, unsafe defaults)
- Performance (obvious hotspots, N+1, unnecessary work)
- Readability & maintainability
- Docs / comments updated if needed
Output format
- Blockers
- Majors
- Minors
- Nits
- Overall summary + next actions