@unifan/pi-review-zh 1.0.14 → 1.0.16

package/index.ts

@@ -200,73 +200,74 @@ type ReviewTarget =
 	| { type: "folder"; paths: string[] };
 
 // 针对不同审查目标的中文提示词
+const LITE_REVIEW_PROMPT =
+	"【极速体检模式】请直接运行 `git diff` 快速排查当前未提交改动中的高危逻辑缺陷与安全漏洞。只抓严重级别最高的问题，快速输出审查结论。严禁输出任何步骤清单。所有输出必须使用纯正中文。";
+
+const FULL_REVIEW_PROMPT =
+	"【全量深度审查模式】请对当前代码改动展开全方位的深度审查。直接运行 `git diff` 全量审阅代码改动，深入排查潜在的业务逻辑隐患、边界异常、并发安全、性能GC开销与架构合规性，直接给出按优先级排序的详尽审查清单。严禁输出任何步骤清单。所有输出必须使用纯正中文。";
+
 const UNCOMMITTED_PROMPT =
-	"请审查当前代码的所有改动（包含暂存区、未暂存区以及新增文件）。请直接运行 `git diff`、`git status` 及必要的文件读取工具深入审查，并给出按优先级排序的具体审查发现。所有输出必须使用纯正中文。";
+	"请审查当前代码的所有改动（包含暂存区、未暂存区以及新增文件）。直接运行 `git diff` 与 `git status` 查看代码差异，并直接输出审查发现。严禁输出任何步骤清单或待办列表。所有输出必须使用纯正中文。";
 
 const BASE_BRANCH_PROMPT_WITH_MERGE_BASE =
-	"请审查当前分支相对于基准分支 '{baseBranch}' 的改动代码。两者的合并基准 commit 为 {mergeBaseSha}。请直接运行 `git diff {mergeBaseSha}` 检查本次改动并给出按优先级排序的具体审查发现。所有输出必须使用纯正中文。";
+	"请审查当前分支相对于基准分支 '{baseBranch}' 的改动（合并基准 commit: {mergeBaseSha}）。直接运行 `git diff {mergeBaseSha}` 查看改动并直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。";
 
 const BASE_BRANCH_PROMPT_FALLBACK =
-	"请审查当前分支相对于基准分支 '{branch}' 的改动代码。首先使用 `git merge-base HEAD \"$(git rev-parse --abbrev-ref \"{branch}@{upstream}\")\"` 找到合并基准，再通过 `git diff` 检查改动并给出具体审查发现。所有输出必须使用纯正中文。";
+	"请审查当前分支相对于基准分支 '{branch}' 的改动。直接使用 `git merge-base` 与 `git diff` 查看改动并直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。";
 
 const COMMIT_PROMPT_WITH_TITLE =
-	'请审查提交 commit {sha} ("{title}") 引入的代码改动。直接运行 `git show {sha}` 检查差异并给出按优先级排序的具体审查发现。所有输出必须使用纯正中文。';
+	'请审查提交 commit {sha} ("{title}") 引入的代码改动。直接运行 `git show {sha}` 查看差异并直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。';
 
 const COMMIT_PROMPT =
-	"请审查提交 commit {sha} 引入的代码改动。直接运行 `git show {sha}` 检查差异并给出按优先级排序的具体审查发现。所有输出必须使用纯正中文。";
+	"请审查提交 commit {sha} 引入的代码改动。直接运行 `git show {sha}` 查看差异并直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。";
 
 const PULL_REQUEST_PROMPT =
-	'请审查 Pull Request #{prNumber} ("{title}") 相对于基准分支 \'{baseBranch}\' 的改动代码。两者的合并基准 commit 为 {mergeBaseSha}。请直接运行 `git diff {mergeBaseSha}` 检查改动并给出按优先级排序的具体审查发现。所有输出必须使用纯正中文。';
+	'请审查 Pull Request #{prNumber} ("{title}") 相对于基准分支 \'{baseBranch}\' 的改动（合并基准 commit: {mergeBaseSha}）。直接运行 `git diff {mergeBaseSha}` 查看改动并直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。';
 
 const PULL_REQUEST_PROMPT_FALLBACK =
