Default code-quality route for broad code review, PR review, maintainability, correctness, and regression-risk checks. Do not use as the primary route for dedicated OWASP/security audits, review-feedback handling, completion verification, AI-code cleanup, or TDD/test-first work.
SKILL.md
Code Reviewer
Complete toolkit for code reviewer with modern tools and best practices.
Routing Boundary
Use code-reviewer when the user asks for a fresh review of code or a PR:
correctness and likely bugs
maintainability and readability
regression risk
test coverage gaps from a reviewer's point of view
# Use the analyzer script
python scripts/code_quality_checker.py .
# Review recommendations
# Apply fixes
3. Implement Best Practices
Follow the patterns and practices documented in:
references/code_review_checklist.md
references/coding_standards.md
references/common_antipatterns.md
Best Practices Summary
Code Quality
Follow established patterns
Write comprehensive tests
Document decisions
Review regularly
Performance
Measure before optimizing
Use appropriate caching
Optimize critical paths
Monitor in production
Security
Flag obvious security risks during a general review.
Route dedicated OWASP, auth, secret, injection, or threat-model requests to security-reviewer.
Maintainability
Write clear code
Use consistent naming
Add helpful comments
Keep it simple
Common Commands
# Development
npm run dev
npm run build
npm run test
npm run lint
# Analysis
python scripts/code_quality_checker.py .
python scripts/review_report_generator.py --analyze
# Deployment
docker build -t app:latest .
docker-compose up -d
kubectl apply -f k8s/
Troubleshooting
Common Issues
Check the comprehensive troubleshooting section in references/common_antipatterns.md.
