dsh-vscode-mode 0.23.1 → 0.23.2

package/README.md

@@ -521,6 +521,7 @@ DSH 每次启动监听随机端口，`http://127.0.0.1:<port>` 就是浏览器 o
 
 官方 `dsh plugin` 方式，三选一。下面以 `web` profile 为例；Desktop 或其他命名
 profile 请替换为实际目标，不要把示例目录当成当前运行 profile。升级前可阅读
+**v0.23.2** 修复 Keep 在非活跃端（桌面端）点了没反应，共两处成因。①权限：`edrv.review.decide` 原先一律要求本端活跃会话，而同端详情接口的 `capability.keep` 只按审查库推导，于是会话在另一实例时按钮亮着、点击后抛错并被兜底成 HTTP 500「DSH 请求失败（HTTP 500）」。现按写入副作用区分——官方记录 Keep 只写共享审查库（`BEGIN IMMEDIATE` + operationId 幂等 + `id/seq/revision/current/decision` CAS，库为每工作区 SQLite 且 `WAL + busy_timeout=2000`），冷端放行；官方 Undo 需校验会话证据并写回源文件、待审投影条目写本机待审 JSON（无跨进程锁），两者仍限本端活跃会话，并返回可读的跨端原因而非 500。详情响应新增 `capability.live`，冷端差异栏禁用 Undo 并给出跨端说明、Keep 提示补一句「写入共享审查库，仍可执行」，批量预检对冷端待审条目给出准确原因。②选中目标：已结算（`oldText === newText`，累计净变化为零）的待审投影 `capability` 恒为不可 Keep，但单文件通道把它排在官方记录之前、客户端取 `items[0]`，同路径官方待审行又被历史过滤掉，于是差异栏「待审 0」、Keep 禁用、点击无反应。现 `pendingPageOf()` 在交给官方审查通道前剔除已结算投影，官方待审记录重新成为审查目标。另外详情未加载时点 Keep 不再静默返回，改为可读提示；打开文件时若某条官方记录的改动已全部不在当前文件中（无可定位差异且缓冲区干净），自动按 Keep 结案（只标记不写盘），差异栏不再显示「本记录 N 处改动均已不在当前文件中」这类失效叙事；官方元数据首次载入期间差异栏显示 `—` 而不是按历史记录报数，避免重启瞬间闪出几十条假待审。验证：typecheck 0 error、目标单测 57/57、`pnpm build` 通过，并用真实工作区审查库探针确认差异栏首条已回到官方记录且 `capability={keep:true,undo:true,live:true}`；桌面端 GUI 点按验收因 App 重启后恢复的是另一个会话而未完成，缺口保留，见 [0.23.2 发布说明](docs/release-0.23.2.md)。以下保留历史版本事实：
 **v0.23.1** 修复「Keep All / Undo All」在工作区待审超过一页时必定中止：批量按钮原先复用官方审查的分页接口逐页取待审条目，并把每页返回的展示用计数当成全量总数校验，因此官方待审超过 50 条（本仓库常态约 150 条）时会先报「待审记录在分页期间变化」或最终报「待审记录数量不完整」，实际一条都不会被决策（已完成 0 项）。现在批量操作改走一次性完整快照——`edrv.review.list` 新增 `all: true` 参数，服务端循环分页取全官方待审行，与工作区历史改动投影按路径去重后返回 `nextCursor: null` 与真实总数（`pending`），客户端单次取快照、校验条目唯一且仍为待审后才逐项决策；快照总量超过 2000 条时明确拒绝并提示分批处理，条目在决策途中变化则继续整批中止、不做部分写入。差异栏的日常浏览仍走原分页接口，展示行为不变。本版修复只经单测与真实审查库只读核对，未在重启后的 GUI 上点按批量按钮验证，缺口保留，见 [0.23.1 发布说明](docs/release-0.23.1.md)。以下保留历史版本事实：
 **v0.23.0** 的快捷菜单：编辑器内一个可拖动悬浮按钮，点开是自定义菜单树（组 / 按钮，最多 5 层），按钮可携带提示词并直接发送给 AI 或只填入输入框；配置存于 `<DSH_HOME>/dsh-vscode-mode/data/quick-menu.json`，带 `revision` 乐观锁与损坏降级；节点可限定生效工作区（默认全部），按钮可绑定技能与规则，发送时按当前会话可见性校验后只拼装通过的绑定。新增 `skills.list` 只读 RPC 枚举「全局技能 + 当前工作区技能」作为候选，配置界面的工作区、技能、规则全部改为下拉弹层/多选下拉，菜单设置重排为四段并统一按钮盒模型（修掉「导入 JSON」下移不居中）。本版**未做 GUI 真机验收**（本机 pnpm store 损坏、插件未部署），缺口已获用户接受，见 [0.23.0 发布说明](docs/release-0.23.0.md)。以下保留历史版本事实：
 **v0.22.0** 的累计基线待审（非 UI 能力对齐 dsh-diff-approval）：同一文件的多次编辑折叠成一条待审、基线保留最早内容，A→B→C 显示 A→C、A→B→A 自然归零；Keep 只推进基线不改磁盘，Revert 走版本守卫写回并读回校验，撤销/重做用游标模型跨重启保留；新增 `str_replace_editor` 预像配对、`fs/observed` 终像、已入列路径的外部改动吸收与缺失/不可读生命周期、git/svn/Perforce 基线导入，以及旧 sidecar 写入口对已接管路径的服务端拒绝。差异仍在编辑器内行内展示、块级 Keep/Undo 用药丸，文件级在差异栏；详情面板与批注（含「让 AI 应答」）本版移除。验收边界：p4 仅有假 shell 单测（本机无 Perforce 客户端），「让 AI 应答」随批注一并下线故不再验证；本版未在宿主上重启验收 GUI 全流程，见 [0.22.0 发布说明](docs/release-0.22.0.md)。以下保留历史版本事实：
