代码审查与测试指导
帮助编码代理为逻辑变更设计清晰、合适的测试。
技能仅提供测试编写建议,不执行命令、不请求权限,也未发现恶意、窃密或外泄行为;但没有说明敏感数据、外部副作用、用户确认、回滚或测试隔离要求,因此扣分。
内容内部一致,明确了集成测试位置、test_codex、单元测试文件命名和避免测试专用主代码;但没有异常输入、依赖缺失、失败诊断或不可继续时的反馈规则,且本次仅静态阅读,不能超过10分。
适用于修改 Codex agent 逻辑时的测试决策,触发场景较明确;但没有定义非适用场景、输入输出、边界条件、语义触发规则或中文及不同环境适配,因此扣分。
文档短小且结构清晰,给出路径、命名和渐进式选择;仓库提供 Apache-2.0 和官方来源信息,但该技能自身没有版本、变更记录、维护责任、安装说明、示例或故障排查,因此扣分。
能够直接指导选择集成测试或专用单元测试文件,并减少测试专用函数;但没有完整流程、示例、验收输出或与替代方案的比较,静态材料不足以证明结果只需轻量复核,因此不超过7分。
建议可由技能文本和仓库中的测试目录约定审计,且仓库包含测试与 CI 相关材料;但没有针对该技能的专门测试、第三方复现证据或多源交叉验证,本次未执行任何内容,因此仅给有限分数。
- 技能没有明确说明测试失败、环境缺失或无法运行时应如何诊断和反馈。
- 没有定义敏感数据处理、隔离、回滚或用户确认要求。
- 缺少版本、维护责任、变更记录、示例及非适用场景说明。
它能做什么 & 适用场景
该技能为代理提供测试编写规范,重点建议针对代理逻辑变更采用集成测试。集成测试应放在 core/suite,并使用 test_codex 初始化 Codex 测试实例。必要时,单元测试应放在专用的 *_tests.rs 文件中。它还要求列出需要覆盖的主要逻辑变更和用户可见行为,并建议复用现有测试辅助工具。
它指导代理识别主要逻辑变更和用户可见行为,优先在 core/suite 中编写使用 test_codex 的集成测试;在需要单元测试时,将测试放入专用的 *_tests.rs 文件;避免在主实现中加入仅供测试使用的函数;并检查现有辅助工具以提升测试可读性和简洁性。
- 维护 Codex 代理逻辑的工程师需要为一项行为变更确定测试类型时。
- 开发影响用户可见行为的功能后,需要补充集成测试时。
- Rust 项目需要把单元测试与主实现分离时。
- 测试代码变得重复或难以阅读,需要查找现有辅助工具时。
优缺点一览
- 明确优先使用集成测试验证代理逻辑变更。
- 规定了 core/suite、test_codex 和 *_tests.rs 等具体测试位置或方式。
- 提醒避免测试专用函数污染主实现,并鼓励复用现有辅助工具。
- 它只提供测试编写指导,不会自行运行测试或生成测试报告。
- 未提供完整测试示例、失败处理流程或覆盖率标准。
- 内容针对 Codex 仓库结构,移植到其他项目时需要确认对应目录和测试初始化方式。
如何安装
该技能位于 openai/codex 仓库的 .codex/skills/code-review-testing/SKILL.md。README 提供了 Codex CLI 的安装方式,例如 macOS/Linux 可运行 curl -fsSL https://chatgpt.com/codex/install.sh | sh,Windows 可运行 powershell -ExecutionPolicy ByPass -c "irm https://chatgpt.com/codex/install.ps1 | iex",也可使用 npm install -g @openai/codex 或 brew install --cask codex。README 未说明如何将该技能单独安装到其他客户端。
如何使用
在该仓库中使用 Codex CLI,并提出类似“为这项代理逻辑变更设计并实现测试,优先使用 core/suite 中的 test_codex 集成测试;列出需要覆盖的主要逻辑变更和用户可见行为”的请求。SKILL.md 未提供独立命令或专用触发语法。