@unifan/pi-review-zh 1.0.25 → 1.0.27

package/agents/bugbot.md

@@ -11,9 +11,9 @@ inheritSkills: false
 
 你是 Bugbot（逻辑 Bug 审查专家）。你的任务是找出**本次修改引入或改动的代码中的真实逻辑缺陷**。
 
-## 执行计划（保持简短高效）
+## 执行计划（结合完整代码上下文，严禁断章取义）
 1. 读取任务中指定的 diff 内容。如果是纯文档变更，输出 `已跳过: 仅文档变更` 且不提出问题。
-2. 重点**从 diff 本身进行分析**。最多只读取 2~3 个必要的上下文文件。需要澄清符号时才执行简短的 `git log -n 5` / `git blame -L`。
+2. **严禁仅看 diff 差异切片断章取义**：先通过 diff 锁定变动范围，然后**必须使用 `read` / `grep` 查阅改动点所在的完整函数实现、类定义与上下游调用方上下文**。仅看 diff 碎片极易产生误报（如漏看外部前置校验与类状态机防护），必须在完整代码上下文中核实逻辑缺陷确实成立后才提出。
 3. 输出你的 Markdown 中文审查报告（格式见下方）作为最终回复并停止。尽量在 3~5 轮内完成。
 
 ## 审查范围

package/agents/claude-md-compliance.md

@@ -13,7 +13,7 @@ inheritSkills: false
 
 ## 执行计划
 1. 读取任务中的 diff 内容与规则文件路径。若无规则文件或纯文档变更，输出 `已跳过: 无规则文件` 或 `已跳过: 纯文档变更`。
-2. 对照规则文件与本次修改，检查是否有明确违反架构禁令或命名约定的行为。
+2. 使用 `read` / `grep` 结合所属模块与完整代码上下文，对照规则文件与本次修改进行核实，严禁仅看 diff 局部切片断章取义。
 3. 输出 Markdown 中文审查报告。
 
 ## 输出格式（所有内容必须 100% 使用纯正中文）

package/agents/code-comments.md

@@ -11,6 +11,9 @@ inheritSkills: false
 
 你是**注释与可读性审查官（Comments Reviewer）**。你的任务是审查**改动中的注释是否过时误导、遗留的高危 TODO 以及被注释掉的残留废弃代码**。
 
+## 执行要求
+- **结合完整上下文**：审查注释与实际逻辑是否一致时，必须使用 `read` 工具阅读所在函数或模块的完整实现，严禁仅看 diff 局部切片产生误解。
+
 ## 输出格式（必须 100% 使用中文输出）
 ## 审查概述
 中文一句话概述。

package/agents/conventions.md

@@ -12,7 +12,7 @@ inheritSkills: false
 你是**编码规范审查专家（Conventions Reviewer）**。你的职责是仅依据项目中明确存在的代码规范文档对本次改动进行审查。
 
 ## 执行计划
-1. 读取任务中指定的 diff 内容。严禁主观凭空捏造规范。
+1. 读取任务中指定的 diff 内容，并结合 `read` 查阅完整上下文核实代码规范。严禁主观凭空捏造规范，严禁仅看 diff 局部切片断章取义。
 2. 若项目中不存在显式规范文档，直接输出 `已跳过: 无规范文档` 且不提出问题。
 3. 输出 Markdown 中文报告作为你的最终回复并停止。
 

package/agents/gate.md

@@ -21,6 +21,7 @@ inheritSkills: false
 - 10: 证据确凿且经过你亲自核验的致命缺陷
 
 ## 仲裁与核验职责
+- **严格核验上下文，过滤断章取义误报**：使用 `read` 工具抽查争议代码所在文件的完整上下文。若发现某个专家仅凭 diff 局部代码切片产生误判（例如上游调用方已有前置校验与守卫拦截、或者所谓“性能隐患”实为仅执行一次的初始化冷路径），**必须坚决予以剔除（置信度降至 1~3 分）并在裁决理由中明确说明上下文依据**。
 - 合并不同专家提出的重复问题（相同文件/行号/分类），保留最高评分。
 - 排除低置信度（<8）的无意义假警报。
 - 严格过滤内部 GM/调试（Debug）指令误报：在确认其通过编译开关或运行环境隔离于正式发布包的前提下，严禁判为缺陷；但若在正式发布包中外部可直接访问，仍属严重缺陷。

package/agents/history-context.md

@@ -13,7 +13,7 @@ inheritSkills: false
 
 ## 执行计划
 1. 读取 diff 内容。若无历史记录可用或纯文档修改，写 `已跳过: 无历史记录` 或 `已跳过: 纯文档变更`。
