| name | code-review |
| description | Review a PR diff with focused, actionable feedback |
Use this template to review a PR diff with focused, actionable feedback.
INPUTS
- PR diff - the pull request diff to review
- (optional) PR description
- (optional) Ticket links
INSTRUCTIONS
- Scan the PR diff for correctness, style, security, and performance.
- Ground comments in the project context (frameworks, languages, databases,
messaging systems).
- Prefer specific inline suggestions with minimal working patches.
- Flag test gaps and missing docs. Propose concrete test cases.
- Label severity: Blocker, Major, Minor, Nit.
- Keep lines <= 80 chars.
OUTPUT FORMAT
Summary
- Scope:
- Impact:
- Risk level:
Positives
- Code quality wins:
- Good patterns:
- Tests/documentation:
Issues by Severity
Blockers
- [file:line] Problem -> Why it matters -> Fix suggestion
// patchlet
Major
Minor
Nits
Security & Compliance
- Authentication/authorization:
- Input validation/injection prevention:
- Secrets/logging/sensitive data:
- Third-party integrations:
- Data privacy/compliance:
Performance
- Hot paths:
- Database query optimization:
- Caching/TTL:
- Async/concurrency handling:
Testing Gaps
- Unit:
- Integration/e2e:
- Property/fuzz:
- Load/reliability:
Documentation
- Changelog:
- Architecture notes:
- README/code comments:
Inline Review
- [file path]
- line X: comment
- line Y: comment
Review Checklist
