Open Interpreter 的 review-agent 技能剖析只读、缺陷优先的代码审查 Skill 是怎么定义和运行的【免费下载链接】openinterpreterA coding agent for open models like Kimi K3 and GLM 5.3项目地址: https://gitcode.com/GitHub_Trending/op/openinterpreter本文围绕仓库中的 review-agent 技能定义 展开先完整拆解这份 SKILL.md 的审查规则审查目标、基线比较方法、缺陷判定五条件、P0–P3 严重级输出格式再结合codex-rs/skills与codex-rs/app-server的源码说明该技能如何作为系统技能内嵌进客户端、如何被/review与interpreter exec review显式触发以及allow_implicit_invocation: false策略的含义。读完你可以完整理解一个缺陷优先的代码审查 Agent的技能契约设计并能照此思路为自己的项目定制同类只读审查技能。一、review-agent 是什么一个内嵌的系统级代码审查技能SKILL.md 位于codex-rs/skills/src/assets/samples/review-agent/目录下属于 Open Interpreter 随二进制分发的系统技能system skills。它的作用由 frontmatter 一句话概括--- name: review-agent description: Perform a read-only, defect-first review of a specified code change and return every actionable finding. Use when another agent delegates review of uncommitted changes, a base-branch diff, a commit, or custom review instructions. ---frontmatter 只有name和description两个字段description既是技能的自述也是模型判断何时该用这个技能的依据——覆盖未提交改动、基线分支 diff、指定 commit、自定义审查指令四类目标。与该 SKILL.md 配对的 agents/openai.yaml 声明了技能的界面元数据与调用策略interface: display_name: Review Agent short_description: Find actionable bugs in code changes default_prompt: Use $review-agent to review the requested code changes and return actionable findings. policy: allow_implicit_invocation: false从 model.rs 的源码结构看SkillPolicy.allow_implicit_invocation缺省时默认是true见 SkillMetadata::allows_implicit_invocation即模型可以在任务中自主联想到并调用该技能而 review-agent 显式关闭了这一开关。这意味着review-agent 只能被显式点名使用例如审查入口生成的提示词、或用户直接提及该技能模型不会在日常编码中顺手把它拉进来——这对一个会输出大量审查结论的技能是合理的收敛设计避免无关会话被审查输出污染。从源码结构看这些samples/下的技能目录整体通过include_dir!宏编译进二进制并在启动时解压安装到CODEX_HOME/skills/.system/下见 lib.rs 中的SYSTEM_SKILLS_DIR与install_system_skills并用带盐的指纹文件.codex-system-skills.marker避免每次启动重复写入。因此在本地查看当前版本实际生效的 review-agent 技能应到 Open Interpreter 主目录如~/.openinterpreter下的skills/.system/review-agent/SKILL.md技能发现路径的其他层级仓库级.agents/skills/、个人级~/.agents/skills/等可参考 docs/skills.md。二、SKILL.md 全文规则拆解审查契约的每一条2.1 行为红线只读且不产生副作用技能开头立即划定了四条不许做Inspect the requested target directly and return every finding that the author would likely fix. Do not modify files, create commits, push branches, post review comments, or delegate the review to another agent.即直接检查目标本身而不是转述别人给的信息并且不得修改文件、创建 commit、推送分支、发布评审评论也不允许把审查任务再委托给另一个 agent。这使 review-agent 成为一个纯感知-推理-输出的只读角色即使运行在宽松的审批策略下也不会对仓库状态产生副作用。2.2 审查流程四步先读取适用的AGENTS.md指令项目级约定应影响什么算问题的判断检查目标对象的完整 diff并阅读足够的周边代码以理解每个被改动路径的上下文识别该变更引入的具体回归concrete regressions——找到第一个问题后必须继续审查完整个 diff不允许见好就收核对相关测试与调用点确认每一条发现都是真实且可执行的actionable。这四步的顺序暗含优先级先建立项目语境AGENTS.md再建立变更语境diff 周边代码然后才做缺陷判定最后用测试和调用方做证据闭环。2.3 基线分支审查比较实际会合并的内容这是 SKILL.md 中最具工程细节的一段。对 base-branch 审查对应interpreter exec review --base main这类目标文档明确要求比较对象应是最终会合并进基线的变更而不是直接拿本地分支 tip 与基线做 diff避免把本地未推送/超前提交算进审查范围若分支存在 upstream 且 upstream 领先本地分支比较基准应解析到该 upstream否则用本地分支具体操作执行git merge-base HEAD comparison-ref得到 merge-base再检查git diff merge-base-sha若本地分支无法解析应显式尝试其配置的 upstream之后才允许报告目标不可用。这段规则本质上是把git rebase/merge的三方合并语义内化给了模型审查的是合入点两侧的差异而非两个 tip 之间的差异。2.4 缺陷判定五条件全部满足才可上报SKILL.md 规定只有当以下条件同时成立时才允许标记一个问题它确实在正确性、安全性、性能或可维护性上有实质影响它是离散的、可执行的discrete and actionable它是由本次被审查的变更引入的introduced by the reviewed change受影响的场景或调用路径能从代码中被演示出来作者如果知道这个情况大概率会去修它。并明确排除四类噪声臆测性担忧、既有问题pre-existing problems、有意的行为变更、以及不遮挡阅读的纯风格挑剔。这套高门槛 白名单外全部丢弃的规则直接对应技能名称中的 defect-first——宁可漏报风格问题也不产出作者看了会反感的噪声。2.5 输出格式按严重级排序的发现列表结果必须发现优先、按严重度排序每条问题一个条目标题行格式固定为[P1] Imperative finding title — path/to/file.rs:line标题用祈使句并以路径:行号收尾随后跟一个短段落说明受影响场景与行为错误的原因引用范围要尽量小且必须与被审查的 diff 有交集防止指着 diff 之外大段说事。严重级定义如下级别含义P0通用性发布阻断问题或严重故障universal release blocker or critical failureP1应当紧接着修复的紧急缺陷P2应当修复的普通缺陷P3影响较低但仍值得修复的问题若无任何符合条件的发现必须原样输出No findings.禁止为了填满结果而编造发现。发现列表之后再给一段简短的总体评估并提及实质性的测试缺口或残余风险。三、源码纵深review-agent 如何被触发与分发3.1 审查入口/review与interpreter exec review用户侧的触发点在文档中有明确记载交互模式斜杠命令/reviewReview current changes for bugs and regressions见 docs/interactive.md 的用法说明非交互命令interpreter exec review --uncommitted、interpreter exec review --base main、interpreter exec review --commit abc123见 docs/exec.md 与 docs/cli-reference.md。注意 docs/auto-review.md 特意区分了两个概念auto-reviewapprovals_reviewer auto_review是把审批提示交给评审 agent 评估而代码审查要用/review或interpreter exec review——后者正是本文的 review-agent 技能的服务目标。3.2 Detached 交付把技能路径写进提示词在 turn_processor.rs 的review_start_inner中审查请求携带thread_id、target与delivery三个参数。当交付方式为Detached时服务端会构造如下提示词let review_skill_path system_cache_root_dir(self.config.codex_home) .join(review-agent) .join(SKILL.md); let prompt format!( Use $review-agent for this review.\n\n{target_prompt}, review_skill_path.display() );即提示词显式引用安装在本机skills/.system/review-agent/SKILL.md的技能路径让新会话加载该技能后执行审查提交前还会用MAX_USER_INPUT_TEXT_CHARS校验输入长度超限直接报input_too_large错误。Inline交付则走start_inline_review在原线程内完成审查。这与 2.1 节仅显式调用的策略相互印证即使是系统自带的审查流程也是通过点名技能文件来触发的。3.3 分发机制编译期内嵌 启动时指纹安装lib.rs 展示了这套分发如何做到升级即更新、不升级零开销include_dir!($CARGO_MANIFEST_DIR/src/assets/samples)把samples/整个目录含 review-agent、skill-creator、qa-testing 等全部系统技能编译进产物install_system_skills在启动时把内嵌目录写入$CODEX_HOME/skills/.system写入前用目录内所有文件的路径与内容哈希外加 saltv1计算指纹指纹与标记文件.codex-system-skills.marker一致则跳过安装不一致即技能内容有变更则先清空旧目录再整体重写。因此 review-agent 的 SKILL.md 更新会随新版本发布自动替换本地缓存副本无需用户手动同步。配套的单元测试同文件末尾的fingerprint_traverses_nested_entries验证了指纹遍历能覆盖嵌套条目。四、实战要点与适用前提调用方式交互式会话中直接输入/review脚本化场景用interpreter exec review --uncommitted | --base branch | --commit sha。基线审查时SKILL.md 的 merge-base 规则决定了审查范围是实际会合并的变更若你的本地分支落后于远端 upstream比较基准会自动切到 upstream。期望输出一段按 P0→P3 排序的发现列表[Px] 标题 — 文件:行号 一段场景说明或明确的No findings.末尾附总体评估与测试缺口提示。可以据此在 CI 中做结构化解析匹配行首的[P\d]即可提取发现条目。适用前提本技能依赖目标仓库可执行git merge-base/git diff且技能文件只从CODEX_HOME/skills/.system读取——修改本地缓存副本会被下次版本指纹校验覆盖定制应走 docs/skills.md 所述的.agents/skills/仓库级或~/.agents/skills/个人级技能目录同名技能以本地优先。设计可复用性如果想在 Open Interpreter 中自建一个同类只读审查/检查技能review-agent 提供了完整的模板——frontmatter 只保留name/description、用agents/openai.yaml的policy.allow_implicit_invocation: false锁死显式调用、在正文中用红线清单 触发条件 固定输出格式三段式约束模型行为。综上review-agent 技能 展示了 Open Interpreter 系统技能体系的一个完整切片SKILL.md 负责用自然语言精确约束审查行为与输出契约openai.yaml 负责界面与调用策略codex-rs/skills负责编译期打包与启动期安装codex-rs/app-server负责在审查请求中把技能路径写进提示词完成显式唤起——四层各自独立、又拼成一个可离线复现的只读缺陷优先代码审查能力。【免费下载链接】openinterpreterA coding agent for open models like Kimi K3 and GLM 5.3项目地址: https://gitcode.com/GitHub_Trending/op/openinterpreter创作声明：本文部分内容由AI辅助生成（AIGC），仅供参考 Open Interpreter 的 review-agent 技能剖析：只读、缺陷优先的代码审查 Skill 是怎么定义和运行的
Open Interpreter 的 review-agent 技能剖析只读、缺陷优先的代码审查 Skill 是怎么定义和运行的【免费下载链接】openinterpreterA coding agent for open models like Kimi K3 and GLM 5.3项目地址: https://gitcode.com/GitHub_Trending/op/openinterpreter本文围绕仓库中的 review-agent 技能定义 展开先完整拆解这份 SKILL.md 的审查规则审查目标、基线比较方法、缺陷判定五条件、P0–P3 严重级输出格式再结合codex-rs/skills与codex-rs/app-server的源码说明该技能如何作为系统技能内嵌进客户端、如何被/review与interpreter exec review显式触发以及allow_implicit_invocation: false策略的含义。读完你可以完整理解一个缺陷优先的代码审查 Agent的技能契约设计并能照此思路为自己的项目定制同类只读审查技能。一、review-agent 是什么一个内嵌的系统级代码审查技能SKILL.md 位于codex-rs/skills/src/assets/samples/review-agent/目录下属于 Open Interpreter 随二进制分发的系统技能system skills。它的作用由 frontmatter 一句话概括--- name: review-agent description: Perform a read-only, defect-first review of a specified code change and return every actionable finding. Use when another agent delegates review of uncommitted changes, a base-branch diff, a commit, or custom review instructions. ---frontmatter 只有name和description两个字段description既是技能的自述也是模型判断何时该用这个技能的依据——覆盖未提交改动、基线分支 diff、指定 commit、自定义审查指令四类目标。与该 SKILL.md 配对的 agents/openai.yaml 声明了技能的界面元数据与调用策略interface: display_name: Review Agent short_description: Find actionable bugs in code changes default_prompt: Use $review-agent to review the requested code changes and return actionable findings. policy: allow_implicit_invocation: false从 model.rs 的源码结构看SkillPolicy.allow_implicit_invocation缺省时默认是true见 SkillMetadata::allows_implicit_invocation即模型可以在任务中自主联想到并调用该技能而 review-agent 显式关闭了这一开关。这意味着review-agent 只能被显式点名使用例如审查入口生成的提示词、或用户直接提及该技能模型不会在日常编码中顺手把它拉进来——这对一个会输出大量审查结论的技能是合理的收敛设计避免无关会话被审查输出污染。从源码结构看这些samples/下的技能目录整体通过include_dir!宏编译进二进制并在启动时解压安装到CODEX_HOME/skills/.system/下见 lib.rs 中的SYSTEM_SKILLS_DIR与install_system_skills并用带盐的指纹文件.codex-system-skills.marker避免每次启动重复写入。因此在本地查看当前版本实际生效的 review-agent 技能应到 Open Interpreter 主目录如~/.openinterpreter下的skills/.system/review-agent/SKILL.md技能发现路径的其他层级仓库级.agents/skills/、个人级~/.agents/skills/等可参考 docs/skills.md。二、SKILL.md 全文规则拆解审查契约的每一条2.1 行为红线只读且不产生副作用技能开头立即划定了四条不许做Inspect the requested target directly and return every finding that the author would likely fix. Do not modify files, create commits, push branches, post review comments, or delegate the review to another agent.即直接检查目标本身而不是转述别人给的信息并且不得修改文件、创建 commit、推送分支、发布评审评论也不允许把审查任务再委托给另一个 agent。这使 review-agent 成为一个纯感知-推理-输出的只读角色即使运行在宽松的审批策略下也不会对仓库状态产生副作用。2.2 审查流程四步先读取适用的AGENTS.md指令项目级约定应影响什么算问题的判断检查目标对象的完整 diff并阅读足够的周边代码以理解每个被改动路径的上下文识别该变更引入的具体回归concrete regressions——找到第一个问题后必须继续审查完整个 diff不允许见好就收核对相关测试与调用点确认每一条发现都是真实且可执行的actionable。这四步的顺序暗含优先级先建立项目语境AGENTS.md再建立变更语境diff 周边代码然后才做缺陷判定最后用测试和调用方做证据闭环。2.3 基线分支审查比较实际会合并的内容这是 SKILL.md 中最具工程细节的一段。对 base-branch 审查对应interpreter exec review --base main这类目标文档明确要求比较对象应是最终会合并进基线的变更而不是直接拿本地分支 tip 与基线做 diff避免把本地未推送/超前提交算进审查范围若分支存在 upstream 且 upstream 领先本地分支比较基准应解析到该 upstream否则用本地分支具体操作执行git merge-base HEAD comparison-ref得到 merge-base再检查git diff merge-base-sha若本地分支无法解析应显式尝试其配置的 upstream之后才允许报告目标不可用。这段规则本质上是把git rebase/merge的三方合并语义内化给了模型审查的是合入点两侧的差异而非两个 tip 之间的差异。2.4 缺陷判定五条件全部满足才可上报SKILL.md 规定只有当以下条件同时成立时才允许标记一个问题它确实在正确性、安全性、性能或可维护性上有实质影响它是离散的、可执行的discrete and actionable它是由本次被审查的变更引入的introduced by the reviewed change受影响的场景或调用路径能从代码中被演示出来作者如果知道这个情况大概率会去修它。并明确排除四类噪声臆测性担忧、既有问题pre-existing problems、有意的行为变更、以及不遮挡阅读的纯风格挑剔。这套高门槛 白名单外全部丢弃的规则直接对应技能名称中的 defect-first——宁可漏报风格问题也不产出作者看了会反感的噪声。2.5 输出格式按严重级排序的发现列表结果必须发现优先、按严重度排序每条问题一个条目标题行格式固定为[P1] Imperative finding title — path/to/file.rs:line标题用祈使句并以路径:行号收尾随后跟一个短段落说明受影响场景与行为错误的原因引用范围要尽量小且必须与被审查的 diff 有交集防止指着 diff 之外大段说事。严重级定义如下级别含义P0通用性发布阻断问题或严重故障universal release blocker or critical failureP1应当紧接着修复的紧急缺陷P2应当修复的普通缺陷P3影响较低但仍值得修复的问题若无任何符合条件的发现必须原样输出No findings.禁止为了填满结果而编造发现。发现列表之后再给一段简短的总体评估并提及实质性的测试缺口或残余风险。三、源码纵深review-agent 如何被触发与分发3.1 审查入口/review与interpreter exec review用户侧的触发点在文档中有明确记载交互模式斜杠命令/reviewReview current changes for bugs and regressions见 docs/interactive.md 的用法说明非交互命令interpreter exec review --uncommitted、interpreter exec review --base main、interpreter exec review --commit abc123见 docs/exec.md 与 docs/cli-reference.md。注意 docs/auto-review.md 特意区分了两个概念auto-reviewapprovals_reviewer auto_review是把审批提示交给评审 agent 评估而代码审查要用/review或interpreter exec review——后者正是本文的 review-agent 技能的服务目标。3.2 Detached 交付把技能路径写进提示词在 turn_processor.rs 的review_start_inner中审查请求携带thread_id、target与delivery三个参数。当交付方式为Detached时服务端会构造如下提示词let review_skill_path system_cache_root_dir(self.config.codex_home) .join(review-agent) .join(SKILL.md); let prompt format!( Use $review-agent for this review.\n\n{target_prompt}, review_skill_path.display() );即提示词显式引用安装在本机skills/.system/review-agent/SKILL.md的技能路径让新会话加载该技能后执行审查提交前还会用MAX_USER_INPUT_TEXT_CHARS校验输入长度超限直接报input_too_large错误。Inline交付则走start_inline_review在原线程内完成审查。这与 2.1 节仅显式调用的策略相互印证即使是系统自带的审查流程也是通过点名技能文件来触发的。3.3 分发机制编译期内嵌 启动时指纹安装lib.rs 展示了这套分发如何做到升级即更新、不升级零开销include_dir!($CARGO_MANIFEST_DIR/src/assets/samples)把samples/整个目录含 review-agent、skill-creator、qa-testing 等全部系统技能编译进产物install_system_skills在启动时把内嵌目录写入$CODEX_HOME/skills/.system写入前用目录内所有文件的路径与内容哈希外加 saltv1计算指纹指纹与标记文件.codex-system-skills.marker一致则跳过安装不一致即技能内容有变更则先清空旧目录再整体重写。因此 review-agent 的 SKILL.md 更新会随新版本发布自动替换本地缓存副本无需用户手动同步。配套的单元测试同文件末尾的fingerprint_traverses_nested_entries验证了指纹遍历能覆盖嵌套条目。四、实战要点与适用前提调用方式交互式会话中直接输入/review脚本化场景用interpreter exec review --uncommitted | --base branch | --commit sha。基线审查时SKILL.md 的 merge-base 规则决定了审查范围是实际会合并的变更若你的本地分支落后于远端 upstream比较基准会自动切到 upstream。期望输出一段按 P0→P3 排序的发现列表[Px] 标题 — 文件:行号 一段场景说明或明确的No findings.末尾附总体评估与测试缺口提示。可以据此在 CI 中做结构化解析匹配行首的[P\d]即可提取发现条目。适用前提本技能依赖目标仓库可执行git merge-base/git diff且技能文件只从CODEX_HOME/skills/.system读取——修改本地缓存副本会被下次版本指纹校验覆盖定制应走 docs/skills.md 所述的.agents/skills/仓库级或~/.agents/skills/个人级技能目录同名技能以本地优先。设计可复用性如果想在 Open Interpreter 中自建一个同类只读审查/检查技能review-agent 提供了完整的模板——frontmatter 只保留name/description、用agents/openai.yaml的policy.allow_implicit_invocation: false锁死显式调用、在正文中用红线清单 触发条件 固定输出格式三段式约束模型行为。综上review-agent 技能 展示了 Open Interpreter 系统技能体系的一个完整切片SKILL.md 负责用自然语言精确约束审查行为与输出契约openai.yaml 负责界面与调用策略codex-rs/skills负责编译期打包与启动期安装codex-rs/app-server负责在审查请求中把技能路径写进提示词完成显式唤起——四层各自独立、又拼成一个可离线复现的只读缺陷优先代码审查能力。【免费下载链接】openinterpreterA coding agent for open models like Kimi K3 and GLM 5.3项目地址: https://gitcode.com/GitHub_Trending/op/openinterpreter创作声明：本文部分内容由AI辅助生成（AIGC），仅供参考