AI agent skill · by @vstorm-co

Code Review

Systematic code review for bugs, security, style, and performance

Archived 2026-10-23 · Source updated 2026-10-04 · License: MIT. Check the original repository for the latest version.

SKILL.md instructions

# 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