# Code Review

> Technical code review for quality and bugs on recently changed files Use when this capability is needed.

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

---


# Code Review

Perform a technical code review on recently changed files.

## Gather Context

Read CLAUDE.md and key conventions in `.claude/rules/` to understand project standards.

Then examine changes:

```bash
git status
git diff HEAD
git diff --stat HEAD
git ls-files --others --exclude-standard
```

## Review Process

Read each changed and new file **in its entirety** (not just the diff) to understand full context.

For each file, analyze for:

1. **Logic Errors** - Off-by-one, incorrect conditionals, missing error handling, race conditions
2. **Security Issues** - SQL injection, XSS, insecure data handling, exposed secrets
3. **Performance Problems** - N+1 queries, inefficient algorithms, memory leaks
4. **Code Quality** - DRY violations, overly complex functions, poor naming
5. **Pattern Adherence** - CQRS, DDD, module structure, naming conventions, authorization policies

## Verify Issues

- Run specific tests for issues found
- Confirm type errors are legitimate
- Validate security concerns with context

## Output

Save to `.agents/code-reviews/{appropriate-name}.md`

**Stats:**
- Files Modified / Added / Deleted
- Lines added / removed

**For each issue:**

```
severity: critical|high|medium|low
file: path/to/file
line: 42
issue: [one-line description]
detail: [why this is a problem]
suggestion: [how to fix it]
```

If no issues found: "Code review passed. No technical issues detected."

## Important

- Be specific (line numbers, not vague complaints)
- Focus on real bugs, not style preferences
- Suggest fixes, don't just complain
- Flag security issues as CRITICAL

---
> Converted and distributed by [TomeVault](https://tomevault.io/claim/andrewmclachlan) — claim your Tome and manage your conversions.
<!-- tomevault:4.0:skill_md:2026-04-11 -->