ponytail-review
Reviews diffs for over-engineering only. Use when reviewing, desloping, or asking what to delete, inline, shrink, or replace with stdlib/native code.
ソース情報
- リポジトリ
- redpanda-data/ui-harness
- ソースの最終更新活動
- 2026年6月26日 12:54
- 検出された SKILL.md の言語
- 英語
- スター
- 34
- フォーク
- 4
インストール方法
デフォルトでは、最初にソースを確認する Prompt が選択されています。直接コマンドに切り替えるか、ローカルコピーをダウンロードすることもできます。
ソースファイルを確認
インストールを決める前に、SKILL.md と SkillsMP に表示されている付属ファイルをお読みください。
SKILL.md を表示中
SKILL.md
ソースの指示 · 読み取り専用プレビュー- name
- ponytail-review
- description
- Reviews diffs for over-engineering only. Use when reviewing, desloping, or asking what to delete, inline, shrink, or replace with stdlib/native code.
- license
- MIT
- vendored_from
- https://github.com/DietrichGebert/ponytail
- upstream_commit
- 687c1b339872289d70f65c5eaabce850b1663867
# Ponytail Review
Repo/code changes: run `/deslop` before commit, push, PR, or merge.
Vendored from DietrichGebert/ponytail. Complexity only; `/deslop` owns value/defense/test gate.
Caveman terse: one finding per line, no coaching prose.
Review diffs for needless complexity. Best result: diff gets shorter.
## Format
`<file>:L<line>: <tag> <what>. <replacement>.`
Tags:
- `delete:` dead code, unused flexibility, speculative feature. Replace with nothing.
- `stdlib:` hand-rolled stdlib. Name function.
- `native:` dep/code doing platform job. Name feature.
- `yagni:` one impl, one caller, config nobody sets.
- `shrink:` same behavior, fewer lines. Show shorter form.
## Examples
- `L12-38: stdlib: 27-line validator class. "@" check; real validation is confirmation email.`
- `L4: native: moment.js for one format call. Intl.DateTimeFormat, 0 deps.`
- `repo.py:L88: yagni: AbstractRepository with one impl. Inline until second exists.`
- `L52-71: delete: retry wrapper around idempotent local call. Nothing replaces it.`
- `L30-44: shrink: manual dict loop. dict(zip(keys, values)), 1 line.`
## Scoring
End with `net: -<N> lines possible.`
Nothing to cut: `Lean already. Ship.`
## Boundary
Complexity only. Correctness, security, perf, resilience, product value -> normal review and `/deslop`. Behavior test is not bloat. List cuts; do not apply.
GitHubで見る