@thehammer/danx-dashboard-mcp 0.1.326 → 0.1.328

package/dist/design-screenshot.js

@@ -21,6 +21,15 @@ const READY_WAIT_MS = 5000;
 export const READY_MESSAGE = "danx-design:ready";
 const READY_LISTENER = `window.__danxDesignReady = false; window.addEventListener("message", function (e) { ` +
     `if (e.data && e.data.type === ${JSON.stringify(READY_MESSAGE)}) window.__danxDesignReady = true; });`;
+/**
+ * DX-5350: the screenshot renders the page outside `SandboxedFrame`, so `window.danxMiniApp` (which the frame injects, `@danxbot/ui` 27.0.0)
+ * is restated here as a stub: `holdReady`, a once-only `ready` and ready-on-`load` unless held, posting the same ready message the listener
+ * above hears. `reportError` and `onAppearance` are no-ops so a harness never throws.
+ */
+const MINI_APP_STUB = `(function () { var held = false, posted = false; ` +
+    `function ready() { if (posted) return; posted = true; window.postMessage({ type: ${JSON.stringify(READY_MESSAGE)} }, "*"); } ` +
+    `window.danxMiniApp = { holdReady: function () { held = true; }, ready: ready, reportError: function () {}, onAppearance: function () { return function () {}; } }; ` +
+    `window.addEventListener("load", function () { if (!held) ready(); }); })();`;
 function none(reason) {
     return { png_base64: null, screenshot: `none: ${reason}` };
 }
@@ -93,15 +102,21 @@ export async function captureWith(where, offline, drive) {
         await browser.close().catch((err) => console.error(`[danx-dashboard-mcp] could not close the browser: ${oneLine(err)}`));
     }
 }
+/** DX-5319: where the built html is served for the capture: a route answered by the capture itself, so the page is a real navigation and never reaches a network. */
+const DESIGN_PAGE_URL = "https://design.invalid/";
 /**
- * Capture `html` as a 1280x800 viewport PNG (device scale 1, network disabled). The page is `setContent`-ed, then given a short wait
- * for the harness's ready message (the page has loaded either way).
+ * Capture `html` as a 1280x800 viewport PNG (device scale 1, network disabled). The page is served from a route and navigated to, never
+ * `setContent`-ed: an init script runs on a navigation and not on `setContent`'s `document.write`, so the ready listener and the
+ * `window.danxMiniApp` stub would not exist and every capture would wait out the whole ready timeout (DX-5319, measured: 5.2 s, the stub
+ * undefined). It then gets a short wait for the harness's ready message (the page has loaded either way).
  */
 export function captureScreenshot(html, where) {
     return captureWith(where, true, async (page) => {
         await page.addInitScript(READY_LISTENER);
-        await page.setContent(html, { waitUntil: "load" });
-        // The page has loaded; a design that never posts ready (it threw, or its harness is older) is still captured as it stands.
+        await page.addInitScript(MINI_APP_STUB);
+        await page.route(DESIGN_PAGE_URL, (route) => route.fulfill({ status: 200, contentType: "text/html; charset=utf-8", body: html }));
+        await page.goto(DESIGN_PAGE_URL, { waitUntil: "load" });
+        // The page has loaded; a design that never posts ready (it threw) is still captured as it stands.
         await page.waitForFunction("window.__danxDesignReady === true", undefined, { timeout: READY_WAIT_MS }).catch(() => undefined);
     });
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@thehammer/danx-dashboard-mcp",
-  "version": "0.1.326",
+  "version": "0.1.328",
   "description": "Stdio MCP server fronting danxbot's dashboard API for agents (operator sessions and dispatched workers alike): the generic danxbot_api/danxbot_api_list/danxbot_api_spec tools reach any route the caller's credential may call, plus dedicated agent-specific tools (plan_connect, request_permission, attach_file, and danxbot_slack_reply) (DX-3465, PLN-11 R-25).",
   "license": "MIT",
   "type": "module",