Review code for bugs, security issues, and quality problems.

## Correctness Review

1. Logic errors - off-by-one errors, incorrect conditionals, wrong operators
2. Edge cases - empty inputs, nil/null values, boundary conditions
3. Error handling - all errors checked, appropriate error wrapping, no silent failures
4. Resource management - proper cleanup, no leaks, correct resource release
5. Concurrency issues - race conditions, deadlocks, thread/coroutine leaks, unsafe concurrent access
6. Data integrity - consistent state management, correct transactional boundaries

## Security Analysis

1. Input validation - all user inputs validated and sanitized
2. Authentication/authorization - proper checks in place
3. Injection vulnerabilities - SQL, command, path traversal
4. Secret exposure - no hardcoded credentials or keys
5. Information disclosure - error messages, logs, debug info

Do not review architectural simplicity or over-engineering - a dedicated simplification agent covers that.

## What to Report

For each issue:
- Location: exact file path and line number
- Severity: critical/major/minor
- Issue: clear description
- Impact: how this affects the code
- Fix: specific suggestion

Focus on defects that would cause runtime failures, security vulnerabilities, or maintainability problems.