-	'请审查 Pull Request #{prNumber} ("{title}") 相对于基准分支 \'{baseBranch}\' 的改动代码。首先寻找当前分支与 {baseBranch} 的合并基准 (例如 `git merge-base HEAD {baseBranch}`)，再通过 `git diff` 检查改动并给出具体审查发现。所有输出必须使用纯正中文。';
+	'请审查 Pull Request #{prNumber} ("{title}") 相对于基准分支 \'{baseBranch}\' 的改动。直接运行 `git diff` 查看改动并直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。';
 
 const FOLDER_REVIEW_PROMPT =
-	"请对以下目录/文件路径的代码进行快照审查：{paths}。注意这是全量快照审查（非 diff 对比）。请直接读取这些文件并给出按优先级排序的具体审查发现。所有输出必须使用纯正中文。";
+	"请对以下目录/文件路径的代码进行快照审查：{paths}。直接读取这些文件并直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。";
 
 // 权威的中文代码审查准则 (基于 Codex 准则精炼与本土化)
-const REVIEW_RUBRIC = `> 🚨【最高优先级语言要求】：你的所有思维链（Thinking）、推理分析、工具调用说明、审查发现与最终裁决结论必须 100% 全程使用纯正中文！绝对严禁输出任何英文段落、英文标题、英文思考或英文总结！
+const REVIEW_RUBRIC = `> 🚨【最高执行原则】：
+> 1. **全流程纯中文**：思维链（Thinking）、推理分析与最终报告必须 100% 使用纯正中文，严禁包含任何英文段落或英文小标题！
+> 2. **直接审查，拒绝繁琐步骤**：严禁打印任何工作流待办清单（Checklist）或环境测试命令！直接阅读代码改动并立即输出审查报告！
 
-# 核心代码审查准则（资深工程师视角）
+# 核心代码审查准则
 
-你正在作为一名资深技术专家对另一位工程师提交的代码改动进行严格的代码审查。你的目标是帮作者把关并拦截真实风险，给出清晰、可落地、带事实证据的纯中文审查意见。
+你是一名资深技术专家，正在审查提交的代码改动。请直击本质，拦截真实缺陷：
 
-## 重点排查范围（排查什么）
-1. **代码正确性与边界处理**：逻辑缺陷、空指针/未定义引用、边界越界、生命周期异常、未捕获的运行时异常。
-2. **并发与状态安全**：竞态条件、死锁隐患、异步缺少等待、未处理的取消或中断、脏状态残留。
-3. **性能与内存开销**：高频主循环内的大量内存分配 (GC 压力)、不必要的深拷贝、高复杂度算法、资源句柄或网络连接未释放。
-4. **代码健壮性与外部合规**：
-   - 严禁信任外部用户输入（必须检查未参数化的 SQL、路径穿越、未校验的 URL 重定向、危险的反序列化等）。
-   - 报错与异常必须检查稳定的错误码或类型，严禁用脆弱的错误文本字符串做业务分支判断。
-5. **本次改动引入的缺陷**：只审查本次改动实际引入的问题，严禁把改动前既有的历史代码或未改动代码归咎为缺陷。
+## 重点排查范围
+1. **代码正确性与边界**：业务逻辑漏洞、空指针/未定义引用、边界越界、生命周期异常、未捕获的运行时崩溃。
+2. **并发与状态安全**：竞态条件、死锁隐患、异步缺少等待、脏状态残留。
+3. **性能与内存开销**：高频主循环内的大量堆内存分配 (GC 压力)、不必要的深拷贝、高复杂度算法、资源句柄泄露。
+4. **防御与输入校验**：外部未受信任输入未做严格参数化或校验（SQL注入、路径穿越、越权等）。
+5. **只关注本次改动**：严禁将改动前既有的历史代码当作本次改动的缺陷。
 
-## 严格过滤误报（不排查什么）
-- ❌ 严禁提出吹毛求疵、纯属个人审美的废话风格建议（若无明确项目规范强制要求）。
-- ❌ 严禁提出 Linter、类型检查器、编译构建会自动捕获的浅层格式建议。
-- ❌ 严禁基于未证实的纯主观假设进行无端猜测，每一条问题必须有明确的代码事实证据。
+## 严格过滤误报
+- ❌ 严禁提出吹毛求疵、纯属个人审美的风格建议。
+- ❌ 严禁提出 Linter / 类型检查器会自动捕获的浅层格式建议。
+- ❌ 严禁基于纯主观假设进行无端猜测，每条问题必须有明确代码事实证据。
 
