comfyui-mcp 0.52.107 → 0.52.108

package/dist/orchestrator/panel-tools.js

@@ -38,6 +38,7 @@ import { sanitizePanelUpdateNodeResult } from "../services/manager-update-error.
 import { formatQueueStatusPartialNote, getManifestPartialLeftover, } from "../services/manifest-partial.js";
 import { searchPanelNodes } from "../services/manager-node-search.js";
 import { isPanelAnsweredError } from "../services/panel-answered.js";
+import { readWorkflowListReadiness, workflowListReadinessOf, } from "../services/panel-workflow-readiness.js";
 import { isPreExecutorRefusal } from "../services/panel-refusal.js";
 import { createSdkMcpServer, tool } from "@anthropic-ai/claude-agent-sdk";
 import { parse as parseYaml } from "yaml";
@@ -2645,6 +2646,32 @@ function carryPanelAnsweredMark(err, res) {
     });
     return res;
 }
+/** #1785 — preserve the Panel's typed workflow-list readiness refusal across the
+ * bridge-to-ToolResult conversion. The refusal is an explicit answer, not a
+ * silent read failure, and its distinction controls whether a concurrent
+ * re-hello may be treated as successful healing. */
+const WORKFLOW_LIST_READINESS_RESULT = Symbol("panel.workflowListReadinessResult");
+function carryWorkflowListReadinessMark(err, res) {
+    if (res?.isError !== true)
+        return res;
+    const refusal = workflowListReadinessOf(err);
+    if (!refusal)
+        return res;
+    Object.defineProperty(res, WORKFLOW_LIST_READINESS_RESULT, {
+        value: refusal,
+        enumerable: false,
+        configurable: true,
+    });
+    return res;
+}
+function workflowListReadinessResultOf(res) {
+    if (res?.isError !== true)
+        return null;
+    const refusal = res[WORKFLOW_LIST_READINESS_RESULT];
+    return refusal && typeof refusal === "object"
+        ? readWorkflowListReadiness(refusal)
+        : null;
+}
 /** True only for a ToolResult `ctx.call` produced from an error the BRIDGE minted
  *  out of a received reply. A timeout, a routing refusal or a dead socket can never
  *  carry this, whatever their text says — and neither can an error from a panel too
@@ -7391,6 +7418,17 @@ async function rebindWorkflowFence(ctx, opts) {
             const res = await ctx.call({ cmd: "workflow_list" }, 6000);
             syncFenceToCurrentTab();
             if (res?.isError) {
+                const readiness = workflowListReadinessResultOf(res);
+                if (readiness) {
+                    return {
+                        done: {
+                            status: "readiness_refused",
+                            before,
+                            detail: toolResultText(res),
+                            readiness,
+                        },
+                    };
+                }
                 return { done: await unreadableOrHealed(ctx, before, toolResultText(res)) };
             }
             parsed = parseToolResultJson(res);
@@ -7885,6 +7923,17 @@ panelGapNote = "") {
                     `keeps throwing: ${RELOAD_TAB_REMEDY}` +
                     `\n\nUNDERLYING CAUSE: ${r.detail}`,
             };
+        case "readiness_refused":
+            return {
+                binding: "not_recovered",
+                note: ` The panel ANSWERED the workflow_list probe with its typed reconnect-readiness ` +
+                    `refusal (${r.readiness.code}): the probe was not run, changed no workflow target, ` +
+                    `and the live workflow identity was not ready within the panel's bounded window. ` +
+                    `A concurrent re-hello or fence change is not a successful workflow_list proof, so ` +
+                    `this call did NOT restore graph binding. Retry after the panel reports readiness, or ` +
+                    `use panel_open_workflow(<path>) to explicitly re-establish the tab.` +
+                    `\n\nUNDERLYING CAUSE (quoted verbatim): ${r.detail}`,
+            };
         case "unreadable":
             // ALWAYS not_recovered, with or without a prior fence (codex gate). The read
             // we just attempted FAILED, so nothing about this session was observed —
@@ -10838,7 +10887,7 @@ export function makePanelToolCtx(bridge, tabId, workflowTargets, onRunTicketOpen
         const res = await callOnce(cmd, timeoutMs, onDispatchedRid, (err) => {
             failure = err;
         });
-        return carryMidCommandDisconnectMark(failure, carryPanelAnsweredMark(failure, res));
+        return carryWorkflowListReadinessMark(failure, carryMidCommandDisconnectMark(failure, carryPanelAnsweredMark(failure, res)));
     };
     // Human-in-the-loop confirmation for a DESTRUCTIVE op: render a yes/no card in
     // the panel and block on the user's pick. Returns false on decline, timeout, or