comfyui-mcp 0.50.73 → 0.50.75

package/dist/orchestrator/fence-refusal.js

@@ -58,7 +58,20 @@ export function identityReason(tabId, origin, workflowUuid, failure) {
             `observed nor was told the panel's origin; if COMFYUI_URL is unset or unparseable for ` +
             `this session, setting it to the URL the panel is served from is what supplies it.`);
     }
-    return (`the workflow identity did not validate (uuid ${String(workflowUuid)} against origin ` +
-        `${origin}) — a malformed uuid, or one bound to a different origin.`);
+    // #1255 — this used to end "a malformed uuid, or one bound to a different
+    // origin", which describes a comparison the code does not perform.
+    // workflowIdentityParts uses the origin as a PRESENCE gate: it requires one to
+    // exist, and never compares it against a previously bound value (nothing in
+    // the repo has ever read `identity.origin` -- both call sites take `.uuid`).
+    // Reaching here with an origin present therefore means the UUID failed its
+    // shape check, full stop.
+    //
+    // The old wording sent a reader hunting for an origin mismatch that cannot
+    // happen -- it cost a whole investigation and a closed-unmerged PR before the
+    // mechanism was checked rather than believed. A refusal must describe the
+    // check it actually ran.
+    return (`the workflow identity did not validate: the uuid ${String(workflowUuid)} is not a ` +
+        `well-formed workflow uuid. (The origin ${origin} was present and accepted -- this gate ` +
+        `requires an origin to exist, it does not compare it against a previously bound one.)`);
 }
 //# sourceMappingURL=fence-refusal.js.map

package/dist/orchestrator/fence-refusal.js.map

@@ -1 +1 @@
-{"version":3,"file":"fence-refusal.js","sourceRoot":"","sources":["../../src/orchestrator/fence-refusal.ts"],"names":[],"mappings":"AAAA,4EAA4E;AAC5E,EAAE;AACF,iFAAiF;AACjF,gFAAgF;AAChF,yEAAyE;AACzE,2DAA2D;AAC3D,EAAE;AACF,8EAA8E;AAC9E,2EAA2E;AAC3E,4EAA4E;AAC5E,0DAA0D;AAC1D,EAAE;AACF,6EAA6E;AAC7E,8EAA8E;AAC9E,6EAA6E;AAC7E,uEAAuE;AACvE,6EAA6E;AAC7E,+EAA+E;AAC/E,sEAAsE;AACtE,4EAA4E;AAM5E,4EAA4E;AAC5E,SAAS,aAAa,CAAC,KAAa,EAAE,IAAY;IAChD,OAAO,CACL,2DAA2D,IAAI,SAAS,KAAK,eAAe;QAC5F,2EAA2E;QAC3E,mFAAmF;QACnF,+DAA+D,CAChE,CAAC;AACJ,CAAC;AAED,qDAAqD;AACrD,MAAM,UAAU,iBAAiB,CAAC,KAAa,EAAE,OAAuB;IACtE,IAAI,OAAO,KAAK,WAAW;QAAE,OAAO,aAAa,CAAC,KAAK,EAAE,oCAAoC,CAAC,CAAC;IAC/F,OAAO,mBAAmB,KAAK,wDAAwD,CAAC;AAC1F,CAAC;AAED,wDAAwD;AACxD,MAAM,UAAU,gBAAgB,CAAC,KAAa;IAC5C,OAAO,GAAG,KAAK,iFAAiF,CAAC;AACnG,CAAC;AAED;;;;;;GAMG;AACH,MAAM,UAAU,cAAc,CAC5B,KAAa,EACb,MAA0B,EAC1B,YAAqB,EACrB,OAAuB;IAEvB,IAAI,CAAC,MAAM,IAAI,OAAO,KAAK,WAAW,EAAE,CAAC;QACvC,OAAO,aAAa,CAAC,KAAK,EAAE,wCAAwC,CAAC,CAAC;IACxE,CAAC;IACD,IAAI,CAAC,MAAM,EAAE,CAAC;QACZ,6EAA6E;QAC7E,8EAA8E;QAC9E,6EAA6E;QAC7E,wEAAwE;QACxE,mEAAmE;QACnE,OAAO,CACL,wFAAwF;YACxF,oFAAoF;YACpF,uFAAuF;YACvF,uFAAuF;YACvF,mFAAmF,CACpF,CAAC;IACJ,CAAC;IACD,OAAO,CACL,gDAAgD,MAAM,CAAC,YAAY,CAAC,kBAAkB;QACtF,GAAG,MAAM,2DAA2D,CACrE,CAAC;AACJ,CAAC"}
+{"version":3,"file":"fence-refusal.js","sourceRoot":"","sources":["../../src/orchestrator/fence-refusal.ts"],"names":[],"mappings":"AAAA,4EAA4E;AAC5E,EAAE;AACF,iFAAiF;AACjF,gFAAgF;AAChF,yEAAyE;AACzE,2DAA2D;AAC3D,EAAE;AACF,8EAA8E;AAC9E,2EAA2E;AAC3E,4EAA4E;AAC5E,0DAA0D;AAC1D,EAAE;AACF,6EAA6E;AAC7E,8EAA8E;AAC9E,6EAA6E;AAC7E,uEAAuE;AACvE,6EAA6E;AAC7E,+EAA+E;AAC/E,sEAAsE;AACtE,4EAA4E;AAM5E,4EAA4E;AAC5E,SAAS,aAAa,CAAC,KAAa,EAAE,IAAY;IAChD,OAAO,CACL,2DAA2D,IAAI,SAAS,KAAK,eAAe;QAC5F,2EAA2E;QAC3E,mFAAmF;QACnF,+DAA+D,CAChE,CAAC;AACJ,CAAC;AAED,qDAAqD;AACrD,MAAM,UAAU,iBAAiB,CAAC,KAAa,EAAE,OAAuB;IACtE,IAAI,OAAO,KAAK,WAAW;QAAE,OAAO,aAAa,CAAC,KAAK,EAAE,oCAAoC,CAAC,CAAC;IAC/F,OAAO,mBAAmB,KAAK,wDAAwD,CAAC;AAC1F,CAAC;AAED,wDAAwD;AACxD,MAAM,UAAU,gBAAgB,CAAC,KAAa;IAC5C,OAAO,GAAG,KAAK,iFAAiF,CAAC;AACnG,CAAC;AAED;;;;;;GAMG;AACH,MAAM,UAAU,cAAc,CAC5B,KAAa,EACb,MAA0B,EAC1B,YAAqB,EACrB,OAAuB;IAEvB,IAAI,CAAC,MAAM,IAAI,OAAO,KAAK,WAAW,EAAE,CAAC;QACvC,OAAO,aAAa,CAAC,KAAK,EAAE,wCAAwC,CAAC,CAAC;IACxE,CAAC;IACD,IAAI,CAAC,MAAM,EAAE,CAAC;QACZ,6EAA6E;QAC7E,8EAA8E;QAC9E,6EAA6E;QAC7E,wEAAwE;QACxE,mEAAmE;QACnE,OAAO,CACL,wFAAwF;YACxF,oFAAoF;YACpF,uFAAuF;YACvF,uFAAuF;YACvF,mFAAmF,CACpF,CAAC;IACJ,CAAC;IACD,0EAA0E;IAC1E,mEAAmE;IACnE,+EAA+E;IAC/E,4EAA4E;IAC5E,6EAA6E;IAC7E,2EAA2E;IAC3E,0BAA0B;IAC1B,EAAE;IACF,2EAA2E;IAC3E,8EAA8E;IAC9E,0EAA0E;IAC1E,yBAAyB;IACzB,OAAO,CACL,oDAAoD,MAAM,CAAC,YAAY,CAAC,YAAY;QACpF,0CAA0C,MAAM,yCAAyC;QACzF,sFAAsF,CACvF,CAAC;AACJ,CAAC"}

