dsh-vscode-mode 0.23.4 → 0.23.5

package/README.md

@@ -533,13 +533,13 @@ profile 请替换为实际目标，不要把示例目录当成当前运行 profi
 
 ```bash
 # ① Git 安装（clone + prepare 构建；推荐打固定 tag）
-dsh plugin --profile web add github:Lenonss/DSH_VsCodeMode#v0.23.4
+dsh plugin --profile web add github:Lenonss/DSH_VsCodeMode#v0.23.5
 
 # ② npm 注册表（发布到 npm 后）
-dsh plugin --profile web add dsh-vscode-mode@0.23.4
+dsh plugin --profile web add dsh-vscode-mode@0.23.5
 
 # ③ GitHub Release tgz 直装（Release asset 可用后）
-dsh plugin --profile web add https://github.com/Lenonss/DSH_VsCodeMode/releases/download/v0.23.4/dsh-vscode-mode-0.23.4.tgz
+dsh plugin --profile web add https://github.com/Lenonss/DSH_VsCodeMode/releases/download/v0.23.5/dsh-vscode-mode-0.23.5.tgz
 ```
 
 > **DSH 版本政策（0.17.0 起）**：六项 DSH peer 仍统一声明为 `*`，另以
@@ -880,7 +880,7 @@ Keep All / Undo All 却是亮的。修复：单文件 Keep / Undo 覆盖冲突
 完整变更见 [GitHub Releases](https://github.com/Lenonss/DSH_VsCodeMode/releases)。
 近期关键版本：
 
-- **v0.23.4**：修复桌面端点计划卡「查看全文」会先闪开一页官方计划预览、随即自动关闭。根因：官方卡片走 `ctx.sidebarRight.openResource(planAddress(callId))`，而本插件以 extension 优先级认领了 `dsh-resource://plan/**` 与 `dsh-resource://plan-review/**`，于是侧栏先按本插件类型建出一个页签（正文「正在打开计划预览…」），挂载后才把计划转进「文件编辑」、再把这个页签关掉——用户看到的就是这次建而即关。现由新增 `src/client/planDirect.ts` 在装配期包装 `sidebarRight.openResource`：只接管「2 段计划地址 + 会话为在屏会话」，正文依次取回顾卡片的 `params.planReview`、插件侧地址→正文缓存、可选 `resources.source()` 资源流（上限 1.5s），拿到正文后经 `requestOpen(..., { preview: true })` 直通本插件编辑器的只读计划页签，不再创建侧栏页签；非计划地址、跨会话与 subagent 5 段地址、正文不可得、编辑器 5s 未回执、以及服务对象只读导致包装赋值失败，一律回落原生 `openResource`，原「认领 + 挂载后转移」链路保留为后备。`openReceipt` 新增可选 `timeoutMs`，`planState` 新增有界（64 条）地址→正文缓存并由 live `exit_plan_mode` 写入。验证：typecheck 0 error；`tests/planDirect.test.ts`（新增 9 例）、`tests/planBridge.test.ts`（13）、`tests/planState.test.ts`（10）、`tests/openReceipt.test.ts`（9）四套 41/41；`npm run build` 双 bundle 通过（含客户端产物完整性校验）；浏览器确认新 bundle 已加载且 Console 无「计划直通装配失败」；GUI 点按由用户在本机 Web 端手动复验后发布。见 [发布说明](docs/release-0.23.4.md)。
+- **v0.23.5**：修复差异栏把同一文件算成两条（差异文件数偏大）。待审条目存的是写入时的相对路径，官方导航用绝对身份，而 `mergedPage()` 与 `diffSummary` 的去重都只按 path 建覆盖集，相对路径的待审行压不掉绝对身份的官方行，于是同一文件同时留下待审行与官方汇总行；现两侧覆盖集都补入条目身份（与官方 identity 同源）。另修复本会话新建的文件被渲染成一堆修改块：累计基线保留「最早内容」，新建文件首次采集时磁盘已有会话自己的中间产物，基线因此非空，「新增 516 行」被切成若干 `-`/`+` 混合块，与官方整文件新增（`before=false`、`oldLines=0`）口径相反；现按每个文件在该会话内最早的官方记录判定整文件新增，命中时待审侧基线置空，列表与详情只出一块全加差异、增删计数等于整文件行数，`addedFiles()` 查询失败则回退旧口径。验证：typecheck 0 error；全量单测通过；`npm run build` 通过；本版未做 GUI 点按验收（缺口保留）。见 [0.23.5 发布说明](docs/release-0.23.5.md)。- **v0.23.4**：修复桌面端点计划卡「查看全文」会先闪开一页官方计划预览、随即自动关闭。根因：官方卡片走 `ctx.sidebarRight.openResource(planAddress(callId))`，而本插件以 extension 优先级认领了 `dsh-resource://plan/**` 与 `dsh-resource://plan-review/**`，于是侧栏先按本插件类型建出一个页签（正文「正在打开计划预览…」），挂载后才把计划转进「文件编辑」、再把这个页签关掉——用户看到的就是这次建而即关。现由新增 `src/client/planDirect.ts` 在装配期包装 `sidebarRight.openResource`：只接管「2 段计划地址 + 会话为在屏会话」，正文依次取回顾卡片的 `params.planReview`、插件侧地址→正文缓存、可选 `resources.source()` 资源流（上限 1.5s），拿到正文后经 `requestOpen(..., { preview: true })` 直通本插件编辑器的只读计划页签，不再创建侧栏页签；非计划地址、跨会话与 subagent 5 段地址、正文不可得、编辑器 5s 未回执、以及服务对象只读导致包装赋值失败，一律回落原生 `openResource`，原「认领 + 挂载后转移」链路保留为后备。`openReceipt` 新增可选 `timeoutMs`，`planState` 新增有界（64 条）地址→正文缓存并由 live `exit_plan_mode` 写入。验证：typecheck 0 error；`tests/planDirect.test.ts`（新增 9 例）、`tests/planBridge.test.ts`（13）、`tests/planState.test.ts`（10）、`tests/openReceipt.test.ts`（9）四套 41/41；`npm run build` 双 bundle 通过（含客户端产物完整性校验）；浏览器确认新 bundle 已加载且 Console 无「计划直通装配失败」；GUI 点按由用户在本机 Web 端手动复验后发布。见 [发布说明](docs/release-0.23.4.md)。
 - **v0.23.3**：修复 issue #13（已删除文件的差异条目把人卡死）与 #12（重启后成串「采集不完整」）。①文件在磁盘上被删掉时，客户端申请整文件删除预览被「已有官方审查记录」拒绝，而拒绝只是一条普通 `Error`，于是落到通用加载失败分支——「文件加载失败：…」+ 进度钉在 0%、重试同错、标签页关不掉；`edrv.review.detail` 还把 Undo 不可用解释成「缺少可验证的原始字节」，而真实原因是文件已不在。现缺失文件预览的可预期拒绝带稳定码（`official-covered` / `official-decided` / `no-records` / `no-evidence` / `too-large`），客户端据此显示「文件已在磁盘删除（· 有官方待审改动）」终态：标题 + 原因 + 重试 / 关闭标签页，不再画 0% 进度条；官方待审记录在磁盘已删除时给出「Keep 按删除结案、Undo 需要读写该文件」的准确原因（Keep 语义不变，仍只写共享审查库），记录都已结案时提示可直接关闭；`!review` 的磁盘状态抖动改为最多重读 2 次后给终态，消除缺失↔重现的无限递归。v0.22.0 起 `hasOfficial()` 已用 `typeof path !== 'string'` 守卫堵掉把不透明 FsTarget 绑进 SQL 导致的 `Unknown named parameter 'targetKey'`，本版不再复现。②官方 `workspace/changes` 的 summary 只存在宿主内存里，插件启动补采原先无条件把历史事件全部入队，重启后每一轮都必然抛「summary 已失效或尚不可用」并写成不可修复的 `gap`（面板成串「轮次 N 采集不完整」）；现补采先 `summary()` 探测再入队，失效事件只记一条汇总日志。验证：typecheck 与基线同为 37 个既有错误（`fflate/browser` 依赖缺失）、零新增；定向单测 111/111（含新增 `tests/deletedFileReview.test.ts` 5 例与 `tests/officialBackfill.test.ts` 2 例）、EditorView 相关 82/82；`npm run build` 四个 bundle 通过。见 [发布说明](docs/release-0.23.3.md)。
 - **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)。

