feat: Add comprehensive review command with quality analysis (#24)

* feat: Add comprehensive review command with quality analysis

- Add new /review command with --files, --commit, --pr flags
- Implement evidence-based quality assessment patterns
- Create comprehensive review patterns YAML resource
- Update command index to reflect 19 total commands
- Integrate with existing analysis workflow

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Kevin Pauer <kevinpauer@users.noreply.github.com>

* fix: remove new custom .yml and reuse existing ones; adjust compression-performance-patterns

* feat: add command documentation

---------

Signed-off-by: Kevin Pauer <kevinpauer@users.noreply.github.com>
Co-authored-by: Kevin Pauer <kevinpauer@users.noreply.github.com>
Co-authored-by: Claude <noreply@anthropic.com>

.claude/commands/index.md

@@ -3,12 +3,12 @@
 @include shared/universal-constants.yml#Universal_Legend
 
 ## Ultra-Compressed Reference
-Commands: `/cmd --flags` | 18 total | Universal flags available
+Commands: `/cmd --flags` | 19 total | Universal flags available
 
 @include shared/flag-inheritance.yml#Universal_Always
 
 ## Command Categories
-**Analysis**: `/analyze` `/scan` `/explain`
+**Analysis**: `/analyze` `/scan` `/explain` `/review`
 **Build**: `/build` `/deploy` `/migrate` 
 **Manage**: `/task` `/load` `/cleanup`
 **Dev**: `/test` `/troubleshoot` `/improve`
@@ -16,7 +16,7 @@ Commands: `/cmd --flags` | 18 total | Universal flags available
 
 ## Commands w/ Primary Flags
 **Dev**: `/build` --init|feature|react | `/dev-setup` --install|ci | `/test` --coverage|e2e
-**Analysis**: `/analyze` --code|arch | `/troubleshoot` --fix|prod | `/improve` --perf|quality | `/explain` --depth
+**Analysis**: `/analyze` --code|arch | `/review` --files|commit|pr | `/troubleshoot` --fix|prod | `/improve` --perf|quality | `/explain` --depth
 **Ops**: `/deploy` --env|rollback | `/migrate` --dry-run | `/scan` --security | `/estimate` --detailed | `/cleanup` --all | `/git` --commit|sync
 **Design**: `/design` --api|ddd | `/document` --api|user | `/spawn` --task
 **Manage**: `/task` --create|status | `/load` --context
@@ -24,7 +24,7 @@ Commands: `/cmd --flags` | 18 total | Universal flags available
 **Setup**: `/load` → `/dev-setup --install` → `/build --init` → `/test`
 **Feature**: `/analyze` → `/design --api` → `/build --tdd` → `/test --e2e` → `/deploy`
 **Debug**: `/troubleshoot --fix` → `/test` → `/git --commit`
-**Quality**: `/analyze --code --think` → `/improve --quality` → `/scan --validate`
+**Quality**: `/review --quality --evidence` → `/improve --quality` → `/scan --validate`
 **Security**: `/scan --security --owasp` → `/improve` → `/scan --validate`
 
 ### Advanced Flag Combinations

.claude/commands/review.md

@@ -0,0 +1,75 @@
+**Purpose**: AI-powered code review and quality analysis
+
+---
+
+@include shared/universal-constants.yml#Universal_Legend
+
+## Command Execution
+Execute: immediate. --plan→show plan first
+Legend: Generated based on symbols used in command
+Purpose: "[Action][Subject] in $ARGUMENTS"
+
+Perform comprehensive code review and quality analysis on files, commits, or pull requests specified in $ARGUMENTS.
+
+@include shared/flag-inheritance.yml#Universal_Always
+
+Examples:
+- `/review --files src/auth.ts --persona-security` - Security-focused file review
+- `/review --commit HEAD --quality --evidence` - Quality review with sources
+- `/review --pr 123 --all --interactive` - Comprehensive PR review
+- `/review --files src/ --persona-performance --think` - Performance analysis
+
+## Command-Specific Flags
+--files: "Review specific files or directories"
+--commit: "Review changes in specified commit (HEAD, hash, range)"
+--pr: "Review pull request changes (git diff main..branch)"
+--quality: "Focus on code quality issues (DRY, SOLID, complexity)"
+--evidence: "Include sources and documentation for all suggestions"
+--fix: "Suggest specific fixes for identified issues"
+--summary: "Generate executive summary of review findings"
+
+@include shared/quality-patterns.yml#Code_Quality_Metrics
+
+@include shared/security-patterns.yml#OWASP_Top_10
+
+@include shared/compression-performance-patterns.yml#Performance_Baselines
+
+@include shared/architecture-patterns.yml#DDD_Building_Blocks
+
+## Review Process & Methodology
+
+**1. Context Analysis:** Understanding codebase patterns | Identifying architectural style | Recognizing team conventions | Establishing review scope
+
+**2. Multi-Dimensional Scan:** Quality assessment across all dimensions | Persona-specific deep dives | Cross-reference analysis | Dependency impact review
+
+**3. Evidence Collection:** Research best practices via Context7 | Cite authoritative sources | Reference documentation | Provide measurable metrics
+
+**4. Prioritized Findings:** Critical issues first | Security vulnerabilities highlighted | Performance bottlenecks identified | Quality improvements suggested
+
+**5. Actionable Recommendations:** Specific fix suggestions | Alternative approaches | Refactoring opportunities | Prevention strategies
+
+**Evidence-Based Analysis:** All suggestions must cite authoritative sources | Reference official docs via Context7 | Cross-reference industry standards | Performance claims require measurable evidence
+
+**Persona Specialization:** Security→Vulnerabilities+compliance | Performance→Bottlenecks+optimization | Architecture→Patterns+maintainability | QA→Coverage+validation
+
+@include shared/research-patterns.yml#Mandatory_Research_Flows
+
+@include shared/quality-patterns.yml#Validation_Sequence
+
+## Persona Integration
+
+**--persona-security:** Security-first analysis | Threat modeling | Vulnerability scanning | Compliance checking | Risk assessment
+
+**--persona-performance:** Performance optimization focus | Bottleneck identification | Resource analysis | Scalability review
+
+**--persona-architect:** System design evaluation | Pattern assessment | Maintainability review | Technical debt analysis
+
+**--persona-qa:** Testing coverage analysis | Edge case identification | Quality metrics | Validation strategies
+
+**--persona-refactorer:** Code improvement opportunities | Refactoring suggestions | Cleanup recommendations | Pattern application
+
+@include shared/execution-patterns.yml#Servers
+
+@include shared/docs-patterns.yml#Standard_Notifications
+
+@include shared/universal-constants.yml#Standard_Messages_Templates

.claude/commands/shared/compression-performance-patterns.yml

@@ -121,6 +121,12 @@ Performance_Baselines:
     CPU_Utilization: "Track intensive operations"
     Network_Usage: "MCP server communication overhead"
     Disk_I_O: "File operations efficiency"
+    
+  Review_Scope_Limits:
+    Single_File: "Files >1000 lines flagged for review scope"
+    Directory: "Directories >50 files require batch processing"  
+    Repository: "Repos >10K files get incremental analysis"
+    Large_File_Warning: "Files >5MB may require streaming analysis"
 ```
 
 ## MCP Performance Integration