---

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