dsh-as-aistudio 0.2.0 → 0.2.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-as-aistudio",
-  "version": "0.2.0",
+  "version": "0.2.2",
   "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",
@@ -47,7 +47,7 @@
   },
   "dependencies": {
     "dsh-delete-turn": "^0.1.7",
-    "dsh-edit-turn": "^0.2.15",
+    "dsh-edit-turn": "^0.2.25",
     "dsh-markdown-bubble": "^0.1.4",
     "dsh-rerun-turn": "^0.1.24"
   },
@@ -71,7 +71,7 @@
     "compatibility": {
       "dshReleases": {
         "0.1.6-alpha.2": "compatible",
-        "0.2.0-rc.1": "compatible"
+        "0.2.2-rc.1": "compatible"
       }
     }
   }

package/src/client.js

@@ -4779,7 +4779,7 @@ window.__ModuleLoader__.load({
   }
 
     // --- which factories the studio mounts ---
-    const VENDORED_TABLE = { factory_dsh_edit_turn, factory_dsh_rerun_turn, factory_dsh_delete_turn, factory_dsh_markdown_bubble }
+    const VENDORED_TABLE = { "dsh-edit-turn": factory_dsh_edit_turn, "dsh-rerun-turn": factory_dsh_rerun_turn, "dsh-delete-turn": factory_dsh_delete_turn, "dsh-markdown-bubble": factory_dsh_markdown_bubble }
 
     // --- the studio own browser half ---
 
@@ -4795,7 +4795,7 @@ window.__ModuleLoader__.load({
     const NS = 'dsh-as-aistudio'
 
     /** Keep in sync with package.json and src/components.js. */
-    const PLUGIN_VERSION = '0.2.0'
+    const PLUGIN_VERSION = '0.2.2'
 
     /** 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.0'
+export const STUDIO_VERSION = '0.2.2'
 
 /**
  * The AI Studio action strip this studio composes, in the order the user meets

package/src/index.js

@@ -64,22 +64,32 @@ export async function mountComponentHosts(ctx) {
       mounted.set(component.id, { mounted: false, reason: 'no-plugin-shape' })
       continue
     }
+    // A mount is only honest once the fiber has SETTLED. A component whose
+    // apply() threw synchronously is caught below, but one whose start failed
+    // asynchronously — a tool registered twice, a route refused — rejects after
+    // ctx.plugin() has already returned. Reporting it optimistically as mounted
+    // is exactly the lie that shipped: the host log said "failed to start" while
+    // /status said 4/4. So the record is created optimistic and CORRECTED when
+    // the fiber settles, and statusOf reads the record, not the promise.
+    const record = { mounted: true, reason: null, dispose: null }
     try {
       const fiber = ctx.plugin(plugin)
-      // A rejected start is still a start we know nothing about yet: record it
-      // when it settles rather than leaving the row silently half-mounted.
+      if (fiber !== undefined && fiber !== null && typeof fiber.dispose === 'function') {
+        record.dispose = fiber.dispose.bind(fiber)
+      }
       if (fiber && typeof fiber.then === 'function') {
-        fiber.then(
+        Promise.resolve(fiber).then(
           () => {},
           (error) => {
+            record.mounted = false
+            record.reason = 'start-failed'
             console.error('[' + STUDIO_ID + '] component ' + component.id + ' failed to start:', error)
           },
         )
       }
-      mounted.set(component.id, {
-        mounted: true,
-        dispose: typeof fiber !== 'undefined' && fiber !== null && typeof fiber.dispose === 'function' ? fiber.dispose.bind(fiber) : null,
-      })
+      // A synchronous apply failure also arrives as a throw here, and it must win
+      // over the optimistic value we are about to store.
+      mounted.set(component.id, record)
     } catch (error) {
       mounted.set(component.id, { mounted: false, reason: 'apply-failed', detail: String((error && error.message) || error) })
     }

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.0'
+    const PLUGIN_VERSION = '0.2.2'
 
     /** Presence marker the live verifier reads. */
     const DEBUG_KEY = '__DSH_AS_AISTUDIO__'

package/tools/build.mjs

@@ -61,7 +61,17 @@ for (const id of ORDER) {
   parts.push('    // ===== vendored ' + id + ' =====')
   parts.push('    const factory_' + id.replace(/-/g, '_') + ' = ' + vendored(id))
 }
-parts.push(read('tail.txt').replace('__T__', ORDER.map((id) => 'factory_' + id.replace(/-/g, '_')).join(', ')))
+// The table is keyed by COMPONENT ID, because that is what the studio looks up
+// (VENDORED_TABLE[id] with id like 'dsh-edit-turn'). Emitting the bare factory
+// names would build an object whose keys are the variable names instead, every
+// lookup would be undefined, and every component would silently fail to mount -
+// which is exactly the regression this line now prevents.
+parts.push(
+  read('tail.txt').replace(
+    '__T__',
+    ORDER.map((id) => JSON.stringify(id) + ': factory_' + id.replace(/-/g, '_')).join(', '),
+  ),
+)
 parts.push(studio)
 parts.push(read('tail2.txt'))