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