@@ -879,6 +880,7 @@ Keep All / Undo All 却是亮的。修复：单文件 Keep / Undo 覆盖冲突
 完整变更见 [GitHub Releases](https://github.com/Lenonss/DSH_VsCodeMode/releases)。
 近期关键版本：
 
+- **v0.23.2**：修复 Keep 在非活跃端（桌面端）点了没反应，两处成因。①`edrv.review.decide` 原先一律要求本端活跃会话，而能力位只按审查库推导，会话在另一实例时按钮亮着、点击后抛错被兜底成 HTTP 500。现按写入副作用区分：官方记录 Keep 只写共享审查库（`BEGIN IMMEDIATE` + 幂等键 + 版本 CAS），冷端放行；官方 Undo（校验证据并写回源文件）与待审投影条目（本机 JSON 无跨进程锁）仍限活跃端，返回可读跨端原因而非 500；详情新增 `capability.live`，冷端禁用 Undo 并说明、Keep 提示补「写入共享审查库，仍可执行」。②已结算待审投影的 `capability.keep` 恒为假，却因排在 `items[0]`、同路径官方待审行被历史过滤，把官方记录挤出差异栏（表现为「待审 0」+Keep 禁用+点击无反应）；现 `pendingPageOf()` 剔除已结算投影，官方记录重新成为审查目标。另详情未加载时点 Keep 不再静默失败；打开文件时若某条官方记录的改动已全部不在当前文件中（无可定位差异且缓冲区干净），自动按 Keep 结案（只标记不写盘），差异栏不再显示「本记录 N 处改动均已不在当前文件中」；官方元数据首次载入期间差异栏显示 `—` 而非按历史记录报数，重启瞬间不再闪出几十条假待审。验证：typecheck 0 error、目标单测 57/57、build 通过、真实审查库探针确认首条回到官方记录且 `capability={keep:true,undo:true,live:true}`；桌面端 GUI 点按验收因重启后恢复的是另一个会话未完成，缺口保留。发布不自动部署，升级后重启 DSH 并刷新页面。见 [发布说明](docs/release-0.23.2.md)。
 - **v0.23.1**：修复 Keep All / Undo All 在待审超过一页时必定中止——批量按钮原先复用分页接口并把每页的展示用计数当全量总数校验，官方待审 150 条时先报「待审记录在分页期间变化」或最终报「待审记录数量不完整」，实际 0 项被决策。现改为一次性完整快照（`edrv.review.list` 新增 `all: true`，服务端循环取全并与历史改动投影去重，`nextCursor: null`、`pending` 为真实总数），客户端单次取快照、校验唯一且仍待审后逐项决策；超过 2000 条明确拒绝并提示分批，途中变化仍整批中止不部分写入。差异栏浏览仍走原分页接口。本版未在重启后的 GUI 点按验证，缺口保留；发布不自动部署，升级后重启 DSH 并刷新页面。见 [发布说明](docs/release-0.23.1.md)。
 - **v0.23.0**：新增快捷菜单——可拖动悬浮按钮 + 自定义菜单树（组/按钮，最多 5 层），按钮携带提示词并「直接发送给 AI」或「只填入输入框」；配置存 `<DSH_HOME>/dsh-vscode-mode/data/quick-menu.json`，带 `revision` 乐观锁、冲突重拉不覆盖、损坏降级。节点可限定工作区（默认全部、子节点继承），按钮可绑定技能与规则，发送时按会话可见性校验后只拼装通过的绑定（规则支持引用/内联）。新增 `skills.list` 只读 RPC 枚举全局技能 + 当前工作区技能；配置界面工作区改下拉弹层、技能/规则改分组多选下拉并移除手输技能名，菜单设置重排四段、统一按钮盒模型并修掉「导入 JSON」下移不居中。本版 GUI 未真机验收（pnpm store 损坏、插件未部署），缺口已获用户接受；发布不自动部署，升级后重启 DSH 并刷新页面。见 [发布说明](docs/release-0.23.0.md)。
 - **v0.22.0**：新增累计基线待审——同文件多次编辑折叠成一条、基线保留最早内容（A→B→C 显示 A→C、A→B→A 归零）；Keep 只推进基线不改磁盘，Revert 带版本守卫写回并读回，撤销/重做游标模型跨重启保留；补 `str_replace_editor` 预像配对、外部改动吸收与缺失生命周期、git/svn/Perforce 基线导入、旧入口服务端拒绝。差异仍在编辑器内行内展示，详情面板与批注（含「让 AI 应答」）移除。p4 仅假 shell 单测（本机无 Perforce 客户端），GUI 全流程未重启验收；发布不自动部署，升级后重启 DSH 并刷新页面。见 [发布说明](docs/release-0.22.0.md)。

package/lib/client.js

@@ -3554,6 +3554,10 @@ window.__ModuleLoader__.load({
 			}
 		}
 		//#endregion
+		//#region src/shared/pendingReview.ts
+		/** 待审条目在面板列表里的 id 前缀；客户端与 Host 共用这一份口径。 */
+		const PENDING_PREFIX = "pending:";
+		//#endregion
 		//#region src/client/reviewBatch.ts
 		/**
 		* @public
@@ -3589,6 +3593,7 @@ window.__ModuleLoader__.load({
 				for (const item of items) {
 					port.guard();
 					const detail = await port.detail(item.id);
+					if (detail.capability.live === false && item.id.startsWith("pending:")) throw new Error(`${item.display}：待审条目需要在会话活跃的一端判定，请先在那一端处理或在本端打开该会话`);
 					if (!detail.item.current || detail.item.seq !== item.seq || detail.item.revision !== item.revision || !detail.capability.keep) throw new Error(`${item.display}：审查版本变化，请刷新后重试`);
 					if (reject && !detail.capability.undo) throw new Error(`${item.display}：${detail.capability.reason}`);
 					details.push(detail);
@@ -3754,10 +3759,6 @@ window.__ModuleLoader__.load({
 			return reviewBlocks(diff).map(placeRegion);
 		}
 		//#endregion
-		//#region src/shared/pendingReview.ts
-		/** 待审条目在面板列表里的 id 前缀；客户端与 Host 共用这一份口径。 */
-		const PENDING_PREFIX = "pending:";
-		//#endregion
 		//#region src/shared/pathKey.ts
 		/**
 		* 路径比较口径：Host 与客户端共用同一套「远端 / 盘符 / 相对路径」判定，只在平台折叠上分工。
@@ -382531,7 +382532,8 @@ td.UniverTableCell, th.UniverTableCell{mso-padding-alt:0cm 5.4pt 0cm 5.4pt;}
 		* @returns 持久化的全工作区控制条
 		*/
 		function ReviewAllRow(props) {
-			const total = props.total ?? 0;
+			const known = typeof props.total === "number";
+			const total = known ? props.total : 0;
 			const current = total && props.current > 0 ? props.current : 0;
 			const disabled = props.busy || !!props.unavailable || !props.pending;
 			return /* @__PURE__ */ react$1.default.createElement("div", {
@@ -382544,7 +382546,7 @@ td.UniverTableCell, th.UniverTableCell{mso-padding-alt:0cm 5.4pt 0cm 5.4pt;}
 				className: "edrv-review-total",
 				onClick: props.onOpen,
 				title: "打开当前工作区全部待审差异文件"
-			}, "差异文件 ", /* @__PURE__ */ react$1.default.createElement("strong", null, current, " / ", total)), /* @__PURE__ */ react$1.default.createElement("span", { className: "edrv-review-all-actions" }, /* @__PURE__ */ react$1.default.createElement(_deepseek_ai_dsh_client_ui_primitives.Button, {
+			}, "差异文件 ", /* @__PURE__ */ react$1.default.createElement("strong", null, known ? `${current} / ${total}` : "—")), /* @__PURE__ */ react$1.default.createElement("span", { className: "edrv-review-all-actions" }, /* @__PURE__ */ react$1.default.createElement(_deepseek_ai_dsh_client_ui_primitives.Button, {
 				size: "sm",
 				variant: "ghost",
 				onClick: props.onKeepAll,
@@ -382591,6 +382593,19 @@ td.UniverTableCell, th.UniverTableCell{mso-padding-alt:0cm 5.4pt 0cm 5.4pt;}
 				"aria-hidden": "true"
 			}, /* @__PURE__ */ react$1.default.createElement("path", { d: marks[mark] }));
 		}
+		/**
+		* @private
+		* @author ddj 2026年10月31号
+		* @description: Keep 按钮提示。会话不在本端活跃时 Keep 仍可执行（只写共享审查库），
+		* 提示里补一句说明，免得用户把它当成跨端不可用的按钮。
+		* @param hint 差异栏给出的 Keep 说明
+		* @param live 会话是否在本端活跃
+		* @returns 按钮 title 文本
+		*/
+		function keepTitle(hint, live) {
+			if (live) return hint;
+			return `${hint}（会话在另一端激活：Keep 写入共享审查库，仍可执行）`;
+		}
 		/** @public @author ddj 2026年10月27号 @param props Existing review callbacks and immutable record metadata. @returns Responsive review toolbar; no disk or decision side effects until an explicit action. */
 		function OfficialDiffBar(props) {
 			const { official, pendingRegions = [], staleRegions = [], diffIdx = 0 } = props;
@@ -382647,7 +382662,7 @@ td.UniverTableCell, th.UniverTableCell{mso-padding-alt:0cm 5.4pt 0cm 5.4pt;}
 				role: "region",
 				"aria-label": "文件差异审查"
 			}, /* @__PURE__ */ react$1.default.createElement(ReviewAllRow, {
-				total: props.fileTotal ?? 0,
+				total: props.fileTotal ?? null,
 				current: (props.fileTotal ?? 0) && (props.fileIdx ?? -1) >= 0 ? (props.fileIdx ?? 0) + 1 : 0,
 				busy: props.reviewBatchBusy,
 				unavailable: props.batchUnavailable,
@@ -382692,7 +382707,7 @@ td.UniverTableCell, th.UniverTableCell{mso-padding-alt:0cm 5.4pt 0cm 5.4pt;}
 				icon: barIcon("check"),
 				onClick: props.onAcceptFile,
 				disabled: props.reviewBatchBusy || !detail?.capability.keep,
-				title: detail?.capability.keep ? view.keepHint : detail?.capability.reason
+				title: detail?.capability.keep ? keepTitle(view.keepHint, detail.capability.live !== false) : detail?.capability.reason
 			}, "Keep"), /* @__PURE__ */ react$1.default.createElement(_deepseek_ai_dsh_client_ui_primitives.Menu, {
 				open: menu === "more",
 				anchor: moreTrigger,
@@ -382780,7 +382795,7 @@ td.UniverTableCell, th.UniverTableCell{mso-padding-alt:0cm 5.4pt 0cm 5.4pt;}
 				role: "region",
 				"aria-label": "文件差异审查"
 			}, /* @__PURE__ */ react$1.default.createElement(ReviewAllRow, {
-				total: props.fileTotal ?? 0,
+				total: props.fileTotal ?? null,
 				current: (props.fileTotal ?? 0) && (props.fileIdx ?? -1) >= 0 ? (props.fileIdx ?? 0) + 1 : 0,
 				busy,
 				pending: props.allPendingCount,
@@ -393802,6 +393817,7 @@ td.UniverTableCell, th.UniverTableCell{mso-padding-alt:0cm 5.4pt 0cm 5.4pt;}
 				review.page
 			]);
 			const reviewOpsRef = react$1.default.useRef(/* @__PURE__ */ new Map());
+			const settledRef = react$1.default.useRef(/* @__PURE__ */ new Set());
 			const [officialDetail, setOfficialDetail] = react$1.default.useState(null);
 			const [officialRecords, setOfficialRecords] = react$1.default.useState([]);
 			const [selectedReview, setSelectedReview] = react$1.default.useState(null);
@@ -393816,7 +393832,9 @@ td.UniverTableCell, th.UniverTableCell{mso-padding-alt:0cm 5.4pt 0cm 5.4pt;}
 			react$1.default.useEffect(() => {
 				const controller = new AbortController();
 				const pendingPath = (review.page?.items ?? []).filter((item) => String(item.id).startsWith(PENDING_PREFIX)).map((item) => item.currentPath || item.path).find((path) => sameFile(path, active));
-				const reviewPath = review.page?.overview?.coveredPaths.find((path) => sameFile(path, active)) ?? pendingPath;
+				const coveredPath = review.page?.overview?.coveredPaths.find((path) => sameFile(path, active));
+				const reviewPath = coveredPath ?? pendingPath;
+				dbg(sessionId, `[DEBUG reviewPath] active=${active ?? "(null)"} covered=${coveredPath ?? "(none)"} pending=${pendingPath ?? "(none)"} coveredCount=${review.page?.overview?.coveredPaths.length ?? -1} items=${review.page?.items?.length ?? -1} overview=${review.page?.overview ? 1 : 0}`);
 				if (!active || !reviewPath) {
 					setOfficialDetail(null);
 					setOfficialRecords([]);
@@ -393832,6 +393850,7 @@ td.UniverTableCell, th.UniverTableCell{mso-padding-alt:0cm 5.4pt 0cm 5.4pt;}
 						if (controller.signal.aborted) return;
 						if (!page.ok) throw new Error(page.error);
 						const selected = page.page.items.find((item) => item.id === selectedReview) ?? page.page.items[0];
+						dbg(sessionId, `[DEBUG reviewList] path=${reviewPath} items=${page.page.items.length} waiting=${page.page.pending} selected=${selected?.id ?? "(none)"}`);
 						if (!selected) {
 							setOfficialRecords([]);
 							setOfficialDetail(null);
@@ -393842,6 +393861,7 @@ td.UniverTableCell, th.UniverTableCell{mso-padding-alt:0cm 5.4pt 0cm 5.4pt;}
 							id: selected.id
 						}, controller.signal);
 						if (controller.signal.aborted) return;
+						dbg(sessionId, `[DEBUG reviewDetail] id=${selected.id} ok=${result.ok ? 1 : 0} keep=${result.ok ? result.detail.capability?.keep === true ? 1 : 0 : "-"} live=${result.ok ? String(result.detail.capability?.live) : "-"} err=${result.ok ? "" : String(result.error).slice(0, 120)}`);
 						if (!result.ok) throw new Error(result.error);
 						setOfficialRecords((previous) => JSON.stringify(previous) === JSON.stringify(page.page.items) ? previous : page.page.items);
 						setOfficialDetail((previous) => JSON.stringify(previous) === JSON.stringify(result.detail) ? previous : result.detail);
@@ -393929,6 +393949,20 @@ td.UniverTableCell, th.UniverTableCell{mso-padding-alt:0cm 5.4pt 0cm 5.4pt;}
 			const staleAmbiguous = staleRegions.filter((region) => region.staleReason === "ambiguous").length;
 			const noBlocks = !!officialDetail && (officialDetail.diff.kind !== "text" || officialDetail.diff.hunks.length === 0);
 			const decideRegions = react$1.default.useMemo(() => [...pendingRegions, ...staleRegions], [pendingRegions, staleRegions]);
+			react$1.default.useEffect(() => {
+				if (!officialDetail || contentReady !== true || batchBusyRef.current) return;
+				if (pendingRegions.length > 0 || staleRegions.length === 0) return;
+				if (noBlocks || officialDetail.capability?.keep !== true) return;
+				if (dirtyRef.current[active]) return;
+				settleInvisibleRecord(officialDetail);
+			}, [
+				officialDetail,
+				contentReady,
+				pendingRegions,
+				staleRegions,
+				noBlocks,
+				active
+			]);
 			hoverRegionsRef.current = pendingRegions;
 			lineRegionMapRef.current = react$1.default.useMemo(() => {
 				const map = /* @__PURE__ */ new Map();
@@ -396783,18 +396817,17 @@ td.UniverTableCell, th.UniverTableCell{mso-padding-alt:0cm 5.4pt 0cm 5.4pt;}
 				hunkIndex: r.idx,
 				decision: reject ? "rejected" : "accepted"
 			});
-			/** @private @author ddj 2026年10月27号 @param reject Undo instead of Keep. @param blockId Optional contiguous block. @returns Persisted decision result; never reloads over a dirty buffer. */
-			async function decideOfficial(reject, blockId) {
-				if (!officialDetail || batchBusyRef.current) return false;
-				if (reject && (dirtyRef.current[active] || savingRef.current.has(active) || saveTimerRef.current.isPending())) {
-					setError("存在未保存或正在保存的修改，请等待完成后重试");
-					return false;
-				}
-				batchBusyRef.current = true;
-				const source = officialDetail.item;
-				const undoPath = active;
-				const undoModel = editorRef.current?.getModel?.();
-				const undoVersion = undoModel?.getVersionId?.();
+			/**
+			* @private
+			* @author ddj 2026年10月31号
+			* @description: 提交一条官方记录的判定；operationId 按决策键复用，网络重试保持幂等。
+			* @param detail 目标记录（含冻结的 seq / revision）
+			* @param reject true=Undo
+			* @param blockId 可选的连续块；缺省为整文件
+			* @returns 成功返回提交结果，失败抛出 host 给出的原因
+			*/
+			async function submitOfficialDecision(detail, reject, blockId) {
+				const source = detail.item;
 				const opKey = JSON.stringify([
 					sessionId,
 					source.id,
@@ -396808,25 +396841,68 @@ td.UniverTableCell, th.UniverTableCell{mso-padding-alt:0cm 5.4pt 0cm 5.4pt;}
 					operationId = crypto.randomUUID();
 					reviewOpsRef.current.set(opKey, operationId);
 				}
+				const result = await rpc("edrv.review.decide", {
+					sessionId,
+					id: source.id,
+					seq: source.seq,
+					revision: source.revision,
+					operationId,
+					scope: blockId ? "hunk" : "file",
+					blockId,
+					decision: reject ? "rejected" : "accepted"
+				});
+				if (!result.ok) throw new Error(result.error);
+				reviewOpsRef.current.delete(opKey);
+				return result;
+			}
+			/**
+			* @private
+			* @author ddj 2026年10月31号
+			* @description: 结案一条改动已不在当前文件中的官方记录：等价于 Keep（只标记已采纳，不改写磁盘）。
+			* 这类记录没有可见差异可审，留在差异栏只会让用户去处理一个不存在的东西。
+			* @param detail 待结案的记录
+			*/
+			async function settleInvisibleRecord(detail) {
+				if (batchBusyRef.current) return;
+				const settled = settledRef.current;
+				if (settled.has(detail.item.id)) return;
+				settled.add(detail.item.id);
+				batchBusyRef.current = true;
+				try {
+					await submitOfficialDecision(detail, false);
+					refreshReview(sessionId);
+				} catch (cause) {
+					settled.delete(detail.item.id);
+					dbg(sessionId, `[DEBUG settleInvisible] id=${detail.item.id} err=${String(cause).slice(0, 160)}`);
+				} finally {
+					batchBusyRef.current = false;
+				}
+			}
+			/** @private @author ddj 2026年10月27号 @param reject Undo instead of Keep. @param blockId Optional contiguous block. @returns Persisted decision result; never reloads over a dirty buffer. */
+			async function decideOfficial(reject, blockId) {
+				dbg(sessionId, `[DEBUG decideOfficial] reject=${reject ? 1 : 0} detail=${officialDetail ? 1 : 0} busy=${batchBusyRef.current ? 1 : 0} active=${active ?? "(null)"}`);
+				if (!officialDetail || batchBusyRef.current) {
+					if (!officialDetail && !batchBusyRef.current) setError("官方差异详情尚未加载完成，请稍后重试；若一直如此，请切换文件或刷新页面。");
+					return false;
+				}
+				if (reject && (dirtyRef.current[active] || savingRef.current.has(active) || saveTimerRef.current.isPending())) {
+					setError("存在未保存或正在保存的修改，请等待完成后重试");
+					return false;
+				}
+				batchBusyRef.current = true;
+				const detail = officialDetail;
+				const undoPath = active;
+				const undoModel = editorRef.current?.getModel?.();
+				const undoVersion = undoModel?.getVersionId?.();
 				try {
-					const result = await rpc("edrv.review.decide", {
-						sessionId,
-						id: source.id,
-						seq: source.seq,
-						revision: source.revision,
-						operationId,
-						scope: blockId ? "hunk" : "file",
-						blockId,
-						decision: reject ? "rejected" : "accepted"
-					});
-					if (!result.ok) throw new Error(result.error);
-					reviewOpsRef.current.delete(opKey);
+					await submitOfficialDecision(detail, reject, blockId);
 					if (reject && activeRef.current === undoPath && !dirtyRef.current[undoPath] && !savingRef.current.has(undoPath) && undoModel?.getVersionId?.() === undoVersion) reloadFile(true);
 					else if (reject) setError("Undo 已完成，编辑器在等待期间变化；未覆盖当前缓冲区，请核对磁盘后刷新");
 					refreshReview(sessionId);
 					setStatus(reject ? "已 Undo 选中差异" : "已 Keep 选中差异");
 					return true;
 				} catch (cause) {
+					dbg(sessionId, `[DEBUG decideOfficialFail] id=${detail.item.id} reject=${reject ? 1 : 0} err=${String(cause).slice(0, 160)}`);
 					setError(String(cause));
 					return false;
 				} finally {
@@ -397043,13 +397119,24 @@ td.UniverTableCell, th.UniverTableCell{mso-padding-alt:0cm 5.4pt 0cm 5.4pt;}
 				}
 				return operationId;
 			}
+			/**
+			* @private
+			* @author ddj 2026年10月31号
+			* @description: 批量操作要求官方待审列表已就绪；切换会话后列表还在载入时，
+			* 先触发一次刷新并给出可读提示，而不是直接报错中止（用户看到红色「尚未就绪」会以为功能坏了）。
+			* @param action 面向用户的动作名
+			* @returns 列表已就绪时为 true
+			*/
+			function reviewReadyForBatch(action) {
+				if (review.page?.overview) return true;
+				refreshReview(sessionId);
+				setStatus(`官方待审列表正在载入，载入完成后请再次点击${action}`);
+				return false;
+			}
 			/** @private @author ddj 2026年10月27号 @param reject Undo or Keep all workspace pending records, including previous rounds. @returns Explicit partial results; legacy proceeds only when official stage fully succeeds. */
 			async function actReviewAll(reject) {
 				if (batchBusyRef.current) return;
-				if (!review.page?.overview) {
-					setError("官方待审列表尚未就绪，未执行批量操作");
-					return;
-				}
+				if (!reviewReadyForBatch(reject ? "Undo All" : "Keep All")) return;
 				if (!sum.pendingFiles.length) return;
 				const token = reviewLifeRef.current;
 				const undoPath = activeRef.current;
@@ -397116,10 +397203,7 @@ td.UniverTableCell, th.UniverTableCell{mso-padding-alt:0cm 5.4pt 0cm 5.4pt;}
 			/** @private @author ddj 2026年10月27号 @param reject Whether to undo pending files. @description Open a DSH confirmation without starting any mutations. */
 			function askReviewAll(reject) {
 				if (batchBusyRef.current || !sum.pendingFiles.length) return;
-				if (!review.page?.overview) {
-					setError("官方待审列表尚未就绪，未执行批量操作");
-					return;
-				}
+				if (!reviewReadyForBatch(reject ? "Undo All" : "Keep All")) return;
 				setReviewAllPrompt({
 					reject,
 					count: sum.pendingFiles.length,
@@ -398617,6 +398701,7 @@ td.UniverTableCell, th.UniverTableCell{mso-padding-alt:0cm 5.4pt 0cm 5.4pt;}
 			const sessionDown = isSessionMissingError(rawReviewError);
 			const reviewErrorTone = sessionDown ? "info" : "error";
 			const reviewErrorText = !rawReviewError ? void 0 : sessionDown ? SESSION_DOWN_NOTICE : sessionErrorText(rawReviewError);
+			const reviewLoading = review.page === null && !review.error;
 			const buildDockSnapshot = () => ({
 				mode: editorDockMode(active),
 				missingDeletion: !!missingNow,
@@ -398644,7 +398729,7 @@ td.UniverTableCell, th.UniverTableCell{mso-padding-alt:0cm 5.4pt 0cm 5.4pt;}
 				onUndoFile: undoFile,
 				onAcceptAllFiles: acceptAllFiles,
 				onUndoAllFiles: undoAllFiles,
-				allPendingCount: allPending.length + (review.page?.pending ?? 0),
+				allPendingCount: reviewLoading ? 0 : allPending.length + (review.page?.pending ?? 0),
 				reviewBatchBusy,
 				batchUnavailable: !review.page?.overview ? "官方待审列表尚未就绪" : void 0,
 				onRollback: rollbackFile,
@@ -398661,7 +398746,7 @@ td.UniverTableCell, th.UniverTableCell{mso-padding-alt:0cm 5.4pt 0cm 5.4pt;}
 				contentReady,
 				diffTotal: displayDiffTotal(contentReady, pendingRegions.length, sum.files.find((file) => sameFile(file.path, active))?.pending ?? 0),
 				fileIdx,
-				fileTotal: sum.pendingFiles.length,
+				fileTotal: reviewLoading ? null : sum.pendingFiles.length,
 				onPrevDiff: () => gotoDiff(-1),
 				onNextDiff: () => gotoDiff(1),
 				onPrevFile: () => gotoFile(-1),

package/lib/client.js.map

@@ -1,4 +1,4 @@
 [diffend] Oversized file quarantined before diffing.
 name: package/lib/client.js.map
-size: 35662220 bytes
-sha256: 0204857e50fdc9778b1c9c5e5b128880ec0af54eb8d94edbd9d2b38cb2a63d3d
+size: 35670356 bytes
+sha256: 4a298d4c08a69a99f2aa5a4626c9aa23ff928609a2af4f4cb9328d3ba4b25114

package/lib/index.js

@@ -15971,6 +15971,10 @@ function finishUndo(cwd, request) {
 //#region src/reviewRpc.ts
 /** 批量写入快照的硬上限：待审积压超过它说明状态异常，宁可拒绝也不做无界写循环。 */
 const BATCH_LIMIT = 2e3;
+/** 冷端 Undo 说明：Undo 会校验证据并写回源文件，只能在持有该会话的那一端执行。 */
+const UNDO_NEEDS_LIVE = "Undo 需要在本端活跃的会话上执行：该会话当前在另一端（Web / 桌面）激活。Keep 不受影响，可在任一端执行；或在本端打开该会话后重试。";
+/** 冷端待审判定说明：待审库是按工作区的本地存储，判定需要与正在编辑的那一端保持一致。 */
+const PENDING_NEEDS_LIVE = "待审条目的判定需要在本端活跃的会话上执行：该会话当前在另一端激活，请在那一端处理，或在本端打开该会话后重试。";
 /**
 * @private
 * @author ddj 2026年10月31号
@@ -16020,19 +16024,61 @@ function batchSnapshot(cwd, pending) {
 * @returns 审查相关处理器表
 */
 function reviewHandlers(ctx, registry) {
-	/** @private @author ddj 2026年10月28号 @param sessionId 请求作用域 @returns 已迁移工作区（允许跨实例冷读） */
-	async function workspace(sessionId) {
+	/**
+	* @private
+	* @author ddj 2026年10月31号
+	* @description: 解析会话作用域，并标明本端是否持有该会话的活跃实例。
+	* 工作区一律由 host 解析，绝不采信调用方 cwd；冷读作用域只带 cwd，不带会话实例。
+	* @param sessionId 请求作用域
+	* @returns 已迁移工作区与活跃标记
+	* @throws 会话不存在、会话无工作区或活跃库迁移未完成
+	*/
+	async function scopeOf(sessionId) {
 		const scope = await resolveScope(ctx, sessionId);
 		if ("err" in scope) throw new Error("会话不存在或没有工作区");
 		await bucketOf(registry, ctx, scope.cwd);
-		return scope.cwd;
+		return {
+			cwd: scope.cwd,
+			live: !!scope.session
+		};
 	}
-	/** @private @author ddj 2026年10月28号 @param sessionId 请求作用域 @returns 本端 live 会话的工作区；冷读作用域一律拒绝写入 */
-	async function liveWorkspace(sessionId) {
-		const scope = await resolveScope(ctx, sessionId);
-		if ("err" in scope || !scope.session) throw new Error("会话不存在或没有工作区");
-		await bucketOf(registry, ctx, scope.cwd);
-		return scope.cwd;
+	/**
+	* @private
+	* @author ddj 2026年10月31号
+	* @description: 本端是否持有该会话的活跃实例（纯诊断用，不解析工作区）。
+	* @param sessionId 请求作用域
+	* @returns 本端加载了该会话时为 true
+	*/
+	function liveOf(sessionId) {
+		return !!sessionOf(ctx, sessionId);
+	}
+	/**
+	* @private
+	* @author ddj 2026年10月31号
+	* @description: 按本端是否活跃覆写面板能力位。活跃端保持数据库结论；
+	* 冷端关闭依赖会话的操作（Undo、待审判定）并给出跨端原因，Keep 仍按数据库结论放行。
+	* @param capability 数据库推导的能力位
+	* @param live 本端是否持有活跃会话
+	* @param pending 是否为待审投影条目
+	* @returns 面板可用的能力位
+	*/
+	function liveCapabilityOf(capability, live, pending) {
+		if (live) return {
+			...capability,
+			live: true
+		};
+		if (pending) return {
+			keep: false,
+			undo: false,
+			reason: PENDING_NEEDS_LIVE,
+			live: false
+		};
+		return {
+			...capability,
+			undo: false,
+			live: false,
+			reason: capability.undo ? UNDO_NEEDS_LIVE : capability.reason
+		};
 	}
 	/**
 	* @private
@@ -16045,7 +16091,11 @@ function reviewHandlers(ctx, registry) {
 	*/
 	function pendingPageOf(cwd, sessionId) {
 		try {
-			return loadPage(cwd, sessionId);
+			const page = loadPage(cwd, sessionId);
+			return page.items.some((item) => item.settled === true) ? {
+				...page,
+				items: page.items.filter((item) => item.settled !== true)
+			} : page;
 		} catch {
 			return {
 				items: [],
@@ -16057,7 +16107,7 @@ function reviewHandlers(ctx, registry) {
 	}
 	return {
 		"edrv.review.list": async (args) => {
-			const cwd = await workspace(args.sessionId);
+			const { cwd } = await scopeOf(args.sessionId);
 			if (args.all && !args.history && args.path === void 0) return batchSnapshot(cwd, pendingPageOf(cwd, args.sessionId));
 			const official = reviewPage(cwd, args.cursor, args.history, args.path);
 			const pending = pendingPageOf(cwd, args.sessionId);
@@ -16077,63 +16127,97 @@ function reviewHandlers(ctx, registry) {
 					}
 				};
 			}
+			const merged = mergedPage(official, pending, cwd);
+			log.debug(`[review] list live=${liveOf(args.sessionId) ? 1 : 0} path=${args.path ?? "(all)"} items=${merged.items.length} pending=${merged.pending} covered=${merged.overview?.coveredPaths.length ?? -1}`);
 			return {
 				ok: true,
-				page: mergedPage(official, pending, cwd)
+				page: merged
 			};
 		},
 		"edrv.review.detail": async (args) => {
-			const cwd = await workspace(args.sessionId);
+			const scope = await scopeOf(args.sessionId);
+			const cwd = scope.cwd;
 			if (args.id.startsWith("pending:")) {
 				const detail = loadDetail(cwd, args.id.slice(8));
 				if (!detail) return {
 					ok: false,
 					error: "待审条目不存在"
 				};
+				const mapped = pendingDetailOf(detail, new Date(detail.item.updatedAt).toISOString(), cwd);
 				return {
 					ok: true,
-					detail: pendingDetailOf(detail, new Date(detail.item.updatedAt).toISOString(), cwd)
+					detail: {
+						...mapped,
+						capability: liveCapabilityOf(mapped.capability, scope.live, true)
+					}
 				};
 			}
 			const detail = reviewDetail(cwd, args.id);
-			return detail ? {
+			if (!detail) {
+				log.debug(`[review] detail live=${scope.live ? 1 : 0} id=${args.id} found=0`);
+				return {
+					ok: false,
+					error: "官方审查记录不存在"
+				};
+			}
+			const capability = liveCapabilityOf(detail.capability, scope.live, false);
+			log.debug(`[review] detail live=${scope.live ? 1 : 0} id=${args.id} found=1 keep=${capability.keep ? 1 : 0} undo=${capability.undo ? 1 : 0}`);
+			return {
 				ok: true,
-				detail
-			} : {
-				ok: false,
-				error: "官方审查记录不存在"
+				detail: {
+					...detail,
+					capability
+				}
 			};
 		},
 		"edrv.review.decide": async (args) => {
-			const cwd = await liveWorkspace(args.sessionId);
-			if (args.id.startsWith("pending:")) {
-				const result = await applyDecide(ctx.get("fs"), cwd, {
-					sessionId: args.sessionId,
-					id: args.id.slice(8),
-					revision: args.revision,
-					operationId: args.operationId,
-					decision: args.decision,
-					scope: args.scope === "hunk" ? "block" : "file",
-					...args.blockId === void 0 ? {} : { blockId: args.blockId }
-				});
-				if (!result.ok) return {
-					ok: false,
-					error: result.reason
-				};
-				return {
-					ok: true,
-					revision: result.flags.revision
-				};
-			}
 			try {
+				const scope = await scopeOf(args.sessionId);
+				if (args.id.startsWith("pending:")) {
+					if (!scope.live) return {
+						ok: false,
+						error: PENDING_NEEDS_LIVE
+					};
+					const result = await applyDecide(ctx.get("fs"), scope.cwd, {
+						sessionId: args.sessionId,
+						id: args.id.slice(8),
+						revision: args.revision,
+						operationId: args.operationId,
+						decision: args.decision,
+						scope: args.scope === "hunk" ? "block" : "file",
+						...args.blockId === void 0 ? {} : { blockId: args.blockId }
+					});
+					if (!result.ok) return {
+						ok: false,
+						error: result.reason
+					};
+					return {
+						ok: true,
+						revision: result.flags.revision
+					};
+				}
+				if (args.decision === "rejected") {
+					if (!scope.live) return {
+						ok: false,
+						error: UNDO_NEEDS_LIVE
+					};
+					return {
+						ok: true,
+						...await undoReview(ctx, sessionOf(ctx, args.sessionId), scope.cwd, args)
+					};
+				}
+				const result = decideReview(scope.cwd, args);
+				if (!scope.live) log.info(`[review] 冷端 Keep 官方记录 ${args.id}`);
 				return {
 					ok: true,
-					...args.decision === "rejected" ? await undoReview(ctx, sessionOf(ctx, args.sessionId), cwd, args) : decideReview(cwd, args)
+					...result
 				};
 			} catch (error) {
+				const message = error instanceof Error ? error.message : String(error);
+				log.warn(`[review] decide 失败 id=${args.id} decision=${args.decision}: ${message}`);
 				return {
 					ok: false,
-					error: error instanceof Error ? error.message : String(error)
+					error: message
 				};
 			}
 		}