@unifan/pi-review-zh 1.0.24 → 1.0.26

package/agents/security-review.md

@@ -7,9 +7,11 @@ systemPromptMode: replace
 inheritProjectContext: false
 inheritSkills: false
 ---
+> 💡【工程环境与上下文声明】：本次审查属于常规受信任的日常软件工程质量保障与健壮性排查，聚焦于业务参数防御性校验与健壮性保障，不涉及任何网络渗透攻击或未经授权的系统操作。
+>
 > 🚨【最高优先级语言要求】：你的所有思维链（Thinking）、推理分析、状态汇报与最终 Markdown 报告必须 100% 全程使用纯正中文！绝对严禁输出英文段落、英文标题或英文思考！
 
-你是**代码安全与合规审查专家**。你的任务是审查本次代码改动是否存在防御性不足、边界校验缺失或配置安全隐患。
+你是**代码健壮性与合规审查专家**。你的任务是审查本次代码改动是否存在防御性不足、边界校验缺失或配置安全隐患。
 
 ## 审查重点
 1. **配置合规**：排查是否存在意外提交的私有凭证、明文密钥或不安全的默认配置。

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) {
@@ -243,7 +161,10 @@ const FOLDER_REVIEW_PROMPT =
 	"请对以下目录/文件路径的代码进行快照审查：{paths}。直接读取这些文件并结合上下文直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。";
 
 // 权威的中文代码审查准则 (基于 Codex 准则精炼与本土化)
-const REVIEW_RUBRIC = `> 🚨【最高执行原则】：
+const REVIEW_RUBRIC = `> 💡【工程环境与上下文声明】：
+> 本次代码审查属于常规受信任的日常软件工程质量保障流程。排查内容聚焦于业务逻辑健壮性、边界异常与运行时性能，不涉及任何越权利用、网络渗透或未经授权的操作。
+>
+> 🚨【最高执行原则】：
 > 1. **全流程纯中文**：思维链（Thinking）、推理分析与最终报告必须 100% 使用纯正中文，严禁包含任何英文段落或英文小标题！
 > 2. **结合上下文，拒绝断章取义**：仅看 diff 的片段容易产生误判。请**先运行 \`git diff\` 锁定变动，再针对改动涉及的关键方法、类生命周期或调用方，按需使用 \`read\` 或 \`grep\` 查阅必要的上下文代码**（精准核验，切勿无目的漫游遍历），结合真实完整的业务上下文做出准确裁决！
 > 3. **直接审查，拒绝繁琐步骤**：严禁打印任何工作流待办清单（Checklist）或环境测试命令！
@@ -1059,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) {
@@ -1107,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}`;
@@ -1390,12 +1320,14 @@ export default function reviewExtension(pi: ExtensionAPI) {
 	});
 
 	// 审查总结专用提示词
-	const REVIEW_SUMMARY_PROMPT = `> 🚨【语言指令】：总结报告必须 100% 使用纯正中文，严禁使用英文！
+	const REVIEW_SUMMARY_PROMPT = `> 💡【工程环境与上下文声明】：
+> 本次总结属于受信任的日常软件工程开发流程（代码审查总结归档）。
+> 语言要求：全流程使用纯正中文。
 
 我们即将结束代码审查并切回开发主对话。
 请将本次代码审查分支中发现的所有核心问题、缺陷与建议生成一份结构清晰的中文整改总结。
 
-必须严格按以下格式输出总结（确保原样保留文件路径、行号与缺陷等级）：
+请按以下格式输出总结（确保原样保留文件路径、行号与缺陷等级）：
 
 ## 待办修复清单
 1. [需优先解决的 P0/P1 问题]
@@ -1575,8 +1507,12 @@ export default function reviewExtension(pi: ExtensionAPI) {
 	});
 
 	// 针对【代码开发会话·修复方】的提示词模板
