# Aico Code Review

> 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)

- Skill: `aibot88/aico-code-review` (Agent Skill, multi-file: 2 files)
- Install (CLI): `npx skillmds@latest add aibot88/aico-code-review`
- Raw SKILL.md: https://api.skillmd.com/api/skills/aibot88/aico-code-review/raw
- Safety review: pending (external: skill-scanner PASS, skillspector PASS)
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: Security
- Author: aibot88 (https://skillmd.com/u/aibot88)
- Updated: 2026-09-09
- Page: https://skillmd.com/skills/aibot88/aico-code-review

---


# 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

```markdown
## 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