What problem does it solve?
Reviews often miss subtle security and performance issues, resulting in fragile code and technical debt. This skill provides a structured, end-to-end review approach that emphasizes security, maintainability, and adherence to best practices.
Core Features & Use Cases
- Understand Context: Before evaluating changes, identify purpose, requirements, tests, and scope.
- Code Quality Analysis: Check naming, SRP, DRY, error handling, logging, structure, and readability against a standard checklist.
- Security Review: Verify input validation, authentication/authorization checks, and safe dependency usage.
- Performance Review: Detect N+1 queries, unnecessary loops, memory concerns, and caching opportunities.
- Test & Documentation Review: Ensure adequate test coverage, edge-case handling, and update documentation.
- Use Case: Ideal for PR reviews, pre-merge checks, and periodic code quality audits across projects.
Quick Start
Review the target PR by following the six steps to evaluate context, quality, security, performance, tests, and documentation, then provide actionable feedback.