mcpField
C
codex-review

codex-review

github.com/ulpi-io测试与质量5140 次安装9 周前更新
代码仓库

关于 codex-review

codex-review 是一项通过 Codex CLI 协调外部代码审查的技能,面向需要审查代码变更的开发者。它要求先检查实际差异、聚焦变更风险,并在采取行动前核实审查结果。

主要功能

  1. 01检查 Codex 是否可用及已认证
  2. 02选择并检查审查差异范围
  3. 03编写针对变更的审查指令
  4. 04运行 codex review 并捕获输出
  5. 05核实发现并记录排除项

适用场景

  1. 01审查未提交的变更或指定提交
  2. 02聚焦变更文件中的风险
  3. 03后续审查时避免重复报告已修复问题

安装

复制下面的命令,在终端中运行。

$npx skills add https://github.com/ulpi-io/skills --skill codex-review

安装后在 Claude Code 中输入/codex-review确认已加载。

详细介绍

此技能用于协调外部审查,必须严格遵守流程。

不可妥协的规则:

  1. 编写 Codex 指令前,先阅读实际差异。
  2. 指令必须针对已更改的部分及其可能存在的风险。
  3. 绝不在审查指令中传递秘密或凭据值。
  4. 后续轮次必须沿用排除列表。
  5. 采取行动前,先验证返回的发现。

Codex 审查

输入

  • $request:可选的范围提示,例如 last commit、uncommitted、auth focus 或 round 2

目标

使用 codex review 进行外部审查,以便:

  • 读取正确范围的差异
  • 聚焦变更集中的实际风险区域
  • 返回结构化发现,而不是泛泛而谈

步骤 0:确认 Codex 是否可用

检查:

  • which codex
  • 当前配置所需的最简身份验证或环境检查

如果 CLI 不可用或未通过身份验证,说明阻碍并停止。

成功标准:可以在当前代码库中成功调用 Codex。

步骤 1:确定审查范围

确定要审查的范围:

  • 相对于基准分支的整个分支
  • 未提交的更改
  • 指定的提交

编写指令前,先查看实际差异摘要和已更改文件列表。

如果没有差异,停止并明确说明。

成功标准:审查目标明确,并有实际差异作为依据。

步骤 2:编写聚焦的审查指令

创建一个简短的临时指令文件,其中包含:

  • 更改内容
  • 最相关的风险区域
  • 后续轮次中需要排除的任何已修复问题
  • 要求根据实际代码验证发现
  • 简洁的预期输出格式

指令要具体。泛泛的提示会导致审查结果质量不佳。

成功标准:指令文件针对实际变更集编写。

步骤 3:运行 codex review

根据所选范围使用相应的调用方式:

  • 分支审查使用 --base <branch>
  • 工作树审查使用 --uncommitted
  • 单个提交审查使用 --commit <sha>

关键标志:

  • sandbox_permissions — Codex 需要磁盘读取权限才能验证发现: -c 'sandbox_permissions=["disk-full-read-access","disk-full-write-access","network-full-access"]'
  • instructions — 指向步骤 2 中的聚焦指令文件: -c 'instructions="/tmp/codex-review-instructions.md"'
  • --title "<description>" — 描述性审查标题

始终使用 2>&1 捕获 stderr(Codex 会将日志写入 stderr)。

如果预计审查会运行较长时间,可以在后台执行。

成功标准:Codex 针对预期范围运行,并返回可解析的输出。

步骤 4:总结发现

报告:

  • 审查范围
  • 按优先级分类的发现
  • 可用时注明文件和行号
  • 若未发现实质性问题,明确说明结果干净

如果用户希望修复问题,先在本地验证每项发现,再修改代码。

成功标准:用户得到清晰、范围明确的审查摘要,而不是原始 CLI 输出。

步骤 5:仅在带有排除项的情况下迭代

后续轮次中:

  • 将已修复的发现添加到排除部分
  • 尽可能缩小范围,聚焦新更改
  • 避免为重复、泛泛的全分支审查付费

成功标准:后续轮次关注新问题,而不是重复报告旧问题。

防护原则

  • 不要主动运行此技能;仅在用户明确要求时运行。
  • 不要在指令文件中放入秘密、令牌或私有配置值。
  • 不要盲目信任发现;必须在本地验证。
  • 不要用 Codex 审查代替先阅读差异。

输出约定

报告:

  1. 审查范围
  2. 提供给 Codex 的主要关注点
  3. 按优先级分类的发现,并在可用时注明位置
  4. 如果未发现实质性问题,明确说明结果干净
  5. 下一轮是否应排除之前已修复的问题

评论

支持 Markdown