code-review-framework

What problem does it solve?

This Skill provides a comprehensive, structured framework for conducting code reviews, addressing issues like inconsistent feedback, missed critical bugs, and subjective comments. It ensures thorough evaluation across correctness, security, performance, and maintainability, leading to higher code quality and fewer post-release defects.

Core Features & Use Cases

• Structured Checklist: Offers a detailed checklist covering correctness (error handling, edge cases), security (input validation, SQL injection), and performance (N+1 queries, async usage).
• Severity & Categorization: Guides on categorizing comments by ReviewCategory (e.g., SECURITY, PERFORMANCE) and ReviewSeverity (BLOCKING, MAJOR, MINOR).
• Specific Feedback: Emphasizes providing concrete suggestions with code examples, rather than vague comments.
• Review Summary: Provides a template for summarizing review findings, including blocking issues, major concerns, and positive feedback.
• Use Case: A senior engineer is reviewing a pull request for a critical backend service. This skill helps them systematically check for SQL injection vulnerabilities, N+1 query problems, and proper error handling, ensuring the code meets high standards before merging.

Quick Start

Review the attached Python code for correctness, security vulnerabilities, and performance issues, providing specific suggestions.