（3934期）Ponytail Review – 极简代码审查专家

极简代码审查专家 来自DietrichGebert的开源代码审查工具，专精识别过度工程化代码，通过精准删减实现代码瘦身，让diff越改越短。 使用说明 核心用法 Ponytail-review是一款专注于代码过度工程化审查的轻量级技能。其核心使命非常明确：让代码diff变得更短，而非更长。使用时只需传入代码diff，技能会以极精简的格式输出审查结果——每行一个发现，包含位置、待删除内容和替代方案。 输出格式遵循严格规范：L<line>: <tag> <what>. <replacement>. 或使用 <file>:L<line>: ... 处理多文件diff。五个标签各司其职： delete: 识别死代码、未使用的灵活性、推测性功能，直接删除 stdlib: 发现重复造轮子的自定义实现，替换为标准库函数 native: 标记可用原生平台特性替代的外部依赖 yagni: 揪出单一实现的抽象层、无人使用的配置、仅一个调用者的分层 shrink: 相同逻辑但更简洁的写法 审查以唯一关键指标收尾：net: -<N> lines possible. 若无删减空间，则直接判定 Lean already. Ship. 显著优点 极度聚焦：明确排除正确性bug、安全漏洞和性能问题，专注于"过度工程化"这一单一维度，避免审查范围无限膨胀 可行动输出：每个发现都附带具体替换方案，而非模糊的"考虑简化"式建议 量化成果：用行数净减少量作为唯一度量，让代码瘦身效果一目了然 语言无关：基于diff文本分析，理论上适用于任何编程语言 开源可信：MIT许可证，GitHub公开仓库，来源透明可审计 交互灵活：支持stop ponytail-review指令随时切换回传统详细审查模式 潜在缺点与局限性 功能单一：仅能识别过度工程化，无法替代完整代码审查流程，必须与其他审查工具配合使用 上下文缺失：单行输出格式虽简洁，但可能丢失复杂重构所需的背景信息 机械判断风险："yagni"标签可能误伤合理的提前抽象，尤其是在快速迭代的初创场景中 无自动修复：仅列出待删内容，不实际应用修改，仍需人工执行 测试要求模糊：提及"smoke test或assert自检"为最低标准，但未明确界定何为"过度测试" 生态依赖：作为MCP/Agent技能，实际效果取决于宿主平台的集成质量 适合的目标群体 技术负责人/Code Reviewer：需要快速批量识别团队代码中的冗余设计 开源维护者：审查外部贡献时快速剔除过度复杂的实现 重构专项团队：聚焦代码瘦身项目，建立"以行数减少为荣"的团队文化 初级开发者导师：通过具体案例学习"简单优于复杂"的工程哲学 精益创业团队：资源受限环境下优先选择最小可行实现 使用风险 误删风险：过度激进的"delete"建议可能移除实际需要的扩展点，建议关键删除操作需二次确认 团队摩擦："yagni"判断高度依赖上下文共识，在缺乏设计一致性的团队中可能引发争议 依赖更新成本："native"和"stdlib"替换建议可能引入行为差异，需配合回归测试 认知负荷：极简输出格式对新手不够友好，需配合示例文档理解标签体系 性能无关：明确声明不处理性能问题，切勿用于性能关键路径的优化决策 原始说明 Review a diff for over-engineering. Finds what to delete: reinvented stdlib, needless deps, speculative abstractions. One line per finding.