---
name: review-pr
description: Perform a structured code review of a pull request. Checks code quality, test coverage, security, and documentation.
disable-model-invocation: true
---

# PR Review

Review the current changes systematically:

## 1. Scope Check
- Does the PR do one thing? (Single responsibility)
- Is the PR a reasonable size? (< 400 lines preferred)
- Does the title/description match the changes?

## 2. Code Quality
- Is the code readable?
- Are names descriptive?
- Is there unnecessary duplication?
- Are there any code smells?

## 3. Tests
- Are new features tested?
- Are edge cases covered?
- Do tests have descriptive names?
- Is test coverage adequate?

## 4. Security
- Any user input handling changes?
- Auth/authz changes reviewed?
- Secrets or PII exposure?

## 5. Documentation
- Are public APIs documented?
- Is the README updated if needed?
- Are breaking changes noted?

Provide a summary: ✅ Approve / 🔄 Request Changes / ❓ Questions