dsh-as-aistudio 0.2.4 → 0.2.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-as-aistudio",
-  "version": "0.2.4",
+  "version": "0.2.5",
   "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; the four components are dependencies mounted by it. Each still installs alone and works alone.",
   "type": "module",
   "main": "src/index.js",
@@ -48,7 +48,7 @@
     "verify:manifest": "node tools/verify-manifest.mjs"
   },
   "dependencies": {
-    "dsh-delete-turn": "^0.1.7",
+    "dsh-delete-turn": "^0.1.8",
     "dsh-edit-turn": "^0.2.16",
     "dsh-markdown-bubble": "^0.1.4",
     "dsh-rerun-turn": "^0.1.24"

package/src/client.js

@@ -3131,6 +3131,8 @@ window.__ModuleLoader__.load({
         this.animateOnce = false
         this.view = Object.freeze({
           hidden: new Map(),
+          // replacement seq -> the seq it was recorded as shadowing. See hiddenViaOf.
+          hiddenVia: new Map(),
           surface: new Set(),
           replyTurns: new Set(),
           edits: new Map(),
@@ -3195,6 +3197,7 @@ window.__ModuleLoader__.load({
             for (const item of Array.isArray(data.hidden) ? data.hidden : []) {
               if (item && typeof item.seq === 'number') hidden.set(item.seq, typeof item.mode === 'string' ? item.mode : 'message')
             }
+            const hiddenVia = hiddenViaOf(data.hidden)
             const surface = new Set()
             for (const seq of Array.isArray(data.surface) ? data.surface : []) surface.add(seq)
             const replyTurns = new Set()
@@ -3208,7 +3211,7 @@ window.__ModuleLoader__.load({
               if (typeof turn === 'number') markerTurns.add(turn)
             }
             const surfaceThrough = typeof data.lastSeq === 'number' ? data.lastSeq : -1
-            this.publish({ hidden, surface, replyTurns, edits, markerTurns, surfaceReady: true, surfaceThrough, loaded: true, loadError: false })
+            this.publish({ hidden, hiddenVia, surface, replyTurns, edits, markerTurns, surfaceReady: true, surfaceThrough, loaded: true, loadError: false })
           })
           .catch(() => {
             this.publish({ loadError: true })
@@ -3260,8 +3263,10 @@ window.__ModuleLoader__.load({
           for (const item of Array.isArray(data.hidden) ? data.hidden : []) {
             if (item && typeof item.seq === 'number') hidden.set(item.seq, typeof item.mode === 'string' ? item.mode : target.mode)
           }
+          const hiddenVia = new Map(this.view.hiddenVia)
+          for (const [via, seq] of hiddenViaOf(data.hidden)) hiddenVia.set(via, seq)
           this.animateOnce = true
-          this.publish({ pending: false, dialog: null, hidden, loaded: true, loadError: false })
+          this.publish({ pending: false, dialog: null, hidden, hiddenVia, loaded: true, loadError: false })
         } catch {
           this.publish({ pending: false, failure: 'generic' })
         }
@@ -3419,9 +3424,27 @@ window.__ModuleLoader__.load({
       return seq === target.seq ? target : { ...target, seq }
     }
 
-    function isRowHidden(hidden, seqs) {
+    // Index a deletion by the REPLACEMENT it landed, not only by the seq it
+    // shadowed. A deleted message leaves a carrier standing in its row, and the
+    // transcript row names the live node (the carrier), so a row checked against
+    // 'hidden' alone never matches: the deletion succeeds, the model context
+    // loses the message, and the row is simply never hidden - an empty bubble
+    // left behind. The link is already on every hidden entry ('replacement'), so
+    // it is indexed here rather than derived from 'edits': that chain
+    // deliberately excludes this plugin's own carriers (they rewrite context,
+    // not content) and so cannot bridge the gap.
+    function hiddenViaOf(items) {
+      const via = new Map()
+      for (const item of Array.isArray(items) ? items : []) {
+        if (item && typeof item.seq === 'number' && typeof item.replacement === 'number') via.set(item.replacement, item.seq)
+      }
+      return via
+    }
+
+    function isRowHidden(hidden, hiddenVia, seqs) {
       for (const seq of seqs) {
         if (hidden.has(seq)) return true
+        if (hiddenVia !== undefined && hiddenVia.has(seq)) return true
       }
       return false
     }
@@ -3684,13 +3707,13 @@ window.__ModuleLoader__.load({
           const anchor = anchorKey === undefined ? undefined : snapshot.nodes.get(anchorKey)
           if (anchor !== undefined) {
             const seqs = expandSeqs(seqsFor(anchor), view.edits)
-            setRowHidden(row, isRowHidden(view.hidden, seqs), animate)
+            setRowHidden(row, isRowHidden(view.hidden, view.hiddenVia, seqs), animate)
           }
           continue
         }
         const seqs = expandSeqs(seqsFor(node), view.edits)
         for (const seq of seqs) if (typeof seq === 'number' && seq > maxSeq) maxSeq = seq
-        const hidden = isRowHidden(view.hidden, seqs)
+        const hidden = isRowHidden(view.hidden, view.hiddenVia, seqs)
         setRowHidden(row, hidden, animate)
         const target = hidden ? null : liveTarget(targetFor(node), view.edits)
         const covered = target !== null || slotCoversRow(node)
@@ -4795,7 +4818,7 @@ window.__ModuleLoader__.load({
     const NS = 'dsh-as-aistudio'
 
     /** Keep in sync with package.json and src/components.js. */
-    const PLUGIN_VERSION = '0.2.4'
+    const PLUGIN_VERSION = '0.2.5'
 
     /** Presence marker the live verifier reads. */
     const DEBUG_KEY = '__DSH_AS_AISTUDIO__'

package/src/components.js

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

package/src/studio.js

@@ -49,7 +49,7 @@ window.__ModuleLoader__.load({
     const NS = 'dsh-as-aistudio'
 
     /** Keep in sync with package.json and src/components.js. */
-    const PLUGIN_VERSION = '0.2.4'
+    const PLUGIN_VERSION = '0.2.5'
 
     /** Presence marker the live verifier reads. */
     const DEBUG_KEY = '__DSH_AS_AISTUDIO__'

package/src/vendor/dsh-delete-turn.js

@@ -1,6 +1,6 @@
 // dsh-delete-turn browser half, VENDORED by tools/vendor.mjs - DO NOT EDIT.
 //
-// Source: ../dsh-delete-turn/src/client.js  (v0.1.7, sha256 516518d8b6b8e5e4)
+// Source: ../dsh-delete-turn/src/client.js  (v0.1.8, sha256 09de24fd97ecd4ec)
 // Regenerate with `npm run vendor`; test/vendor-sync.test.js fails when this is stale.
 //
 // The component registers its factory with the host module loader; inside this
@@ -154,6 +154,8 @@ export default (require) => {
         this.animateOnce = false
         this.view = Object.freeze({
           hidden: new Map(),
+          // replacement seq -> the seq it was recorded as shadowing. See hiddenViaOf.
+          hiddenVia: new Map(),
           surface: new Set(),
           replyTurns: new Set(),
           edits: new Map(),
@@ -218,6 +220,7 @@ export default (require) => {
             for (const item of Array.isArray(data.hidden) ? data.hidden : []) {
               if (item && typeof item.seq === 'number') hidden.set(item.seq, typeof item.mode === 'string' ? item.mode : 'message')
             }
+            const hiddenVia = hiddenViaOf(data.hidden)
             const surface = new Set()
             for (const seq of Array.isArray(data.surface) ? data.surface : []) surface.add(seq)
             const replyTurns = new Set()
@@ -231,7 +234,7 @@ export default (require) => {
               if (typeof turn === 'number') markerTurns.add(turn)
             }
             const surfaceThrough = typeof data.lastSeq === 'number' ? data.lastSeq : -1
-            this.publish({ hidden, surface, replyTurns, edits, markerTurns, surfaceReady: true, surfaceThrough, loaded: true, loadError: false })
+            this.publish({ hidden, hiddenVia, surface, replyTurns, edits, markerTurns, surfaceReady: true, surfaceThrough, loaded: true, loadError: false })
           })
           .catch(() => {
             this.publish({ loadError: true })
@@ -283,8 +286,10 @@ export default (require) => {
           for (const item of Array.isArray(data.hidden) ? data.hidden : []) {
             if (item && typeof item.seq === 'number') hidden.set(item.seq, typeof item.mode === 'string' ? item.mode : target.mode)
           }
+          const hiddenVia = new Map(this.view.hiddenVia)
+          for (const [via, seq] of hiddenViaOf(data.hidden)) hiddenVia.set(via, seq)
           this.animateOnce = true
-          this.publish({ pending: false, dialog: null, hidden, loaded: true, loadError: false })
+          this.publish({ pending: false, dialog: null, hidden, hiddenVia, loaded: true, loadError: false })
         } catch {
           this.publish({ pending: false, failure: 'generic' })
         }
@@ -442,9 +447,27 @@ export default (require) => {
       return seq === target.seq ? target : { ...target, seq }
     }
 
-    function isRowHidden(hidden, seqs) {
+    // Index a deletion by the REPLACEMENT it landed, not only by the seq it
+    // shadowed. A deleted message leaves a carrier standing in its row, and the
+    // transcript row names the live node (the carrier), so a row checked against
+    // 'hidden' alone never matches: the deletion succeeds, the model context
+    // loses the message, and the row is simply never hidden - an empty bubble
+    // left behind. The link is already on every hidden entry ('replacement'), so
+    // it is indexed here rather than derived from 'edits': that chain
+    // deliberately excludes this plugin's own carriers (they rewrite context,
+    // not content) and so cannot bridge the gap.
+    function hiddenViaOf(items) {
+      const via = new Map()
+      for (const item of Array.isArray(items) ? items : []) {
+        if (item && typeof item.seq === 'number' && typeof item.replacement === 'number') via.set(item.replacement, item.seq)
+      }
+      return via
+    }
+
+    function isRowHidden(hidden, hiddenVia, seqs) {
       for (const seq of seqs) {
         if (hidden.has(seq)) return true
+        if (hiddenVia !== undefined && hiddenVia.has(seq)) return true
       }
       return false
     }
@@ -707,13 +730,13 @@ export default (require) => {
           const anchor = anchorKey === undefined ? undefined : snapshot.nodes.get(anchorKey)
           if (anchor !== undefined) {
             const seqs = expandSeqs(seqsFor(anchor), view.edits)
-            setRowHidden(row, isRowHidden(view.hidden, seqs), animate)
+            setRowHidden(row, isRowHidden(view.hidden, view.hiddenVia, seqs), animate)
           }
           continue
         }
         const seqs = expandSeqs(seqsFor(node), view.edits)
         for (const seq of seqs) if (typeof seq === 'number' && seq > maxSeq) maxSeq = seq
-        const hidden = isRowHidden(view.hidden, seqs)
+        const hidden = isRowHidden(view.hidden, view.hiddenVia, seqs)
         setRowHidden(row, hidden, animate)
         const target = hidden ? null : liveTarget(targetFor(node), view.edits)
         const covered = target !== null || slotCoversRow(node)