@lucascouts/claude-agent-acp-plus 0.3.0 → 0.4.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
package/dist/acp-agent.js CHANGED
@@ -63,6 +63,47 @@ const QUERY_RECREATE_INIT_TIMEOUT_MS = 30_000;
63
63
  * completed/stalled without the host turn resolving (issue #825). */
64
64
  const TURN_NO_RESULT_MESSAGE = "The turn ended without a result: the agent went idle while this prompt was still in flight " +
65
65
  "(e.g. the model stream dropped mid-turn). Any partial output may be incomplete; please retry.";
66
+ /** Result-message origin kinds that mark an AUTONOMOUS cycle — work the
67
+ * model did on its own (a task-notification followup, a peer/coordinator/
68
+ * observer message it handled) rather than the user's prompt. Absent,
69
+ * `human`, and `channel` origins are the user's own turn (this adapter's
70
+ * prompts arrive as the ACP channel on some CLI configurations — ALL
71
+ * channel servers are treated as user, so a foreign channel integration's
72
+ * autonomously-handled result is misclassified as the user's; accepted,
73
+ * see below), and `auto-continuation` continues the user's turn, so its
74
+ * result is the turn's real terminal.
75
+ *
76
+ * Deliberately fail-OPEN: an unknown future kind defaults to the user
77
+ * lane. Misrouting a USER result into the autonomous lane hangs the
78
+ * prompt un-detectably (the result is skipped, its trailing idle absorbed
79
+ * as owed, so the #825 detector can't fire); misrouting an autonomous
80
+ * result into the user lane is the bounded misattribution class this set
81
+ * exists to reduce. */
82
+ const AUTONOMOUS_RESULT_ORIGINS = new Set([
83
+ "task-notification",
84
+ "peer",
85
+ "coordinator",
86
+ "observer",
87
+ "observer-activity",
88
+ ]);
89
+ /** Whether this turn's terminal result arrived but its settlement is being
90
+ * held for background subagents it spawned (see Turn.deferredSettle). The
91
+ * single spelling of the hold predicate, shared by the consumer's settle
92
+ * lanes and cancel(). */
93
+ function isHeldOpen(turn) {
94
+ return turn != null && turn.deferredSettle !== undefined && !turn.settled;
95
+ }
96
+ /** Disarm the force-cancel backstop (see Session.forceCancelTimer). Every
97
+ * path that settles the active turn must run this so a timer can never fire
98
+ * on an already-settled turn — and must leave the field undefined, or the
99
+ * arm site's !forceCancelTimer guard would refuse to arm the backstop for
100
+ * the NEXT turn's cancel. */
101
+ function disarmForceCancel(session) {
102
+ if (session.forceCancelTimer) {
103
+ clearTimeout(session.forceCancelTimer);
104
+ session.forceCancelTimer = undefined;
105
+ }
106
+ }
66
107
  /** Compute a stable fingerprint of the session-defining params so we can
67
108
  * detect when a loadSession/resumeSession call requires tearing down and
68
109
  * recreating the underlying Query process. MCP servers are sorted by name
@@ -71,6 +112,78 @@ function computeSessionFingerprint(params) {
71
112
  const servers = [...(params.mcpServers ?? [])].sort((a, b) => a.name.localeCompare(b.name));
72
113
  return JSON.stringify({ cwd: params.cwd, mcpServers: servers });
73
114
  }
115
+ /**
116
+ * The single provider ID this agent exposes via `providers/*`. Claude Code has
117
+ * one LLM backend selected by protocol (anthropic / bedrock / vertex), so there
118
+ * is exactly one configurable provider.
119
+ */
120
+ const PROVIDER_ID = "main";
121
+ /**
122
+ * Protocols the `main` provider can be configured with. These mirror the
123
+ * env-var mappings understood by {@link createEnvForProvider}.
124
+ */
125
+ const SUPPORTED_PROTOCOLS = ["anthropic", "bedrock", "vertex"];
126
+ /**
127
+ * Advance the lexer state across the fragment appended since the last delta:
128
+ * just enough JSON awareness (string/escape, nesting depth) to spot commas
129
+ * that sit at the top level of the input object — everything before such a
130
+ * comma is a set of complete fields. Returns true once the input object's
131
+ * closing brace arrives.
132
+ */
133
+ function scanStreamedToolInput(state) {
134
+ let complete = false;
135
+ for (let index = state.scannedTo; index < state.partialJson.length; index++) {
136
+ const character = state.partialJson[index];
137
+ if (state.inString) {
138
+ if (state.escaped) {
139
+ state.escaped = false;
140
+ }
141
+ else if (character === "\\") {
142
+ state.escaped = true;
143
+ }
144
+ else if (character === '"') {
145
+ state.inString = false;
146
+ }
147
+ continue;
148
+ }
149
+ if (character === '"') {
150
+ state.inString = true;
151
+ }
152
+ else if (character === "{") {
153
+ state.objectDepth++;
154
+ }
155
+ else if (character === "}") {
156
+ state.objectDepth--;
157
+ if (state.objectDepth === 0) {
158
+ complete = true;
159
+ }
160
+ }
161
+ else if (character === "[") {
162
+ state.arrayDepth++;
163
+ }
164
+ else if (character === "]") {
165
+ state.arrayDepth--;
166
+ }
167
+ else if (character === "," && state.objectDepth === 1 && state.arrayDepth === 0) {
168
+ state.lastTopLevelComma = index;
169
+ }
170
+ }
171
+ state.scannedTo = state.partialJson.length;
172
+ return complete;
173
+ }
174
+ /** Parse the complete top-level fields before a top-level comma by closing the
175
+ * object at that boundary. */
176
+ function recoveredToolInput(prefix) {
177
+ try {
178
+ const value = JSON.parse(prefix + "}");
179
+ return value !== null && typeof value === "object" && !Array.isArray(value)
180
+ ? value
181
+ : undefined;
182
+ }
183
+ catch {
184
+ return undefined;
185
+ }
186
+ }
74
187
  export async function claudeCliPath() {
75
188
  if (process.env.CLAUDE_CODE_EXECUTABLE) {
76
189
  return process.env.CLAUDE_CODE_EXECUTABLE;
@@ -203,6 +316,36 @@ export function stripLocalCommandMetadata(content) {
203
316
  export function isLocalCommandMetadata(content) {
204
317
  return stripLocalCommandMetadata(content) === null;
205
318
  }
319
+ /**
320
+ * True for the synthetic assistant message the CLI injects into the transcript
321
+ * when a turn fails authentication (e.g. "Not logged in · Please run /login",
322
+ * "Session expired. Please run /login to sign in again."). The `/login`
323
+ * instruction is Claude Code TUI-specific and meaningless to ACP clients
324
+ * (issue #863). The live prompt loop suppresses the text and fails the turn
325
+ * with `authRequired` so the client can run its own auth flow; replay must
326
+ * skip it too — both for parity with what the client saw live and because the
327
+ * message stays in the transcript forever, so it would resurface on every
328
+ * session/load even after the user has logged back in.
329
+ *
330
+ * Takes the API message (`message.message`), which replay only knows as
331
+ * `unknown`. The persisted record's structured `error: "authentication_failed"`
332
+ * marker is stripped by `getSessionMessages`, so the synthetic model + text is
333
+ * all both paths have to match on.
334
+ */
335
+ export function isSyntheticLoginMessage(apiMessage) {
336
+ if (!apiMessage || typeof apiMessage !== "object") {
337
+ return false;
338
+ }
339
+ const { model, content } = apiMessage;
340
+ if (model !== "<synthetic>" || !Array.isArray(content) || content.length !== 1) {
341
+ return false;
342
+ }
343
+ const block = content[0];
344
+ return (!!block &&
345
+ block.type === "text" &&
346
+ typeof block.text === "string" &&
347
+ block.text.includes("Please run /login"));
348
+ }
206
349
  const PERMISSION_MODE_ALIASES = {
207
350
  auto: "auto",
208
351
  default: "default",
@@ -316,6 +459,10 @@ export class ClaudeAcpAgent {
316
459
  clientCapabilities;
317
460
  logger;
318
461
  gatewayAuthRequest;
462
+ /** Client-managed LLM routing set via `providers/set`. Process-scoped and
463
+ * never persisted to disk (see the Configurable LLM Providers RFD). When
464
+ * set, it takes precedence over {@link gatewayAuthRequest}. */
465
+ providerConfig;
319
466
  /** Grace period before a `session/cancel` forces a wedged prompt loop to
320
467
  * return "cancelled". See {@link DEFAULT_FORCE_CANCEL_GRACE_MS}. Mutable so
321
468
  * tests can shrink it. */
@@ -441,6 +588,10 @@ export class ClaudeAcpAgent {
441
588
  auth: {
442
589
  logout: {},
443
590
  },
591
+ // Client-managed LLM routing via `providers/list`, `providers/set`, and
592
+ // `providers/disable`. Advertised unconditionally; there is no client
593
+ // capability prerequisite for the provider methods.
594
+ providers: {},
444
595
  loadSession: true,
445
596
  sessionCapabilities: {
446
597
  additionalDirectories: {},
@@ -564,11 +715,94 @@ export class ClaudeAcpAgent {
564
715
  }
565
716
  throw new Error("Method not implemented.");
566
717
  }
718
+ /**
719
+ * `providers/list` — returns the single client-configurable custom gateway
720
+ * provider (`main`). `current` carries only non-secret routing (never headers,
721
+ * which may hold secrets); only `apiType`/`baseUrl` are surfaced for UI
722
+ * display, and is `null` when the provider is not configured/disabled. The
723
+ * provider is optional (`required: false`): while disabled/unconfigured the
724
+ * agent falls back to its own default routing (normal Claude login).
725
+ */
726
+ async unstable_listProviders(_params) {
727
+ const config = this.resolveProviderConfig();
728
+ const provider = {
729
+ providerId: PROVIDER_ID,
730
+ supported: SUPPORTED_PROTOCOLS,
731
+ required: false,
732
+ current: config ? { apiType: config.apiType, baseUrl: config.baseUrl } : null,
733
+ };
734
+ return { providers: [provider] };
735
+ }
736
+ /**
737
+ * `providers/set` — replace the full configuration for the `main` provider.
738
+ * Rejects unknown IDs, unsupported protocols, and empty/invalid base URLs with
739
+ * `invalid_params`. Config is process-scoped and applies to sessions created or
740
+ * loaded after this call.
741
+ */
742
+ async unstable_setProvider(params) {
743
+ if (params.providerId !== PROVIDER_ID) {
744
+ throw RequestError.invalidParams({ providerId: params.providerId }, `Unknown provider ID "${params.providerId}"; expected "${PROVIDER_ID}".`);
745
+ }
746
+ if (!SUPPORTED_PROTOCOLS.includes(params.apiType)) {
747
+ throw RequestError.invalidParams({ apiType: params.apiType, supported: SUPPORTED_PROTOCOLS }, `Unsupported apiType "${params.apiType}" for provider "${PROVIDER_ID}".`);
748
+ }
749
+ if (!isValidBaseUrl(params.baseUrl)) {
750
+ throw RequestError.invalidParams({ baseUrl: params.baseUrl }, "baseUrl must be a non-empty absolute http(s) URL.");
751
+ }
752
+ const config = {
753
+ apiType: params.apiType,
754
+ baseUrl: params.baseUrl,
755
+ headers: params.headers ?? {},
756
+ };
757
+ // Vertex requires project + region, which the standard payload cannot
758
+ // carry, so they arrive via `_meta.claudeCode.vertex`.
759
+ if (params.apiType === "vertex") {
760
+ const vertex = params._meta?.claudeCode?.vertex;
761
+ if (!vertex ||
762
+ typeof vertex.projectId !== "string" ||
763
+ vertex.projectId.trim() === "" ||
764
+ typeof vertex.region !== "string" ||
765
+ vertex.region.trim() === "") {
766
+ throw RequestError.invalidParams(undefined, "vertex apiType requires non-empty `_meta.claudeCode.vertex.projectId` and `_meta.claudeCode.vertex.region`.");
767
+ }
768
+ config.vertex = { projectId: vertex.projectId, region: vertex.region };
769
+ }
770
+ this.providerConfig = config;
771
+ return {};
772
+ }
773
+ /**
774
+ * `providers/disable` — disabling the `main` provider clears any client-managed
775
+ * routing (both a `providers/set` config and the legacy gateway auth request),
776
+ * so the agent reverts to its own default routing and `providers/list` reports
777
+ * `current: null`. Disabling any other (unknown) ID is treated as a successful
778
+ * no-op per the RFD's idempotency rule.
779
+ */
780
+ async unstable_disableProvider(params) {
781
+ if (params.providerId === PROVIDER_ID) {
782
+ this.providerConfig = undefined;
783
+ this.gatewayAuthRequest = undefined;
784
+ }
785
+ // Unknown provider: idempotent success.
786
+ return {};
787
+ }
788
+ /**
789
+ * Resolve the effective client-managed routing config. `providers/set` takes
790
+ * precedence; otherwise fall back to the legacy gateway auth request. Returns
791
+ * `null` when neither is configured.
792
+ */
793
+ resolveProviderConfig() {
794
+ if (this.providerConfig) {
795
+ return this.providerConfig;
796
+ }
797
+ return gatewayRequestToProviderConfig(this.gatewayAuthRequest);
798
+ }
567
799
  async logout(_params) {
568
- // Clear in-memory gateway credentials supplied via `authenticate`. The
569
- // gateway method never touches the on-disk credential store, so dropping
570
- // this reference is the whole logout for that path.
800
+ // Clear in-memory gateway credentials supplied via `authenticate` and any
801
+ // provider routing set via `providers/set`. Neither touches the on-disk
802
+ // credential store, so dropping these references is the whole logout for
803
+ // those paths.
571
804
  this.gatewayAuthRequest = undefined;
805
+ this.providerConfig = undefined;
572
806
  // For the Claude/Console login methods the credentials live in the native
573
807
  // CLI's store (keychain or config dir), which only the binary can clear.
574
808
  // `claude auth logout` is non-interactive and idempotent.
@@ -778,23 +1012,34 @@ export class ClaudeAcpAgent {
778
1012
  // gateways that don't carry a stable/matching id across the stream and the
779
1013
  // consolidated message. Reset after each consolidated message consumes it.
780
1014
  const streamedBlocks = [];
1015
+ // Tool-use blocks start streaming before their JSON input. Keep the
1016
+ // partial input per parent message and block index so completed top-level
1017
+ // fields can refine the pending tool call while it streams. Entries are
1018
+ // dropped at block/message boundaries; the whole map is swept when a turn
1019
+ // settles, since an interrupted subagent stream (keyed by a
1020
+ // parent_tool_use_id that never recurs) has no boundary event of its own.
1021
+ const streamedToolInputs = new Map();
781
1022
  // Stop reason accumulated for the active turn (result subtype, refusal,
782
1023
  // max_tokens, …). Reset per turn; read when the turn settles at idle.
783
1024
  let stopReason = "end_turn";
784
- // How many trailing `session_state_changed: idle` messages are already
785
- // accounted for: every user-turn result that terminates a turn (settle,
786
- // reject, or orphan skip) is followed by one, as is a cancelled turn
787
- // settled by the next turn's echo hand-off. The idle handler absorbs owed
788
- // idles; an idle that arrives when NONE is owed while the active turn is
789
- // still unsettled means the SDK ended the turn without ever emitting its
790
- // result, so the turn will never settle on its own (issue #825).
791
- // Stream-level debt, deliberately NOT reset per turn: a lagged idle can
792
- // arrive after the next turn has already activated (issue #773), and the
793
- // debt is what attributes it to the turn that owed it. Over-counting (an
794
- // idle the SDK never emits, e.g. CLI binaries without session-state
795
- // events issue #497) is benign: the counter just absorbs one future
796
- // idle, and detection degrades to the status quo rather than misfiring.
797
- let owedTrailingIdles = 0;
1025
+ /** The consumer's single send chokepoint: every `sessionUpdate` in this
1026
+ * loop goes through here (never `this.client.sessionUpdate` directly) so
1027
+ * answer-delivery tracking is a property of sending, not something each
1028
+ * emission site must remember. A top-level `agent_message_chunk` marks
1029
+ * the stretch's answer as delivered; subagent-attributed chunks are
1030
+ * recognizable by the `parentToolUseId` meta that toAcpNotifications
1031
+ * stamps from `parent_tool_use_id`, and never reach the top-level feed
1032
+ * as the turn's answer. */
1033
+ const sendUpdate = async (notification) => {
1034
+ const { update } = notification;
1035
+ if (update.sessionUpdate === "agent_message_chunk") {
1036
+ const claudeMeta = update._meta?.claudeCode;
1037
+ if (!claudeMeta?.parentToolUseId) {
1038
+ session.emittedAssistantText = true;
1039
+ }
1040
+ }
1041
+ await this.client.sessionUpdate(notification);
1042
+ };
798
1043
  const resetTurnScratch = () => {
799
1044
  lastAssistantTotalUsage = null;
800
1045
  lastAssistantUsage = null;
@@ -830,6 +1075,31 @@ export class ClaudeAcpAgent {
830
1075
  session.activeTurn = turn;
831
1076
  session.cancelled = false;
832
1077
  session.pendingOrphanResults = 0;
1078
+ session.orphanCommands?.clear();
1079
+ // Two-phase sweep of registry entries the level signal ended (see
1080
+ // the endedPerLevel field doc): armed at the first activation,
1081
+ // deleted at the second — the same activation-time self-heal as the
1082
+ // orphan lanes, and the growth bound for leaked entries whose settle
1083
+ // bookends never arrive. The one-activation grace lets a corrective
1084
+ // inclusive level rescue a live async agent that a racing payload
1085
+ // absent-marked (deletion is irreversible: levels never ADD entries).
1086
+ // Local-only commands don't advance the clock: two quick /context
1087
+ // calls would otherwise burn the whole grace in seconds of wall time
1088
+ // while the corrective level is still in flight, and they interact
1089
+ // with no tasks — a later real turn still bounds growth.
1090
+ if (!turn.isLocalOnlyCommand) {
1091
+ for (const [taskId, record] of session.liveBackgroundTasks) {
1092
+ if (!record.endedPerLevel) {
1093
+ continue;
1094
+ }
1095
+ if (record.endedPerLevel === "sweep-armed") {
1096
+ session.liveBackgroundTasks.delete(taskId);
1097
+ }
1098
+ else {
1099
+ record.endedPerLevel = "sweep-armed";
1100
+ }
1101
+ }
1102
+ }
833
1103
  resetTurnScratch();
834
1104
  };
835
1105
  /** Ensure there is an active turn before a user-turn result that carries no
@@ -849,18 +1119,165 @@ export class ClaudeAcpAgent {
849
1119
  * result), so we skip those and only promote once the count is drained. */
850
1120
  const ensureActiveTurn = () => {
851
1121
  if (session.activeTurn) {
852
- return;
853
- }
854
- const head = (session.turnQueue ?? []).find((t) => !t.settled);
855
- if (!head) {
856
- return;
1122
+ if (!isHeldOpen(session.activeTurn)) {
1123
+ return;
1124
+ }
1125
+ // A held turn (Turn.deferredSettle) already produced its result, so
1126
+ // this incoming user-turn result cannot be its — it belongs to the
1127
+ // next queued command (an echo-less one, e.g. `/context` sent while
1128
+ // the hold drains; a normal prompt's echo would have handed the held
1129
+ // turn off before its result). Settle the held turn with its
1130
+ // recorded outcome — the user moving on outranks the hold, same
1131
+ // contract as the echo hand-off — and fall through to promote the
1132
+ // queue head, which this result belongs to. Without this, the head
1133
+ // would never be promoted (echo-less turns have no other promotion
1134
+ // path) and its prompt would hang, while this result's outcome
1135
+ // overwrote the held turn's. Orphan lanes below are necessarily
1136
+ // empty while a turn is held: orphans are seeded by cancel(), which
1137
+ // inline-settles a held turn, and activation cleared older ones.
1138
+ // settleActive also closes the held turn's delivery stretch, so the
1139
+ // promoted command's own delivery decision is not judged against the
1140
+ // held turn's followup text (issue #453) — the caller snapshots the
1141
+ // flag AFTER this runs.
1142
+ settleActive(session.activeTurn.deferredSettle);
857
1143
  }
1144
+ // Orphan accounting runs BEFORE the head check: an orphan's echo-less
1145
+ // result can arrive with an EMPTY queue (the common post-cancel
1146
+ // timeline — the active turn settled at the interrupt's idle and the
1147
+ // user hasn't typed yet), and it must still be consumed here. Skipping
1148
+ // the bookkeeping when there is nothing to promote would leave a
1149
+ // phantom entry/count that swallows the next live echo-less result
1150
+ // (e.g. /compact) instead.
858
1151
  if ((session.pendingOrphanResults ?? 0) > 0) {
859
1152
  session.pendingOrphanResults--;
860
1153
  return;
861
1154
  }
1155
+ // msg_lifecycle_v1 lane. Attribute this echo-less result using the
1156
+ // entries' states — turns run sequentially and frames arrive in stream
1157
+ // order, so at any result: every "zombie" is from an already-dead turn
1158
+ // whose own result already passed before the frame that created the
1159
+ // newest entry (or never existed), every "started" entry was dispatched
1160
+ // into THE turn that emitted this result (an older turn's entries got
1161
+ // their terminal frames before a newer turn's "started" frames), and a
1162
+ // "pending" entry was not dispatched before it. One result therefore
1163
+ // covers ALL started and zombie entries at once (N coalesced commands
1164
+ // share ONE result); their outstanding terminal frames then no-op on
1165
+ // the missing entries. NOTE this ordering argument is asserted from
1166
+ // observed CLI behavior, not a documented wire contract — if a dead
1167
+ // turn's late result could lag past the NEXT turn's dispatch frames,
1168
+ // deleting a zombie and a started entry on one result would
1169
+ // double-consume it. The unexpected-transition logging in the frame
1170
+ // handler is the tripwire for that class of drift.
1171
+ if (session.orphanCommands?.size) {
1172
+ let consumedOrphanResult = false;
1173
+ let oldestPending;
1174
+ for (const [uuid, state] of session.orphanCommands) {
1175
+ if (state === "started" || state === "zombie") {
1176
+ consumedOrphanResult = true;
1177
+ session.orphanCommands.delete(uuid);
1178
+ }
1179
+ else {
1180
+ oldestPending ??= uuid;
1181
+ }
1182
+ }
1183
+ if (consumedOrphanResult) {
1184
+ return;
1185
+ }
1186
+ if (oldestPending !== undefined) {
1187
+ // No dispatch was seen before this result, so it is very likely a
1188
+ // live turn's — but a lost "started" frame would mean it IS the
1189
+ // orphan's (dup-over-loss: prefer one wrong skip over
1190
+ // misattributing a dead turn's outcome to a live prompt). Grant
1191
+ // each pending entry exactly one skip, like the count lane did.
1192
+ session.orphanCommands.delete(oldestPending);
1193
+ return;
1194
+ }
1195
+ }
1196
+ const head = firstUnsettledQueuedTurn();
1197
+ if (!head) {
1198
+ return;
1199
+ }
862
1200
  activateTurn(head);
863
1201
  };
1202
+ /** Result-time bookkeeping that must run whether or not the result can be
1203
+ * attributed to a turn. (1) Latch `commandResultSeen` on every queued
1204
+ * turn whose command is known dispatched with no terminal frame yet —
1205
+ * the emitting turn is the one it was dispatched (possibly folded) into,
1206
+ * so its result has now passed; a later cancel() must not seed an orphan
1207
+ * entry that waits for it (see Turn.commandResultSeen). (2) When a turn
1208
+ * is ACTIVE, the result is attributed to it and never reaches
1209
+ * ensureActiveTurn — but it still covers the map's started entries
1210
+ * (commands folded into the active turn share its result) and zombies
1211
+ * (their late results have already passed or never existed), so drain
1212
+ * them here or they would zombify/linger and swallow a later live
1213
+ * echo-less result. */
1214
+ const recordResultForOrphanCommands = () => {
1215
+ for (const turn of session.turnQueue ?? []) {
1216
+ if (!turn.settled && turn.commandStarted && !turn.commandFinished) {
1217
+ turn.commandResultSeen = true;
1218
+ }
1219
+ }
1220
+ if (session.activeTurn && session.orphanCommands?.size) {
1221
+ for (const [uuid, state] of session.orphanCommands) {
1222
+ if (state === "started" || state === "zombie") {
1223
+ session.orphanCommands.delete(uuid);
1224
+ }
1225
+ }
1226
+ }
1227
+ };
1228
+ /** The unsettled in-flight turn owning this prompt uuid, if any. */
1229
+ const findUnsettledTurn = (uuid) => (session.turnQueue ?? []).find((t) => t.promptUuid === uuid && !t.settled);
1230
+ /** The first queued turn still awaiting its outcome, if any — the single
1231
+ * spelling of "a prompt is pending" shared by the head promotion and
1232
+ * the autonomous stretch-close guard. */
1233
+ const firstUnsettledQueuedTurn = () => (session.turnQueue ?? []).find((t) => !t.settled);
1234
+ /** Whether any background subagent this turn spawned is still live —
1235
+ * while true, the turn's settlement stays deferred so the subagent's
1236
+ * output and permission requests land inside it (see
1237
+ * Turn.deferredSettle). */
1238
+ const turnAwaitingSubagents = (turn) => {
1239
+ if (!turn.spawnedTaskIds?.size) {
1240
+ return false;
1241
+ }
1242
+ for (const taskId of turn.spawnedTaskIds) {
1243
+ const record = session.liveBackgroundTasks.get(taskId);
1244
+ // The isSubagent read is defense in depth for the shells-never-defer
1245
+ // contract: spawnedTaskIds only ever holds subagent ids today, but a
1246
+ // future add site must not silently let a long-lived shell hold a
1247
+ // prompt open. endedPerLevel entries are kept for attribution only —
1248
+ // the level signal says the task is gone (or its bookends were
1249
+ // lost), so a hold must not wait on them.
1250
+ if (record?.isSubagent && !record.endedPerLevel) {
1251
+ return true;
1252
+ }
1253
+ }
1254
+ return false;
1255
+ };
1256
+ /** Settle the active turn's stored deferred outcome once none of its
1257
+ * spawned subagents is live. The single drain rule shared by the
1258
+ * followup-result and idle settle sites, so the two lanes can't drift. */
1259
+ const settleDeferredIfDrained = () => {
1260
+ const turn = session.activeTurn;
1261
+ if (isHeldOpen(turn) && !turnAwaitingSubagents(turn)) {
1262
+ settleActive(turn.deferredSettle);
1263
+ }
1264
+ };
1265
+ /** Settle the active turn with `outcome` now — unless subagents it
1266
+ * spawned are still live, in which case store the outcome and hold the
1267
+ * turn open (see Turn.deferredSettle). Every result-time settle of a
1268
+ * turn that can have spawned subagents must route through here: a site
1269
+ * calling settleActive directly bypasses the hold and re-opens the
1270
+ * out-of-turn permission deadlock (issue #866) through its lane. */
1271
+ const settleOrDefer = (outcome) => {
1272
+ if (session.activeTurn &&
1273
+ !session.activeTurn.settled &&
1274
+ turnAwaitingSubagents(session.activeTurn)) {
1275
+ session.activeTurn.deferredSettle = outcome;
1276
+ }
1277
+ else {
1278
+ settleActive(outcome);
1279
+ }
1280
+ };
864
1281
  /** Settle the active turn's deferred exactly once, disarm the force-cancel
865
1282
  * backstop (the turn is over), and drop it from the queue. */
866
1283
  const settleActive = (result) => {
@@ -868,22 +1285,34 @@ export class ClaudeAcpAgent {
868
1285
  if (!turn || turn.settled) {
869
1286
  return;
870
1287
  }
1288
+ // Captured before the settled flip below (isHeldOpen tests !settled).
1289
+ const wasHeld = isHeldOpen(turn);
871
1290
  turn.settled = true;
872
- if (session.forceCancelTimer) {
873
- clearTimeout(session.forceCancelTimer);
874
- session.forceCancelTimer = undefined;
875
- }
1291
+ disarmForceCancel(session);
876
1292
  session.turnQueue = (session.turnQueue ?? []).filter((t) => t !== turn);
877
1293
  session.activeTurn = null;
1294
+ streamedToolInputs.clear();
1295
+ if (wasHeld) {
1296
+ // Settling a held turn is its delivery-stretch boundary: the turn's
1297
+ // answer finished long ago, so text streamed since the last boundary
1298
+ // is normally its followups' — left latched it would suppress a
1299
+ // following replayed turn's issue-#453 result-text fallback (the
1300
+ // common post-hold sequence). Known trade: at the echo hand-off an
1301
+ // incoming turn's pre-echo deltas share this one boolean, so a
1302
+ // STREAMING replay on a usage-omitting backend could re-emit its
1303
+ // answer — the flag cannot attribute text to a turn before its
1304
+ // echo, and the suppression direction is the common one, so the
1305
+ // clear wins. Every held-settle lane inherits this: the drain
1306
+ // settle, both hand-offs, and stream-done; cancel()'s inline mirror
1307
+ // carries its own copy.
1308
+ session.emittedAssistantText = false;
1309
+ }
878
1310
  turn.resolve(result);
879
1311
  };
880
1312
  /** Reject the active turn (auth required, error result, …) without tearing
881
1313
  * down the consumer: the stream continues to idle and later turns proceed. */
882
1314
  const failActive = (error) => {
883
- if (session.forceCancelTimer) {
884
- clearTimeout(session.forceCancelTimer);
885
- session.forceCancelTimer = undefined;
886
- }
1315
+ disarmForceCancel(session);
887
1316
  const turn = session.activeTurn;
888
1317
  if (!turn || turn.settled) {
889
1318
  return;
@@ -891,14 +1320,17 @@ export class ClaudeAcpAgent {
891
1320
  turn.settled = true;
892
1321
  session.turnQueue = (session.turnQueue ?? []).filter((t) => t !== turn);
893
1322
  session.activeTurn = null;
1323
+ streamedToolInputs.clear();
1324
+ // A failed turn's stretch is over, and some failure lanes (the issue
1325
+ // #825 idle-fail) never see the result whose `finally` would close it —
1326
+ // start the next stretch clean, or its stale delivery record would
1327
+ // suppress the next turn's issue-#453 result-text fallback.
1328
+ session.emittedAssistantText = false;
894
1329
  turn.reject(error);
895
1330
  };
896
1331
  /** Reject every in-flight turn — used when the stream dies. */
897
1332
  const failAllTurns = (error) => {
898
- if (session.forceCancelTimer) {
899
- clearTimeout(session.forceCancelTimer);
900
- session.forceCancelTimer = undefined;
901
- }
1333
+ disarmForceCancel(session);
902
1334
  const turns = session.activeTurn
903
1335
  ? [session.activeTurn, ...(session.turnQueue ?? []).filter((t) => t !== session.activeTurn)]
904
1336
  : [...(session.turnQueue ?? [])];
@@ -906,8 +1338,18 @@ export class ClaudeAcpAgent {
906
1338
  session.turnQueue = [];
907
1339
  for (const turn of turns) {
908
1340
  if (!turn.settled) {
1341
+ const wasHeld = isHeldOpen(turn);
909
1342
  turn.settled = true;
910
- turn.reject(error);
1343
+ if (wasHeld) {
1344
+ // A held turn's answer already streamed and its outcome is
1345
+ // recorded — a stream death during the post-answer hold is a
1346
+ // background failure, not the turn's. Resolve with the real
1347
+ // outcome, mirroring the stream-done path.
1348
+ turn.resolve(turn.deferredSettle);
1349
+ }
1350
+ else {
1351
+ turn.reject(error);
1352
+ }
911
1353
  }
912
1354
  }
913
1355
  };
@@ -961,9 +1403,45 @@ export class ClaudeAcpAgent {
961
1403
  // turn being abandoned. Stale counts self-heal: activation resets
962
1404
  // them (see activateTurn).
963
1405
  if (session.activeTurn && !session.activeTurn.settled) {
964
- session.pendingOrphanResults = (session.pendingOrphanResults ?? 0) + 1;
1406
+ // Seed by what the frames already told us, mirroring cancel()'s
1407
+ // queued-turn sweep — the consumer may have drained the wedged
1408
+ // turn's result and/or terminal frame before the backstop fired,
1409
+ // and an entry seeded for a result or frame that is already
1410
+ // spent would never drain (it would swallow an unrelated later
1411
+ // echo-less result instead).
1412
+ const active = session.activeTurn;
1413
+ if (active.commandFinished === "completed" || active.commandFinished === "discarded") {
1414
+ // Finished SDK-side; any result already passed. Nothing to
1415
+ // track.
1416
+ }
1417
+ else if (active.commandFinished === "cancelled") {
1418
+ // Aborted after dispatch: its late result may still come —
1419
+ // unless it already did.
1420
+ if (!active.commandResultSeen) {
1421
+ this.trackOrphanCommand(session, active.promptUuid, "zombie");
1422
+ }
1423
+ }
1424
+ else if (active.commandResultSeen) {
1425
+ // Its result was already consumed (dropped at the cancelled
1426
+ // guard); only the terminal frame is outstanding, which no-ops
1427
+ // with no entry. Nothing to track.
1428
+ }
1429
+ else {
1430
+ // The wedged turn WAS dispatched (it's active), so track it
1431
+ // "started": its late result (if the SDK recovers) is skipped
1432
+ // echo-less, and its terminal frame — or that skip plus
1433
+ // activation's clear when the frame is lost to the wedge — is
1434
+ // what drains it.
1435
+ this.trackOrphanCommand(session, active.promptUuid, "started");
1436
+ }
965
1437
  }
966
1438
  settleActive({ stopReason: "cancelled", usage: sessionUsage(session) });
1439
+ // The cancelled turn's result may never come (that's why the
1440
+ // backstop fired) — close its delivery stretch here so partial
1441
+ // streamed text can't suppress the next turn's issue-#453 fallback.
1442
+ // If a late orphan result does arrive, its `finally` clears again;
1443
+ // FIFO ordering means no live turn's text can have streamed yet.
1444
+ session.emittedAssistantText = false;
967
1445
  // If the session is being torn down — or this consumer was
968
1446
  // superseded by a lazy query recreate — abandon the in-flight
969
1447
  // next() (swallowing any later rejection so it can't surface as
@@ -999,11 +1477,15 @@ export class ClaudeAcpAgent {
999
1477
  // turn's real outcome.
1000
1478
  //
1001
1479
  // Settle the turn that was in flight so its prompt() doesn't hang:
1002
- // cancelled if a cancel is pending, otherwise the accumulated outcome.
1003
- settleActive({
1004
- stopReason: session.cancelled ? "cancelled" : stopReason,
1005
- usage: sessionUsage(session),
1006
- });
1480
+ // cancelled if a cancel is pending, otherwise the outcome a
1481
+ // deferred turn already recorded (see Turn.deferredSettle) or the
1482
+ // accumulated scratch outcome. The scratch currently still equals
1483
+ // a deferred turn's stored outcome (followup results never mutate
1484
+ // it), but the stored one is the authoritative source.
1485
+ const inFlight = session.activeTurn;
1486
+ settleActive(session.cancelled
1487
+ ? { stopReason: "cancelled", usage: sessionUsage(session) }
1488
+ : (inFlight?.deferredSettle ?? { stopReason, usage: sessionUsage(session) }));
1007
1489
  // Queued turns the SDK never started never ran, so reject them rather
1008
1490
  // than reporting a success (end_turn) — or a misleading "cancelled" —
1009
1491
  // for a prompt that produced no output. (A cancel already settled the
@@ -1030,19 +1512,116 @@ export class ClaudeAcpAgent {
1030
1512
  message: message,
1031
1513
  });
1032
1514
  }
1515
+ // CLIs 2.1.206+ (capability msg_lifecycle_v1) report the fate of every
1516
+ // uuid-stamped queued command (queued/started/completed/cancelled/
1517
+ // discarded) as `command_lifecycle` frames — 2-3 per prompt, since
1518
+ // prompt() stamps a uuid on every message. The frame is @internal and
1519
+ // absent from the SDKMessage union, so handle it BEFORE the exhaustive
1520
+ // switch: it must not reach `unreachable`'s error log, and a `case`
1521
+ // for it wouldn't typecheck. It feeds only the orphan accounting (see
1522
+ // Session.orphanCommands); turn settlement stays driven by
1523
+ // echoes/results/idle. (Raw-mode emission above still forwards these
1524
+ // frames.)
1525
+ if (message.type === "command_lifecycle") {
1526
+ const frame = message;
1527
+ switch (frame.state) {
1528
+ case "started": {
1529
+ // Remember dispatch on the live turn so a cancel() that orphans
1530
+ // it seeds the right state (see Turn.commandStarted)...
1531
+ const queued = findUnsettledTurn(frame.command_uuid);
1532
+ if (queued) {
1533
+ queued.commandStarted = true;
1534
+ }
1535
+ // ...and promote an already-orphaned command: once dispatched,
1536
+ // a bare `cancelled` no longer means "dropped without running".
1537
+ const state = session.orphanCommands?.get(frame.command_uuid);
1538
+ if (state === "pending") {
1539
+ session.orphanCommands.set(frame.command_uuid, "started");
1540
+ }
1541
+ else if (state === "zombie") {
1542
+ // "started" after the command's terminal frame: the ordering
1543
+ // the whole lane rests on has been violated (frames are
1544
+ // per-uuid FIFO). Surface it — a silent drift here degrades
1545
+ // into swallowed or misattributed results.
1546
+ this.logger.error(`Session ${params.sessionId}: command_lifecycle "started" for ${frame.command_uuid} after its terminal frame; orphan accounting may be off for this cancel.`);
1547
+ }
1548
+ break;
1549
+ }
1550
+ case "completed":
1551
+ case "discarded":
1552
+ case "cancelled": {
1553
+ // Terminal frames. Latch the fate on a still-queued turn so a
1554
+ // later cancel() doesn't seed an orphan entry for a command
1555
+ // whose one-and-only terminal frame has already been consumed
1556
+ // (nothing would ever drain that entry).
1557
+ const queued = findUnsettledTurn(frame.command_uuid);
1558
+ if (queued) {
1559
+ queued.commandFinished = frame.state;
1560
+ }
1561
+ if (frame.state === "cancelled") {
1562
+ // Ambiguous by design (dup-over-loss): dropped before
1563
+ // dispatch (no result will ever come — safe to forget) vs
1564
+ // consumed into a turn that was aborted/failed. For the
1565
+ // latter, any result the dead turn managed to emit has
1566
+ // already deleted the entry (see
1567
+ // recordResultForOrphanCommands / ensureActiveTurn), so a
1568
+ // still-"started" entry means no result was seen since
1569
+ // dispatch — it becomes a zombie for the next
1570
+ // echo-less-result skip.
1571
+ const state = session.orphanCommands?.get(frame.command_uuid);
1572
+ if (state === "pending") {
1573
+ session.orphanCommands?.delete(frame.command_uuid);
1574
+ }
1575
+ else if (state === "started") {
1576
+ session.orphanCommands?.set(frame.command_uuid, "zombie");
1577
+ }
1578
+ break;
1579
+ }
1580
+ // Exactly-one-terminal: the command is finished. "completed" is
1581
+ // emitted after any result its turn produced (fresh turn) or the
1582
+ // command folded into another turn whose result is attributed
1583
+ // elsewhere — either way no echo-less result remains to skip.
1584
+ // "discarded" = session ended with it still queued; no result.
1585
+ session.orphanCommands?.delete(frame.command_uuid);
1586
+ break;
1587
+ }
1588
+ default:
1589
+ // "queued" carries no fate information. Anything else is a
1590
+ // state this adapter doesn't know — likely a CLI that grew the
1591
+ // v1 vocabulary. The entry still drains by result coverage or
1592
+ // activation's clear (bounded damage), but log it so the
1593
+ // degradation is visible instead of silent.
1594
+ if (frame.state !== "queued") {
1595
+ this.logger.error(`Session ${params.sessionId}: unknown command_lifecycle state "${frame.state}" for ${frame.command_uuid}; treating as uninformative.`);
1596
+ }
1597
+ break;
1598
+ }
1599
+ continue;
1600
+ }
1033
1601
  switch (message.type) {
1034
1602
  case "system":
1035
1603
  switch (message.subtype) {
1036
1604
  case "init":
1605
+ // Latch the lifecycle capability so cancel() routes orphan
1606
+ // accounting through `orphanCommands` (per-uuid, exact)
1607
+ // instead of the coalescing-blind count. Never unlatch: init
1608
+ // re-emits per turn and the capability can't be lost mid-CLI.
1609
+ if (message.capabilities?.includes("msg_lifecycle_v1")) {
1610
+ session.msgLifecycleV1 = true;
1611
+ }
1037
1612
  // A fresh `system`/init (e.g. after reinitialize) can carry an
1038
1613
  // updated Fast mode state; reconcile it with what we seeded at
1039
1614
  // session creation.
1040
1615
  await this.syncFastModeState(message.session_id, session, message.fast_mode_state);
1041
1616
  break;
1042
1617
  case "status": {
1618
+ // These banners count as delivered text (via sendUpdate), so
1619
+ // an echo-less turn that only ever emits them (e.g. `/compact`,
1620
+ // promoted at its own result) doesn't have its result text
1621
+ // re-emitted by the issue-#453 fallback.
1043
1622
  if (message.status === "compacting") {
1044
1623
  compactionInProgress = true;
1045
- await this.client.sessionUpdate({
1624
+ await sendUpdate({
1046
1625
  sessionId: message.session_id,
1047
1626
  update: {
1048
1627
  sessionUpdate: "agent_message_chunk",
@@ -1055,7 +1634,7 @@ export class ClaudeAcpAgent {
1055
1634
  // message carrying `compact_result`, not the `compact_boundary`
1056
1635
  // message (which only fires when there's content to compact).
1057
1636
  compactionInProgress = false;
1058
- await this.client.sessionUpdate({
1637
+ await sendUpdate({
1059
1638
  sessionId: message.session_id,
1060
1639
  update: {
1061
1640
  sessionUpdate: "agent_message_chunk",
@@ -1066,7 +1645,7 @@ export class ClaudeAcpAgent {
1066
1645
  else if (message.compact_result === "failed" && compactionInProgress) {
1067
1646
  compactionInProgress = false;
1068
1647
  const reason = message.compact_error ? `: ${message.compact_error}` : ".";
1069
- await this.client.sessionUpdate({
1648
+ await sendUpdate({
1070
1649
  sessionId: message.session_id,
1071
1650
  update: {
1072
1651
  sessionUpdate: "agent_message_chunk",
@@ -1091,9 +1670,9 @@ export class ClaudeAcpAgent {
1091
1670
  // dropped dramatically) and replaced within seconds by the next
1092
1671
  // result message.
1093
1672
  //
1094
- // `size` keeps coming from session.contextWindowSize (learned
1095
- // from modelUsage / the model heuristic) getContextUsage's
1096
- // window field under-reports extended 1M windows.
1673
+ // `size` keeps coming from session.contextWindowSize
1674
+ // compaction frees occupancy, it doesn't change the model's
1675
+ // window.
1097
1676
  //
1098
1677
  // The "Compacting completed." text is emitted from the `status`
1099
1678
  // handler (keyed on `compact_result`), not here, so the failure
@@ -1101,7 +1680,7 @@ export class ClaudeAcpAgent {
1101
1680
  const usedTokens = await fetchContextUsedTokens(session.query, this.logger);
1102
1681
  lastAssistantUsage = null;
1103
1682
  lastAssistantTotalUsage = usedTokens ?? 0;
1104
- await this.client.sessionUpdate({
1683
+ await sendUpdate({
1105
1684
  sessionId: message.session_id,
1106
1685
  update: {
1107
1686
  sessionUpdate: "usage_update",
@@ -1112,7 +1691,7 @@ export class ClaudeAcpAgent {
1112
1691
  break;
1113
1692
  }
1114
1693
  case "local_command_output": {
1115
- await this.client.sessionUpdate({
1694
+ await sendUpdate({
1116
1695
  sessionId: message.session_id,
1117
1696
  update: {
1118
1697
  sessionUpdate: "agent_message_chunk",
@@ -1122,6 +1701,7 @@ export class ClaudeAcpAgent {
1122
1701
  break;
1123
1702
  }
1124
1703
  case "session_state_changed": {
1704
+ session.lastSessionState = message.state;
1125
1705
  if (message.state === "idle") {
1126
1706
  // A non-cancelled turn normally settled at its terminal
1127
1707
  // `result` already (issue #773), and that result recorded an
@@ -1154,14 +1734,38 @@ export class ClaudeAcpAgent {
1154
1734
  // when the cancel pre-empted the result (wedge/force-cancel).
1155
1735
  if (session.cancelled && session.activeTurn && !session.activeTurn.settled) {
1156
1736
  settleActive({ stopReason: "cancelled", usage: sessionUsage(session) });
1737
+ // An interrupt can pre-empt the turn's result entirely
1738
+ // (nothing ran the result-case `finally`), so close the
1739
+ // delivery stretch here: idle is the SDK's authoritative
1740
+ // turn-over signal, and stale partial-text state would
1741
+ // suppress the next turn's issue-#453 fallback.
1742
+ session.emittedAssistantText = false;
1743
+ }
1744
+ else if (isHeldOpen(session.activeTurn)) {
1745
+ // A turn held open for its background subagents (see
1746
+ // Turn.deferredSettle). Idles keep their normal cadence
1747
+ // during the hold — the CLI emits one per processing
1748
+ // cycle (the turn's own trailer, then one per followup),
1749
+ // NOT one final "all drained" signal — so each one
1750
+ // absorbs an outstanding trailer debt, and the turn only
1751
+ // settles once none of its spawned subagents is left
1752
+ // (the followup-result settle usually got there first;
1753
+ // this is the fallback when no followup came). Mid-hold
1754
+ // idles never fall through: a held turn HAS its result,
1755
+ // so reading its idle as "turn abandoned without a
1756
+ // result" (issue #825) would fail a healthy prompt.
1757
+ if (session.owedTrailingIdles > 0) {
1758
+ session.owedTrailingIdles--;
1759
+ }
1760
+ settleDeferredIfDrained();
1157
1761
  }
1158
- else if (owedTrailingIdles > 0) {
1762
+ else if (session.owedTrailingIdles > 0) {
1159
1763
  // Absorb a settled turn's trailing idle. Also covers a
1160
1764
  // cancel that landed between a turn's counted result and
1161
1765
  // this lagged idle (no active turn to settle): the idle
1162
1766
  // still belongs to that settled turn, and skipping the
1163
1767
  // decrement would leak the debt permanently.
1164
- owedTrailingIdles--;
1768
+ session.owedTrailingIdles--;
1165
1769
  }
1166
1770
  else if (!session.cancelled &&
1167
1771
  session.activeTurn &&
@@ -1203,7 +1807,7 @@ export class ClaudeAcpAgent {
1203
1807
  const title = isSynthesis
1204
1808
  ? "Recalled synthesized memory"
1205
1809
  : `Recalled ${count} ${count === 1 ? "memory" : "memories"}`;
1206
- await this.client.sessionUpdate({
1810
+ await sendUpdate({
1207
1811
  sessionId: message.session_id,
1208
1812
  update: {
1209
1813
  sessionUpdate: "tool_call",
@@ -1230,7 +1834,7 @@ export class ClaudeAcpAgent {
1230
1834
  // list with this payload: supportedCommands() is captured once
1231
1835
  // at initialize and never reflects mid-session changes, so we
1232
1836
  // forward message.commands directly rather than re-querying.
1233
- await this.client.sessionUpdate({
1837
+ await sendUpdate({
1234
1838
  sessionId: message.session_id,
1235
1839
  update: {
1236
1840
  sessionUpdate: "available_commands_update",
@@ -1254,7 +1858,7 @@ export class ClaudeAcpAgent {
1254
1858
  // rejection reason — otherwise the client shows a tool call
1255
1859
  // that silently never resolves.
1256
1860
  const reason = message.decision_reason ?? message.message;
1257
- await this.client.sessionUpdate({
1861
+ await sendUpdate({
1258
1862
  sessionId: message.session_id,
1259
1863
  update: {
1260
1864
  sessionUpdate: "tool_call_update",
@@ -1286,10 +1890,14 @@ export class ClaudeAcpAgent {
1286
1890
  // instead of a silent stop. ACP's agent_message_chunk has no
1287
1891
  // severity field, so fold the level into the text for the more
1288
1892
  // prominent levels ('info' is transcript-only noise — leave plain).
1893
+ // Sending via sendUpdate also marks the notice as this stretch's
1894
+ // delivered text: a hook-blocked turn's result repeats the block
1895
+ // reason with zero output tokens, and the issue-#453 fallback
1896
+ // must not emit it a second time.
1289
1897
  const text = message.level === "info"
1290
1898
  ? message.content
1291
1899
  : `**${message.level[0].toUpperCase()}${message.level.slice(1)}:** ${message.content}`;
1292
- await this.client.sessionUpdate({
1900
+ await sendUpdate({
1293
1901
  sessionId: message.session_id,
1294
1902
  update: {
1295
1903
  sessionUpdate: "agent_message_chunk",
@@ -1302,16 +1910,51 @@ export class ClaudeAcpAgent {
1302
1910
  case "hook_progress":
1303
1911
  case "hook_response":
1304
1912
  case "files_persisted":
1913
+ case "task_progress":
1914
+ break;
1305
1915
  case "task_started":
1916
+ // For subagent tasks `task_id` is the subagent's agent id (the
1917
+ // SDK keys its task registry by agent id) and `tool_use_id` is
1918
+ // the Agent/Task tool_use that spawned it — recorded so the
1919
+ // subagent's permission requests, which reach canUseTool with
1920
+ // only `agentID`, can attribute their eagerly-emitted
1921
+ // tool_call to the parent tool call. Non-subagent tasks (e.g.
1922
+ // background Bash) land here too; their task_ids never match
1923
+ // an agentID, so those entries are inert for attribution.
1924
+ //
1925
+ // `isSubagent` marks Task/Agent-tool subagents — the tasks
1926
+ // whose completion wakes the model for a followup, so the
1927
+ // ones worth deferring turn settlement for. A sync subagent
1928
+ // is pruned (terminal task_updated) before its turn's result
1929
+ // can arrive, so registry membership at result time means an
1930
+ // async subagent. Their spawn is also recorded on the active
1931
+ // turn: a turn only ever waits on its own subagents, and a
1932
+ // spawn during a held-open drain window (an agent chain)
1933
+ // extends that turn's hold.
1934
+ session.liveBackgroundTasks.set(message.task_id, {
1935
+ parentToolUseId: message.tool_use_id,
1936
+ isSubagent: !!message.subagent_type,
1937
+ });
1938
+ if (message.subagent_type && session.activeTurn && !session.activeTurn.settled) {
1939
+ (session.activeTurn.spawnedTaskIds ??= new Set()).add(message.task_id);
1940
+ }
1941
+ break;
1306
1942
  case "task_notification":
1307
- case "task_progress":
1943
+ // The task settled — no further tool calls can originate
1944
+ // from it, so its registry entry can be dropped.
1945
+ session.liveBackgroundTasks.delete(message.task_id);
1946
+ break;
1308
1947
  case "task_updated":
1309
- case "background_tasks_changed":
1310
- case "control_request_progress":
1311
- // `background_tasks_changed` is a level signal of live background
1312
- // tasks; this adapter drives lifecycle off the task edge bookends.
1313
- // `control_request_progress` only reports on side_question control
1314
- // requests, which this adapter never issues.
1948
+ // terminal-status task_updated patch and a (deduplicated)
1949
+ // task_notification when a task settles, but only the patch is
1950
+ // guaranteed per transition prune on it too so the registry
1951
+ // can't grow for the session's lifetime if a notification is
1952
+ // skipped.
1953
+ if (message.patch.status === "completed" ||
1954
+ message.patch.status === "failed" ||
1955
+ message.patch.status === "killed") {
1956
+ session.liveBackgroundTasks.delete(message.task_id);
1957
+ }
1315
1958
  break;
1316
1959
  case "worker_shutting_down":
1317
1960
  // A Remote Control worker announced a graceful teardown. This is a
@@ -1363,7 +2006,7 @@ export class ClaudeAcpAgent {
1363
2006
  const outcome = persistent
1364
2007
  ? `The session will continue on ${message.fallback_model}.`
1365
2008
  : `The session stays on ${message.original_model}.`;
1366
- await this.client.sessionUpdate({
2009
+ await sendUpdate({
1367
2010
  sessionId: message.session_id,
1368
2011
  update: {
1369
2012
  sessionUpdate: "agent_message_chunk",
@@ -1397,183 +2040,334 @@ export class ClaudeAcpAgent {
1397
2040
  lastRefusalExplanation = message.api_refusal_explanation ?? message.content;
1398
2041
  }
1399
2042
  break;
2043
+ // `control_request_progress` only reports on side_question
2044
+ // control requests, which this adapter never issues.
2045
+ case "control_request_progress":
2046
+ break;
2047
+ case "background_tasks_changed":
2048
+ // A level signal: the full live background-task set on every
2049
+ // membership change, with REPLACE semantics. Used only to
2050
+ // reconcile `liveBackgroundTasks` — dropping (or, for
2051
+ // subagent entries, unpinning) any entry whose settle
2052
+ // bookend (task_notification / terminal task_updated) was
2053
+ // lost, so a leaked subagent entry can't defer its spawning
2054
+ // turn's settlement forever. Growth of retained
2055
+ // (endedPerLevel) subagent entries is bounded by the
2056
+ // activation-time sweep in activateTurn, not here. It never
2057
+ // ADDS entries (the payload carries no attribution or
2058
+ // subagent marker), so the unspecified ordering vs. the edge
2059
+ // bookends is safe: a level that precedes its task_started
2060
+ // simply no-ops here.
2061
+ if (session.liveBackgroundTasks.size > 0) {
2062
+ const live = new Set(message.tasks.map((t) => t.task_id));
2063
+ for (const [taskId, record] of session.liveBackgroundTasks) {
2064
+ if (live.has(taskId)) {
2065
+ // The level proves the task live in the background
2066
+ // universe (e.g. a foreground agent was backgrounded
2067
+ // after an earlier absent-marking, or that marking was
2068
+ // a racing payload built before the task registered) —
2069
+ // un-end it so a hold waits on it again, and disarm
2070
+ // the activation sweep.
2071
+ record.endedPerLevel = undefined;
2072
+ continue;
2073
+ }
2074
+ if (record.isSubagent) {
2075
+ // The level's universe is BACKGROUND tasks only, so a
2076
+ // live sync (foreground) subagent is legitimately
2077
+ // absent — deleting its entry would strand its
2078
+ // permission attribution (#859). Keep the entry but
2079
+ // stop any hold from waiting on the id: an absent id
2080
+ // can equally be a leaked async entry whose settle
2081
+ // bookends were lost.
2082
+ record.endedPerLevel ??= "ended";
2083
+ }
2084
+ else {
2085
+ session.liveBackgroundTasks.delete(taskId);
2086
+ }
2087
+ }
2088
+ }
2089
+ break;
1400
2090
  default:
1401
2091
  unreachable(message, this.logger);
1402
2092
  break;
1403
2093
  }
1404
2094
  break;
1405
2095
  case "result": {
1406
- // Task-notification followups are autonomous work triggered by a
1407
- // task-notification system message, not by the user's prompt.
1408
- // They should not influence the user-turn lifecycle (stop reason,
1409
- // slash-command output forwarding) but their cost is real.
1410
- const isTaskNotification = message.origin?.kind === "task-notification";
1411
- // Reconcile the Fast mode toggle with the SDK's reported state.
1412
- // Gated to user-driven turns like every other side effect below; a
1413
- // background followup's state lands on the next user turn's result.
1414
- // Runs even when the turn errors or was cancelled.
1415
- if (!isTaskNotification) {
1416
- await this.syncFastModeState(params.sessionId, session, message.fast_mode_state);
1417
- }
1418
- // A user-turn result needs an active turn so its stop reason is
1419
- // attributed and the turn settles at idle. Local-only commands carry
1420
- // no user-message echo to promote them, so do it here from the head.
1421
- // Promote BEFORE accumulating usage, since activation resets the
1422
- // accumulator — promoting after would discard this result's tokens.
1423
- if (!isTaskNotification) {
1424
- ensureActiveTurn();
1425
- }
1426
- // Every user-turn result terminates a turn (settle, reject, or
1427
- // orphan skip) and the SDK follows it with a trailing
1428
- // `session_state_changed: idle` — record the debt so the idle
1429
- // handler absorbs that idle rather than reading it as a turn the
1430
- // SDK abandoned (issue #825). One exclusion: the cancelled ACTIVE
1431
- // turn's own result. It is dropped at the `session.cancelled`
1432
- // guard, and either the idle itself settles the turn (consuming
1433
- // the trailer) or the next echo's hand-off does (which records
1434
- // the debt there instead) — counting here too would double it.
1435
- // Results skipped while cancelled with NO active turn — orphaned
1436
- // queued turns the SDK still ran, or a force-cancelled turn's
1437
- // late result after the backstop settled it — get no such settle,
1438
- // so their trailers must be counted here or they'd later be read
1439
- // as the next healthy turn being abandoned and false-fail it.
1440
- if (!isTaskNotification && (!session.cancelled || !session.activeTurn)) {
1441
- owedTrailingIdles++;
1442
- }
1443
- // Accumulate usage into the user turn's tally. Skip task-notification
1444
- // followups: their cost is real but is reported separately via the
1445
- // usage_update below, and `session.accumulatedUsage` is only reset on
1446
- // turn activation — so folding a task-notification result that lands
1447
- // after the next turn is active (but before it settles) would leak
1448
- // those tokens into that turn's PromptResponse.usage.
1449
- if (!isTaskNotification) {
1450
- session.accumulatedUsage.inputTokens += message.usage.input_tokens;
1451
- session.accumulatedUsage.outputTokens += message.usage.output_tokens;
1452
- session.accumulatedUsage.cachedReadTokens += message.usage.cache_read_input_tokens;
1453
- session.accumulatedUsage.cachedWriteTokens +=
1454
- message.usage.cache_creation_input_tokens;
1455
- }
1456
- const matchingModelUsage = lastAssistantModel
1457
- ? getMatchingModelUsage(message.modelUsage, lastAssistantModel)
1458
- : null;
1459
- // Only overwrite when we have an authoritative value — a miss
1460
- // (e.g. a turn with no top-level assistant message) would
1461
- // otherwise discard the window learned on a prior turn and
1462
- // leave the next prompt's mid-stream updates reporting 200k.
1463
- if (matchingModelUsage) {
1464
- session.contextWindowSize = matchingModelUsage.contextWindow;
1465
- }
1466
- // Send usage_update notification
1467
- if (lastAssistantTotalUsage !== null) {
1468
- await this.client.sessionUpdate({
1469
- sessionId: params.sessionId,
1470
- update: {
1471
- sessionUpdate: "usage_update",
1472
- used: lastAssistantTotalUsage,
1473
- size: session.contextWindowSize,
1474
- cost: {
1475
- amount: message.total_cost_usd,
1476
- currency: "USD",
1477
- },
1478
- ...(message.origin && {
1479
- _meta: { "_claude/origin": message.origin },
1480
- }),
1481
- },
1482
- });
1483
- }
1484
- if (session.cancelled) {
1485
- if (!isTaskNotification) {
1486
- stopReason = "cancelled";
2096
+ // A result from an autonomous cycle a task-notification
2097
+ // followup, or a peer/coordinator/observer message the model
2098
+ // handled on its own (see AUTONOMOUS_RESULT_ORIGINS) is not
2099
+ // the user's prompt's. Autonomous results must never touch the
2100
+ // user-turn lifecycle (stop reason, settles, failActive,
2101
+ // slash-command output forwarding), though their cost is real.
2102
+ const isAutonomousResult = message.origin != null && AUTONOMOUS_RESULT_ORIGINS.has(message.origin.kind);
2103
+ try {
2104
+ // Reconcile the Fast mode toggle with the SDK's reported state.
2105
+ // Gated to user-driven turns like every other side effect below;
2106
+ // an autonomous cycle's state lands on the next user turn's
2107
+ // result. Runs even when the turn errors or was cancelled.
2108
+ if (!isAutonomousResult) {
2109
+ await this.syncFastModeState(params.sessionId, session, message.fast_mode_state);
1487
2110
  }
1488
- break;
1489
- }
1490
- // A refusal can arrive on any result subtype (and may even set
1491
- // is_error), so handle it before the subtype switch — otherwise the
1492
- // is_error throw below would surface it as an internal error. The
1493
- // refused assistant message carries no visible content, so surface
1494
- // the classifier's explanation (when available) and report ACP's
1495
- // dedicated `refusal` stop reason.
1496
- if (message.stop_reason === "refusal" && !isTaskNotification) {
1497
- if (lastRefusalExplanation) {
1498
- await this.client.sessionUpdate({
2111
+ // A user-turn result needs an active turn so its stop reason is
2112
+ // attributed and the turn settles at idle. Local-only commands carry
2113
+ // no user-message echo to promote them, so do it here from the head.
2114
+ // Promote BEFORE accumulating usage, since activation resets the
2115
+ // accumulator promoting after would discard this result's tokens.
2116
+ // The orphan bookkeeping runs first: it covers folded/zombie
2117
+ // commands whose shared or late result this is, even when the
2118
+ // result is the ACTIVE turn's (ensureActiveTurn never looks at
2119
+ // the map in that case).
2120
+ if (!isAutonomousResult) {
2121
+ recordResultForOrphanCommands();
2122
+ ensureActiveTurn();
2123
+ }
2124
+ // A result closes the stretch of output it terminates: snapshot
2125
+ // the delivery record — AFTER ensureActiveTurn, whose held-turn
2126
+ // hand-off closes the held stretch, so an echo-less command
2127
+ // promoted here is judged on its own delivery, not on the held
2128
+ // turn's followup text — and before the handling below can emit
2129
+ // anything of its own; the `finally` then clears it so every
2130
+ // exit from this case (the cancelled-guard and refusal breaks
2131
+ // included) starts the next stretch clean. Clearing up front
2132
+ // instead would let result-time emissions (refusal explanation,
2133
+ // result-text forwarding) taint the next stretch and suppress a
2134
+ // following replayed turn's fallback. Autonomous cycles run
2135
+ // alongside a user turn and must not clear its flag (they exit
2136
+ // through the early break below, which the gated `finally`
2137
+ // leaves alone).
2138
+ const deliveredAssistantText = session.emittedAssistantText;
2139
+ // Every user-turn result terminates a turn (settle, reject, or
2140
+ // orphan skip) and the SDK follows it with a trailing
2141
+ // `session_state_changed: idle` — record the debt so the idle
2142
+ // handler absorbs that idle rather than reading it as a turn the
2143
+ // SDK abandoned (issue #825). One exclusion: the cancelled ACTIVE
2144
+ // turn's own result. It is dropped at the `session.cancelled`
2145
+ // guard, and either the idle itself settles the turn (consuming
2146
+ // the trailer) or the next echo's hand-off does (which records
2147
+ // the debt there instead) — counting here too would double it.
2148
+ // Results skipped while cancelled with NO active turn — orphaned
2149
+ // queued turns the SDK still ran, or a force-cancelled turn's
2150
+ // late result after the backstop settled it — get no such settle,
2151
+ // so their trailers must be counted here or they'd later be read
2152
+ // as the next healthy turn being abandoned and false-fail it.
2153
+ // Autonomous results (followups, peer/channel/coordinator
2154
+ // cycles) are counted too: each is its own processing cycle with
2155
+ // its own trailing idle, and that idle can lag past the next
2156
+ // prompt's echo — which, un-owed, would be read as the fresh
2157
+ // turn being abandoned (#825 false-fail). That lag was mostly
2158
+ // unreachable when such cycles only ran with no pending turn,
2159
+ // but a held turn settling AT a followup result unblocks the
2160
+ // client at exactly that point, making the race the common
2161
+ // case.
2162
+ // The cancelled-ACTIVE-turn exclusion applies only to that
2163
+ // turn's OWN result — a followup result arriving inside the
2164
+ // cancel window still gets its own trailer and must be counted,
2165
+ // or that idle would later false-fail the next prompt.
2166
+ if (isAutonomousResult || !session.cancelled || !session.activeTurn) {
2167
+ session.owedTrailingIdles++;
2168
+ }
2169
+ // Accumulate usage into the user turn's tally. Skip autonomous
2170
+ // results: their cost is real but is reported separately via the
2171
+ // usage_update below, and `session.accumulatedUsage` is only reset on
2172
+ // turn activation — so folding an autonomous result that lands
2173
+ // after the next turn is active (but before it settles) would leak
2174
+ // those tokens into that turn's PromptResponse.usage.
2175
+ if (!isAutonomousResult) {
2176
+ session.accumulatedUsage.inputTokens += message.usage.input_tokens;
2177
+ session.accumulatedUsage.outputTokens += message.usage.output_tokens;
2178
+ session.accumulatedUsage.cachedReadTokens += message.usage.cache_read_input_tokens;
2179
+ session.accumulatedUsage.cachedWriteTokens +=
2180
+ message.usage.cache_creation_input_tokens;
2181
+ }
2182
+ const matchingModelUsage = lastAssistantModel
2183
+ ? getMatchingModelUsage(message.modelUsage, lastAssistantModel)
2184
+ : null;
2185
+ // Only overwrite when we have an authoritative value — a miss
2186
+ // (e.g. a turn with no top-level assistant message) would
2187
+ // otherwise discard the window learned on a prior turn and
2188
+ // leave the next prompt's mid-stream updates reporting 200k.
2189
+ if (matchingModelUsage) {
2190
+ session.contextWindowSize = matchingModelUsage.contextWindow;
2191
+ }
2192
+ // Send usage_update notification
2193
+ if (lastAssistantTotalUsage !== null) {
2194
+ await sendUpdate({
1499
2195
  sessionId: params.sessionId,
1500
2196
  update: {
1501
- sessionUpdate: "agent_message_chunk",
1502
- content: { type: "text", text: lastRefusalExplanation },
2197
+ sessionUpdate: "usage_update",
2198
+ used: lastAssistantTotalUsage,
2199
+ size: session.contextWindowSize,
2200
+ cost: {
2201
+ amount: message.total_cost_usd,
2202
+ currency: "USD",
2203
+ },
2204
+ ...(message.origin && {
2205
+ _meta: { "_claude/origin": message.origin },
2206
+ }),
1503
2207
  },
1504
2208
  });
1505
2209
  }
1506
- stopReason = "refusal";
1507
- settleActive({ stopReason: "refusal", usage: sessionUsage(session) });
1508
- break;
1509
- }
1510
- switch (message.subtype) {
1511
- case "success": {
1512
- if (message.result.includes("Please run /login")) {
1513
- failActive(RequestError.authRequired());
1514
- break;
1515
- }
1516
- if (message.stop_reason === "max_tokens") {
1517
- if (!isTaskNotification) {
1518
- stopReason = "max_tokens";
1519
- }
1520
- break;
2210
+ if (session.cancelled) {
2211
+ if (!isAutonomousResult) {
2212
+ stopReason = "cancelled";
1521
2213
  }
1522
- if (message.is_error) {
1523
- failActive(RequestError.internalError(errorKindData(lastAssistantError), message.result));
1524
- break;
2214
+ break;
2215
+ }
2216
+ // A held turn (see Turn.deferredSettle) settles at its
2217
+ // followup's terminal result: this is the earliest point at which
2218
+ // the promised summary has fully streamed — the trailing idle
2219
+ // would work too, but a client should not wait out another idle
2220
+ // round-trip for a response whose content is already complete.
2221
+ // (While the turn still awaits another of its subagents —
2222
+ // parallel spawns — the helper holds; the next notification's
2223
+ // followup settles it instead. Other autonomous origins — peer/
2224
+ // channel/coordinator cycles — reach here too: settling a
2225
+ // drained hold at their results is as good as the idle
2226
+ // fallback.) Then stop: everything below is user-turn
2227
+ // lifecycle, and an autonomous outcome must never touch it —
2228
+ // its is_error or "Please run /login" text would otherwise
2229
+ // failActive a live turn (the held one, or the user's next
2230
+ // prompt) whose own result recorded a different outcome.
2231
+ if (isAutonomousResult) {
2232
+ settleDeferredIfDrained();
2233
+ // With no turn in flight OR QUEUED (also after the settle
2234
+ // above), the stretch holds only autonomous prose — close
2235
+ // it, so a replayed next prompt isn't silently suppressed by
2236
+ // the issue-#453 delivery check. A live turn's flag may
2237
+ // guard the USER's already-streamed text — and so may a
2238
+ // QUEUED turn's: with mid-message echo lag its deltas stream
2239
+ // before the echo activates it (activeTurn still null), and
2240
+ // clearing then would re-emit that answer via the fallback,
2241
+ // the duplicate direction the flag's doc forbids.
2242
+ if (!session.activeTurn && !firstUnsettledQueuedTurn()) {
2243
+ session.emittedAssistantText = false;
1525
2244
  }
1526
- // For local-only commands (no model invocation), the result
1527
- // text is the command output — forward it to the client.
1528
- // Task-notification followups never originate from a user
1529
- // slash command, so skip the forwarding for them.
1530
- if (session.activeTurn?.isLocalOnlyCommand && !isTaskNotification) {
1531
- for (const notification of toAcpNotifications(message.result, "assistant", params.sessionId, session.toolUseCache, this.client, this.logger)) {
1532
- await this.client.sessionUpdate(notification);
1533
- }
2245
+ break;
2246
+ }
2247
+ // A refusal can arrive on any result subtype (and may even set
2248
+ // is_error), so handle it before the subtype switch — otherwise the
2249
+ // is_error throw below would surface it as an internal error. The
2250
+ // refused assistant message carries no visible content, so surface
2251
+ // the classifier's explanation (when available) and report ACP's
2252
+ // dedicated `refusal` stop reason.
2253
+ if (message.stop_reason === "refusal") {
2254
+ if (lastRefusalExplanation) {
2255
+ await sendUpdate({
2256
+ sessionId: params.sessionId,
2257
+ update: {
2258
+ sessionUpdate: "agent_message_chunk",
2259
+ content: { type: "text", text: lastRefusalExplanation },
2260
+ },
2261
+ });
1534
2262
  }
2263
+ stopReason = "refusal";
2264
+ // Through the deferral gate, not settleActive: a refusal can
2265
+ // land on a turn whose spawned subagents are still live, and
2266
+ // settling it out from under them would strand their output
2267
+ // and permission requests out-of-turn (issue #866's deadlock,
2268
+ // through the refusal lane).
2269
+ settleOrDefer({ stopReason: "refusal", usage: sessionUsage(session) });
1535
2270
  break;
1536
2271
  }
1537
- case "error_during_execution": {
1538
- if (message.stop_reason === "max_tokens") {
1539
- if (!isTaskNotification) {
2272
+ switch (message.subtype) {
2273
+ case "success": {
2274
+ if (message.result.includes("Please run /login")) {
2275
+ failActive(RequestError.authRequired());
2276
+ break;
2277
+ }
2278
+ if (message.stop_reason === "max_tokens") {
1540
2279
  stopReason = "max_tokens";
2280
+ break;
2281
+ }
2282
+ if (message.is_error) {
2283
+ failActive(RequestError.internalError(errorKindData(lastAssistantError), message.result));
2284
+ break;
2285
+ }
2286
+ // The result text is forwarded in two cases. Local-only
2287
+ // commands (no model invocation): the result IS the command
2288
+ // output. Otherwise the result is normally a trailing copy of
2289
+ // text that already streamed — but a cache-replayed turn
2290
+ // generates no tokens, and some CLIs then skip streaming
2291
+ // entirely and answer on the `result` alone: no `stream_event`
2292
+ // deltas, no consolidated `assistant` message (issue #453).
2293
+ // Forward it rather than end the turn silently:
2294
+ // `deliveredAssistantText` covers whatever already reached the
2295
+ // client (a turn that showed its answer cannot emit it twice),
2296
+ // and the output-token check keeps the fallback to the
2297
+ // replayed turns it was reported for. `?? 0`: typed non-null,
2298
+ // but third-party backends have been observed omitting usage
2299
+ // token fields (see snapshotFromUsage), and the replay lane
2300
+ // was reported from exactly such a backend — treat a missing
2301
+ // count as the replay signature rather than silently disabling
2302
+ // the fallback there. (Autonomous results never get here —
2303
+ // they exit at the early break above — so no background
2304
+ // prose can be injected into the feed.)
2305
+ if (session.activeTurn?.isLocalOnlyCommand ||
2306
+ (!deliveredAssistantText && (message.usage.output_tokens ?? 0) === 0)) {
2307
+ for (const notification of toAcpNotifications(message.result, "assistant", params.sessionId, session.toolUseCache, this.client, this.logger)) {
2308
+ await sendUpdate(notification);
2309
+ }
1541
2310
  }
1542
2311
  break;
1543
2312
  }
1544
- if (message.is_error) {
1545
- failActive(RequestError.internalError(errorKindData(lastAssistantError), message.errors.join(", ") || message.subtype));
1546
- break;
1547
- }
1548
- if (!isTaskNotification) {
2313
+ case "error_during_execution": {
2314
+ if (message.stop_reason === "max_tokens") {
2315
+ stopReason = "max_tokens";
2316
+ break;
2317
+ }
2318
+ if (message.is_error) {
2319
+ failActive(RequestError.internalError(errorKindData(lastAssistantError), message.errors.join(", ") || message.subtype));
2320
+ break;
2321
+ }
1549
2322
  stopReason = "end_turn";
1550
- }
1551
- break;
1552
- }
1553
- case "error_max_budget_usd":
1554
- case "error_max_turns":
1555
- case "error_max_structured_output_retries":
1556
- if (message.is_error) {
1557
- failActive(RequestError.internalError(errorKindData(lastAssistantError), message.errors.join(", ") || message.subtype));
1558
2323
  break;
1559
2324
  }
1560
- if (!isTaskNotification) {
2325
+ case "error_max_budget_usd":
2326
+ case "error_max_turns":
2327
+ case "error_max_structured_output_retries":
2328
+ if (message.is_error) {
2329
+ failActive(RequestError.internalError(errorKindData(lastAssistantError), message.errors.join(", ") || message.subtype));
2330
+ break;
2331
+ }
1561
2332
  stopReason = "max_turn_requests";
1562
- }
1563
- break;
1564
- default:
1565
- unreachable(message, this.logger);
1566
- break;
2333
+ break;
2334
+ default:
2335
+ unreachable(message, this.logger);
2336
+ break;
2337
+ }
2338
+ // Settle the user turn at its terminal result so the client unlocks
2339
+ // as soon as the answer is done, rather than waiting for the SDK's
2340
+ // trailing `idle` (which can lag while background work runs — issue
2341
+ // #773). The consumer keeps draining afterward (absorbing idle and
2342
+ // forwarding any background output).
2343
+ //
2344
+ // One exception: while background subagents this turn spawned are
2345
+ // still live, settling now would strand their remaining work
2346
+ // outside any turn — ACP allows out-of-turn session/update, but
2347
+ // many clients stop consuming at the prompt response, and a
2348
+ // subagent's permission request would block on an RPC nobody
2349
+ // answers (issues #864/#866). Hold the turn open instead: store
2350
+ // the outcome and settle with it once the subagents are done —
2351
+ // at their followup's terminal result (see the deferred-settle
2352
+ // block above the subtype switch) or at an idle with none of
2353
+ // them left — so the subagents' streamed output, their
2354
+ // permission requests, and the model's promised summary all land
2355
+ // inside the turn. `session/cancel` and the next prompt's echo
2356
+ // hand-off still settle a deferred turn early, so a long-running
2357
+ // subagent never holds the prompt hostage.
2358
+ //
2359
+ // is_error/auth already settled via failActive (activeTurn is null
2360
+ // then, so both branches no-op); cancellation is left to the
2361
+ // idle/abort path. settleActive is idempotent, so a duplicate
2362
+ // idle is a no-op.
2363
+ if (!session.cancelled) {
2364
+ settleOrDefer({ stopReason, usage: sessionUsage(session) });
2365
+ }
1567
2366
  }
1568
- // Settle the user turn at its terminal result so the client unlocks
1569
- // as soon as the answer is done, rather than waiting for the SDK's
1570
- // trailing `idle` (which can lag while background work runs — issue
1571
- // #773). The consumer keeps draining afterward (absorbing idle and
1572
- // forwarding any background output). is_error/auth already settled
1573
- // via failActive; cancellation is left to the idle/abort path.
1574
- // settleActive is idempotent, so a duplicate idle is a no-op.
1575
- if (!isTaskNotification && !session.cancelled) {
1576
- settleActive({ stopReason, usage: sessionUsage(session) });
2367
+ finally {
2368
+ if (!isAutonomousResult) {
2369
+ session.emittedAssistantText = false;
2370
+ }
1577
2371
  }
1578
2372
  break;
1579
2373
  }
@@ -1636,11 +2430,10 @@ export class ClaudeAcpAgent {
1636
2430
  const model = message.event.message.model;
1637
2431
  if (model && model !== "<synthetic>") {
1638
2432
  lastAssistantModel = model;
1639
- // Only upgrade from the default — once a `result` has given
1640
- // us an authoritative window, trust it over the heuristic.
1641
- // Model switches invalidate the cached window via
1642
- // `syncSessionConfigState`, which resets us back to the
1643
- // default so this branch runs again for the new model.
2433
+ // Only upgrade from the default — once the SDK has given us
2434
+ // an authoritative window (seeded at session creation,
2435
+ // refreshed on model switches in `applyConfigOptionValue`,
2436
+ // confirmed by each `result`), trust it over the heuristic.
1644
2437
  if (session.contextWindowSize === DEFAULT_CONTEXT_WINDOW) {
1645
2438
  const inferred = inferContextWindowFromModel(model);
1646
2439
  if (inferred !== null) {
@@ -1666,7 +2459,7 @@ export class ClaudeAcpAgent {
1666
2459
  const nextUsage = totalTokens(lastAssistantUsage);
1667
2460
  if (nextUsage !== lastAssistantTotalUsage) {
1668
2461
  lastAssistantTotalUsage = nextUsage;
1669
- await this.client.sessionUpdate({
2462
+ await sendUpdate({
1670
2463
  sessionId: params.sessionId,
1671
2464
  update: {
1672
2465
  sessionUpdate: "usage_update",
@@ -1682,8 +2475,11 @@ export class ClaudeAcpAgent {
1682
2475
  taskState: session.taskState,
1683
2476
  emittedToolCalls: session.emittedToolCalls,
1684
2477
  messageId: currentStreamMessageId,
2478
+ streamedToolInputs,
1685
2479
  })) {
1686
- await this.client.sessionUpdate(notification);
2480
+ // sendUpdate records delivery; a subagent stream's chunks carry
2481
+ // the stamped parentToolUseId meta and are excluded there.
2482
+ await sendUpdate(notification);
1687
2483
  }
1688
2484
  break;
1689
2485
  }
@@ -1704,7 +2500,7 @@ export class ClaudeAcpAgent {
1704
2500
  // is still promoted — activateTurn() clears the flag. The turn's own
1705
2501
  // echo is then dropped from the feed (the client already shows it).
1706
2502
  if (message.type === "user" && "uuid" in message && message.uuid) {
1707
- const queued = (session.turnQueue ?? []).find((t) => t.promptUuid === message.uuid && !t.settled);
2503
+ const queued = findUnsettledTurn(message.uuid);
1708
2504
  if (queued) {
1709
2505
  // Only (re)activate if this isn't already the active turn — a
1710
2506
  // turn promoted early (e.g. by a result that preceded its echo)
@@ -1722,15 +2518,38 @@ export class ClaudeAcpAgent {
1722
2518
  // debt so that lagged idle is absorbed rather than read
1723
2519
  // as the freshly-activated turn ending without a result
1724
2520
  // (which would false-fail a healthy turn — issue #825).
1725
- owedTrailingIdles++;
2521
+ // Counted for a DEFERRED turn too, even though its own
2522
+ // result already recorded a debt that may still be
2523
+ // outstanding: the interrupt can produce a trailer of
2524
+ // its own, and over-counting is benign (absorbs one
2525
+ // future idle) while under-counting risks the false
2526
+ // fail this debt exists to prevent.
2527
+ session.owedTrailingIdles++;
1726
2528
  // Before activateTurn resets the accumulator, so the
1727
2529
  // usage still belongs to the cancelled turn.
1728
2530
  settleActive({ stopReason: "cancelled", usage: sessionUsage(session) });
1729
2531
  }
2532
+ else if (isHeldOpen(session.activeTurn)) {
2533
+ // A turn held open for its background subagents (see
2534
+ // Turn.deferredSettle) hands off with the real outcome
2535
+ // its result recorded, not a guessed end_turn — the
2536
+ // user moving on must not block behind a long-running
2537
+ // subagent, but it must not rewrite the stop reason
2538
+ // either. Its trailing-idle debt stands and is absorbed
2539
+ // when the drain idle eventually arrives.
2540
+ settleActive(session.activeTurn.deferredSettle);
2541
+ }
1730
2542
  else {
1731
2543
  settleActive({ stopReason: "end_turn", usage: sessionUsage(session) });
1732
2544
  }
1733
2545
  }
2546
+ // Unlike the no-result teardown lanes, this hand-off must
2547
+ // NOT clear emittedAssistantText for a NON-held previous
2548
+ // turn (a held one's settleActive above closes its own
2549
+ // stretch): the echo can land
2550
+ // mid-message, so deltas already streamed belong to the turn
2551
+ // being activated — clearing would forget them and let its
2552
+ // result re-emit the answer.
1734
2553
  activateTurn(queued);
1735
2554
  }
1736
2555
  break;
@@ -1783,7 +2602,7 @@ export class ClaudeAcpAgent {
1783
2602
  taskState: session.taskState,
1784
2603
  messageId: messageIdForGrouping(message),
1785
2604
  })) {
1786
- await this.client.sessionUpdate(notification);
2605
+ await sendUpdate(notification);
1787
2606
  }
1788
2607
  }
1789
2608
  else {
@@ -1807,12 +2626,7 @@ export class ClaudeAcpAgent {
1807
2626
  if (message.message.role === "system") {
1808
2627
  break;
1809
2628
  }
1810
- if (message.type === "assistant" &&
1811
- message.message.model === "<synthetic>" &&
1812
- Array.isArray(message.message.content) &&
1813
- message.message.content.length === 1 &&
1814
- message.message.content[0].type === "text" &&
1815
- message.message.content[0].text.includes("Please run /login")) {
2629
+ if (message.type === "assistant" && isSyntheticLoginMessage(message.message)) {
1816
2630
  failActive(RequestError.authRequired());
1817
2631
  break;
1818
2632
  }
@@ -1895,13 +2709,18 @@ export class ClaudeAcpAgent {
1895
2709
  taskState: session.taskState,
1896
2710
  emittedToolCalls: session.emittedToolCalls,
1897
2711
  messageId: messageIdForGrouping(message),
2712
+ toolUseResult: message.type === "user" ? message.tool_use_result : undefined,
1898
2713
  })) {
1899
- await this.client.sessionUpdate(notification);
2714
+ // sendUpdate records delivery. Subagent text/thinking is
2715
+ // filtered out of `content` above; blocks that do pass through
2716
+ // (e.g. a subagent image) carry the stamped parentToolUseId
2717
+ // meta and are excluded there.
2718
+ await sendUpdate(notification);
1900
2719
  }
1901
2720
  break;
1902
2721
  }
1903
2722
  case "tool_progress": {
1904
- await this.client.sessionUpdate({
2723
+ await sendUpdate({
1905
2724
  sessionId: message.session_id,
1906
2725
  update: {
1907
2726
  sessionUpdate: "tool_call_update",
@@ -1919,7 +2738,7 @@ export class ClaudeAcpAgent {
1919
2738
  }
1920
2739
  case "rate_limit_event": {
1921
2740
  if (lastAssistantTotalUsage !== null) {
1922
- await this.client.sessionUpdate({
2741
+ await sendUpdate({
1923
2742
  sessionId: message.session_id,
1924
2743
  update: {
1925
2744
  sessionUpdate: "usage_update",
@@ -1940,7 +2759,7 @@ export class ClaudeAcpAgent {
1940
2759
  case "conversation_reset":
1941
2760
  break;
1942
2761
  default:
1943
- unreachable(message);
2762
+ unreachable(message, this.logger);
1944
2763
  break;
1945
2764
  }
1946
2765
  }
@@ -1984,6 +2803,30 @@ export class ClaudeAcpAgent {
1984
2803
  }
1985
2804
  }
1986
2805
  }
2806
+ /** Route one orphaned command into the session's orphan-accounting lane:
2807
+ * the per-uuid map on msg_lifecycle_v1 CLIs (drained by the command's own
2808
+ * terminal lifecycle frame and the echo-less-result skip), the plain count
2809
+ * elsewhere (the count lane can't express per-command states, so `state`
2810
+ * only matters on the map lane). Both orphan-producing paths — cancel()'s
2811
+ * queued-turn sweep and the consumer's force-cancel wedge path — must seed
2812
+ * through here so the lane split stays a single mechanism.
2813
+ *
2814
+ * Known window: `msgLifecycleV1` is only learnable from the stream's first
2815
+ * `system`/init (the control-channel initialize carries no capabilities),
2816
+ * so a cancel that beats that drain seeds the COUNT lane on a
2817
+ * lifecycle-capable CLI — where command coalescing can leave the count
2818
+ * stale by N-1 (the pre-map bug, confined to this sub-second window and
2819
+ * still healed by the next activation's reset). Structural until the SDK
2820
+ * exposes capabilities before the stream starts. */
2821
+ trackOrphanCommand(session, uuid, state) {
2822
+ if (session.msgLifecycleV1) {
2823
+ session.orphanCommands ??= new Map();
2824
+ session.orphanCommands.set(uuid, state);
2825
+ }
2826
+ else {
2827
+ session.pendingOrphanResults = (session.pendingOrphanResults ?? 0) + 1;
2828
+ }
2829
+ }
1987
2830
  async cancel(params) {
1988
2831
  const session = this.sessions[params.sessionId];
1989
2832
  if (!session) {
@@ -2006,11 +2849,18 @@ export class ClaudeAcpAgent {
2006
2849
  return;
2007
2850
  }
2008
2851
  session.cancelled = true;
2852
+ // Capture the orphan-accounting lane before anything can await: the
2853
+ // consumer latches msgLifecycleV1 when it drains the first system/init,
2854
+ // which can happen DURING the awaited interrupt() below — the receipt
2855
+ // reconciliation must act on the same lane the seeding used, or a
2856
+ // count-lane orphan would be left for the map-lane receipt path (which
2857
+ // never decrements the count) to miss.
2858
+ const lifecycleLane = session.msgLifecycleV1 === true;
2009
2859
  // Settle queued turns that haven't started yet (no echo seen) right away —
2010
2860
  // they have no in-flight SDK work to interrupt. The active turn is settled
2011
2861
  // by the consumer when it observes the interrupt's trailing idle (or via the
2012
2862
  // backstop below). Mirrors the old pendingMessages cancellation.
2013
- const orphanedUuids = [];
2863
+ const orphanedTurns = [];
2014
2864
  if (session.turnQueue) {
2015
2865
  for (const turn of session.turnQueue) {
2016
2866
  if (turn !== session.activeTurn && !turn.settled) {
@@ -2018,16 +2868,98 @@ export class ClaudeAcpAgent {
2018
2868
  // Deliberately no `usage`: a queued turn never ran, so the session
2019
2869
  // accumulator (the active turn's tally) is not its spend.
2020
2870
  turn.resolve({ stopReason: "cancelled" });
2021
- orphanedUuids.push(turn.promptUuid);
2871
+ orphanedTurns.push(turn);
2022
2872
  }
2023
2873
  }
2024
2874
  // Each removed queued turn's user message was already pushed to the SDK,
2025
2875
  // which processes input FIFO and will still emit a result for it with no
2026
- // uuid to match. Count those so the consumer skips them (see
2876
+ // uuid to match. Track those so the consumer skips them (see
2027
2877
  // ensureActiveTurn) rather than misattributing them to the head.
2028
- session.pendingOrphanResults = (session.pendingOrphanResults ?? 0) + orphanedUuids.length;
2878
+ // msg_lifecycle_v1 CLIs get per-uuid tracking drained by the command's
2879
+ // own terminal lifecycle frame — exact under command coalescing, where
2880
+ // N queued commands fold into ONE turn emitting one result and a plain
2881
+ // count would go stale by N-1 and swallow a later echo-less result.
2882
+ // Older CLIs keep the count and its activation-time self-heal (they
2883
+ // never see lifecycle frames, so commandStarted/commandFinished stay
2884
+ // unset and every turn takes the plain-seed path below).
2885
+ for (const turn of orphanedTurns) {
2886
+ if (turn.commandFinished === "completed" || turn.commandFinished === "discarded") {
2887
+ // The command already finished SDK-side and its terminal frame was
2888
+ // consumed while the turn sat queued — nothing is left to skip, and
2889
+ // a seeded entry would never drain.
2890
+ continue;
2891
+ }
2892
+ if (turn.commandFinished === "cancelled") {
2893
+ // Terminal frame already consumed. Dispatched-then-aborted: the
2894
+ // dead turn's late result may still come — seed the zombie the
2895
+ // frame handler would have made — unless that result already
2896
+ // passed pre-cancel (commandResultSeen: e.g. the command folded
2897
+ // into the active turn and their shared result was attributed
2898
+ // there), in which case a zombie would be a phantom that swallows
2899
+ // an unrelated later result. Never dispatched: dropped, no result
2900
+ // coming, nothing to track.
2901
+ if (turn.commandStarted && !turn.commandResultSeen) {
2902
+ this.trackOrphanCommand(session, turn.promptUuid, "zombie");
2903
+ }
2904
+ continue;
2905
+ }
2906
+ if (turn.commandStarted && turn.commandResultSeen) {
2907
+ // Dispatched and its turn's result already passed; only its
2908
+ // terminal frame is outstanding, which no-ops with no entry.
2909
+ continue;
2910
+ }
2911
+ this.trackOrphanCommand(session, turn.promptUuid, turn.commandStarted ? "started" : "pending");
2912
+ }
2029
2913
  session.turnQueue = session.turnQueue.filter((turn) => turn === session.activeTurn && !turn.settled);
2030
2914
  }
2915
+ // A deferred active turn (see Turn.deferredSettle) already has its
2916
+ // result — it is only held open for its background subagents, which the
2917
+ // interrupt below tears down. Settle it "cancelled" NOW: during the hold
2918
+ // the session is typically already in state idle (the CLI's trailer
2919
+ // fired at the result), so the interrupt may produce no fresh idle for
2920
+ // the consumer's cancelled-settle path to run on, and the cancel would
2921
+ // otherwise stall until the force-cancel backstop. Any outstanding
2922
+ // trailer debt is absorbed by the idle handler when its idle does come.
2923
+ // The turn's own usage snapshot is reported per the cancelled-usage
2924
+ // contract (issue #844).
2925
+ {
2926
+ const active = session.activeTurn;
2927
+ if (isHeldOpen(active)) {
2928
+ active.settled = true;
2929
+ // Mirror settleActive's invariants (it is consumer-scoped and
2930
+ // unreachable from here): disarm the backstop — none should be
2931
+ // armed for a held turn, but a drift here must not leave a timer
2932
+ // firing on a settled turn — and drop the turn from the queue.
2933
+ disarmForceCancel(session);
2934
+ session.turnQueue = (session.turnQueue ?? []).filter((t) => t !== active);
2935
+ session.activeTurn = null;
2936
+ // Settling a held turn closes its delivery stretch: any streamed
2937
+ // text since the last boundary was its followups', and left latched
2938
+ // it would suppress a following replayed turn's issue-#453 fallback.
2939
+ session.emittedAssistantText = false;
2940
+ // When the interrupt below pre-empts a live cycle — running, or
2941
+ // blocked on a permission request (requires_action, the #866 shape
2942
+ // users cancel out of) — it produces a trailer idle with no counted
2943
+ // result; with the hold's own trailer typically already absorbed,
2944
+ // that idle would be un-owed and could lag past the next prompt's
2945
+ // echo — read as the fresh turn ending without a result (issue #825
2946
+ // false-fail). Pre-count it unless the session sits idle: there the
2947
+ // interrupt emits nothing, and a debt that never drains would mask
2948
+ // one future #825 detection. (lastSessionState is last-CONSUMED, so
2949
+ // both stale reads exist and both are accepted one-cycle windows: a
2950
+ // running transition still in the backlog reads as stale idle and
2951
+ // under-counts — that false-fail additionally needs the trailer to
2952
+ // lag past the next echo — while a cycle already completed into the
2953
+ // backlog reads as stale non-idle and over-counts, masking one
2954
+ // future #825 detection. Undefined — no state event consumed —
2955
+ // pre-counts; that only occurs on CLIs whose missing idle events
2956
+ // also disable the detector the debt could mask.)
2957
+ if (session.lastSessionState !== "idle") {
2958
+ session.owedTrailingIdles++;
2959
+ }
2960
+ active.resolve({ stopReason: "cancelled", usage: active.deferredSettle.usage });
2961
+ }
2962
+ }
2031
2963
  // Arm a backstop before interrupting: if a turn is actively consuming the
2032
2964
  // query and interrupt() doesn't make the SDK yield (e.g. a wedged TaskOutput
2033
2965
  // block — issue #680), force the consumer to settle the active turn
@@ -2065,11 +2997,30 @@ export class ClaudeAcpAgent {
2065
2997
  // receipt, so a bare `{}` success from a gateway can't read as "everything
2066
2998
  // was dropped") — keep the count-everything behavior and its
2067
2999
  // activation-time self-heal.
2068
- if (Array.isArray(receipt?.still_queued) && orphanedUuids.length > 0) {
3000
+ if (Array.isArray(receipt?.still_queued) && orphanedTurns.length > 0) {
2069
3001
  const stillQueued = new Set(receipt.still_queued);
2070
- const dropped = orphanedUuids.filter((uuid) => !stillQueued.has(uuid)).length;
2071
- if (dropped > 0) {
2072
- session.pendingOrphanResults = Math.max(0, (session.pendingOrphanResults ?? 0) - dropped);
3002
+ if (lifecycleLane) {
3003
+ // Lifecycle lane: forget dropped orphans by uuid. Only entries still
3004
+ // "pending" an orphan absent from `still_queued` because it was
3005
+ // DISPATCHED before the interrupt (not dropped) has usually been
3006
+ // promoted to "started" by its lifecycle frame by now, and its own
3007
+ // terminal frame must stay in charge of its fate. (If that frame is
3008
+ // still in the consumer's backlog we mis-forget — the same exposure
3009
+ // the count lane has always had for a dropped-then-run command.)
3010
+ // Mostly redundant with the "cancelled"-frame removal, but a receipt
3011
+ // survives paths where that frame was never emitted.
3012
+ for (const turn of orphanedTurns) {
3013
+ if (!stillQueued.has(turn.promptUuid) &&
3014
+ session.orphanCommands?.get(turn.promptUuid) === "pending") {
3015
+ session.orphanCommands.delete(turn.promptUuid);
3016
+ }
3017
+ }
3018
+ }
3019
+ else {
3020
+ const dropped = orphanedTurns.filter((turn) => !stillQueued.has(turn.promptUuid)).length;
3021
+ if (dropped > 0) {
3022
+ session.pendingOrphanResults = Math.max(0, (session.pendingOrphanResults ?? 0) - dropped);
3023
+ }
2073
3024
  }
2074
3025
  }
2075
3026
  }
@@ -2117,10 +3068,7 @@ export class ClaudeAcpAgent {
2117
3068
  // after the floor, and clear the timer so it can't outlive the deleted
2118
3069
  // session (it isn't unref'd and would otherwise keep the event loop alive
2119
3070
  // until it fires).
2120
- if (session.forceCancelTimer) {
2121
- clearTimeout(session.forceCancelTimer);
2122
- session.forceCancelTimer = undefined;
2123
- }
3071
+ disarmForceCancel(session);
2124
3072
  session.cancelController?.abort();
2125
3073
  this.closeQueryStream(session);
2126
3074
  // Abort the SDK abort signal only on explicit destroy. closeQueryStream
@@ -2338,6 +3286,12 @@ export class ClaudeAcpAgent {
2338
3286
  if (replaySession && replayMessageId && message.uuid) {
2339
3287
  replaySession.messageIdToUuid.set(replayMessageId, message.uuid);
2340
3288
  }
3289
+ // The live prompt loop converts the synthetic "Please run /login"
3290
+ // assistant message into an authRequired error instead of showing its
3291
+ // TUI-specific text; skip it on replay too (issue #863).
3292
+ if (message.type === "assistant" && isSyntheticLoginMessage(message.message)) {
3293
+ continue;
3294
+ }
2341
3295
  // @ts-expect-error - untyped in SDK but we handle all of these
2342
3296
  let content = message.message.content;
2343
3297
  // @ts-expect-error - untyped in SDK but we handle all of these
@@ -2376,14 +3330,14 @@ export class ClaudeAcpAgent {
2376
3330
  * outcome or a `requestCancelled` rejection). Either way we surface the same
2377
3331
  * "Tool use aborted" the callers already expect, so a cancelled dialog no
2378
3332
  * longer leaves the `await` hanging. */
2379
- async requestPermissionFromClient(params, toolName, signal) {
3333
+ async requestPermissionFromClient(params, toolName, signal, parentToolUseId) {
2380
3334
  // The SDK may invoke `canUseTool` (and therefore this permission request)
2381
3335
  // before the assistant message's tool_use block streams to us. Some ACP clients
2382
3336
  // expect the `tool_call` a permission request references to already exist,
2383
3337
  // so emit it now if it hasn't been sent yet. The streamed tool_use chunk
2384
3338
  // later refines it with a `tool_call_update` rather than emitting a
2385
3339
  // duplicate (see `emittedToolCalls` in `toAcpNotifications`).
2386
- await this.ensureToolCallEmitted(params.sessionId, toolName, params.toolCall.toolCallId, params.toolCall.rawInput);
3340
+ await this.ensureToolCallEmitted(params.sessionId, toolName, params.toolCall.toolCallId, params.toolCall.rawInput, parentToolUseId);
2387
3341
  try {
2388
3342
  return await this.client.requestPermission(params, signal);
2389
3343
  }
@@ -2400,10 +3354,15 @@ export class ClaudeAcpAgent {
2400
3354
  * instead of emitting a duplicate (see `emittedToolCalls`). Built via the same
2401
3355
  * `toolCallNotification` helper as the streamed path so the two are identical.
2402
3356
  * Tools the stream renders as a plan (TodoWrite) or suppresses (Task*) are
2403
- * skipped so a permission prompt for them never surfaces a stray tool_call. */
2404
- async ensureToolCallEmitted(sessionId, toolName, toolCallId, toolInput) {
3357
+ * emitted too: a permission request referencing a tool call the client has
3358
+ * never seen can trip strict clients (issue #851), so the reference must
3359
+ * always resolve. Since the streamed path never completes those calls, they
3360
+ * are resolved at tool_result time instead (see `toAcpNotifications`).
3361
+ * `parentToolUseId` attributes a subagent's tool call to the Agent/Task call
3362
+ * that spawned it, matching the streamed path's `_meta`. */
3363
+ async ensureToolCallEmitted(sessionId, toolName, toolCallId, toolInput, parentToolUseId) {
2405
3364
  const session = this.sessions[sessionId];
2406
- if (!session || !shouldEmitToolCall(toolName)) {
3365
+ if (!session) {
2407
3366
  return;
2408
3367
  }
2409
3368
  if (session.emittedToolCalls.has(toolCallId)) {
@@ -2411,13 +3370,20 @@ export class ClaudeAcpAgent {
2411
3370
  }
2412
3371
  session.emittedToolCalls.add(toolCallId);
2413
3372
  const supportsTerminalOutput = this.clientCapabilities?._meta?.["terminal_output"] === true;
2414
- await this.client.sessionUpdate({
2415
- sessionId,
2416
- update: toolCallNotification({ id: toolCallId, name: toolName, input: toolInput }, toolInput, supportsTerminalOutput, session.cwd),
2417
- });
3373
+ const update = toolCallNotification({ id: toolCallId, name: toolName, input: toolInput }, toolInput, supportsTerminalOutput, session.cwd);
3374
+ if (parentToolUseId) {
3375
+ update._meta = {
3376
+ ...update._meta,
3377
+ claudeCode: {
3378
+ ...(update._meta?.claudeCode || {}),
3379
+ parentToolUseId,
3380
+ },
3381
+ };
3382
+ }
3383
+ await this.client.sessionUpdate({ sessionId, update });
2418
3384
  }
2419
3385
  canUseTool(sessionId) {
2420
- return async (toolName, toolInput, { signal, suggestions, toolUseID }) => {
3386
+ return async (toolName, toolInput, { signal, suggestions, toolUseID, agentID }) => {
2421
3387
  const alwaysAllowLabel = describeAlwaysAllow(suggestions, toolName);
2422
3388
  const supportsTerminalOutput = this.clientCapabilities?._meta?.["terminal_output"] === true;
2423
3389
  const session = this.sessions[sessionId];
@@ -2427,6 +3393,23 @@ export class ClaudeAcpAgent {
2427
3393
  message: "Session not found",
2428
3394
  };
2429
3395
  }
3396
+ // When the tool call originates inside a subagent, attribute the eagerly
3397
+ // emitted tool_call (and the permission request itself) to the Agent/Task
3398
+ // tool call that spawned the subagent, mirroring the streamed subagent
3399
+ // path's `_meta.claudeCode.parentToolUseId` (see `liveBackgroundTasks`).
3400
+ const parentToolUseId = agentID
3401
+ ? session.liveBackgroundTasks.get(agentID)?.parentToolUseId
3402
+ : undefined;
3403
+ if (agentID && !parentToolUseId) {
3404
+ // The attribution rests on an undocumented SDK invariant
3405
+ // (task_started.task_id === canUseTool's agentID for subagent tasks;
3406
+ // verified against the bundled CLI). Should an SDK bump break it — or
3407
+ // the consumer lose the race with task_started — the lookup misses and
3408
+ // the request goes out unattributed; log it so the regression is
3409
+ // observable rather than silent.
3410
+ this.logger.log(`[claude-agent-acp] No parent tool_use recorded for subagent ${agentID}; ` +
3411
+ `sending the ${toolName} permission request unattributed`);
3412
+ }
2430
3413
  // AskUserQuestion is surfaced to us as a normal permission check (the SDK
2431
3414
  // routes it through canUseTool whenever a callback is registered, rather
2432
3415
  // than the interactive dialog). Present it as an ACP form elicitation and
@@ -2434,7 +3417,7 @@ export class ClaudeAcpAgent {
2434
3417
  if (toolName === "AskUserQuestion" && this.clientCapabilities?.elicitation?.form) {
2435
3418
  // Like permission requests, the elicitation references this toolUseID, so
2436
3419
  // make sure the tool_call has surfaced to the client before we send it.
2437
- await this.ensureToolCallEmitted(sessionId, toolName, toolUseID, toolInput);
3420
+ await this.ensureToolCallEmitted(sessionId, toolName, toolUseID, toolInput, parentToolUseId);
2438
3421
  return this.handleAskUserQuestion(sessionId, toolInput, toolUseID, signal);
2439
3422
  }
2440
3423
  // Fallback for clients WITHOUT `elicitation.form`: route each question
@@ -2503,8 +3486,13 @@ export class ClaudeAcpAgent {
2503
3486
  toolCallId: toolUseID,
2504
3487
  rawInput: toolInput,
2505
3488
  ...toolInfoFromToolUse({ name: toolName, input: toolInput, id: toolUseID }, supportsTerminalOutput, session?.cwd),
3489
+ // `claudeCode` metas always carry `toolName` (see ToolUpdateMeta),
3490
+ // so clients can rely on one shape everywhere.
3491
+ ...(parentToolUseId
3492
+ ? { _meta: { claudeCode: { toolName, parentToolUseId } } }
3493
+ : {}),
2506
3494
  },
2507
- }, toolName, signal);
3495
+ }, toolName, signal, parentToolUseId);
2508
3496
  if (signal.aborted || response.outcome?.outcome === "cancelled") {
2509
3497
  throw new Error("Tool use aborted");
2510
3498
  }
@@ -2562,8 +3550,13 @@ export class ClaudeAcpAgent {
2562
3550
  toolCallId: toolUseID,
2563
3551
  rawInput: toolInput,
2564
3552
  ...toolInfoFromToolUse({ name: toolName, input: toolInput, id: toolUseID }, supportsTerminalOutput, session?.cwd),
3553
+ // `claudeCode` metas always carry `toolName` (see ToolUpdateMeta),
3554
+ // so clients can rely on one shape everywhere.
3555
+ ...(parentToolUseId
3556
+ ? { _meta: { claudeCode: { toolName, parentToolUseId } } }
3557
+ : {}),
2565
3558
  },
2566
- }, toolName, signal);
3559
+ }, toolName, signal, parentToolUseId);
2567
3560
  if (signal.aborted || response.outcome?.outcome === "cancelled") {
2568
3561
  throw new Error("Tool use aborted");
2569
3562
  }
@@ -2739,12 +3732,17 @@ export class ClaudeAcpAgent {
2739
3732
  // carries no "1m" token.
2740
3733
  const newModelInfo = session.modelInfos.find((m) => m.value === value);
2741
3734
  if (session.models.currentModelId !== value) {
2742
- // The cached context window was learned for the previous model; reset
2743
- // to the new model's heuristic so mid-stream updates between now and
2744
- // the next `result` reflect the user's selection instead of the old
2745
- // model's window.
3735
+ // The cached context window was learned for the previous model. The
3736
+ // SDK is already running the new model here (user-driven switches call
3737
+ // `query.setModel` before this, and the refusal-fallback sync only
3738
+ // reconciles a switch the SDK already made), so ask it for the new
3739
+ // window; fall back to the text heuristic so mid-stream updates
3740
+ // between now and the next `result` reflect the user's selection
3741
+ // instead of the old model's window.
2746
3742
  session.contextWindowSize =
2747
- inferContextWindowFromModel(value, newModelInfo?.displayName, newModelInfo?.description) ?? DEFAULT_CONTEXT_WINDOW;
3743
+ (await fetchContextWindowSize(session.query, this.logger)) ??
3744
+ inferContextWindowFromModel(value, newModelInfo?.resolvedModel, newModelInfo?.displayName, newModelInfo?.description) ??
3745
+ DEFAULT_CONTEXT_WINDOW;
2748
3746
  }
2749
3747
  session.models = { ...session.models, currentModelId: value };
2750
3748
  // Recompute availableModes for the new model and clamp the current
@@ -3369,7 +4367,10 @@ export class ClaudeAcpAgent {
3369
4367
  env: {
3370
4368
  ...process.env,
3371
4369
  ...userProvidedOptions?.env,
3372
- ...createEnvForGateway(this.gatewayAuthRequest),
4370
+ // Client-managed LLM routing: `providers/set` config wins, else the
4371
+ // legacy gateway auth request. Baked into the query at creation, so it
4372
+ // only affects sessions started after the change (matching the RFD).
4373
+ ...createEnvForProvider(this.resolveProviderConfig()),
3373
4374
  // Opt-in to session state events like when the agent is idle
3374
4375
  CLAUDE_CODE_EMIT_SESSION_STATE_EVENTS: "1",
3375
4376
  },
@@ -3654,6 +4655,22 @@ export class ClaudeAcpAgent {
3654
4655
  effortLevel: initialEffort.currentValue,
3655
4656
  });
3656
4657
  }
4658
+ // Seed the context window from the SDK's authoritative report. Text
4659
+ // inference alone misses aliases that resolve to extended-context models
4660
+ // with no "1m" token anywhere in their id or description (e.g. `sonnet` →
4661
+ // claude-sonnet-5, natively ~1M): those streamed `usage_update.size:
4662
+ // 200000` until the first result's modelUsage corrected it — again on
4663
+ // every process restart or session re-creation, since the learned window
4664
+ // lives only on the Session (issue #596).
4665
+ //
4666
+ // The inference fallback is deliberately keyed to the allowlisted entry: a
4667
+ // fallback-resolved sibling's resolvedModel/displayName/description can
4668
+ // describe a different context lane than the verbatim live id (e.g. an
4669
+ // "opus[1m]" row matched for a bare 200k id), so on the fallback path only
4670
+ // the id itself is a trustworthy window signal.
4671
+ const contextWindowSize = (await fetchContextWindowSize(q, this.logger)) ??
4672
+ inferContextWindowFromModel(models.currentModelId, catalogModelInfo?.resolvedModel, catalogModelInfo?.displayName, catalogModelInfo?.description) ??
4673
+ DEFAULT_CONTEXT_WINDOW;
3657
4674
  this.sessions[sessionId] = {
3658
4675
  query: q,
3659
4676
  input: input,
@@ -3683,16 +4700,13 @@ export class ClaudeAcpAgent {
3683
4700
  fastModeEnabled,
3684
4701
  abortController,
3685
4702
  emitRawSDKMessages: sessionMeta?.claudeCode?.emitRawSDKMessages ?? false,
3686
- contextWindowSize:
3687
- // Deliberately keyed to the catalog entry: a fallback-resolved
3688
- // sibling's displayName/description can describe a different context
3689
- // lane than the verbatim live id (e.g. an "opus[1m]" row matched for
3690
- // a bare 200k id), so on the fallback path only the id itself is a
3691
- // trustworthy window signal.
3692
- inferContextWindowFromModel(models.currentModelId, catalogModelInfo?.displayName, catalogModelInfo?.description) ?? DEFAULT_CONTEXT_WINDOW,
4703
+ contextWindowSize,
3693
4704
  taskState,
3694
4705
  toolUseCache: {},
3695
4706
  emittedToolCalls: new Set(),
4707
+ liveBackgroundTasks: new Map(),
4708
+ emittedAssistantText: false,
4709
+ owedTrailingIdles: 0,
3696
4710
  messageIdToUuid: new Map(),
3697
4711
  };
3698
4712
  return {
@@ -3762,27 +4776,76 @@ function snapshotFromUsage(usage) {
3762
4776
  cache_creation_input_tokens: usage.cache_creation_input_tokens ?? 0,
3763
4777
  };
3764
4778
  }
3765
- function createEnvForGateway(request) {
4779
+ /**
4780
+ * Adapt a legacy gateway `authenticate` request into the shared
4781
+ * {@link ProviderConfig} shape. Returns `null` when no gateway request is
4782
+ * present. `methodId` selects the protocol: `gateway-bedrock` → bedrock,
4783
+ * otherwise anthropic.
4784
+ */
4785
+ function gatewayRequestToProviderConfig(request) {
3766
4786
  if (!request?._meta) {
4787
+ return null;
4788
+ }
4789
+ return {
4790
+ apiType: request.methodId === "gateway-bedrock" ? "bedrock" : "anthropic",
4791
+ baseUrl: request._meta.gateway.baseUrl,
4792
+ headers: request._meta.gateway.headers,
4793
+ };
4794
+ }
4795
+ /**
4796
+ * Map a resolved provider config into the Claude Code env vars that redirect API
4797
+ * traffic and inject headers. Returns an empty object when routing is
4798
+ * unconfigured. The token/bypass placeholders (`" "`) are required so the CLI
4799
+ * skips its normal login/credential checks when a gateway is in use.
4800
+ */
4801
+ function createEnvForProvider(config) {
4802
+ if (!config) {
3767
4803
  return {};
3768
4804
  }
3769
- const customHeaders = Object.entries(request._meta.gateway.headers)
4805
+ const customHeaders = Object.entries(config.headers)
3770
4806
  .map(([key, value]) => `${key}: ${value}`)
3771
4807
  .join("\n");
3772
- if (request.methodId === "gateway-bedrock") {
4808
+ if (config.apiType === "bedrock") {
3773
4809
  return {
3774
4810
  CLAUDE_CODE_USE_BEDROCK: "1",
3775
4811
  AWS_BEARER_TOKEN_BEDROCK: " ", // Must be non-empty to bypass pass configuration check
3776
- ANTHROPIC_BEDROCK_BASE_URL: request._meta.gateway.baseUrl,
4812
+ ANTHROPIC_BEDROCK_BASE_URL: config.baseUrl,
4813
+ ANTHROPIC_CUSTOM_HEADERS: customHeaders,
4814
+ };
4815
+ }
4816
+ if (config.apiType === "vertex") {
4817
+ // `config.vertex` is guaranteed present for vertex by `unstable_setProvider`
4818
+ // validation; fall back to empty strings defensively.
4819
+ return {
4820
+ CLAUDE_CODE_USE_VERTEX: "1",
4821
+ ANTHROPIC_VERTEX_BASE_URL: config.baseUrl,
4822
+ ANTHROPIC_VERTEX_PROJECT_ID: config.vertex?.projectId ?? "",
4823
+ CLOUD_ML_REGION: config.vertex?.region ?? "",
3777
4824
  ANTHROPIC_CUSTOM_HEADERS: customHeaders,
3778
4825
  };
3779
4826
  }
3780
4827
  return {
3781
- ANTHROPIC_BASE_URL: request._meta.gateway.baseUrl,
4828
+ ANTHROPIC_BASE_URL: config.baseUrl,
3782
4829
  ANTHROPIC_CUSTOM_HEADERS: customHeaders,
3783
4830
  ANTHROPIC_AUTH_TOKEN: " ", // Must be specified to bypass claude login requirement
3784
4831
  };
3785
4832
  }
4833
+ /**
4834
+ * Validate a provider base URL: must be a non-empty absolute http(s) URL.
4835
+ */
4836
+ function isValidBaseUrl(baseUrl) {
4837
+ if (typeof baseUrl !== "string" || baseUrl.trim() === "") {
4838
+ return false;
4839
+ }
4840
+ let parsed;
4841
+ try {
4842
+ parsed = new URL(baseUrl);
4843
+ }
4844
+ catch {
4845
+ return false;
4846
+ }
4847
+ return parsed.protocol === "http:" || parsed.protocol === "https:";
4848
+ }
3786
4849
  /**
3787
4850
  * Build the list of permission modes the agent will advertise for the given
3788
4851
  * model. `auto` is gated by `ModelInfo.supportsAutoMode === true`, which is
@@ -4614,10 +5677,12 @@ function isTaskTool(toolName) {
4614
5677
  toolName === "TaskList" ||
4615
5678
  toolName === "TaskGet");
4616
5679
  }
4617
- /** Whether a tool's tool_use surfaces to the client as a standalone
5680
+ /** Whether the streamed tool_use path surfaces this tool as a standalone
4618
5681
  * `tool_call`. TodoWrite is rendered as a `plan` and Task* tools are
4619
5682
  * suppressed (their plan snapshot is emitted at tool_result time), so neither
4620
- * produces a tool_call. */
5683
+ * produces a streamed tool_call/tool_call_update — which means a
5684
+ * permission-surfaced tool_call for them (see `ensureToolCallEmitted`) must be
5685
+ * resolved explicitly at tool_result time. */
4621
5686
  function shouldEmitToolCall(toolName) {
4622
5687
  return toolName !== "TodoWrite" && !isTaskTool(toolName);
4623
5688
  }
@@ -4652,6 +5717,28 @@ function toolCallNotification(toolUse, rawInput, supportsTerminalOutput, cwd, re
4652
5717
  ...toolInfoFromToolUse(toolUse, supportsTerminalOutput, cwd),
4653
5718
  };
4654
5719
  }
5720
+ /** Refine a pending tool call from the complete top-level fields recovered
5721
+ * from its still-streaming input. Shares `toolInfoFromToolUse` with the
5722
+ * consolidated path but never carries `content`: content built from partial
5723
+ * input is misleading (an Edit missing its `new_string` renders as a pure
5724
+ * deletion) or invalid (a Write diff without `content` lacks the required
5725
+ * `newText`), and the consolidated message supplies it moments later. */
5726
+ function streamedInputRefinement(toolUse, input, supportsTerminalOutput, cwd) {
5727
+ // TodoWrite/Task* never surfaced a tool_call to refine (plan lane).
5728
+ if (!shouldEmitToolCall(toolUse.name)) {
5729
+ return undefined;
5730
+ }
5731
+ const { title, kind, locations } = toolInfoFromToolUse({ ...toolUse, input }, supportsTerminalOutput, cwd);
5732
+ return {
5733
+ _meta: { claudeCode: { toolName: toolUse.name } },
5734
+ toolCallId: toolUse.id,
5735
+ sessionUpdate: "tool_call_update",
5736
+ rawInput: input,
5737
+ title,
5738
+ kind,
5739
+ ...(locations ? { locations } : {}),
5740
+ };
5741
+ }
4655
5742
  /**
4656
5743
  * Convert an SDKAssistantMessage (Claude) to a SessionNotification (ACP).
4657
5744
  * Only handles text, image, and thinking chunks for now.
@@ -4683,6 +5770,15 @@ export function toAcpNotifications(content, role, sessionId, toolUseCache, clien
4683
5770
  }
4684
5771
  return [{ sessionId, update }];
4685
5772
  }
5773
+ // `tool_use_result` is message-level and carries no tool_use_id of its own:
5774
+ // it describes "the" tool_result block of the message it rode in on. If
5775
+ // several tool_result blocks were ever batched into one message it couldn't
5776
+ // be attributed, so it is only honored when the message carries exactly one.
5777
+ const toolUseResult = options?.toolUseResult !== undefined &&
5778
+ content.filter((c) => typeof c === "object" && c !== null && c.type === "tool_result")
5779
+ .length === 1
5780
+ ? options.toolUseResult
5781
+ : undefined;
4686
5782
  const output = [];
4687
5783
  // Only handle the first chunk for streaming; extend as needed for batching
4688
5784
  for (const chunk of content) {
@@ -4824,12 +5920,59 @@ export function toAcpNotifications(content, role, sessionId, toolUseCache, clien
4824
5920
  case "bash_code_execution_tool_result":
4825
5921
  case "text_editor_code_execution_tool_result":
4826
5922
  case "mcp_tool_result": {
5923
+ const wasEmitted = options?.emittedToolCalls?.has(chunk.tool_use_id) === true;
4827
5924
  options?.emittedToolCalls?.delete(chunk.tool_use_id);
4828
5925
  const toolUse = toolUseCache[chunk.tool_use_id];
4829
5926
  if (!toolUse) {
5927
+ // The permission flow may have surfaced this tool_call even though
5928
+ // its tool_use never reached the cache (e.g. the assistant message
5929
+ // carrying it was dropped by the cancelled-turn guard and a straggler
5930
+ // result landed later). Resolve the surfaced call anyway so it can't
5931
+ // stay pending in the client forever; without the cache entry the
5932
+ // tool name is unknown, so no claudeCode meta is attached.
5933
+ if (wasEmitted) {
5934
+ output.push({
5935
+ sessionId,
5936
+ update: {
5937
+ toolCallId: chunk.tool_use_id,
5938
+ sessionUpdate: "tool_call_update",
5939
+ status: "is_error" in chunk && chunk.is_error
5940
+ ? "failed"
5941
+ : "completed",
5942
+ rawOutput: chunk.content,
5943
+ },
5944
+ });
5945
+ }
4830
5946
  logger.error(`[claude-agent-acp] Got a tool result for tool use that wasn't tracked: ${chunk.tool_use_id}`);
4831
5947
  break;
4832
5948
  }
5949
+ // A permission request may have surfaced a plan-rendered (TodoWrite) or
5950
+ // suppressed (Task*) tool as a real tool_call so the request referenced
5951
+ // a tool call the client knows about (see `ensureToolCallEmitted`,
5952
+ // issue #851). The branches below never emit a tool_call_update for
5953
+ // those tools, which would leave the surfaced call pending in the
5954
+ // client forever — resolve it here. `wasEmitted` is only ever true for
5955
+ // these tools via the permission flow: the streamed plan/suppressed
5956
+ // branches don't record emissions.
5957
+ if (wasEmitted && !shouldEmitToolCall(toolUse.name)) {
5958
+ output.push({
5959
+ sessionId,
5960
+ update: {
5961
+ _meta: {
5962
+ claudeCode: {
5963
+ toolName: toolUse.name,
5964
+ ...(options?.parentToolUseId ? { parentToolUseId: options.parentToolUseId } : {}),
5965
+ },
5966
+ },
5967
+ toolCallId: chunk.tool_use_id,
5968
+ sessionUpdate: "tool_call_update",
5969
+ status: "is_error" in chunk && chunk.is_error
5970
+ ? "failed"
5971
+ : "completed",
5972
+ rawOutput: chunk.content,
5973
+ },
5974
+ });
5975
+ }
4833
5976
  if (isTaskTool(toolUse.name)) {
4834
5977
  // Headless/SDK sessions emit Task* tools instead of TodoWrite.
4835
5978
  // TaskCreate / TaskUpdate mutate the accumulated task list; TaskList
@@ -4853,7 +5996,7 @@ export function toAcpNotifications(content, role, sessionId, toolUseCache, clien
4853
5996
  }
4854
5997
  }
4855
5998
  else if (toolUse.name !== "TodoWrite") {
4856
- const { _meta: toolMeta, ...toolUpdate } = toolUpdateFromToolResult(chunk, toolUseCache[chunk.tool_use_id], supportsTerminalOutput);
5999
+ const { _meta: toolMeta, ...toolUpdate } = toolUpdateFromToolResult(chunk, toolUseCache[chunk.tool_use_id], supportsTerminalOutput, toolUseResult);
4857
6000
  // When terminal output is supported, send terminal_output as a
4858
6001
  // separate notification to match codex-acp's streaming lifecycle:
4859
6002
  // 1. tool_call → _meta.terminal_info (already sent above)
@@ -4930,35 +6073,105 @@ export function toAcpNotifications(content, role, sessionId, toolUseCache, clien
4930
6073
  }
4931
6074
  export function streamEventToAcpNotifications(message, sessionId, toolUseCache, client, logger, options) {
4932
6075
  const event = message.event;
6076
+ const streamKey = message.parent_tool_use_id ?? "";
6077
+ const streamedToolInputs = options?.streamedToolInputs;
6078
+ const forwardedOptions = {
6079
+ clientCapabilities: options?.clientCapabilities,
6080
+ parentToolUseId: message.parent_tool_use_id,
6081
+ cwd: options?.cwd,
6082
+ taskState: options?.taskState,
6083
+ emittedToolCalls: options?.emittedToolCalls,
6084
+ messageId: options?.messageId,
6085
+ };
4933
6086
  switch (event.type) {
4934
- case "content_block_start":
4935
- return toAcpNotifications([event.content_block], "assistant", sessionId, toolUseCache, client, logger, {
4936
- clientCapabilities: options?.clientCapabilities,
4937
- parentToolUseId: message.parent_tool_use_id,
4938
- cwd: options?.cwd,
4939
- taskState: options?.taskState,
4940
- emittedToolCalls: options?.emittedToolCalls,
4941
- messageId: options?.messageId,
4942
- });
4943
- case "content_block_delta":
4944
- return toAcpNotifications([event.delta], "assistant", sessionId, toolUseCache, client, logger, {
4945
- clientCapabilities: options?.clientCapabilities,
4946
- parentToolUseId: message.parent_tool_use_id,
4947
- cwd: options?.cwd,
4948
- taskState: options?.taskState,
4949
- emittedToolCalls: options?.emittedToolCalls,
4950
- messageId: options?.messageId,
4951
- });
6087
+ case "content_block_start": {
6088
+ const block = event.content_block;
6089
+ if (streamedToolInputs &&
6090
+ (block.type === "tool_use" ||
6091
+ block.type === "server_tool_use" ||
6092
+ block.type === "mcp_tool_use")) {
6093
+ let inputsForMessage = streamedToolInputs.get(streamKey);
6094
+ if (!inputsForMessage) {
6095
+ inputsForMessage = new Map();
6096
+ streamedToolInputs.set(streamKey, inputsForMessage);
6097
+ }
6098
+ inputsForMessage.set(event.index, {
6099
+ id: block.id,
6100
+ name: block.name,
6101
+ partialJson: "",
6102
+ scannedTo: 0,
6103
+ inString: false,
6104
+ escaped: false,
6105
+ objectDepth: 0,
6106
+ arrayDepth: 0,
6107
+ lastTopLevelComma: -1,
6108
+ emittedThroughComma: -1,
6109
+ });
6110
+ }
6111
+ return toAcpNotifications([block], "assistant", sessionId, toolUseCache, client, logger, forwardedOptions);
6112
+ }
6113
+ case "content_block_delta": {
6114
+ if (event.delta.type === "input_json_delta") {
6115
+ const streamedInput = streamedToolInputs?.get(streamKey)?.get(event.index);
6116
+ if (!streamedInput)
6117
+ return [];
6118
+ streamedInput.partialJson += event.delta.partial_json;
6119
+ if (scanStreamedToolInput(streamedInput)) {
6120
+ // Input complete: the consolidated assistant message replays the
6121
+ // block with its full input and refines the call there; emitting
6122
+ // here too would send a duplicate identical update.
6123
+ const inputsForMessage = streamedToolInputs?.get(streamKey);
6124
+ inputsForMessage?.delete(event.index);
6125
+ if (inputsForMessage?.size === 0)
6126
+ streamedToolInputs?.delete(streamKey);
6127
+ return [];
6128
+ }
6129
+ if (streamedInput.lastTopLevelComma <= streamedInput.emittedThroughComma) {
6130
+ return [];
6131
+ }
6132
+ streamedInput.emittedThroughComma = streamedInput.lastTopLevelComma;
6133
+ const input = recoveredToolInput(streamedInput.partialJson.slice(0, streamedInput.lastTopLevelComma));
6134
+ if (!input)
6135
+ return [];
6136
+ const supportsTerminalOutput = options?.clientCapabilities?._meta?.["terminal_output"] === true;
6137
+ const update = streamedInputRefinement(streamedInput, input, supportsTerminalOutput, options?.cwd);
6138
+ if (!update)
6139
+ return [];
6140
+ if (message.parent_tool_use_id) {
6141
+ update._meta = {
6142
+ ...update._meta,
6143
+ claudeCode: {
6144
+ ...(update._meta?.claudeCode || {}),
6145
+ parentToolUseId: message.parent_tool_use_id,
6146
+ },
6147
+ };
6148
+ }
6149
+ applyMessageId(update, options?.messageId);
6150
+ return [{ sessionId, update }];
6151
+ }
6152
+ return toAcpNotifications([event.delta], "assistant", sessionId, toolUseCache, client, logger, forwardedOptions);
6153
+ }
4952
6154
  // No content. `ping` is a Messages-API keep-alive event that the SDK's
4953
6155
  // `BetaRawMessageStreamEvent` union doesn't include even though the
4954
6156
  // wire format emits it; the `as never` cast lets us no-op it here
4955
6157
  // instead of letting it fall through to `unreachable`.
4956
6158
  case "ping":
4957
- case "message_start":
4958
6159
  case "message_delta":
6160
+ return [];
6161
+ // A message boundary ends every input stream on this lane: message_stop is
6162
+ // the normal end, and a message_start clears anything a prior message on
6163
+ // the lane left behind (e.g. a stream cut short mid-block).
6164
+ case "message_start":
4959
6165
  case "message_stop":
4960
- case "content_block_stop":
6166
+ streamedToolInputs?.delete(streamKey);
6167
+ return [];
6168
+ case "content_block_stop": {
6169
+ const inputsForMessage = streamedToolInputs?.get(streamKey);
6170
+ inputsForMessage?.delete(event.index);
6171
+ if (inputsForMessage?.size === 0)
6172
+ streamedToolInputs?.delete(streamKey);
4961
6173
  return [];
6174
+ }
4962
6175
  default:
4963
6176
  unreachable(event, logger);
4964
6177
  return [];
@@ -5016,6 +6229,9 @@ export function runAcp() {
5016
6229
  .onRequest(methods.agent.session.setMode, (ctx) => agent.setSessionMode(ctx.params))
5017
6230
  .onRequest(methods.agent.session.setConfigOption, (ctx) => agent.setSessionConfigOption(ctx.params))
5018
6231
  .onRequest(methods.agent.authenticate, (ctx) => agent.authenticate(ctx.params))
6232
+ .onRequest(methods.agent.providers.list, (ctx) => agent.unstable_listProviders(ctx.params))
6233
+ .onRequest(methods.agent.providers.set, (ctx) => agent.unstable_setProvider(ctx.params))
6234
+ .onRequest(methods.agent.providers.disable, (ctx) => agent.unstable_disableProvider(ctx.params))
5019
6235
  .onRequest(methods.agent.logout, (ctx) => agent.logout(ctx.params))
5020
6236
  .onRequest(methods.agent.session.prompt, (ctx) => runPromptWithCancellation(agent, ctx.params, ctx.signal))
5021
6237
  .onNotification(methods.agent.session.cancel, (ctx) => agent.cancel(ctx.params))
@@ -5037,12 +6253,14 @@ function commonPrefixLength(a, b) {
5037
6253
  * Anthropic 1M-context variants encode "1m" as a distinct token in the SDK
5038
6254
  * model ID (e.g., "claude-opus-4-6-1m"), which `\b1m\b` catches without also
5039
6255
  * matching things like "10m" or embedded substrings. Semantic aliases like
5040
- * `default` carry no such token in the ID, but the SDK's human-facing
5041
- * `displayName`/`description` do (e.g. "Opus 4.7 (1M context)"), so callers
5042
- * pass those too the same `\b1m\b` token appears in "1M context". The SDK's
5043
- * `ModelInfo` exposes no structured context-window field, so this text scan is
5044
- * the only pre-`result` signal available. A miss falls back to the default
5045
- * window and is corrected by `result.modelUsage` within one turn. */
6256
+ * `default` carry no such token in the ID, but their `resolvedModel` and the
6257
+ * SDK's human-facing `displayName`/`description` can (e.g.
6258
+ * "claude-opus-4-8[1m]", "Opus 4.7 (1M context)"), so callers pass those too.
6259
+ * This text scan can't catch every model some resolve to extended-context
6260
+ * models with no "1m" anywhere (e.g. `sonnet` claude-sonnet-5, natively
6261
+ * ~1M) which is why `fetchContextWindowSize` is preferred wherever a live
6262
+ * query is available. A miss falls back to the default window and is
6263
+ * corrected by `result.modelUsage` within one turn. */
5046
6264
  function inferContextWindowFromModel(...texts) {
5047
6265
  if (texts.some((text) => text != null && /\b1m\b/i.test(text)))
5048
6266
  return 1_000_000;
@@ -5052,12 +6270,7 @@ function inferContextWindowFromModel(...texts) {
5052
6270
  * `getContextUsage` control request. Unlike the per-message API usage numbers
5053
6271
  * (which only count message tokens), this `totalTokens` includes the system
5054
6272
  * prompt, tool schemas, MCP tools, and memory-file overhead — the real
5055
- * occupancy the user sees. Returns `null` on any control-request failure.
5056
- *
5057
- * Note: we deliberately do NOT use this response's window fields for `size`.
5058
- * They have been observed to under-report extended (1M) context windows, so
5059
- * the window keeps coming from `modelUsage` / `inferContextWindowFromModel`,
5060
- * which handle the 1M variants correctly. */
6273
+ * occupancy the user sees. Returns `null` on any control-request failure. */
5061
6274
  async function fetchContextUsedTokens(query, logger) {
5062
6275
  try {
5063
6276
  const usage = await query.getContextUsage();
@@ -5068,6 +6281,28 @@ async function fetchContextUsedTokens(query, logger) {
5068
6281
  return null;
5069
6282
  }
5070
6283
  }
6284
+ /** Fetch the current model's full context window (`rawMaxTokens`) via the
6285
+ * `getContextUsage` control request — the same source `/context` prints.
6286
+ * This is the only pre-`result` signal that covers semantic aliases whose
6287
+ * text carries no "1m" token (e.g. `sonnet` → claude-sonnet-5, natively
6288
+ * ~1M), so it's the primary window source at session creation and on model
6289
+ * switches; `inferContextWindowFromModel` remains the fallback. A returned
6290
+ * window is still superseded by each `result.modelUsage.contextWindow`.
6291
+ *
6292
+ * (Older CLIs under-reported extended 1M windows here — commit 20ef663
6293
+ * dropped the field for that reason — but the CLI vendored by the pinned
6294
+ * SDK reports them correctly again.) Returns `null` on any control-request
6295
+ * failure or a nonsensical (non-positive) window. */
6296
+ async function fetchContextWindowSize(query, logger) {
6297
+ try {
6298
+ const usage = await query.getContextUsage();
6299
+ return usage.rawMaxTokens > 0 ? usage.rawMaxTokens : null;
6300
+ }
6301
+ catch (error) {
6302
+ logger.error("Failed to fetch context window size from SDK:", error);
6303
+ return null;
6304
+ }
6305
+ }
5071
6306
  function parseModelConfig(raw) {
5072
6307
  if (!raw)
5073
6308
  return undefined;