package/dist/services/panel-installer.js

@@ -1479,15 +1479,28 @@ export async function panelStatus(depsIn = defaultDeps) {
                             `serves from, so "not installed" is UNCORROBORATED. On a split install ` +
                             `(Comfy Desktop's --base-directory, #766) the panel lives elsewhere and ` +
                             `installing here would land in a custom_nodes nothing loads. ` +
-                            (baseResolution?.liveRootUnderivable
-                                ? `ComfyUI IS running, but its reported launch arguments did not identify ` +
-                                    `an install root holding custom_nodes (a relative main.py with no ` +
-                                    `reported working directory), so reaching it again will not change this — ` +
-                                    `check install_comfyui (action:"environment")'s local.workspace_path, or ` +
-                                    `relaunch ComfyUI with an ABSOLUTE path to main.py, before installing.`
-                                : `Start/reach ` +
-                                    `ComfyUI so its own install root can be read, or check install_comfyui ` +
-                                    `(action:"environment")'s local.workspace_path, before installing.`)
+                            // #796 — THREE causes, three remedies. The unreadable case used to take
+                            // one of the other two branches, and both give advice that cannot work:
+                            // the argv was fine (so relaunching with an absolute main.py changes
+                            // nothing) and ComfyUI is reachable (so "start/reach ComfyUI" is already
+                            // true). Naming a remedy that cannot apply is worse than naming none.
+                            (baseResolution?.liveRootUnreadable
+                                ? `ComfyUI IS running and DID identify an install root — ` +
+                                    `${baseResolution.liveRootUnreadable} — but its custom_nodes could not be ` +
+                                    `READ from here (a permission error, an IO error, or a network path that ` +
+                                    `is not available to this process), so it could not be confirmed as the ` +
+                                    `serving tree. That is not evidence it lacks one. Make that directory ` +
+                                    `readable by whoever runs this MCP — or run install_comfyui ` +
+                                    `(action:"panel") from a process that can read it — before installing.`
+                                : baseResolution?.liveRootUnderivable
+                                    ? `ComfyUI IS running, but its reported launch arguments did not identify ` +
+                                        `an install root holding custom_nodes (a relative main.py with no ` +
+                                        `reported working directory), so reaching it again will not change this — ` +
+                                        `check install_comfyui (action:"environment")'s local.workspace_path, or ` +
+                                        `relaunch ComfyUI with an ABSOLUTE path to main.py, before installing.`
+                                    : `Start/reach ` +
+                                        `ComfyUI so its own install root can be read, or check install_comfyui ` +
+                                        `(action:"environment")'s local.workspace_path, before installing.`)
                     : `Not installed. Run install_comfyui(action:'panel', panel_action:'install') to add the panel (${PANEL_VERSION}). Restart ComfyUI afterwards.`;
     }
     else {