dsh-edit-turn 0.5.7 → 0.5.8

package/README.md

@@ -277,6 +277,16 @@ ln -s /path/to/dsh-install/node_modules ./node_modules
 
 ### 更新日志
 
+**0.5.8** —— 复制交给**平台写入器这一层**兜底：不管按下走哪条路，落到剪贴板的都是改写后的文本。
+
+- **0.5.7 没修好**：那一版把"回答按下"从按钮节点挪到了文档级捕获监听（按行匹配，节点被重画也不影响），但实测仍然复制出原文。说明这次按下**根本没有经过本插件能拦到的按钮**——具体是哪个环节，从外部看不出来（没有可用的浏览器来读真实 DOM）。
+
+- **所以换到更低的层**：`navigator.clipboard.writeText` 是**所有**复制都要经过的唯一关口——宿主自己的 `writeClipboard` 原语内部就是调它。本插件把这一层包起来：写入的文本如果正是某条被改写消息的**原文**（按行登记，空白归一化后比较），就换成改写后的文本；其余写入一字不动。
+
+- **登记表放在页级**（`globalThis` 上的一个 Symbol），不是模块变量：实时重载会生成新的 bundle，而**旧 bundle 装的包装器还在写入路径上**（包装本身是幂等的）。两份 Map 会让包装器读到一个没人填的登记表。
+
+- 新增测试：宿主**自己**写它那份原文（不经过本插件的按钮接管）时，落地的是改写后的文本；与改写无关的复制原样通过。测试台补了 `navigator.clipboard` 与 `childNodes`。
+
 **0.5.7** —— 改写过的行，复制按钮被宿主重画之后仍然交出改写后的文本。
 
 - **问题**：编辑过一条消息（提示词走官方 surface 替换），界面上显示的是改写后的文本，但点那一排的**复制**按钮，粘出来的是**改写前**的原文。

package/lib/client.js

