refactor

SOLID Refactoring Assistant

Analyze code for SOLID violations and suggest targeted improvements.

Purpose

Identify refactoring opportunities based on:

• SOLID principle violations
• Code smells and anti-patterns
• Complexity metrics
• Duplication detection

Instructions

Step 1: Scope Analysis

Determine the refactoring scope from user input:

• Single file: Deep analysis
• Directory: Pattern detection across files
• Function/class: Focused extraction suggestions

# Get file/directory stats
if [ -f "$TARGET" ]; then
  wc -l "$TARGET"
  echo "Single file analysis"
elif [ -d "$TARGET" ]; then
  find "$TARGET" -type f \( -name "*.ts" -o -name "*.js" -o -name "*.py" \) | wc -l
  echo "Directory analysis"
fi

Step 2: SOLID Violations Detection

S - Single Responsibility

Look for:

• Files > 300 lines
• Functions > 50 lines
• Classes with > 10 methods
• Mixed concerns (data + UI + business logic)

# Find large files
find . -name "*.{ts,js,py}" -exec wc -l {} + 2>/dev/null | sort -rn | head -10

# Functions with high line count (approximate)
grep -rn "function\|def \|fn " --include="*.{ts,js,py,rs}" . | head -20

O - Open/Closed Principle

Look for:

• Switch/case statements on types
• Repeated if/else type checking
• Direct modifications vs extensions

L - Liskov Substitution

Look for:

• Overridden methods that throw "not implemented"
• Type checks before method calls
• Empty method overrides

I - Interface Segregation

Look for:

• Large interfaces (> 10 methods)
• Classes implementing unused interface methods
• Fat service classes

D - Dependency Inversion

Look for:

• Direct instantiation of dependencies (new Service())
• Hardcoded class references
• Missing dependency injection

Step 3: Code Smells

# Duplication patterns
grep -rn --include="*.{ts,js,py}" . 2>/dev/null | \
  awk -F: '{print $3}' | sort | uniq -c | sort -rn | head -10

# Long parameter lists (> 4 params)
grep -rn "function.*,.*,.*,.*," --include="*.{ts,js}" . 2>/dev/null | head -10

# Deep nesting (4+ levels)
grep -rn "^\s\{16,\}" --include="*.{ts,js,py}" . 2>/dev/null | head -10