dsh-edit-turn 0.5.10 → 0.5.12

package/README.md

@@ -277,6 +277,20 @@ ln -s /path/to/dsh-install/node_modules ./node_modules
 
 ### 更新日志
 
+**0.5.12** —— `/state` 只折一次表面；临时诊断撤掉。
+
+- **复制已确认修好**（根因是 `/state` 太大太慢：25 MB / 13 秒，那段时间里浏览器半部没有任何编辑条目，按下复制不会被接管）。所以 0.5.10 的自报诊断按计划撤掉。
+
+- **再提速**：`rollbackLedger` 自己又折了一遍表面，而 `stateOf` 已经折过一次 —— 31k 事件的会话等于把整份日志走了两遍，还各自重建了 `bySeq`、`turnIndex`。现在折一次、索引复用，四个遍历共用同一份。实测同一份日志：`editableReplies` 的轻量路径 **2,227 ms → 29 ms**（0.5.11），这一版再省掉重复的那一遍折叠。
+
+- **缓存试过又撤了**：按"会话 + 事件数 + 配置"缓存 state 能把重复打开变成瞬时，但测试里暴露出陈旧风险（状态快照没跟上新事件时，键不变、结果却是旧的）。宁可慢一点，也不要给编辑器一份过期的状态。这条以后要做得从宿主的 seq 严格判定失效。
+
+**0.5.11** —— `/state` 的回复列表加轻量路径：不再为每条回复构建分块。
+
+- 0.5.9 把回复全文从 `/state` 拿掉了，但服务端**仍然为每条回复构建了全部分块**（还要为每条回复遍历它那一整轮的所有事件做合并）。实测一个会话 290 条回复：**2,227 ms、22.4 MB 的中间数据** —— 而 `/state` 只需要身份。
+
+- 新增 `light` 选项：跳过分块构建与"整轮合并"，只保留判定所需的字段。同一份日志：**29 ms、1.1 MB**（77 倍）。`/reply`（按需取）仍走完整路径。
+
 **0.5.10** —— 临时诊断：复制没被接管时，把原因写进标签页标题。
 
 - 这台机器上没有可用的浏览器（浏览器工具找不到 Chrome），拿不到真实 DOM，而两次按代码推断的修法都没命中。所以加一条**自报**：如果这次复制按下没被接管，标签页标题会变成 `[复制诊断] <原因>`（例如「这一行没登记文本（登记 3 行）」「按钮不在行内」「写入没匹配上」）；接管成功时标题自动恢复。不污染剪贴板、不改页面内容。

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.10'
+    const PLUGIN_VERSION = '0.5.12'
 
     // --- copy -----------------------------------------------------------------
 
@@ -2001,24 +2001,6 @@ window.__ModuleLoader__.load({
     })()
     const rowCopyText = copyRegistry.text
 
