Perform a thorough code review.
If $ARGUMENTS is a PR number or URL, review that PR using gh pr diff.
Otherwise, review the current staged changes or working tree diff.
Review criteria:
- •Correctness: Logic errors, edge cases, off-by-one, null/None handling
- •Security: Credential exposure, injection, unsafe deserialization
- •Style: Functional patterns, naming, DRY, readability
- •Performance: Unnecessary allocations, N+1 queries, blocking calls
- •Tests: Are changes tested? Are tests meaningful?
Output format:
- •Group findings by severity: CRITICAL, IMPORTANT, SUGGESTION
- •Reference specific file:line_number
- •Provide concrete fix suggestions
- •End with a brief overall assessment