Code Review Skill
Overview
A systematic approach to code review that moves beyond "it looks good" to rigorous quality verification. This skill provides specific checklists and procedures for different review types.
When To Use
- •Self-Review: Before submitting a PR or finishing
/work - •Peer Review: When reviewing another agent's or human's code (
/resolve_pr) - •Plan Review: When validating an implementation plan (
/plan_review)
Instrumentation
bash
# Log usage when using this skill ./scripts/log-skill.sh "code-review" "manual" "$$"
What do you want to do?
- •Security Review (Auth, RLS, Input) →
workflows/security-pass.md - •Performance Review (Database, Re-renders) →
workflows/performance-pass.md - •Architecture Review (State, Data Flow) →
workflows/architecture-pass.md - •General Quality Check →
checklists/pre-merge.md
Key Principles
- •Review in Passes: Don't check everything at once. Do a security pass, then a performance pass, etc.
- •Reference Patterns: Always check against
docs/solutions/patterns/critical-patterns.md. - •Verify, Don't Guess: If you see a potential issue, verify it with a quick test or script.