dsh-vscode-mode 0.23.0 → 0.23.2

package/lib/index.js

@@ -15969,6 +15969,51 @@ function finishUndo(cwd, request) {
 }
 //#endregion
 //#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号
+* @description: 批量写入用的一次性完整快照。合并口径与列表一致（待审投影优先、同路径官方旧行让位），
+* 但一次取全，且 items 与 pending 同源，客户端才能把它当成写前不变式。
+* @param cwd 工作区
+* @param pending 该会话可见的待审页
+* @returns 完整快照；待审积压超过上限或分页期间数量漂移时返回错误
+*/
+function batchSnapshot(cwd, pending) {
+	const first = reviewPage(cwd);
+	if (first.pending > BATCH_LIMIT) return {
+		ok: false,
+		error: `待审记录过多（${first.pending} 条，上限 ${BATCH_LIMIT}），请先分批处理后重试`
+	};
+	const official = [];
+	let page = first;
+	while (page) {
+		for (const item of page.items) if (item.current && item.decision === "pending") official.push(item);
+		page = page.nextCursor === null ? void 0 : reviewPage(cwd, page.nextCursor);
+	}
+	if (official.length !== first.pending) return {
+		ok: false,
+		error: "待审记录数量不完整，未执行批量操作"
+	};
+	const live = pending.items.filter((item) => !item.settled);
+	const kept = official.filter((row) => !live.some((entry) => pendingSamePath(entry.path, row.currentPath) || pendingSamePath(entry.path, row.path)));
+	const items = [...live.map((entry) => pendingItemOf(entry, cwd)), ...kept];
+	return {
+		ok: true,
+		page: {
+			items,
+			pending: items.length,
+			nextCursor: null,
+			gaps: first.gaps,
+			...first.overview ? { overview: first.overview } : {}
+		}
+	};
+}
 /**
 * @public
 * @author ddj 2026年10月26号 / 2026年10月28号
@@ -15979,19 +16024,61 @@ function finishUndo(cwd, request) {
 * @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
@@ -16004,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: [],
@@ -16016,7 +16107,8 @@ 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);
 			if (args.history) return {
@@ -16035,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
 				};
 			}
 		}