code-review

# 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.