comfyui-mcp 0.52.161 → 0.52.163

package/dist/orchestrator/panel-tools.js

@@ -62,7 +62,7 @@ import { tabRouteCarriesPath } from "./turn-origins.js";
 import { NODE_ID_MESSAGE, NODE_ID_PATTERN, normalizeNodeId, PLAIN_NODE_ID_PATTERN, unionErrorFor, } from "./node-id.js";
 import { unrecognizedKeysError } from "./unrecognized-keys.js";
 import { FIND_NODES_DEFAULT_FIELDS, FIND_NODES_FIELDS, GROUP_ID_MESSAGE, GROUP_ID_PATTERN, GROUP_REF_MESSAGE, normalizeGroupId, normalizeShowMediaItem, normalizeTodoItemKeys, pickAlias, projectFindNodesPayload, } from "./panel-arg-aliases.js";
-import { addressedNodeMatchesPersistRemedy, parseAmbiguousPromotedWidgetRefusal, parseContradictoryPromotedWidgetRefusal, parseLinkDrivenPromotedWriteWarning, parseSubgraphScopeRefusal, parseUnpromotedControlPersistRemedy, promotedScopeWitnessFromEnvelope, promotedViewingMatchesScope, resolveLegacyInnerPromotedTarget, resolveInnerPromotedTarget, validatePromotedSubgraphEnvelope, isInnerLinkDrivenWriteWarning, shapeParentAuthoritativePromotedWrite, } from "./promoted-widget.js";
+import { addressedNodeMatchesPersistRemedy, parseAmbiguousPromotedWidgetRefusal, parseContradictoryPromotedWidgetRefusal, parseLinkDrivenPromotedWriteWarning, parseSubgraphScopeRefusal, parseUnpromotedControlPersistRemedy, promotedScopeWitnessFromEnvelope, promotedViewingMatchesScope, resolveLegacyInnerPromotedTarget, promotedInnerWidgetIsLinkDriven, resolveInnerPromotedTarget, validatePromotedSubgraphEnvelope, isInnerLinkDrivenWriteWarning, shapeParentAuthoritativePromotedWrite, } from "./promoted-widget.js";
 import { includeRequestedCreateGroupMembers } from "./create-group-membership.js";
 import { sizeForUncollapse } from "./edit-node-uncollapse.js";
 import { fastGroupsFilterPropertyNote, isFastGroupsFilterProperty, } from "./rgthree-fast-groups-property.js";
@@ -6417,8 +6417,12 @@ const PROMOTED_PRIMITIVE_INNER_TYPES = new Set([
 /** #2488 — a promoted subgraph INPUT whose inner widget is a different name
  * (frame_counts → length) or a linked primitive must be written on the host.
  * Same-name proxyWidgets promotions keep the inner write so #2314's known-bad
- * live probe still runs before any container mutation. */
+ * 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. */
 function shouldWritePromotedHostFirst(plan) {
+    if (plan.innerLinkDriven)
+        return true;
     if (!plan.inner.parentRail)
         return false;
     if (plan.hostWidget.toLowerCase() !== plan.inner.widget.toLowerCase())
@@ -6920,6 +6924,7 @@ async function preparePromotedWidgetWrite(ctx, nodeId, widget) {
         ...(outerNodeIdentity ? { outerNodeIdentity } : {}),
         inner,
         innerNodeType,
+        innerLinkDriven: promotedInnerWidgetIsLinkDriven(innerNode, inner.widget, inner.terminal),
         ...(inner.terminal ? { terminal: inner.terminal } : {}),
         scope,
         binding: {
@@ -16974,6 +16979,40 @@ 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.
+                    if (plan.innerLinkDriven) {
+                        if (plan.terminal) {
+                            const terminalBlocked = refuseKnownBadPromotedTerminal(plan.terminal);
+                            if (terminalBlocked)
+                                return terminalBlocked;
+                        }
+                        const hostWidget = plan.hostWidget;
+                        const hostScope = plan.outerScope;
+                        const written = await write(plan.outerNodeId, hostWidget, plan.outerNodeType, () => {
+                            const error = currentPromotedBindingError(ctx, plan.binding);
+                            if (error) {
+                                throw new Error(`Refusing promoted container graph_set_widget: ${error}. No graph_set_widget was dispatched.`);
+                            }
+                            if (hostScope) {
+                                const scopeError = currentPromotedExpectedScopeError(ctx, hostScope);
+                                if (scopeError) {
+                                    throw new Error(`Refusing promoted container graph_set_widget: ${scopeError}. No graph_set_widget was dispatched.`);
+                                }
+                            }
+                        }, hostScope, plan.outerNodeIdentity);
+                        if (written.isError) {
+                            return appendToolResultText(written, `\n\n(Tried the enclosing subgraph widget on node ${plan.outerNodeId} ` +
+                                `"${hostWidget}" first because the promoted inner widget is link-driven. ` +
+                                `Navigation scope was left unchanged.)`);
+                        }
+                        markPanelSchemaReady(panelSchemaKey(ctx));
+                        const persisted = await persistUnpromotedControlAfterGenerate(written, ctx, plan.outerNodeId);
+                        return appendToolResultText(persisted, promotedHostAppliedNote(plan));
+                    }
                     const enter = await ctx.call({ cmd: "graph_enter_subgraph", node_id: plan.outerNodeId }, 15000);
                     if (enter.isError) {
                         return promotedWriteRefusal(args.widget, `panel_enter_subgraph failed (${textOfToolResult(enter)})`);