---
id: gh-ponytail-review
name: "ponytail-review"
url: https://skills.yangsir.net/skill/gh-ponytail-review
author: dietrichgebert
domain: ai-code-generation-quality
tags: ["code-review", "refactoring", "code-quality", "over-engineering", "simplification"]
install_count: 54400
rating: 4.70 (120 reviews)
github: https://github.com/dietrichgebert/ponytail/tree/main/.openclaw/skills/ponytail-review
---

# ponytail-review

> 此技能专门审查代码差异，以识别过度工程化、不必要的复杂性和冗余代码。它专注于发现可删除的内容，例如重复的标准库功能、不必要的依赖项和推测性抽象，从而帮助开发者编写更精简、更易维护的代码，提升代码质量并减少技术债务。

**Stats**: 54,400 installs · 4.7/5 (120 reviews)

## Before / After 对比

### 代码审查效率与质量提升

**Before**:

开发者在手动审查代码时，常因疏忽或经验不足而遗漏过度工程化和冗余代码，导致代码库膨胀、维护成本增加，且耗费大量时间。

**After**:

该 Skill 能够快速、准确地识别出代码中过度工程化的部分，并提供具体的删除或替换建议，显著提升了代码审查的效率和质量，有效减少了技术债务。

| Metric | Before | After | Change |
|---|---|---|---|
| 发现过度工程化代码的平均时间 | 90分钟 | 10分钟 | -89% |

## Readme

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<line>: <tag> <what>. <replacement>.`, or `<file>:L<line>: ...` 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: -<N> 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.


---
*Source: https://skills.yangsir.net/skill/gh-ponytail-review*
*Markdown mirror: https://skills.yangsir.net/api/skill/gh-ponytail-review/markdown*