dsh-better-workspace 0.9.0 → 0.9.1

package/README.md

@@ -61,7 +61,7 @@ web/                 ← 虚拟分组(不是真实目录,纯命名层级)
 - **文件夹式收起**:点工作区行 = 收起/展开该工作区的全部会话(收起后行上保留会话数角标);会话分组同样可收起;搜索时自动全部展开。
 - **拖拽还原原版能力**:工作区行可拖拽——拖到另一工作区行上/下半即调整顺序(写回宿主 registry 序),拖到分组行则移动进该分组;会话行可拖拽——同工作区拖到另一会话上/下调整顺序,拖到会话子分组即移动进该分组。显示顺序 = 宿主手动序,拖拽结果立即生效。拖拽工作区期间,单链合并显示的行(如 工作/A)临时展开回文件夹树——路径里的每一级都是可投放的分组行,松手后自动合并回去。
 - **不丢原生能力**:会话行(打开/重命名/分叉/归档)、每工作区「新会话」(点 + 总是先展开该工作区,让新会话立刻可见)、当前会话高亮、官方状态灯(蓝色运行环 / 琥珀待交互 / 绿色完成提醒)、**状态呼吸灯**(折叠藏起的状态灯向外冒泡:工作区/分组行图标按状态色呼吸发光——琥珀待交互/绿完成,自定义过发光的标题一起呼吸;运行中只在会话行本层蓝色呼吸,不外渗;可在设置关闭)、**活跃定时任务角标**(有活动 Schedule 的会话显示闹钟图标,与官方行一致)、未分组会话兜底、中英双语跟随界面语言。
-- **URL 标题感知**:会话/工作区标题里的 URL(如 `https://host/api/v1/x`)按 URL 语义分层——域名与协议合成一段、路径各段自成层级,同前缀 URL 自动聚到同一分组下,不再被撕成伪分组。
+- **URL/斜杠标题不拆层**:宿主自动生成的会话标题落地时若含 `/`(如复述了 URL),整串自动包上 `“”` 原样显示,绝不解析成层级;引号内(含手动添加)的 `/` 永不切分;无引号但含 URL 的旧标题也有平铺兜底。你手动命名的 `/` 分组标题不受影响。
 - **对话空态页同样生效**:会话空态页的「添加工作区」菜单走的也是本插件的拾取交互,同样带所属分组弹窗。
 
 ## 安装

package/README_EN.md

@@ -59,7 +59,7 @@ Session titles nest on `/` too (e.g. `test1/plugin maintenance`). Session groups
 - **Folder-style collapse**: clicking a workspace row collapses/expands all of its sessions (the row keeps a session counter when collapsed); session groups collapse too; search force-expands everything.
 - **Native drag restored**: drag a workspace row above/below another to reorder (writes back the host registry order) or onto a folder row to move it into that group; drag session rows to reorder within the same workspace or onto a session group to move. Display order = host manual order, drag results are visible immediately. While dragging a workspace, merged single-chain rows (e.g. work/A) temporarily re-expand into folder rows — every level of the path becomes a drop target, and chains merge back when the drag ends.
 - **Native capabilities kept**: open/rename/fork/archive sessions, per-workspace new-session (the + button always expands the workspace first so the new session is immediately visible), current-session highlight, official status dots (blue running ring / amber pending / green done), **status breathing lights** (dots hidden by collapse bubble outward: workspace/folder rows breathe on their icon in the status color — amber pending / green done, custom-glow labels breathe along; running breathes blue only on the session row itself and never relays; off-able in settings), **active-schedule badge** (alarm clock icon for sessions with active Schedule records, matching the official rows), ungrouped-session fallback, zh/en localization.
-- **URL-aware titles**: URLs in session/workspace titles (e.g. `https://host/api/v1/x`) split the way URLs mean it — scheme://host as one segment, each path segment a level — so same-prefix URLs nest together instead of shredding into pseudo-folders.
+- **Slash-bearing titles never nest from URLs**: when a host-generated session title lands containing `/` (e.g. echoing a pasted URL), it is auto-wrapped in `“”` and shown verbatim — quotes (manual ones too) never split on `/`; unquoted URL tails still fall back flat. Your manually named `/` groupings are untouched.
 - **Conversation hero too**: the empty-state “Add workspace” menu uses the same picking interaction with the same group popup.
 
 ## Install

package/dsh.plugin.json

