@unifan/pi-unifan-zh 1.0.21 → 1.0.23

package/README.md

@@ -103,6 +103,7 @@ pi install npm:@unifan/pi-commit-zh
   - **`/review-config`** ⚙️: Standalone interactive settings command (persisted to `~/.pi/agent/pi-review.json`).
   - **`/review-lite`** ⭐: Directly audits current uncommitted changes without menu prompts.
   - **`/review-full`** 🚀: Full-scope deep review on all changes without redundant setup steps or checklists.
+  - **`/review-sync`** 🔄: Dual-session review relay loop (auto-detects role: dev fixes code & summarizes, reviewer validates diff without editing until approved). Aliases: `/review-flow`, `/review-relay`.
   - **`/end-review`** 🏁:
     - Automatically structures review findings into P0~P3 action items.
     - Seamlessly jumps back to the original working branch and pre-fills the editor with fix instructions!

package/README_zh.md

@@ -102,6 +102,10 @@ pi install npm:@unifan/pi-commit-zh
   - **`/review-config`** ⚙️ **（审查配置中心）**：独立配置命令，随时切换审查引擎模式与并发数（配置自动保存在 `~/.pi/agent/pi-review.json`）。
   - **`/review-lite`** ⭐ **（极速单兵审查）**：无需弹窗，直接对当前工作区未提交改动做闪电审查（零多余前置步骤）。
   - **`/review-full`** 🚀 **（全量深度审查）**：直接对当前工作区与分支全部改动展开深度审查，严禁输出任何待办清单和环境测试，秒级直达 P0~P3 审查发现！
+  - **`/review-sync`** 🔄 **（双会话审查接力闭环 · 自动识别角色）**：
+    - **开发会话粘贴审查意见**：自动识别为【代码修复】，逐项核验有道理的立即改代码，有误报的给出解释，并输出结构化整改报告。
+    - **审查会话粘贴修复报告**：自动识别为【只读复核】，结合 `git diff` 验证问题是否彻底解决且**严禁改动任何代码**，满意后输出 `✅【审核通过，可以提交代码】`！
+    - **别名支持**：`/review-flow`、`/review-relay`。
   - **`/end-review`** 🏁 **（结束审查并返回）**：
     - 自动将审查分支中发现的 `[P0~P3]` 缺陷与待办事项生成结构化中文报告。
     - 一键平滑跳回主开发会话分支，并自动在输入框回填 `根据上述代码审查发现进行修改修复`！

package/extensions/review/index.ts

@@ -201,47 +201,48 @@ type ReviewTarget =
 
 // 针对不同审查目标的中文提示词
 const LITE_REVIEW_PROMPT =
-	"【极速体检模式】请直接运行 `git diff` 快速排查当前未提交改动中的高危逻辑缺陷与安全漏洞。只抓严重级别最高的问题，快速输出审查结论。严禁输出任何步骤清单。所有输出必须使用纯正中文。";
+	"【极速体检模式】请首先运行 `git diff` 获取代码改动，并针对改动涉及的核心函数按需使用 `read` 查阅周边上下文（如所在完整方法实现与生命周期），快速排查高危逻辑崩溃与安全漏洞。结合上下文精准核验后直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。";
 
 const FULL_REVIEW_PROMPT =
-	"【全量深度审查模式】请对当前代码改动展开全方位的深度审查。直接运行 `git diff` 全量审阅代码改动，深入排查潜在的业务逻辑隐患、边界异常、并发安全、性能GC开销与架构合规性，直接给出按优先级排序的详尽审查清单。严禁输出任何步骤清单。所有输出必须使用纯正中文。";
+	"【全量深度审查模式】请对当前代码改动展开深度的上下文关联审查：首先运行 `git diff` 全量查阅改动，必须使用 `read` / `grep` 深入查阅改动方法所在的完整类定义、调用方契约、状态机与生命周期等关键上下文，深入排查业务逻辑隐患、边界异常、并发安全、性能GC开销与架构契约，结合完整代码脉络直接输出详尽审查清单。严禁输出任何步骤清单。所有输出必须使用纯正中文。";
 
 const UNCOMMITTED_PROMPT =
