oh-my-pi /review 代码审查完整指南P0 到 P3 优先级排序一键裁决代码能否发布【免费下载链接】oh-my-pi⌥ Coding agent with the IDE wired in项目地址: https://gitcode.com/GitHub_Trending/oh/oh-my-pioh-my-pi是一款把 IDE 深度接入的编程智能体Coding agent其中的/review代码审查命令可以把你的改动交给专门的审查智能体先按P0 到 P3 四级优先级给问题排序再输出一个明确的能否发布裁决——是correct可合并还是incorrect存在阻断性缺陷全程只需一个斜杠命令。3 步启动 /review 代码审查/review是一个内置的交互式命令输入后会出现审查模式选择菜单。核心流程只有 3 步输入/review可直接跟在命令后附加自定义要求如重点关注并发安全选择审查范围选一个模式并确认等待裁决审查智能体逐文件分析后输出带优先级的 findings 和最终 verdict内置的 5 种审查模式模式适用场景说明 检测到的 PR对话中提到过 PR 链接从会话上下文自动识别无需手动输入 对比基线分支PR 风格审查选择 base 分支自动 diff 当前分支 未提交改动本地开发中审查 staged unstaged 全部改动 指定 commit追溯历史提交从最近 20 条提交中选一条✍️ 自定义指令专项检查自己写审查要求如只看性能问题命令的完整实现在 review/index.ts斜杠命令的发现与调度机制可参考 docs/slash-command-internals.md。P0 到 P3AI 代码审查的四级严重程度每条 finding 都必须标注 0–3 的优先级这是/review给出可执行结论的关键。官方定义的分级标准如下源自 reviewer.md级别判定标准典型例子处理时机P0阻断发布/线上运维与输入无关必然触发数据损坏、鉴权绕过立即修复否则不发布P1高风险高负载下的竞态条件下一迭代必须修复P2中风险边缘场景处理不当计划内修复⚪P3信息级次优但正确锦上添花发布裁决由 3 个字段组成的能否发布结论审查结束前审查智能体必须交出一个结构化裁决而不是含糊的看起来还行overall_correctness整体正确性correct 无 bug/无阻断项可以合并incorrect 存在必须处理的问题explanation裁决说明1–3 句纯文本总结直接回答能不能发confidence置信度0.0–1.0告诉你这个裁决有多可靠注意一个容易忽略的细节风格、文档、命名等 nit 类问题不计入正确性裁决。也就是说只要没有 P0/P1 级缺陷即使有一堆 P3 建议结论依然是可发布——这正是裁决与唠叨的区别。严格过滤准则为什么 AI 审查不钻牛角尖很多 AI 审查工具最大的问题是误报泛滥。oh-my-pi 的审查智能体要求每条 finding 必须同时满足 6 条准则影响可证明——能指出具体受影响的代码路径拒绝猜测可执行——有明确的修复动作而非建议优化 X非故意——明显不是作者刻意的设计选择补丁引入——不标记存量旧 bug只查本次改动无隐藏假设——不对代码库或作者意图做未声明假设严格度对等——不提出代码库其他地方都不具备的严苛要求此外还有专门的跨边界检查任何穿过函数/模块边界的新类型、消息、枚举值都必须去消费端确认存在匹配的分发分支防止生产者发了消息、消费者静默丢弃这类集成 bug 漏审。大型改动审查并行审查员与自动噪音过滤当 diff 变大时/review会自动做两件事并行分派按改动行数和文件数计算审查员数量小改动 1 个超大改动最多 16 个并按同目录同智能体、测试跟随实现的原则分组通过task工具并行派发噪音过滤lock 文件、构建产物、图片字体、node_modules等自动排除且会在结果中列出排除清单审查不跑偏这套分发 过滤逻辑让几千行的巨型 PR 也能被审查得条理清晰。相关机制可阅读 docs/task-agent-discovery.md 与 docs/tools/task.md。关键文件与延伸阅读资料路径/review 命令实现review/index.ts审查员智能体定义含 P0-P3 标准reviewer.md审查请求提示词模板review-request.md无头headless审查模板review-headless-request.md斜杠命令内部机制docs/slash-command-internals.mdCLI 完整参考docs/cli-reference.md常见问题Q审查会修改我的代码吗不会。审查智能体的 Bash 工具是只读的只允许git diff、git log、gh pr diff等命令明确禁止编辑文件或触发构建。Q和 /security 安全扫描有什么区别/review关注本次改动是否正确、能否合并/security是独立的安全扫描流水线计划、扫描、导入 SARIF、对比 lineage两者互补。Q审查结论能当最终把关吗建议把incorrect裁决和 P0/P1 发现当作硬性阻断项处理P2/P3 按排期跟进——这样既能拦住真正的事故又不会被低级别建议拖慢发布节奏。 小结oh-my-pi 的/review把AI 代码审查从一堆零散评论升级成了分级 裁决的工程流程——P0 到 P3 排序告诉你要先修什么correct/incorrect裁决直接回答你能不能发布。【免费下载链接】oh-my-pi⌥ Coding agent with the IDE wired in项目地址: https://gitcode.com/GitHub_Trending/oh/oh-my-pi创作声明：本文部分内容由AI辅助生成（AIGC），仅供参考
oh-my-pi /review 代码审查完整指南P0 到 P3 优先级排序一键裁决代码能否发布【免费下载链接】oh-my-pi⌥ Coding agent with the IDE wired in项目地址: https://gitcode.com/GitHub_Trending/oh/oh-my-pioh-my-pi是一款把 IDE 深度接入的编程智能体Coding agent其中的/review代码审查命令可以把你的改动交给专门的审查智能体先按P0 到 P3 四级优先级给问题排序再输出一个明确的能否发布裁决——是correct可合并还是incorrect存在阻断性缺陷全程只需一个斜杠命令。3 步启动 /review 代码审查/review是一个内置的交互式命令输入后会出现审查模式选择菜单。核心流程只有 3 步输入/review可直接跟在命令后附加自定义要求如重点关注并发安全选择审查范围选一个模式并确认等待裁决审查智能体逐文件分析后输出带优先级的 findings 和最终 verdict内置的 5 种审查模式模式适用场景说明 检测到的 PR对话中提到过 PR 链接从会话上下文自动识别无需手动输入 对比基线分支PR 风格审查选择 base 分支自动 diff 当前分支 未提交改动本地开发中审查 staged unstaged 全部改动 指定 commit追溯历史提交从最近 20 条提交中选一条✍️ 自定义指令专项检查自己写审查要求如只看性能问题命令的完整实现在 review/index.ts斜杠命令的发现与调度机制可参考 docs/slash-command-internals.md。P0 到 P3AI 代码审查的四级严重程度每条 finding 都必须标注 0–3 的优先级这是/review给出可执行结论的关键。官方定义的分级标准如下源自 reviewer.md级别判定标准典型例子处理时机P0阻断发布/线上运维与输入无关必然触发数据损坏、鉴权绕过立即修复否则不发布P1高风险高负载下的竞态条件下一迭代必须修复P2中风险边缘场景处理不当计划内修复⚪P3信息级次优但正确锦上添花发布裁决由 3 个字段组成的能否发布结论审查结束前审查智能体必须交出一个结构化裁决而不是含糊的看起来还行overall_correctness整体正确性correct 无 bug/无阻断项可以合并incorrect 存在必须处理的问题explanation裁决说明1–3 句纯文本总结直接回答能不能发confidence置信度0.0–1.0告诉你这个裁决有多可靠注意一个容易忽略的细节风格、文档、命名等 nit 类问题不计入正确性裁决。也就是说只要没有 P0/P1 级缺陷即使有一堆 P3 建议结论依然是可发布——这正是裁决与唠叨的区别。严格过滤准则为什么 AI 审查不钻牛角尖很多 AI 审查工具最大的问题是误报泛滥。oh-my-pi 的审查智能体要求每条 finding 必须同时满足 6 条准则影响可证明——能指出具体受影响的代码路径拒绝猜测可执行——有明确的修复动作而非建议优化 X非故意——明显不是作者刻意的设计选择补丁引入——不标记存量旧 bug只查本次改动无隐藏假设——不对代码库或作者意图做未声明假设严格度对等——不提出代码库其他地方都不具备的严苛要求此外还有专门的跨边界检查任何穿过函数/模块边界的新类型、消息、枚举值都必须去消费端确认存在匹配的分发分支防止生产者发了消息、消费者静默丢弃这类集成 bug 漏审。大型改动审查并行审查员与自动噪音过滤当 diff 变大时/review会自动做两件事并行分派按改动行数和文件数计算审查员数量小改动 1 个超大改动最多 16 个并按同目录同智能体、测试跟随实现的原则分组通过task工具并行派发噪音过滤lock 文件、构建产物、图片字体、node_modules等自动排除且会在结果中列出排除清单审查不跑偏这套分发 过滤逻辑让几千行的巨型 PR 也能被审查得条理清晰。相关机制可阅读 docs/task-agent-discovery.md 与 docs/tools/task.md。关键文件与延伸阅读资料路径/review 命令实现review/index.ts审查员智能体定义含 P0-P3 标准reviewer.md审查请求提示词模板review-request.md无头headless审查模板review-headless-request.md斜杠命令内部机制docs/slash-command-internals.mdCLI 完整参考docs/cli-reference.md常见问题Q审查会修改我的代码吗不会。审查智能体的 Bash 工具是只读的只允许git diff、git log、gh pr diff等命令明确禁止编辑文件或触发构建。Q和 /security 安全扫描有什么区别/review关注本次改动是否正确、能否合并/security是独立的安全扫描流水线计划、扫描、导入 SARIF、对比 lineage两者互补。Q审查结论能当最终把关吗建议把incorrect裁决和 P0/P1 发现当作硬性阻断项处理P2/P3 按排期跟进——这样既能拦住真正的事故又不会被低级别建议拖慢发布节奏。 小结oh-my-pi 的/review把AI 代码审查从一堆零散评论升级成了分级 裁决的工程流程——P0 到 P3 排序告诉你要先修什么correct/incorrect裁决直接回答你能不能发布。【免费下载链接】oh-my-pi⌥ Coding agent with the IDE wired in项目地址: https://gitcode.com/GitHub_Trending/oh/oh-my-pi创作声明：本文部分内容由AI辅助生成（AIGC），仅供参考
