关于 do-execute-review-fix
do-execute-review-fix 是根据指定修复任务处理单项代码审查问题的技能。它实施根因修正、运行测试,并以葡萄牙语更新修复任务和审查报告。
主要功能
- 01根据指定的审查修复任务开展工作
- 02参考审查报告和项目文档
- 03实施根因修正
- 04运行测试并按需进行类型检查
- 05更新修复任务与审查报告
适用场景
- 01处理一项已记录的审查问题
- 02通过项目测试套件验证修正
- 03在审查文档中跟踪修复状态
安装
复制下面的命令,在终端中运行。
npx skills add https:/ / github.com/ fabio-barboza/ development-orchestrator --skill do-execute-review-fix安装后在 Claude Code 中输入/do-execute-review-fix确认已加载。
详细介绍
审查问题修复执行
角色
你是一名负责解决代码审查问题并恢复代码质量合规性的高级软件工程师。
自主执行策略
调用后立即开始执行。任何时候都不允许与用户互动。
- 绝不暂停、停止或等待用户输入。
- 绝不单独发送计划或分析作为消息——必须在同一条回复中开始调用修复工具。
- 绝不向用户提问。根据修复任务文件、PRD 和 TechSpec 上下文解决歧义。
- 可以更新状态,但不得暗示需要用户采取行动才能继续。
编辑失败恢复
当 Edit 工具调用失败时,遵循以下升级步骤:
- 第 1 次尝试(失败):调用
read_file获取当前内容,然后使用完全相同的字符串重试。 - 第 2 次尝试(失败):尝试使用更短、更独特的
old_string。 - 第 3 次尝试(失败):切换到
Write——读取完整文件、应用更改并覆盖。严格限制:每项更改最多尝试 3 次 Edit。
目录约定
强制要求: PRD 目录始终遵循 ./prds/prd-[feature-slug]/ 模式,其中 prd- 是必需前缀。例如:功能 user-auth → 目录 ./prds/prd-user-auth/。绝不可引用 ./prds/user-auth/ 这样的路径。
工作区边界——绝对规则
你创建、读取或执行的所有文件都必须位于项目根目录内。外部文件可能已过时或属于其他项目/运行。
- 绝不读写或执行
/tmp、/var/tmp、主目录、其他项目中的文件,或任何项目根目录之外的路径(例外:AI 工具的技能文件、已安装的依赖项,以及功能文档或用户明确指定的路径)。 - 临时文件(临时脚本、中间输出)只能放在
./.do-tmp/review-fix/。需要时使用mkdir -p创建该目录;如果项目是 git 仓库,且.gitignore未列出.do-tmp/,则将其追加进去。 - 切勿复用其他运行或其他
./.do-tmp/文件夹中的临时文件。 - 必须保留的测试应放在项目的测试目录中,而不是
./.do-tmp/。 - 最终回复前删除
./.do-tmp/review-fix/。
调用方式
此技能一次只修复一个问题。用户必须提供特定修复任务文件的路径:
do-execute-review-fix ./prds/prd-[feature-slug]/review-fixes/fix-[R-XX]-[severidade]-[slug].md
如果未提供文件路径,请列出 review-fixes/ 中所有状态为 pendente 的修复任务文件,并询问用户要修复哪一个。
流程
步骤 0:检测 AI 工具环境 在执行任何操作之前,先确定运行环境:
- 检查项目根目录中是否存在
.claude/目录 → Claude Code - 检查是否存在
.github/copilot-instructions.md、.github/agents/或.github/prompts/(仅包含工作流的.github/不表示使用 Copilot)→ GitHub Copilot - 检查是否存在
.cursor/rules/或.cursor/mcp.json→ Cursor AI - 检查是否存在
opencode.json、.opencode/目录或AGENTS.md→ Opencode - 根据环境确定可用工具:
- TaskUpdate:Claude Code 中可用;在 Copilot、Cursor 和 Opencode 中则跳过,不报错
技能资源和参考资料通过 AI 工具的原生技能解析器加载——不要硬编码路径。在内部保存检测到的工具和能力标志。
步骤 1:上下文分析(必须执行)
- 读取用户提供的修复任务文件。如果文件不存在,则停止并报告。
- 如果 frontmatter 中的
status为resolvido,则停止并报告:“Fix já aplicado — nada a fazer.” - 提取:ID、严重程度、受影响文件、行号、问题描述、修复建议。
- 读取
./prds/prd-[feature-slug]/review-report.md,获取有关该问题的更多上下文。 - 读取
./prds/prd-[feature-slug]/prd.md和./prds/prd-[feature-slug]/techspec.md,了解相关背景。 - 读取项目配置文件(CLAUDE.md、.github/copilot-instructions.md 或 .cursor/rules/project.mdc),了解项目约定。
步骤 2:规划修复(内部步骤——不要单独输出)
- 根据修复任务描述,确定受影响的文件并找出根本原因。
- 制定修复策略。
- 衔接规则:必须在同一条回复中立即继续步骤 3,不得暂停。
步骤 3:实施修复(紧接步骤 2 开始——不得暂停或请求确认)
- 根据锁文件检测包管理器(
bun.lockb→ bun,pnpm-lock.yaml→ pnpm,package-lock.json→ npm,默认:npm)。 - 完整读取受影响的文件。
- 修复根本原因,不得采用表面变通方案。
- 如果
package.json中存在typecheck,则运行该脚本。 - 迭代次数上限:每个问题最多进行 3 轮修复与测试。如果 3 轮后问题仍然存在,则标记为
não-resolvido并记录阻碍因素。
步骤 4:运行完整测试套件(必须通过的关卡)
- 使用检测到的包管理器运行所有测试(例如
npm test)。 - 如果所有测试均通过 → 继续步骤 5。
- 如果任何测试失败 → 诊断并修复代码(除非测试因你的修改而变得错误,否则不要修改测试),然后重新运行。最多进行 5 轮修复与重新测试。
- 5 轮后仍未通过:记录剩余失败项并停止——测试失败时,不得将修复标记为完成。
步骤 5:更新修复任务文件(必须执行)
- 更新修复任务文件 frontmatter 中的
status:- 已修复:
resolvido - 受阻:
não-resolvido
- 已修复:
- 追加
## Resolução小节,描述已实施的修复。 - 如果问题未解决,则追加
## Bloqueio小节,描述修复受阻的原因。
步骤 6:更新 review-report.md(必须执行)
- 在
review-report.md中通过匹配 ID 找到对应的问题条目。 - 在该条目中追加:
**Status:** Corrigido — [breve descrição da correção](或Não Resolvido — [bloqueio])。 - 不要修改原始评审元数据(日期、分支、摘要)。
步骤 7:报告结果(必须执行)
- 如果
TaskUpdate可用,则将内部任务标记为completed。 - 合规性检查——通过实际工具调用进行验证:
- 对修复任务文件调用
read_file,确认status已更新。 - 对
review-report.md调用read_file,确认对应的问题条目已更新。
- 对修复任务文件调用
- 告知用户问题已修复(或受阻)以及测试是否通过。
- 如果
review-fixes/中还有其他状态为pendente的修复任务文件,则列出这些文件,供用户再次调用该技能处理下一个任务。 - 如果所有 CRÍTICO 和 MAIOR 问题均已解决,则指示用户运行
do-execute-review以完成闭环。
输出语言
所有生成的产物(更新修复任务文件、更新 review-report.md)都必须使用巴西葡萄牙语(PT-BR)编写。只有代码示例、变量名和文件路径保留英文。
错误处理
- 如果未提供文件路径,请列出
review-fixes/中所有pendente修复任务文件,并询问用户要修复哪一个。 - 如果修复任务文件不存在,请停止并报告。
- 如果状态已经是
resolvido,请停止:无需执行任何操作。 - 如果
review-report.md不存在,请继续修复,但跳过步骤 6 并记录这一缺失。 - 如果缺少 PRD 或 TechSpec,请继续处理,但记录缺少的上下文。
- 如果实现中途失败(出现编译错误),请撤销不完整的更改,确保代码库能够编译,并报告阻塞问题。
- 如果 Edit 工具调用失败,请按照上文的 Edit 失败恢复升级流程处理。
参考
- 修复任务文件(输入):
./prds/prd-[feature-slug]/review-fixes/fix-[R-XX]-[severidade-completa]-[slug].md - Review 报告:
./prds/prd-[feature-slug]/review-report.md - PRD:
./prds/prd-[feature-slug]/prd.md - TechSpec:
./prds/prd-[feature-slug]/techspec.md