comfyui-mcp 0.52.170 → 0.52.171

package/dist/orchestrator/panel-tools.js

@@ -15652,9 +15652,11 @@ function runLateAckGraceMs() {
  *
  * Returns null when there is nothing to reconcile. The negatives are deliberate:
  *
- *  - NOT an unacked post-write (timeout or disconnect), or the Panel's explicit
- *    id-less queued_unknown receipt (#1728) ⇒ nothing to wait for.
- *    Gated on the bridge's own typed marks, never on message text: an acked
+ *  - A `queued_unknown` with no exact receipt and no scoped queue-attribution
+ *    contract ⇒ nothing to claim. Full-graph uncertainty waits only for the
+ *    bridge's rid-correlated `run_receipt`; scoped uncertainty may additionally
+ *    use its existing bounded queue observation. Gated on the bridge's own typed
+ *    marks, never on message text: an acked
  *    panel error can quote our timeout/OUTCOME UNKNOWN sentence verbatim
  *    (#1468 round 2), and the difference is whether the tab answered, which
  *    only the bridge knows.
@@ -15696,7 +15698,7 @@ function latePanelReceiptResult(parsed, latePromptIds) {
         ...(ids.length > 1 ? { prompt_ids: ids } : {}),
     };
 }
-async function reconcileLateRunAck(ctx, res, rid, preRunningPromptId, allowPanelQueueUnknown = false, expectedTabId, expectedPromptCount = 1) {
+async function reconcileLateRunAck(ctx, res, rid, preRunningPromptId, allowPanelQueueUnknownReceipt = false, expectedTabId, expectedPromptCount = 1, allowQueueUnknownInference = false) {
     const parsed = parseToolResultJson(res);
     // An exact Panel receipt is safe even when a queued_unknown reply already
     // carried some ids: the bridge event is the producer's own prompt capture,
@@ -15705,10 +15707,15 @@ async function reconcileLateRunAck(ctx, res, rid, preRunningPromptId, allowPanel
     // available after the transport failed. The old queue path remains
     // id-less-only below.
     const receiptEligible = Boolean(rid) &&
-        ((allowPanelQueueUnknown && isPanelQueueUncertainty(parsed)) ||
+        ((allowPanelQueueUnknownReceipt && isPanelQueueUncertainty(parsed)) ||
             isReplyTimeoutResult(res) ||
             isMidCommandDisconnectResult(res));
-    const queueEligible = isUnackedGraphRunResult(res, allowPanelQueueUnknown);
+    // A run_receipt is exact Panel evidence for BOTH full and scoped runs. Queue
+    // observation is weaker: only the scoped Panel contract makes a new queue id
+    // attributable to this dispatch. Keep those authorities separate so enabling
+    // full-graph receipt recovery cannot turn a foreign back-to-back render into
+    // this run's prompt id.
+    const queueEligible = isUnackedGraphRunResult(res, allowQueueUnknownInference);
     if (!receiptEligible && !queueEligible)
         return null;
     const fromQueue = () => {
@@ -18902,9 +18909,13 @@ export function buildPanelToolDefs() {
             // guidance. Rebuilding the reply here (rather than describing it) is what
             // makes that possible — `ok()` is exactly what ctx.call would have produced.
             const reconcileRun = async () => {
-                const recovered = await reconcileLateRunAck(ctx, res, runRid, pre.runningPromptId, typeof args.to_node_id === "number", runTicketTab, typeof args.batch_count === "number"
+                const recovered = await reconcileLateRunAck(ctx, res, runRid, pre.runningPromptId, 
+                // #2143 — queued_unknown is receipt-recoverable for full graph runs
+                // too. Only exact run_receipt evidence is allowed there; the weaker
+                // QueueMonitor inference remains scoped-run-only below.
+                true, runTicketTab, typeof args.batch_count === "number"
                     ? Math.max(1, Math.min(64, Math.floor(args.batch_count)))
-                    : 1);
+                    : 1, typeof args.to_node_id === "number");
                 if (!recovered)
                     return;
                 for (const entry of recovered.completionKeys ?? []) {