@animalabs/agent-framework 0.26.1 → 0.26.2

package/dist/src/framework.js

@@ -45,7 +45,7 @@ import { safeSlice } from './safe-slice.js';
 import { SEND_TOOLS, bareToolName } from './send-tools.js';
 import { boundedErrorMessage, errorMessage } from './error-message.js';
 import { ProviderWaits, EVERY_MODEL, isReleaseModel, withoutControls } from './provider-waits.js';
-import { nonErrorResultText, shownToScript, toolResultDataToHistoryString, truncateForHistory, DEFAULT_TOOL_RESULT_INLINE_MAX_CHARS, } from './tool-result-history.js';
+import { nonErrorResultText, shownToScript, shownToScriptFromServer, toolResultDataToHistoryString, truncateForHistory, DEFAULT_TOOL_RESULT_INLINE_MAX_CHARS, } from './tool-result-history.js';
 import { TOOL_IMAGE_REF_RE, ToolImageLedger, cutStoredToolResultToFit, formatKeptImageSlots, parseImagePlaceholders, sha256Hex, storedImageSlots, toolResultCutAt, toolResultShare, } from './tool-image-ledger.js';
 import { randomUUID, createHash } from 'node:crypto';
 import { describeCodeExecution, CODE_EXECUTION_TOOL_NAME, isolateSettings, validateCodeExecutionConfig, } from './code-execution/tool-definition.js';
