dsh-mobile-plugin 0.1.72 → 0.1.74

package/lib/relay/lan-proxy.js

@@ -459,7 +459,7 @@ document.getElementById('f').addEventListener('submit', async (e) => {
                   ip: suppliedDevice.ip || ip,
                 }
                 onApprove(device)
-                  .then(({ approved, token }) => {
+                  .then(({ approved, token, platform }) => {
                     if (!approved) {
                       // 与「配对码错误」返回同一响应，配对码不再可通过状态码探测。
                       pairDenied(res);
@@ -479,6 +479,9 @@ document.getElementById('f').addEventListener('submit', async (e) => {
                       deviceId: issued.deviceId,
                       deviceCredential: issued.credential,
                       credentialExpiresAt: issued.expiresAt,
+                      // 宿主自己的系统标识，不是手机的。手机把它存进设备记录，
+                      // 好让设备行在第一次 status 轮询之前就画得出图标。
+                      ...(platform ? { hostPlatform: platform } : {}),
                     }));
                   })
                   .catch((err) => {

package/lib/relay/pairing-policy.js

@@ -42,4 +42,18 @@ function evaluate(value, device) {
   }
 }
 
-module.exports = { POLICIES, DEFAULT_POLICY, normalize, evaluate }
+/**
+ * 宿主的系统标识（`process.platform` 的取值：darwin / win32 / linux …）。
+ *
+ * 与 status 那条路同源，但**送达时机完全不同**：status 要走轮询，手机刚绑好、
+ * 还没轮到第一次探测时读不到；图标却要在设备行出现的那一刻就画得出来。
+ * 配对是一台设备「是什么」这个事实唯一的确定时刻，所以在这里就把它交给手机，
+ * 由手机存进设备记录。轮询回来的平台值只当补充（老设备补登记用），不再承担首次绘制。
+ *
+ * @returns {{ platform: string }}
+ */
+function hostPlatform() {
+  return { platform: process.platform }
+}
+
+module.exports = { POLICIES, DEFAULT_POLICY, normalize, evaluate, hostPlatform }

package/lib/relay/tunnel.js

@@ -381,12 +381,17 @@ class RelayTunnel {
     if (this.connected) this.sendJson(T.updatePairing, 0, { pairingCode: code });
   }
 
-  /** 回传桌面端弹窗授权结果：approved + 桌面签发的移动 token（拒绝时无 token）。 */
-  sendPairConfirm(requestId, approved, token) {
+  /**
+   * 回传桌面端弹窗授权结果：approved + 桌面签发的移动 token（拒绝时无 token）。
+   * `hostPlatform` 是**宿主自己的** `process.platform`，经中转原样回给手机，
+   * 让配对那一刻就把「这台是什么系统」落盘（设备行不等 status 轮询）。
+   */
+  sendPairConfirm(requestId, approved, token, hostPlatform) {
     this.sendJson(T.pairConfirm, 0, {
       requestId,
       approved,
       ...(approved && token ? { token } : {}),
+      ...(approved && typeof hostPlatform === 'string' && hostPlatform ? { hostPlatform } : {}),
     });
   }
 

package/lib/routes/control-plane.js

@@ -260,7 +260,9 @@ function registerControlPlaneRoutes(deps) {
     const decision = pairingApproval.evaluate(state.pairingPolicy, body.device)
     // Audit trail: the removed prompt showed the device before letting it in.
     process.stderr.write(`[dsh-mobile-plugin] pairing ${decision.approved ? 'approved' : 'denied'} policy=${decision.policy} device=${decision.device}${decision.ip ? ` ip=${decision.ip}` : ''}\n`)
-    return sendJson(res, 200, { data: { approved: decision.approved, reason: decision.reason } })
+    // `hostPlatform` 一并回给中转：手机要靠它把「这台宿主是什么系统」在配对那一刻就落盘，
+    // 不能等到第一次 status 轮询（那之前设备行画不出图标）。被拒时不给，没有意义。
+    return sendJson(res, 200, { data: { approved: decision.approved, reason: decision.reason, ...(decision.approved ? pairingApproval.hostPlatform() : {}) } })
   } })
   // 控制面状态里额外带上面板页的绝对地址：客户端半边（lib/client.js）那一项现在只是
   // 一个「打开配置页面」的入口，链接必须是绝对的（原因见 panelUrlOf）。地址只在这里

package/lib/routes/mobile-api.js

@@ -162,6 +162,7 @@ function registerMobileRoutes(deps) {
   route('/api/mobile/v1/directory/create', workspaces.handleCreateDirectory)
   route('/api/mobile/v1/workspace/create', workspaces.handleCreateWorkspace)
   route('/api/mobile/v1/workspace/delete', workspaces.handleDeleteWorkspace)
+  route('/api/mobile/v1/workspace/move', workspaces.handleMoveWorkspace)
   route('/api/mobile/v1/workspace/move-session', workspaces.handleMoveSession)
   // App 的 `renameWorkspace()` / `archiveSession()` 早就在打这两个地址
   // （`app/src/services/apiPaths.ts` 的 workspaceRename / workspaceArchiveSession），
@@ -173,8 +174,8 @@ function registerMobileRoutes(deps) {
   route('/api/mobile/v1/workspace/unarchive-session', workspaces.handleUnarchiveSession)
   route('/api/mobile/v1/workspace/pin-session', workspaces.handlePinSession)
   route('/api/mobile/v1/workspace/unpin-session', workspaces.handleUnpinSession)
-  // 「清理所有已归档的对话」（侧栏归档视图的文字按钮）。与 session_cleanup 工具同一口径：
-  // 只删已归档且不活跃的会话，且 body 必须带 confirm: true（磁盘删除不可恢复）。
+  // 「清理所有已归档的对话」（侧栏归档视图的文字按钮）：按 workspace.list 的归档集合全量清理，
+  // 且 body 必须带 confirm: true（磁盘删除不可恢复）。
   route('/api/mobile/v1/workspace/clear-archived', archiveClean.handleClearArchived)
   // 桌面端 `/export` 其实是两步：命令本身只留一行 transcript 标记，真正的下载由
   // @deepseek-ai/dsh-session-log-export 在浏览器侧监听 command/executed 后拉

package/lib/session-cleanup.cjs

@@ -3,17 +3,34 @@
  *
  * 两个入口共用 deleteSessionData：DSH agent 工具 registerSessionCleanupTool（action=delete
  * 必须 confirm:true 且只针对已归档）与移动 API /workspace/clear-archived（archive-clean.js）。
+ *
+ * 认哪份文件、怎么读 id 都以宿主自己的规则为准，不按目录名猜：
+ *  - 文件名 = `session[.vN].jsonl[.zstd]`（宿主 `dsh-session-format` 的
+ *    CANONICAL_LOG_FILENAME + compressionSuffix：0 代不带 vN，压缩时才带 .zstd）。
+ *    曾经只认 `session.jsonl.zstd`，于是 0.2.0 真在用的 `session.v4.jsonl.zstd`
+ *    一个都认不到 —— 表现是「清理已归档」恒回 `deleted: 0`，静默不删。
+ *  - 删之前先从宿主归档集合里摘掉这些 id：宿主不会自己修剪，删完还留着就是「已归档」
+ *    里一排点不开的死行，且每清一次只增不减。
+ *  - 解压用 Node 自带的 zstd（DSH 自己就是这么读写日志的，宿主并不需要系统 zstd
+ *    命令），不再 shell 出去：没装 zstd 命令的机器上它一缺就是 readHeader 静默返回
+ *    undefined，整条删除路径空转。id 仍然取自日志第一行那条 header。
  */
 'use strict'
 
 const fs = require('node:fs')
 const path = require('node:path')
-const { execFileSync } = require('node:child_process')
+const zlib = require('node:zlib')
 
 const DEFAULT_OLDER_THAN_MINUTES = 30
 const MINUTES = 60 * 1000
 
-const { envelopeValue } = require('./mobile/envelope.js')
+// 宿主认可的会话日志名。非规范名（迁移中途的临时文件、大写、前导零、带 v0）宿主自己
+// 也不认，这里同样不认 —— 认多了会把没提交的代次当会话删掉。
+const CANONICAL_LOG_FILENAME = /^session(?:\.v[1-9][0-9]*)?\.jsonl(?:\.zstd)?$/
+// 只需要第一行那条 header；给个上限，别为了读一行把整份日志解进内存。
+const HEADER_MAX_BYTES = 64 * 1024
+
+const { envelopeValue, unwrap } = require('./mobile/envelope.js')
 
 function asItems(response) {
   const value = envelopeValue(response)
@@ -48,13 +65,28 @@ function projectCount(items, cwd) {
 
 const { dshHome } = require('./shared/paths')
 
-function readHeader(file) {
+/** 读日志第一行那条 header（id 的真源）。压缩件用 Node 自带的 zstd 流式解，读到换行就停。 */
+async function readHeader(file) {
+  const source = fs.createReadStream(file)
+  const stream = file.endsWith('.zstd') ? source.pipe(zlib.createZstdDecompress()) : source
+  const chunks = []
+  let size = 0
   try {
-    const firstLine = execFileSync('zstd', ['-dc', file], { encoding: 'utf8', maxBuffer: 16384 }).split('\n', 1)[0]
-    return JSON.parse(firstLine)
+    for await (const chunk of stream) {
+      chunks.push(chunk)
+      size += chunk.length
+      const joined = Buffer.concat(chunks, size)
+      const end = joined.indexOf(0x0a)
+      if (end >= 0) return JSON.parse(joined.subarray(0, end).toString('utf8'))
+      if (size >= HEADER_MAX_BYTES) return undefined
+    }
   } catch {
     return undefined
+  } finally {
+    stream.destroy()
+    source.destroy()
   }
+  return undefined
 }
 
 function sessionFiles(root) {
@@ -65,19 +97,19 @@ function sessionFiles(root) {
     for (const entry of entries) {
       const file = path.join(directory, entry.name)
       if (entry.isDirectory()) visit(file)
-      else if (entry.name === 'session.jsonl.zstd') result.push(file)
+      else if (CANONICAL_LOG_FILENAME.test(entry.name)) result.push(file)
     }
   }
   visit(root)
   return result
 }
 
-function deleteSessionData(sessionIds) {
+async function deleteSessionData(sessionIds) {
   const ids = new Set(sessionIds)
   const removed = []
   const root = path.join(dshHome(), 'sessions')
   for (const file of sessionFiles(root)) {
-    const header = readHeader(file)
+    const header = await readHeader(file)
     if (!header || !ids.has(header.id)) continue
     try {
       fs.rmSync(path.dirname(file), { recursive: true, force: true })
@@ -95,6 +127,33 @@ function deleteSessionData(sessionIds) {
   return removed
 }
 
+/**
+ * 把已经从磁盘删掉的 id 从宿主归档集合里擦掉（删除路径的最后一步，best-effort）。
+ *
+ * 宿主不会自己修剪这份集合：日志删了它照样留着那些 id，于是「已归档」里是一排点不开的
+ * 死行，而且每清一次就多几条、只增不减。它排在 `deleteSessionData` **之后**：日志没删成
+ * 前先擦集合，中途失败就会留下「已取消归档、日志还在」的会话——用户看到的是清理只把会话
+ * 移出了归档，还要重启宿主才恢复。先删后擦则日志一没就是真的删掉了，立刻生效；擦失败最多
+ * 留一行死行，重试即可。宿主对本来就未归档的 id 是 no-op。
+ *
+ * 只传**真删掉**的 id：没匹配到日志的还留着日志，擦掉它的名字只会让那一行消失而没有东西被删。
+ *
+ * 返回擦失败的 id（`failed`），**不抛异常**：日志已经删成功，擦集合坏了不该让整条清理
+ * 报错——那等于告诉用户「没删掉」，而事实上已经删掉了。
+ */
+async function forgetArchivedSessions(ctx, sessionIds, rpcId) {
+  const failed = []
+  const unarchive = ctx && ctx.apiProxy && ctx.apiProxy.workspace ? ctx.apiProxy.workspace.unarchiveSession : undefined
+  if (typeof unarchive !== 'function') return failed
+  for (const sessionId of sessionIds) {
+    try {
+      const { error } = unwrap(await unarchive({ rpcId, method: 'workspace.unarchiveSession', payload: { sessionId } }))
+      if (error) failed.push(sessionId)
+    } catch { failed.push(sessionId) }
+  }
+  return failed
+}
+
 async function cleanupSessions(ctx, args) {
   const list = ctx.apiProxy?.sessions?.list
   const workspaceList = ctx.apiProxy?.workspace?.list
@@ -129,7 +188,11 @@ async function cleanupSessions(ctx, args) {
   }
   if (args.confirm !== true) throw new Error('永久删除需要 confirm: true')
   const ids = selected.map(item => item.sessionId)
-  const deletedIds = deleteSessionData(ids)
+  // 与移动端那条路由同一口径：**先删日志、再擦归档集合**。反过来会在链路中途失败时
+  // 留下「名字已从归档集合消失、日志还在」的会话（看着像只是被移出了归档，得重启宿主
+  // 才恢复）；先删则日志一没就是真的删掉了，立刻生效。擦集合是收尾，失败不改变结果。
+  const deletedIds = await deleteSessionData(ids)
+  await forgetArchivedSessions(ctx, deletedIds, 'dsh-mobile-plugin-session-cleanup')
   return { action: args.action, cwd, selected: ids.length, archived: 0, deleted: deletedIds.length, failed: ids.length - deletedIds.length, sessionIds: deletedIds }
 }
 
@@ -156,4 +219,4 @@ function registerSessionCleanupTool(ctx) {
   }))
 }
 
-module.exports = { cleanupSessions, registerSessionCleanupTool, deleteSessionData }
+module.exports = { cleanupSessions, registerSessionCleanupTool, deleteSessionData, forgetArchivedSessions }

package/lib/sidecar-facade-worker.cjs

@@ -211,7 +211,7 @@ function startTunnel() {
       try {
         const result = await parentJson('/api/relay/internal/approval', { device: message.device || {} })
         if (tunnel !== current) return
-        if (result.approved) current.sendPairConfirm(message.requestId, true, core.signLanToken(config.lanSecret, config.instanceId, core.MOBILE_BEARER_TTL_SEC))
+        if (result.approved) current.sendPairConfirm(message.requestId, true, core.signLanToken(config.lanSecret, config.instanceId, core.MOBILE_BEARER_TTL_SEC), result.platform)
         else current.sendPairConfirm(message.requestId, false)
       } catch { if (tunnel === current) current.sendPairConfirm(message.requestId, false) }
     },
@@ -234,7 +234,11 @@ function startLan() {
     onApprove: async (device) => {
       try {
         const result = await parentJson('/api/relay/internal/approval', { device: device || {} })
-        return { approved: Boolean(result.approved), token: result.approved ? core.signLanToken(config.lanSecret, config.instanceId, core.MOBILE_BEARER_TTL_SEC) : null }
+        return {
+          approved: Boolean(result.approved),
+          token: result.approved ? core.signLanToken(config.lanSecret, config.instanceId, core.MOBILE_BEARER_TTL_SEC) : null,
+          platform: typeof result.platform === 'string' ? result.platform : null,
+        }
       } catch { return { approved: false, token: null } }
     },
   })

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-mobile-plugin",
-  "version": "0.1.72",
+  "version": "0.1.74",
   "description": "Mobile API (/api/mobile/v1) and relay client for the DSH mobile app — desktop profile build (0.2.0-rc.2 compatible, peers resolved by the host)",
   "license": "MIT",
   "type": "module",