-    // TEMP DIAG (删除时机：复制修好后). 复制按下没被接管时，把原因写进标签页标题：
-    // 不污染剪贴板、不改页面内容，用户念一眼标题即可；成功接管时自动恢复原标题。
-    const copyDiag = (page) => {
-      try {
-        if (page.__dshetTitle === undefined) page.__dshetTitle = document.title
-        const base = String(page.__dshetTitle ?? '')
-        const show = (reason) => {
-          document.title = `[复制诊断] ${reason}${base === '' ? '' : ' · ' + base}`
-        }
-        const clear = () => {
-          if (String(document.title).startsWith('[复制诊断]')) document.title = base
-        }
-        return { show, clear }
-      } catch {
-        return { show: () => {}, clear: () => {} }
-      }
-    }
-    const titleDiag = typeof globalThis !== 'undefined' ? copyDiag(globalThis) : { show: () => {}, clear: () => {} }
     const rowHostText = copyRegistry.host
 
     /**
@@ -2150,11 +2132,6 @@ window.__ModuleLoader__.load({
       const native = clipboard.writeText.bind(clipboard)
       const wrapped = (value) => {
         const rewritten = rewriteCopiedText(String(value ?? ''))
-        if (rewritten === null && rowCopyText.size > 0) {
-          titleDiag.show(`写入没匹配上（登记 ${rowCopyText.size} 行，长度 ${String(value ?? '').length}）`)
-        } else if (rewritten !== null) {
-          titleDiag.clear()
-        }
         return rewritten === null ? native(value) : native(rewritten)
       }
       wrapped[CLIPBOARD_WRAPPED] = true
@@ -4995,15 +4972,11 @@ window.__ModuleLoader__.load({
           // 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) { titleDiag.show('按钮不在行内'); return }
+          if (row === null) return
           const key = row.getAttribute('data-chat-flow-key')
-          if (typeof key !== 'string' || key === '') { titleDiag.show('行没有 key'); return }
+          if (typeof key !== 'string' || key === '') return
           const text = rowCopyText.get(key)
-          if (typeof text !== 'string') {
-            titleDiag.show(`这一行没登记文本（登记 ${rowCopyText.size} 行）`)
-            return
-          }
-          titleDiag.clear()
+          if (typeof text !== 'string') return
           const write = primitives !== null && primitives !== undefined && typeof primitives.writeClipboard === 'function'
             ? primitives.writeClipboard
             : null

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.10'
+export const PLUGIN_VERSION = '0.5.12'
 
 // 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
@@ -699,10 +699,13 @@ function copyBlock(block) {
  * @returns `{ hidden, edits }`: one hidden entry per shadowed seq, and one
  *   record per landed rollback.
  */
