devclarityai Use this template to review a PR diff with focused, actionable feedback.
INPUTS
- PROJECT_CONTEXT
- PR_DIFF
- (optional) PR_DESCRIPTION
- (optional) TICKET_LINKS
INSTRUCTIONS
- Scan PR_DIFF for correctness, style, security, and performance.
- Ground comments in 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
- Builds/CI green
- Lint/format pass
- Tests updated/added
- Backward compatible
- Feature flagged
- Observability added