@@ -671,6 +671,63 @@ function announcedDomainsOf(announce) {
 function resumesLogicalTurn(reason) {
     return reason === 'context_budget_restart' || reason === 'tool_result_guard_retry';
 }
+/**
+ * Marks a held turn's retry: the request a provider hold keeps for a turn
+ * whose call failed into it, or whose compile it held with nothing sent.
+ * That turn has begun and not ended: it starts again when the hold releases
+ * (a new start of its open turn, as 0.26.0 started it). So, as a
+ * continuation on its way does (continuesTurn), it keeps its agent's turn
+ * alive meanwhile, passes a quiesce, and goes with an abandon, a stop() or
+ * its agent's removal. A symbol, not a field: hosts build requests too, and
+ * only the framework may say one is a turn's own retry. Spreads and rests
+ * copy it, as every copy of a request the scheduler makes is one.
+ */
+const TURN_RETRY = Symbol('turn-retry');
+/**
+ * The batch a start that carries a turn on (continuesTurn) was merged from,
+ * when the scheduler merged wakes that aren't the turn's into it: every
+ * request it took in, as each was. Until the start takes its turn token it
+ * stands for them all. A scheduler pass that takes it back puts the batch in
+ * its place, each request with its own identity and flags; a quiesce at the
+ * auxiliary park sends the batch back to the scheduler; and a caller's stop
+ * or an abandon that drops the start puts back the batch's wakes, the
+ * requests that don't carry the turn on (dropContinuations), where they run
+ * as a turn of their own, as wakes queued while a turn streams do. Once the
+ * start takes its token the turn reads them, and beginAgentTurn drops the
+ * record. A symbol, as TURN_RETRY is, so spreads and rests copy it.
+ */
+const MERGED_BATCH = Symbol('merged-batch');
+function mergedBatch(request) {
+    return request[MERGED_BATCH];
+}
+/** The wakes of a merged start's batch (MERGED_BATCH): its requests that
+ *  don't carry the turn on. */
+function mergedWakes(request) {
+    return (mergedBatch(request) ?? []).filter((r) => !continuesTurn(r));
+}
+function isTurnRetry(request) {
+    return request[TURN_RETRY] === true;
+}
+/** How to go on sooner than a hold keeping a turn's retry lets the turn go
+ *  on: what surgery's and the puppet's refusals advise. */
+function heldTurnWayOn(statedWait) {
+    return 'a quiesce lets a held turn start when its hold ends and finish, so to go on sooner, end it with quiesce ' +
+        'with abandon: true, which ends it once the quiesce\'s wait for the drain runs out (at once on a host ' +
+        'already quiesced)' +
+        (statedWait
+            ? ", or lift a stated wait with release-provider-wait, so the turn tries again once no host's or " +
+                'acceleration hold binds it'
+            : '');
+}
+/** A copy of a held turn's trigger, marked as its retry (TURN_RETRY). */
+function asTurnRetry(trigger) {
+    return { ...trigger, [TURN_RETRY]: true };
+}
+/** A request that carries its agent's turn on: a continuation of it, or the
+ *  retry a provider hold keeps for it (TURN_RETRY). */
+function continuesTurn(request) {
+    return resumesLogicalTurn(request.reason) || isTurnRetry(request);
+}
 /**
  * The answers dispatchScriptToolCall's safety timer made, when no result came
  * for a script's call: the tool may still be running, or its result was lost,
@@ -678,6 +735,19 @@ function resumesLogicalTurn(reason) {
  * place in the host taken until its run ends.
  */
 const SCRIPT_CALL_NO_RESULT = new WeakSet();
+/**
+ * An MCPL peer's results whose blocks couldn't be given RFC-005's treatment
+ * for a script, because JSON couldn't hold them (dispatchMcplToolCall's
+ * reading), each with why. The failure is the script lane's alone: the
+ * model's lane reads the same answer with no such round trip. So such a
+ * result keeps the peer's own outcome, its blocks dropped, and the trace and
+ * the call's terminal report what the peer answered, as on v0.26.1, while
+ * the script lane's exit answers it as a result it can't convert to JSON
+ * (scriptResultAnswer). A module's result that JSON can't hold differs: no
+ * route can store or send it, so its dispatch makes it an error result for
+ * everyone (checkedToolResult).
+ */
+const SCRIPT_RESULT_UNCONVERTIBLE = new WeakMap();
 /** The last TurnRouting.turnId given out: ids increase, and are unique within the process. */
 let lastTurnId = 0;
 function freshTurnRouting() {
@@ -705,8 +775,10 @@ function ephemeralRunStopped(name, observed) {
 }
 /** The reason on the inference:aborted that ends a turn whose retry stop()
  *  cancelled, whichever frame was waiting to retry: beginAgentTurn's, when
- *  the stream couldn't start, or driveStream's, when it failed once started.
- *  Its inference:failed came first, and this says no retry follows it. */
+ *  the stream couldn't start, or driveStream's, when it failed once started;
+ *  or whose failure a provider hold would have kept for its retry once stop()
+ *  had closed admission (endFailureAtStop). Its inference:failed came first,
+ *  and this says no retry follows it. */
 const RETRY_CANCELLED_BY_STOP = 'a retry waiting while the framework was stopping is dropped before it sends anything';
 /** The reason on the inference:aborted that ends a turn whose start waited to
  *  retry a failed compile across a stop() and a start(): the retry would
@@ -719,6 +791,11 @@ function isTurnContinuation(reason) {
     return reason === 'context_budget_restart' || reason === 'tool_results_ready'
         || reason === 'tool_result_guard_retry';
 }
+/** A request a quiesce lets through, so a turn already alive can finish:
+ *  a continuation of the turn, or a held turn's retry (TURN_RETRY). */
+function passesQuiesce(request) {
+    return request !== undefined && (isTurnContinuation(request.reason) || isTurnRetry(request));
+}
 /** Wake sources the framework generates for itself (bounce wakes,
  *  continuations, retries, unsticks, admission requeues). A wake from any
  *  other source — a channel message, a heartbeat or other server event, a
@@ -1190,12 +1267,14 @@ function scriptResultBrief(content) {
 }
 /**
  * What a script gets for a tool result that can't be converted to JSON,
- * such as a cyclic object or a BigInt that an in-process module returned.
+ * such as a cyclic object or a BigInt that an in-process module returned,
+ * or an MCPL peer's blocks nested deeper than JSON.stringify can take.
  * It names the tool, and says the call itself completed, so a script that
  * reports it doesn't invite a blind retry. The host's log gets the same
- * line, since the tool that returned it needs fixing whether or not the
- * script reports it. The reason and the tool's own error are cut to
- * VALUE_CHARS each, so the failure reaches the script whatever its memory.
+ * line, since the tool, a module's or a server's, returns what a script
+ * can't receive, whether or not the script reports it. The reason and the
+ * tool's own error are cut to VALUE_CHARS each, so the failure reaches the
+ * script whatever its memory.
  */
 function unconvertibleScriptResult(toolName, err, ownError) {
     let reason;
@@ -1230,7 +1309,9 @@ const SCRIPT_TEXT_ANSWER_ENVELOPE_BYTES = Buffer.byteLength(JSON.stringify({ con
  * dense in small objects or a script already holding most of its memory.
  * Such a result, or one past the figure that doesn't fit, reaches the script
  * as ResultTooLarge, saying the call took effect, and nothing of it is cut or
- * saved.
+ * saved. One the host can't convert to JSON at all, such as one nested deeper
+ * than its runtime's JSON.stringify can take, is answered instead as a
+ * failure naming the tool and saying the call itself completed.
  */
 const scriptResultMaxChars = (memoryMaxBytes) => Math.max(0, Math.floor((Math.min(maxResultBytes(memoryMaxBytes), bufferConstants.MAX_STRING_LENGTH) - SCRIPT_TEXT_ANSWER_ENVELOPE_BYTES) / 6));
 /**
@@ -1974,12 +2055,14 @@ export class AgentFramework {
      * is still the agent's, so a start that ends after a later frame opened
      * one, such as its retry's stream waiting to retry, leaves that one open.
      * An operator's abandon marks the window, and the frame's check collects
-     * the mark, which goes with the window. abortInference() and the abandon
-     * count a turn with no stream only here; a stop made anywhere else, as
-     * while an agent is idle, a turn finishes its last delivery or a provider
-     * hold has parked it, has no check to end it, and the next start drops
-     * it. Keyed by the Agent, so a replacement agent under the same name
-     * never shares one.
+     * the mark, which goes with the window. A stop of a turn with no stream
+     * lands here, or, for a turn whose continuation or retry is on its way (a
+     * provider hold's, a park's, the queue's or a pass's), through
+     * abortInference(), an abandon or stop(), which drop that continuation or
+     * retry (dropContinuations); a stop made anywhere else, as while an agent
+     * is idle or a turn finishes its last delivery, has no check to end it,
+     * and the next start drops it. Keyed by the Agent, so a replacement agent
+     * under the same name never shares one.
      */
     checkAhead = new Map();
     /** Structured calls executed in the current logical turn. Unlike the
@@ -3649,6 +3732,23 @@ export class AgentFramework {
             a.coalescingSubject === b.coalescingSubject && batchSubjects(a) === batchSubjects(b) &&
             a.readsDeferredWrite === b.readsDeferredWrite;
     }
+    /** Keep `request` among a hold's `held` requests, once by
+     *  sameInferenceRequest. A held turn's retry (TURN_RETRY) whose equal is
+     *  already held marks that one, so the hold still keeps the turn alive. */
+    holdRequest(held, request) {
+        const same = held.find((r) => this.sameInferenceRequest(r, request));
+        if (!same) {
+            held.push(request);
+            return;
+        }
+        if (isTurnRetry(request))
+            same[TURN_RETRY] = true;
+        // A merged start's batch (MERGED_BATCH) stays recorded on the request
+        // kept in its place, which then stands for it, as the start did.
+        const batch = mergedBatch(request);
+        if (batch)
+            same[MERGED_BATCH] = [...new Set([...(mergedBatch(same) ?? []), ...batch])];
+    }
     consultProviderHold(error, agentName) {
         if (!this.providerHoldHook)
             return undefined;
@@ -3706,15 +3806,21 @@ export class AgentFramework {
         // failure streak), and the wait applies to the calls that follow it.
         if (!acceleration && !hostHold) {
             return wait !== undefined && isMembraneError(error) && error.retryable === true
-                && this.holdForProviderWait(agent, wait, trigger ? [trigger] : [], true);
+                && this.holdForProviderWait(agent, wait, trigger ? [asTurnRetry(trigger)] : [], true);
         }
+        // As in holdForProviderWait: once stop() has closed admission, nothing is
+        // held, and nothing started.
+        if (this.providerAdmissionClosed)
+            return true;
         const now = Date.now();
         const pacing = hostHold ? undefined : this.accelerationCooldownMs(agent.name, error);
         const delayMs = hostHold ? hostHold.holdMs : pacing.delayMs;
         const existing = this.providerAccelerationCooldowns.get(agent.name);
         const held = existing?.heldRequests ?? [];
-        if (trigger && !held.some((r) => this.sameInferenceRequest(r, trigger)))
-            held.push(trigger);
+        // The failed turn's own request, kept as its retry (TURN_RETRY). An
+        // auxiliary call's failure passes no trigger: it holds no turn of its own.
+        if (trigger)
+            this.holdRequest(held, asTurnRetry(trigger));
         if (existing)
             clearTimeout(existing.timer);
         const timer = setTimeout(() => this.releaseProviderAccelerationCooldown(agent.name), Math.min(delayMs, MAX_TIMER_DELAY_MS));
@@ -3767,7 +3873,7 @@ export class AgentFramework {
             return undefined;
         if (this.conversationAgentHomes.has(agent.name)) {
             const wait = this.recordProviderWait(agent.name, agent.model, error);
-            return error.retryable === true && wait !== undefined && this.holdForProviderWait(agent, wait, trigger ? [trigger] : [], true)
+            return error.retryable === true && wait !== undefined && this.holdForProviderWait(agent, wait, trigger ? [asTurnRetry(trigger)] : [], true)
                 ? 'held'
                 : undefined;
         }
@@ -3922,19 +4028,33 @@ export class AgentFramework {
      * Hold this agent's primary turns for a provider wait on its model. The
      * requests are kept and released together, into one fresh compile, when
      * the wait ends: the same path an acceleration cooldown takes, without
-     * parking auxiliary calls to other models.
+     * parking auxiliary calls to other models. `report`, once the wait is
+     * found to bind, runs before anything is held: a caller that reports its
+     * turn's end there (inference:failed) does so before the hold takes the
+     * turn's retry, as every failure site does.
      */
-    holdForProviderWait(agent, wait, requests, failed = false) {
+    holdForProviderWait(agent, wait, requests, failed = false, report) {
         const now = Date.now();
         const until = wait.until ?? Number.POSITIVE_INFINITY;
         if (until <= now)
             return false;
+        report?.();
+        // stop() has cleared every hold and dropped every turn on its way: a
+        // failure or start reaching here after it holds nothing, so no retry
+        // outlives the stop as a turn still alive. Nothing is started either.
+        if (this.providerAdmissionClosed)
+            return true;
+        // A listener of the report may have released the wait
+        // (releaseProviderWait) before anything was held: nothing binds now, so
+        // the requests start, as the release of an installed hold starts them.
+        if (report && !this.providerWaits?.active(agent.name, wait.model)) {
+            this.pendingRequests.push(...requests);
+            return true;
+        }
         const existing = this.providerAccelerationCooldowns.get(agent.name);
         const held = existing?.heldRequests ?? [];
-        for (const request of requests) {
-            if (!held.some((r) => this.sameInferenceRequest(r, request)))
-                held.push(request);
-        }
+        for (const request of requests)
+            this.holdRequest(held, request);
         // An acceleration or host hold already parks the whole agent; its release
         // re-checks this wait at admission.
         if (existing && existing.waitModel === undefined)
@@ -4514,9 +4634,11 @@ export class AgentFramework {
     }
     /**
      * Abort an in-flight inference for an agent. True when there was one to
-     * stop: a stream, or a turn with no stream and one of its checks ahead (its
+     * stop: a stream; a turn with no stream and one of its checks ahead (its
      * hooks or compile, a wait to retry, the tool-result guard's wait), which
-     * ends at that check, before it sends anything more.
+     * ends at that check, before it sends anything more; or a turn whose
+     * continuation or retry is on its way (queued, kept by a provider hold,
+     * parked at admission or in a scheduler pass), which ends here.
      */
     abortInference(agentName, reason) {
         const agent = this.agents.get(agentName);
@@ -4546,6 +4668,20 @@ export class AgentFramework {
             checkWindow.wake?.();
             return true;
         }
+        // A turn whose continuation or retry is on its way (a budget restart or
+        // a guard retry, or a failed turn's retry a provider hold keeps:
+        // pendingContinuationAgents) ends here, as a stop of a turn's start ends
+        // one, whether or not the frame that handed it on has let go of its
+        // token yet (that frame ends on its own and settles nothing more): the
+        // continuation or retry is dropped, with one inference:aborted carrying
+        // the stop's reason. Wakes that arrived meanwhile aren't the turn's, and
+        // still run; a hold of such wakes alone keeps no turn, and has nothing to
+        // stop. Every failure site reports inference:failed before a hold takes
+        // the retry, so a listener of it finds nothing held, and reports false.
+        if (!result && this.pendingContinuationAgents().has(agentName)) {
+            this.dropContinuations(agentName, { kind: 'caller', reason, why: 'a stop ended the turn' });
+            return true;
+        }
         return !!result;
     }
     /**
@@ -5433,9 +5569,10 @@ export class AgentFramework {
      * it queues, or a tool-result guard's retry, which its frame starts at
      * once) holds no turn token until its start takes one. Once its turn's
      * frame has let go of its token, its start can wait in the queue, a
-     * provider hold (a retry too, when a wait holds it), a park on admission
-     * or a scheduler pass's hands. Its turn is alive meanwhile. These are the
-     * agents with one on its way. A
+     * provider hold, a park on admission or a scheduler pass's hands. Its turn
+     * is alive meanwhile. So is a turn whose call failed into a provider hold,
+     * which keeps the turn's retry (TURN_RETRY), until that retry starts,
+     * wherever it waits. These are the agents with one on its way. A
      * `tool_results_ready` continuation follows a tool round left waiting with
      * no stream, which only the direct Agent API makes (Agent's
      * runInferenceWithInjections; the framework's own rounds wait with their
@@ -5444,7 +5581,7 @@ export class AgentFramework {
      */
     pendingContinuationAgents() {
         const agents = new Set();
-        const add = (r) => { if (resumesLogicalTurn(r.reason))
+        const add = (r) => { if (continuesTurn(r))
             agents.add(r.agentName); };
         // Read defensively: a framework built from the prototype, as the
         // operator tests build one, sets none of these.
@@ -5462,7 +5599,7 @@ export class AgentFramework {
         return agents;
     }
     /** The agents with a turn alive: holding a turn token, or with a
-     *  continuation of their turn on its way. */
+     *  continuation of their turn, or its retry, on its way. */
     turnHolders() {
         const holders = this.pendingContinuationAgents();
         for (const name of this.activeTurnTokens.keys())
@@ -5472,6 +5609,26 @@ export class AgentFramework {
     hasTurn(agentName) {
         return this.activeTurnTokens.has(agentName) || this.pendingContinuationAgents().has(agentName);
     }
+    /** The provider hold keeping `agentName`'s turn for its retry (TURN_RETRY),
+     *  in words naming its end and its reason, and whether a stated wait holds
+     *  it; undefined when no hold keeps one. */
+    heldTurnRetry(agentName) {
+        // Optional, as pendingContinuationAgents reads it: a framework built from
+        // its prototype, as some suites build one, has no map of holds.
+        const hold = this.providerAccelerationCooldowns?.get(agentName);
+        if (!hold?.heldRequests.some(isTurnRetry))
+            return undefined;
+        // A deadline past what a Date can name (an infinite wait, or a stated
+        // one too long for an instant) is one only a release ends.
+        const end = new Date(hold.until);
+        const until = Number.isNaN(end.getTime()) ? 'until released' : `until ${end.toISOString()}`;
+        return {
+            says: `a provider hold keeps its turn to start again, ${until}: ${hold.reason}`,
+            // release-provider-wait lifts a stated wait's hold, not a host's or an
+            // acceleration cooldown's own pacing.
+            statedWait: hold.waitModel !== undefined || hold.statedWait !== undefined,
+        };
+    }
     getHostModeStatus() {
         const parkedAdmissions = this.parkedAdmissionCount();
         const holders = this.turnHolders();
@@ -5483,7 +5640,7 @@ export class AgentFramework {
             activeTurns: holders.size,
             activeTurnAgents: [...holders].sort(),
             parkedAdmissions,
-            gatedRequests: this.pendingRequests.filter((r) => !isTurnContinuation(r.reason)).length,
+            gatedRequests: this.pendingRequests.filter((r) => !passesQuiesce(r)).length,
             backgroundScripts: this.codeExecution?.scripts.leftRunning() ?? 0,
             deferredWrites: this.deferredMessages.length,
             ...(this.lastUnabandonable.length > 0 ? { unabandonable: [...this.lastUnabandonable] } : {}),
@@ -5499,8 +5656,9 @@ export class AgentFramework {
      * Enter quiesce mode: write the deferred-write queue, persist the flag,
      * suppress gate deliveries, pause all MCPL data planes, and wait for
      * in-flight turns to settle (turn-alive is a turn token, which spans
-     * dequeue → settled teardown, or a continuation of the turn on its way to
-     * its start: turnHolders). Idempotent, but for the queue's write, which
+     * dequeue → settled teardown, a continuation of the turn on its way to its
+     * start, or the turn's retry, which a provider hold keeps or which waits
+     * to start: turnHolders). Idempotent, but for the queue's write, which
      * every call makes first: when the queue can't reach disk, because its
      * write fails or it's over the cap, the call rejects with a
      * QuiesceBlockedError and the host's mode as it was, unless it has
@@ -5510,8 +5668,8 @@ export class AgentFramework {
      * On drain timeout the host STAYS quiesced (`drained: false` in the
      * result); with `abandon: true` the undrained streams are cancelled via a
      * dedicated cancel kind that settles the turn without feeding the
-     * inference-failure accounting, and a turn whose continuation hasn't
-     * started ends with the continuation dropped. Drain covers the framework's
+     * inference-failure accounting, and a turn whose continuation or retry
+     * hasn't started ends with it dropped. Drain covers the framework's
      * own turns: a caller of the direct Agent API (`Agent.runInference`) owns
      * its turns' timing.
      */
@@ -5638,7 +5796,7 @@ export class AgentFramework {
         // to the continuation.
         const handedOff = new Set();
         for (const agentName of this.pendingContinuationAgents()) {
-            console.error(`[host-mode] abandoning ${agentName}'s turn before its continuation starts: the continuation is dropped`);
+            console.error(`[host-mode] abandoning ${agentName}'s turn before its continuation or held retry starts: it is dropped`);
             this.dropContinuations(agentName, { kind: 'quiesce_abandoned', why: 'its turn was abandoned' });
             handedOff.add(agentName);
         }
@@ -8987,7 +9145,8 @@ export class AgentFramework {
      * by stop() while it runs.
      *
      * The branch moves under every agent sharing the store, so the switch
-     * reserves the store as a surgery does: every agent must be idle, or it is
+     * reserves the store as a surgery does: every agent must be idle with no
+     * turn alive (no continuation or held retry of one on its way), or it is
      * refused (OperatorActionError 'agent-busy'), and no turn starts until the
      * switch is done.
      */
@@ -9201,18 +9360,35 @@ export class AgentFramework {
             throw new OperatorActionError('agent-busy', `Cannot ${verb}: a live ${this.surgeryHold.verb} for ${this.surgeryHold.agentName} is already in progress`);
         }
         const busy = [];
+        let heldTurns = false;
+        let statedWaits = false;
         for (const [name, a] of this.agents) {
-            if (a.state.status !== 'idle')
+            if (a.state.status !== 'idle') {
                 busy.push(`${name} is ${a.state.status}`);
-            else if (this.hasTurn(name))
+                continue;
+            }
+            if (!this.hasTurn(name))
+                continue;
+            // A provider hold keeping the turn's retry is named: a quiesce alone
+            // doesn't end that turn.
+            const held = this.heldTurnRetry(name);
+            if (held) {
+                heldTurns = true;
+                if (held.statedWait)
+                    statedWaits = true;
+                busy.push(`${name} is idle+turn-alive (${held.says})`);
+            }
+            else {
                 busy.push(`${name} is idle+turn-alive`);
+            }
         }
         for (const name of this.activeTurnTokens.keys()) {
             if (!this.agents.has(name))
                 busy.push(`${name} is turn-alive`);
         }
         if (busy.length > 0) {
-            throw new OperatorActionError('agent-busy', `Cannot ${verb} while ${busy.join(', ')} — every agent sharing the store must be idle (quiesce the host first)`);
+            throw new OperatorActionError('agent-busy', `Cannot ${verb} while ${busy.join(', ')} — every agent sharing the store must be idle (quiesce the host first)` +
+                (heldTurns ? `; ${heldTurnWayOn(statedWaits)}` : ''));
         }
         const reserved = new Map();
         for (const name of this.agents.keys()) {
@@ -9994,9 +10170,10 @@ export class AgentFramework {
             // Direct inference requests (e.g. runEphemeralToCompletion) bypass the
             // event queue — without this the loop can exit before they're drained.
             // While quiesced, gate-parked requests are NOT progress and must not
-            // block idle (only a budget restart can still run); identity-equivalent
-            // to `.length > 0` when not quiesced.
-            this.pendingRequests.some((r) => !this.quiesced || isTurnContinuation(r.reason)) ||
+            // block idle; what a quiesce lets through (passesQuiesce: a turn's
+            // continuation, or a held turn's retry) can still run, and is.
+            // Identity-equivalent to `.length > 0` when not quiesced.
+            this.pendingRequests.some((r) => !this.quiesced || passesQuiesce(r)) ||
             this.activeStreams.size > 0 ||
             Array.from(this.agents.values()).some((a) => a.state.status !== 'idle')) {
             await this.processNextEvent();
@@ -11553,9 +11730,12 @@ export class AgentFramework {
         // (request dropped as stale, inference policy veto) would otherwise stay
         // registered forever. No turn, no active stream and no pending request =
         // done. A turn holds its token from dequeue, through its hooks and
-        // compile, before its stream registers, so a fork mid-compile is busy.
+        // compile, before its stream registers, so a fork mid-compile is busy;
+        // and a turn is alive while its continuation or a provider hold's retry
+        // of it is on its way (hasTurn), so a closure turn held for its retry
+        // finishes when the hold ends.
         for (const agentName of [...this.closingConversationAgents]) {
-            if (!this.activeTurnTokens.has(agentName) &&
+            if (!this.hasTurn(agentName) &&
                 !this.activeStreams.has(agentName) &&
                 !this.pendingRequests.some((r) => r.agentName === agentName)) {
                 this.disposeConversationAgent(agentName);
@@ -12342,6 +12522,24 @@ export class AgentFramework {
         for (const cooldown of this.providerAccelerationCooldowns.values()) {
             cooldown.heldRequests = cooldown.heldRequests.filter((r) => r.coalescingSubject !== subject);
         }
+        // A wake of the subject's merged into a start that carries a turn on
+        // (MERGED_BATCH) is withdrawn from the start's batch, so no stop puts
+        // it back and no pass unpacks it.
+        const unrecord = (r) => {
+            const batch = mergedBatch(r);
+            if (batch?.some((w) => w.coalescingSubject === subject)) {
+                r[MERGED_BATCH] = batch.filter((w) => w.coalescingSubject !== subject);
+            }
+        };
+        for (const r of this.pendingRequests)
+            unrecord(r);
+        for (const cooldown of this.providerAccelerationCooldowns.values())
+            cooldown.heldRequests.forEach(unrecord);
+        for (const r of this.parkedContinuations ?? [])
+            unrecord(r);
+        for (const pass of this.passRequests ?? [])
+            for (const requests of pass.values())
+                requests.forEach(unrecord);
     }
     /**
      * A placement in the deferred queue is upgraded in place once the flush has
@@ -12720,7 +12918,7 @@ export class AgentFramework {
                 });
                 console.error(`[inference-dropped] agent=${agentName} reason=agent_not_found requests=${requests.length}`);
                 // A continuation among them leaves its turn to end here.
-                if (requests.some((r) => resumesLogicalTurn(r.reason))) {
+                if (requests.some(continuesTurn)) {
                     this.endContinuedTurn(agentName, { kind: 'dropped', why: 'its agent is gone' });
                 }
                 continue;
@@ -12731,11 +12929,8 @@ export class AgentFramework {
                 // selected now, the turn proceeds (the wait stays recorded).
                 const otherModel = providerCooldown.waitModel !== undefined && providerCooldown.waitModel !== agent.model;
                 if (!otherModel && (now < providerCooldown.until || this.extendHostProviderHold(agentName, providerCooldown))) {
-                    for (const req of requests) {
-                        if (!providerCooldown.heldRequests.some((r) => this.sameInferenceRequest(r, req))) {
-                            providerCooldown.heldRequests.push(req);
-                        }
-                    }
+                    for (const req of requests)
+                        this.holdRequest(providerCooldown.heldRequests, req);
                     const recovery = this.providerAccelerationRecoveries.get(agentName);
                     if (recovery)
                         recovery.heldRequests = providerCooldown.heldRequests.length;
@@ -12751,6 +12946,16 @@ export class AgentFramework {
                     recovery.releasedAt = now;
                 requests = [...providerCooldown.heldRequests, ...requests];
             }
+            // A start that carries a turn on and came back before it took its
+            // token stands for the batch it was merged from (MERGED_BATCH),
+            // including one a hold released just now. The batch takes its place,
+            // each request as it was, judged one by one from here: a wait holds
+            // them apart, a quiesce parks the wakes while the turn's request
+            // passes, and the read check and the policy decide on them all; the
+            // merge below records the batch anew.
+            if (requests.some((r) => mergedBatch(r) !== undefined)) {
+                requests = requests.flatMap((r) => mergedBatch(r) ?? [r]);
+            }
             // A provider's stated wait on this agent's primary model, including one
             // recorded before a restart, holds the turn until it passes. An
             // ephemeral run records nothing, its name being single-use, so no wait,
@@ -12784,9 +12989,10 @@ export class AgentFramework {
             // bounds a multi-day maintenance window's accumulation.
             if (this.quiesced) {
                 // Continuations of a held turn (budget restart, non-streaming tool
-                // results) pass; everything else parks.
-                const passThrough = requests.filter((r) => isTurnContinuation(r.reason));
-                const parked = requests.filter((r) => !isTurnContinuation(r.reason));
+                // results) pass, and so does a held turn's retry (passesQuiesce), so
+                // a turn alive at the quiesce can finish; everything else parks.
+                const passThrough = requests.filter((r) => passesQuiesce(r));
+                const parked = requests.filter((r) => !passesQuiesce(r));
                 // Coalesce per (reason, addressed): every channel wake shares one
                 // reason, so keying on reason alone would let an ambient message
                 // parked later evict a DM/mention parked earlier — and the
@@ -12888,7 +13094,7 @@ export class AgentFramework {
                 // A continuation inherits its predecessor's EventGate liveness and
                 // logical turn. If policy drops it, no driveStream remains to end them,
                 // so its turn ends here (endContinuedTurn).
-                if (requests.some((r) => resumesLogicalTurn(r.reason))) {
+                if (requests.some(continuesTurn)) {
                     this.endContinuedTurn(agentName, { kind: 'dropped', why: 'the inference policy dropped its continuation' });
                 }
                 this.endPendingPrimary(agentName);
@@ -12970,7 +13176,7 @@ export class AgentFramework {
             // exists only once assembly renders it. Recorded for the whole batch
             // (not inherited from requests[0]), so a requeued trigger stays honest.
             const batchOnly = !budgetRestart && requests.every((r) => r.coalescingBatch === true && !!r.coalescingSubject);
-            await this.startAgentStream(agent, {
+            const merged = {
                 ...trigger,
                 coalescingBatch: batchOnly || undefined,
                 coalescingBatchSubjects: batchOnly
@@ -12998,7 +13204,19 @@ export class AgentFramework {
                 counterparty: budgetRestart ? undefined : provReq?.counterparty,
                 wakeChannelId: budgetRestart ? undefined : provReq?.wakeChannelId,
                 wakeAt: budgetRestart ? undefined : provReq?.wakeAt,
-            });
+            };
+            // A held turn's retry merged with other wakes leaves the start its
+            // turn's retry (TURN_RETRY), whichever request leads the merge: held
+            // again before it takes a token, it keeps the turn alive as before.
+            const start = requests.some(isTurnRetry) ? asTurnRetry(merged) : merged;
+            // A start that carries a turn on and took in wakes that don't records
+            // the batch it was merged from (MERGED_BATCH): until it takes its
+            // token it stands for them all.
+            if (continuesTurn(start) && requests.some((r) => !continuesTurn(r)))
+                start[MERGED_BATCH] = [...requests];
+            else
+                delete start[MERGED_BATCH];
+            await this.startAgentStream(agent, start);
         }
     }
     /**
@@ -15243,7 +15461,7 @@ export class AgentFramework {
                 // Do not await inside processInferenceRequests: one slow auxiliary call for
                 // this resident must not block events or other residents. Admission stays
                 // owned and the continuation receives it after the in-flight call settles.
-                const parkedContinuation = trigger !== undefined && resumesLogicalTurn(trigger.reason);
+                const parkedContinuation = trigger !== undefined && continuesTurn(trigger);
                 if (parkedContinuation)
                     this.parkedContinuations.add(trigger);
                 void this.waitForAuxiliaryIdle(agent.name).then(async () => {
@@ -15277,8 +15495,9 @@ export class AgentFramework {
                     // park; starting here would bypass it and run a turn against a
                     // context the operator believes frozen. Give admission back and
                     // requeue — the scheduler re-parks it (coalesced) until resume.
-                    // Continuations of a held turn pass, as they do at the gate.
-                    if (this.quiesced && !isTurnContinuation(trigger?.reason ?? '')) {
+                    // Continuations of a held turn, and a held turn's retry, pass, as
+                    // they do at the gate (passesQuiesce).
+                    if (this.quiesced && !passesQuiesce(trigger)) {
                         this.releasePrimaryProviderGate(agent.name);
                         this.pendingRequests.push(trigger ?? {
                             agentName: agent.name, reason: 'provider-admission:requeue', source: 'scheduler', timestamp: Date.now(),
@@ -15286,6 +15505,16 @@ export class AgentFramework {
                         console.error(`[provider-admission] ${agent.name}: host quiesced while parked — wake requeued, not started`);
                         return;
                     }
+                    // A start that stands for a merged batch (MERGED_BATCH) goes back
+                    // to the scheduler as that batch: its gate lets the turn's request
+                    // through alone and parks the wakes until resume.
+                    const batch = trigger && mergedBatch(trigger);
+                    if (this.quiesced && batch) {
+                        this.releasePrimaryProviderGate(agent.name);
+                        this.pendingRequests.push(...batch);
+                        console.error(`[provider-admission] ${agent.name}: host quiesced while parked — the merged start goes back as its ${batch.length} requests`);
+                        return;
+                    }
                     await this.startAgentStream(agent, trigger, attempt, true);
                 }).catch((error) => {
                     this.releasePrimaryProviderGate(agent.name);
@@ -15424,12 +15653,25 @@ export class AgentFramework {
      * caller has ended the turn already.
      */
     dropContinuations(agentName, drop) {
-        const isContinuation = (r) => r.agentName === agentName && resumesLogicalTurn(r.reason);
+        const isContinuation = (r) => r.agentName === agentName && continuesTurn(r);
+        // A caller's stop and an abandon end the turn, not the wakes merged into
+        // a start of it that hasn't taken its token (MERGED_BATCH): they go back
+        // in the queue. A stop() or a disposal ends the agent's work, and puts
+        // nothing back.
+        const putsBack = drop?.kind === 'caller' || drop?.kind === 'quiesce_abandoned';
+        const back = [];
         let dropped = false;
         const take = (requests) => {
-            const left = requests.filter((r) => !isContinuation(r));
-            if (left.length < requests.length)
-                dropped = true;
+            const left = [];
+            for (const r of requests) {
+                if (!isContinuation(r))
+                    left.push(r);
+                else {
+                    dropped = true;
+                    if (putsBack)
+                        back.push(...mergedWakes(r));
+                }
+            }
             return left;
         };
         this.pendingRequests = take(this.pendingRequests);
@@ -15446,6 +15688,8 @@ export class AgentFramework {
             this.parkedContinuations.delete(parked);
             this.releasePrimaryProviderGate(agentName);
             dropped = true;
+            if (putsBack)
+                back.push(...mergedWakes(parked));
         }
         for (const pass of this.passRequests) {
             const requests = pass.get(agentName);
@@ -15455,6 +15699,7 @@ export class AgentFramework {
             requests.length = 0;
             requests.push(...left);
         }
+        this.pendingRequests.push(...back);
         if (!dropped || !drop)
             return;
         // A primary a provider hold left pending for what it released won't run
@@ -15481,11 +15726,11 @@ export class AgentFramework {
         const startedAt = this.lastInferenceAt.get(agentName)?.startedAt ?? Date.now();
         if (agent && drop.kind !== 'dropped') {
             const routing = this.openLogical.get(agentName) ?? this.routingOf(agentName);
-            this.endStoppedStart(agent, { kind: drop.kind }, routing, startedAt, undefined, false);
+            this.endStoppedStart(agent, drop.kind === 'caller' ? { kind: 'caller', reason: drop.reason } : { kind: drop.kind }, routing, startedAt, undefined, false);
         }
         else {
             const owner = this.activeTurnTokens.get(agentName);
-            this.abortDroppedTurn(agentName, owner ?? -1, drop.kind === 'dropped' ? CONTINUATION_DROPPED : drop.kind, startedAt);
+            this.abortDroppedTurn(agentName, owner ?? -1, drop.kind === 'dropped' ? CONTINUATION_DROPPED : drop.kind === 'caller' ? (drop.reason ?? 'user') : drop.kind, startedAt);
             this.channelRegistry?.stopTyping();
             if (agent) {
                 agent.toolResultGuard.recovering = false;
@@ -15508,6 +15753,34 @@ export class AgentFramework {
             this.ackDeferredWrites();
         }
     }
+    /**
+     * After a failure a provider hold would take: once stop() has closed
+     * admission, the hold installs nothing (holdForProviderWait,
+     * holdProviderAcceleration), so no retry follows the turn's
+     * inference:failed, and the turn ends as a retry a stop cancelled ends, with
+     * an inference:aborted saying so. Nothing is written. True when it ended
+     * the turn.
+     */
+    endFailureAtStop(agentName, turnToken, startedAt) {
+        if (!this.providerAdmissionClosed)
+            return false;
+        this.abortDroppedTurn(agentName, turnToken, RETRY_CANCELLED_BY_STOP, startedAt);
+        return true;
+    }
+    /**
+     * A turn's start whose failure a provider hold took (or, once stop() has
+     * closed admission, would have: endFailureAtStop) ends here. Its window
+     * closes now, not at the start's finally a few microtasks on: no check of
+     * this start is ahead, and a stop made after the failure's report must
+     * find the held retry (abortInference's drop), not a window nothing will
+     * collect.
+     */
+    endHeldStart(agent, checkWindow, turnToken, startedAt) {
+        if (this.checkAhead.get(agent) === checkWindow)
+            this.checkAhead.delete(agent);
+        this.endFailureAtStop(agent.name, turnToken, startedAt);
+        this.eventGate?.onInferenceEnded(agent.name);
+    }
     /**
      * End a turn that beginAgentTurn drops before it sends anything. The turn
      * emitted inference:started, and listeners that bracket a turn, such as a
@@ -15726,6 +15999,12 @@ export class AgentFramework {
             const { coalescingSubject: _subject, coalescingBatch: _batch, coalescingBatchSubjects: _subjects, readsDeferredWrite: _read, ...retry } = trigger;
             trigger = retry;
         }
+        // The wakes merged into this start are the turn's now: it reads them,
+        // so a later drop of its retry puts none of them back (MERGED_BATCH).
+        if (trigger && mergedBatch(trigger)) {
+            const { [MERGED_BATCH]: _batch, ...own } = trigger;
+            trigger = own;
+        }
         // Record turn checkpoint before inference (only on first attempt, not retries)
         if (attempt === 0 && trigger?.reason !== 'tool_result_guard_retry') {
             this.recordTurnCheckpoint(agent.name);
@@ -16011,9 +16290,8 @@ export class AgentFramework {
             // hold the turn as the boundary would rather than send it into the wait.
             if (this.conversationAgentHomes.has(agent.name)) {
                 const wait = this.providerWaits?.active(agent.name, agent.model);
-                if (wait && this.holdForProviderWait(agent, wait, trigger ? [trigger] : [])) {
-                    console.error(`[provider-wait] agent=${agent.name} model=${agent.model} binds after the compile ` +
-                        `(recorded while the turn compiled); turn held, nothing sent`);
+                // The turn has begun (inference:started), so the hold keeps its retry.
+                if (wait && this.holdForProviderWait(agent, wait, trigger ? [asTurnRetry(trigger)] : [], false, () => {
                     // The stream the compile made won't be sent: it's cancelled, and
                     // this start's window closes, before the hold is reported, so a stop
                     // made while inference:failed is delivered finds neither, and the
@@ -16025,8 +16303,15 @@ export class AgentFramework {
                         this.checkAhead.delete(agent);
                     // The turn emitted inference:started before its compile; every turn
                     // that does gets one terminal event, which listeners (a typing loop)
-                    // rely on, as the failure sites' holds emit inference:failed.
+                    // rely on. It's reported before the hold takes the turn's retry, as
+                    // at every failure site, so a listener's stop finds nothing held.
                     this.emitTrace({ type: 'inference:failed', agentName: agent.name, error: providerWaitText(agent.name, agent.model, wait) });
+                })) {
+                    console.error(`[provider-wait] agent=${agent.name} model=${agent.model} binds after the compile ` +
+                        `(recorded while the turn compiled); ${this.providerAdmissionClosed ? 'the framework is stopping, so nothing is held'
+                            : this.providerAccelerationCooldowns.get(agent.name)?.heldRequests.some(isTurnRetry) ? 'turn held'
+                                : 'the wait was released as the failure was reported, so the retry is queued'}, nothing sent`);
+                    this.endFailureAtStop(agent.name, turnToken, turnStartedAt);
                     this.activeStreams.delete(agent.name);
                     this.pendingAssistantBlocks.delete(agent.name);
                     this.channelRegistry?.stopTyping();
@@ -16101,14 +16386,16 @@ export class AgentFramework {
             if (ownsProviderGate && this.holdProviderAcceleration(agent, err, trigger)) {
                 // Capacity scheduling is not poisoned history. The cooldown timer owns
                 // one later fresh compile; no immediate retry or exhaustion marker.
-                this.eventGate?.onInferenceEnded(agent.name);
+                // (Once stop() has closed admission, nothing is held, and the turn
+                // ends here: endFailureAtStop.)
+                this.endHeldStart(agent, checkWindow, turnToken, turnStartedAt);
                 return false;
             }
             const statedWait = ownsProviderGate ? undefined : this.nonOwningProviderWait(agent, err, trigger);
             if (statedWait === 'held') {
                 // A conversation agent held by the provider's stated wait, as a
                 // resident is: the hold's release owns the later compile.
-                this.eventGate?.onInferenceEnded(agent.name);
+                this.endHeldStart(agent, checkWindow, turnToken, turnStartedAt);
                 return false;
             }
             const action = statedWait === 'fail' ? { retry: false } : this.errorPolicy.onInferenceError(err, agent.name, attempt);
@@ -16141,7 +16428,16 @@ export class AgentFramework {
                 }
                 // The retry re-enters startAgentStream, which replaces this frame's
                 // turn token with its own; cleanup belongs to the innermost frame.
-                await this.startAgentStream(agent, trigger, attempt + 1);
+                // The retry is its turn's (TURN_RETRY): this frame has begun the turn,
+                // and once the retry's start is held at the last admission boundary
+                // (a conversation agent's own compression can record a wait during
+                // the policy's delay), this frame lets go of its token. No check of
+                // this start is ahead now, so its window closes first: a stop made
+                // once the retry is held finds the held retry (abortInference's
+                // drop), not a window nothing will collect.
+                if (this.checkAhead.get(agent) === checkWindow)
+                    this.checkAhead.delete(agent);
+                await this.startAgentStream(agent, trigger && asTurnRetry(trigger), attempt + 1);
             }
             else {
                 // Terminal failure with no driveStream: this frame owns teardown.
@@ -17625,13 +17921,17 @@ export class AgentFramework {
                         if (agent.streamId !== myStreamId)
                             break;
                         this.abandonAgentScripts(agent.name, 'stream error', streamRouting);
+                        // A hold takes the turn's retry, or, once stop() has closed
+                        // admission, the turn ends here (endFailureAtStop).
                         if (ownsProviderGate && this.holdProviderAcceleration(agent, err, trigger)) {
+                            this.endFailureAtStop(agent.name, myTurnToken ?? -1, startTime);
                             if (this.agents.get(agent.name) === agent && agent.streamId === myStreamId)
                                 this.eventGate?.onInferenceEnded(agent.name);
                             break;
                         }
                         const statedWait = ownsProviderGate ? undefined : this.nonOwningProviderWait(agent, err, trigger);
                         if (statedWait === 'held') {
+                            this.endFailureAtStop(agent.name, myTurnToken ?? -1, startTime);
                             if (this.agents.get(agent.name) === agent && agent.streamId === myStreamId)
                                 this.eventGate?.onInferenceEnded(agent.name);
                             break;
@@ -17670,7 +17970,8 @@ export class AgentFramework {
                                     }
                                     return;
                                 }
-                                await this.startAgentStream(agent, trigger, attempt + 1);
+                                // Marked as its turn's retry, as at the start's failure.
+                                await this.startAgentStream(agent, trigger && asTurnRetry(trigger), attempt + 1);
                             }
                             else {
                                 // The turn ends here, and its inference:failed doesn't say so,
@@ -17980,10 +18281,16 @@ export class AgentFramework {
             const durationMs = Date.now() - startTime;
             // The stream won't go on: the agent stops holding it before the
             // failure is reported, as at the stream's error event, so a stop made
-            // while inference:failed is delivered finds no stream.
+            // while inference:failed is delivered finds no stream. It's reported
+            // once, before a provider hold takes the turn's retry, as at the
+            // stream's error event and at a start's failure: a listener's stop
+            // finds nothing held, and one made after it ends the held turn.
             agent.reset();
+            this.emitTrace({ type: 'inference:failed', agentName: agent.name, error: err.message, stack: err.stack });
             if (ownsProviderGate && this.holdProviderAcceleration(agent, err, trigger)) {
-                this.emitTrace({ type: 'inference:failed', agentName: agent.name, error: err.message, stack: err.stack });
+                // Once stop() has closed admission, nothing is held, and the turn
+                // ends here (endFailureAtStop).
+                this.endFailureAtStop(agent.name, myTurnToken ?? -1, startTime);
                 logStream({ success: false,
                     error: `Provider acceleration cooldown: ${err.message}`,
                     request: compiledRequest ?? { note: 'streaming request rate-limited' }, durationMs });
@@ -18000,7 +18307,7 @@ export class AgentFramework {
             // says why it ends.
             const statedWait = ownsProviderGate ? undefined : this.nonOwningProviderWait(agent, err, trigger);
             if (statedWait === 'held') {
-                this.emitTrace({ type: 'inference:failed', agentName: agent.name, error: err.message, stack: err.stack });
+                this.endFailureAtStop(agent.name, myTurnToken ?? -1, startTime);
                 logStream({ success: false,
                     error: `Provider wait: ${err.message}`,
                     request: compiledRequest ?? { note: 'streaming request rate-limited' }, durationMs });
@@ -18011,12 +18318,6 @@ export class AgentFramework {
                     this.eventGate?.onInferenceEnded(agent.name);
                 return;
             }
-            this.emitTrace({
-                type: 'inference:failed',
-                agentName: agent.name,
-                error: err.message,
-                stack: err.stack,
-            });
             this.settleAgent(agent.name, {
                 stopReason: 'exhausted',
                 speech: '',
@@ -18224,8 +18525,11 @@ export class AgentFramework {
             // frame that ends the turn disposes it, after the flush above, so what
             // was held for the turn's end lands in the fork's store instead of
             // being dropped as addressed to no agent. A frame that hands the turn
-            // to a successor (a guard or budget restart) leaves the fork to it.
-            if (frameReachedTerminal && ownsPhysicalStream && this.closingConversationAgents.has(agent.name)) {
+            // to a successor (a guard or budget restart) leaves the fork to it, and
+            // so does one whose failure a provider hold keeps for its retry: the
+            // turn isn't over (pendingContinuationAgents).
+            if (frameReachedTerminal && ownsPhysicalStream && this.closingConversationAgents.has(agent.name)
+                && !this.pendingContinuationAgents().has(agent.name)) {
                 this.disposeConversationAgent(agent.name);
             }
         }
@@ -18307,8 +18611,14 @@ export class AgentFramework {
         }
         if (agent.state.status !== 'idle' || this.hasTurn(agentName)) {
             const shown = agent.state.status === 'idle' ? 'idle+turn-alive' : agent.state.status;
-            throw new Error(`puppet refused: agent ${agentName} is ${shown} (requires idle — ` +
-                `injecting a turn under an active stream corrupts wire ordering)`);
+            // A turn a provider hold keeps for its retry has no stream: the refusal
+            // names the hold, and how to go on sooner, as surgery's does.
+            const held = agent.state.status === 'idle' ? this.heldTurnRetry(agentName) : undefined;
+            throw new Error(held
+                ? `puppet refused: agent ${agentName} is idle+turn-alive (${held.says}; requires idle); ` +
+                    heldTurnWayOn(held.statedWait)
+                : `puppet refused: agent ${agentName} is ${shown} (requires idle — ` +
+                    `injecting a turn under an active stream corrupts wire ordering)`);
         }
         const onSurface = this.getToolsForAgent(agentName)
             .some((t) => t.name === toolName && agent.canUseTool(t.name));
@@ -19233,7 +19543,9 @@ export class AgentFramework {
      * content where it sent one, `false`, `0` and `null` included, else null.
      * A result that can't be converted to JSON (a cyclic object, a BigInt),
      * which the dispatch's own check makes unlikely here, is a failure naming
-     * the tool and saying the call itself completed.
+     * the tool and saying the call itself completed. So is an MCPL peer's
+     * result whose blocks were too deep for the dispatch's treatment, which
+     * the dispatch marks (SCRIPT_RESULT_UNCONVERTIBLE).
      */
     async scriptResultAnswer(result, toolName, toServer) {
         const structured = Object.prototype.hasOwnProperty.call(result, 'structured');
@@ -19277,6 +19589,10 @@ export class AgentFramework {
                 brief: error,
             };
         };
+        // An MCPL peer's result whose blocks JSON couldn't hold, so that RFC-005's
+        // treatment couldn't be given them at dispatch.
+        if (SCRIPT_RESULT_UNCONVERTIBLE.has(result))
+            return failed(SCRIPT_RESULT_UNCONVERTIBLE.get(result));
         let content;
         let text;
         // A structured result's blocks, as #538 gives them: none when its content
@@ -24507,8 +24823,8 @@ export class AgentFramework {
         });
     }
     /** An MCP tool call's trace: a result the server marked an error (or one
-     *  the reading couldn't complete) is tool:failed, as on the other tool
-     *  paths; anything else is tool:completed. */
+     *  the reading takes as one, such as malformed content) is tool:failed, as
+     *  on the other tool paths; anything else is tool:completed. */
     traceMcpToolOutcome(serverId, tool, callId, result, durationMs) {
         if (result.isError) {
             this.emitTrace({ type: 'tool:failed', module: `mcpl:${serverId}`, tool, callId, durationMs, error: result.error ?? 'tool returned an error' });
@@ -24917,17 +25233,44 @@ export class AgentFramework {
         // generated JSON) stands in for what the server sent. An MCPL peer's
         // content blocks are the exception: what a script prints comes back to
         // the model, so each gets the treatment RFC-005 gives it on every lane
-        // the model reads, here, where the server that sent them is known. A
-        // module's blocks get it at the script lane's exit (scriptResultAnswer).
-        // A standard peer's blocks reach the script as sent.
+        // the model reads, here, where the server that sent them is known, and
+        // so does what RFC-005 governs at any depth within them, by the same
+        // peer reading, bound to that server (shownToScriptFromServer). A
+        // module's blocks get it at
+        // the script lane's exit (scriptResultAnswer). A standard peer's blocks
+        // reach the script as sent.
         const forScript = this.scriptToolWaiters?.has(call.id) ?? false;
         const reading = async (result, mcpl) => {
             if (!forScript || !Object.prototype.hasOwnProperty.call(result, 'structuredContent')) {
                 return this.mcpToolResult(result, call.id, mcpl, serverId, call.name);
             }
             const direct = AgentFramework.directToolResult(result, serverId, call.name);
-            if (mcpl)
-                return Array.isArray(direct.data) ? { ...direct, data: direct.data.map((block) => referenceBlockForScript(block, serverId)) } : direct;
+            if (mcpl) {
+                if (!Array.isArray(direct.data))
+                    return direct;
+                // Each block by the classifier, then what's nested in each, at any
+                // depth. The blocks came from the wire as JSON, so the round trip
+                // changes nothing else. But a value that parsed can still fail to
+                // go back: past some depth JSON.stringify overflows the stack where
+                // JSON.parse didn't (on Node 24.19 an array 5,000 deep does, on Bun
+                // 1.4.4 one 200,000 deep). That fails the script lane's treatment
+                // alone, and the peer answered: the result keeps the peer's outcome
+                // for the trace and the call's terminal, and the script lane's exit
+                // answers it as a result it can't convert, not as a failed dispatch
+                // (SCRIPT_RESULT_UNCONVERTIBLE). Its blocks are dropped, since they
+                // are untreated and the result travels on, to the waiter and to the
+                // listeners of its event's trace. Its structured content is kept, as
+                // every result keeps it, since RFC-005's treatment doesn't reach it.
+                const blocks = direct.data.map((block) => referenceBlockForScript(block, serverId));
+                try {
+                    return { ...direct, data: JSON.parse(JSON.stringify(blocks, (_key, value) => shownToScriptFromServer(value, serverId))) };
+                }
+                catch (err) {
+                    const unconvertible = { ...direct, data: [] };
+                    SCRIPT_RESULT_UNCONVERTIBLE.set(unconvertible, err);
+                    return unconvertible;
+                }
+            }
             // A standard peer's content that isn't a list of blocks is a malformed
             // result on the model's reading (normalizeStandardToolResult), so the
             // script reads it that way too, its structured content kept.
@@ -25125,7 +25468,8 @@ export class AgentFramework {
      * direct path (a module's callTool, a channel's tool call) doesn't come
      * here: it returns the server's result itself (directToolResult), for
      * either engine, as does a script's call whose result is structured, with
-     * an MCPL peer's blocks given RFC-005's treatment (referenceBlockForScript).
+     * an MCPL peer's blocks given RFC-005's treatment (referenceBlockForScript),
+     * and what's nested in them at any depth (shownToScriptFromServer).
      */
     async mcpToolResult(result, callId, mcplPeer, serverId, callName) {
         const hasStructured = Object.prototype.hasOwnProperty.call(result, 'structuredContent');