dsh-as-aistudio 0.4.0 → 0.4.1

package/CHANGELOG.md

@@ -4,6 +4,12 @@
 
 ## 中文
 
+**0.4.1** —— 编辑过的消息，复制按钮被宿主重画之后仍然复制改写后的文本。
+
+- **组件跟进**：`dsh-edit-turn` **0.5.7**（另三个不变：`dsh-rerun-turn` 0.2.6、`dsh-delete-turn` 0.1.15、`dsh-markdown-bubble` 0.1.5）。
+
+- **复制拿到的是改写前的原文**：编辑一条消息后，界面上显示的是改写后的文本，但点那一排的复制按钮，粘出来的是原文。原因是这次按下原本由**挂在宿主那个按钮节点上**的捕获期监听回答，而动作条是宿主 React 画的——任何一次重渲染都可能换掉那个按钮节点，监听就跟着旧节点一起消失，按下于是落到宿主自己的处理上（它复制"这一行被画出来时的那条消息"）。现在改由**文档级捕获期监听按行**回答：与按钮是不是同一个节点无关，捕获期在 React 根节点之前，宿主自己的处理看不到这次按下。本包已把修好的 `dsh-edit-turn` 0.5.7 重新 vendor 进来并重建了浏览器 bundle。
+
 **0.4.0** —— 少要一个平台包；四个组件跟进到最新。
 
 - **不再要求 `dsh-tools`**：本包内置的宿主半部以前要 `@deepseek-ai/dsh-tools`，唯一原因是 `dsh-edit-turn` 向模型暴露了一个工具（`edit_turn_targets`）。那个工具已经删掉——编辑是靠 UI 面板做的，模型不需要知道这件事——于是整族插件都不再声明模型可见的工具，宿主半部的解析只需要 `dsh-session`。少一个平台包，就少一处跟宿主版本绑定的地方。

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-as-aistudio",
-  "version": "0.4.0",
+  "version": "0.4.1",
   "description": "Google-AI-Studio-style turn controls for DeepSeek Harness, in one install: edit a prompt and re-run it, re-run a reply in place, delete a turn from the model context, and render sent prompts as Markdown. Self-contained: the plugin list shows this one row, and the four components are vendored inside it - installing the studio installs nothing else.",
   "type": "module",
   "main": "src/index.js",

package/src/client.js

@@ -43,7 +43,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.6'
+    const PLUGIN_VERSION = '0.5.7'
 
     // --- copy -----------------------------------------------------------------
 
@@ -1935,6 +1935,17 @@ 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
+    /**
+     * Row key -> the text that row's copy button must hand over.
+     *
+     * The host's copy button copies the message the row was drawn for, so on a row
+     * this plugin rewrote it would copy the wording the user just replaced. The
+     * press is answered from a document-level listener keyed by ROW rather than by
+     * button node: the host re-renders its action bar, and a listener planted on
+     * the button it drew goes away with that node (the button it draws next is a
+     * different one).
+     */
+    const rowCopyText = new Map()
 
     /**
      * A button another plugin put in this bar.
@@ -1999,6 +2010,13 @@ window.__ModuleLoader__.load({
      * takeover that cannot copy is worse than none.
      */
     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)
+      }
       const button = nativeCopyButton(row, t)
       if (button === null) return
       const write = primitives !== null && primitives !== undefined && typeof primitives.writeClipboard === 'function'
@@ -2127,6 +2145,8 @@ 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)
       const button = nativeCopyButton(row)
       if (button !== null) releaseCopyButton(button)
     }
