| name | aico-code-review |
| description | Request structured code review to catch correctness, security, performance, and readability issues. Reviews should happen early and often.
Use this skill when:
- Completing a task and need quality check
- After major feature implementation
- Before merging to main branch
- When stuck and need fresh perspective on code
- User asks for "code review", "review my code", "check my code"
- Subagent-driven workflow needs spec compliance or quality review
Review categories: Correctness, Tests, Security, Performance, Readability, Error Handling
Severity levels: Critical (fix now), Important (fix before proceeding), Minor (note for later)
|
Code Review
When to Request Review
Mandatory:
- After completing each task
- After implementing major feature
- Before merge to main
Optional:
- When stuck (fresh perspective)
- Before refactoring
- After fixing complex bug
Review Checklist
| Category | Check |
|---|
| Correctness | Does it do what it should? |
| Tests | Are there tests? Do they pass? |
| Security | Any vulnerabilities? |
| Performance | Any obvious bottlenecks? |
| Readability | Is code clear and maintainable? |
| Error Handling | Are errors handled properly? |
Issue Severity
| Severity | Action |
|---|
| Critical | Fix immediately, blocks progress |
| Important | Fix before proceeding |
| Minor | Note for later, can proceed |
Review Output Template
## Code Review: [Feature/Task Name]
### Files Modified
- `path/to/file.ts` - [what changed]
### Issues
**Critical:**
- [ ] [Issue description]
**Important:**
- [ ] [Issue description]
**Minor:**
- [ ] [Issue description]
### Assessment
- [ ] Ready to proceed
- [ ] Needs fixes (see issues above)
Key Rules
- ALWAYS review after each task completion
- MUST fix Critical issues immediately
- MUST fix Important issues before proceeding
- Minor issues can be noted for later
- If reviewer is wrong, push back with technical reasoning
Common Mistakes
- ❌ Skip review because "it's simple" → ✅ Review everything
- ❌ Ignore Critical issues → ✅ Fix immediately
- ❌ Proceed with Important issues → ✅ Fix first