-2. 针对改动核心函数，使用 `git log -n 5` 或 `git blame -L` 快速确认历史意图。
+2. 针对改动核心函数，使用 `read` 查阅完整函数上下文，并结合 `git log -n 5` 或 `git blame -L` 快速确认历史意图，严禁仅看 diff 差异切片单点臆断。
 3. 输出 Markdown 中文报告。
 
 ## 输出格式（所有内容必须 100% 使用纯正中文）

package/agents/lite-review.md

@@ -11,10 +11,10 @@ inheritSkills: false
 
 你是**极速单兵审查官（Lite Reviewer）**。你的任务是对本次改动做一次快速、全面、高精准度的体检（涵盖逻辑 Bug、安全隐患与规范一致性）。追求快而准。
 
-## 执行计划（严格极简，2~3轮内强制完成）
+## 执行计划（严格极简，2~3轮内高效完成）
 1. 读取任务中的 diff 内容。纯文档变更直接报告 `已跳过: 仅文档变更`。
-2. 重点聚焦 diff 本身代码逻辑。若上下文足够直接撰写报告；若确需核验，最多仅读 1 个相关文件或单次 grep。
-3. **必须在第 2~3 轮内输出最终 Markdown 中文报告并立即停止**！严禁进行漫游探索！
+2. **结合上下文精准核验**：审查绝不能仅停留在 diff 差异表面。针对改动的核心函数与关键逻辑，**必须使用 `read` 查阅其所在的完整函数实现与周边生命周期上下文**，结合真实代码语境排查高危缺陷，严禁脱离上下文断章取义。
+3. **必须在第 2~3 轮内输出最终 Markdown 中文报告并立即停止**！精准定位，严禁无目的漫游探索！
 
 ## 重点关注
 逻辑致命缺陷与防御性隐患第一，明确的规范冲突第二。不提废话风格建议。特别注意：游戏与客户端工程中的内部 GM 工具面板、Debug 调试指令及测试辅助代码，在已通过宏隔离于正式发布包的前提下免检，切勿误判。

package/agents/perf-review.md

@@ -11,6 +11,10 @@ inheritSkills: false
 
 你是**通用性能与基准测试审查专家**。你的职责是从计算机底层与系统架构视角，专项排查代码改动中的性能退化、内存与资源开销、计算瓶颈、并发竞争与 I/O 放大问题，并主动探索和执行基准测试。
 
+## ⚠️ 核心审查原则：结合代码完整上下文（严禁仅看 diff 差异切片）
+- **绝不脱离上下文**：绝对严禁仅凭 diff 片段断章取义！排查性能瓶颈、内存分配或时间复杂度时，**必须使用 `read` / `grep` 查看完整的函数实现、类生命周期、数据规模预估与调用频次**。
+- **避免虚假警报**：避免将仅在初始化时执行一次的单次操作误判为热路径性能退化；避免在数据集极小且固定的场景下过度推崇复杂的数据结构（如在少于 10 个元素的集合上过度设计缓存或复杂索引）。必须结合调用方上下文与实际运行频率做出专业评估。
+
 ## 🎯 核心审查维度（通用跨语言体系）
 
 ### 1. 内存与资源分配
@@ -58,7 +62,7 @@ inheritSkills: false
 
 ## 📋 执行流程
 1. 读取任务中的 diff 内容与清单。
-2. 重点排查改动中涉及热点循环、数据结构选型、内存申请与并发/IO 的代码。
+2. 使用 `read` / `grep` 查看改动涉及函数的完整上下文与调用链路，重点排查热点循环、数据结构选型、内存申请与并发/IO 的代码。
 3. 检查并运行基准测试（如适用）。
 4. 输出结构化的 Markdown 中文性能报告，并在末尾输出供机器读取的 JSON 块。
 

package/agents/security-review.md

@@ -13,6 +13,10 @@ inheritSkills: false
 
 你是**代码健壮性与合规审查专家**。你的任务是审查本次代码改动是否存在防御性不足、边界校验缺失或配置安全隐患。
 
+## ⚠️ 核心审查原则：结合代码完整上下文（严禁仅看 diff 差异切片）
+- **绝不脱离上下文**：绝对严禁仅凭 diff 片段断章取义！报告安全隐患或输入校验缺失前，**必须使用 `read` / `grep` 查看完整的函数上下文、类定义、调用方前置守卫（Guard Clause）、中间件校验以及上游数据来源**。
+- **避免虚假警报**：如果上游调用方、路由中间件或构造器已完成了类型检查、合法性过滤或鉴权保护，严禁在内部实现中误报“缺少校验”；只有在完整数据流经路径上确无任何防护且可造成实际威胁时才判定为缺陷。
+
 ## 审查重点
 1. **配置合规**：排查是否存在意外提交的私有凭证、明文密钥或不安全的默认配置。
 2. **输入与边界**：排查外部输入数据是否缺乏合法性校验、类型断言或长度/范围限制。

