code-review

Files (1)

• SKILL.md

SKILL.md (first lines)

# Code Review Skill

## Overview
Automated code review skill that analyzes code quality, detects anti-patterns, provides improvement suggestions, and performs basic security checks.

## Features
- Code quality analysis
- Anti-pattern detection
- Improvement suggestions
- Basic security checks
- Code metrics calculation
- Best practices validation

## Usage Examples

### Basic Code Review
```bash
/code-review path/to/file.py
```

### Review with Focus Areas
```bash
/code-review --focus security path/to/code/
```

### Generate Report
```bash
/code-review --format markdown --output report.md path/to/project/
```

## Implementation

### Core Components

#### CodeQualityAnalyzer
```python
class CodeQualityAnalyzer:
    """Analyzes code quality metrics and patterns"""

    def __init__(self):
        self.metrics = {
            'complexity': {},
            'maintainability': {},
            'coverage': {}
        }

    def analyze_file(self, filepath):
        """Analyze a single file for code quality"""
        # Implementation
        pass

    def generate_report(self, results):
        """Generate comprehensive quality report"""
        # Implementation
        pass
```

#### AntiPatternDetector
```python
class AntiPatternDetector:
    """Detects common code anti-patterns"""

    PATTERNS = {
        'god_class': r'class\s+\w+\s*{[^}]*\n[^}]*\n[^}]*}',
        'long_method': r'def\s+\w+\([^)]*\):\s*[^:]*\n(?:[^:\n]*\n){10,}',
        'deep_nesting': r'if.*:\s*if.*:\s*if.*:',
    }

    def detect_patterns(self, code):
        """Scan code for anti-patterns"""
        # Implementation
        pass
```

### Security Checker
```python
class SecurityChecker:
    """Performs basic security analysis"""

    def check_injection_vulnerabilities(self, code):
        """Detect potential injection vulnerabilities"""
        # SQL injection, XSS, command injection checks
        pass

    def validate_input_validation(self, code):
        """Check input validation patterns"""
        # Implementation
        pass
```

## Best Practices

### Code Metrics to Track
- **Cyclomatic Complexity**: Keep below 10
- **Lines per Method**: Max 20-30 lines
- **File Size**: Max 500 lines
- **Coupling Score**: Aim for low coupling
- **Cohesion Score**: Aim for high cohesion

### Security Checklist
- [ ] Input validation present
- [ ] SQL parameterized queries
- [ ] Output encoding for HTML
- [ ] Authentication checks
- [ ] Authorization enforcement
- [ ] Error messages sanitized

### Performance Patterns
- Use generators for large datasets
- Implement proper caching strategies
- Avoid premature optimization
- Profile before optimizing

## Integration

### with CI/CD Pipeline
```yaml
- name: Code Quality Check
  run: |

From eroslifestyle/Claude-Orchestrator-Plugin, MIT.

More from eroslifestyle/Claude-Orchestrator-Plugin