Ponytail audit skill

Audit the whole repo for over-engineering.

by DietrichGebert·MIT license·★ 153,538 Stars on the repo·GitHub ↗

Use now

Files of Ponytail audit

DietrichGebert/main1 file shown
SKILL.md
Show the full text42 lines
ponytail-audit/SKILL.md42 lines · 1.7 KB

ponytail-review, repo-wide. Scan the whole tree instead of a diff. Rank findings biggest cut first.

Tags

Same as ponytail-review:

  • 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.
  • reuse: equivalent helper, util, or pattern already in this repo. Name the path.
  • yagni: abstraction with one implementation, config nobody sets, layer with one caller.
  • shrink: same logic, fewer lines. Show the shorter form.

Hunt

Deps the stdlib or platform already ships, single-implementation interfaces, factories with one product, wrappers that only delegate, files exporting one thing, dead flags and config, hand-rolled stdlib, helpers duplicating an equivalent that already lives in this repo. Before emitting delete:, grep the whole tree for the symbol, including tests, fixtures and string or dynamic references.

Output

One line per finding, ranked: <tag> <what to cut>. <replacement>. [path]. End with net: -<N> lines, -<M> deps possible. Nothing to cut: Lean already. Ship.

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. Lists findings, applies nothing. One-shot. "stop ponytail-audit" or "normal mode" to revert.

1---
2name: ponytail-audit
3description: "Audit the whole repo for over-engineering. A ranked list of what to delete, simplify, or replace with stdlib or native features."
4homepage: https://github.com/DietrichGebert/ponytail
5license: MIT
6---
7 
8ponytail-review, repo-wide. Scan the whole tree instead of a diff. Rank
9findings biggest cut first.
10 
11## Tags
12 
13Same as ponytail-review:
14 
15- `delete:` dead code, unused flexibility, speculative feature. Replacement: nothing.
16- `stdlib:` hand-rolled thing the standard library ships. Name the function.
17- `native:` dependency or code doing what the platform already does. Name the feature.
18- `reuse:` equivalent helper, util, or pattern already in this repo. Name the path.
19- `yagni:` abstraction with one implementation, config nobody sets, layer with one caller.
20- `shrink:` same logic, fewer lines. Show the shorter form.
21 
22## Hunt
23 
24Deps the stdlib or platform already ships, single-implementation interfaces,
25factories with one product, wrappers that only delegate, files exporting one
26thing, dead flags and config, hand-rolled stdlib, helpers duplicating an
27equivalent that already lives in this repo.
28Before emitting `delete:`, grep the whole tree for the symbol, including tests,
29fixtures and string or dynamic references.
30 
31## Output
32 
33One line per finding, ranked: `<tag> <what to cut>. <replacement>. [path]`.
34End with `net: -<N> lines, -<M> deps possible.` Nothing to cut: `Lean already. Ship.`
35 
36## Boundaries
37 
38Scope: over-engineering and complexity only. Correctness bugs, security holes,
39and performance are explicitly out of scope. Route them to a normal review
40pass. Lists findings, applies nothing. One-shot.
41"stop ponytail-audit" or "normal mode" to revert.
42 

Discussion