comfyui-mcp 0.52.143 → 0.52.144

package/dist/orchestrator/panel-tools.js

@@ -38,6 +38,7 @@ import { getModelInventoryDisclosure, } from "../services/model-inventory-disclo
 import { sanitizePanelUpdateNodeResult } from "../services/manager-update-error.js";
 import { formatQueueStatusPartialNote, getManifestPartialLeftover, } from "../services/manifest-partial.js";
 import { searchPanelNodes } from "../services/manager-node-search.js";
+import { listPanelNodes } from "../services/manager-node-list.js";
 import { isPanelAnsweredError } from "../services/panel-answered.js";
 import { readWorkflowListReadiness, workflowListReadinessOf, } from "../services/panel-workflow-readiness.js";
 import { isPreExecutorRefusal } from "../services/panel-refusal.js";
@@ -7185,6 +7186,38 @@ function acceptedPromptId(parsed) {
 function isPanelQueueUncertainty(parsed) {
     return parsed?.queued_unknown === true;
 }
+/**
+ * #2438 — the live-canvas surface has no render-queue inspector. Panel
+ * `retry_guidance` historically named `queue (action:"list")`, which is a
+ * headless stdio tool and may be unreachable (ECONNREFUSED) even while the
+ * panel is connected. The only safe next step on this surface is to wait for
+ * an eventual UNDETERMINED completion and not re-dispatch.
+ */
+const PANEL_QUEUED_UNKNOWN_RETRY_GUIDANCE = `Do NOT re-run panel_run: the /prompt request already left the panel, and a second ` +
+    `dispatch may queue a duplicate render. This live-canvas surface has no render-queue ` +
+    `inspection tool. End your turn and wait for an eventual UNDETERMINED completion ` +
+    `event for this tab; if one arrives, treat it as this run's possible outcome rather ` +
+    `than queuing another.`;
+function applyQueuedUnknownRetryGuidance(res) {
+    const parsed = parseToolResultJson(res);
+    if (!parsed || !isPanelQueueUncertainty(parsed))
+        return res;
+    if (parsed.retry_guidance === PANEL_QUEUED_UNKNOWN_RETRY_GUIDANCE)
+        return res;
+    return {
+        ...res,
+        content: [
+            {
+                type: "text",
+                text: JSON.stringify({
+                    ...parsed,
+                    retry_guidance: PANEL_QUEUED_UNKNOWN_RETRY_GUIDANCE,
+                }),
+            },
+            ...res.content.slice(1),
+        ],
+    };
+}
 function detectRunRejection(res) {
     // Bridge/transport/executor error: never a queue. Preserve it verbatim (#248),
     // no success note (#331). fail() already carries err.message (incl. any stack).
@@ -7194,9 +7227,9 @@ function detectRunRejection(res) {
     if (!parsed)
         return null; // unparseable non-error reply — don't regress a success
     // #2143 — `queued_unknown` is an explicit Panel receipt, not the ComfyUI
-    // rejection channel. Its `error` field explains the timeout and its
-    // `retry_guidance` tells the caller how to settle it; keep the receipt intact
-    // and never redispatch from here.
+    // rejection channel. Its `error` field explains the timeout. Keep the
+    // receipt intact and never redispatch from here; the handler rewrites
+    // `retry_guidance` onto this surface (#2438) after this gate.
     if (isPanelQueueUncertainty(parsed))
         return null;
     const topError = parsed.error;
@@ -7336,6 +7369,8 @@ export const __panelRunTestHooks = {
     isRetryableRunToNodeStampRace,
     isSeedRunToNodeStampRace,
     describeDroppedOutputs,
+    applyQueuedUnknownRetryGuidance,
+    PANEL_QUEUED_UNKNOWN_RETRY_GUIDANCE,
 };
 /** Drop a trailing .json (case-insensitive) so filename/path forms compare equal. */
 function stripJsonExt(s) {
@@ -13403,20 +13438,20 @@ function panelLateReceiptNote(promptId) {
         `panel_run or you may queue a duplicate render.`);
 }
 /**
- * #2143 — a normal Panel timeout receipt is neither a confirmed queue nor a
- * refusal. Keep its retry guidance in the original JSON and add only the
- * local fact that this handler did not dispatch a second run.
+ * #2143 / #2438 — a normal Panel timeout receipt is neither a confirmed queue
+ * nor a refusal. Name the next step that exists on THIS surface rather than
+ * forwarding Panel retry_guidance that points at a headless queue inspector.
  */
 function panelQueueUncertaintyNote(queuedPromptIds) {
     if (queuedPromptIds.length > 0) {
         return (`\n\n[UNCERTAIN] The Panel confirmed these prompt id(s) left its queue path: ` +
             `${queuedPromptIds.join(", ")}. Completion tickets were opened for them. ` +
-            `Any remaining request(s) are still uncertain; follow the Panel's retry_guidance above ` +
-            `before retrying. No second panel_run was dispatched.`);
+            `Any remaining request(s) are still uncertain. ${PANEL_QUEUED_UNKNOWN_RETRY_GUIDANCE} ` +
+            `No second panel_run was dispatched.`);
     }
     return (`\n\n[UNCERTAIN] The Panel could not determine whether this run entered ComfyUI's queue. ` +
         `This is not a confirmed refusal, and no completion ticket can be opened without a prompt id. ` +
-        `Follow the Panel's retry_guidance above before retrying. No second panel_run was dispatched.`);
+        `${PANEL_QUEUED_UNKNOWN_RETRY_GUIDANCE} No second panel_run was dispatched.`);
 }
 async function pollLateAskReply(bridge, askId, timing, hardDeadline) {
     const probe = bridge;
@@ -16456,6 +16491,11 @@ export function buildPanelToolDefs() {
                             `If the running job is genuinely stuck, queue (action:"cancel") with clear_pending:true interrupts it AND drops pending, then escalate to restart_comfyui if it reports the job wedged.`;
                 }
             }
+            // #2438 — rewrite Panel retry_guidance BEFORE the notes are spliced so
+            // a headless `queue (action:"list")` inspector is not left in the JSON
+            // for a surface that cannot call it.
+            if (panelQueueUncertain)
+                res = applyQueuedUnknownRetryGuidance(res);
             if (res.content?.[0]?.type === "text") {
                 return {
                     ...res,
@@ -18558,7 +18598,20 @@ CHECKED FOR YOU: the graph read this message prescribes was just run, and it ` +
             });
             return out.via === "panel" ? out.value : ok(out.value);
         }),
-        def("panel_list_nodes", "List the custom-node PACKS currently installed in the user's ComfyUI (via the built-in Manager). Takes no arguments — it returns the whole installed set. Read-only. This lists packs, NOT node classes: to find out whether a specific node type is available (e.g. LTXVContextWindowsGuideAware), use comfy_cli action:\"search_nodes\", which fuzzy-searches real node classes and falls back to the connected server's live /object_info. panel_search_nodes is a third thing again — it searches INSTALLABLE packs in the registry, not what is already here.", {}, async (_args, ctx) => ctx.call({ cmd: "nodes_list" }, 20000)),
+        def("panel_list_nodes", "List the custom-node PACKS currently installed in the user's ComfyUI (via the built-in Manager). Takes no arguments — it returns the whole installed set. Read-only. This lists packs, NOT node classes: to find out whether a specific node type is available (e.g. LTXVContextWindowsGuideAware), use comfy_cli action:\"search_nodes\", which fuzzy-searches real node classes and falls back to the connected server's live /object_info. panel_search_nodes is a third thing again — it searches INSTALLABLE packs in the registry, not what is already here. If the panel tab is gone, the listing is served from the ComfyUI host's Manager HTTP instead of failing as a dispatch error.", {}, async (_args, ctx) => {
+            // #2459 — pack listing is a Manager / object_info server-state read.
+            // Try the live tab first (Manager dialect + /object_info + #1496
+            // filter). When dispatch never reaches the tab, serve the same
+            // inventory from host Manager HTTP instead of dying as tab-loss.
+            const out = await listPanelNodes({
+                panelList: () => ctx.call({ cmd: "nodes_list" }, 20000),
+            });
+            if (out.via === "panel")
+                return out.value;
+            if (out.via === "fallback")
+                return ok(out.value);
+            return fail(out.message);
+        }),
         def("panel_install_node", "Install a custom-node pack into the user's ComfyUI through the BUILT-IN Manager. Registry ids are supported: pass an id from panel_search_nodes (or another known registry id), then poll panel_node_queue_status and verify with panel_list_nodes before restarting. The panel runtime refuses Git URLs before Manager v4 queueing because v4 ignores the supplied repository and resolves by bare name, so a URL is not a successful v4 from-source install. Use an id from panel_search_nodes; use install_custom_node(source:'git') only when its local target is the same ComfyUI as this panel. If the panel drives another machine, install the repository on the ComfyUI host instead. The repository input remains normalized for legacy Manager 3.x direct-URL routing, but that compatibility path is not a v4 guarantee. The channel input can select a Manager channel for a supported registry-id install; it does not bypass the v4 Git-URL refusal. Install packs one at a time and confirm each is present before restarting.", {
             id: z.string().optional().describe("Registry id returned by panel_search_nodes."),
             repository: z