Review pull request: $ARGUMENTS
Follow this checklist:
- Read the PR: Run
gh pr view $ARGUMENTSandgh pr diff $ARGUMENTS - Understand scope: What is this PR trying to accomplish?
- Review for correctness:
- Does the code do what it claims?
- Are there edge cases not handled?
- Are there off-by-one errors, null pointer risks, or race conditions?
- Review for architecture:
- Does this follow existing patterns in the codebase?
- Is there code duplication that should be extracted?
- Are new dependencies justified?
- Review for security:
- Input validation on external data?
- SQL injection, XSS, or other OWASP risks?
- Secrets or credentials exposed?
- Review for testing:
- Are new behaviors covered by tests?
- Are edge cases tested?
- Summarize: Provide a clear summary with specific, actionable feedback