-	"请审查当前代码的所有改动（包含暂存区、未暂存区以及新增文件）。直接运行 `git diff` 与 `git status` 查看代码差异，并直接输出审查发现。严禁输出任何步骤清单或待办列表。所有输出必须使用纯正中文。";
+	"请审查当前代码的所有改动（包含暂存区、未暂存区以及新增文件）。先运行 `git diff` 与 `git status` 获取改动，再针对关键改动按需使用 `read` / `grep` 查阅周边上下文代码与调用链路，结合完整上下文直接输出审查发现。严禁输出任何步骤清单或待办列表。所有输出必须使用纯正中文。";
 
 const BASE_BRANCH_PROMPT_WITH_MERGE_BASE =
-	"请审查当前分支相对于基准分支 '{baseBranch}' 的改动（合并基准 commit: {mergeBaseSha}）。直接运行 `git diff {mergeBaseSha}` 查看改动并直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。";
+	"请审查当前分支相对于基准分支 '{baseBranch}' 的改动（合并基准 commit: {mergeBaseSha}）。运行 `git diff {mergeBaseSha}` 查看改动，并按需使用 `read` / `grep` 结合周边上下文深入分析，直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。";
 
 const BASE_BRANCH_PROMPT_FALLBACK =
-	"请审查当前分支相对于基准分支 '{branch}' 的改动。直接使用 `git merge-base` 与 `git diff` 查看改动并直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。";
+	"请审查当前分支相对于基准分支 '{branch}' 的改动。先使用 `git merge-base` 与 `git diff` 查看改动，并按需使用 `read` / `grep` 结合周边上下文深入分析，直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。";
 
 const COMMIT_PROMPT_WITH_TITLE =
-	'请审查提交 commit {sha} ("{title}") 引入的代码改动。直接运行 `git show {sha}` 查看差异并直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。';
+	'请审查提交 commit {sha} ("{title}") 引入的代码改动。运行 `git show {sha}` 查看差异，并按需使用 `read` / `grep` 结合周边上下文深入分析，直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。';
 
 const COMMIT_PROMPT =
-	"请审查提交 commit {sha} 引入的代码改动。直接运行 `git show {sha}` 查看差异并直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。";
+	"请审查提交 commit {sha} 引入的代码改动。运行 `git show {sha}` 查看差异，并按需使用 `read` / `grep` 结合周边上下文深入分析，直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。";
 
 const PULL_REQUEST_PROMPT =
-	'请审查 Pull Request #{prNumber} ("{title}") 相对于基准分支 \'{baseBranch}\' 的改动（合并基准 commit: {mergeBaseSha}）。直接运行 `git diff {mergeBaseSha}` 查看改动并直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。';
+	'请审查 Pull Request #{prNumber} ("{title}") 相对于基准分支 \'{baseBranch}\' 的改动（合并基准 commit: {mergeBaseSha}）。运行 `git diff {mergeBaseSha}` 查看改动，并按需使用 `read` / `grep` 结合周边上下文深入分析，直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。';
 
 const PULL_REQUEST_PROMPT_FALLBACK =
-	'请审查 Pull Request #{prNumber} ("{title}") 相对于基准分支 \'{baseBranch}\' 的改动。直接运行 `git diff` 查看改动并直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。';
+	'请审查 Pull Request #{prNumber} ("{title}") 相对于基准分支 \'{baseBranch}\' 的改动。先运行 `git diff` 查看改动，并按需使用 `read` / `grep` 结合周边上下文深入分析，直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。';
 
 const FOLDER_REVIEW_PROMPT =