@@ -4775,6 +4795,45 @@ window.__ModuleLoader__.load({
       // left". Blur cannot be that signal: a host re-render replaces the node
       // (blur without a related target) and a save temporarily disables it -
       // both used to drop the caret for good.
+      // The press that matters: the host's own copy button on a row this plugin
+      // rewrote. Capture phase, so the host's handler (which listens at the React
+      // root, below the document) never sees the press; by row key, so a button
+      // the host re-rendered is still answered.
+      react.useEffect(() => {
+        if (typeof document.addEventListener !== 'function' || typeof document.removeEventListener !== 'function') {
+          return undefined
+        }
+        const onCopyPress = (event) => {
+          const target = event.target
+          if (target === null || target === undefined || typeof target.closest !== 'function') return
+          const button = target.closest('button')
+          if (button === null) return
+          // Our own buttons sit in the same bar: theirs is not this press.
+          if (pluginOwned(button)) return
+          const row = button.closest('[data-chat-flow-key]')
+          if (row === null) return
+          const key = row.getAttribute('data-chat-flow-key')
+          if (typeof key !== 'string' || key === '') return
+          const text = rowCopyText.get(key)
+          if (typeof text !== 'string') return
+          const write = primitives !== null && primitives !== undefined && typeof primitives.writeClipboard === 'function'
+            ? primitives.writeClipboard
+            : null
+          if (write === null) return
+          if (typeof event.preventDefault === 'function') event.preventDefault()
+          if (typeof event.stopPropagation === 'function') event.stopPropagation()
+          if (typeof event.stopImmediatePropagation === 'function') event.stopImmediatePropagation()
+          Promise.resolve(write(text)).then(
+            (ok) => {
+              if (ok === true) flashRowCopy(button, typeof t === 'function' ? t('action.copied') : null)
+            },
+            () => {},
+          )
+        }
+        document.addEventListener('click', onCopyPress, true)
+        return () => document.removeEventListener('click', onCopyPress, true)
+      }, [t])
+
       react.useEffect(() => {
         if (typeof document.addEventListener !== 'function' || typeof document.removeEventListener !== 'function') {
           return undefined
@@ -8979,7 +9038,7 @@ window.__ModuleLoader__.load({
     const NS = 'dsh-as-aistudio'
 
     /** Keep in sync with package.json and src/components.js. */
-    const PLUGIN_VERSION = '0.4.0'
+    const PLUGIN_VERSION = '0.4.1'
 
     /** Presence marker the live verifier reads. */
     const DEBUG_KEY = '__DSH_AS_AISTUDIO__'

package/src/components.js

@@ -26,7 +26,7 @@
 export const STUDIO_ID = 'dsh-as-aistudio'
 
 /** Keep in sync with package.json and src/client.js. */
-export const STUDIO_VERSION = '0.4.0'
+export const STUDIO_VERSION = '0.4.1'
 
 /**
  * The AI Studio action strip this studio composes, in the order the user meets

package/src/vendor/host/dsh-edit-turn.js

@@ -1,6 +1,6 @@
 // dsh-edit-turn host half (dsh-edit-turn.js), VENDORED by tools/vendor.mjs - DO NOT EDIT.
 //
-// Source: ../dsh-edit-turn/lib/index.js  (v0.5.6, sha256 c4da00d187ff461d)
+// Source: ../dsh-edit-turn/lib/index.js  (v0.5.7, sha256 32dc3ed4d66bb063)
 // Regenerate with `npm run vendor`; test/sync.test.js fails when this is stale.
 //
 // src/shell.js imports the copy named after the component, which is why the
@@ -85,7 +85,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.6'
+export const PLUGIN_VERSION = '0.5.7'
 
 // 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/src/vendor/host/dsh-edit-turn.patch.js

@@ -1,6 +1,6 @@
 // dsh-edit-turn host half (dsh-edit-turn.patch.js), VENDORED by tools/vendor.mjs - DO NOT EDIT.
 //
-// Source: ../dsh-edit-turn/lib/patch.js  (v0.5.6, sha256 5e1e8cce1d168f6c)
+// Source: ../dsh-edit-turn/lib/patch.js  (v0.5.7, sha256 5e1e8cce1d168f6c)
 // Regenerate with `npm run vendor`; test/sync.test.js fails when this is stale.
 //
 // src/shell.js imports the copy named after the component, which is why the

package/src/vendor/versions.json

@@ -1,5 +1,5 @@
 {
-  "dsh-edit-turn": "0.5.6",
+  "dsh-edit-turn": "0.5.7",
   "dsh-rerun-turn": "0.2.6",
   "dsh-delete-turn": "0.1.15",
   "dsh-markdown-bubble": "0.1.5"