comfyui-mcp 0.50.71 → 0.50.73

package/dist/orchestrator/panel-tools.js

@@ -4704,8 +4704,21 @@ export function makePanelToolCtx(bridge, tabId, workflowTargets) {
             if (!opts?.scopeRecoveryConsent || !pinProvenDead) {
                 return { previous, current: previous, rebound: false };
             }
-            const repinned = bridge.repinScopeToActive?.(previous);
-            return { previous, current: previous, rebound: Boolean(repinned) };
+            // #1077 Finding 2 — carry the REASON out when nothing was repinned. This
+            // collapsed to `Boolean(repinned)`, so four different refusals arrived at
+            // the caller as one bare `false` and the user was told only that the
+            // adoption was refused. One of those refusals repeats forever (an active
+            // tab on another backend's conversation while this one has several), and
+            // nothing in the reply hinted that naming a workflow is the way out.
+            const outcome = bridge.repinScopeToActive?.(previous);
+            const repinned = typeof outcome === "string" ? outcome : undefined;
+            const repinRefusal = outcome && typeof outcome === "object" ? outcome.reason : undefined;
+            return {
+                previous,
+                current: previous,
+                rebound: Boolean(repinned),
+                ...(repinRefusal ? { repinRefusal } : {}),
+            };
         }
         // A healthy binding is left untouched (never disturb a live session). Recovery only
         // fires for an orphaned/stale tab id.
@@ -7624,7 +7637,16 @@ export function buildPanelToolDefs() {
                     // THE explicit scope-recovery consent (#884 gate 3) — the only
                     // caller that may escape a DEAD scope pin (a healthy pin still
                     // stays put; see rebindToActiveTab's double gate).
-                    ctx.rebindToActiveTab({ scopeRecoveryConsent: true });
+                    const rebind = ctx.rebindToActiveTab({ scopeRecoveryConsent: true });
+                    // #1077 Finding 2 — a scope repin that declined now says WHY, and
+                    // this is where the user reads it. The refusal used to be a bare
+                    // boolean, so a session stuck in the one state that repeats forever
+                    // (the active tab belongs to another backend's conversation while
+                    // this one has several eligible tabs) saw no difference from a
+                    // healthy pin being correctly left alone.
+                    if (rebind?.repinRefusal) {
+                        rebindNote += ` The session pin was NOT moved: ${rebind.repinRefusal}.`;
+                    }
                 }
                 catch (err) {
                     // #474: with 2+ live tabs the rebind is AMBIGUOUS — fail so the user picks.