Original skill document
Rendered from the source repository. Headings, examples, code, tables, links, and referenced images are preserved.
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
markdown
## 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.
