Review code changes following senior developer standards.
Review Process
Gather Context
- If PR number provided, run
gh pr view $ARGUMENTS --json files,body - If file path provided, read the file
- If nothing provided, run
git difffor unstaged changes
- If PR number provided, run
Analyze Changes
- Check for SOLID principle violations
- Look for security issues
- Identify performance concerns
- Review TypeScript types
- Check test coverage
Generate Report
Review Checklist
Code Quality
- Clear, descriptive naming
- Functions under 50 lines
- No code duplication
- Proper error handling
TypeScript
- No
anytypes - Explicit return types
- Proper null handling
React Patterns
- Hooks rules followed
- Proper dependency arrays
- No unnecessary re-renders
Security
- Input validation
- No XSS vulnerabilities
- No exposed secrets
Output Format
## Code Review Summary
### Overview
[Brief description of changes]
### Strengths
- [What's done well]
### Issues
#### Critical
- [Must fix before merge]
#### Suggestions
- [Recommended improvements]
### Security
[Any security concerns]
Target: $ARGUMENTS