Commit Graph

10 Commits

Author SHA1 Message Date
1b7f31ff73 Clean up: remove diagnostic logging and unnecessary info.outputs logic
The actual fix was setting slot.label alongside slot.name. The
info.outputs approach was wrong (it always contains Python defaults).
Widget values were being restored correctly all along.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-01 00:12:27 +01:00
c5901bb438 Fix: set output label alongside name for LiteGraph rendering 2026-03-01 00:06:19 +01:00
64736c03ab Add post-rename and delayed diagnostic logging 2026-03-01 00:00:12 +01:00
efe0be3224 Add diagnostic logging to onConfigure for debugging output name persistence
Temporary instrumentation to trace data flow during workflow reload.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-02-28 23:48:59 +01:00
5f9ab80f2f Fix output names not surviving page refresh in JSONDynamicLoader
Read output names from info.outputs (serialized node data) instead of
hidden widget values, which ComfyUI may not persist across reloads.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-02-28 23:43:35 +01:00
ea57b28812 Fix dynamic outputs breaking Kijai Set/Get nodes on workflow load
Defer removal of the 32 default Python outputs from onNodeCreated using
queueMicrotask so they remain available during graph loading. ComfyUI
creates all nodes before configuring them, and nodes like Kijai's SetNode
resolve links during their configure step — if outputs were already
removed, the resolution failed with "node input undefined". The deferred
cleanup only runs for new nodes; loaded workflows set _configured=true
in onConfigure first. Also adds a fallback to sync widget values from
serialized outputs when widget restoration fails.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-02-28 00:22:53 +01:00
f05735dcc0 Fix refresh wiping all links when JSON file is missing
Some checks failed
Publish to Comfy registry / Publish Custom Node to registry (push) Has been cancelled
When the file at json_path didn't exist, clicking Refresh Outputs
returned empty keys causing all outputs and links to be removed.
Now the API returns an error flag and the frontend bails out early,
preserving existing outputs.

Bump version to 1.2.1.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-02-24 14:19:05 +01:00
b5b4a26f6d Add PreviewToLoad node for bridging preview images to LoadImage nodes
Previews an image (like PreviewImage) and saves a copy to input/ so a
LoadImage node can reference it.  JS extension adds a target-node-ID
widget and "Send to Load Image" button that updates the target's combo.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-02-24 11:49:13 +01:00
c909f854f4 Rename node to JSONDynamicLoader to avoid collision with JSON Manager
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-02-23 19:36:28 +01:00
79157f4289 Initial commit: JSON Loader (Dynamic) ComfyUI node
Auto-discovers JSON keys and exposes them as typed output slots.
Includes JS frontend for refresh, connection-safe updates, and
workflow persistence of keys and types.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-02-23 19:32:57 +01:00