From 187b85b054764bfd6cc8f34be5640b353bc08616 Mon Sep 17 00:00:00 2001
From: Ethanfel <ethan.fel@ts-pc.fr>
Date: Sun, 1 Mar 2026 00:12:30 +0100
Subject: [PATCH] Clean up: remove unnecessary info.outputs logic, set label on
 reused slots

The actual fix was setting slot.label alongside slot.name. Reverted
onConfigure to read from widget values (which work correctly) and
ensured label is set on both new and reused output slots.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 web/project_dynamic.js | 49 ++++++++++--------------------------------
 1 file changed, 11 insertions(+), 38 deletions(-)

diff --git a/web/project_dynamic.js b/web/project_dynamic.js
index 15b525e..7f2de6e 100644
--- a/web/project_dynamic.js
+++ b/web/project_dynamic.js
@@ -143,6 +143,7 @@ app.registerExtension({
                     if (key in oldSlots) {
                         const slot = this.outputs[oldSlots[key]];
                         slot.type = type;
+                        slot.label = key;
                         newOutputs.push(slot);
                         delete oldSlots[key];
                     } else {
@@ -197,37 +198,12 @@ app.registerExtension({
             const okWidget = this.widgets?.find(w => w.name === "output_keys");
             const otWidget = this.widgets?.find(w => w.name === "output_types");
 
-            // Primary source: read output names from serialized node info.
-            // Hidden widget values may not survive ComfyUI's serialization,
-            // but info.outputs always contains the correct saved output names.
-            let keys = [];
-            let types = [];
-            const savedOutputs = info.outputs || [];
-            for (let i = 0; i < savedOutputs.length; i++) {
-                if (savedOutputs[i].name === "total_sequences") continue;
-                if (/^output_\d+$/.test(savedOutputs[i].name)) continue;
-                keys.push(savedOutputs[i].name);
-                types.push(savedOutputs[i].type || "*");
-            }
-
-            // Fallback: try hidden widget values
-            if (keys.length === 0) {
-                const wKeys = okWidget?.value
-                    ? okWidget.value.split(",").filter(k => k.trim())
-                    : [];
-                if (wKeys.length > 0) {
-                    keys = wKeys;
-                    types = otWidget?.value
-                        ? otWidget.value.split(",")
-                        : [];
-                }
-            }
-
-            // Update hidden widgets so the Python backend has keys for execution
-            if (keys.length > 0) {
-                if (okWidget) okWidget.value = keys.join(",");
-                if (otWidget) otWidget.value = types.join(",");
-            }
+            const keys = okWidget?.value
+                ? okWidget.value.split(",").filter(k => k.trim())
+                : [];
+            const types = otWidget?.value
+                ? otWidget.value.split(",")
+                : [];
 
             // Ensure slot 0 is total_sequences (INT)
             if (this.outputs.length === 0 || this.outputs[0].name !== "total_sequences") {
@@ -267,13 +243,10 @@ app.registerExtension({
                     this.removeOutput(this.outputs.length - 1);
                 }
             } else if (this.outputs.length > 1) {
-                const dynamicOutputs = this.outputs.slice(1).filter(
-                    o => !/^output_\d+$/.test(o.name)
-                );
-                if (dynamicOutputs.length > 0) {
-                    if (okWidget) okWidget.value = dynamicOutputs.map(o => o.name).join(",");
-                    if (otWidget) otWidget.value = dynamicOutputs.map(o => o.type).join(",");
-                }
+                // Widget values empty but serialized dynamic outputs exist — sync widgets
+                const dynamicOutputs = this.outputs.slice(1);
+                if (okWidget) okWidget.value = dynamicOutputs.map(o => o.name).join(",");
+                if (otWidget) otWidget.value = dynamicOutputs.map(o => o.type).join(",");
             }
 
             this.setSize(this.computeSize());