-export function rollbackLedger(events) {
-  const folded = foldSurface(events)
-  const bySeq = new Map(events.map((event) => [event.seq, event]))
-  const turnOf = turnIndex(events)
+export function rollbackLedger(events, options = {}) {
+  // A caller that already folded the surface (the state route does) hands it in:
+  // folding twice over a long session is the single most expensive thing this
+  // route did - 31k events, and every fold walks all of them.
+  const folded = options.folded ?? foldSurface(events)
+  const bySeq = options.bySeq ?? new Map(events.map((event) => [event.seq, event]))
+  const turnOf = options.turnOf ?? turnIndex(events)
   const hidden = []
   const edits = []
   for (const replacement of folded.replacements) {
@@ -1244,8 +1247,8 @@ export function buildReplayWrites(events, plan, baseTurn, replayId, startSeq, pr
  * @returns one entry per editable prompt, in conversation order.
  */
 export function editableTurns(events, surfaceNodes, options = {}) {
-  const eventBySeq = new Map(events.map((event) => [event.seq, event]))
-  const turnOf = turnIndex(events)
+  const eventBySeq = options.bySeq ?? new Map(events.map((event) => [event.seq, event]))
+  const turnOf = options.turnOf ?? turnIndex(events)
   const head = surfaceNodes[0]
   const out = []
   // Surface order, not log order: a prompt rewritten in place travels as a
@@ -1301,10 +1304,15 @@ export function editableTurns(events, surfaceNodes, options = {}) {
  */
 export function editableReplies(events, surfaceNodes, options = {}) {
   const nodeIndex = new Map(surfaceNodes.map((seq, index) => [seq, index]))
-  const turnOf = turnIndex(events)
+  const turnOf = options.turnOf ?? turnIndex(events)
   const head = surfaceNodes[0]
   const patches = options.patches instanceof Map ? options.patches : new Map()
   const patch = options.patch === true
+  // `/state` publishes identities only, so the blocks never leave this function.
+  // Skipping them also skips the per-turn widening below, which walks every reply
+  // of the same turn for every reply: that widening is what made one refresh of a
+  // long session take seconds.
+  const light = options.light === true
   const out = []
   for (const event of events) {
     if (!isAssistantReply(event)) continue
@@ -1324,7 +1332,7 @@ export function editableReplies(events, surfaceNodes, options = {}) {
     // text of that turn is offered, each annotated with the seq it lives in.
     let listed = blocks.map((block) => ({ ...block, seq: event.seq }))
     const carriesAnswer = blocks.some((block) => block.kind === 'text' && block.text.trim() !== '')
-    if (patch && carriesAnswer && typeof turn === 'number') {
+    if (!light && patch && carriesAnswer && typeof turn === 'number') {
       const acrossTurn = []
       for (const other of events) {
         if (!isAssistantReply(other) || turnOf.get(other.seq) !== turn) continue
@@ -1338,6 +1346,7 @@ export function editableReplies(events, surfaceNodes, options = {}) {
       if (acrossTurn.length > listed.length) listed = acrossTurn
     }
     out.push({
+      ...(light ? {} : { blocks: listed }),
       seq: event.seq,
       turn: typeof turn === 'number' ? turn : null,
       messageId: messageIdOf(event) ?? null,
@@ -1763,7 +1772,9 @@ async function stateOf(ctx, sessionId, config) {
     }
   }
   const folded = foldSurface(events)
-  const ledger = rollbackLedger(events)
+  const bySeq = new Map(events.map((event) => [event.seq, event]))
+  const turnOf = turnIndex(events)
+  const ledger = rollbackLedger(events, { folded, bySeq, turnOf })
   const patches = patchesOf(events)
   // A patch whose target has been retired (a later rerun shadowed it) is inert:
   // the projection skips it and no row can show it. Reporting it in /state made
@@ -1777,8 +1788,8 @@ async function stateOf(ctx, sessionId, config) {
   // now, so a store that is unmounted (or not yet mounted) cannot leave the
   // editor promising to keep blocks the host would have to drop.
   const store = resolveAttachmentStore(ctx)
-  const turns = editableTurns(events, folded.nodes, { preview: canPreviewAttachments(store) })
-  const replies = editableReplies(events, folded.nodes, { patches, patch: config.patch === true }).map(lightReply)
+  const turns = editableTurns(events, folded.nodes, { preview: canPreviewAttachments(store), bySeq, turnOf })
+  const replies = editableReplies(events, folded.nodes, { patches, patch: config.patch === true, light: true, bySeq, turnOf }).map(lightReply)
   noteRequest({
     kind: 'state',
     sessionId,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-edit-turn",
-  "version": "0.5.10",
+  "version": "0.5.12",
   "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",
@@ -45,14 +45,17 @@
     "verify:ui": "node tools/verify-live-ui.mjs"
   },
   "dependencies": {
+    "@deepseek-ai/cordis": "^4.0.5",
+    "@deepseek-ai/dsh-scope": "^0.2.1-rc.5",
     "@deepseek-ai/schemastery": "^3.18.1"
   },
   "devDependencies": {
-    "@deepseek-ai/dsh-session": "^0.2.1-rc.2"
+    "@deepseek-ai/dsh-session": "^0.2.1-rc.2",
+    "@deepseek-ai/dsh-session-format-catalog": "^0.2.1-rc.5"
   },
   "peerDependencies": {
-    "@deepseek-ai/dsh-settings": "^0.1.0-rc.7 || ^0.1.1-rc.2 || ^0.1.2-alpha.2 || ^0.1.3-alpha.2 || ^0.1.5-alpha.1 || ^0.2.0-rc.1",
-    "@deepseek-ai/dsh-session": "^0.2.0-rc.1 || ^0.2.1-rc.2"
+    "@deepseek-ai/dsh-session": "^0.2.0-rc.1 || ^0.2.1-rc.2",
+    "@deepseek-ai/dsh-settings": "^0.1.0-rc.7 || ^0.1.1-rc.2 || ^0.1.2-alpha.2 || ^0.1.3-alpha.2 || ^0.1.5-alpha.1 || ^0.2.0-rc.1"
   },
   "dsh": {
     "bundle": {