- name
- reviewing-code
- description
- Provides structured code review with prioritized feedback. Use when reviewing PRs, analyzing code quality, checking for bugs, or auditing changes. Triggers on "review this", "check this code", PR reviews, or code quality questions.
- allowed-tools
- Read, Grep, Glob
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 wellCommon 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.