code-review

# Code Review

Systematic 9-phase code review methodology using Language Server Protocol tools. Adapts to available LSP capabilities across programming languages.

## Skill Methodology

Systematic 9-phase code review methodology using Language Server Protocol tools. Extends DEVELOPER profile with sequential analysis phases assessing code quality, architecture, and maintainability.

> [!IMPORTANT]
> The skill embodies Analyze → Verify → Document → Deliver
>
> - Process skill instructions systematically
> - Take time to read, understand, and apply each section's logic carefully
> - Rushing past documented procedures causes **fatal** execution errors

### Phase Overview

1. **Phase 1 (Project Discovery)** - Establish tool inventory, understand project structure, identify technology stack
2. **Phase 2 (Structural Analysis)** - Analyze code organization, module structure, architectural patterns
3. **Phase 3 (Dependency Mapping)** - Map import relationships, call hierarchies, dependency flow
4. **Phase 4 (Type Safety)** - Assess type coverage, identify type safety issues, verify type inference
5. **Phase 5 (Usage Analysis)** - Analyze how symbols are used throughout the codebase
6. **Phase 6 (Code Quality)** - Evaluate error handling, resource management, code maintainability
7. **Phase 7 (Refactoring Safety)** - Test rename operations, assess refactoring risk before changes
8. **Phase 8 (Consistency)** - Verify naming conventions, style consistency, architectural patterns
9. **Phase 9 (Report)** - Synthesize findings into actionable report with prioritized recommendations

> [!IMPORTANT]
> Review phases provide thorough investigation, tool-verified findings, and incremental validation for code quality assessment.

### Systematic Approach

The phases must be completed **sequentially**:

- **DO complete each phase fully** - Each phase delivers specific analysis findings
- **DO wait for approval** - Confirm deliverables before proceeding to next phase
- **DO adapt approach*