dsh-edit-turn 0.2.9 → 0.2.11

package/README.md

@@ -219,6 +219,17 @@ ln -s /path/to/dsh-install/node_modules ./node_modules
 
 ### 更新日志
 
+**0.2.11** —— 修「字怎么变大了」。
+
+- **修复：改写后的气泡与编辑器不跟随平台的"内容字号"设置**。平台的气泡用 `font-size: var(--dsh-content-font-size,14px)` + `line-height: calc(22px + var(--dsh-content-font-delta,0px))`；我们的替代气泡写成 `font:inherit`（继承了行的 16px，比气泡的 15px 大），编辑器又写死 15px——读者一改内容字号，两处都对不上，看起来就是"字变大了"。现在两者都取平台同一个变量/公式（该变量定义在 body 上，编辑器所在的浮层是其子节点，可直接继承）。
+- 真机验证：默认设置下平台气泡 / 替代气泡 / 编辑器均为 15px；把 `--dsh-content-font-size` 改成 18px 后，三者同步变为 18px。+ 2 条静态检查；110 单测 / 80 契约 / 静态 / verify:live / verify:ui 全绿。
+
+**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.11'
 
     // --- copy -----------------------------------------------------------------
 
@@ -145,7 +145,12 @@ window.__ModuleLoader__.load({
       // a replacement message, so this one is ours. Styled from the plugin own
       // namespaced variables, which follow the active theme without borrowing
       // CSS-module hashes that change between builds.
-      '.dshet-revision{box-sizing:border-box;align-self:flex-end;width:fit-content;max-width:min(72ch,86%);margin:6px 0 6px auto;padding:9px 14px;border-radius:16px;background:var(--dshet-field);color:var(--dshet-ink);font:inherit;line-height:inherit;white-space:pre-wrap;overflow-wrap:anywhere;min-width:0}',
+      // Font metrics copied from the platform's own user bubble
+      // (`.Sixlwa_bubble`): a fixed size or `font:inherit` is wrong here -
+      // inherit picks up the row's 16px, fixed ignores the reader's content
+      // font setting, and either way the rewritten text changed size against
+      // the bubble it replaces.
+      '.dshet-revision{box-sizing:border-box;align-self:flex-end;width:fit-content;max-width:min(72ch,86%);margin:6px 0 6px auto;padding:9px 14px;border-radius:16px;background:var(--dshet-field);color:var(--dshet-ink);font:inherit;font-size:var(--dsh-content-font-size,14px);line-height:calc(22px + var(--dsh-content-font-delta,0px));white-space:pre-wrap;overflow-wrap:anywhere;min-width:0}',
       '.dshet-revision:empty{display:none}',
       '.dshet-revision{position:relative}',
       // Fallback only: a row with no action bar to keep is displayed away
@@ -187,7 +192,7 @@ window.__ModuleLoader__.load({
       '.dshet-layer .dshet-editor{position:absolute;pointer-events:auto}',
       '.dshet-editor{margin:0;border:1px solid var(--dshet-line);border-radius:16px;background:var(--dshet-panel);backdrop-filter:blur(18px) saturate(1.2);box-shadow:var(--dshet-shadow);color:var(--dshet-ink);transition:border-color .12s}',
       '.dshet-editor:focus-within{border-color:var(--dsw-alias-button-primary-fill,var(--dshet-accent))}',
-      '.dshet-editor textarea{display:block;width:100%;min-height:47px;max-height:46vh;overflow-y:auto;resize:none;box-sizing:border-box;padding:11px 14px 0;border:0;background:transparent;color:inherit;font:inherit;font-size:15px;line-height:25px;outline:none}',
+      '.dshet-editor textarea{display:block;width:100%;min-height:47px;max-height:46vh;overflow-y:auto;resize:none;box-sizing:border-box;padding:11px 14px 0;border:0;background:transparent;color:inherit;font:inherit;font-size:var(--dsh-content-font-size,14px);line-height:calc(22px + var(--dsh-content-font-delta,0px));outline:none}',
       '.dshet-editor textarea::placeholder{color:var(--dshet-ink-dim)}',
       '.dshet-editor textarea:disabled{opacity:.65}',
       '.dshet-note,.dshet-warn,.dshet-error{margin:4px 14px 0;font-size:12px;line-height:19px}',
@@ -264,6 +269,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 +554,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 +1012,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 +1481,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.11;
 // 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.11'
 
 // 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.11",
   "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",