package/lib/client.js

@@ -4117,6 +4117,7 @@ window.__ModuleLoader__.load({
 				if (!item.id.startsWith("pending:")) continue;
 				const path = item.currentPath || item.path;
 				covered.add(diffPathKey(path));
+				covered.add(diffPathKey(item.id.slice(8)));
 				rows.push({
 					path,
 					label: item.display || path,

package/lib/client.js.map

@@ -1,4 +1,4 @@
 [diffend] Oversized file quarantined before diffing.
 name: package/lib/client.js.map
-size: 35694036 bytes
-sha256: 5a0053a6aa5733ebef51b077a624f0fc5bcf1a9c555232c357762a7eb8bdf2ef
+size: 35694514 bytes
+sha256: 7fa559b003ee01e2e53a8698796d4a4e567f8e63d0b6d00548272a0057026c6d

package/lib/index.js

@@ -7652,6 +7652,27 @@ function reviewPage(cwd, cursor = 0, history = false, path) {
 /**
 * @public
 * @author ddj 2026年10月31号
+* @description: 该会话里由官方记录判为「整文件新增」的文件身份集合。待审条目命中时基线要置空：
+* 文件是本会话新建的，累计基线只是会话自己的中间产物，拿它做旧侧会把新增文件渲染成一堆修改块。
+* @param cwd 迁移后的工作区
+* @param sessionId 会话
+* @returns 整文件新增的文件身份集合
+*/
+function addedFiles(cwd, sessionId) {
+	try {
+		const db = dbFor(cwd);
+		if (!db.prepare("SELECT 1 FROM sqlite_master WHERE name='review_files' AND type='table'").get()) return /* @__PURE__ */ new Set();
+		const rows = db.prepare(`SELECT identity FROM (SELECT identity,
+      json_extract(diff,'$.before') AS before, ROW_NUMBER() OVER (PARTITION BY identity ORDER BY turn, seq) AS rank
+      FROM review_files WHERE session_id=?) WHERE rank=1 AND before=0`).all(sessionId);
+		return new Set(rows.map((row) => String(row.identity)));
+	} catch {
+		return /* @__PURE__ */ new Set();
+	}
+}
+/**
+* @public
+* @author ddj 2026年10月31号
 * @description: 该文件是否还有未处理的官方待审改动。缺失文件预览据此区分「需要去差异栏 Keep 结案」
 * 与「记录都已处理、可直接关闭标签页」，避免已结案的删除文件继续提示待审。
 * @param cwd Migrated workspace
@@ -9297,10 +9318,22 @@ function capabilityOf(entry, flags) {
 * @author ddj 2026年10月29号
 * @description: 供详情接口复用的差异构造。
 * @param entry 待审条目
+* @param baselineEmpty 基线置空：文件由本次会话新建，旧基线只是会话自己的中间产物
 * @returns 官方差异
 */
-function diffOf(entry) {
-	return textDiff(entry.path, entry.path, entry.oldText, entry.newText);
+function diffOf(entry, baselineEmpty = false) {
+	if (!baselineEmpty) return textDiff(entry.path, entry.path, entry.oldText, entry.newText);
+	if (entry.newText === "") return textDiff(entry.path, entry.path, "", "");
+	const diff = textDiff(entry.path, entry.path, "", entry.newText);
+	if (diff.kind !== "text") return diff;
+	return {
+		...diff,
+		hunks: diff.hunks.map((hunk) => ({
+			...hunk,
+			oldLines: 0,
+			lines: hunk.lines.filter((line) => line !== "-")
+		}))
+	};
 }
 //#endregion
 //#region src/pending/persist.ts
@@ -9560,10 +9593,11 @@ function flagsOfRow(row) {
 * @author ddj 2026年10月29号
 * @description: 统计一次差异的增删行数；非文本差异统一记 0/0。
 * @param entry 待审条目
+* @param baselineEmpty 基线置空（新建文件按全量新增计）
 * @returns 增删行数
 */
-function countsOf(entry) {
-	const diff = diffOf(entry);
+function countsOf(entry, baselineEmpty = false) {
+	const diff = diffOf(entry, baselineEmpty);
 	if (diff.kind !== "text") return {
 		added: 0,
 		deleted: 0
@@ -9633,11 +9667,12 @@ function foldWrite(stored, write) {
 * @author ddj 2026年10月29号
 * @description: 条目转列表投影，已解决条目仍保留但不可再决定。
 * @param stored 条目与标记
+* @param baselineEmpty 基线置空（新建文件按全量新增计）
 * @returns 列表项
 */
-function itemOf(stored) {
+function itemOf(stored, baselineEmpty = false) {
 	const { entry, flags } = stored;
-	const counts = countsOf(entry);
+	const counts = countsOf(entry, baselineEmpty);
 	const item = {
 		id: entry.id,
 		path: entry.path,
@@ -9662,12 +9697,13 @@ function itemOf(stored) {
 * @description: 读一页待审条目，按更新时间升序；会话过滤走血缘集合。
 * @param cwd 工作区
 * @param sessionId 当前会话
+* @param wholeAdd 本次会话新建的文件身份集合，命中时基线置空
 * @returns 列表
 */
-function loadPage(cwd, sessionId) {
+function loadPage(cwd, sessionId, wholeAdd) {
 	const all = loadEntries(cwd);
 	const visible = sessionId ? all.filter((stored) => stored.entry.sessionIds.includes(sessionId)) : all;
-	const items = visible.map(itemOf);
+	const items = visible.map((stored) => itemOf(stored, wholeAdd?.has(stored.entry.id) === true));
 	const gaps = [];
 	for (const stored of visible) if (stored.flags.oversized) gaps.push(`${stored.entry.path}: 文本超出上限，仅保留标记`);
 	return {
@@ -9682,14 +9718,15 @@ function loadPage(cwd, sessionId) {
 * @description: 读单条详情，包含可交给现有面板渲染的差异与块决定。
 * @param cwd 工作区
 * @param id 条目身份
+* @param baselineEmpty 基线置空（新建文件按全量新增渲染）
 * @returns 详情，条目不存在时 null
 */
-function loadDetail(cwd, id) {
+function loadDetail(cwd, id, baselineEmpty = false) {
 	const stored = loadEntries(cwd).find((item) => item.entry.id === id);
 	if (!stored) return null;
-	const diff = diffOf(stored.entry);
+	const diff = diffOf(stored.entry, baselineEmpty);
 	return {
-		item: itemOf(stored),
+		item: itemOf(stored, baselineEmpty),
 		diff,
 		blocks: blockStatesOf(stored.entry),
 		capability: capabilityOf(stored.entry, stored.flags)
@@ -10249,7 +10286,11 @@ function pendingCovers(cwd, path) {
 * @returns 合并后的页
 */
 function mergedPage(official, pending, cwd) {
-	const covered = new Set(pending.items.map((item) => pathKey$1(item.path)));
+	const covered = /* @__PURE__ */ new Set();
+	for (const item of pending.items) {
+		covered.add(pathKey$1(item.path));
+		covered.add(pathKey$1(item.id));
+	}
 	const kept = official.items.filter((item) => !covered.has(pathKey$1(item.currentPath)) && !covered.has(pathKey$1(item.path)));
 	return {
 		items: [...pending.items.map((item) => pendingItemOf(item, cwd)), ...kept],
@@ -16154,7 +16195,7 @@ function reviewHandlers(ctx, registry) {
 	*/
 	function pendingPageOf(cwd, sessionId) {
 		try {
-			const page = loadPage(cwd, sessionId);
+			const page = loadPage(cwd, sessionId, addedFiles(cwd, sessionId));
 			return page.items.some((item) => item.settled === true) ? {
 				...page,
 				items: page.items.filter((item) => item.settled !== true)
@@ -16201,7 +16242,8 @@ function reviewHandlers(ctx, registry) {
 			const scope = await scopeOf(args.sessionId);
 			const cwd = scope.cwd;
 			if (args.id.startsWith("pending:")) {
-				const detail = loadDetail(cwd, args.id.slice(8));
+				const entryId = args.id.slice(8);
+				const detail = loadDetail(cwd, entryId, addedFiles(cwd, args.sessionId).has(entryId));
 				if (!detail) return {
 					ok: false,
 					error: "待审条目不存在"
@@ -16350,7 +16392,7 @@ function readHandlers(kit) {
 	return {
 		"edrv.pending.list": async (args) => {
 			const cwd = await kit.read(args.sessionId);
-			const page = loadPage(cwd, args.sessionId);
+			const page = loadPage(cwd, args.sessionId, addedFiles(cwd, args.sessionId));
 			for (const item of page.items) trackPendingCover(cwd, item.path, true);
 			return {
 				ok: true,
@@ -16358,7 +16400,9 @@ function readHandlers(kit) {
 			};
 		},
 		"edrv.pending.detail": async (args) => {
-			const detail = loadDetail(await kit.read(args.sessionId), pendingRawId(args.id));
+			const cwd = await kit.read(args.sessionId);
+			const rawId = pendingRawId(args.id);
+			const detail = loadDetail(cwd, rawId, addedFiles(cwd, args.sessionId).has(rawId));
 			return detail ? {
 				ok: true,
 				detail