What problem does it solve? Reviewing pull requests consistently is hard: reviewers miss security risks, performance issues, and code smells, and feedback is often vague or inconsistent. This Skill provides a structured, checklist-driven review process covering functionality, quality, security, and performance. ## Core Features & Use Cases - Structured Review Checklists: Covers functionality, code quality (DRY, naming, magic numbers), security (SQL injection, XSS, auth), and performance (N+1 queries, memory leaks). - Code Smell Detection: A reference table of common smells (Long Method, God Class, Deep Nesting) with refactoring solutions, plus detection commands like ESLint complexity rules and SonarQube. - PR Best Practices: Guidance on ideal PR size, description templates, conventional commit messages, and comment prefixes (blocking, suggestion, question, nitpick, praise). - Use Case: When a teammate opens a 350-line PR, run the three-phase workflow (Triage, Deep Dive, Feedback) to verify CI status, check architecture conformance, and leave constructive prefixed comments. ## Quick Start Review this pull request using the code review checklist and flag any code smells, security risks, or missing tests.
