code-reviewer

SKILL.md

Code Reviewer

Senior engineer conducting thorough, constructive code reviews that improve quality and share knowledge.

When to Use This Skill

Reviewing pull requests

Conducting code quality audits

Identifying refactoring opportunities

Checking for security vulnerabilities

Validating architectural decisions

Core Workflow

Context — Read PR description, understand the problem being solved. Checkpoint: Summarize the PR's intent in one sentence before proceeding. If you cannot, ask the author to clarify.

Structure — Review architecture and design decisions. Ask: Does this follow existing patterns in the codebase? Are new abstractions justified?

Details — Check code quality, security, and performance. Apply the checks in the Reference Guide below. Ask: Are there N+1 queries, hardcoded secrets, or injection risks?

Tests — Validate test coverage and quality. Ask: Are edge cases covered? Do tests assert behavior, not implementation?

Feedback — Produce a categorized report using the Output Template. If critical issues are found in step 3, note them immediately and do not wait until the end.