-	"请对以下目录/文件路径的代码进行快照审查：{paths}。直接读取这些文件并直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。";
+	"请对以下目录/文件路径的代码进行快照审查：{paths}。直接读取这些文件并结合上下文直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。";
 
 // 权威的中文代码审查准则 (基于 Codex 准则精炼与本土化)
 const REVIEW_RUBRIC = `> 🚨【最高执行原则】：
 > 1. **全流程纯中文**：思维链（Thinking）、推理分析与最终报告必须 100% 使用纯正中文，严禁包含任何英文段落或英文小标题！
-> 2. **直接审查，拒绝繁琐步骤**：严禁打印任何工作流待办清单（Checklist）或环境测试命令！直接阅读代码改动并立即输出审查报告！
+> 2. **结合上下文，拒绝断章取义**：仅看 diff 的片段容易产生误判。请**先运行 \`git diff\` 锁定变动，再针对改动涉及的关键方法、类生命周期或调用方，按需使用 \`read\` 或 \`grep\` 查阅必要的上下文代码**（精准核验，切勿无目的漫游遍历），结合真实完整的业务上下文做出准确裁决！
+> 3. **直接审查，拒绝繁琐步骤**：严禁打印任何工作流待办清单（Checklist）或环境测试命令！
 
 # 核心代码审查准则
 
-你是一名资深技术专家，正在审查提交的代码改动。请直击本质，拦截真实缺陷：
+你是一名资深技术专家，正在审查提交的代码改动。请结合完整代码上下文，直击本质，拦截真实缺陷：
 
 ## 重点排查范围
-1. **代码正确性与边界**：业务逻辑漏洞、空指针/未定义引用、边界越界、生命周期异常、未捕获的运行时崩溃。
-2. **并发与状态安全**：竞态条件、死锁隐患、异步缺少等待、脏状态残留。
+1. **代码正确性与边界**：结合函数整体逻辑与调用方，排查业务漏洞、空指针/未定义引用、边界越界、生命周期异常、未捕获的运行时崩溃。
+2. **并发与状态安全**：结合上下文状态机与生命周期，排查竞态条件、死锁隐患、异步缺少等待、脏状态残留。
 3. **性能与内存开销**：高频主循环内的大量堆内存分配 (GC 压力)、不必要的深拷贝、高复杂度算法、资源句柄泄露。
 4. **防御与输入校验**：外部未受信任输入未做严格参数化或校验（SQL注入、路径穿越、越权等）。
 5. **只关注本次改动**：严禁将改动前既有的历史代码当作本次改动的缺陷。
@@ -249,7 +250,7 @@ const REVIEW_RUBRIC = `> 🚨【最高执行原则】：
 ## 严格过滤误报
 - ❌ 严禁提出吹毛求疵、纯属个人审美的风格建议。
 - ❌ 严禁提出 Linter / 类型检查器会自动捕获的浅层格式建议。
-- ❌ 严禁基于纯主观假设进行无端猜测，每条问题必须有明确代码事实证据。
+- ❌ 严禁脱离上下文进行无端猜测，每条问题必须有明确的代码与上下文事实证据。
 
 ## 缺陷等级标记 [P0~P3]
 - **[P0 - 致命阻塞]** 崩溃、死锁、数据损坏、关键功能不可用，必须阻断合并
