comfyui-mcp 0.50.76 → 0.50.77

package/dist/utils/optional-dep.js

@@ -6,6 +6,28 @@
 // clear "install <pkg>" error instead of crashing on import.
 import { ModelError } from "./errors.js";
 const cache = new Map();
+export function classifyImportFailure(spec, err) {
+    const e = err;
+    const message = typeof e?.message === "string" ? e.message : String(err);
+    const notFound = e?.code === "ERR_MODULE_NOT_FOUND" || e?.code === "MODULE_NOT_FOUND";
+    if (!notFound)
+        return { kind: "unloadable", detail: message };
+    // "Cannot find package 'x' imported from …" / "Cannot find module 'x'".
+    const named = /Cannot find (?:package|module) ['"]([^'"]+)['"]/.exec(message)?.[1];
+    if (named === undefined) {
+        // A not-found we cannot attribute. Treat it as the requested package rather
+        // than inventing a transitive one — this is the pre-existing behaviour and
+        // the common case.
+        return { kind: "missing" };
+    }
+    // SUBPATHS RUN BOTH WAYS, and only one direction is obvious. Requesting
+    // "pkg/sub" makes Node report `Cannot find package 'pkg'` — it names the bare
+    // PACKAGE, not what was asked for — so a `named === spec` test fails and the
+    // package gets blamed as its own dependency. Found by a surviving mutant and
+    // confirmed against real Node before fixing.
+    const isSelf = named === spec || named.startsWith(`${spec}/`) || spec.startsWith(`${named}/`);
+    return isSelf ? { kind: "missing" } : { kind: "missing-transitive", detail: named };
+}
 export async function requireOptionalDep(spec, options) {
     const cached = cache.get(spec);
     if (cached !== undefined)
@@ -18,9 +40,21 @@ export async function requireOptionalDep(spec, options) {
     catch (err) {
         const feature = options?.feature ?? spec;
         const installHint = options?.installHint ?? `npm install ${spec}`;
-        throw new ModelError(`Optional dependency for ${feature} is not installed. ` +
-            `To enable it, run: ${installHint}`, {
-            code: "OPTIONAL_DEP_MISSING",
+        const why = classifyImportFailure(spec, err);
+        throw new ModelError(why.kind === "missing"
+            ? `Optional dependency for ${feature} is not installed. To enable it, run: ${installHint}`
+            : why.kind === "missing-transitive"
+                ? `Optional dependency for ${feature} IS installed, but it could not load because ` +
+                    `${why.detail} is missing — one of its own dependencies, not this package. ` +
+                    `Running "${installHint}" will report nothing to do. Reinstall it so its ` +
+                    `dependency tree is resolved (npm install --force ${spec}), or install ${why.detail} directly.`
+                : `Optional dependency for ${feature} IS installed, but it failed to LOAD: ${why.detail}. ` +
+                    `Running "${installHint}" will report nothing to do — the package is present and ` +
+                    `broken, which is usually a native binary built for a different Node/ABI or platform. ` +
+                    `Reinstall it (npm install --force ${spec}), or check that binary's own requirements.`, {
+            // The code still distinguishes the two for programmatic callers: only a
+            // genuinely absent package is MISSING.
+            code: why.kind === "missing" ? "OPTIONAL_DEP_MISSING" : "OPTIONAL_DEP_UNLOADABLE",
             package: spec,
             cause: err instanceof Error ? err.message : String(err),
         });

package/dist/utils/optional-dep.js.map

@@ -1 +1 @@
-{"version":3,"file":"optional-dep.js","sourceRoot":"","sources":["../../src/utils/optional-dep.ts"],"names":[],"mappings":"AAAA,iDAAiD;AACjD,EAAE;AACF,6EAA6E;AAC7E,8EAA8E;AAC9E,6EAA6E;AAC7E,6DAA6D;AAE7D,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAC;AAEzC,MAAM,KAAK,GAAG,IAAI,GAAG,EAAmB,CAAC;AAEzC,MAAM,CAAC,KAAK,UAAU,kBAAkB,CACtC,IAAY,EACZ,OAGC;IAED,MAAM,MAAM,GAAG,KAAK,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;IAC/B,IAAI,MAAM,KAAK,SAAS;QAAE,OAAO,MAAW,CAAC;IAC7C,IAAI,CAAC;QACH,MAAM,GAAG,GAAG,CAAC,MAAM,MAAM,CAAC,IAAI,CAAC,CAAM,CAAC;QACtC,KAAK,CAAC,GAAG,CAAC,IAAI,EAAE,GAAG,CAAC,CAAC;QACrB,OAAO,GAAG,CAAC;IACb,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,MAAM,OAAO,GAAG,OAAO,EAAE,OAAO,IAAI,IAAI,CAAC;QACzC,MAAM,WAAW,GAAG,OAAO,EAAE,WAAW,IAAI,eAAe,IAAI,EAAE,CAAC;QAClE,MAAM,IAAI,UAAU,CAClB,2BAA2B,OAAO,qBAAqB;YACrD,sBAAsB,WAAW,EAAE,EACrC;YACE,IAAI,EAAE,sBAAsB;YAC5B,OAAO,EAAE,IAAI;YACb,KAAK,EAAE,GAAG,YAAY,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC;SACxD,CACF,CAAC;IACJ,CAAC;AACH,CAAC"}
+{"version":3,"file":"optional-dep.js","sourceRoot":"","sources":["../../src/utils/optional-dep.ts"],"names":[],"mappings":"AAAA,iDAAiD;AACjD,EAAE;AACF,6EAA6E;AAC7E,8EAA8E;AAC9E,6EAA6E;AAC7E,6DAA6D;AAE7D,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAC;AAEzC,MAAM,KAAK,GAAG,IAAI,GAAG,EAAmB,CAAC;AAiCzC,MAAM,UAAU,qBAAqB,CAAC,IAAY,EAAE,GAAY;IAC9D,MAAM,CAAC,GAAG,GAAmD,CAAC;IAC9D,MAAM,OAAO,GAAG,OAAO,CAAC,EAAE,OAAO,KAAK,QAAQ,CAAC,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC;IACzE,MAAM,QAAQ,GAAG,CAAC,EAAE,IAAI,KAAK,sBAAsB,IAAI,CAAC,EAAE,IAAI,KAAK,kBAAkB,CAAC;IACtF,IAAI,CAAC,QAAQ;QAAE,OAAO,EAAE,IAAI,EAAE,YAAY,EAAE,MAAM,EAAE,OAAO,EAAE,CAAC;IAE9D,wEAAwE;IACxE,MAAM,KAAK,GAAG,iDAAiD,CAAC,IAAI,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC;IACnF,IAAI,KAAK,KAAK,SAAS,EAAE,CAAC;QACxB,4EAA4E;QAC5E,2EAA2E;QAC3E,mBAAmB;QACnB,OAAO,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC;IAC7B,CAAC;IACD,wEAAwE;IACxE,8EAA8E;IAC9E,6EAA6E;IAC7E,6EAA6E;IAC7E,6CAA6C;IAC7C,MAAM,MAAM,GACV,KAAK,KAAK,IAAI,IAAI,KAAK,CAAC,UAAU,CAAC,GAAG,IAAI,GAAG,CAAC,IAAI,IAAI,CAAC,UAAU,CAAC,GAAG,KAAK,GAAG,CAAC,CAAC;IACjF,OAAO,MAAM,CAAC,CAAC,CAAC,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC,CAAC,CAAC,EAAE,IAAI,EAAE,oBAAoB,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC;AACtF,CAAC;AAED,MAAM,CAAC,KAAK,UAAU,kBAAkB,CACtC,IAAY,EACZ,OAGC;IAED,MAAM,MAAM,GAAG,KAAK,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;IAC/B,IAAI,MAAM,KAAK,SAAS;QAAE,OAAO,MAAW,CAAC;IAC7C,IAAI,CAAC;QACH,MAAM,GAAG,GAAG,CAAC,MAAM,MAAM,CAAC,IAAI,CAAC,CAAM,CAAC;QACtC,KAAK,CAAC,GAAG,CAAC,IAAI,EAAE,GAAG,CAAC,CAAC;QACrB,OAAO,GAAG,CAAC;IACb,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,MAAM,OAAO,GAAG,OAAO,EAAE,OAAO,IAAI,IAAI,CAAC;QACzC,MAAM,WAAW,GAAG,OAAO,EAAE,WAAW,IAAI,eAAe,IAAI,EAAE,CAAC;QAClE,MAAM,GAAG,GAAG,qBAAqB,CAAC,IAAI,EAAE,GAAG,CAAC,CAAC;QAC7C,MAAM,IAAI,UAAU,CAClB,GAAG,CAAC,IAAI,KAAK,SAAS;YACpB,CAAC,CAAC,2BAA2B,OAAO,yCAAyC,WAAW,EAAE;YAC1F,CAAC,CAAC,GAAG,CAAC,IAAI,KAAK,oBAAoB;gBACjC,CAAC,CAAC,2BAA2B,OAAO,+CAA+C;oBACjF,GAAG,GAAG,CAAC,MAAM,+DAA+D;oBAC5E,YAAY,WAAW,mDAAmD;oBAC1E,oDAAoD,IAAI,iBAAiB,GAAG,CAAC,MAAM,YAAY;gBACjG,CAAC,CAAC,2BAA2B,OAAO,yCAAyC,GAAG,CAAC,MAAM,IAAI;oBACzF,YAAY,WAAW,2DAA2D;oBAClF,uFAAuF;oBACvF,qCAAqC,IAAI,6CAA6C,EAC5F;YACE,wEAAwE;YACxE,uCAAuC;YACvC,IAAI,EAAE,GAAG,CAAC,IAAI,KAAK,SAAS,CAAC,CAAC,CAAC,sBAAsB,CAAC,CAAC,CAAC,yBAAyB;YACjF,OAAO,EAAE,IAAI;YACb,KAAK,EAAE,GAAG,YAAY,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC;SACxD,CACF,CAAC;IACJ,CAAC;AACH,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "comfyui-mcp",
-  "version": "0.50.76",
+  "version": "0.50.77",
   "mcpName": "io.github.artokun/comfyui-mcp",
   "description": "Local-first, agent-native control plane for ComfyUI — MCP server + autonomous sidebar agent that drives your live graph in natural language on ANY LLM: Claude/ChatGPT/Gemini on your subscription (no API key), free local models via Ollama (fully offline), or any hosted model via an OpenAI-compatible endpoint (DeepSeek, GLM, MiMo, OpenRouter). Generate images, video & audio, author and run workflows, manage models and custom nodes; a compact tool-router mode keeps even 4B models effective, and a built-in LLM Arena benchmarks them on real ComfyUI tasks. Also a Claude Code plugin with skills, slash commands, and installer packs. Local, LAN, VPS, or Comfy Cloud.",
   "homepage": "https://comfyui-mcp.artokun.io/docs",