Architecture simplification

Skill yeaight7/agent-powerups/skills/architecture-simplification

Curated power-ups for coding agents: skills, slash commands, MCP configs, hooks, AGENTS.md templates, and workflows for serious software engineering. Claude Code, Codex, Antigravity CLI, Cursor and more

Install

npx -y skills add yeaight7/agent-powerups --skill architecture-simplification

Assembled from the repository path, not quoted from the project. Check it against their README if it does not work.

One thing to look at

6 stars6 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.

What its author says it does

Copied from the file, not written here

Use when a codebase carries over-engineered abstractions, unnecessary layers, or redundant logic that should be collapsed without changing behavior.

SKILL.md

2.5 KB, as published. Nobody here has run it

Purpose

Over time, codebases accumulate "just in case" abstractions. This skill guides the safe removal of unnecessary complexity: remove the noise around the core logic without changing the core logic itself.

When to Use

An interface has only one implementation and no second one is planned
A wrapper, factory, or layer only passes arguments straight through
Two code paths do the same thing and should be consolidated

Inputs

The suspect abstraction(s) and their call sites
A green test suite covering the affected area

Workflow

Identify the abstraction cost. Does this interface have only one implementation? Does this wrapper class just pass arguments straight through? Measure before cutting:
```
grep -rn "implements IUserRepository" src/    # count implementations
grep -rn "IUserRepository" src/ | wc -l       # count references
```
Run the tests first. The affected area must be green before any removal — this is the behavioral baseline.
Inline the logic. Move the logic from the unnecessary abstraction directly into the caller.
Delete the dead code. Remove the interface, wrapper, or factory that is no longer needed.
Test verification. Re-run the same tests; the observable behavior of the system must not have changed.

Example: if a UserRepository implements IUserRepository but there is only ever one database, inline UserRepository and delete IUserRepository.

Output

The simplified code with the abstraction removed
Before/after test evidence showing unchanged behavior

Verification

Tests covering the area were green before the change (baseline)
Same tests green after the change — observable behavior preserved
No references to the removed abstraction remain (searched, not assumed)
Diff contains only removal/inlining — no core-logic rewrites

Failure Modes

Rewrite disguised as simplification — do not rewrite the entire subsystem; simplification removes the noise around the core logic, not the logic itself.
Cutting without a baseline — without a green pre-change test run, "tests pass after" proves nothing.
Speculative retention — keeping the interface "in case we need it later" recreates the original problem.