# Ponytail Review > Revisión de código centrada solo en sobre-ingeniería: qué borrar, stdlib reinventada, dependencias innecesarias, abstracciones especulativas. Una línea por hallazgo. Complementa la revisión de corrección normal. Fuente: https://skillsagentes.com/skills/dietrichgebert/ponytail/ponytail-review Markdown: https://skillsagentes.com/skills/dietrichgebert/ponytail/ponytail-review.md Repositorio: https://github.com/DietrichGebert/ponytail Autor: DietrichGebert Licencia: MIT Actualizado: hace 3 meses Coste de contexto: 114 tok instalada, 596 tok al activarse, 596 tok con todos los archivos del bundle Bundle: 1 archivo, 2 KB Permisos que pide: ninguno declarado ## Instalación Un skill son archivos markdown: los mismos archivos valen para cualquier agente y lo único que cambia es el directorio de destino, es decir la bandera `--agent`. Añade `-g` para instalarlo en todos los proyectos de la máquina. ```bash # Claude Code npx -y skills add DietrichGebert/ponytail --skill ponytail-review --agent claude-code # Cursor npx -y skills add DietrichGebert/ponytail --skill ponytail-review --agent cursor # Codex npx -y skills add DietrichGebert/ponytail --skill ponytail-review --agent codex # Gemini CLI npx -y skills add DietrichGebert/ponytail --skill ponytail-review --agent gemini # Windsurf npx -y skills add DietrichGebert/ponytail --skill ponytail-review --agent windsurf # Cline npx -y skills add DietrichGebert/ponytail --skill ponytail-review --agent cline ``` ## Qué hace - Revisa un diff (no el repo completo) buscando solo sobre-ingeniería: qué borrar y con qué reemplazarlo, una línea por hallazgo. - Usa las mismas etiquetas que ponytail-audit (delete/stdlib/native/yagni/shrink) con formato `L: . .` - Termina con `net: - lines possible` o "Lean already. Ship." si no hay nada que cortar. ## Cuándo usarla - Revisar un diff específico por sobre-ingeniería o complejidad innecesaria. - Preguntar si algo está sobre-diseñado o qué se puede simplificar en un cambio. ## Cuándo no - Bugs de corrección, huecos de seguridad o rendimiento: el archivo los excluye explícitamente y los remite a un review normal. - Un smoke test o self-check con assert no cuenta como bloat y nunca debe marcarse para borrar, según el archivo. ## Qué la activa - "/ponytail-review" - "revisa este diff por sobre-ingeniería" - "qué se puede borrar de este cambio" ## Antes de instalar - Ninguno: solo lista hallazgos, no aplica los fixes ("does not apply the fixes"). ## Archivos - SKILL.md — 2 KB ## SKILL.md Reproducido tal cual desde DietrichGebert/ponytail bajo MIT. Esta sección es el documento original y está en inglés. Review diffs for unnecessary complexity. One line per finding: location, what to cut, what replaces it. The diff's best outcome is getting shorter. ## Format `L: . .`, or `:L: ...` for multi-file diffs. Tags: - `delete:` dead code, unused flexibility, speculative feature. Replacement: nothing. - `stdlib:` hand-rolled thing the standard library ships. Name the function. - `native:` dependency or code doing what the platform already does. Name the feature. - `yagni:` abstraction with one implementation, config nobody sets, layer with one caller. - `shrink:` same logic, fewer lines. Show the shorter form. ## Examples ❌ "This EmailValidator class might be more complex than necessary, have you considered whether all these validation rules are needed at this stage?" ✅ `L12-38: stdlib: 27-line validator class. "@" in email, 1 line, real validation is the confirmation mail.` ✅ `L4: native: moment.js imported for one format call. Intl.DateTimeFormat, 0 deps.` ✅ `repo.py:L88: yagni: AbstractRepository with one implementation. Inline it until a second one exists.` ✅ `L52-71: delete: retry wrapper around an idempotent local call. Nothing replaces it.` ✅ `L30-44: shrink: manual loop builds dict. dict(zip(keys, values)), 1 line.` ## Scoring End with the only metric that matters: `net: - lines possible.` If there is nothing to cut, say `Lean already. Ship.` and stop. ## Boundaries Scope: over-engineering and complexity only. Correctness bugs, security holes, and performance are explicitly out of scope. Route them to a normal review pass, not this one. A single smoke test or `assert`-based self-check is the ponytail minimum, not bloat, never flag it for deletion. Does not apply the fixes, only lists them. "stop ponytail-review" or "normal mode": revert to verbose review style. ## Dónde encaja - Categoría: [Herramientas para desarrolladores](https://skillsagentes.com/categorias/herramientas-desarrollo.md) — Skills que cambian cómo tu agente escribe, revisa y despliega código. - Creador: [DietrichGebert](https://skillsagentes.com/creators/dietrichgebert.md) — 6 skills en el directorio - [Todas las skills](https://skillsagentes.com/skills.md) - [Ranking de instalaciones](https://skillsagentes.com/ranking.md) ## Otras skills del mismo repositorio - [Ponytail](https://skillsagentes.com/skills/dietrichgebert/ponytail/ponytail.md): Impone la solución más perezosa que funcione: la más simple, corta y mínima. YAGNI antes que código nuevo, stdlib y nativo antes que dependencias. Para cualquier tarea de código; no para pedidos que no son de código. - [Ponytail Help](https://skillsagentes.com/skills/dietrichgebert/ponytail/ponytail-help.md): Tarjeta de referencia rápida con todos los modos, skills y comandos de ponytail. Despliegue de una sola vez, no es un modo persistente. - [Ponytail Audit](https://skillsagentes.com/skills/dietrichgebert/ponytail/ponytail-audit.md): Auditoría de sobre-ingeniería en todo el repo, no solo un diff: lista rankeada de qué borrar, simplificar o reemplazar por stdlib/nativo. Reporte de una sola vez, no aplica los cambios. - [Ponytail Debt](https://skillsagentes.com/skills/dietrichgebert/ponytail/ponytail-debt.md): Recolecta los comentarios `ponytail:` del repo en un ledger de deuda técnica, para que los atajos y aplazamientos deliberados no se olviden. Reporte de una sola vez, no cambia nada. - [Ponytail Gain](https://skillsagentes.com/skills/dietrichgebert/ponytail/ponytail-gain.md): Muestra el impacto medido de ponytail como un scoreboard compacto: menos código, menos costo, más velocidad, según medianas de benchmark. Despliegue de una sola vez, no es una cifra por-repo. --- Skills Agentes · [Índice de páginas en markdown](https://skillsagentes.com/sitemap.md) · [Inicio](https://skillsagentes.com/index.md)