dsh-claude-style 0.6.1 → 0.6.4

package/CHANGELOG.md

@@ -2,6 +2,26 @@
 
 All notable changes to `dsh-claude-style` are documented here, newest first.
 
+## [0.6.4] - 2026-09-24
+
+[中文](#cn-0.6.4) | [English](#en-0.6.4)
+
+<h3 id="cn-0.6.4">问题修复</h3>
+
+- 修复 **刚启动、账号抽屉还没展开过时，页脚里的插件条目一直留在账户行旁边**：隐藏页脚条目与把条目镜像进抽屉原本挤在同一次调用里，而抽屉没打开时这次调用会提前返回，于是宿主重新渲染出来的插件图标继续画在侧栏底部、紧挨账户行，直到抽屉被展开过一次才被标记隐藏——这正是该现象只在首次启动后出现的原因。现在隐藏条目每次同步都跑，只有镜像需要抽屉容器。
+- 修复 **底栏统计弹层偶尔只剩一段（只显示会话统计，或只显示 Token 用量）**：皮肤按标签文字区分宿主的两颗药丸，读取窗口又只有 300 毫秒，而宿主的两个面板按它自己的提交节奏挂载，晚到一步就丢掉那一段。现在按面板自身的标记判定类型，等待拉长到 800 毫秒、读失败时重试一次；药丸按钮每一步都从 DOM 重新取，窗口内还会再按一次——宿主重渲染换掉节点时，按旧节点等于没按；首次只读到一段时安排一次补读把缺的那段补上，卡片只增不减，已经打开的卡片不会被一次失败的读取改小。会话本身没有计时数据时宿主也不渲染计时面板，此时只有 Token 用量是正确的。
+- 修复 **0.1.7-rc.2 里工作区会话行的状态圈消失**：新宿主把会话行的前导座位交给槽位出口渲染，座位里因此始终有一个 `div[data-slot]` 包裹层（`display: contents`），座位不再是空元素，画在 `:empty` 上的圆圈就不再出现。圆圈现在也挂在空的槽位出口上；座位里带运行状态点时仍由状态点自己绘制。
+- 修复 **账户抽屉收起时它的行与图标停在账户行上方**：自建抽屉的内容要在收起状态下预先对账（镜像行只在收起时同步、展开时冻结），所以收起时面板连同齿轮图标一直挂在账户行正上方，此前只用透明度隐藏，内容仍留在绘制与命中树里。收起状态改用 `visibility` 隐藏（保留布局，关闭的淡出照常），展开时恢复可见。
+
+<h3 id="en-0.6.4">Bug Fixes</h3>
+
+- Fix **the footer's plugin entries staying beside the account row after a fresh start until the drawer has been opened once**: hiding the entries and mirroring them into the drawer shared one call that returned early while no drawer was up, so the plugin icons the host re-rendered stayed painted at the sidebar's bottom next to the account row until the drawer had been opened once — which is why the symptom only appeared after a fresh start. The hiding now runs on every sync; only the mirroring needs the drawer's container.
+- Fix **the stats card occasionally showing only one section (session statistics alone, or Token usage alone)**: the skin told the host's two pills apart by their label text and read with a 300 ms window, while the host mounts its panels on its own commit — one late panel lost that section. The read now identifies each panel by its own marker, waits up to 800 ms and retries once; it re-resolves the pill from the DOM at every step and presses the live node again while the window lasts, because a press on a node the host has since re-rendered goes nowhere; and a card that did come up short schedules one late re-read that fills the missing section in, so the card only ever grows and a failed read never shrinks a card already on screen. A session with no timing data renders no timing panel in the host either, where Token usage alone is correct.
+- Fix **the workspace session rows' status circle disappearing on 0.1.7-rc.2**: the newer host renders the row's leading seat through a slot outlet, so the seat always carries a `div[data-slot]` anchor (`display: contents`) and is never empty — the circle drawn on `:empty` stopped appearing. The circle now hangs on the empty outlet anchor as well; a seat carrying the running status dot still keeps the dot's own paint.
+- Fix **the account drawer's rows and icons parked above the account row while it is closed**: the self-built drawer's content is reconciled while closed by design (the mirror sync only runs then and freezes while open), so the panel and its gear icon sit right above the account row; opacity alone left that content in the paint and hit-test tree. The closed state now hides it with `visibility` (layout kept, closing fade intact) and the open state restores it.
+
+**Full Changelog**: [v0.6.1...v0.6.4](https://github.com/Nwflower/dsh-claude-style/compare/v0.6.1...v0.6.4)
+
 ## [0.6.1] - 2026-09-24
 
 [中文](#cn-0.6.1) | [English](#en-0.6.1)

package/lib/client.js

@@ -2920,7 +2920,12 @@ window.__ModuleLoader__.load({
       "body[data-dsh-claude-style] [class*=\"projectRow\"][aria-expanded=\"false\"] [class*=\"chevron\"]::after {",
       "  transform: rotate(-90deg);",
       "}",
-      "/* Session status circle indicator (matching Claude style) */",
+      "/* Session status circle indicator (matching Claude style). The host's newer",
+      "   sidebar renders the leading seat through a slot outlet, whose anchor",
+      "   (`div[data-slot]`, display:contents) is a child of the seat even when nothing",
+      "   is registered — an idle row's seat is therefore no longer :empty, so the",
+      "   circle hangs on the empty anchor as well. A seat carrying status dots or",
+      "   another plugin's content keeps its own paint and gets no circle. */",
       "body[data-dsh-claude-style] [class*=\"sessionRow\"] [class*=\"slot\"] {",
       "  width: 14px !important;",
       "  height: 16px !important;",
@@ -2928,7 +2933,8 @@ window.__ModuleLoader__.load({
       "  align-items: center !important;",
       "  justify-content: center !important;",
       "}",
-      "body[data-dsh-claude-style] [class*=\"sessionRow\"] [class*=\"slot\"]:empty::after {",
+      "body[data-dsh-claude-style] [class*=\"sessionRow\"] [class*=\"slot\"]:empty::after,",
+      "body[data-dsh-claude-style] [class*=\"sessionRow\"] [class*=\"slot\"] > [data-slot]:empty::after {",
       "  content: \"\";",
       "  display: inline-block;",
       "  width: 5px;",
@@ -3255,15 +3261,24 @@ window.__ModuleLoader__.load({
       "  flex-direction: column !important;",
       "  gap: 6px !important;",
       "  opacity: 0 !important;",
+      "  /* The drawer's rows are built and reconciled while it is closed (the mirror",
+      "     sync only runs then), so the panel sits right above the account row with",
+      "     its icons in it. opacity alone leaves that content in the paint and",
+      "     hit-test tree; visibility takes it out of both, and the delayed transition",
+      "     keeps the closing fade. Layout is untouched, so the closed drawer's box is",
+      "     still measurable. */",
+      "  visibility: hidden !important;",
       "  pointer-events: none !important;",
       "  transform: translateY(6px) scale(0.98) !important;",
       "  transform-origin: bottom left !important;",
-      "  transition: opacity 0.15s ease, transform 0.15s ease !important;",
+      "  transition: opacity 0.15s ease, transform 0.15s ease, visibility 0s linear 0.15s !important;",
       "}",
       "body[data-dsh-claude-style] .dsh-claude-account-popover[data-open=\"true\"] {",
       "  opacity: 1 !important;",
+      "  visibility: visible !important;",
       "  pointer-events: auto !important;",
       "  transform: translateY(0) scale(1) !important;",
+      "  transition: opacity 0.15s ease, transform 0.15s ease, visibility 0s !important;",
       "}",
       "body[data-dsh-claude-style][data-ds-dark-theme] .dsh-claude-account-popover {",
       "  background: #1e1e1d !important;",
@@ -6731,49 +6746,114 @@ window.__ModuleLoader__.load({
           return rows
         }
 
-        function statsPanelData(selector) {
-          var panel = document.querySelector(selector)
-          return {
-            title: panel === null ? '' : (panel.getAttribute('aria-label') || ''),
-            rows: statsRowsFrom(panel),
+        /** How long one pill's panel may take to mount (the host commits on its own schedule). */
+        var STATS_PANEL_POLL_MS = 25
+        var STATS_PANEL_ATTEMPTS = 32
+        /** When a click is repeated inside that window, and how often. */
+        var STATS_PANEL_REPRESS_AT = 12
+        var STATS_PANEL_REPRESS_MAX = 2
+        /** How often a read that could not open a panel is retried, and after what delay. */
+        var STATS_READ_RETRIES = 1
+        var STATS_RETRY_MS = 120
+        /** How long after a short card the one late re-read runs. */
+        var STATS_FILL_MS = 700
+
+        /** The stats panel that is open right now, with the kind its own marker names. */
+        function openStatsPanel() {
+          var details = document.querySelector('[data-session-stats-details]')
+          if (details !== null) {
+            var detailsDialog = details.closest('[role="dialog"]')
+            if (detailsDialog !== null) return { kind: 'details', panel: detailsDialog }
           }
+          var usage = document.querySelector('[data-session-stats-usage]')
+          if (usage !== null) {
+            var usageDialog = usage.closest('[role="dialog"]')
+            if (usageDialog !== null) return { kind: 'usage', panel: usageDialog }
+          }
+          return null
         }
 
-        function withStatsPanel(button, selector, done) {
-          if (button === null) { done({ title: '', rows: [] }); return }
-          if (button.getAttribute('aria-expanded') !== 'true') button.click()
+        /** The row's pill buttons as they are right now (the host replaces them on re-render). */
+        function statsPillAt(index) {
+          var root = statsRoot()
+          if (root === null) return null
+          var buttons = root.querySelectorAll('button')
+          return index < buttons.length ? buttons[index] : null
+        }
+
+        /**
+         * Open one pill, read whichever stats panel it shows, and close it again.
+         *
+         * The read is kind-agnostic on purpose: which panel appears is the host's
+         * own answer, so a pill is never mislabelled by its label text (the timing
+         * pill's label carries a tok/s reading as well, and the timing pill is a
+         * plain span — no trigger at all — while the session has no timing yet).
+         * The panel counts only while that pill reports itself expanded, so a
+         * sibling's panel that has not unmounted yet is never read as this one's.
+         * The pill is re-resolved from the DOM at every step and pressed again
+         * while the window lasts: the host re-renders the row on its own schedule,
+         * and a click on a node it has since replaced goes nowhere. `null` means
+         * the row has no such pill; a pill that shows no panel inside the window
+         * reports a failure, which is not an absent section.
+         */
+        function readStatsPill(index, done) {
+          var presses = 0
+          function press() {
+            var pill = statsPillAt(index)
+            if (pill === null) return false
+            if (pill.getAttribute('aria-expanded') !== 'true') pill.click()
+            return true
+          }
+          if (!press()) { done(null); return }
           var attempts = 0
           function read() {
-            var panel = document.querySelector(selector)
-            if (panel === null && attempts < 15) {
+            var pill = statsPillAt(index)
+            if (pill === null) { done({ failed: true, rows: [] }); return }
+            var open = pill.getAttribute('aria-expanded') === 'true' ? openStatsPanel() : null
+            if (open === null && attempts < STATS_PANEL_ATTEMPTS) {
               attempts += 1
-              setTimeout(read, 20)
+              if (attempts === STATS_PANEL_REPRESS_AT && presses < STATS_PANEL_REPRESS_MAX) {
+                presses += 1
+                press()
+              }
+              setTimeout(read, STATS_PANEL_POLL_MS)
               return
             }
-            var data = statsPanelData(selector)
-            if (button.getAttribute('aria-expanded') === 'true') button.click()
-            done(data)
+            if (pill.getAttribute('aria-expanded') === 'true') pill.click()
+            if (open === null) { done({ failed: true, rows: [] }); return }
+            done({ kind: open.kind, title: open.panel.getAttribute('aria-label') || '', rows: statsRowsFrom(open.panel) })
           }
-          setTimeout(read, 20)
+          setTimeout(read, STATS_PANEL_POLL_MS)
         }
 
+        /**
+         * Read both of the host's pills.
+         *
+         * `done(sections, incomplete)`: the sections in the host's own order
+         * (session statistics, then token usage), and `incomplete` true when a
+         * pill the row carries could not be read — the caller must not mistake
+         * that for a section the host does not have.
+         */
         function collectStatsData(done) {
-          var root = statsRoot()
-          if (root === null) { done([]); return }
-          var buttons = root.querySelectorAll('button')
-          var timeBtn = null
-          var usageBtn = null
-          for (var i = 0; i < buttons.length; i++) {
-            var aria = buttons[i].getAttribute('aria-label') || ''
-            if (/轮|步|turns?|steps?/i.test(aria)) timeBtn = buttons[i]
-            else usageBtn = buttons[i]
-          }
-          withStatsPanel(timeBtn, '[role="dialog"]:has([data-session-stats-details])', function (timeData) {
-            withStatsPanel(usageBtn, '[role="dialog"]:has([data-session-stats-usage])', function (usageData) {
+          if (statsRoot() === null) { done([], false); return }
+          readStatsPill(0, function (first) {
+            readStatsPill(1, function (second) {
+              var reads = [first, second]
               var sections = []
-              if (timeData.rows.length > 0) sections.push(timeData)
-              if (usageData.rows.length > 0) sections.push(usageData)
-              done(sections)
+              var incomplete = false
+              for (var i = 0; i < reads.length; i++) {
+                var read = reads[i]
+                if (read === null) continue
+                if (read.failed === true || read.rows.length === 0) {
+                  incomplete = true
+                  continue
+                }
+                sections.push(read)
+              }
+              sections.sort(function (a, b) {
+                return (a.kind === 'details' ? 0 : 1) - (b.kind === 'details' ? 0 : 1)
+              })
+              done(sections, incomplete)
             })
           })
         }
@@ -6841,12 +6921,34 @@ window.__ModuleLoader__.load({
           pop.innerHTML = html
         }
 
-        /** The last full read; a thinner one never replaces it while the card is up. */
+        /** Whether the card is up, so a failed re-read can leave its content alone. */
+        function isStatsPopoverOpen() {
+          return statsPopover !== null && statsPopover.getAttribute('data-open') === 'true'
+        }
 
-        var statsSections = null
+        /** How many sections the card currently shows. */
+        var renderedSections = 0
 
-        var statsSectionsAt = 0
+        /** Place the card above the stats row, clamped to the viewport. */
+        function placeStatsPopover(anchor) {
+          var pop = ensureStatsPopover()
+          var live = statsRoot() || anchor
+          if (live === null) return
+          var rect = live.getBoundingClientRect()
+          var width = pop.offsetWidth
+          var height = pop.offsetHeight
+          var left = Math.max(8, Math.min(rect.left + rect.width / 2 - width / 2, window.innerWidth - width - 8))
+          var top = Math.max(8, rect.top - height - 8)
+          pop.style.left = left + 'px'
+          pop.style.top = top + 'px'
+        }
 
+        function showStatsSections(anchor, sections) {
+          renderStatsPopover(sections)
+          renderedSections = sections.length
+          ensureStatsPopover().setAttribute('data-open', 'true')
+          placeStatsPopover(anchor)
+        }
 
         function showStatsPopover(anchor) {
           // The card reads the host's two dialogs, which only the detailed row
@@ -6858,29 +6960,40 @@ window.__ModuleLoader__.load({
             clearTimeout(statsHideTimer)
             statsHideTimer = null
           }
-          collectStatsData(function (sections) {
+          readStatsCard(anchor, 0)
+        }
+
+        /**
+         * Collect the host's panels and render the card.
+         *
+         * A read that could not open one of the panels is retried, and one that
+         * still comes back short while the card is already up leaves the card as
+         * it is: the host's panels mount on its own commit, so a read that lost
+         * the race must never read as a section the host does not have — that was
+         * the 'click and it drops to Token usage only' card. A card that did come
+         * up short schedules one late re-read, which fills it in and never
+         * shrinks it.
+         */
+        function readStatsCard(anchor, attempt) {
+          collectStatsData(function (sections, incomplete) {
             if (sections.length === 0) return
-            // A click re-collects, and the host's two pills can be mid-flight then: a read
-            // that comes back with fewer sections than the card already shows must not
-            // shrink it — that is the 'click and it drops to Token usage only' bug.
-            var fresh = Date.now() - statsSectionsAt < 180000
-            if (statsSections !== null && fresh && sections.length < statsSections.length) {
-              sections = statsSections
-            } else {
-              statsSections = sections
-              statsSectionsAt = Date.now()
+            if (incomplete && attempt < STATS_READ_RETRIES) {
+              setTimeout(function () { readStatsCard(anchor, attempt + 1) }, STATS_RETRY_MS)
+              return
             }
-            renderStatsPopover(sections)
-            var pop = ensureStatsPopover()
-            pop.setAttribute('data-open', 'true')
-            var live = statsRoot() || anchor
-            var rect = live.getBoundingClientRect()
-            var width = pop.offsetWidth
-            var height = pop.offsetHeight
-            var left = Math.max(8, Math.min(rect.left + rect.width / 2 - width / 2, window.innerWidth - width - 8))
-            var top = Math.max(8, rect.top - height - 8)
-            pop.style.left = left + 'px'
-            pop.style.top = top + 'px'
+            if (incomplete && isStatsPopoverOpen()) return
+            showStatsSections(anchor, sections)
+            if (incomplete) setTimeout(function () { fillStatsCard(anchor) }, STATS_FILL_MS)
+          })
+        }
+
+        /** One late re-read for a card that came up short; it only ever adds. */
+        function fillStatsCard(anchor) {
+          collectStatsData(function (sections, incomplete) {
+            if (incomplete || sections.length === 0) return
+            if (!isStatsPopoverOpen()) return
+            if (sections.length <= renderedSections) return
+            showStatsSections(anchor, sections)
           })
         }
 
@@ -11056,6 +11169,12 @@ window.__ModuleLoader__.load({
         var footerEntries = syncFooterActionVisibility(footerActions)
         var body = options.body()
 
+        // A closed host drawer has no container to mirror into, but the entries
+        // above are hidden in place all the same: the host re-renders them, and a
+        // pass that skips this leaves their icons painted in the sidebar until the
+        // drawer is opened once.
+        if (body === null) return
+
         // While the popover is open, its mirrors must stay completely static:
         // a content rewrite, reorder, or embedded-clone replacement under the
         // pointer cancels the browser's :hover state and can swallow the click
@@ -11746,10 +11865,15 @@ window.__ModuleLoader__.load({
         syncAccountRowBox(entryRow, footArea)
         if (hostTrigger !== null) bindHostRowHover(hostTrigger)
 
+        // The host's footer entries are hidden in place on every pass, drawer or
+        // no drawer: the host re-renders them, and a pass that skipped this left
+        // their icons painted beside the account row until the drawer had been
+        // opened once. Only the mirroring needs a container to write into.
+        mirror.sync(footArea)
+
         var root = surface.mode() === 'host' ? surface.container() : accountPopover
         if (root === null) return
         rows.syncHeader(root, hostTrigger)
-        mirror.sync(footArea)
         // The rail toggle (and any reflow) moves the anchor without a window
         // resize or a page scroll, so an open drawer re-resolves its position at
         // the end of its own pass.

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "dsh-claude-style",
   "description": "Claude Code Desktop theme for DeepSeek Harness (dsh) web GUI — visual & interaction overhaul",
-  "version": "0.6.1",
+  "version": "0.6.4",
   "type": "module",
   "main": "lib/index.js",
   "icon": "lib/claude-mark.svg",