dsh-better-workspace 0.9.5 → 0.9.6

package/README.md

@@ -33,7 +33,7 @@ web/                 ← 虚拟分组(不是真实目录,纯命名层级)
 
 工作区里的会话名称同样用 `/` 分层(如 `测试1/两个插件维护`)。会话分组用**次级配色且无文件夹图标**渲染,和工作区文件夹一眼可辨;会话分组可整组重命名(批量改写成员标题)。
 
-**重启后标题不回退**:DSH 重启后的冷列表只为命中投影缓存的会话携带标题——分叉出的会话与从未写过检查点的会话会暂时回退成「工作区名」显示,真实的标题其实还在会话日志里。本插件把每个会话**最近一次的真实标题**持久化在浏览器里(dsh 客户端 store),回退窗口内直接用记忆标题渲染,`/` 分级分组照常成立;打开会话后官方数据到位即自动纠正。
+**重启后标题不回退**:DSH 重启后的冷列表只为命中投影缓存的会话携带标题——分叉出的会话与从未写过检查点的会话会暂时回退成「工作区名」显示,真实的标题其实还在会话日志里。本插件把每个会话**最近一次的真实标题**持久化在浏览器里(带防抖的本地缓存,变更合并整写一次),回退窗口内直接用记忆标题渲染,`/` 分级分组照常成立;打开会话后官方数据到位即自动纠正。
 
 **跨端手动同步(Web ↔ 桌面客户端)**:外观自定义、显式分组与开关默认保存在各设备自己的浏览器里;设置卡片新增「跨端同步」区——Web 上点「从客户端获取」、客户端上点「从 Web 获取」,可选**覆盖本设备**或**合并两端**(并集保留各自独有项),另一端的数据经宿主设置存储(~/.dsh/settings.yaml,两端共享)送达;旁边有「发送本设备数据」按钮,旧版本的历史数据首次同步推一次即可,之后每次修改都会自动写入宿主、对端随手可取。折叠状态仍保持每端独立。
 

package/README_EN.md

@@ -31,7 +31,7 @@ Right-click any folder row: new subfolder (parent prefix pre-filled), new worksp
 
 Session titles nest on `/` too (e.g. `test1/plugin maintenance`). Session groups render in a secondary color without a folder icon, so they are clearly distinct from workspace folders; a session group can be renamed as a whole (rewrites member titles).
 
-**No title regression after a restart**: the cold list right after a DSH restart only carries titles for sessions that hit the persisted projection cache — fork-born and never-checkpointed sessions temporarily fall back to the workspace *basename*, even though the real title still lives in the session log. This plugin remembers each session's **last real wire title** in the browser (dsh client store) and renders from memory during that window, so `/` grouping keeps working; the moment a session opens, the official data takes over and self-heals.
+**No title regression after a restart**: the cold list right after a DSH restart only carries titles for sessions that hit the persisted projection cache — fork-born and never-checkpointed sessions temporarily fall back to the workspace *basename*, even though the real title still lives in the session log. This plugin remembers each session's **last real wire title** in the browser (a debounced local cache: real changes coalesce into one write) and renders from memory during that window, so `/` grouping keeps working; the moment a session opens, the official data takes over and self-heals.
 
 **Manual cross-device sync (web ↔ desktop app)**: appearance, explicit folders, and toggles live in each device's own browser by default. The settings card adds a "Cross-device sync" section — "Pull from desktop app" on the web / "Pull from web" in the desktop app, with an **overwrite-this-device** or **merge-both-sides** mode (the union keeps each side's unique entries); the other surface's data travels through the host settings store (~/.dsh/settings.yaml, shared by both surfaces). A "Send this device's data" button next to it covers history created before this version — push once, and afterwards every edit is written to the host automatically for the other surface to pull. Expansion state stays per-device.
 

package/dsh.plugin.json

