direct-review

name	direct-review
description	CODE REVIEW for over-engineering ONLY. Finds what to delete: reinvented stdlib, unneeded deps, speculative abstractions, dead flexibility. One line per finding: location, what to cut, replacement. Trigger: "review for over-engineering", "what can we delete", "is this over-engineered", "simplify review", "/ponytail-review". Complements correctness review. Does NOT apply fixes.
version	2.0.0
last_updated	2026-06-28T00:00:00.000Z
category	direct

Ponytail Review: Over-Engineering Scanner

TAGS

Tag	Meaning	Replacement
delete:	Dead code, unused flexibility, speculative feature	Nothing
stdlib:	Hand-rolled stdlib function	Name the function
native:	Dep/code doing what platform already does	Name the feature
yagni:	Abstraction with one impl, unused config, layer with one caller	Inline
shrink:	Same logic, fewer lines	Show shorter form

FORMAT

L<line>: <tag> <what>. <replacement>.
<file>:L<line>: ...   (for multi-file diffs)

EXAMPLES

L12-38: stdlib: 27-line EmailValidator. "@" in email, 1 line, real validation is confirmation mail.
L4: native: moment.js for one format call. Intl.DateTimeFormat, 0 deps.
repo.py:L88: yagni: AbstractRepository with one implementation. Inline it.
L52-71: delete: retry wrapper around idempotent local call. Nothing replaces it.
L30-44: shrink: manual loop builds dict. dict(zip(keys, values)), 1 line.

SCORING

net: -<N> lines possible.

Nothing to cut: Lean already. Ship.

BOUNDARIES

Scope: over-engineering and complexity ONLY. Correctness bugs, security holes, performance out of scope. A single smoke test is minimum, not bloat, never flag. Does NOT apply fixes. One-shot. "stop ponytail-review" or "normal mode" to revert.