code-reviewlisted

# Code Review Skill Perform a thorough code review with quality metrics and actionable feedback. ## When to Use This skill auto-activates when: - User asks to review a PR or pull request - User asks to review code changes or a branch - User wants code quality analysis on changes - User mentions "review", "code review", or "check my changes" ## Workflow 1. **Get changes to review** - If PR number: fetch PR diff via GitHub MCP - If branch: `git_diff` against main/master - If file path: read the specific file - If "staged": `git_diff(staged=true)` - If empty: `git_diff` for unstaged changes 2. **Analyze code quality** (in parallel) - Call `analyze_method_complexity(root, threshold=10)` on changed files - Call `find_god_classes(root)` on changed files - Call `build_and_extract_errors` to check compilation 3. **Review checklist** ### Correctness - [ ] Logic is correct and handles edge cases - [ ] Error handling is appropriate - [ ] Null checks where needed - [ ] Async/await used correctly - [ ] Disposal of resources (IDisposable, using) ### Design - [ ] Single Responsibility Principle - [ ] Dependencies are injected, not created - [ ] Interfaces used for abstraction - [ ] No circular dependencies introduced - [ ] Follows existing patterns in codebase ### Security - [ ] No SQL injection vulnerabilities - [ ] No XSS vulnerabilities - [ ] Input validation present - [ ] Sensitive data not logged -