| name | baton-lean-review |
| description | 审查当前改动/全仓的过度工程与可精简点,输出报告。只读,不修改代码。触发:用户要求精简审查、代码评审、减少复杂度、过度工程检查时使用。 |
baton-lean-review(只读精简审查)
审查当前 diff 或指定范围,按净收益排序输出报告。本 skill 绝不修改代码;不替代正确性/安全/Fidelity Reviewer。
优先级(不可违反)
用户明确要求/FROZEN > 安全与数据完整性 > Contract/DoD/Fidelity > 正确性 > 记忆闭环 > 精简目标 > 输出风格。
决策阶梯(按顺序停止在第一个可行层级)
- 这项需求是否真实需要?推测性未来需求先不做。
- 仓库是否已有 helper/类型/模式/组件?优先复用。
- 标准库是否覆盖?
- OS/浏览器/数据库/框架原生能力是否覆盖?
- 已安装依赖是否覆盖?
- 能否通过删除、配置或组合现有能力完成?
- 前六项都不成立,才新增最小实现。
标签
| 标签 | 含义 |
|---|
delete | 死代码、投机功能 |
reuse | 仓库已有能力可复用 |
stdlib | 手写标准库已有等价物 |
native | 平台原生能力可替代 |
yagni | 单实现接口、单产品工厂、无人使用配置 |
shrink | 逻辑等价但可更小 |
keep | 安全/正确性/兼容性强制保留 |
输出格式
文件:行号 | 标签 | 建议 | 替代物 | 风险门禁 | 预计净变化
keep 标签必须说明保留原因(安全/正确性/兼容性)。
- 涉及安全校验、防数据丢失、凭据处理、FROZEN/Fidelity 的代码默认 keep,除非有更强证据。
- 每条建议给出可验证的替代方案;没有替代方案的不建议。
边界
- 只读:不修改任何文件、不提交、不 push。
- 不替代正确性/安全/Fidelity Reviewer。
- 报告末尾标注:审查范围、审查身份(无法确认写 unknown)、未覆盖部分。