code-review

# Code Review Skill ## Overview Perform comprehensive code reviews analyzing bugs, style issues, performance bottlenecks, security vulnerabilities, and adherence to best practices. ## Workflow ### 1. Scan Code Structure - Use `view` tool to read all relevant files in the codebase - Identify file types, frameworks, and languages used - Map dependencies and module relationships ### 2. Analyze Code Quality Check for the following categories: **Bugs & Logic Errors:** - Null/undefined handling - Off-by-one errors - Race conditions - Memory leaks - Incorrect algorithm implementation **Security Issues:** - SQL injection vulnerabilities - XSS vulnerabilities - Authentication/authorization flaws - Sensitive data exposure - Unsafe dependencies **Performance Problems:** - Inefficient algorithms (O(n²) where O(n) possible) - Unnecessary database queries - Memory overuse - Blocking operations - Missing caching **Code Style:** - Naming conventions - Code formatting inconsistencies - Magic numbers/strings - Dead code - Overly complex functions **Best Practices:** - DRY (Don't Repeat Yourself) violations - SOLID principles adherence - Error handling patterns - Testing coverage - Documentation quality ### 3. Categorize Findings Group issues by: - **Critical**: Security vulnerabilities, data loss risks - **High**: Bugs that cause crashes/errors - **Medium**: Performance issues, maintainability problems - **Low**: Style issues, minor improvements ### 4. Provide Solutions For each iss...