# Code Review

> Review code for quality, security, and maintainability. Use when reviewing pull requests, examining code changes, doing code review, QA checks, or architecture review for COMPLEX tasks. Includes QA checklist and CTO review template.

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

---


# Code Review

## QA Checklist (every review)

### Code Quality
- [ ] 0 linter errors
- [ ] 0 TypeScript errors
- [ ] No console.log (except debug)
- [ ] Tests pass
- [ ] Edge cases handled
- [ ] Error cases handled

### Naming Standards

| Type | Convention | Example |
|------|-----------|---------|
| Files (components) | PascalCase | `ComponentName.tsx` |
| Files (utils) | camelCase | `utilName.ts` |
| Functions | camelCase | `functionName()` |
| Components | PascalCase | `ComponentName` |
| Constants | UPPER_SNAKE | `CONSTANT_NAME` |

### JTBD Verification (for user-facing features)

- [ ] Job Story implemented
- [ ] UI text about benefits, not features
- [ ] Minimum steps to result
- [ ] No confusing terminology

## Feedback Format

- **CRITICAL**: Must fix before merge
- **SUGGESTION**: Consider improving
- **NICE TO HAVE**: Optional enhancement

## CTO Review (for COMPLEX tasks)

For architecture decisions, use the full template in [CTO-REVIEW.md](references/CTO-REVIEW.md).

Apply CTO Review when:
- Change affects >5 files
- API change used by others
- Data migration
- New architectural concept