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