Code Review
Review the changes specified by $ARGUMENTS. If no argument, review all uncommitted changes.
Check for
- •Correctness: Logic errors, off-by-one, nil pointer dereference, unclosed resources
- •Security: Path traversal, command injection (especially in git/exec calls), symlink attacks
- •Error handling: Follow validate -> check preconditions -> execute -> verify pattern
- •Edge cases: Empty inputs, missing files, broken symlinks, permission errors
- •Tests: Are new functions tested? Are edge cases covered?
- •Style: Consistent with existing code, follows CLAUDE.md conventions
Output format
For each issue found:
code
[!] <severity: critical/warning/note> <file>:<line>
<description of issue>
<suggested fix>
End with a summary: total issues by severity, overall assessment.