@idoall/dsh-session-colors 0.1.1 → 0.1.2

package/CHANGELOG.md

@@ -8,6 +8,22 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.1.2] - 2026-09-22
+
+### Fixed
+
+- Chips no longer stay at their old coordinates for up to 20 seconds when a
+  workspace is collapsed or expanded. Each mutation option set now gets its own
+  `MutationObserver`: a second `observe()` call on the same target **replaces**
+  the first one's options rather than merging them, so sharing one observer for
+  `childList` and for the `body` class had silently stopped row additions and
+  removals from being watched at all. The chips only moved when the 20-second peer
+  poll happened to re-render the layer.
+
+### Added
+
+- A regression test that fails if two option sets ever share one observer again.
+
 ## [0.1.1] - 2026-09-22
 
 Documentation only — no code change.

package/README.md

@@ -114,12 +114,13 @@ stored as HSVA, so alpha round-trips exactly.
 
 ## Compatibility
 
-Current release: plugin **`0.1.1`**, verified against DeepSeek Harness
+Current release: plugin **`0.1.2`**, verified against DeepSeek Harness
 **`0.1.6-alpha.2`**.
 
 | Plugin version | Verified DeepSeek Harness | npm status | What it is |
 | --- | --- | --- | --- |
-| **`0.1.1`** | `0.1.6-alpha.2` | `latest` | Documentation only: the packaged README no longer says "not published", and the market screenshots are declared. No code change. |
+| **`0.1.2`** | `0.1.6-alpha.2` | `latest` | Fixes chips staying at their old coordinates for ~20s when a workspace is collapsed or expanded: a second `MutationObserver.observe()` on the same target had silently replaced the `childList` watch. |
+| `0.1.1` | `0.1.6-alpha.2` | published | Documentation only: the packaged README no longer says "not published", and the market screenshots are declared. |
 | `0.1.0` | `0.1.6-alpha.2` | published | First release: Session colour marks, Host-side storage, visible inside a phone's sidebar drawer |
 
 - **Verified DeepSeek Harness** is the exact DSH version this plugin was actually

package/README.zh.md

@@ -88,11 +88,12 @@ dsh plugin --profile <profile> add link:/path/to/dsh-session-colors
 
 ## 兼容性
 
-当前版本：插件 **`0.1.1`** 已针对 DeepSeek Harness **`0.1.6-alpha.2`** 验证。
+当前版本：插件 **`0.1.2`** 已针对 DeepSeek Harness **`0.1.6-alpha.2`** 验证。
 
 | 插件版本 | 已验证的 DeepSeek Harness | npm 发布状态 | 该版本是什么 |
 | --- | --- | --- | --- |
-| **`0.1.1`** | `0.1.6-alpha.2` | `latest` | 仅文档：打包进 npm 的 README 不再写"尚未发布"，并声明了市场截图。无代码改动。 |
+| **`0.1.2`** | `0.1.6-alpha.2` | `latest` | 修掉折叠/展开工作区后色块停在旧位置约 20 秒：同一个 target 上第二次 `MutationObserver.observe()` 把 `childList` 监听顶掉了。 |
+| `0.1.1` | `0.1.6-alpha.2` | 已发布 | 仅文档：打包进 npm 的 README 不再写"尚未发布"，并声明了市场截图。 |
 | `0.1.0` | `0.1.6-alpha.2` | 已发布 | 首个版本：会话颜色标记、宿主侧存储、手机抽屉内可见 |
 
 - **已验证的 DeepSeek Harness** 是这个插件**实际跑过**的确切 DSH 版本。这份清单只有一个存放处——[`package.json`](package.json) 的 `dsh.compatibility.dshReleases`——并且有测试保证两个 README 的兼容性段落与它逐字一致、且落在 `peerDependencies` 声明的范围内。未列出的 DSH 版本**不会被宣称为兼容**。

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@idoall/dsh-session-colors",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "type": "module",
   "description": "DSH web plugin: give any Session a colour mark, shown as a chip in the sidebar and picked from the conversation header. Marks are stored on the Host, so a colour chosen on one device is visible on every other one — including over LAN and tunnels.",
   "main": "./src/index.js",

package/src/client.js

@@ -621,16 +621,31 @@ window.__ModuleLoader__.load({
         }
         document.addEventListener('transitionrun', onTransition, true)
         document.addEventListener('transitionend', onTransition, true)
-        let observer
+        // Each option set needs its OWN observer. A second `observe()` call on
+        // the same target REPLACES the first one's options rather than merging
+        // them (DOM spec), so sharing one observer silently stopped childList
+        // from being watched at all: collapsing a workspace removed its rows
+        // without any re-sync, and the chips sat at their old coordinates until
+        // the peer poll happened to re-render the layer ~20s later.
+        const observers = []
         try {
-          observer = new MutationObserver(sync)
-          observer.observe(document.body, { childList: true, subtree: true })
+          const rowsObserver = new MutationObserver(sync)
+          rowsObserver.observe(document.body, { childList: true, subtree: true })
+          observers.push(rowsObserver)
           // The phone adapter opens its drawer by toggling a class on `body`.
           // With transitions disabled the geometry still changes, just without a
           // transition event to follow.
-          observer.observe(document.body, { attributes: true, attributeFilter: ['class'] })
+          const drawerObserver = new MutationObserver(sync)
+          drawerObserver.observe(document.body, { attributes: true, attributeFilter: ['class'] })
+          observers.push(drawerObserver)
+          // A workspace collapse that hides rows instead of removing them
+          // produces no childList mutation, so follow the ARIA state and
+          // `hidden` as well.
+          const stateObserver = new MutationObserver(sync)
+          stateObserver.observe(document.body, { attributes: true, attributeFilter: ['aria-expanded', 'hidden'], subtree: true })
+          observers.push(stateObserver)
         } catch {
-          observer = undefined
+          // Without observers the layer still follows scroll, resize and transitions.
         }
         return () => {
           cancelFrame(frame)
@@ -638,7 +653,7 @@ window.__ModuleLoader__.load({
           window.removeEventListener('resize', sync)
           document.removeEventListener('transitionrun', onTransition, true)
           document.removeEventListener('transitionend', onTransition, true)
-          if (observer !== undefined) observer.disconnect()
+          for (const watcher of observers) watcher.disconnect()
         }
       }, [snapshot])