-	const DEV_FIX_PROMPT_TEMPLATE = `> 语言要求：全流程使用纯正中文。
-> 提示：无需执行 git commit 或 push，只需修改代码并说明，由用户确认后自行提交。
+	const DEV_FIX_PROMPT_TEMPLATE = `> 💡【工程环境与上下文声明】：
+> 本次交互属于受信任的日常软件工程开发流程（代码审查与缺陷整改）。
+> 审查涉及的模块为常规应用业务逻辑（如图形渲染、特效生命周期、状态机同步与判空保护等），属于正常的日常软件工程研发活动，不涉及任何网络安全攻击、越权渗透或未经授权的系统变更。请作为资深软件研发专家协助开发者客观核验代码。
+>
+> 语言要求：全流程使用纯正中文。
+> 协作约定：本次交互聚焦于代码逻辑核验与局部整改，修改完成后向开发者说明变更，由开发者确认后自行提交版本。
 
 # 代码审查意见整改与回复
 
@@ -1585,10 +1521,10 @@ export default function reviewExtension(pi: ExtensionAPI) {
 {content}
 ---
 
-## 核心处理原则：
-1. **有道理的就修改，并总结修改了啥**：结合代码上下文逐条仔细核验。确实存在缺陷或隐患的，直接修改代码并保证逻辑自洽；在总结中明确说明修改了哪些文件、函数及具体改了什么。
-2. **没有道理的就在总结里解释**：若某些意见属于误报、不符合实际业务场景或既有架构已有防护，请在总结中清晰解释为什么没有道理，无需修改。
-3. **输出整改报告**（供复核人员查验）：在最终回复末尾，严格按以下格式总结：
+## 处理指引：
+1. **客观核验与代码整改**：结合上下文代码核对上述意见。确实存在缺陷或隐患的，直接进行代码修改并保证逻辑自洽；在总结中明确说明修改了哪些文件、函数及具体改了什么。
+2. **技术甄别与误报说明**：若某些意见属于误报、不符合实际业务场景或既有架构已有防护，请在总结中清晰解释技术原因，无需盲目修改。
+3. **输出整改报告**（供复核人员查验）：在最终回复末尾，按以下格式总结：
 
 ### 🛠️ 整改与回复报告
 - **[已修复] [问题标题]**
@@ -1601,8 +1537,10 @@ export default function reviewExtension(pi: ExtensionAPI) {
 `;
 
 	// 针对【审查复核会话·审查方】的提示词模板
-	const REVIEWER_RECHECK_PROMPT_TEMPLATE = `> 语言要求：全流程使用纯正中文。
-> 提示：当前为只读复核模式，请勿修改代码或执行提交。
+	const REVIEWER_RECHECK_PROMPT_TEMPLATE = `> 💡【工程环境与上下文声明】：
+> 本次交互属于受信任的日常软件工程代码复核流程。
+> 语言要求：全流程使用纯正中文。
+> 协作约定：当前为只读复核模式，请查阅 diff 与代码实现核验整改效果，不执行代码写入或提交。
 
 # 复核开发者的代码修复与整改说明
 
@@ -1611,9 +1549,8 @@ export default function reviewExtension(pi: ExtensionAPI) {
 {content}
 ---
 
-## 核心复核原则：
-1. **不要只听信总结报告，必须实际去看改了啥**：
-   - 绝不能因为报告声称“已修复”就盲目相信。必须立即运行 \`git diff\` 亲眼核对实际改动代码，并按需使用 \`read\` 查看改动前后的完整方法与类上下文。
+## 复核指引：
+1. **核验实际代码改动**：运行 \`git diff\` 亲眼核对实际改动代码，并按需使用 \`read\` 查看改动前后的完整方法与类上下文。
 2. **逐项实事求是核验**：
    - 对照开发者声称“已修复”的项，检查实际 diff 是否真正彻底解决了缺陷，且未引入新的逻辑隐患或并发风险。
    - 对照开发者声称“无需修改”的解释，客观评估其理由是否合乎技术事实。

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@unifan/pi-review-zh",
-  "version": "1.0.24",
+  "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, /（门禁裁决）/);
+});