Open Interpreter code-review Skill 实战子代理编排、800 行变更限制与模型上下文治理【免费下载链接】openinterpreterA coding agent for open models like Kimi K3 and GLM 5.3项目地址: https://gitcode.com/GitHub_Trending/op/openinterpreterOpen Interpreter 仓库内置了一套面向 Pull Request 最终评审的code-review技能它本身是一个编排器通过为每个code-review-*子技能派生独立子代理来并行执行多维度审查。读完本文你将掌握该技能的编排协议、输出规范以及破坏性变更、变更规模、模型可见上下文、测试规范四个审查维度的具体判定标准并看到这些标准在codex-rs源码中的实现落点。一、code-review 技能定位PR 最终评审的编排器该技能定义在 .codex/skills/code-review/SKILL.md是一个标准的 Skill目录内含SKILL.md文件头部用 frontmatter 声明元数据name: code-review description: Run a final code review on a pull requestdescription是技能被选中触发的依据因此写得非常具体——对 Pull Request 执行最终代码审查。技能正文只有一条核心指令和若干输出约束Use subagents to review code using all code-review-* skills other than this orchestrator. One subagent per skill. Pass full skill path to subagents. Use xhigh reasoning.其余规则如下不得遗漏You must return every single issue from every subagent. You can return an unlimited number of findings.——编排者必须原样汇总所有子代理的发现数量不设上限报告格式使用原始 Markdown 报告发现项必须编号便于引用回溯且每一条发现必须包含具体的文件路径与行号GitHub 集成约束仅当执行评审的 GitHub 用户是该 PR 的 owner 时才为 PR 添加code-reviewed标签除非被明确要求否则不得在 GitHub 上留下评论。从源码结构看这是一个典型的编排器 原子技能模式编排器不自己写审查规则而是把审查职责完全下放给同目录下的子技能自己只负责派工、汇总和格式约束。二、子代理编排模型一个子技能对应一个子代理编排指令中的三句话各解决一个工程问题Use subagents——子技能在独立子代理中执行。每个子代理拥有独立的上下文窗口破坏性变更扫描、上下文审查这类需要大量文件检索的任务不会互相挤占 tokenOne subagent per skill——一个子代理只负责一个子技能维度之间相互隔离。某个维度的长分析例如遍历 app-server 协议的全部 schema 变更不会阻塞其他维度也避免了先找到一个问题就停止的短路行为Pass full skill path to subagents——向子代理传递的是子技能的完整路径如.codex/skills/code-review-testing/SKILL.md让子代理按路径加载完整技能文本而不是靠模型回忆规则Use xhigh reasoning——要求子代理以最高推理档位运行对应到各子技能中那些需要跨文件、跨调用链推理的判定例如变更能否拆分为可评审的阶段。汇总侧的约束返回每一条 issue、编号、文件路径 行号保证最终报告可直接定位到代码位置也使code-reviewed标签的授予有据可查只有当报告完整覆盖所有子技能输出时才符合该标签的语义。仓库中该编排器的四个被调用子技能分别是子技能文件审查维度code-breaking-changes.codex/skills/code-review-breaking-changes/SKILL.md外部集成面的破坏性变更code-review-change-size.codex/skills/code-review-change-size/SKILL.md变更规模限制code-review-context.codex/skills/code-review-context/SKILL.md模型可见上下文的完整性code-review-testing.codex/skills/code-review-testing/SKILL.md测试编写规范三、维度一破坏性变更扫描code-review-breaking-changes该子技能聚焦外部集成面external integration surfaces要求审查者主动搜索以下四个面上的破坏性变更app-server APIsapp-server 对外暴露的协议与接口。从源码结构看协议层集中在 codex-rs/app-server-protocol/其schema/目录包含由代码生成的大量.ts与.jsonschema 产物——协议字段增删、重命名或语义变化都会直接波及下游消费方是这一维度的主要检查点CLI parameters命令行参数是用户脚本与 CI 的集成面参数移除、改名或默认值变化都可能破坏现有调用configuration loading配置加载行为的变化键名、解析规则、优先级同样会影响存量部署resuming sessions from existing rollouts从既有 rollout 恢复会话的能力。rollout 机制对应 codex-rs/rollout/ 模块若持久化格式或恢复逻辑变更旧会话将无法恢复属于典型的行为破坏。子技能还有一条行为约束值得注意Do not stop after finding one issue; analyze all possible ways breaking changes can happen.——与编排器返回所有发现的要求呼应要求穷举式分析而非抽样式分析。四、维度二变更规模限制code-review-change-size这是四个维度中唯一给出量化硬指标的技能规则明确除非变更是机械性的mechanical如纯重命名、格式化、批量替换变更总行数不得超过 800 行对于复杂逻辑变更规模应控制在 500 行以内若变更超限审查者必须说明它能否拆分为可评审的阶段reviewable stages并基于实际 diff、依赖关系与受影响的调用点指出应当最先合入的最小完整阶段。这条规则把变更太大无法评审从一个主观感受变成了可执行的判定审查结论不是建议拆分四个字而必须给出具体的拆分方案与首发阶段。五、维度三模型可见上下文治理code-review-context该技能针对 Codex 推理请求中发送给模型的上下文历史消息制定六条规则是四个维度中约束最细的一条No history rewrite——上下文必须增量式构建禁止改写历史避免频繁触发 cache miss——对上下文的频繁变更会击穿前缀缓存应尽量避免No unbounded items——注入模型上下文的任何内容都必须有界bounded size且带硬性上限hard cap单项不得大于 10K tokens新增的、可能超过 1K token 的单项视为 P0需要额外的手动评审所有注入片段必须定义为core/context中的 struct并实现ContextualUserFragmenttrait。第 6 条在源码中有清晰落点。codex-rs/core/src/context/mod.rs 模块头部注释即为 Context fragments injected into model input该文件集中声明并导出各个上下文片段实现CurrentTimeReminder、PermissionsInstructions、RolloutBudgetContext、TokenBudgetContext、ImageResizeNotice等三十余个片段模块。其中 mod.rs 第 46 行 将 trait 从codex_context_fragmentscrate 重新导出pub use codex_context_fragments::ContextualUserFragment;对应的工作区 crate 为 codex-rs/context-fragments/。也就是说新增上下文注入必须实现ContextualUserFragment这一审查标准与核心库中片段即 trait 实现 集中导出的代码组织方式一一对应审查者在 PR 中看到任何绕过该 trait 直接拼接上下文的写法即应判为违规。前几条规则同样可映射到具体机制RolloutBudgetContext、TokenBudgetContext等片段本身就是预算/上限类注入物印证了第 3、4 条对有界性的强调而避免 cache miss对应的是前缀缓存友好的消息追加模型——这也是第 1 条禁止历史改写的原因。六、维度四测试编写规范code-review-testing该子技能对 agent 相关变更的测试策略给出明确偏好优先集成测试而非单元测试集成测试位于core/suite下使用test_codex搭建 Codex 的测试实例。测试脚手架test_codex定义在 codex-rs/core/tests/common/test_codex.rs它负责构造ThreadManager、Config、认证与模型提供商等完整运行环境从文件导入可见其依赖codex_core::CodexThread、codex_core::config::Config、codex_login::CodexAuth等codex-rs/core/tests/suite/下的用例如compact_remote.rs、remote_env.rs均基于它编写改变 agent 逻辑的功能必须新增集成测试且审查者要求提交方列出需要被覆盖的主要逻辑变更与用户可见行为清单单元测试放独立文件确需单测时放入专门的*_tests.rs测试文件避免在主实现代码中引入仅供测试的函数test-only functions复用既有 helper审查者应检查是否已有现成 helper 可以让测试更简洁可读而不是复制粘贴新的脚手架。这条维度把测试是否到位从有没有测试升级为测试层级是否与变更性质匹配、是否遵循仓库既有测试基建与codex-rs中统一的测试目录约定tests/common放脚手架、tests/suite放集成用例保持一致。七、技能格式与加载机制背景理解上述技能如何被触发可以参考仓库的技能规范文档 docs/skills.md。其中说明一个技能是包含SKILL.md及可选支持文件scripts/、references/、assets/的文件夹Open Interpreter 先读取技能元数据frontmatter 中的name/description仅当请求匹配时才加载完整技能内容description控制技能何时被选中应写得具体——code-review技能的 Run a final code review on a pull request 正是这种面向触发场景的描述文档给出的技能位置包括仓库级的.agents/skills/目录等本仓库自身使用的评审技能则放在.codex/skills/目录下code-review与四个code-review-*子技能同处该目录保证Pass full skill path to subagents时子代理能直接按路径命中。编排器模式在这一格式下尤其自然code-review的 SKILL.md 正文极短只讲派工与汇总协议而各审查维度的完整规则被隔离在各自的 SKILL.md 中按需加载、互不干扰。八、小结code-review技能的设计要点可以概括为三层编排层一个子技能一个子代理、传递完整技能路径、最高推理档位汇总时零遗漏并统一为编号 文件路径 行号的可定位报告规则层破坏性变更面向 app-server API、CLI 参数、配置加载、rollout 会话恢复四个集成面穷举变更规模以 800/500 行为硬指标并强制给出拆分方案上下文注入遵循增量构建、有界、10K tokens 上限、1K tokens 以上新单项 P0、统一实现ContextualUserFragment的六条规则测试优先集成、脚手架统一走test_codex落点层每条规则都能在当前仓库找到对应物——codex-rs/core/src/context/的片段体系、codex-rs/app-server-protocol/schema/的协议产物、codex-rs/core/tests/common/test_codex.rs的测试基建——这使得审查标准不是纸面规范而是与代码组织方式互相印证的可执行清单。对贡献者而言提交 PR 前按这四个维度自查集成面兼容性、diff 行数、新增上下文的有界性与 trait 实现、集成测试覆盖就是让code-review技能在最终评审中快速给出干净报告的最短路径。【免费下载链接】openinterpreterA coding agent for open models like Kimi K3 and GLM 5.3项目地址: https://gitcode.com/GitHub_Trending/op/openinterpreter创作声明：本文部分内容由AI辅助生成（AIGC），仅供参考