SKILL.md
Code Review Skill
Purpose
Provide consistent, high-signal code reviews by focusing on correctness, clarity, maintainability, security, performance, tests, and developer experience.
Inputs
- PR description and linked issues
- Code diffs and file context
- Existing conventions in the repository
Review Checklist
Correctness
- Does the change satisfy the requirements?
- Are edge cases handled?
- Are errors propagated/handled appropriately?
Readability & Maintainability
- Is the code easy to follow?
- Are names clear and consistent?
- Is complexity justified?
- Are public APIs documented?
Security
- Validate/encode untrusted input
- Avoid leaking secrets/PII
- Check authZ/authN boundaries
Performance
- Watch for N+1 queries, unnecessary allocations, inefficient loops
- Ensure any tradeoffs are explained
Testing
- Are tests added/updated?
- Do tests cover failure cases and boundaries?
- Are tests deterministic and fast?
Style & Consistency
- Follows project lint/format rules
- No dead code or commented-out blocks
Output Format
- Summary of change
- Key issues (must-fix)
- Suggestions (nice-to-have)
- Questions/uncertainties
Tone
Be constructive and specific. Prefer actionable suggestions with examples.