@@ -1,6 +1,6 @@
 {
   "id": "dsh-external/dsh-better-workspace",
-  "version": "0.9.0",
+  "version": "0.9.1",
   "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.0",
+  "version": "0.9.1",
   "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

@@ -217,45 +217,58 @@ window.__ModuleLoader__.load({
       return i === -1 ? s : s.slice(i + 1)
     }
     /**
-     * URL-aware segment splitting for titles. A bare "/" split shreds URLs
-     * ("https:" → empty → host → path...), so when a "://" shows up the tail is
-     * parsed as a URL and split the way URLs mean it: scheme://host is ONE
-     * segment (the authority is not a hierarchy level), each pathname segment
-     * is a level, and query/hash glue to the last segment. Consequences:
-     * several sessions sharing a URL prefix (".../v1/m", ".../v1/n") nest
-     * under the same derived folder instead of piling up as long flat rows,
-     * while "docs/see https://x/a/b" keeps the plain "docs" head. Titles whose
-     * URL-looking tail does not parse stay opaque from the last "/" before the
-     * scheme marker. Grouping is a pure projection of titles, so sessions that
-     * were previously shredded into pseudo-folders re-flow on reload.
+     * Plain-text splitting with the URL tail kept opaque: a bare "/" split
+     * shreds URLs ("https:" → empty → host → path...), so from the first
+     * "://" onward the tail is ONE leaf; text before the marker splits
+     * normally at the last "/" before it. This is the FALLBACK guard —
+     * freshly generated titles get their whole "/"-bearing title wrapped in
+     * quotes by the quote-on-land effect (see BetterBrowser), which is the
+     * primary mechanism.
      */
-    const urlSegsOf = (raw) => {
-      let u = null
-      try { u = new URL(raw) } catch { return null }
-      const host = u.protocol + '//' + u.host
-      const pathSegs = u.pathname.split('/').filter(Boolean)
-      const tail = (u.search || '') + (u.hash || '')
-      if (pathSegs.length === 0) return [host + tail]
-      if (tail !== '') pathSegs[pathSegs.length - 1] += tail
-      return [host].concat(pathSegs)
-    }
-
-    const splitTitleSegs = (title) => {
-      const s = String(title || '')
+    const splitPlainSegs = (text) => {
+      const s = String(text || '')
       const schemeAt = s.indexOf('://')
       if (schemeAt === -1) return s.split('/').map(x => x.trim()).filter(Boolean)
-      let start = schemeAt
-      while (start > 0 && /[a-zA-Z0-9+.-]/.test(s[start - 1])) start--
-      const head = s.slice(0, start).split('/').map(x => x.trim()).filter(Boolean)
-      const urlSegs = urlSegsOf(s.slice(start).trim())
-      if (urlSegs !== null) return head.concat(urlSegs)
-      // Not a parseable URL after all: opaque from the last "/" before the marker.
       const cut = s.lastIndexOf('/', schemeAt)
       const segs = (cut === -1 ? '' : s.slice(0, cut)).split('/').map(x => x.trim()).filter(Boolean)
-      const tailText = s.slice(cut + 1).trim()
-      if (tailText !== '') segs.push(tailText)
+      const tail = s.slice(cut + 1).trim()
+      if (tail !== '') segs.push(tail)
       return segs
     }
+
+    /**
+     * Title → hierarchy segments, QUOTE-AWARE: spans wrapped in “…” (or "…")
+     * are verbatim — slashes inside quotes never split. Quoted spans stay one
+     * leaf including the quotes; text between quoted spans splits through
+     * splitPlainSegs (URL-tail opaque). An unterminated opening quote makes
+     * the rest of the title one verbatim span. Grouping is a pure projection,
+     * so previously shredded titles re-flow on reload.
+     */
+    const splitTitleSegs = (title) => {
+      const s = String(title || '')
+      if (!/["“]/.test(s)) return splitPlainSegs(s)
+      const out = []
+      let plain = ''
+      let i = 0
+      while (i < s.length) {
+        const ch = s[i]
+        if (ch === '"' || ch === '“') {
+          for (const seg of splitPlainSegs(plain)) out.push(seg)
+          plain = ''
+          const close = ch === '“' ? '”' : '"'
+          const j = s.indexOf(close, i + 1)
+          const end = j === -1 ? s.length : j + 1
+          const quoted = s.slice(i, end).trim()
+          if (quoted !== '') out.push(quoted)
+          i = end
+          continue
+        }
+        plain += ch
+        i += 1
+      }
+      for (const seg of splitPlainSegs(plain)) out.push(seg)
+      return out
+    }
     const normPath = (p) => splitTitleSegs(p).join('/')
 
     /** Render a primitives icon by name; unknown names degrade to null, never crash. */
@@ -1301,6 +1314,44 @@ window.__ModuleLoader__.load({
       // source element, and Chromium cancels the whole gesture. dragEnd clears
       // the timer so a same-tick cancel never leaves a ghost drag behind.
       const wsDragArmTimer = React.useRef(null)
+      // Session ids that have EVER had a durable title this mount. The
+      // blank→titled transition marks the host's automatic title landing
+      // (LLM provider or deterministic fallback — the wire shape is the
+      // same); user renames always happen AFTER landing and never re-trigger.
+      // The FIRST snapshot only registers: titles that predate this mount may
+      // be deliberate user groupings and must never be touched.
+      const titledSeenRef = React.useRef(null)
+
+      // Quote-on-land: when a freshly landed automatic title contains "/",
+      // wrap the WHOLE title in “…” so splitTitleSegs keeps it verbatim (the
+      // AI/fallback title echoing a URL must not nest). Idempotent — quoted
+      // titles and already-seen sessions are skipped — and best-effort: a
+      // failed rename never retries (the unquoted URL-tail fallback in
+      // splitPlainSegs still keeps the display flat).
+      React.useEffect(() => {
+        if (!list || !list.byId || typeof renameSession !== 'function') return
+        let seen = titledSeenRef.current
+        const first = seen === null
+        if (first) { seen = new Set(); titledSeenRef.current = seen }
+        const fixes = []
+        for (const id of Object.keys(list.byId)) {
+          const summary = list.byId[id]
+          if (!summary || summary.blank) continue
+          const title = summary.title
+          if (!title || String(title).trim() === '') continue
+          if (seen.has(id)) continue
+          seen.add(id)
+          if (first) continue // pre-existing on load: register only
+          const text = String(title)
+          if (!text.includes('/')) continue
+          if (text.includes('“') || text.includes('"')) continue
+          fixes.push([id, '“' + text + '”'])
+        }
+        if (fixes.length === 0) return
+        Promise.resolve()
+          .then(async () => { for (const [id, next] of fixes) await renameSession(id, next) })
+          .catch(() => { /* keep the flat fallback display */ })
+      }, [list])
       const normalizedQuery = query.trim().toLowerCase()
       const now = Date.now()