@@ -45,7 +45,7 @@ window.__ModuleLoader__.load({
     const RERUN_PREFIX = '/dsh-rerun-turn'
 
     /** Keep in sync with package.json and lib/index.js. */
-    const PLUGIN_VERSION = '0.5.7'
+    const PLUGIN_VERSION = '0.5.8'
 
     // --- copy -----------------------------------------------------------------
 
@@ -1937,6 +1937,10 @@ window.__ModuleLoader__.load({
     const COPY_TIMER = 'dshetCopyTimer'
     /** The host draws its own check for a second; this is the same second. */
     const COPY_FEEDBACK_MS = 1000
+    /** 标记：这一页的剪贴板写入已经被本半部包过一次（重载会复用同一个 page）。 */
+    const CLIPBOARD_WRAPPED = Symbol.for('dshetClipboardWrapped')
+    /** 页级登记表：重载后新 bundle 与旧包装器共用同一份。 */
+    const COPY_REGISTRY = Symbol.for('dshetCopyRegistry')
     /**
      * Row key -> the text that row's copy button must hand over.
      *
@@ -1947,7 +1951,25 @@ window.__ModuleLoader__.load({
      * the button it drew goes away with that node (the button it draws next is a
      * different one).
      */
-    const rowCopyText = new Map()
+    /**
+     * Row key -> the text that row's copy must hand over, and the wording the host
+     * would hand over instead.
+     *
+     * Both live on the PAGE rather than in this module: a live reload builds a new
+     * bundle with new maps, and the clipboard wrapper the previous bundle installed
+     * is still the one in the write path (wrapping is idempotent, by design). Two
+     * maps would leave that wrapper reading a registry nobody fills.
+     */
+    const copyRegistry = (() => {
+      const page = typeof globalThis !== 'undefined' ? globalThis : {}
+      const existing = page[COPY_REGISTRY]
+      if (existing !== undefined && existing !== null && existing.text instanceof Map) return existing
+      const created = { text: new Map(), host: new Map() }
+      page[COPY_REGISTRY] = created
+      return created
+    })()
+    const rowCopyText = copyRegistry.text
+    const rowHostText = copyRegistry.host
 
     /**
      * A button another plugin put in this bar.
@@ -2011,13 +2033,93 @@ window.__ModuleLoader__.load({
      * host's own clipboard writer the button keeps the host's behaviour too: a
      * takeover that cannot copy is worse than none.
      */
+    /**
+     * The wording the host's own copy button would hand over for this row.
+     *
+     * It is the message the row was drawn for - the pre-edit text - and it is read
+     * off the row itself, because that is where it still is: this plugin hides the
+     * row's own bubble and stands its revision in front of it.
+     *
+     * @param {Element} row - the row.
+     * @returns {string|null} the host's text, or null when it cannot be read.
+     */
+    function hostCopyTextOf(row) {
+      try {
+        const bubble = ownRevisionBubble(row)
+        let text = ''
+        const children = row.childNodes
+        if (children === undefined || children === null) return null
+        for (const child of children) {
+          if (bubble !== null && child === bubble) continue
+          if (typeof child.textContent === 'string') text += child.textContent
+        }
+        const normalized = text.replace(/\s+/g, ' ').trim()
+        return normalized === '' ? null : normalized
+      } catch {
+        return null
+      }
+    }
+
+    /**
+     * The wording that must actually land on the clipboard.
+     *
+     * The host's copy button copies the message its row was drawn for, so on a row
+     * this plugin rewrote it hands over the pre-edit text. The press itself is
+     * answered earlier when this half can find the button, but the action bar is the
+     * host's own React and it may draw a button this half never marked - so the last
+     * say belongs to the write itself, which no button and no handler can bypass.
+     *
+     * @param {string} text - what the platform is about to write.
+     * @returns {string|null} the text to write instead, or null to leave it alone.
+     */
+    function rewriteCopiedText(text) {
+      const wanted = text.replace(/\s+/g, ' ').trim()
+      if (wanted === '') return null
+      for (const [key, edited] of rowCopyText) {
+        if (typeof edited !== 'string') continue
+        if (edited.replace(/\s+/g, ' ').trim() === wanted) return null
+        const hostText = rowHostText.get(key)
+        if (typeof hostText === 'string' && (hostText === wanted || hostText.includes(wanted))) return edited
+      }
+      return null
+    }
+
+    /**
+     * Wrap the platform's clipboard writer, once per page.
+     *
+     * `navigator.clipboard.writeText` is the single choke point every copy goes
+     * through - the host's own `writeClipboard` primitive calls it, and so does a
+     * browser's native copy of a selection.
+     *
+     * @returns {() => void} undo.
+     */
+    function installClipboardRewrite() {
+      const clipboard = typeof navigator !== 'undefined' ? navigator.clipboard : undefined
+      if (clipboard === undefined || clipboard === null || typeof clipboard.writeText !== 'function') return () => {}
+      if (clipboard.writeText[CLIPBOARD_WRAPPED] === true) return () => {}
+      const native = clipboard.writeText.bind(clipboard)
+      const wrapped = (value) => {
+        const rewritten = rewriteCopiedText(String(value ?? ''))
+        return rewritten === null ? native(value) : native(rewritten)
+      }
+      wrapped[CLIPBOARD_WRAPPED] = true
+      clipboard.writeText = wrapped
+      return () => {
+        if (clipboard.writeText === wrapped) clipboard.writeText = native
+      }
+    }
+
     function takeOverRowCopy(row, text, t) {
       // Registered before the button lookup on purpose: the handover has to work
       // even when this pass cannot find the button to mark (the next one may, and
       // the document-level press answers from this table either way).
       {
         const key = row.getAttribute('data-chat-flow-key')
-        if (typeof key === 'string' && key !== '') rowCopyText.set(key, text)
+        if (typeof key === 'string' && key !== '') {
+          rowCopyText.set(key, text)
+          const hostText = hostCopyTextOf(row)
+          if (hostText !== null) rowHostText.set(key, hostText)
+        }
       }
       const button = nativeCopyButton(row, t)
       if (button === null) return
@@ -2148,7 +2250,10 @@ window.__ModuleLoader__.load({
     /** The copy handover of a row, wherever the last pass left it. */
     function releaseRowCopy(row) {
       const key = row.getAttribute('data-chat-flow-key')
-      if (typeof key === 'string' && key !== '') rowCopyText.delete(key)
+      if (typeof key === 'string' && key !== '') {
+        rowCopyText.delete(key)
+        rowHostText.delete(key)
+      }
       const button = nativeCopyButton(row)
       if (button !== null) releaseCopyButton(button)
     }
@@ -4793,6 +4898,10 @@ window.__ModuleLoader__.load({
         controller.load()
       }, [controller])
 
+      // The last say on what lands: whatever route the press takes, the write
+      // itself goes through here.
+      react.useEffect(() => installClipboardRewrite(), [])
+
       // Pressing outside the editor is the one signal that means "the user
       // left". Blur cannot be that signal: a host re-render replaces the node
       // (blur without a related target) and a save temporarily disables it -

package/lib/index.js

@@ -78,7 +78,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.5.7'
+export const PLUGIN_VERSION = '0.5.8'
 
 // 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.5.7",
+  "version": "0.5.8",
   "description": "Edit any message in DeepSeek Harness in place: a prompt is replaced where it stands; a model reply block (answer text or reasoning) is patched through a plugin record and a message projection, so the turn stays where it is, later turns are untouched and the log grows by one small record. Falls back to the replay splice on hosts without the projection API. The append-only session log is never rewritten.",
   "type": "module",
   "main": "lib/index.js",