-## 缺陷严重等级标记
-每一条审查发现必须在标题中清晰标注严重等级：
-- **[P0 - 致命阻塞]** 导致系统崩溃、死锁、数据损坏、关键功能完全不可用或高危漏洞，必须立即修复，阻断合并。
-- **[P1 - 紧急待修]** 明确的逻辑缺陷、高概率边界异常、严重性能退化或破坏公共接口契约，应在本次合并前修复。
-- **[P2 - 普通建议]** 局部的健壮性隐患、轻度可读性或次要设计问题，建议在后续优化。
-- **[P3 - 细节优化]** 极轻微的细节优化建议，不影响业务。
-
-## 输出格式（所有内容必须使用纯正中文）
-请按以下规范输出审查报告：
+## 缺陷等级标记 [P0~P3]
+- **[P0 - 致命阻塞]** 崩溃、死锁、数据损坏、关键功能不可用，必须阻断合并
+- **[P1 - 紧急待修]** 明确逻辑缺陷、边界异常、严重性能退化，合并前需修复
+- **[P2 - 普通建议]** 局部的健壮性隐患、轻度设计问题
+- **[P3 - 细节优化]** 细节优化与可读性
 
+## 输出格式规范（直接输出，无多余寒暄与准备动作）
 ### 审查发现清单
-每个问题按以下格式列出（若完全无缺陷，请明确输出：\`未发现存活的代码缺陷，代码质量良好，建议合并。\`）：
+每个问题按以下格式列出（若无存活缺陷，明确输出：\`未发现存活的代码缺陷，代码质量良好，建议合并。\`）：
 - **[P0|P1|P2|P3] 简短标题**：\`文件路径:行号\`
-  - **缺陷说明**：简明扼要说明该问题会导致什么后果以及在何种场景下被触发。
-  - **代码证据**：引用 1~3 行具体代码或调用链路。
-  - **修改建议**：给出最小化的修复思路或直接附带精准的代码替换块（可使用 \`\`\`suggestion 代码块）。
+  - **缺陷说明**：简明说明后果与触发场景。
+  - **代码证据**：引用 1~3 行关键代码。
+  - **修改建议**：给出最小化修复思路或代码替换块（可使用 \`\`\`suggestion）。
 
 ### 综合裁决
 - **最终结论**：\`通过\` 或 \`需要修改 (存在 P0/P1 阻塞问题)\`
-- **总结说明**：一句话中文总评。
-
-> 🚨【语言警示】：全篇所有内容必须 100% 使用纯正中文，严禁包含任何未翻译的英文短语或小标题！`;
+- **总结说明**：一句话中文总评。`;
 
 /**
  * 尝试加载项目本地的专属审查准则文件 (REVIEW_GUIDELINES.md 或 AGENTS.md / CLAUDE.md)
@@ -430,9 +431,15 @@ async function getDefaultBranch(pi: ExtensionAPI): Promise<string> {
 	return "main";
 }
 
-async function buildReviewPrompt(pi: ExtensionAPI, target: ReviewTarget): Promise<string> {
+async function buildReviewPrompt(
+	pi: ExtensionAPI,
+	target: ReviewTarget,
+	reviewStyle: "lite" | "full" | "standard" = "standard",
+): Promise<string> {
 	switch (target.type) {
 		case "uncommitted":
+			if (reviewStyle === "lite") return LITE_REVIEW_PROMPT;
+			if (reviewStyle === "full") return FULL_REVIEW_PROMPT;
 			return UNCOMMITTED_PROMPT;
 
 		case "baseBranch": {
@@ -921,6 +928,7 @@ export default function reviewExtension(pi: ExtensionAPI) {
 		target: ReviewTarget,
 		useFreshSession: boolean,
 		runtimeSettings?: Partial<ReviewSettings>,
+		reviewStyle: "lite" | "full" | "standard" = "standard",
 	): Promise<void> {
 		if (reviewOriginId) {
 			ctx.ui.notify("当前已有正在进行的审查会话。请先使用 /end-review 结束。", "warning");
@@ -969,29 +977,26 @@ export default function reviewExtension(pi: ExtensionAPI) {
 			pi.appendEntry(REVIEW_STATE_TYPE, { active: true, originId: lockedOriginId });
 		}
 
-		const prompt = await buildReviewPrompt(pi, target);
+		const prompt = await buildReviewPrompt(pi, target, reviewStyle);
 		const hint = getUserFacingHint(target);
 		const projectGuidelines = await loadProjectReviewGuidelines(ctx.cwd);
 
-		const MANDATORY_CHINESE_HEADER = `> 🚨【核心语言指令 - 最高优先级】\n> 你的**所有思维链（Thinking）、中间分析过程、代码排查笔记、审查发现与最终裁决结论**必须 100% 全程使用纯正中文进行思考与撰写！绝对禁止输出任何英文思考段落、英文标题或英文总结！`;
-
-		let fullPrompt = `${MANDATORY_CHINESE_HEADER}\n\n${REVIEW_RUBRIC}`;
+		let fullPrompt = REVIEW_RUBRIC;
 
 		if (settings.mode === "subagents") {
 			fullPrompt += `\n\n---\n\n${buildSubagentOrchestrationPrompt(settings.concurrency, settings.gateEnabled)}`;
 		}
 
-		fullPrompt += `\n\n---\n\n## 本次审查目标与任务指示\n\n${prompt}`;
+		fullPrompt += `\n\n---\n\n## 本次审查任务指示\n\n${prompt}`;
 
 		if (projectGuidelines) {
 			fullPrompt += `\n\n## 本项目附加规范指南\n\n${projectGuidelines}`;
 		}
 
