@unifan/pi-review-zh 1.0.25 → 1.0.26

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) {
@@ -1062,6 +980,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 +1035,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.26",
   "description": "Pi 交互式 AI 代码审查扩展（支持单模型与多 Subagent 并发 2~6 专家自由切换，Codex 经典架构）",
   "type": "module",
   "license": "MIT",

package/src/orchestration.ts

@@ -0,0 +1,90 @@
+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. **并发调用子代理**：先调用 \`subagent({ action: "list", capabilities: true })\`，确认上述专家均可执行且未禁用；再通过下方**一个异步工作流**并行启动全部专家。每个专家必须使用示例中的 \`model\` 参数，继承本次会话模型与思考档位（${thinkingLevel}），不得省略或改用代理默认档位。运行失败时报告具体原因并停止，不得静默更换模型或执行方式：
+${fence}js
+${callsExample}
+${fence}
+2. **主审裁判长汇总整理**：当所有专家子代理执行完毕返回发现后，请你作为主审裁判长${gateEnabled ? "（门禁裁决）" : ""}：
+   - 全面综合各专家的审查意见，对相同问题进行去重，剔除误报和低置信度内容（注意：项目中合法的内部 GM / Debug 调试工具在确保与正式生产环境隔离的前提下免检）。
+   - 严格按照《核心代码审查准则》的 **[P0~P3]** 等级标准排布审查清单。
+   - 给出最终综合裁决与一句话中文总评。
+3. **语言强制要求**：所有任务入参、思考分析过程、综合汇报与最终报告必须 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, /（门禁裁决）/);
+});