Ponytail 代码精简评审

要解决的问题

代码评审中，除了明显的逻辑 Bug 和安全隐患，最消耗注意力的往往是不必要的复杂度：手写的校验逻辑其实标准库已有；为一个格式化调用引入庞大的日期库；为只有一个实现的接口建立抽象层；以及包装幂等调用的重试装饰器。这些代码能跑，但增加了阅读和维护成本。

技能如何工作

该技能专注于找出 Diff 中可以删掉的复杂度，输出极简的替换建议：

• 逐行标记：每个发现只占一行，指明位置、删什么、用什么替换。格式为 L12-38，多文件 Diff 使用 repo.py:L88。
• 核心标签：

- delete：彻底删除死代码或投机性功能，无需替换。
- stdlib：标准库已提供的功能，如用 re 替代手写的 27 行邮箱校验。
- native：平台原生支持，如用 Intl.DateTimeFormat 替代 moment.js。
- yagni：只有一个调用或实现的抽象层，建议内联。
- shrink：逻辑不变但可以更短，如用 dict(zip(keys, values)) 替代手动循环。

• 量化收益：最后给出唯一指标 net: - lines possible，直观展示可精简的行数。

适用边界

• 只谈复杂度：明确排除正确性 Bug、安全漏洞和性能优化，这些应走常规评审流程。
• 不删测试：一个冒烟测试或基础断言是最低保障，绝不能因为“冗余”而建议删除。
• 只建议不动手：仅列出精简清单，不会直接应用修改。
• 随时退出：输入 stop ponytail-review 或 normal mode 即可切换回详尽的评审模式。

如果 Diff 中没有可删减的内容，技能会直接回复 Lean already. Ship. 并停止。