# Code Review

> Code quality review checklist

- Skill: `hongmaple0820/code-review` (Agent Skill)
- Install (CLI): `npx skillmds@latest add hongmaple0820/code-review`
- Raw SKILL.md: https://api.skillmd.com/api/skills/hongmaple0820/code-review/raw
- Safety review: pending
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: Coding & Dev Tools
- Author: hongmaple0820 (https://skillmd.com/u/hongmaple0820)
- Updated: 2026-09-17
- Page: https://skillmd.com/skills/hongmaple0820/code-review

---


# Code Review Checklist

Review all code before merge.

## Security

- [ ] No hardcoded secrets
- [ ] Input validated at boundaries
- [ ] SQL uses parameterized queries
- [ ] No XSS vulnerabilities
- [ ] Auth/authorization correct

## Quality

- [ ] Functions < 50 lines
- [ ] Files < 800 lines
- [ ] No deep nesting (>4 levels)
- [ ] Errors handled explicitly
- [ ] No mutation of inputs

## Performance

- [ ] No N+1 queries
- [ ] Pagination on large datasets
- [ ] Expensive operations cached
- [ ] No unbounded loops

## Testing

- [ ] Coverage >= 80%
- [ ] Edge cases covered
- [ ] Error paths tested

## Severity

| Level | Action |
|-------|--------|
| CRITICAL | Block merge |
| HIGH | Fix before merge |
| MEDIUM | Consider fixing |
| LOW | Optional |