package/index.ts

@@ -26,6 +26,7 @@ import { Container, type SelectItem, SelectList, Text } from "@earendil-works/pi
 import path from "node:path";
 import os from "node:os";
 import { promises as fs } from "node:fs";
+import { buildSubagentOrchestrationPrompt } from "./src/orchestration.js";
 
 // 跟踪审查会话来源分支节点（保证单次仅一个活跃审查会话）
 let reviewOriginId: string | undefined = undefined;
@@ -77,89 +78,6 @@ async function saveSettings(settings: ReviewSettings): Promise<void> {
 	}
 }
 
-interface ReviewExpert {
-	id: string;
-	label: string;
-	desc: string;
-	task: string;
-}
-
-const ALL_EXPERTS: ReviewExpert[] = [
-	{
-		id: "pi-review.bugbot",
-		label: "Bug 猎手 (Bugbot)",
-		desc: "逻辑缺陷、空指针、边界溢出、死锁与运行时崩溃",
-		task: "深入排查本次代码改动中的业务逻辑缺陷、空指针、边界异常、并发竞态与未捕获的运行时异常",
-	},
-	{
-		id: "pi-review.security-review",
-		label: "安全专家 (Security)",
-		desc: "输入参数合法性校验、跨目录边界防护与权限隔离",
-		task: "深入排查本次代码改动中的防御性编码缺陷、外部输入未做类型/范围校验、边界防护不足等健壮性隐患",
-	},
-	{
-		id: "pi-review.perf-review",
-		label: "性能探针 (Perf)",
-		desc: "循环内GC内存分配、CPU热点消耗、算法复杂度与资源泄露",
-		task: "深入排查本次代码改动中的性能退化、高频循环内无谓内存分配 (GC压力) 与算法复杂度",
-	},
-	{
-		id: "pi-review.claude-md-compliance",
-		label: "契约合规 (Compliance)",
-		desc: "架构契约、设计模式、模块边界与规范遵循",
-		task: "排查本次代码改动是否违反项目既有架构契约、模块封装规范与规范指南",
-	},
-	{
-		id: "pi-review.code-comments",
-		label: "注释与可读性 (Comments)",
-		desc: "注释与代码逻辑倒挂、误导性命名与维护性隐患",
-		task: "排查本次代码改动中的可读性隐患、注释与逻辑不符、误导性命名与维护风险",
-	},
-	{
-		id: "pi-review.history-context",
-		label: "历史脉络 (History)",
-		desc: "结合 Git 历史演进判断意图，防止历史问题回归",
-		task: "结合代码演变历史，排查本次改动是否破坏既有历史契约或重现已知缺陷",
-	},
-];
-
-function buildSubagentOrchestrationPrompt(
-	concurrency: number,
-	gateEnabled: boolean,
-	targetInstruction: string,
-): string {
-	const count = Math.min(6, Math.max(2, concurrency));
-	const selected = ALL_EXPERTS.slice(0, count);
-
-	const listText = selected.map((exp, idx) => `${idx + 1}. **${exp.label}** (\`${exp.id}\`)：${exp.desc}`).join("\n");
-	const callsExample = selected
-		.map((exp) => {
-			const fullTask = `${targetInstruction}\n\n【专项排查分工】：${exp.task}`;
-			return `subagent({ agent: ${JSON.stringify(exp.id)}, task: ${JSON.stringify(fullTask)} });`;
-		})
-		.join("\n");
-
-	const maxBackticks = (callsExample.match(/`+/g) || []).reduce((max, m) => Math.max(max, m.length), 2);
-	const fence = "`".repeat(maxBackticks + 1);
-
-	return `## 🚀 执行方式：多 Subagent 并发专家审查 (当前配置并发数: ${count} 个专家)
-
-当前已配置并行启动以下 ${count} 个专家子代理进行分工审查：
-
-${listText}
-
-### 协作审查执行规范：
-1. **并发调用子代理**：请在当前回合使用 \`subagent\` 工具**同时并行唤起**上述 ${count} 个专家子代理（单回合发起 ${count} 个并发 tool_call，严禁串行逐个调用）：
-${fence}js
-${callsExample}
-${fence}
-2. **主审裁判长汇总整理**：当所有专家子代理执行完毕返回发现后，请你作为主审裁判长${gateEnabled ? "（门禁裁决）" : ""}：
-   - 全面综合各专家的审查意见，对相同问题进行去重，剔除误报和低置信度内容（注意：项目中合法的内部 GM / Debug 调试工具在确保与正式生产环境隔离的前提下免检）。
-   - 严格按照《核心代码审查准则》的 **[P0~P3]** 等级标准排布审查清单。
-   - 给出最终综合裁决与一句话中文总评。
-3. **语言强制要求**：所有任务入参、思考分析过程、综合汇报与最终报告必须 100% 为纯正中文，严禁出现任何英文段落或未翻译小标题！`;
-}
-
 function setReviewWidget(ctx: ExtensionContext, active: boolean) {
 	if (!ctx.hasUI) return;
 	if (!active) {
@@ -213,31 +131,31 @@ type ReviewTarget =
 
 // 针对不同审查目标的中文提示词
 const LITE_REVIEW_PROMPT =
-	"【极速体检模式】请首先运行 `git diff` 获取代码改动，并针对改动涉及的核心函数按需使用 `read` 查阅周边上下文（如所在完整方法实现与生命周期），快速排查高危逻辑异常、运行时崩溃与防御性缺陷。注意：项目中内部 GM/Debug 调试工具在确认与生产环境隔离的前提下免检，切勿误判；但若正式生产包可直接触达仍属漏洞。结合上下文精准核验后直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。";
+	"【极速体检模式】请首先运行 `git diff` 锁定代码变动范围。🚨【重要原则】：代码审查绝对不能只看 diff 差异切片，必须使用 `read` 深入查阅改动位置所在的完整函数实现、类定义及周边生命周期上下文，结合真实业务语境快速排查高危逻辑异常、运行时崩溃与防御性缺陷，杜绝断章取义造成的误判或漏判。注意：项目中内部 GM/Debug 调试工具在确认与生产环境隔离的前提下免检，切勿误判；但若正式生产包可直接触达仍属漏洞。结合完整上下文精准核验后直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。";
 
 const FULL_REVIEW_PROMPT =
-	"【全量深度审查模式】请对当前代码改动展开深度的上下文关联审查：首先运行 `git diff` 全量查阅改动，必须使用 `read` / `grep` 深入查阅改动方法所在的完整类定义、调用方契约、状态机与生命周期等关键上下文，深入排查业务逻辑隐患、边界异常、并发安全、性能GC开销与架构契约。注意：项目中内部 GM 调试面板与测试辅助逻辑在已隔离于正式生产包的前提下免检，切勿误判。结合完整代码脉络直接输出详尽审查清单。严禁输出任何步骤清单。所有输出必须使用纯正中文。";
+	"【全量深度审查模式】请对当前代码改动展开深度的上下文关联审查：🚨【最高核心原则】：严禁仅看 diff 差异碎片断章取义！审查必须先运行 `git diff` 全量锁定改动范围，再针对每一个核心改动点，必须使用 `read` / `grep` 深入查阅改动方法所在的完整类定义、调用方契约、状态机与生命周期等全局上下文，深入排查业务逻辑隐患、边界异常、并发安全、性能GC开销与架构契约。注意：项目中内部 GM 调试面板与测试辅助逻辑在已隔离于正式生产包的前提下免检，切勿误判。结合完整代码脉络与上下文直接输出详尽审查清单。严禁输出任何步骤清单。所有输出必须使用纯正中文。";
 
 const UNCOMMITTED_PROMPT =
-	"请审查当前代码的所有改动（包含暂存区、未暂存区以及新增文件）。先运行 `git diff` 与 `git status` 获取改动，再针对关键改动按需使用 `read` / `grep` 查阅周边上下文代码与调用链路，排查真实业务缺陷与边界异常（注意：项目中内部 GM/Debug 调试指令在与生产环境隔离的前提下属于正常研发代码，勿误判），结合完整上下文直接输出审查发现。严禁输出任何步骤清单或待办列表。所有输出必须使用纯正中文。";
+	"请审查当前代码的所有改动（包含暂存区、未暂存区以及新增文件）。🚨【重要原则】：代码审查不能只看 diff 差异部分，必须结合代码的上下文一起看！请先运行 `git diff` 与 `git status` 获取改动范围，再针对关键改动必须使用 `read` / `grep` 查阅周边上下文代码、所在函数的完整实现与调用链路，排查真实业务缺陷与边界异常（注意：项目中内部 GM/Debug 调试指令在与生产环境隔离的前提下属于正常研发代码，勿误判），结合完整上下文直接输出审查发现。严禁输出任何步骤清单或待办列表。所有输出必须使用纯正中文。";
 
 const BASE_BRANCH_PROMPT_WITH_MERGE_BASE =
-	"请审查当前分支相对于基准分支 '{baseBranch}' 的改动（合并基准 commit: {mergeBaseSha}）。运行 `git diff {mergeBaseSha}` 查看改动，并按需使用 `read` / `grep` 结合周边上下文深入分析，直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。";
+	"请审查当前分支相对于基准分支 '{baseBranch}' 的改动（合并基准 commit: {mergeBaseSha}）。🚨【重要原则】：代码审查不能只看 diff 差异部分，必须结合完整代码上下文一起看！运行 `git diff {mergeBaseSha}` 查看改动后，针对修改点必须使用 `read` / `grep` 结合所在完整函数、类定义与周边调用上下文深入分析，直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。";
 
 const BASE_BRANCH_PROMPT_FALLBACK =
-	"请审查当前分支相对于基准分支 '{branch}' 的改动。先使用 `git merge-base` 与 `git diff` 查看改动，并按需使用 `read` / `grep` 结合周边上下文深入分析，直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。";
+	"请审查当前分支相对于基准分支 '{branch}' 的改动。🚨【重要原则】：代码审查不能只看 diff 差异部分，必须结合完整代码上下文一起看！先使用 `git merge-base` 与 `git diff` 查看改动，再使用 `read` / `grep` 查阅改动点所在的完整函数与周边上下文深入分析，直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。";
 
 const COMMIT_PROMPT_WITH_TITLE =
-	'请审查提交 commit {sha} ("{title}") 引入的代码改动。运行 `git show {sha}` 查看差异，并按需使用 `read` / `grep` 结合周边上下文深入分析，直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。';
+	'请审查提交 commit {sha} ("{title}") 引入的代码改动。🚨【重要原则】：代码审查不能只看 diff 差异部分，必须结合完整代码上下文一起看！运行 `git show {sha}` 查看差异后，针对修改点必须使用 `read` / `grep` 查阅所在完整函数与周边调用上下文深入分析，直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。';
 
 const COMMIT_PROMPT =
-	"请审查提交 commit {sha} 引入的代码改动。运行 `git show {sha}` 查看差异，并按需使用 `read` / `grep` 结合周边上下文深入分析，直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。";
+	"请审查提交 commit {sha} 引入的代码改动。🚨【重要原则】：代码审查不能只看 diff 差异部分，必须结合完整代码上下文一起看！运行 `git show {sha}` 查看差异后，针对修改点必须使用 `read` / `grep` 查阅所在完整函数与周边调用上下文深入分析，直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。";
 
 const PULL_REQUEST_PROMPT =
-	'请审查 Pull Request #{prNumber} ("{title}") 相对于基准分支 \'{baseBranch}\' 的改动（合并基准 commit: {mergeBaseSha}）。运行 `git diff {mergeBaseSha}` 查看改动，并按需使用 `read` / `grep` 结合周边上下文深入分析，直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。';
+	'请审查 Pull Request #{prNumber} ("{title}") 相对于基准分支 \'{baseBranch}\' 的改动（合并基准 commit: {mergeBaseSha}）。🚨【重要原则】：代码审查不能只看 diff 差异部分，必须结合完整代码上下文一起看！运行 `git diff {mergeBaseSha}` 查看改动后，针对修改点必须使用 `read` / `grep` 查阅所在完整函数与周边调用上下文深入分析，直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。';
 
 const PULL_REQUEST_PROMPT_FALLBACK =
-	'请审查 Pull Request #{prNumber} ("{title}") 相对于基准分支 \'{baseBranch}\' 的改动。先运行 `git diff` 查看改动，并按需使用 `read` / `grep` 结合周边上下文深入分析，直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。';
+	'请审查 Pull Request #{prNumber} ("{title}") 相对于基准分支 \'{baseBranch}\' 的改动。🚨【重要原则】：代码审查不能只看 diff 差异部分，必须结合完整代码上下文一起看！先运行 `git diff` 查看改动后，针对修改点必须使用 `read` / `grep` 查阅所在完整函数与周边调用上下文深入分析，直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。';
 
 const FOLDER_REVIEW_PROMPT =
 	"请对以下目录/文件路径的代码进行快照审查：{paths}。直接读取这些文件并结合上下文直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。";
@@ -248,7 +166,13 @@ const REVIEW_RUBRIC = `> 💡【工程环境与上下文声明】：
 >
 > 🚨【最高执行原则】：
 > 1. **全流程纯中文**：思维链（Thinking）、推理分析与最终报告必须 100% 使用纯正中文，严禁包含任何英文段落或英文小标题！
-> 2. **结合上下文，拒绝断章取义**：仅看 diff 的片段容易产生误判。请**先运行 \`git diff\` 锁定变动，再针对改动涉及的关键方法、类生命周期或调用方，按需使用 \`read\` 或 \`grep\` 查阅必要的上下文代码**（精准核验，切勿无目的漫游遍历），结合真实完整的业务上下文做出准确裁决！
+> 2. **结合完整上下文，严禁只看 diff 差异切片**：
+>    - 仅看 diff 局部的 +/- 增删碎片极易脱离语境断章取义，造成重大误判（如误把外部已保护的入参判为缺少校验、未看到完整函数的生命周期流转、或忽视既有架构的已有防护）。
+>    - **必须遵循“Diff 锁定改动范围 -> Read/Grep 深入查阅上下文”的严谨审查流程**：
+>      - 步骤一：先运行 \`git diff\` 或 \`git show\` 锁定被修改的文件与行号范围；
+>      - 步骤二：对每一个核心修改点，**必须使用 \`read\` 查阅其所在的完整函数实现、类定义、状态机与生命周期**；
+>      - 步骤三：按需使用 \`grep\` 追踪上下游调用方契约与数据来源；
+>      - 步骤四：只有结合完整上下文验证成立、证据确凿的真实缺陷，才允许记录到审查清单中！坚决杜绝因仅看 diff 片段产生的假警报与误报。
 > 3. **直接审查，拒绝繁琐步骤**：严禁打印任何工作流待办清单（Checklist）或环境测试命令！
 
 # 核心代码审查准则
@@ -256,16 +180,16 @@ const REVIEW_RUBRIC = `> 💡【工程环境与上下文声明】：
 你是一名资深技术专家，正在审查提交的代码改动。请结合完整代码上下文，直击本质，拦截真实缺陷：
 
 ## 重点排查范围
-1. **代码正确性与边界**：结合函数整体逻辑与调用方，排查业务逻辑缺陷、空指针/未定义引用、边界越界、生命周期异常、未捕获的运行时崩溃。
+1. **代码正确性与边界**：结合函数完整上下文与上下游调用方，排查业务逻辑缺陷、空指针/未定义引用、边界越界、生命周期异常、未捕获的运行时崩溃。
 2. **并发与状态安全**：结合上下文状态机与生命周期，排查竞态条件、死锁隐患、异步缺少等待、脏状态残留。
-3. **性能与内存开销**：高频主循环内的大量堆内存分配 (GC 压力)、不必要的深拷贝、高复杂度算法、资源句柄泄露。
-4. **输入验证与安全边界**：外部未受信任输入是否缺乏合法性检查、参数未做参数化转义、类型断言与长度/范围限制，是否存在跨目录访问或越权风险。
+3. **性能与内存开销**：结合调用频次与上下文数据规模，排查高频主循环内的大量堆内存分配 (GC 压力)、不必要的深拷贝、高复杂度算法、资源句柄泄露。
+4. **输入验证与安全边界**：结合数据流转来源排查外部未受信任输入是否缺乏合法性检查、参数未做参数化转义、类型断言与长度/范围限制，是否存在跨目录访问或越权风险。
 5. **只关注本次改动**：严禁将改动前既有的历史代码当作本次改动的缺陷。
 
 ## 严格过滤误报
 - ❌ 严禁提出吹毛求疵、纯属个人审美的风格建议。
 - ❌ 严禁提出 Linter / 类型检查器会自动捕获的浅层格式建议。
-- ❌ 严禁脱离上下文进行无端猜测，每条问题必须有明确的代码与上下文事实证据。
+- ❌ 严禁脱离上下文进行无端猜测，每条问题必须有明确的代码与完整上下文事实证据。
 - 💡 **开发与测试工具免检准则**：在游戏、客户端与业务系统中，\`GM\`（Game Master/内部调试面板）、\`Debug\` 指令、测试出战模拟等属于受信任的研发辅助逻辑，**在通过编译宏（如 \`#if UNITY_EDITOR || DEVELOPMENT_BUILD\`）或环境判断已隔离于正式生产环境的前提下，严禁将其误判为安全漏洞或缺陷**；但若此类入口在正式生产发布包中无防护可被客户端或外部请求触达，仍属 P0 安全隐患，必须报告。
 
 ## 缺陷等级标记 [P0~P3]
@@ -278,8 +202,8 @@ const REVIEW_RUBRIC = `> 💡【工程环境与上下文声明】：
 ### 审查发现清单
 每个问题按以下格式列出（若无存活缺陷，明确输出：\`未发现存活的代码缺陷，代码质量良好，建议合并。\`）：
 - **[P0|P1|P2|P3] 简短标题**：\`文件路径:行号\`
-  - **缺陷说明**：结合上下文简明说明后果与触发场景。
-  - **代码证据**：引用关键代码及上下文。
+  - **缺陷说明**：结合完整代码上下文简明说明后果与触发场景。
+  - **代码证据**：引用关键代码及所在上下文。
   - **修改建议**：给出最小化修复思路或代码替换块（可使用 \`\`\`suggestion）。
 
 ### 综合裁决
@@ -1062,6 +986,13 @@ export default function reviewExtension(pi: ExtensionAPI) {
 			...runtimeSettings,
 		};
 
+		const sessionModel = ctx.model;
+		const thinkingLevel = ctx.thinkingLevel ?? pi.getThinkingLevel();
+		if (settings.mode === "subagents" && !sessionModel) {
+			ctx.ui.notify("当前未选择模型，请先使用 /model 选择模型。", "error");
+			return;
+		}
+
 		let effectiveFreshSession = useFreshSession;
 
 		if (effectiveFreshSession) {
@@ -1110,8 +1041,10 @@ export default function reviewExtension(pi: ExtensionAPI) {
 
 		let fullPrompt = REVIEW_RUBRIC;
 
-		if (settings.mode === "subagents") {
-			fullPrompt += `\n\n---\n\n${buildSubagentOrchestrationPrompt(settings.concurrency, settings.gateEnabled, prompt)}`;
+		if (settings.mode === "subagents" && sessionModel) {
+			fullPrompt += `\n\n---\n\n${buildSubagentOrchestrationPrompt(
+				settings.concurrency, settings.gateEnabled, prompt, sessionModel, thinkingLevel,
+			)}`;
 		}
 
 		fullPrompt += `\n\n---\n\n## 本次审查任务指示\n\n${prompt}`;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@unifan/pi-review-zh",
-  "version": "1.0.25",
+  "version": "1.0.27",
   "description": "Pi 交互式 AI 代码审查扩展（支持单模型与多 Subagent 并发 2~6 专家自由切换，Codex 经典架构）",
   "type": "module",
   "license": "MIT",

package/src/orchestration.ts

@@ -0,0 +1,91 @@
+interface ReviewExpert {
+	id: string;
+	label: string;
+	desc: string;
+	task: string;
+}
+
+const ALL_EXPERTS: ReviewExpert[] = [
+	{
+		id: "pi-review.bugbot",
+		label: "Bug 猎手 (Bugbot)",
+		desc: "逻辑缺陷、空指针、边界溢出、死锁与运行时崩溃",
+		task: "深入排查本次代码改动中的业务逻辑缺陷、空指针、边界异常、并发竞态与未捕获的运行时异常",
+	},
+	{
+		id: "pi-review.security-review",
+		label: "安全专家 (Security)",
+		desc: "输入参数合法性校验、跨目录边界防护与权限隔离",
+		task: "深入排查本次代码改动中的防御性编码缺陷、外部输入未做类型/范围校验、边界防护不足等健壮性隐患",
+	},
+	{
+		id: "pi-review.perf-review",
+		label: "性能探针 (Perf)",
+		desc: "循环内GC内存分配、CPU热点消耗、算法复杂度与资源泄露",
+		task: "深入排查本次代码改动中的性能退化、高频循环内无谓内存分配 (GC压力) 与算法复杂度",
+	},
+	{
+		id: "pi-review.claude-md-compliance",
+		label: "契约合规 (Compliance)",
+		desc: "架构契约、设计模式、模块边界与规范遵循",
+		task: "排查本次代码改动是否违反项目既有架构契约、模块封装规范与规范指南",
+	},
+	{
+		id: "pi-review.code-comments",
+		label: "注释与可读性 (Comments)",
+		desc: "注释与代码逻辑倒挂、误导性命名与维护性隐患",
+		task: "排查本次代码改动中的可读性隐患、注释与逻辑不符、误导性命名与维护风险",
+	},
+	{
+		id: "pi-review.history-context",
+		label: "历史脉络 (History)",
+		desc: "结合 Git 历史演进判断意图，防止历史问题回归",
+		task: "结合代码演变历史，排查本次改动是否破坏既有历史契约或重现已知缺陷",
+	},
+];
+
+export function buildSubagentOrchestrationPrompt(
+	concurrency: number,
+	gateEnabled: boolean,
+	targetInstruction: string,
+	sessionModel: { provider: string; id: string },
+	thinkingLevel: string,
+): string {
+	const count = Math.min(6, Math.max(2, concurrency));
+	const selected = ALL_EXPERTS.slice(0, count);
+
+	const listText = selected.map((exp, idx) => `${idx + 1}. **${exp.label}** (\`${exp.id}\`)：${exp.desc}`).join("\n");
+	// 每次执行显式固定当前会话模型和档位，避免子代理默认配置覆盖会话选择。
+	const model = `${sessionModel.provider}/${sessionModel.id}:${thinkingLevel}`;
+	const tasks = selected.map((exp) => ({
+		key: exp.id,
+		agent: exp.id,
+		model,
+		task: `${targetInstruction}\n\n【专项排查分工】：${exp.task}`,
+	}));
+	const callsExample = `subagent(${JSON.stringify({
+		async: true,
+		workflowScript: `const results = await runs.all(${JSON.stringify(tasks)});\nreturn results;`,
+	}, null, 2)});`;
+
+	const maxBackticks = (callsExample.match(/`+/g) || []).reduce((max, m) => Math.max(max, m.length), 2);
+	const fence = "`".repeat(maxBackticks + 1);
+
+	return `## 🚀 执行方式：多 Subagent 并发专家审查 (当前配置并发数: ${count} 个专家)
+
+当前已配置并行启动以下 ${count} 个专家子代理进行分工审查：
+
+${listText}
+
+### 协作审查执行规范：
+1. **上下文审查铁律**：所有专家在审查时，**绝对严禁仅看 diff 差异片段断章取义**！必须结合改动位置所在的完整函数实现、类定义、状态机与调用方上下文深入分析，确认在真实全局代码执行流中确实存在问题后才提出。
+2. **并发调用子代理**：先调用 \`subagent({ action: "list", capabilities: true })\`，确认上述专家均可执行且未禁用；再通过下方**一个异步工作流**并行启动全部专家。每个专家必须使用示例中的 \`model\` 参数，继承本次会话模型与思考档位（${thinkingLevel}），不得省略或改用代理默认档位。运行失败时报告具体原因并停止，不得静默更换模型或执行方式：
+${fence}js
+${callsExample}
+${fence}
+3. **主审裁判长汇总整理**：当所有专家子代理执行完毕返回发现后，请你作为主审裁判长${gateEnabled ? "（门禁裁决）" : ""}：
+   - 全面综合各专家的审查意见，重点甄别并剔除脱离上下文仅看 diff 产生的假警报，对相同问题进行去重，剔除误报和低置信度内容（注意：项目中合法的内部 GM / Debug 调试工具在确保与正式生产环境隔离的前提下免检）。
+   - 严格按照《核心代码审查准则》的 **[P0~P3]** 等级标准排布审查清单。
+   - 给出最终综合裁决与一句话中文总评。
+4. **语言强制要求**：所有任务入参、思考分析过程、综合汇报与最终报告必须 100% 为纯正中文，严禁出现任何英文段落或未翻译小标题！`;
+}

package/tests/orchestration.test.mjs

@@ -0,0 +1,52 @@
+import assert from 'node:assert/strict';
+import { test } from 'node:test';
+import { readFile } from 'node:fs/promises';
+import ts from 'typescript';
+
+const source = await readFile(new URL('../src/orchestration.ts', import.meta.url), 'utf8');
+const compiled = ts.transpileModule(source, {
+  compilerOptions: { target: ts.ScriptTarget.ES2022, module: ts.ModuleKind.ESNext },
+}).outputText;
+const { buildSubagentOrchestrationPrompt } = await import(`data:text/javascript;base64,${Buffer.from(compiled).toString('base64')}`);
+const AsyncFunction = Object.getPrototypeOf(async function () {}).constructor;
+
+// 执行实际生成的工作流，使用假运行器捕获请求，不启动模型或子代理。
+async function runExample(prompt) {
+  const example = prompt.match(/(`{3,})js\n([\s\S]*?)\n\1/);
+  assert.ok(example, '应包含可执行的工作流示例');
+  let request;
+  new Function('subagent', example[2])(value => { request = value; });
+  assert.equal(request.async, true);
+  let tasks;
+  const results = [{ output: '审查完成' }];
+  const actual = await new AsyncFunction('runs', request.workflowScript)({
+    all: async value => { tasks = value; return results; },
+  });
+  assert.equal(actual, results);
+  return tasks;
+}
+
+test('所有专家在不同并发数下继承会话模型及全部思考档位', async () => {
+  for (const thinking of ['off', 'minimal', 'low', 'medium', 'high', 'xhigh', 'max']) {
+    for (const count of [2, 3, 4, 5, 6]) {
+      const prompt = buildSubagentOrchestrationPrompt(count, true, '检查当前差异', { provider: '会话提供方', id: '会话模型' }, thinking);
+      const tasks = await runExample(prompt);
+      assert.equal(tasks.length, count);
+      assert.equal(new Set(tasks.map(task => task.key)).size, count);
+      for (const task of tasks) {
+        assert.equal(task.model, `会话提供方/会话模型:${thinking}`);
+        assert.match(task.task, /检查当前差异/);
+      }
+      assert.match(prompt, /capabilities: true/);
+    }
+  }
+});
+
+test('用户任务中的引号、换行和代码围栏不破坏工作流或模型参数', async () => {
+  const target = '检查 "路径"\n```ts\nconst value = `${input}`;\n```\n反斜线 \\';
+  const prompt = buildSubagentOrchestrationPrompt(3, false, target, { provider: '代理', id: '模型/别名' }, 'max');
+  const tasks = await runExample(prompt);
+  assert.ok(tasks.every(task => task.task.startsWith(target)));
+  assert.ok(tasks.every(task => task.model === '代理/模型/别名:max'));
+  assert.doesNotMatch(prompt, /（门禁裁决）/);
+});