dsh-mobile-plugin 0.1.73 → 0.1.74

package/lib/session-cleanup.cjs

@@ -128,20 +128,30 @@ async function deleteSessionData(sessionIds) {
 }
 
 /**
- * 把马上要从磁盘上删掉的 id 从宿主归档集合里摘掉。
+ * 把已经从磁盘删掉的 id 从宿主归档集合里擦掉（删除路径的最后一步，best-effort）。
  *
  * 宿主不会自己修剪这份集合：日志删了它照样留着那些 id，于是「已归档」里是一排点不开的
- * 死行，而且每清一次就多几条、只增不减。摘的动作放在删**之前**：摘失败就什么都没丢，
- * 删失败则退化成「已取消归档但没删掉」（会话重新可见、还能再清一次）——比「删干净了却
- * 还挂在归档集合里」好收场。宿主对本来就未归档的 id 是 no-op。
+ * 死行，而且每清一次就多几条、只增不减。它排在 `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') throw new Error('会话清理能力不可用：workspace.unarchiveSession')
+  if (typeof unarchive !== 'function') return failed
   for (const sessionId of sessionIds) {
-    const { error } = unwrap(await unarchive({ rpcId, method: 'workspace.unarchiveSession', payload: { sessionId } }))
-    if (error) throw new Error(`取消归档 ${sessionId} 失败：${error.message || error.code || '未知错误'}`)
+    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) {
@@ -178,8 +188,11 @@ async function cleanupSessions(ctx, args) {
   }
   if (args.confirm !== true) throw new Error('永久删除需要 confirm: true')
   const ids = selected.map(item => item.sessionId)
-  await forgetArchivedSessions(ctx, ids, 'dsh-mobile-plugin-session-cleanup')
+  // 与移动端那条路由同一口径：**先删日志、再擦归档集合**。反过来会在链路中途失败时
+  // 留下「名字已从归档集合消失、日志还在」的会话（看着像只是被移出了归档，得重启宿主
+  // 才恢复）；先删则日志一没就是真的删掉了，立刻生效。擦集合是收尾，失败不改变结果。
   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 }
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-mobile-plugin",
-  "version": "0.1.73",
+  "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",