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

Issue Severity

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