Over-Engineering Detector

VerifiedSafe

Scans codebases for unnecessary complexity, speculative generality, and over-engineered patterns. Useful for spotting premature abstractions, excessive indirection, and unused flexibility, then offers actionable simplifications to reduce technical debt.

Sby Skills Guide Bot
DevelopmentIntermediate
1406/2/2026
Claude Code
#over-engineering#code-quality#refactoring#complexity#simplification

Recommended for

Our review

Analyzes code to detect unnecessary complexity, speculative generality, and over-engineered solutions.

Strengths

  • Identifies premature abstractions and excessive layers
  • Categorizes issues by severity (critical, warning, info)
  • Provides concrete simplification suggestions with effort estimates

Limitations

  • May produce false positives if business context is not understood
  • Does not replace human review of architectural trade-offs
  • Depends on pattern detection quality (may miss subtle cases)
When to use it

During code review or quality audits to simplify a complex codebase.

When not to use it

When complexity is justified by known future requirements or during rapid prototyping.

Security analysis

Safe
Quality score88/100

The skill only uses read-only tools (Read, Glob, Grep, Task) to analyze code for complexity patterns. It does not execute any code, modify files, or exfiltrate data.

No concerns found

Examples

Analyze entire codebase
Analyze the entire codebase for over-engineering, speculative generality, and unnecessary complexity.
Analyze specific directory
detect over-engineering in src/services/
Analyze a file pattern
detect over-engineering in **/*.cs

name: detect-over-engineering description: Detect unnecessary complexity, speculative generality, and over-engineered solutions in code allowed-tools: Read, Glob, Grep, Task argument-hint: "[path-or-pattern]"

Detect Over-Engineering Command

Analyze code to identify unnecessary complexity, speculative generality, and over-engineered solutions.

Usage

/enterprise-architecture:detect-over-engineering [path-or-pattern]

Arguments

  • path-or-pattern (optional): Path to analyze
    • If provided: Analyze the specified path or pattern
    • If omitted: Analyze the entire codebase

Examples

/enterprise-architecture:detect-over-engineering
/enterprise-architecture:detect-over-engineering src/services/
/enterprise-architecture:detect-over-engineering **/*.cs

Workflow

  1. Scan for Complexity Indicators

    • Search for patterns indicating over-engineering
    • Identify abstraction layers
    • Find unused flexibility points
  2. Spawn Over-Engineering Detector Agent Use the over-engineering-detector agent to analyze. The agent detects:

    • Speculative Generality - Abstractions without multiple implementations
    • Premature Abstraction - Complexity before demonstrated need
    • Gold Plating - Features beyond requirements
    • Astronaut Architecture - Excessive layers and indirection
  3. Present Findings Display findings organized by:

    • Critical - Significant complexity with no benefit
    • Warning - Potential over-engineering
    • Info - Minor simplification opportunities

Detection Categories

Speculative Generality

  • Abstract classes with single implementation
  • Interfaces with only one implementer
  • Generic type parameters never varied
  • Configuration options never used
  • Plugin architectures with no plugins

Premature Abstraction

  • Factory patterns for single object creation
  • Strategy patterns with one strategy
  • Excessive dependency injection
  • Over-normalized data structures

Gold Plating

  • Features beyond documented requirements
  • Configurability that's never configured
  • Extensibility points never extended

Output Format

## Over-Engineering Detection Report

### Summary
- Files analyzed: [N]
- Issues found: [N] (Critical: X, Warning: Y, Info: Z)

### Critical Issues

#### [Issue Type]: [Location]
**Pattern:** [What was found]
**Problem:** [Why it's over-engineered]
**Simplification:** [How to simplify]
**Effort:** [Low/Medium/High]

### Warnings

[Same structure]

### Info

[Same structure]

### Simplification Roadmap
1. [Quick win] - [Impact]
2. [Medium effort] - [Impact]
3. [Larger refactor] - [Impact]

### Metrics
- Estimated lines removable: [N]
- Abstraction layers reducible: [N]
- Complexity score improvement: [X]%
Related skills