{/* This page is auto-generated from the skill's SKILL.md by website/scripts/generate-skill-docs.py. Edit the source SKILL.md, not this page. */}

请求代码审查

提交前审查：安全扫描、质量门控、自动修复。

Skill 元数据


来源	内置（默认安装）
路径	`skills/software-development/requesting-code-review`
版本	`2.0.0`
作者	Hermes Agent（改编自 obra/superpowers + MorAlekss）
许可证	MIT
平台	linux, macos, windows
标签	`code-review`, `security`, `verification`, `quality`, `pre-commit`, `auto-fix`
相关 skill	`subagent-driven-development`, `writing-plans`, `test-driven-development`, `github-code-review`

参考：完整 SKILL.md

信息

以下是 Hermes 在触发此 skill 时加载的完整 skill 定义。这是 skill 激活时 agent 所看到的指令内容。

提交前代码验证

代码落地前的自动化验证流水线。包含静态扫描、基线感知质量门控、独立审查子 agent 以及自动修复循环。

核心原则： 任何 agent 都不应验证自己的工作。全新上下文能发现你遗漏的问题。

使用时机

实现功能或修复 bug 后，在 git commit 或 git push 之前

当用户说"commit"、"push"、"ship"、"done"、"verify"或"review before merge"时

在 git 仓库中完成包含 2 个以上文件编辑的任务后

在 subagent-driven-development 的每个任务后（两阶段审查）

跳过情形： 仅文档变更、纯配置调整，或用户说"skip verification"时。

本 skill 与 github-code-review 的区别： 本 skill 在提交前验证你自己的变更。github-code-review 用于在 GitHub 上审查他人的 PR 并添加行内评论。

第 1 步 — 获取 diff

若为空，依次尝试 git diff，再尝试 git diff HEAD~1 HEAD。

若 git diff --cached 为空但 git diff 显示有变更，告知用户先执行 git add <files>。若仍为空，运行 git status — 无内容可验证。

若 diff 超过 15,000 个字符，按文件拆分：

第 2 步 — 静态安全扫描

仅扫描新增行。任何匹配项均作为安全隐患输入第 5 步。

第 3 步 — 基线测试与 lint 检查

检测项目语言并运行相应工具。将你的变更作为 baseline_failures（暂存变更、运行、弹出）捕获变更前的失败数量。只有你的变更引入的新失败才会阻止提交。

测试框架（根据项目文件自动检测）：

Lint 检查与类型检查（仅在已安装时运行）：

基线对比： 若基线干净而你的变更引入了失败，则为回归。若基线本已有失败，仅统计新增失败数。

第 4 步 — 自查清单

在派发审查者之前快速扫描：

无硬编码密钥、API key 或凭据

对用户提供的数据进行输入验证

SQL 查询使用参数化语句

文件操作验证路径（防止路径遍历）

外部调用有错误处理（try/catch）

未遗留调试用 print/console.log

无注释掉的代码

新代码有测试（若测试套件存在）

第 5 步 — 独立审查子 agent

直接调用 delegate_task — 它不可在 execute_code 或脚本内部使用。

审查者仅获得 diff 和静态扫描结果，与实现者无共享上下文。失败关闭原则：无法解析的响应 = 失败。

第 6 步 — 评估结果

综合第 2、3、5 步的结果。

全部通过： 进入第 8 步（提交）。

任何失败： 报告失败内容，然后进入第 7 步（自动修复）。

VERIFICATION FAILED

Security issues: [list from static scan + reviewer]
Logic errors: [list from reviewer]
Regressions: [new test failures vs baseline]
New lint errors: [details]
Suggestions (non-blocking): [list]

第 7 步 — 自动修复循环

最多 2 次修复并重新验证的循环。

派生第三个 agent 上下文 — 不是你（实现者），也不是审查者。它仅修复已报告的问题：

修复 agent 完成后，重新运行第 1-6 步（完整验证循环）。

通过：进入第 8 步

失败且尝试次数 < 2：重复第 7 步

2 次尝试后仍失败：将剩余问题上报给用户，并建议执行 git stash 或 git reset 撤销变更

第 8 步 — 提交

若验证通过：

[verified] 前缀表示此变更已通过独立审查者批准。

参考：常见需标记的模式

Python

JavaScript

与其他 Skill 的集成

subagent-driven-development： 在每个任务后运行本 skill 作为质量门控。两阶段审查（规格合规性 + 代码质量）使用本流水线。

test-driven-development： 本流水线验证是否遵循了 TDD 纪律 — 测试存在、测试通过、无回归。

writing-plans： 验证实现是否符合计划需求。

注意事项

空 diff — 检查 git status，告知用户无内容可验证

非 git 仓库 — 跳过并告知用户

大 diff（>15k 字符） — 按文件拆分，逐一审查

delegate_task 返回非 JSON — 重试一次并使用更严格的 prompt（提示词），否则视为失败

误报 — 若审查者标记了有意为之的内容，在修复 prompt 中注明

未找到测试框架 — 跳过回归检查，审查者裁决仍然执行

Lint 工具未安装 — 静默跳过该检查，不视为失败

自动修复引入新问题 — 计为新失败，循环继续

"请求代码审查 — 提交前审查：安全扫描、质量门控、自动修复"

请求代码审查#

Skill 元数据#

参考：完整 SKILL.md#

提交前代码验证#

使用时机#

第 1 步 — 获取 diff#

第 2 步 — 静态安全扫描#

第 3 步 — 基线测试与 lint 检查#

第 4 步 — 自查清单#

第 5 步 — 独立审查子 agent#

第 6 步 — 评估结果#

第 7 步 — 自动修复循环#

第 8 步 — 提交#

参考：常见需标记的模式#

Python#

JavaScript#

与其他 Skill 的集成#

注意事项#

请求代码审查

Skill 元数据

参考：完整 SKILL.md

提交前代码验证

使用时机

第 1 步 — 获取 diff

第 2 步 — 静态安全扫描

第 3 步 — 基线测试与 lint 检查

第 4 步 — 自查清单

第 5 步 — 独立审查子 agent

第 6 步 — 评估结果

第 7 步 — 自动修复循环

第 8 步 — 提交

参考：常见需标记的模式

Python

JavaScript

与其他 Skill 的集成

注意事项