-		fullPrompt += `\n\n> 🚨【最终语言校验】：请再次检查你的输出，确保通篇 100% 均为流畅纯正的中文，绝无任何英文说明！`;
-
+		const styleTag = reviewStyle === "lite" ? "[极速]" : reviewStyle === "full" ? "[全量]" : "";
 		const modeLabel = settings.mode === "single" ? "单模型模式" : `多Subagent并发(${settings.concurrency}专家)`;
 		const modeHint = useFreshSession ? " (独立审查分支)" : "";
-		ctx.ui.notify(`正在启动代码审查: ${hint}${modeHint} [${modeLabel}]`, "info");
+		ctx.ui.notify(`正在启动代码审查 ${styleTag}: ${hint}${modeHint} [${modeLabel}]`, "info");
 
 		pi.sendUserMessage(fullPrompt);
 	}
@@ -1191,7 +1196,44 @@ export default function reviewExtension(pi: ExtensionAPI) {
 				return;
 			}
 
-			await executeReview(ctx, { type: "uncommitted" }, false);
+			await executeReview(ctx, { type: "uncommitted" }, false, undefined, "lite");
+		},
+	});
+
+	// 全量深度审查命令：/review-full
+	pi.registerCommand("review-full", {
+		description: "全量深度代码审查 (直接对工作区与分支全部改动展开深度审查，无多余前置步骤与待办清单)",
+		handler: async (args, ctx) => {
+			if (!ctx.hasUI) {
+				ctx.ui.notify("代码审查需要交互式终端环境", "error");
+				return;
+			}
+
+			if (reviewOriginId) {
+				ctx.ui.notify("当前已有正在进行的审查。请先输入 /end-review 完成审查并返回。", "warning");
+				return;
+			}
+
+			const { code } = await pi.exec("git", ["rev-parse", "--git-dir"]);
+			if (code !== 0) {
+				ctx.ui.notify("当前目录不是有效的 Git 仓库", "error");
+				return;
+			}
+
+			const { target, settingsOverride } = parseArgs(args);
+			let finalTarget = target && target.type !== "pr" ? target : null;
+
+			if (!finalTarget) {
+				const defaultType = await getSmartDefault();
+				if (defaultType === "baseBranch") {
+					const defaultBranch = await getDefaultBranch(pi);
+					finalTarget = { type: "baseBranch", branch: defaultBranch };
+				} else {
+					finalTarget = { type: "uncommitted" };
+				}
+			}
+
+			await executeReview(ctx, finalTarget, false, settingsOverride, "full");
 		},
 	});
 

package/package.json

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