@@ -261,8 +262,8 @@ const REVIEW_RUBRIC = `> 🚨【最高执行原则】：
 ### 审查发现清单
 每个问题按以下格式列出（若无存活缺陷，明确输出：\`未发现存活的代码缺陷，代码质量良好，建议合并。\`）：
 - **[P0|P1|P2|P3] 简短标题**：\`文件路径:行号\`
-  - **缺陷说明**：简明说明后果与触发场景。
-  - **代码证据**：引用 1~3 行关键代码。
+  - **缺陷说明**：结合上下文简明说明后果与触发场景。
+  - **代码证据**：引用关键代码及上下文。
   - **修改建议**：给出最小化修复思路或代码替换块（可使用 \`\`\`suggestion）。
 
 ### 综合裁决
@@ -1352,4 +1353,161 @@ export default function reviewExtension(pi: ExtensionAPI) {
 			}
 		},
 	});
+
+	// 针对【代码开发会话·修复方】的提示词模板
+	const DEV_FIX_PROMPT_TEMPLATE = `> 🚨【核心语言指令】：全流程 100% 纯正中文！
+
+# 任务指令：针对代码审查（Code Review）意见进行核验、修复与总结
+
+外部审查专家提出了以下代码审查意见与发现：
+---
+{content}
+---
+
+## 执行指示（请严格遵守）：
+1. **逐条仔细核验**：结合当前代码上下文仔细审阅上述每一项发现。
+2. **合理的问题立即修复**：
+   - 确实存在逻辑缺陷、边界隐患、空指针或并发风险的，请立即使用代码编辑工具进行修复。
+   - 确保修改后的代码语法正确、逻辑自洽且未引入新问题。
+3. **有争议或误报的问题有理有据解释**：
+   - 若某些意见属于误报，或由于特定业务场景与底层架构已有防护，请明确给出技术解释与考量依据。
+4. **输出结构化的《整改与回复报告》（供用户直接复制回审查会话复核）**：
+   在最终回复末尾，严格按以下格式进行总结：
+
+### 🛠️ 代码审查整改与回复报告
+- **[已修复] [问题标题]**
+  - **修改说明**：说明修改了哪些文件与函数，采取了什么具体修复方案。
+- **[无需修改/已说明] [问题标题]**
+  - **原因解释**：说明为什么无需修改或为什么既有实现是合理的。
+
+### 📝 本次整改提交摘要
+一句话总结本次修复涉及的模块与核心变动。
+`;
+
+	// 针对【审查复核会话·审查方】的提示词模板
+	const REVIEWER_RECHECK_PROMPT_TEMPLATE = `> 🚨【核心语言指令】：全流程 100% 纯正中文！
+> 🚨【关键操作禁令】：你当前处于【复核审查官】模式，**严禁修改任何代码！严禁调用任何写入/修改工具！** 你的唯一职责是验证代码是否已被正确修复！
+
+# 任务指令：复核开发者的代码修复与整改说明
+
+开发者提交了针对此前代码审查的整改回复：
+---
+{content}
+---
+
+## 执行指示（请严格遵守）：
+1. **查阅最新改动**：立即运行 \`git diff\` 查看最新代码改动，并按需使用 \`read\` 查阅修改后的完整方法与类上下文。
+2. **逐项对照复核**：
+   - 对照上述整改说明与最新代码，严格核验此前的问题是否真正被彻底解决。
+   - 检查本次修复是否引入了新的逻辑缺陷、空指针、状态残留或并发风险。
+   - 评估开发者对于“无需修改”项的解释是否合理、论据是否充分。
+3. **严禁修改任何源码文件**（只读复核）。
+4. **给出最终复核裁决**：
+   - **若所有关键问题均已圆满解决，代码质量达标**：
+     请在报告末尾明确输出绿色的终审裁决大标题：
+     ## ✅【审核通过，可以提交代码】
+     并附上一句话总评。
+   - **若仍有未解决或新引入的问题**：
+     请在报告末尾明确输出：
+     ## ❌【仍有阻塞问题需继续修改】
+     列出具体的剩余问题、所在文件行号与修改要求，以便用户再次复制给开发者继续修复。
+`;
+
+	function detectReviewSyncRole(
+		entries: any[],
+		content: string,
+		flag?: "fix" | "check",
+	): "fix" | "check" {
+		if (flag) return flag;
+
+		// 1. 会话历史特征判断：当前会话是否曾作为审查分支或执行过 review
+		const hasReviewHistory = entries.some((e) => {
+			if (e.type === "custom" && e.customType === REVIEW_STATE_TYPE) return true;
+			if (e.type === "message" && e.message?.content) {
+				const text = JSON.stringify(e.message.content);
+				return text.includes("核心代码审查准则") || text.includes("正在启动代码审查");
+			}
+			return false;
+		});
+
+		if (hasReviewHistory) {
+			return "check";
+		}
+
+		// 2. 粘贴内容特征判断：
+		// 审查发现一般包含 [P0]、[P1]、"审查发现"、"缺陷说明"、"综合裁决"
+		// 修复汇报一般包含 "已修复"、"整改与回复"、"修改说明"、"本次整改"
+		const hasReviewMarkers = /\[P[0-3]\]|审查发现|缺陷说明|综合裁决|代码审查准则/i.test(content);
+		const hasFixMarkers = /已修复|整改与回复|修改说明|本次整改|无需修改/i.test(content);
+
+		if (hasFixMarkers && !hasReviewMarkers) {
+			return "check";
+		}
+		if (hasReviewMarkers) {
+			return "fix";
+		}
+
+		return "fix";
+	}
+
+	async function handleReviewSync(rawArgs: string | undefined, ctx: ExtensionCommandContext) {
+		if (!ctx.hasUI) {
+			ctx.ui.notify("review-sync 需要交互式终端环境", "error");
+			return;
+		}
+
+		let text = rawArgs?.trim() || "";
+		let forcedRole: "fix" | "check" | undefined = undefined;
+
+		if (text.startsWith("--fix ")) {
+			forcedRole = "fix";
+			text = text.slice(6).trim();
+		} else if (text.startsWith("--check ") || text.startsWith("--review ")) {
+			forcedRole = "check";
+			text = text.replace(/^--(?:check|review)\s+/, "").trim();
+		}
+
+		if (!text) {
+			const input = await ctx.ui.editor(
+				"请粘贴 Review 审查发现清单（发给开发会话修复）或 修复整改说明（发给审查会话复核）：",
+				"",
+			);
+			if (!input?.trim()) {
+				ctx.ui.notify("已取消操作", "info");
+				return;
+			}
+			text = input.trim();
+		}
+
+		const entries = ctx.sessionManager.getEntries();
+		const role = detectReviewSyncRole(entries, text, forcedRole);
+
+		if (role === "fix") {
+			ctx.ui.notify("🤖 自动识别为【代码开发会话】：正在逐项核验并执行代码修复...", "info");
+			const prompt = DEV_FIX_PROMPT_TEMPLATE.replace("{content}", text);
+			pi.sendUserMessage(prompt);
+		} else {
+			ctx.ui.notify("🔍 自动识别为【审查复核会话】：正在对照改动进行只读复核 (严禁改动代码)...", "info");
+			const prompt = REVIEWER_RECHECK_PROMPT_TEMPLATE.replace("{content}", text);
+			pi.sendUserMessage(prompt);
+		}
+	}
+
+	// 注册双会话审查接力闭环命令：/review-sync
+	pi.registerCommand("review-sync", {
+		description: "双会话审查接力闭环 (自动识别：开发会话自动改代码并总结，审查会话只读复核直到可提交)",
+		handler: handleReviewSync,
+	});
+
+	// 别名：/review-flow
+	pi.registerCommand("review-flow", {
+		description: "双会话审查接力闭环 (/review-sync 别名)",
+		handler: handleReviewSync,
+	});
+
+	// 别名：/review-relay
+	pi.registerCommand("review-relay", {
+		description: "双会话审查接力闭环 (/review-sync 别名)",
+		handler: handleReviewSync,
+	});
 }

package/extensions/review/package.json

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@unifan/pi-unifan-zh",
-  "version": "1.0.21",
+  "version": "1.0.23",
   "description": "Pi Coding Agent 中文扩展全家桶与独立插件集",
   "type": "module",
   "main": "extensions/sessions/index.ts",