dsh-edit-turn 0.2.4 → 0.2.6

package/README.md

@@ -218,6 +218,15 @@ ln -s /path/to/dsh-install/node_modules ./node_modules
 
 ### 更新日志
 
+**0.2.6** —— 修「改写过的提示词，再点它的笔没反应」。
+
+- **修复：折叠改写行上的铅笔是死的**。改写后的提示词由一个替换事件承载，而平台**不为替换事件渲染行**——于是"哪一行覆盖被编辑的 seq"永远匹配不上，编辑器根本不渲染，点击无声无息（无报错）。现在"覆盖"也认**该行的替换指向**（`revisions.get(行seq) === 编辑seq`），并且编辑器自 0.2.3 起挂在独立浮层上，折叠行承载它没有任何问题。新增用例「改写过的提示词，点笔仍能打开编辑器（预填替代文本）」+ 1 条静态检查；真机复现：重载后点击折叠行上的笔，编辑器打开并预填「回复2」。
+
+**0.2.5** —— 修「每保存一次，模型的回答往下掉一格」。
+
+- **修复：被回滚的「轮次尾巴」没有隐藏，堆成了一列空条**。回复编辑会留下旧轮次的尾巴（用时/用量/动作条）——它不是消息行，属于它的消息已经被回滚，尾巴上没有任何可操作的东西。此前我们按"消息行"的规则给它保留了动作条（那条规则本是为了让被回滚的用户消息还能复制/删除），于是**每保存一次就多一条空条**，把对话一格格顶下去。现在按 `data-chat-flow-kind="turn-tail"` 识别：尾巴整条隐藏（并清理旧版本给它打过的 `data-dshet-keep-actions`），消息行照旧保留动作条。新增用例「被回滚的轮次尾巴整条消失，而不是叠一条空条」+ 2 条静态检查；`verify:ui` 的现场断言改为三条事实——轮次尾巴整条隐藏、没有动作条的行整条隐藏、有动作条的消息行必须保住动作条（真机 6 条孤儿尾巴全部按新规则消失）。
+- 补一句 0.2.4 的发布说明：`verify:ui` 与 `verify:live` 在 0.2.4 上均已通过；本缺陷是发布后由使用现场发现的。
+
 **0.2.4** —— 修一个**会把会话写坏**的致命缺陷（回复编辑的写入非法），并把「读取路径校验」补进契约测试。
 
 - **修复（致命）：回复编辑会把整份会话写到读不出来**。旧写法把两样非法内容写进日志：①静默载体是一个空的 `system/message`，而格式**只允许 system-prompt 来源的 system 消息**；②修正消息带着**已经关闭的**原轮次 turn/step 追加。追加是宽松的（当场不报错），但**读取**会拒绝：DSH 从此对该会话返回 `session-not-found`，「历史加载失败：stored log is corrupt: SessionFormatError: system/message does not match an open turn and step」。现场表现就是那个会话里所有编辑入口消失、历史打不开。现在：回复编辑先在日志里**开一个新轮次**（`turn/start` + `step/start`），静默载体改为**空 `developer/message`**（空内容不投影给模型、替换不进转录、格式合法），修正消息与 `step/end`/`turn/end` 都在这个轮次内收尾；运行时从 `turn/start` 事件投影 `lastTurn`，所以它下一轮用 N+2，序号一致。

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.4'
+    const PLUGIN_VERSION = '0.2.6'
 
     // --- copy -----------------------------------------------------------------
 
@@ -804,10 +804,25 @@ window.__ModuleLoader__.load({
     }
 
     function setRowHidden(row, hide, animate) {
+      // A turn tail is not a message row. It is the strip a turn's end renders
+      // (duration, usage, actions); when the turn behind it is rolled back the
+      // strip is an orphan with nothing left to act on. Keeping it, which is
+      // right for a message row, stacked one empty strip per edit and pushed
+      // the conversation down the page - the report was "every save moves the
+      // model's answer further down".
+      const isTail = row.getAttribute('data-chat-flow-kind') === 'turn-tail'
       if (hide) {
-        const keepsStrip = collapseRowContent(row, true)
-        if (keepsStrip && row.dataset.dshetKeepActions !== '1') row.dataset.dshetKeepActions = '1'
-        if (row.dataset.dshetHidden === '1') return
+        const keepsStrip = isTail ? false : collapseRowContent(row, true)
+        if (keepsStrip) {
+          if (row.dataset.dshetKeepActions !== '1') row.dataset.dshetKeepActions = '1'
+        } else if (row.dataset.dshetKeepActions === '1') {
+          // Older passes marked this tail as keep-actions; with the rule fixed
+          // the attribute has to go, or the row stays visible through it.
+          delete row.dataset.dshetKeepActions
+        }
+        if (row.dataset.dshetHidden === '1') {
+          if (keepsStrip || row.style.display === 'none') return
+        }
         row.dataset.dshetHidden = '1'
         if (keepsStrip) return
         if (!animate || typeof requestAnimationFrame !== 'function') {
@@ -1237,12 +1252,18 @@ window.__ModuleLoader__.load({
         // The editor anchors to whichever row covers the seq being edited, not to
         // the row that owns the entry: a reply's entry now lives in the turn-tail
         // action strip, while its editor still belongs under the reply text.
-        if (
-          !hidden &&
-          view.editing !== null &&
-          typeof view.editing.seq === 'number' &&
-          seqsFor(node).includes(view.editing.seq)
-        ) {
+        //
+        // A rewritten prompt is covered by no row at all: the revision is a
+        // replacement event and the platform draws no row for those, so its seq
+        // appears in no node. The row that STANDS for it - the collapsed one
+        // whose pencil was just pressed - covers it through its revision
+        // mapping, and the editor is hosted on the layer anyway, so rendering it
+        // under a collapsed row costs nothing and works.
+        const editing = view.editing
+        const coversEditing = editing !== null && typeof editing.seq === 'number' &&
+          (seqsFor(node).includes(editing.seq) ||
+            seqsFor(node).some((seq) => view.revisions.get(seq) === editing.seq))
+        if (coversEditing && (!hidden || row.style.display !== 'none')) {
           renderEditor(row, controller, view, 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.4;
+// The silent replacement carrier is an EMPTY `developer/message` (0.2.6;
 // 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.4'
+export const PLUGIN_VERSION = '0.2.6'
 
 // 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.4",
+  "version": "0.2.6",
   "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",