dsh-edit-turn 0.2.9 → 0.2.10

package/README.md

@@ -219,6 +219,12 @@ ln -s /path/to/dsh-install/node_modules ./node_modules
 
 ### 更新日志
 
+**0.2.10** —— 修「编辑器浮层留在设置页上」。
+
+- **修复：会话视图卸载后编辑器没有跟着走**。编辑器挂在 `body` 的独立浮层上（这是为了躲开宿主重绘），而没有任何框架会替它收尾：切到设置页时对话视图（连同注册在输入框卡上的条目）卸载了，编辑器却留在浮层里**飘在设置页上**。现在条目卸载时清掉自己的编辑器与参考，空浮层也一并移除。
+- 卸载判定不能"一有 cleanup 就清"：这个 effect 在每次视图变化时都会先 cleanup 再重跑，硬清会把编辑器反复拆掉（焦点/草稿语义都会乱）。现在是**延迟清理**——下一次 effect 重跑会取消它，只有真正的卸载（不再重跑）才让它落地。
+- 新增用例「会话视图卸载后编辑器跟着消失（浮层也不留）」+ 2 条静态检查；110 单测 / 80 契约 / 静态 / verify:live / verify:ui 全绿。
+
 **0.2.9** —— 修「同一条消息出现两个一样的气泡」。
 
 - **修复：一行两个替代气泡，两个来源都堵掉**。①气泡此前只认「行的直系子节点 / 紧邻前一个兄弟」，一旦被宿主挪开就认不出，下一趟又种一个；现在气泡带**行的 key 标记**（`data-dshet-revision-for`），挪到哪都找得回来，且每行只允许一个（多余的按标记清除）。②更隐蔽的一个：MutationObserver 的回调可能**晚于它所属的那次渲染**，带着**过期视图**（`hidden` 为空）跑一遍，把已折叠的行重新展开——原气泡就回到了替代气泡旁边；现在这趟 pass 从 `controller.getSnapshot()` 现取当前视图，过期视图再也不可能把行展开。

package/lib/client.js

@@ -30,7 +30,7 @@ window.__ModuleLoader__.load({
     const ROUTE_PREFIX = '/dsh-edit-turn'
 
     /** Keep in sync with package.json and lib/index.js. */
-    const PLUGIN_VERSION = '0.2.9'
+    const PLUGIN_VERSION = '0.2.10'
 
     // --- copy -----------------------------------------------------------------
 
@@ -264,6 +264,7 @@ window.__ModuleLoader__.load({
         this.editorRow = null
         this.editorFocus = false
         this.editorDraft = ''
+        this.clearTimer = null
         this.confirming = false
         this.noticeTimer = null
         this.view = Object.freeze({
@@ -548,6 +549,9 @@ window.__ModuleLoader__.load({
       }
 
       dispose() {
+        if (this.clearTimer !== null) window.clearTimeout(this.clearTimer)
+        this.clearTimer = null
+        clearEditor()
         this.listeners.clear()
         if (this.noticeTimer !== null) window.clearTimeout(this.noticeTimer)
         this.noticeTimer = null
@@ -1003,6 +1007,12 @@ window.__ModuleLoader__.load({
     function clearEditor() {
       const elements = document.querySelectorAll('.dshet-editor')
       for (const element of elements) element.remove()
+      // An empty layer has no business staying on the page: the editor is
+      // hosted on the body, where nothing else clears up after it.
+      if (editorLayer !== null && editorLayer.isConnected === true && editorLayer.children.length === 0) {
+        editorLayer.remove()
+        editorLayer = null
+      }
     }
 
     // The layer the editor is hosted on. It is created once, hangs off the
@@ -1466,8 +1476,25 @@ window.__ModuleLoader__.load({
           else window.setTimeout(run, 16)
         })
         observer.observe(document.body, { childList: true, subtree: true })
+        // This effect re-runs whenever the view changes, and its cleanup runs
+        // first every time; only a REAL unmount means the conversation view
+        // went away (switching to Settings, say) and the editor has to go with
+        // it - left behind, it floated over whatever came next. So the clear is
+        // deferred, and the next run of this effect cancels it.
+        if (controller.clearTimer !== null) {
+          window.clearTimeout(controller.clearTimer)
+          controller.clearTimer = null
+        }
         return () => {
           observer.disconnect()
+          if (controller.clearTimer !== null) window.clearTimeout(controller.clearTimer)
+          controller.clearTimer = window.setTimeout(() => {
+            controller.clearTimer = null
+            clearEditor()
+            controller.editorBox = null
+            controller.editorRow = null
+            controller.editorElement = null
+          }, 250)
         }
       }, [snapshot, view, controller, t])
 

package/lib/index.js

@@ -22,7 +22,7 @@
 //      admission path. It resumes the Session when cold and starts exactly one
 //      new turn from the revised text.
 //
-// The silent replacement carrier is an EMPTY `developer/message` (0.2.9;
+// The silent replacement carrier is an EMPTY `developer/message` (0.2.10;
 // before that an empty `system/message`, which no reader accepts - see the
 // README). Empty content projects to no model message, so the rollback adds
 // nothing the model can see and the derived context after an edit is exactly
@@ -60,7 +60,7 @@ export const PLUGIN_ID = 'dsh-edit-turn'
 export const name = PLUGIN_ID
 
 /** Keep in sync with package.json and lib/client.js. */
-export const PLUGIN_VERSION = '0.2.9'
+export const PLUGIN_VERSION = '0.2.10'
 
 // A ring of the last few requests, answering "did the browser half even ask, and
 // what did it get?" without needing its console. Read-only diagnostics: the

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-edit-turn",
-  "version": "0.2.9",
+  "version": "0.2.10",
   "description": "Edit any user turn in DeepSeek Harness and re-run it: an inline editor on the message row rolls the conversation back to just before that turn through the official surface-replace contract, then re-prompts the agent with the revised text. The append-only session log is never rewritten.",
   "type": "module",
   "main": "lib/index.js",