ponytail-review

Identify unnecessary complexity and over-engineering in code diffs.

Updated Jun 26, 2026
One-click install
npx skills add https://github.com/marmelab/atomic-crm-demo-art-teaching --skill ponytail-review-marmelab
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: ponytail-review
Source: https://github.com/marmelab/atomic-crm-demo-art-teaching/tree/main/.claude/skills/ponytail-review
Command: npx skills add https://github.com/marmelab/atomic-crm-demo-art-teaching --skill ponytail-review-marmelab

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

Code reviews often get bogged down by unnecessary complexity, redundant implementations, unneeded dependencies, and speculative abstractions that add maintenance burden without delivering immediate value, making it hard to focus on meaningful improvements.

Core Features & Use Cases

  • Over-Engineering Detection: Scans code diffs for common bloat patterns including hand-rolled standard library replacements, unused dependencies, single-implementation abstractions, and dead flexible code.
  • Actionable Findings: Outputs one-line recommendations per issue with exact line locations, what to cut, and what to replace it with, no vague feedback.
  • Use Case: A development team reviewing a new feature PR can use this skill to quickly identify and remove unnecessary complexity before merging, keeping the codebase lean without duplicating effort from correctness or security review passes.

Quick Start

Use the ponytail-review skill to scan the latest feature branch pull request diff for over-engineering issues and get a prioritized list of simplifications to implement.

Frequently Asked Questions about ponytail-review

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

FAQPage Schema
How do I detect over-engineering and speculative abstractions in a pull request code review?

What is code bloat and how does unnecessary complexity impact a codebase during review?

What is code bloat and how does unnecessary complexity impact a codebase during review?

Code bloat is unnecessary complexity like redundant implementations and speculative abstractions that add maintenance burden without delivering immediate value. Identifying and removing bloat during code review keeps the codebase lean without duplicating effort from correctness or security passes.

How do I identify single-implementation abstractions and unused dependencies in a code diff?

Identifying single-implementation abstractions and unused dependencies in a code diff involves scanning for common over-engineering patterns like hand-rolled standard library replacements and dead flexible code. The process outputs one-line recommendations per issue with exact line locations and possible line reduction metrics.

Can I use code simplification techniques to reduce technical debt on existing feature branches?

Yes, code simplification techniques can be applied to existing feature branches to reduce technical debt. Scanning the pull request diff identifies unnecessary complexity and outputs a prioritized list of simplifications to implement before merging, keeping the codebase lean.

What is the best way to remove redundant implementations and hand-rolled standard library replacements?

The best way to remove redundant implementations and hand-rolled standard library replacements is to scan code diffs for bloat patterns and apply concise, actionable findings. This approach provides exact line locations, what to cut, replacement suggestions, and a total possible line reduction metric.