comfyui-mcp 0.52.163 → 0.52.165

package/dist/orchestrator/panel-tools.js

@@ -6407,6 +6407,21 @@ function promotedHostAppliedNote(plan) {
         `"${plan.hostWidget}". The promoted inner mapping node ${plan.inner.innerNodeId} ` +
         `"${plan.inner.widget}" is synchronized from that host rail.)`);
 }
+/** #2514 — an enclosing-subgraph host write is the parent-authoritative
+ * receipt. Advertise parent_widget_synced even when we never entered the
+ * inner scope (#2500/#2533 host-first), matching the remapped-inner path. */
+function promotedHostWriteReceipt(res, plan) {
+    const payload = parseToolResultJson(res);
+    const note = promotedHostAppliedNote(plan);
+    if (!payload)
+        return appendToolResultText(res, note);
+    const shaped = shapeParentAuthoritativePromotedWrite(payload, {
+        nodeId: plan.outerNodeId,
+        widget: plan.hostWidget,
+        synced: true,
+    });
+    return appendToolResultText(rewriteToolResultJson(res, shaped), note);
+}
 const PROMOTED_PRIMITIVE_INNER_TYPES = new Set([
     "PrimitiveInt",
     "PrimitiveFloat",
@@ -6419,7 +6434,9 @@ const PROMOTED_PRIMITIVE_INNER_TYPES = new Set([
  * Same-name proxyWidgets promotions keep the inner write so #2314's known-bad
  * live probe still runs before any container mutation.
  * #2500 — a Primitive* `value` that exists as a live input is driven by the
- * promoted rail even when the parent-rail witness is missing; write the host. */
+ * promoted rail even when the parent-rail witness is missing; write the host.
+ * #2533 — an authoritative parent rail whose inner widget is a live input
+ * (unet_name, clip_name, labelled prompt) is the same store; write the host. */
 function shouldWritePromotedHostFirst(plan) {
     if (plan.innerLinkDriven)
         return true;
@@ -6924,7 +6941,7 @@ async function preparePromotedWidgetWrite(ctx, nodeId, widget) {
         ...(outerNodeIdentity ? { outerNodeIdentity } : {}),
         inner,
         innerNodeType,
-        innerLinkDriven: promotedInnerWidgetIsLinkDriven(innerNode, inner.widget, inner.terminal),
+        innerLinkDriven: promotedInnerWidgetIsLinkDriven(innerNode, inner.widget, inner.terminal, inner.parentRail),
         ...(inner.terminal ? { terminal: inner.terminal } : {}),
         scope,
         binding: {
@@ -16979,11 +16996,12 @@ export function buildPanelToolDefs() {
                     const afterMappingError = currentPromotedBindingError(ctx, plan.binding);
                     if (afterMappingError)
                         return promotedWriteRefusal(args.widget, afterMappingError);
-                    // #2500 — a link-driven inner Primitive (duration/value_2, fps/value_3)
-                    // accepts a write and warns that rendering still reads the enclosing
-                    // subgraph widget. Write that container first, at the caller's current
-                    // graph, and do not enter the inner scope. This is the #1655 fallback
-                    // path when the #2488 host-rail write already contradicted.
+                    // #2500/#2533 — a link-driven inner (Primitive value, or a same-name
+                    // COMBO/STRING live input with an authoritative parent rail) accepts
+                    // a write and warns that rendering still reads the enclosing subgraph
+                    // widget. Write that container first, at the caller's current graph,
+                    // and do not enter the inner scope. This is the #1655 fallback path
+                    // when the #2488 host-rail write already contradicted.
                     if (plan.innerLinkDriven) {
                         if (plan.terminal) {
                             const terminalBlocked = refuseKnownBadPromotedTerminal(plan.terminal);
@@ -17011,7 +17029,7 @@ export function buildPanelToolDefs() {
                         }
                         markPanelSchemaReady(panelSchemaKey(ctx));
                         const persisted = await persistUnpromotedControlAfterGenerate(written, ctx, plan.outerNodeId);
-                        return appendToolResultText(persisted, promotedHostAppliedNote(plan));
+                        return promotedHostWriteReceipt(persisted, plan);
                     }
                     const enter = await ctx.call({ cmd: "graph_enter_subgraph", node_id: plan.outerNodeId }, 15000);
                     if (enter.isError) {
@@ -17188,7 +17206,7 @@ export function buildPanelToolDefs() {
                     if (!hostWritten.isError) {
                         markPanelSchemaReady(panelSchemaKey(ctx));
                         const persisted = await persistUnpromotedControlAfterGenerate(hostWritten, ctx, promotedPlan.outerNodeId);
-                        return appendToolResultText(persisted, promotedHostAppliedNote(promotedPlan));
+                        return promotedHostWriteReceipt(persisted, promotedPlan);
                     }
                     const hostRefusal = parseContradictoryPromotedWidgetRefusal(textOfToolResult(hostWritten), hostWidget);
                     if (!hostRefusal || String(hostRefusal.nodeId) !== String(promotedPlan.outerNodeId)) {
@@ -17209,7 +17227,7 @@ export function buildPanelToolDefs() {
                         if (!remappedHost.isError) {
                             markPanelSchemaReady(panelSchemaKey(ctx));
                             const persisted = await persistUnpromotedControlAfterGenerate(remappedHost, ctx, promotedPlan.outerNodeId);
-                            return appendToolResultText(persisted, promotedHostAppliedNote(remapPlan));
+                            return promotedHostWriteReceipt(persisted, remapPlan);
                         }
                         if (!parseContradictoryPromotedWidgetRefusal(textOfToolResult(remappedHost), hostRefusal.widget)) {
                             return remappedHost;