ponytail-review

Reviews code diffs for over-engineering and lists what to delete.

Updated Aug 28, 2026
One-click install
npx skills add https://github.com/epicodic/drift --skill ponytail-review-epicodic
Or copy as Structured Prompt for Agent▼
Please help me install this Agent Skill.
Skill: ponytail-review
Source: https://github.com/epicodic/drift/tree/main/.agents/skills/ponytail-review
Command: npx skills add https://github.com/epicodic/drift --skill ponytail-review-epicodic

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve? Codebases accumulate unnecessary complexity: reinvented standard library functions, speculative abstractions, unused dependencies, and dead flexibility. This Skill performs a focused review pass that hunts only over-engineering, telling you exactly what to cut and what replaces it. ## Core Features & Use Cases - Complexity-Only Review: Flags dead code, hand-rolled stdlib equivalents, unneeded dependencies, and speculative abstractions using five tags: delete, stdlib, native, yagni, shrink. - One-Line Findings: Each finding is a single line with location, what to cut, and the replacement, ending with a net line-reduction score. - Use Case: After a large feature branch, run this review on the diff to find that a 27-line validator class can be replaced by a one-line check, or that moment.js was imported for a single format call that Intl.DateTimeFormat handles natively. ## Quick Start Ask the assistant to review this diff for over-engineering and tell me what can be deleted.

Frequently Asked Questions about ponytail-review

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I review code for over-engineering?▼

Run this review on a diff and it flags unnecessary complexity with five tags: delete, stdlib, native, yagni, and shrink. Each finding is one line showing the location, what to cut, and what replaces it, ending with a net line-reduction score.

What kinds of complexity does this code review detect?▼

It detects dead code and unused flexibility, hand-rolled logic the standard library already ships, dependencies duplicating native platform features, abstractions with a single implementation, and verbose logic that has a shorter equivalent form.

Does this review catch bugs or security issues?▼

No. Correctness bugs, security holes, and performance problems are explicitly out of scope and should go through a normal review pass. This review only hunts complexity and over-engineering.

Will this review apply the suggested code deletions automatically?▼

No. It only lists findings with locations and replacements; it never modifies the code. You decide which suggestions to apply yourself.

When should I not use an over-engineering review?▼

Skip it when you need correctness, security, or performance feedback, since those are out of scope. It also never flags a single smoke test or assert-based self-check, which it treats as the minimum, not bloat.