Reviewing Code
Prioritized, actionable feedback. Assume good intent.
Priority Levels
| Level | Prefix | Meaning |
|---|---|---|
| Blocker | 🚨 | Must fix before merge |
| Suggestion | ⚠️ | Should consider |
| Nit | 💭 | Optional polish |
| Question | ❓ | Need clarification |
| Praise | ✨ | Done well |
Checklist
Blockers (Security & Correctness)
- No secrets/credentials in code
- User input validated/sanitized
- Auth checks present where needed
- Edge cases handled (null, empty, errors)
- No race conditions in async code
High Priority (Performance & Logic)
- No N+1 queries
- Large lists paginated
- Expensive ops not in render loops
- Error handling appropriate
Medium Priority (Maintainability)
- Functions have single responsibility
- No copy-paste duplication
- Naming is clear
Output Format
## Summary
[Approve / Request Changes / Discuss]
## 🚨 Blockers
- `file:line` - Issue and fix
## ⚠️ Suggestions
- `file:line` - Suggestion and why
## 💭 Nits
- `file:line` - Minor improvement
## ✨ Good Stuff
- What was done well
Common Issues
React: Missing useEffect deps, state that should be derived, prop drilling
TypeScript: any types, missing null checks
API: Missing loading/error states, unbounded queries
For detailed React/hooks patterns and accessibility checks, see REACT-PATTERNS.md.