dsh-session-rewind 1.3.1 → 1.3.2

package/lib/client.js

@@ -66,7 +66,7 @@ loader.load({
     const CUTS_ROUTE = '/api/session-rewind.cuts';
     const DIAGNOSE_ROUTE = '/api/session-rewind.diagnose';
     const TAIL_ROUTE = '/api/session-rewind.tail';
-    const BOOT_REVISION = '1.3.1';
+    const BOOT_REVISION = '1.3.2';
     const HIDDEN_ATTR = 'data-session-rewind-hidden';
 
     /** One per-message bridge per session, shared by every TurnRewind instance. */
@@ -951,11 +951,10 @@ loader.load({
       };
 
       const runRewind = async (seq, restoreFiles, notice, fromTurn = null) => {
-        notice(tr('notice.working'));
         try {
+          notice(tr('notice.working'));
           const result = await postJson(INPLACE_ROUTE, { sessionId, seq, restoreFiles });
           if (state.disposed) return;
-          if (Number.isInteger(result.hiddenFromTurn)) state.cutTurn = result.hiddenFromTurn;
           if (Array.isArray(result.hiddenTurns) && result.hiddenTurns.length > 0) {
             for (const turn of result.hiddenTurns) {
               if (Number.isInteger(turn)) state.hiddenTurns.add(turn);
@@ -1005,7 +1004,7 @@ loader.load({
         }
       };
 
-      const openPopover = (anchor, seq, notice) => {
+      const openPopover = (anchor, seq, notice, turn = null) => {
         closePopover();
         const popover = document.createElement('div');
         popover.setAttribute('data-session-rewind-popover', '');
@@ -1048,7 +1047,18 @@ loader.load({
           button.addEventListener('mouseleave', () => {
             button.style.background = 'transparent';
           });
-          button.addEventListener('click', onClick);
+          button.addEventListener('click', () => {
+            try {
+              onClick();
+            } catch (error) {
+              sendDiagnose(sessionId, {
+                reason: 'option-failed',
+                error: String(error && error.message ? error.message : error),
+                boot: BOOT_REVISION,
+                sequence: seq,
+              });
+            }
+          });
           return button;
         };
         popover.appendChild(makeButton(tr('popover.chat'), () => {
@@ -1186,7 +1196,7 @@ loader.load({
             event.preventDefault();
             event.stopPropagation();
             sendDiagnose(sessionId, { reason: 'click', turn, seq, buttons: state.buttons.size, mapped: state.turnToSeq.size });
-            openPopover(button, seq, notice);
+            openPopover(button, seq, notice, turn);
           });
           anchor.appendChild(button);
           state.buttons.set(row, button);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-session-rewind",
-  "version": "1.3.1",
+  "version": "1.3.2",
   "type": "module",
   "description": "Rewind a finished Turn: branch the conversation at any Turn (session header or per-Turn control) and restore every workspace file that Turn changed. · 回合回退：从任意已完成的回合重新开始，并把该回合改动过的工作区文件恢复原状。",
   "icon": "./icon.svg",

package/test/client-smoke.mjs

@@ -8,6 +8,7 @@
 
 import assert from 'node:assert/strict';
 import { readFile } from 'node:fs/promises';
+import { readFileSync } from 'node:fs';
 
 //#region stubs
 const listeners = new Map();
@@ -437,4 +438,15 @@ assert.match(textOf(headerTree), /session-rewind\.notice\.unavailable/u);
 assert.equal(forkCalls.length, 0);
 //#endregion
 
+// --- popover wiring --------------------------------------------------------
+// v1.3.1 shipped a popover whose option callbacks referenced a `turn` the
+// enclosing function never declared. That throws inside a DOM listener, which the
+// browser swallows: clicking a control did nothing at all. Pin the wiring.
+{
+  const source = readFileSync(new URL('../lib/client.js', import.meta.url), 'utf8');
+  assert.match(source, /const openPopover = \(anchor, seq, notice, turn/u);
+  assert.match(source, /runRewind\(seq, false, notice, turn\)/u);
+  assert.match(source, /runRewind\(seq, true, notice, turn\)/u);
+}
+
 console.log('client smoke: all assertions passed');