simplify

简化

概述

将此流程用作评审前的清理关卡。目标不是正式批准，而是检查变更代码中可复用之处、代码质量问题和效率问题，然后修复值得处理的问题，同时不改变行为。

何时使用

• 用户希望在 reviewer 或 code-reviewer 之前进行类似 /simplify 的清理
• 功能已完成，差异中可能存在重复代码、粗糙之处或可避免的低效
• 希望在正式评审前减少可避免的评审意见
• 变更涉及的文件或逻辑较多，一轮简化不够

不要将此技能用于：

• 细微差异，只需进行一次局部清理即可
• 会改变需求的行为变更、功能开发或错误修复
• 对未改动模块进行大规模架构重设计
• 正式规范验证或安全评审

在评审流程中的位置

建议放置位置：

implement -> simplify -> code-reviewer and/or reviewer -> receiving-code-review

以下情况可跳过完整的三 Agent 流程：

• 变更很小且显而易见
• 只改动了一两行琐碎代码
• 用户希望立即进行正式评审，而不是先清理

工作流

阶段 1：识别变更

• 如果用户提供了基准版本／头部版本范围，使用该范围
• 如果存在暂存变更，检查 git diff HEAD，以包含暂存和未暂存的变更
• 否则使用 git diff
• 如果没有 git diff，则检查用户提到的或本会话中你之前编辑过的最近修改文件
• 为评审者保留一份简洁的已改动文件摘要，以及完整的差异文本

阶段 2：阅读本地规范

• 首先检查本地指引，例如 AGENTS.md、仓库约定和任何相关技能
• 遵循已改动文件中现有的命名、格式和结构
• 如果本地约定与通用清理偏好冲突，以本地约定为准

阶段 3：并行启动三个评审 Agent

对于中型或大型差异，在同一轮中并行启动三个只读子 Agent。尽可能使用 explorer Agent，因为这是代码库分析，而不是实现工作。

每个 Agent 必须收到：

• 完整差异
• 已改动文件列表
• 清理约束：不改变行为、不编辑文件、仅提供发现

三个评审领域：

• 代码复用评审
• 代码质量评审
• 效率评审

使用以下文件中的提示词模板：

• references/reuse-reviewer.md
• references/quality-reviewer.md
• references/efficiency-reviewer.md

重要事项：

• 评审 Agent 为只读，不会编辑文件
• 主 Agent 汇总发现并执行修复
• 不要并行启动多个工作 Agent 来编辑同一批已改动文件

阶段 4：修复值得处理的问题

• 等待所有评审者完成
• 汇总发现并去除重复项
• 如果问题能在不改变行为的情况下改善代码，则直接修复
• 如果某项发现属于误报或不值得引入改动，则说明原因并跳过，无需长篇争辩

阶段 5：验证

• 对已改动部分运行范围最小且有意义的验证
• 优先运行有针对性的测试，然后视情况运行 lint／构建
• 如果没有自动化验证，请明确说明，并提及残余风险
• 简要总结修复内容，或说明代码已足够整洁

快速参考

目标	优先采用	避免
可读性	清晰的名称、明确的分支、小巧且职责连贯的辅助函数	密集的单行代码、巧妙的小把戏
控制范围	已改动文件、近期差异、用户指定路径	未经请求清理整个仓库
安全性	编辑后进行有针对性的验证	假设重构一定安全
注释	说明意图和约束	复述显而易见的代码
Agent 使用	三个并行只读评审者，由一个控制者应用修复	多个编辑 Agent 修改同一批文件

常见错误

• 把简化变成重新设计
• 过早提取抽象
• 在不相关的文件中大范围重命名
• 追求减少代码行数，而不是提高可维护性
• 因为改动“看起来只是机械操作”而跳过验证
• 让三个 Agent 都编辑代码，而不是只报告发现的问题
• 将此技能用作正式审查或规范验证的替代方案

上游参考

此 Codex 技能将 Claude 工作流中使用的 /simplify 流程改编为原生 Codex 技能。请参阅 references/upstream-agent.md，了解源说明和改编细节。