comfyui-mcp 0.52.55 → 0.52.57

package/dist/orchestrator/panel-tools.js

@@ -58,6 +58,26 @@ import { clearSwitchHold, describeSwitchHold, recordSwitchHold, successProvesSwi
 import { NO_ORIGIN_REMEDY } from "./fence-refusal.js";
 import { completeGetErrorsAudit } from "./get-errors-audit.js";
 import { applyKitchenRecommendation, assessKitchen, assessKitchenGraph, extractLoaders, gatherKitchenStatus, kitchenProactiveHint, } from "../services/kitchen.js";
+export function resolveClientKind(ctx) {
+    const b = ctx.bridge;
+    const routed = typeof b?.liveTabIdFor === "function" ? b.liveTabIdFor(ctx.tabId) : undefined;
+    // A live resolution wins. When nothing resolves: a CONCRETE id still names
+    // the mailbox recipient; a SCOPE address names nobody.
+    // `isScopeAddress` is typed `id is string`; its false branch would otherwise
+    // narrow a `string` tabId to `never` before `.length` is read (tsc TS2339).
+    const address = ctx.tabId;
+    const destination = typeof routed === "string" && routed.length > 0
+        ? routed
+        : typeof address === "string" && address.length > 0 && !isScopeAddress(address)
+            ? address
+            : undefined;
+    if (destination == null)
+        return { kind: "unroutable" };
+    if (typeof b?.isHeadless === "function" && b.isHeadless(destination)) {
+        return { kind: "headless", tabId: destination };
+    }
+    return { kind: "interactive", tabId: destination };
+}
 /** #884 — journal TICKETS (run completions #468, ask answers #486) must be
  *  keyed by the REAL tab a run/card was routed to: the panel reports back under
  *  that tab id, and a ticket keyed by the shared scope address a tool ctx is
@@ -738,6 +758,10 @@ export const __panelToolsTestHooks = {
      *  it fails even where the handler suite does not reach. The handler CALL SITE
      *  is covered separately (deleting the wiring must also fail a named test). */
     annotateShowMediaAck,
+    /** #2012 — the destination-kind helper. Handler call sites are covered
+     *  separately (reverting `panel_show_media` to `isHeadless` on the address
+     *  must also fail a named test). */
+    resolveClientKind,
     /** Direct access to the #742 decline recheck loop so its hard-deadline
      *  guarantee (codex gate r2) can be unit-tested with a custom deadline. */
     probeDeclineRecovery,
@@ -3050,6 +3074,15 @@ function annotateShowMediaAck(res, dispatched) {
                      *  "partial_accounting" or "mailboxed". Structured so a caller can
                      *  branch on it without reading the prose. */
                     unaccounted_because: verdict.reason,
+                    /** #2013 — only on a mailbox receipt. A concrete tab id is a known
+                     *  recipient; `null` means the box is scope-keyed (whoever hellos
+                     *  next). Structured so a caller does not have to parse the prose. */
+                    ...(verdict.reason === "mailboxed"
+                        ? {
+                            queued_for: verdict.queuedFor ?? null,
+                            recipient_known: verdict.queuedFor != null,
+                        }
+                        : {}),
                     /** EVERY dispatched item, even under partial accounting: a reply
                      *  that covers 1 of 2 does not say WHICH, so none can be settled. */
                     unaccounted: dispatched.map((d) => ({ filename: d.filename, kind: d.kind })),
@@ -16420,8 +16453,14 @@ CHECKED FOR YOU: the graph read this message prescribes was just run, and it ` +
                     // bytes HERE and inline them as a data URL. Best-effort: any failure
                     // (fetch error, non-media, too big) falls back to forwarding the ref,
                     // which the client renders as a caption card.
+                    //
+                    // #2012 — judge the DESTINATION, not the address. `ctx.tabId` may be
+                    // a scope / prefix / alias; `isHeadless` on the address is a lookup
+                    // miss that answers false and a phone gets a /view URL it cannot
+                    // fetch. Positive requirement: inline only a proven headless
+                    // destination.
                     let inlined = false;
-                    if (ctx.bridge.isHeadless(ctx.tabId)) {
+                    if (resolveClientKind(ctx).kind === "headless") {
                         try {
                             const base = (process.env.COMFYUI_URL ?? "http://127.0.0.1:8188").replace(/\/+$/, "");
                             const qs = new URLSearchParams({ filename: src.filename, type: src.type ?? "output" });