Agent Skills
返回列表
CodeFlow 代码审查守护

CodeFlow 代码审查守护

开发编程 更新于 2026.08.29

将以下提示词粘贴到你的 AI 对话框中:

请根据 https://skillhub.cn/install/skillhub.md,安装 @user_fc0d4945/codeflow-guard-skills。

技能介绍

要解决什么问题

合并前的代码审查往往缺少统一证据链:有人只看 git diff,有人忘了跑测试,敏感字面量也可能被直接贴进报告。更麻烦的是,风险等级和合并建议若与采集到的事实不一致,复审时很难对齐口径。

CodeFlow Guard 把审查拆成「先取证、再判级、再出报告」:用脚本稳定收集仓库状态、diff、测试与静态信号,再让模型按固定 rubric 和模板写结论,而不是凭印象打分。

如何工作

  1. 上下文采集:优先执行 collect-review-context.js(可传 --repo、--test-cmd、--path),产出 Review Brief、Review Files、Changed Line Anchors 等;脚本不可用时再手动跑 git status、git diff 与测试命令。
  2. 风险判级:读取 references/risk-rubric.md,将问题映射到 P0(阻断合并)~P3(优化建议)。
  3. 结构化报告:按 references/output-template.md 输出中文报告,包含审查上下文、变更摘要、Top 3、path:line 关键风险卡片、测试建议、合并前检查清单与复审标准。

报告硬约束包括:以脚本事实为准;待判断文件逐个进入变更摘要;敏感证据脱敏(前缀 + ***);区分「静态扫描跳过测试总数」与「本次需处理跳过测试」;结论区合并建议、总体风险、摘要等一项一行。

适用边界

  • 最终 P 级与合并建议必须由模型结合 rubric 与上下文判断,脚本信号只是疑似风险,不是定论。
  • 未出现 coverage 工具结果时,不要写覆盖率百分比,只写通过/失败/跳过或「无执行结果」。
  • examples/、test/ 等路径若在待判断列表中,仍要按实际变更审查,不能用目录名跳过。

使用场景

  • 合并 PR 前,需根据 `git diff`、测试命令输出和静态扫描信号写出带 P0~P3 的中文审查报告
  • 只审某次改动的模块时,用 `--path` 限定范围,并让采集脚本自动纳入直接相对引用文件
  • 变更落在登录、鉴权或资金相关路径时,逐项核验 `test.skip`、`it.skip` 等是否导致关键路径未测
  • Skill 安装在 `.codebuddy/skills` 等子目录,但仍要对业务仓库根目录执行 `--repo` 定向或全量审查

适合人员

  • 负责主干或发布分支合并门禁的 Tech Lead,需要 Top 3、`path:line` 关键风险与「可合并 / 补测后合并」的明确结论
  • 在本地先跑用户指定 `test-cmd` 再出报告的后端维护者,要求审查上下文里如实记录测试执行状态与结果
  • 处理 hotfix 的工程师,需要在报告中引用敏感证据但严格遵守前缀脱敏、禁止输出完整密钥
  • 做结对或异步 Code Review 的资深开发,要求 Review Brief 中每个待判断文件都进入变更摘要并被核验