dsh-recall-plugin 2.0.0 → 2.1.1

package/lib/store.js

@@ -13,6 +13,7 @@ import os from 'node:os'
 import crypto from 'node:crypto'
 import * as pwshScripts from './scripts.pwsh.js'
 import * as posixScripts from './scripts.posix.js'
+import { classifyEnvError } from './diagnostics.js'
 
 // home 不可写时迁移重试的节流间隔：避免每条消息都白试一次注定失败的迁移
 const HOME_RETRY_MS = 300000
@@ -20,6 +21,69 @@ const HOME_RETRY_MS = 300000
 // 最近错误环形缓冲容量：设置页排障用，20 条足够回溯一轮快照/gc 的失败
 const ERROR_BUFFER_MAX = 20
 
+// ---- POSIX home 基底解析（模块级纯逻辑，单测直测；工厂内 posixHomeBaseResolve
+// 委托到这里）。放模块级而非工厂闭包：分支行为要在 win32 CI 上可测，而
+// posixHomeBaseResolve 只在 POSIX 运行时被触达。----
+
+// 三档回退选择：bash env $DSH_HOME → Node 主进程 DSH_HOME → os.homedir()。
+// 第三档必须补 /.dsh 子目录（I24）：win32 版第三档是 Join-Path USERPROFILE .dsh
+// （scripts.pwsh.js homeDirScript），POSIX 版曾直接用裸 homedir，快照落
+// ~/dsh-recall-snapshots 而非 ~/.dsh/dsh-recall-snapshots（issue #11 实证）。
+// 返回 third 标记是否走了第三档——只有第三档才涉及旧容器迁移（存量用户
+// 的数据在旧位，改 base 前要先搬）。
+export function selectPosixHomeBase({ probed, envHome, homedir }) {
+  if (probed) return { base: probed, third: false }
+  if (envHome) return { base: envHome, third: false }
+  return { base: homedir + '/.dsh', third: true }
+}
+
+// 第三档命中时的一次性旧容器迁移编排（best-effort，数据安全优先）：
+// legacyHomeMigrateScript 只在「旧容器存在且新容器不存在」时整容器 mv，
+// 输出四态由这里裁决——MIGRATE_OK / OLD_ABSENT 落规范位置（~/.dsh/…）；
+// BOTH_PRESENT（双容器并存）/ MIGRATE_FAIL（mv 失败）沿用旧位并 recordError，
+// 与 tryUpgradeToHome 的非致命迁移哲学一致：数据不丢永远优先于路径规范。
+// 探测命令自身失败按同策略回落旧位——此刻无法判断旧容器是否存在，选新位
+// 会让存量用户「看不到」历史快照，选旧位对新装机只是维持修复前的行为。
+export async function resolvePosixHomeBase(deps, { probed, envHome, homedir }) {
+  const sel = selectPosixHomeBase({ probed, envHome, homedir })
+  if (!sel.third) return sel.base
+  try {
+    const out = String(await deps.runShell(deps.scripts.legacyHomeMigrateScript(homedir), { timeoutMs: 300000, stdoutMaxBytes: 4096 })).trim()
+    if (out === 'MIGRATE_OK' || out === 'OLD_ABSENT') return sel.base
+    deps.recordError(
+      out === 'BOTH_PRESENT'
+        ? 'recall home store 新旧容器并存（' + homedir + '/dsh-recall-snapshots 与 ' + homedir + '/.dsh/dsh-recall-snapshots），沿用旧位，未做任何改动'
+        : 'recall 旧快照容器迁移失败（MIGRATE_FAIL），沿用旧位 ' + homedir + '/dsh-recall-snapshots'
+    )
+    return homedir
+  } catch (error) {
+    deps.recordError('recall 旧快照容器迁移探测失败，沿用旧位: ' + String(error))
+    return homedir
+  }
+}
+
+// 失败清扫脚本输出解读（M3 纯逻辑，供单测；cleanupAfterGitFailure 消费）。
+// killOrphansScript 的三级出口：CLEANUP_OTHER_INSTANCE <pid> = 检测到另一个
+// 活实例正在使用同一快照库（心跳有效且进程存活），清扫已让路；
+// CLEANUP_SKIPPED_FRESH_LOCK = 存在 5 分钟内的新锁（疑似 git 操作进行中），
+// 清扫未触碰；CLEANUP_DONE = 原有清扫路径执行完毕。解析按标记行匹配，
+// 与模板输出逐字对应（改标记必须两侧同步）。
+export function parseCleanupResult(out) {
+  const m = String(out || '').match(/CLEANUP_OTHER_INSTANCE\s+(\d+)/)
+  if (m) return { otherPid: parseInt(m[1], 10), skippedFresh: false }
+  if (String(out || '').indexOf('CLEANUP_SKIPPED_FRESH_LOCK') >= 0) return { otherPid: null, skippedFresh: true }
+  return { otherPid: null, skippedFresh: false }
+}
+
+// rename 步 ENOENT 判定（模块级纯逻辑，单测直接覆盖）：POSIX mv 与
+// pwsh Move-Item 的「目标不存在」文案集合，且错误必须提到 tmp 文件名
+//（basename）——只认 rename 步的错误形态，误吞面最小。
+export function isTmpConsumedError(error, basename) {
+  const s = String(error || '')
+  if (!basename || s.indexOf(basename) < 0) return false
+  return /No such file/i.test(s) || /does not exist/i.test(s) || /cannot find path/i.test(s)
+}
+
 export function createRuntime(ctx, config) {
   const shell = ctx.shell
   const sessions = ctx.sessions
@@ -53,9 +117,19 @@ export function createRuntime(ctx, config) {
   // 最近错误环形缓冲：Host 侧所有失败原本只进 console.error（宿主进程
   // 日志，用户在页面上不可见），这里留最近 20 条经 /api/recall/status
   // 下发给设置页展示。同时转发 console.error 保持原有宿主日志不变。
+  // 尾部去重（issue #11）：环境性错误随每条消息重复抛出，逐条 push 会把
+  // 20 条环形缓冲刷成同一条目、console.error 同步刷屏，其他诊断信息全被
+  // 挤掉。相邻重复只更新 time/count——间隔其他错误的重复仍新建条目，错误
+  // 时序不丢；kind 随条目富集（classifyEnvError），供 status 端点机器分流。
   function recordError(text) {
     const message = String(text)
-    state.errors.push({ time: Date.now(), message })
+    const last = state.errors[state.errors.length - 1]
+    if (last && last.message === message) {
+      last.time = Date.now()
+      last.count += 1
+      return
+    }
+    state.errors.push({ time: Date.now(), message, count: 1, kind: classifyEnvError(message) })
     if (state.errors.length > ERROR_BUFFER_MAX) state.errors.splice(0, state.errors.length - ERROR_BUFFER_MAX)
     console.error(message)
   }
@@ -67,8 +141,10 @@ export function createRuntime(ctx, config) {
   // 基底走 probeHomeScript + Node 侧推导；常量与转义工具不承载命令）。
   ;(function checkScriptParity() {
     // fileWriteCmd 仅 pwsh 版存在：POSIX 的文本落盘走 stdin（store.js
-    // writeTextViaShell 的 POSIX 分支不经命令行传参），不需要该模板函数
-    const SKIP = new Set(['homeDirScript', 'probeHomeScript', 'fileWriteCmd'])
+    // writeTextViaShell 的 POSIX 分支不经命令行传参），不需要该模板函数；
+    // legacyHomeMigrateScript 仅 posix 版存在：旧容器迁移是 POSIX 漂移
+    // （I24）专属的存量数据兜底，win32 无此问题
+    const SKIP = new Set(['homeDirScript', 'probeHomeScript', 'fileWriteCmd', 'legacyHomeMigrateScript'])
     const pwshKeys = Object.keys(pwshScripts).filter((k) => !SKIP.has(k) && typeof pwshScripts[k] === 'function')
     const posixKeys = Object.keys(posixScripts).filter((k) => !SKIP.has(k) && typeof posixScripts[k] === 'function')
     const missing = pwshKeys.filter((k) => posixKeys.indexOf(k) < 0)
@@ -84,7 +160,12 @@ export function createRuntime(ctx, config) {
   // home，快照被迫降级进项目目录（污染）；read-only 会话连项目都写不了，
   // 回退恢复直接失败。pwsh-sandbox / bash-sandbox 对 danger-full-access
   // 直接不约束（等价本地执行器），无沙箱后端的部署则忽略该字段，两边都成立。
-  async function runShell(command, opts) {
+  // F-G3：runShell 的元数据变体——stdout 截断可判定（官方 ShellRunResult.stdout
+  // 是 CollectedOutput{text, truncated, spillPath?}，见 dsh-shell 与
+  // dsh-subprocess 的 lib/types/types.d.ts；截断时 text 只剩流尾部）。需要
+  // 「解析完整 stdout」的调用方（loadIndex）用它区分「读截断」与「内容损坏」；
+  // 其余调用方继续用 runShell 拿纯文本，签名不变。
+  async function runShellMeta(command, opts) {
     const sp = ctx.get('sandboxPolicy')
     const spec = shell.resolve({
       // 编码前导：pwsh 侧统一 UTF-8 输出（中文机器 GBK 代码页不再乱码）；
@@ -108,7 +189,14 @@ export function createRuntime(ctx, config) {
       const err = ((res && res.stderr && res.stderr.text) || '').trim() || ('exit ' + String(res.exitCode))
       throw new Error(err.slice(0, 1500))
     }
-    return out
+    return {
+      text: out,
+      truncated: Boolean(res && res.stdout && res.stdout.truncated),
+    }
+  }
+
+  async function runShell(command, opts) {
+    return (await runShellMeta(command, opts)).text
   }
 
   // 从脚本文本提取影子仓库 git-dir：两套模板的 git 命令脚本都以
@@ -129,7 +217,13 @@ export function createRuntime(ctx, config) {
     const gitDir = extractGitDir(command)
     if (!gitDir) return
     try {
-      await runShell(scripts.killOrphansScript(gitDir), { timeoutMs: 60000, stdoutMaxBytes: 4096 })
+      const out = await runShell(scripts.killOrphansScript(gitDir), { timeoutMs: 60000, stdoutMaxBytes: 4096 })
+      // M3：清扫让路的两种情形都值得一条记录——前者把 issue #11 的「疑似
+      // 多实例」升级为「确认」（点名 PID），后者解释了环境为何没有被自动
+      // 清理。recordError 的尾部去重保证逐消息重复失败不会刷屏。
+      const r = parseCleanupResult(out)
+      if (r.otherPid !== null) recordError('recall 检测到另一个 DSH 实例（PID ' + r.otherPid + '）正在使用此快照库，失败清扫已让路：未杀进程、未动锁')
+      else if (r.skippedFresh) recordError('recall 检测到 5 分钟内的新锁文件，疑似 git 操作正在进行，失败清扫已让路（锁陈旧后会自动清理）')
     } catch (error) { /* best-effort：清扫失败不影响原始错误的抛出 */ }
   }
 
@@ -210,8 +304,11 @@ export function createRuntime(ctx, config) {
 
   // POSIX：shell 侧只探 bash env 里显式的 $DSH_HOME（DSH 执行器洗刷
   // DSH_* 变量后通常为空）；为空时依次回退 Node 主进程的 DSH_HOME
-  // （宿主进程 env，用户导出可见）与 os.homedir()。哈希用 Node crypto
-  // 统一算，规避 Linux sha256sum / macOS shasum 的二选一移植成本。
+  // （宿主进程 env，用户导出可见）与 os.homedir()（补 /.dsh 层，见
+  // selectPosixHomeBase 的 I24 注释）。哈希用 Node crypto 统一算，规避
+  // Linux sha256sum / macOS shasum 的二选一移植成本。三档选择与旧容器
+  // 迁移编排都委托模块级纯函数（resolvePosixHomeBase），本方法只负责探测
+  // 输入与结果缓存（迁移随缓存每进程至多跑一次）。
   async function posixHomeBaseResolve() {
     if (state.posixHomeBase === null) {
       let probed = ''
@@ -220,7 +317,10 @@ export function createRuntime(ctx, config) {
       } catch (error) {
         probed = ''
       }
-      state.posixHomeBase = probed || process.env.DSH_HOME || os.homedir()
+      state.posixHomeBase = await resolvePosixHomeBase(
+        { runShell, scripts, recordError },
+        { probed, envHome: (process.env && process.env.DSH_HOME) || '', homedir: os.homedir() }
+      )
     }
     return state.posixHomeBase
   }
@@ -362,45 +462,83 @@ export function createRuntime(ctx, config) {
     }
   }
 
-  // 任意长度文本落盘（index.json / exclude.txt 共用）：win32 走 base64
+  // 任意长度文本落盘（index.json / exclude.txt 共用），原子写（H2）：
+  // 先写 <file>.tmp 再 rename 替换目标——多块序列中途崩溃最多留一个无害
+  // 的 .tmp 残留（下次写覆盖），绝不会留下截断 JSON。win32 走 base64
   // 分块内联（每块 20000 字符，规避 Windows 命令行 32767 上限——DSH 的
   // pwsh 执行器把命令串作为 -Command 的单个 argv 元素 spawn，快照攒到
   // 几百条就超限），首块覆盖、续块追加；POSIX 用官方 ShellExecRequest
-  // 的 stdin 契约字段直写全文，不经命令行传参，天然没有 argv 上限。
+  // 的 stdin 契约字段直写全文到 tmp，不经命令行传参，天然没有 argv 上限。
   // 空内容也落一次写（清空配置/空索引是合法状态），所以 base64 为空串
-  // 时仍发一块空 piece，而不是整段跳过留下旧文件。
+  // 时仍发一块空 piece，而不是整段跳过留下旧文件。rename 是同卷 O(1)
+  // 元数据操作，索引写频率为每消息一次，额外开销可忽略。
+  // rename 步的 ENOENT 容忍（WSL 双实例实弹发现）：每实例写同一个
+  // <file>.tmp 路径，并发时一方 rename 把 tmp 消费掉，另一方 rename 报
+  // 「No such file / does not exist」。容忍是安全的，因为能走到 rename
+  // 的前提是写侧（分块 Add-Content / cat）已完整成功——任一写步失败都
+  // 在写侧直接抛（POSIX set -e / pwsh EAP=Stop），进不到这里；所以此刻
+  // tmp 消失只可能是同伴先把完整内容 rename 到了目标——本侧写语义已被
+  // 达成。Windows 侧「偶发一次 Move-Item: index.json.tmp does not
+  // exist」即同根。不进 recordError（用户错误列表刷屏正是要消除的
+  // 症状），console.error 留诊断痕迹；其余错误原样抛出。
+  async function renameTmpQuietly(tmp, file) {
+    try {
+      await runShell(scripts.renameFileCmd(tmp, file), { stdoutMaxBytes: 4096 })
+    } catch (error) {
+      const basename = tmp.slice(tmp.lastIndexOf(SEP) + 1)
+      if (isTmpConsumedError(error, basename)) {
+        console.error('recall writeTextViaShell: ' + basename + ' 已被并发写者 rename 消费，视同成功')
+        return
+      }
+      throw error
+    }
+  }
+
   async function writeTextViaShell(file, text) {
     const body = String(text == null ? '' : text)
+    const tmp = file + '.tmp'
     if (isWin) {
       const b64 = Buffer.from(body, 'utf8').toString('base64')
       const chunks = b64 ? b64.match(/.{1,20000}/g) : ['']
       let first = true
       for (const chunk of chunks) {
         const piece = "[Text.Encoding]::UTF8.GetString([Convert]::FromBase64String('" + chunk + "')) | "
-        await runShell(scripts.fileWriteCmd(file, piece, first), { stdoutMaxBytes: 4096 })
+        await runShell(scripts.fileWriteCmd(tmp, piece, first), { stdoutMaxBytes: 4096 })
         first = false
       }
+      await renameTmpQuietly(tmp, file)
     } else {
-      await runShell('cat > ' + scripts.psq(file), { stdin: body, stdoutMaxBytes: 4096 })
+      await runShell('cat > ' + scripts.psq(tmp), { stdin: body, stdoutMaxBytes: 4096 })
+      await renameTmpQuietly(tmp, file)
     }
   }
 
   // 建立影子仓库（幂等：gitReady 命中后直接跳过，省掉每条消息一次的
   // config/exclude 重写）。同时回读 gc.stamp 种子化 gc 节流：让「上次 gc
   // 时间」跨重启续存，避免天天重启的机器每开机都来一次全量 gc。
+  // 返回 {ok, error}（M1-D2）：失败原因必须传出——captureSnapshot 要把它
+  // 分类成 snapFeedback 的可行动提示（此前吞成布尔，客户端空轮询 20 次、
+  // 用户零感知，issue #11 主线缺口）；init/预热调用方忽略返回值，不受形状
+  // 变化影响。
   async function ensureGit(root, store) {
-    if (state.gitReady.has(store.git)) return true
+    if (state.gitReady.has(store.git)) return { ok: true }
     const gitExe = await resolveGit()
-    if (!gitExe) return false
+    if (!gitExe) {
+      // git 缺失分支：原先静默 return false，连 recordError 都没有（用户
+      // 重启也查不到原因的盲区）。进错误环靠上方尾部去重天然免刷屏。
+      const error = '未检测到 git CLI，快照不可用'
+      recordError('recall ensureGit: ' + error + '：请安装 git 或检查其是否在 PATH 中')
+      return { ok: false, error }
+    }
     try {
       const out = scripts.stripBom(await runShell(scripts.ensureGitScript(store, gitExe, config.baseExcludes), { stdoutMaxBytes: 4096 }))
       state.gitReady.add(store.git)
       const m = out.match(/GIT_OK\s+(\d+)/)
       state.gcLastAt.set(store.git, m ? parseInt(m[1], 10) * 1000 : Date.now())
-      return true
+      return { ok: true }
     } catch (error) {
       recordError('recall ensureGit failed: ' + String(error))
-      return false
+      return { ok: false, error: String(error) }
     }
   }
 
@@ -412,5 +550,5 @@ export function createRuntime(ctx, config) {
     runShell(scripts.legacyRmScript(root + SEP + '.dsh-recall-snapshots'), { timeoutMs: 120000, stdoutMaxBytes: 4096 }).catch(() => {})
   }
 
-  return { state, isWin, scripts, recordError, runShell, writeTextViaShell, resolveRoot, resolveGit, homeDirFor, resolveHomeContainer, resolveStore, storeFromDir, tryUpgradeToHome, ensureGit, cleanupLegacy }
+  return { state, isWin, scripts, recordError, runShell, runShellMeta, writeTextViaShell, resolveRoot, resolveGit, homeDirFor, resolveHomeContainer, resolveStore, storeFromDir, tryUpgradeToHome, ensureGit, cleanupLegacy, cleanupAfterGitFailure }
 }

package/package.json

@@ -1,61 +1,65 @@
-{
-  "name": "dsh-recall-plugin",
-  "version": "2.0.0",
-  "description": "DSH 消息撤回插件：在用户消息气泡旁加「撤回」按钮，把项目文件（独立影子 git 仓库快照）与对话历史（官方 fork）一并回退到该消息发送之前。",
-  "type": "module",
-  "repository": {
-    "type": "git",
-    "url": "git+https://github.com/limbo947/dsh-recall-plugin.git"
-  },
-  "publishConfig": {
-    "access": "public"
-  },
-  "main": "lib/index.js",
-  "exports": {
-    ".": "./lib/index.js",
-    "./client": "./lib/client.js",
-    "./package.json": "./package.json"
-  },
-  "files": [
-    "lib",
-    "cordis.patch.yml",
-    "README.md",
-    "README.en.md",
-    "CHANGELOG.md",
-    "LICENSE"
-  ],
-  "engines": {
-    "node": ">=20"
-  },
-  "scripts": {
-    "test": "vitest run tests/unit",
-    "test:probe": "vitest run tests/probe"
-  },
-  "dsh": {
-    "bundle": {
-      "patch": "./cordis.patch.yml"
-    },
-    "client": {
-      "platform": "web",
-      "inject": [
-        "@deepseek-ai/dsh-client-web-react"
-      ]
-    }
-  },
-  "peerDependencies": {
-    "@deepseek-ai/cordis": "^4.0.1",
-    "@deepseek-ai/dsh-client-web-react": "^0.1.1-rc.2",
-    "@deepseek-ai/dsh-host-webserver": "^0.1.1-rc.2",
-    "@deepseek-ai/dsh-sandbox-policy": "^0.1.1-rc.2",
-    "@deepseek-ai/dsh-session": "^0.1.1-rc.2",
-    "@deepseek-ai/dsh-session-query": "^0.1.1-rc.2",
-    "@deepseek-ai/dsh-settings": "^0.1.1-rc.2",
-    "@deepseek-ai/dsh-shell": "^0.1.1-rc.2",
-    "@deepseek-ai/schemastery": "^3.18.1",
-    "react": "^18.2.0"
-  },
-  "license": "MIT",
-  "devDependencies": {
-    "vitest": "^4.1.11"
-  }
-}
+{
+  "name": "dsh-recall-plugin",
+  "version": "2.1.1",
+  "description": "DSH 消息撤回插件：在用户消息气泡旁加「撤回」按钮，把项目文件（独立影子 git 仓库快照）与对话历史（官方 fork）一并回退到该消息发送之前。",
+  "type": "module",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/limbo947/dsh-recall-plugin.git"
+  },
+  "publishConfig": {
+    "access": "public"
+  },
+  "main": "lib/index.js",
+  "exports": {
+    ".": "./lib/index.js",
+    "./client": "./lib/client.js",
+    "./package.json": "./package.json"
+  },
+  "files": [
+    "lib",
+    "cordis.patch.yml",
+    "README.md",
+    "README.en.md",
+    "CHANGELOG.md",
+    "LICENSE"
+  ],
+  "engines": {
+    "node": ">=20"
+  },
+  "scripts": {
+    "check:dsh": "node scripts/check-dsh-version.mjs",
+    "build": "node scripts/build-client.mjs",
+    "test": "vitest run tests/unit",
+    "test:probe": "vitest run tests/probe",
+    "verify:host": "node scripts/verify-host.mjs"
+  },
+  "dsh": {
+    "bundle": {
+      "patch": "./cordis.patch.yml"
+    },
+    "client": {
+      "platform": "web",
+      "inject": [
+        "@deepseek-ai/dsh-client-web-react"
+      ]
+    }
+  },
+  "peerDependencies": {
+    "@deepseek-ai/cordis": "^4.0.1",
+    "@deepseek-ai/dsh-client-web-react": "^0.1.1-rc.2",
+    "@deepseek-ai/dsh-host-webserver": "^0.1.1-rc.2",
+    "@deepseek-ai/dsh-sandbox-policy": "^0.1.1-rc.2",
+    "@deepseek-ai/dsh-session": "^0.1.1-rc.2",
+    "@deepseek-ai/dsh-session-query": "^0.1.1-rc.2",
+    "@deepseek-ai/dsh-settings": "^0.1.1-rc.2",
+    "@deepseek-ai/dsh-shell": "^0.1.1-rc.2",
+    "@deepseek-ai/schemastery": "^3.18.1",
+    "react": "^18.2.0"
+  },
+  "license": "MIT",
+  "devDependencies": {
+    "esbuild": "^0.28.1",
+    "vitest": "^4.1.11"
+  }
+}