---
name: Code Review
slug: code-review-15
category: Quality
description: Code Review performs a systematic review for correctness, security, performance, style, and testing gaps. Use it to inspect changed code and report concrete issues with suggested fixes.
github: "https://github.com/vstorm-co/pydantic-deepagents/tree/main/pydantic_deep/bundled_skills/code-review"
language: Python
stars: 1037
forks: 126
install: "npx degit https://github.com/vstorm-co/pydantic-deepagents/tree/main/pydantic_deep/bundled_skills/code-review ~/.claude/skills/code-review"
installs_to: ~/.claude/skills/code-review
source_path: pydantic_deep/bundled_skills/code-review/SKILL.md
collection_size: 14
category_size: 1354
collection_url: "https://dirskills.com/collections/vstorm-co/pydantic-deepagents"
added: 2026-08-21T05:14:09.187Z
last_synced: 2026-08-21T05:14:09.187Z
canonical_url: "https://dirskills.com/skills/code-review-15"
---

# Code Review

Code Review performs a systematic review for correctness, security, performance, style, and testing gaps. Use it to inspect changed code and report concrete issues with suggested fixes.

**Install:**

```bash
npx degit https://github.com/vstorm-co/pydantic-deepagents/tree/main/pydantic_deep/bundled_skills/code-review ~/.claude/skills/code-review
```

## README

# Code Review

Perform a systematic code review covering these categories:

## Review Checklist

### 1. Correctness
- Logic errors, off-by-one, null/None handling
- Edge cases: empty inputs, large inputs, concurrent access
- Error handling: are exceptions caught and handled properly?

### 2. Security
- Input validation and sanitization
- SQL injection, XSS, command injection
- Secrets in code, hardcoded credentials
- Authentication and authorization checks

### 3. Performance
- Unnecessary loops, N+1 queries
- Missing indexes for database queries
- Large memory allocations, unbounded collections
- Blocking calls in async code

### 4. Style & Maintainability
- Naming clarity (variables, functions, classes)
- Function length — split if >30 lines
- Dead code, commented-out code
- Missing type annotations

### 5. Testing
- Are new code paths covered by tests?
- Are edge cases tested?
- Are error paths tested?

## Output Format

For each issue found:
- **File:line** — category — description — suggested fix
- Severity: critical / warning / suggestion