dsh-comfyui 0.5.1 → 0.5.3

package/lib/analyze.js

@@ -44,14 +44,36 @@ export function analyzeGraph(graph) {
     const isolated = active
         .filter((node) => !linkedIds.has(node.id))
         .map((node) => ({ id: node.id, type: node.type }));
+    // Bypassed nodes are pass-throughs at conversion time (resolveOrigin in
+    // convert.ts: the output follows the FIRST wired input). Mirror that rule
+    // here, or a chain `120 → [406 bypassed] → 119` splits into two components
+    // and "main flow" extraction ships 119 referencing a node it left out
+    // (Issue #8).
+    const linkById = new Map(links.map((link) => [link[0], link]));
+    const nodeById = new Map(nodes.map((node) => [node.id, node]));
+    const passThroughOrigin = (originId) => {
+        let current = originId;
+        const visited = new Set();
+        while (nodeById.get(current)?.mode === 4) {
+            if (visited.has(current))
+                return undefined;
+            visited.add(current);
+            const inputLink = (nodeById.get(current)?.inputs ?? []).find((entry) => entry.link !== null)?.link;
+            const upstream = inputLink === undefined || inputLink === null ? undefined : linkById.get(inputLink);
+            if (upstream === undefined)
+                return undefined;
+            current = upstream[1];
+        }
+        return current;
+    };
     // Connected components over linked active nodes.
     const adjacency = new Map();
     for (const node of active)
         adjacency.set(node.id, new Set());
     for (const link of links) {
-        const a = link[1];
+        const a = passThroughOrigin(link[1]);
         const b = link[3];
-        if (typeof a !== 'number' || typeof b !== 'number')
+        if (a === undefined)
             continue;
         if (!adjacency.has(a) || !adjacency.has(b))
             continue;

package/lib/comfyui.d.ts

@@ -109,7 +109,10 @@ export declare class ComfyUIClient {
     private readonly connectTimeoutMs;
     private readonly maxMediaBytes;
     constructor(baseUrl: string, apiKey: string | undefined, connectTimeoutMs: number, maxMediaBytes: number);
+    private base;
     private endpoint;
+    /** fetch() one route, retrying a 404 under /api until the prefix is known. */
+    private fetchRoute;
     private request;
     /** Best-effort GET of an arbitrary JSON endpoint on the ComfyUI server.
      *

package/lib/comfyui.js

@@ -62,6 +62,14 @@ export function guessContentType(filename) {
 }
 /** The per-process client id ComfyUI uses to correlate queued prompts. */
 export const CLIENT_ID = randomUUID();
+/**
+ * Route prefix learned per base URL: '' when bare routes answer, '/api' when
+ * only the /api-prefixed mirror does. ComfyUI serves every route under both,
+ * but API-only reverse proxies (comfy-api-proxy, Issue #6) forward just
+ * `/api/*`, so the bare /system_stats probe 404s. Learned on the first
+ * request and kept for the process — clients are created per call.
+ */
+const routePrefix = new Map();
 /** HTTP client over the ComfyUI REST API. */
 export class ComfyUIClient {
     baseUrl;
@@ -74,8 +82,34 @@ export class ComfyUIClient {
         this.connectTimeoutMs = connectTimeoutMs;
         this.maxMediaBytes = maxMediaBytes;
     }
+    base() {
+        return this.baseUrl.replace(/\/+$/, '');
+    }
     endpoint(path) {
-        return `${this.baseUrl.replace(/\/+$/, '')}${path}`;
+        if (path.startsWith('/api/'))
+            return `${this.base()}${path}`;
+        return `${this.base()}${routePrefix.get(this.base()) ?? ''}${path}`;
+    }
+    /** fetch() one route, retrying a 404 under /api until the prefix is known. */
+    async fetchRoute(path, init) {
+        const base = this.base();
+        const response = await fetch(this.endpoint(path), init);
+        if (path.startsWith('/api/') || routePrefix.has(base))
+            return response;
+        if (response.ok) {
+            routePrefix.set(base, '');
+            return response;
+        }
+        if (response.status !== 404)
+            return response;
+        const retry = await fetch(`${base}/api${path}`, init);
+        if (!retry.ok) {
+            await retry.body?.cancel();
+            return response;
+        }
+        routePrefix.set(base, '/api');
+        await response.body?.cancel();
+        return retry;
     }
     async request(path, init = {}, timeoutMs) {
         const controller = new AbortController();
@@ -84,7 +118,7 @@ export class ComfyUIClient {
             const headers = { ...init.headers };
             if (this.apiKey !== undefined)
                 headers['Authorization'] = `Bearer ${this.apiKey}`;
-            const response = await fetch(this.endpoint(path), { ...init, headers, signal: controller.signal });
+            const response = await this.fetchRoute(path, { ...init, headers, signal: controller.signal });
             if (!response.ok) {
                 const body = await response.text().catch(() => '');
                 throw new ComfyUIError(`ComfyUI ${path} failed: HTTP ${response.status}${body !== '' ? ` — ${body.slice(0, 300)}` : ''}`, response.status);
@@ -284,7 +318,28 @@ export class ComfyUIClient {
     }
     /** List one user-data subdirectory (e.g. 'workflows') on the ComfyUI server. */
     async listUserData(subdir) {
-        const data = await this.request(`/v2/userdata?path=${encodeURIComponent(subdir)}`);
+        let data;
+        try {
+            data = await this.request(`/v2/userdata?path=${encodeURIComponent(subdir)}`);
+        }
+        catch (error) {
+            // Some builds (e.g. the 0.35 desktop app, Issue #5) have no /v2/userdata;
+            // the v1 listing takes `dir` and returns paths relative to it.
+            if (!(error instanceof ComfyUIError) || error.status !== 404)
+                throw error;
+            const legacy = await this.request(`/userdata?dir=${encodeURIComponent(subdir)}&recurse=true&split=false&full_info=true`);
+            if (!Array.isArray(legacy))
+                return [];
+            return legacy.map((item) => {
+                const rel = (typeof item === 'string' ? item : item.path).replace(/\\/g, '/');
+                const entry = { name: rel, path: `${subdir}/${rel}`, type: 'file' };
+                if (typeof item !== 'string' && item.size !== undefined)
+                    entry.size = item.size;
+                if (typeof item !== 'string' && item.modified !== undefined)
+                    entry.modified = item.modified;
+                return entry;
+            });
+        }
         if (!Array.isArray(data))
             return [];
         if (typeof data[0] === 'string') {
@@ -319,7 +374,7 @@ export class ComfyUIClient {
             const headers = {};
             if (this.apiKey !== undefined)
                 headers['Authorization'] = `Bearer ${this.apiKey}`;
-            const response = await fetch(this.endpoint(`/view?${params.toString()}`), { headers, signal: controller.signal });
+            const response = await this.fetchRoute(`/view?${params.toString()}`, { headers, signal: controller.signal });
             if (!response.ok) {
                 throw new ComfyUIError(`ComfyUI /view failed: HTTP ${response.status}`);
             }
@@ -352,7 +407,7 @@ export class ComfyUIClient {
                 headers['Authorization'] = `Bearer ${this.apiKey}`;
             if (rangeHeader !== undefined)
                 headers['Range'] = rangeHeader;
-            const response = await fetch(this.endpoint(`/view?${params.toString()}`), { headers, method, signal: controller.signal });
+            const response = await this.fetchRoute(`/view?${params.toString()}`, { headers, method, signal: controller.signal });
             if (response.status !== 206 && response.status !== 416 && !response.ok) {
                 throw new ComfyUIError(`ComfyUI /view failed: HTTP ${response.status}`);
             }

package/lib/config.d.ts

@@ -1,86 +1,93 @@
 /**
  * dsh-comfyui host configuration. The same schema drives the Loader entry
- * config (cordis.yml patch) and the `comfyui:` settings section the browser
- * settings page writes, so one shape covers both doors into the same values.
+ * config (cordis.yml patch) and the settings page, so one shape covers both
+ * doors into the same values.
+ *
+ * Fields the plugin reads at use time are `.volatile()`: since dsh 0.1.7 the
+ * settings service only edits volatile fields (anything else fails with
+ * "Plugin entry has no volatile fields", Issue #11), and the Loader commits a
+ * volatile-only change into the running references without restarting the
+ * plugin. `dataDir` / `maxAssets` / `outputDir` stay ordinary: the store is
+ * built from them at startup, so a change there restarts the plugin.
  */
 import z from '@deepseek-ai/schemastery';
-export declare const Config: z<Schemastery.ObjectS<{
+export declare const Config: z<Schemastery.ObjectS<NoInfer<{
     /** ComfyUI HTTP server base URL. */
-    baseUrl: z<string, string>;
+    baseUrl: z<string, string, "volatile-defined">;
     /** Environment-variable name of the optional API key (credentials ref). */
-    apiKeyEnv: z<string, string>;
+    apiKeyEnv: z<string, string, "volatile-defined">;
     /** Per-request connect/read timeout for the ComfyUI HTTP client. */
-    connectTimeoutMs: z<number, number>;
+    connectTimeoutMs: z<number, number, "volatile-defined">;
     /** How long a synchronous generation (or background job) waits for workflow completion. */
-    timeoutMs: z<number, number>;
+    timeoutMs: z<number, number, "volatile-defined">;
     /** History polling interval while waiting for completion. */
-    pollIntervalMs: z<number, number>;
+    pollIntervalMs: z<number, number, "volatile-defined">;
     /** Max media items returned per completed workflow. */
-    maxMediaItems: z<number, number>;
+    maxMediaItems: z<number, number, "volatile-defined">;
     /** Max bytes the media proxy streams for one file. */
-    maxMediaBytes: z<number, number>;
+    maxMediaBytes: z<number, number, "volatile-defined">;
     /** Directory for plugin data (workflow library, asset index); empty means DSH_HOME/data/dsh-comfyui. */
-    dataDir: z<string, string>;
+    dataDir: z<string, string, "defined">;
     /** Max asset records kept in the asset index. */
-    maxAssets: z<number, number>;
+    maxAssets: z<number, number, "defined">;
     /** Directory holding the per-workflow skill packs. Empty = `<dataDir>/skills`.
      * Separate from `dataDir` on purpose: packs are documents a user may want on
      * another drive, in a synced folder, or under version control, while the
      * JSON state files belong with the rest of the plugin data. Must be absolute;
      * a relative value is ignored. */
-    skillsDir: z<string, string>;
+    skillsDir: z<string, string, "volatile-defined">;
     /** External base URL for generated media (e.g. http://192.168.1.5:3080). Empty = auto-detect the browser's request host, then http://127.0.0.1:<webServerPort>. */
-    mediaHost: z<string, string>;
+    mediaHost: z<string, string, "volatile-defined">;
     /** ComfyUI's output directory on this machine, used to delete asset files
      * from the panel. Empty = infer it from the file paths ComfyUI reports;
      * deletion falls back to removing the index record when neither works
      * (e.g. a ComfyUI running on another host). */
-    outputDir: z<string, string>;
+    outputDir: z<string, string, "defined">;
     /** ComfyUI install root(s) on this machine (absolute paths). Multiple
      * entries are allowed because ComfyUI folders can be mapped/mounted (extra
      * models dirs, several installs, portable copies). The agent reads them to
      * locate ComfyUI files directly (workflows, models, the TTS-Audio-Suite
      * voice library) without guessing or asking the user. */
-    comfyuiDirs: z<string[], string[]>;
-}>, Schemastery.ObjectT<{
+    comfyuiDirs: z<NoInfer<string[]>, NoInfer<string[]>, "volatile-defined">;
+}>>, Schemastery.ObjectT<NoInfer<{
     /** ComfyUI HTTP server base URL. */
-    baseUrl: z<string, string>;
+    baseUrl: z<string, string, "volatile-defined">;
     /** Environment-variable name of the optional API key (credentials ref). */
-    apiKeyEnv: z<string, string>;
+    apiKeyEnv: z<string, string, "volatile-defined">;
     /** Per-request connect/read timeout for the ComfyUI HTTP client. */
-    connectTimeoutMs: z<number, number>;
+    connectTimeoutMs: z<number, number, "volatile-defined">;
     /** How long a synchronous generation (or background job) waits for workflow completion. */
-    timeoutMs: z<number, number>;
+    timeoutMs: z<number, number, "volatile-defined">;
     /** History polling interval while waiting for completion. */
-    pollIntervalMs: z<number, number>;
+    pollIntervalMs: z<number, number, "volatile-defined">;
     /** Max media items returned per completed workflow. */
-    maxMediaItems: z<number, number>;
+    maxMediaItems: z<number, number, "volatile-defined">;
     /** Max bytes the media proxy streams for one file. */
-    maxMediaBytes: z<number, number>;
+    maxMediaBytes: z<number, number, "volatile-defined">;
     /** Directory for plugin data (workflow library, asset index); empty means DSH_HOME/data/dsh-comfyui. */
-    dataDir: z<string, string>;
+    dataDir: z<string, string, "defined">;
     /** Max asset records kept in the asset index. */
-    maxAssets: z<number, number>;
+    maxAssets: z<number, number, "defined">;
     /** Directory holding the per-workflow skill packs. Empty = `<dataDir>/skills`.
      * Separate from `dataDir` on purpose: packs are documents a user may want on
      * another drive, in a synced folder, or under version control, while the
      * JSON state files belong with the rest of the plugin data. Must be absolute;
      * a relative value is ignored. */
-    skillsDir: z<string, string>;
+    skillsDir: z<string, string, "volatile-defined">;
     /** External base URL for generated media (e.g. http://192.168.1.5:3080). Empty = auto-detect the browser's request host, then http://127.0.0.1:<webServerPort>. */
-    mediaHost: z<string, string>;
+    mediaHost: z<string, string, "volatile-defined">;
     /** ComfyUI's output directory on this machine, used to delete asset files
      * from the panel. Empty = infer it from the file paths ComfyUI reports;
      * deletion falls back to removing the index record when neither works
      * (e.g. a ComfyUI running on another host). */
-    outputDir: z<string, string>;
+    outputDir: z<string, string, "defined">;
     /** ComfyUI install root(s) on this machine (absolute paths). Multiple
      * entries are allowed because ComfyUI folders can be mapped/mounted (extra
      * models dirs, several installs, portable copies). The agent reads them to
      * locate ComfyUI files directly (workflows, models, the TTS-Audio-Suite
      * voice library) without guessing or asking the user. */
-    comfyuiDirs: z<string[], string[]>;
-}>>;
+    comfyuiDirs: z<NoInfer<string[]>, NoInfer<string[]>, "volatile-defined">;
+}>>, "plain">;
 export type Config = {
     /** ComfyUI HTTP server base URL. */
     baseUrl: string;
@@ -109,3 +116,12 @@ export type Config = {
     /** ComfyUI install root(s) on this machine; the agent uses them to locate files directly. */
     comfyuiDirs: string[];
 };
+/**
+ * Resolve the Loader's parsed entry config into the plain object the plugin
+ * reads. Call again after `loader/volatile-update`: the references are the
+ * same objects, now holding the new values.
+ * @param raw - parsed entry config; fields may be volatile references.
+ * @param defaultDataDir - data directory used when `dataDir` is empty.
+ * @returns a fully populated plain config.
+ */
+export declare function resolveConfig(raw: Partial<Record<keyof Config, unknown>>, defaultDataDir: string): Config;

package/lib/config.js

@@ -1,24 +1,31 @@
 /**
  * dsh-comfyui host configuration. The same schema drives the Loader entry
- * config (cordis.yml patch) and the `comfyui:` settings section the browser
- * settings page writes, so one shape covers both doors into the same values.
+ * config (cordis.yml patch) and the settings page, so one shape covers both
+ * doors into the same values.
+ *
+ * Fields the plugin reads at use time are `.volatile()`: since dsh 0.1.7 the
+ * settings service only edits volatile fields (anything else fails with
+ * "Plugin entry has no volatile fields", Issue #11), and the Loader commits a
+ * volatile-only change into the running references without restarting the
+ * plugin. `dataDir` / `maxAssets` / `outputDir` stay ordinary: the store is
+ * built from them at startup, so a change there restarts the plugin.
  */
 import z from '@deepseek-ai/schemastery';
 export const Config = z.object({
     /** ComfyUI HTTP server base URL. */
-    baseUrl: z.string().default('http://127.0.0.1:8188'),
+    baseUrl: z.string().default('http://127.0.0.1:8188').volatile(),
     /** Environment-variable name of the optional API key (credentials ref). */
-    apiKeyEnv: z.string().default('COMFYUI_API_KEY'),
+    apiKeyEnv: z.string().default('COMFYUI_API_KEY').volatile(),
     /** Per-request connect/read timeout for the ComfyUI HTTP client. */
-    connectTimeoutMs: z.number().min(1_000).max(60_000).default(10_000),
+    connectTimeoutMs: z.number().min(1_000).max(60_000).default(10_000).volatile(),
     /** How long a synchronous generation (or background job) waits for workflow completion. */
-    timeoutMs: z.number().min(5_000).max(3_600_000).default(900_000),
+    timeoutMs: z.number().min(5_000).max(3_600_000).default(900_000).volatile(),
     /** History polling interval while waiting for completion. */
-    pollIntervalMs: z.number().min(200).max(10_000).default(1_000),
+    pollIntervalMs: z.number().min(200).max(10_000).default(1_000).volatile(),
     /** Max media items returned per completed workflow. */
-    maxMediaItems: z.number().min(1).max(50).default(12),
+    maxMediaItems: z.number().min(1).max(50).default(12).volatile(),
     /** Max bytes the media proxy streams for one file. */
-    maxMediaBytes: z.number().min(64 * 1024).max(512 * 1024 * 1024).default(64 * 1024 * 1024),
+    maxMediaBytes: z.number().min(64 * 1024).max(512 * 1024 * 1024).default(64 * 1024 * 1024).volatile(),
     /** Directory for plugin data (workflow library, asset index); empty means DSH_HOME/data/dsh-comfyui. */
     dataDir: z.string().default(''),
     /** Max asset records kept in the asset index. */
@@ -28,9 +35,9 @@ export const Config = z.object({
      * another drive, in a synced folder, or under version control, while the
      * JSON state files belong with the rest of the plugin data. Must be absolute;
      * a relative value is ignored. */
-    skillsDir: z.string().default(''),
+    skillsDir: z.string().default('').volatile(),
     /** External base URL for generated media (e.g. http://192.168.1.5:3080). Empty = auto-detect the browser's request host, then http://127.0.0.1:<webServerPort>. */
-    mediaHost: z.string().default(''),
+    mediaHost: z.string().default('').volatile(),
     /** ComfyUI's output directory on this machine, used to delete asset files
      * from the panel. Empty = infer it from the file paths ComfyUI reports;
      * deletion falls back to removing the index record when neither works
@@ -41,5 +48,45 @@ export const Config = z.object({
      * models dirs, several installs, portable copies). The agent reads them to
      * locate ComfyUI files directly (workflows, models, the TTS-Audio-Suite
      * voice library) without guessing or asking the user. */
-    comfyuiDirs: z.array(z.string()).default([]),
+    comfyuiDirs: z.array(z.string()).default([]).volatile(),
 });
+/** The volatile-reference protocol shared across cosmokit copies (Symbol.for). */
+const VOLATILE_WRITE = Symbol.for('cosmokit.volatile.write');
+/** Read a config value that may be a volatile reference (dsh 0.1.7+) or a plain value (older hosts). */
+function current(value) {
+    if (typeof value === 'object' && value !== null && VOLATILE_WRITE in value) {
+        return value.get();
+    }
+    return value;
+}
+/**
+ * Resolve the Loader's parsed entry config into the plain object the plugin
+ * reads. Call again after `loader/volatile-update`: the references are the
+ * same objects, now holding the new values.
+ * @param raw - parsed entry config; fields may be volatile references.
+ * @param defaultDataDir - data directory used when `dataDir` is empty.
+ * @returns a fully populated plain config.
+ */
+export function resolveConfig(raw, defaultDataDir) {
+    const str = (value, fallback) => (typeof value === 'string' ? value : fallback);
+    const num = (value, fallback) => (typeof value === 'number' ? value : fallback);
+    const dataDir = str(current(raw.dataDir), '');
+    const dirs = current(raw.comfyuiDirs);
+    return {
+        baseUrl: str(current(raw.baseUrl), 'http://127.0.0.1:8188'),
+        apiKeyEnv: str(current(raw.apiKeyEnv), 'COMFYUI_API_KEY'),
+        connectTimeoutMs: num(current(raw.connectTimeoutMs), 10_000),
+        timeoutMs: num(current(raw.timeoutMs), 900_000),
+        pollIntervalMs: num(current(raw.pollIntervalMs), 1_000),
+        maxMediaItems: num(current(raw.maxMediaItems), 12),
+        maxMediaBytes: num(current(raw.maxMediaBytes), 64 * 1024 * 1024),
+        dataDir: dataDir !== '' ? dataDir : defaultDataDir,
+        maxAssets: num(current(raw.maxAssets), 200),
+        skillsDir: str(current(raw.skillsDir), ''),
+        mediaHost: str(current(raw.mediaHost), ''),
+        outputDir: str(current(raw.outputDir), ''),
+        comfyuiDirs: Array.isArray(dirs)
+            ? dirs.filter((dir) => typeof dir === 'string' && dir.trim() !== '')
+            : [],
+    };
+}

package/lib/convert.d.ts

@@ -12,6 +12,18 @@ export type ConvertResult = {
     ok: false;
     error: string;
 };
+/**
+ * Flatten DynamicCombo values that 0.2.0–0.5.1 extraction wrapped as
+ * `{ key, inputs: { sub: value } }` back into the flat API shape
+ * (`master: key`, `master.sub: value`, recursively). Library entries saved by
+ * those versions still carry the wrapped shape; normalizing at queue time
+ * heals them without asking users to re-extract (Issue #10). Returns the same
+ * object when nothing needed flattening.
+ */
+export declare function flattenDynamicCombos<W extends Record<string, {
+    class_type: string;
+    inputs: Record<string, unknown>;
+}>>(workflow: W): W;
 /**
  * Convert a UI-graph workflow (or one extracted component of it) to API
  * format using the live node definitions.

package/lib/convert.js

@@ -30,6 +30,51 @@ function inputSpec(objectInfo, classType, name) {
     }
     return undefined;
 }
+/** Option keys a COMFY_DYNAMICCOMBO_V3 spec declares, or undefined when unreadable. */
+function dynamicComboKeys(spec) {
+    const options = isObject(spec[1]) ? spec[1].options : undefined;
+    if (!Array.isArray(options))
+        return undefined;
+    const keys = options
+        .map((option) => (isObject(option) ? option.key : undefined))
+        .filter((key) => typeof key === 'string');
+    return keys.length > 0 ? keys : undefined;
+}
+/**
+ * Flatten DynamicCombo values that 0.2.0–0.5.1 extraction wrapped as
+ * `{ key, inputs: { sub: value } }` back into the flat API shape
+ * (`master: key`, `master.sub: value`, recursively). Library entries saved by
+ * those versions still carry the wrapped shape; normalizing at queue time
+ * heals them without asking users to re-extract (Issue #10). Returns the same
+ * object when nothing needed flattening.
+ */
+export function flattenDynamicCombos(workflow) {
+    const isWrapped = (value) => isObject(value) && typeof value.key === 'string' && isObject(value.inputs)
+        && Object.keys(value).every((k) => k === 'key' || k === 'inputs');
+    const flatten = (prefix, value, into) => {
+        if (!isWrapped(value)) {
+            into[prefix] = value;
+            return;
+        }
+        into[prefix] = value.key;
+        for (const [sub, subValue] of Object.entries(value.inputs))
+            flatten(`${prefix}.${sub}`, subValue, into);
+    };
+    let changed = false;
+    const out = {};
+    for (const [id, node] of Object.entries(workflow)) {
+        if (!Object.values(node.inputs).some(isWrapped)) {
+            out[id] = node;
+            continue;
+        }
+        changed = true;
+        const inputs = {};
+        for (const [name, value] of Object.entries(node.inputs))
+            flatten(name, value, inputs);
+        out[id] = { ...node, inputs };
+    }
+    return changed ? out : workflow;
+}
 /**
  * Derive the ordered widget-input names for one node. The graph's own input
  * array is the source of truth — widgets_values is stored in UI widget order,
@@ -289,27 +334,48 @@ export function convertGraphToApi(graph, objectInfo, options) {
                     inputs[name] = value;
             }
         }
-        // Collapse DynamicCombo V3 flat keys back into the API object shape:
-        // { key, inputs: { subWidget: value } } — the master combo's value is the
-        // selected option key, and its `master.sub` siblings become the sub-inputs.
+        // DynamicCombo V3 stays FLAT in the API prompt: the master input holds the
+        // selected option key as a string and its sub-widgets sit beside it as
+        // `master.sub` keys. The server rebuilds the nested shape itself
+        // (DynamicCombo._expand_schema_for_dynamic matches the plain value against
+        // option keys); wrapping it as { key, inputs } never matches, the input is
+        // dropped, and the node dies at execute() — Issue #10 (SaveVideo 'codec').
+        // Validate the selected key here so a bad value fails at extraction rather
+        // than after a full generation.
         for (const [name, value] of Object.entries(inputs)) {
             const spec = inputSpec(objectInfo, node.type, name);
             if (!Array.isArray(spec) || spec[0] !== 'COMFY_DYNAMICCOMBO_V3')
                 continue;
-            const sub = {};
-            for (const [other, otherValue] of Object.entries(inputs)) {
-                if (other.startsWith(`${name}.`))
-                    sub[other.slice(name.length + 1)] = otherValue;
+            const keys = dynamicComboKeys(spec);
+            if (keys === undefined || Array.isArray(value))
+                continue;
+            if (typeof value !== 'string' || !keys.includes(value)) {
+                return {
+                    ok: false,
+                    error: `节点 ${node.type}（id ${node.id}）的输入 ${name} 取值 ${JSON.stringify(value)} 不是可选项之一（${keys.join(' / ')}）——请在 ComfyUI 画布上重新选择后再提取`,
+                };
             }
-            inputs[name] = { key: value, inputs: sub };
-            for (const subName of Object.keys(sub))
-                delete inputs[`${name}.${subName}`];
         }
         workflow[String(node.id)] = { class_type: node.type, inputs };
     }
     if (Object.keys(workflow).length === 0) {
         return { ok: false, error: '转换结果为空（图中没有可执行的节点）' };
     }
+    // Reference integrity: every link must point at a node of this same output.
+    // A component that leaves out an upstream node would otherwise extract as
+    // ok and only fail at run time (Issue #8).
+    for (const [id, node] of Object.entries(workflow)) {
+        for (const [name, value] of Object.entries(node.inputs)) {
+            if (!Array.isArray(value) || value.length !== 2 || typeof value[0] !== 'string' || typeof value[1] !== 'number')
+                continue;
+            if (workflow[value[0]] === undefined) {
+                return {
+                    ok: false,
+                    error: `节点 ${node.class_type}（id ${id}）的输入 ${name} 引用了不在本次提取范围内的节点 ${value[0]}——请改用「整体提取」，或把这部分画布整理成独立的流程`,
+                };
+            }
+        }
+    }
     // Fail loudly on nodes whose required inputs are missing — the source graph
     // itself is broken (inputs never wired), and the server would only echo a
     // confusing per-node error at run time. Growable/lazy v3 inputs (template

package/lib/index.d.ts

@@ -21,4 +21,4 @@ export declare const inject: string[];
  * The plugin body. The loader validates the entry config against `Config`
  * (defaults applied), then hands the resolved object to apply.
  */
-export declare function apply(ctx: Context, entryConfig: Partial<ConfigType>): Promise<void>;
+export declare function apply(ctx: Context, entryConfig: Partial<Record<keyof ConfigType, unknown>>): Promise<void>;