@@ -1,6 +1,6 @@
 {
   "id": "dsh-external/dsh-better-workspace",
-  "version": "0.9.5",
+  "version": "0.9.6",
   "main": "./src/index.js",
   "description": "侧边栏工作区层级树:名称中的 / 即虚拟分组(web/前端 · web/后端),添加工作区的目录流附所属分组弹窗,重命名即时重排层级,可新建空分组。Hierarchy tree for the DSH sidebar workspace list.",
   "engines": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-better-workspace",
-  "version": "0.9.5",
+  "version": "0.9.6",
   "description": "DSH web plugin: a hierarchical workspace tree for the sidebar. Workspace titles containing \"/\" group into virtual folders (web/前端 · web/后端); the add-workspace directory flow gains a parent-group popup; renames re-derive the tree; a new-folder button creates empty levels.",
   "type": "module",
   "repository": {

package/src/client.js

@@ -749,31 +749,61 @@ window.__ModuleLoader__.load({
     // last REAL wire title per session id (learned only from snapshots where
     // summary.title exists — never from displayTitle, which is basename
     // degraded in exactly the window being patched), so the tree restores
-    // the "/" grouping until the wire catches up. Same-value writes are
-    // no-ops (immer produces the same state, nothing persists); entries
-    // evict oldest-at past the hard cap. A stale remembered title (renamed
+    // the "/" grouping until the wire catches up. 0.9.6: plain module
+    // state with one debounced whole-file save per real change batch
+    // (issue #1: per-session reactive-store dispatches cost O(sessions)
+    // immer produces per snapshot tick and froze large installs). Same-
+    // value scans allocate nothing; entries evict oldest-at past the hard
+    // cap. A stale remembered title (renamed
     // elsewhere, cleared browser storage) self-heals the moment the wire
     // carries the truth again; a missing entry just leaves today's fallback.
     const TITLE_CACHE_LIMIT = 3000
     const TITLE_CACHE_KEEP = 2400
-    const createTitleCacheStore = () => storeKit.defineStore({
-      init: () => ({ byId: {} }),
-      persist: 'dsh.betterWorkspace.titles.v1',
-      actions: {
-        rememberTitle: (d, id, title, at) => {
-          if (typeof id !== 'string' || id === '' || typeof title !== 'string' || title === '') return
-          if (!d.byId) d.byId = {}
-          const prev = d.byId[id]
-          if (prev && prev.title === title) return
-          d.byId[id] = { title: title, at: typeof at === 'number' ? at : Date.now() }
-          const keys = Object.keys(d.byId)
-          if (keys.length > TITLE_CACHE_LIMIT) {
-            keys.sort((a, b) => ((d.byId[a] && d.byId[a].at) || 0) - ((d.byId[b] && d.byId[b].at) || 0))
-            for (let i = 0; i < keys.length - TITLE_CACHE_KEEP; i++) delete d.byId[keys[i]]
-          }
-        },
-      },
-    })
+    const TITLE_CACHE_KEY = 'dsh.betterWorkspace.titles.v1'
+    const titleCache = { byId: {} }
+    let titleSaveTimer = null
+    const loadTitleCache = () => {
+      try {
+        const raw = localStorage.getItem(TITLE_CACHE_KEY)
+        if (raw !== null) {
+          const parsed = JSON.parse(raw)
+          if (parsed && typeof parsed.byId === 'object' && parsed.byId !== null) titleCache.byId = parsed.byId
+        }
+      } catch (e) { /* storage unavailable/corrupt: cold start without cache */ }
+    }
+    const scheduleTitleSave = () => {
+      clearTimeout(titleSaveTimer)
+      titleSaveTimer = setTimeout(() => {
+        try { localStorage.setItem(TITLE_CACHE_KEY, JSON.stringify(titleCache)) } catch (e) {}
+      }, 200)
+    }
+    // One plain-object batch pass per snapshot (issue #1). Unchanged scans
+    // cost one loop with zero allocation; only real changes mark dirty;
+    // eviction runs once per changed batch followed by ONE debounced save.
+    const rememberAllTitles = (list) => {
+      if (!list || !list.byId) return false
+      const byId = titleCache.byId
+      const now = Date.now()
+      let changed = false
+      for (const id of Object.keys(list.byId)) {
+        const summary = list.byId[id]
+        if (!summary || summary.blank) continue
+        const title = summary.title
+        if (typeof title !== 'string' || title === '') continue
+        const prev = byId[id]
+        if (prev && prev.title === title) continue
+        byId[id] = { title: title, at: now }
+        changed = true
+      }
+      if (!changed) return false
+      const keys = Object.keys(byId)
+      if (keys.length > TITLE_CACHE_LIMIT) {
+        keys.sort((a, b) => ((byId[a] && byId[a].at) || 0) - ((byId[b] && byId[b].at) || 0))
+        for (let i = 0; i < keys.length - TITLE_CACHE_KEEP; i++) delete byId[keys[i]]
+      }
+      scheduleTitleSave()
+      return true
+    }
 
     /* ======================= host settings sync ======================= */
 
@@ -1560,7 +1590,7 @@ window.__ModuleLoader__.load({
       const archivedSet = React.useMemo(() => new Set(archivedSessionIds), [archivedSessionIds])
       const subCounts = React.useMemo(() => subagentRunningCounts(list ? list.byId : {}), [list ? list.byId : null])
       // Last-known real titles for the cold-restart fallback window (see
-      // createTitleCacheStore). Read per render via getSnapshot: no
+      // the title-cache block). Read per render via getSnapshot: no
       // subscription, because a cache write only happens when summary.title
       // exists — at that moment the row already renders the wire truth and
       // no re-render is wanted.
@@ -1572,21 +1602,13 @@ window.__ModuleLoader__.load({
       // Learn real titles from every snapshot: remember exactly what the
       // wire carried (summary.title), never displayTitle — that is already
       // basename-degraded in the very window this patches. Blank rows are
-      // provisional New Sessions and never learn. rememberTitle itself
-      // no-ops on unchanged values, so the loop is cheap when idle.
+      // provisional New Sessions and never learn. One plain-object batch
+      // call per snapshot (issue #1): unchanged scans cost one loop with
+      // zero allocation, real changes schedule one debounced save.
       React.useEffect(() => {
         if (!list || !list.byId) return
-        const remember = titleCacheRef && titleCacheRef.actions
-          ? titleCacheRef.actions.rememberTitle
-          : null
-        if (typeof remember !== 'function') return
-        const now = Date.now()
-        for (const id of Object.keys(list.byId)) {
-          const summary = list.byId[id]
-          if (!summary || summary.blank) continue
-          const title = summary.title
-          if (typeof title !== 'string' || title === '') continue
-          remember(String(id), title, now)
+        if (titleCacheRef && typeof titleCacheRef.rememberAllTitles === 'function') {
+          titleCacheRef.rememberAllTitles(list)
         }
       }, [list ? list.byId : null])
 
@@ -2685,11 +2707,11 @@ window.__ModuleLoader__.load({
       // same persisted state (expansion, folder list, prefs, styling).
       const viewStore = createViewStore()
 
-      // Cold-restart title fallback instance (see createTitleCacheStore).
-      // getSnapshot() during render is intentional: hydration reads
-      // localStorage synchronously at create(), so the first render after a
-      // restart already sees every remembered title.
-      titleCacheRef = createTitleCacheStore().create()
+      // Cold-restart title fallback (see the title-cache block). Hydration
+      // reads localStorage synchronously here, before the first render, so a
+      // restart's first tree already shows every remembered title.
+      loadTitleCache()
+      titleCacheRef = { getSnapshot: () => titleCache, rememberAllTitles }
 
       // Cross-device settings scope (manual sync, see the host-settings-sync
       // block). Binding is best-effort: a missing/failed bind leaves the