clay-server 4.7.0-beta.2 → 4.7.0-beta.4

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.
@@ -1,29 +1,33 @@
1
1
  // Shared ACP Query Handle
2
2
  // -----------------------
3
3
  // Implements the YOKE QueryHandle contract with standard ACP session methods.
4
-
5
4
  var normalizer = require("./acp-event-normalizer");
6
5
  var driverRuntime = require("./acp-driver-runtime");
7
-
6
+ var queryAbort = require("./query-abort");
8
7
  var uuidCounter = 0;
9
8
  function generateUuid(vendor) {
10
9
  uuidCounter++;
11
10
  return (vendor || "acp") + "-" + Date.now().toString(36) + "-" + uuidCounter.toString(36);
12
11
  }
13
-
14
12
  function isAuthError(value) {
15
13
  return /not logged in|sign in|log ?in|authenticate|authentication|credentials|unauthorized|forbidden|\b401\b/i.test(String(value || ""));
16
14
  }
17
-
18
15
  function isApproved(decision) {
19
16
  return decision === true || (decision && decision.behavior === "allow");
20
17
  }
21
-
22
18
  function createAcpQueryHandle(acp, queryOpts) {
23
19
  queryOpts = queryOpts || {};
24
20
  var vendor = queryOpts.vendor || "acp";
25
21
  var driver = queryOpts.driver || null;
26
22
  var abortController = queryOpts.abortController;
23
+ var cancellation = queryAbort.createAbortScope();
24
+ var removeAbort = function () {};
25
+ var pendingPermissions = new Set();
26
+ var stopDelivery = Promise.resolve();
27
+ function send(method, params, timeout) {
28
+ cancellation.check();
29
+ return cancellation.wait(acp.send(method, params, timeout, cancellation.signal));
30
+ }
27
31
  var canUseTool = queryOpts.canUseTool || null;
28
32
  var state = normalizer.createEventState({ vendor: vendor, contextWindow: queryOpts.contextWindow });
29
33
  state.sessionId = queryOpts.resumeSessionId || null;
@@ -76,6 +80,7 @@ function createAcpQueryHandle(acp, queryOpts) {
76
80
  function endIterator() {
77
81
  if (iteratorDone) return;
78
82
  iteratorDone = true;
83
+ removeAbort();
79
84
  if (eventWaiting) {
80
85
  var resolve = eventWaiting;
81
86
  eventWaiting = null;
@@ -128,7 +133,19 @@ function createAcpQueryHandle(acp, queryOpts) {
128
133
  return;
129
134
  }
130
135
 
136
+ var answered = false;
137
+ function cancelPermission() {
138
+ if (answered) return;
139
+ answered = true;
140
+ pendingPermissions.delete(cancelPermission);
141
+ acp.respond(msg.id, { outcome: { outcome: "cancelled" } });
142
+ }
143
+ pendingPermissions.add(cancelPermission);
131
144
  function respond(allowed) {
145
+ if (answered) return;
146
+ if (isCancelled()) { cancelPermission(); return; }
147
+ answered = true;
148
+ pendingPermissions.delete(cancelPermission);
132
149
  var optionId = allowed ? allowId : rejectId;
133
150
  if (!optionId) {
134
151
  acp.respond(msg.id, { outcome: { outcome: "cancelled" } });
@@ -160,7 +177,7 @@ function createAcpQueryHandle(acp, queryOpts) {
160
177
  input: input,
161
178
  options: options,
162
179
  }), function() { return { toolName: toolName, input: input }; }) || { toolName: toolName, input: input };
163
- Promise.resolve(canUseTool(permission.toolName || toolName, permission.input || input, {})).then(function(decision) {
180
+ Promise.resolve(canUseTool(permission.toolName || toolName, permission.input || input, { signal: cancellation.signal })).then(function(decision) {
164
181
  respond(isApproved(decision));
165
182
  }).catch(function(err) {
166
183
  console.error("[yoke/" + vendor + "] canUseTool error:", err.message || err);
@@ -193,7 +210,7 @@ function createAcpQueryHandle(acp, queryOpts) {
193
210
  if (!state.sessionId || !acp.started || value === undefined || value === null) return Promise.resolve();
194
211
  var option = configOptionFor(category, fallbackId);
195
212
  if (option) {
196
- return acp.send("session/set_config_option", {
213
+ return send("session/set_config_option", {
197
214
  sessionId: state.sessionId,
198
215
  configId: option.id,
199
216
  value: value,
@@ -202,7 +219,7 @@ function createAcpQueryHandle(acp, queryOpts) {
202
219
  });
203
220
  }
204
221
  if (category === "mode") {
205
- return acp.send("session/set_mode", { sessionId: state.sessionId, modeId: value }, 15000);
222
+ return send("session/set_mode", { sessionId: state.sessionId, modeId: value }, 15000);
206
223
  }
207
224
  return Promise.reject(new Error("ACP session does not expose configuration option: " + fallbackId));
208
225
  }
@@ -230,7 +247,7 @@ function createAcpQueryHandle(acp, queryOpts) {
230
247
  if (safeModeValue([modes.currentModeId])) return Promise.resolve();
231
248
  var safeMode = safeModeValue(modes.availableModes || []);
232
249
  if (!safeMode) return Promise.reject(new Error("ACP agent started in an unsafe permission mode and exposed no supervised mode"));
233
- return acp.send("session/set_mode", { sessionId: state.sessionId, modeId: safeMode }, 15000);
250
+ return send("session/set_mode", { sessionId: state.sessionId, modeId: safeMode }, 15000);
234
251
  }
235
252
  if (driver && driver.permissionModeGuaranteed === true) return Promise.resolve();
236
253
  return Promise.reject(new Error("ACP agent did not expose a verifiable supervised permission mode"));
@@ -302,6 +319,7 @@ function createAcpQueryHandle(acp, queryOpts) {
302
319
  }
303
320
 
304
321
  function applySessionResult(result) {
322
+ cancellation.check();
305
323
  result = result || {};
306
324
  if (result.sessionId) state.sessionId = result.sessionId;
307
325
  if (Array.isArray(result.configOptions)) state.configOptions = result.configOptions;
@@ -313,7 +331,7 @@ function createAcpQueryHandle(acp, queryOpts) {
313
331
  async function openDefaultSession(handlerEntry) {
314
332
  if (state.sessionId && queryOpts.canResumeSession) {
315
333
  handlerEntry.sessionId = state.sessionId;
316
- var resumed = await acp.send("session/resume", buildSessionParams("resume", {
334
+ var resumed = await send("session/resume", buildSessionParams("resume", {
317
335
  sessionId: state.sessionId,
318
336
  cwd: queryOpts.cwd,
319
337
  mcpServers: queryOpts.mcpServers || [],
@@ -324,7 +342,7 @@ function createAcpQueryHandle(acp, queryOpts) {
324
342
  handlerEntry.sessionId = state.sessionId;
325
343
  state.loadingSession = true;
326
344
  try {
327
- var loaded = await acp.send("session/load", buildSessionParams("load", {
345
+ var loaded = await send("session/load", buildSessionParams("load", {
328
346
  sessionId: state.sessionId,
329
347
  cwd: queryOpts.cwd,
330
348
  mcpServers: queryOpts.mcpServers || [],
@@ -336,7 +354,7 @@ function createAcpQueryHandle(acp, queryOpts) {
336
354
  }
337
355
 
338
356
  state.sessionId = null;
339
- var result = await acp.send("session/new", buildSessionParams("new", {
357
+ var result = await send("session/new", buildSessionParams("new", {
340
358
  cwd: queryOpts.cwd,
341
359
  mcpServers: queryOpts.mcpServers || [],
342
360
  }), 60000);
@@ -355,7 +373,7 @@ function createAcpQueryHandle(acp, queryOpts) {
355
373
  sessionId: state.sessionId,
356
374
  prompt: asPrompt(message),
357
375
  });
358
- return acp.send("session/prompt", params, 30 * 60 * 1000);
376
+ return send("session/prompt", params, 30 * 60 * 1000);
359
377
  });
360
378
  }
361
379
 
@@ -363,17 +381,17 @@ function createAcpQueryHandle(acp, queryOpts) {
363
381
  var currentMessage = prependSystemPrompt(initialMessage);
364
382
  var handlerEntry = acp.addHandler(handleServerEvent);
365
383
  try {
366
- var sessionResult = await openSession(handlerEntry);
384
+ var sessionResult = await cancellation.wait(openSession(handlerEntry));
367
385
  handlerEntry.sessionId = state.sessionId;
368
- await driverRuntime.callAsync(driver, "afterSessionOpen", driverContext({ sessionResult: sessionResult }), function() {});
369
- await ensureSafePermissionMode();
386
+ await cancellation.wait(driverRuntime.callAsync(driver, "afterSessionOpen", driverContext({ sessionResult: sessionResult }), function() {}));
387
+ await cancellation.wait(ensureSafePermissionMode());
370
388
  if (state.model && state.model !== "auto") await setModel(state.model);
371
389
  if (queryOpts.mode) await setConfig("mode", "mode", queryOpts.mode);
372
390
 
373
391
  while (!isCancelled()) {
374
392
  normalizer.resetTurnState(state);
375
393
  pushEvent({ yokeType: "turn_start", uuid: generateUuid(vendor), messageType: "user" });
376
- var result = await prompt(currentMessage);
394
+ var result = await cancellation.wait(prompt(currentMessage));
377
395
  var closing = normalizer.closeOpenBlocks(state);
378
396
  for (var i = 0; i < closing.length; i++) pushEvent(closing[i]);
379
397
  if (isCancelled() || (result && result.stopReason === "cancelled")) {
@@ -395,6 +413,7 @@ function createAcpQueryHandle(acp, queryOpts) {
395
413
  : { yokeType: "error", text: message });
396
414
  }
397
415
  } finally {
416
+ await stopDelivery;
398
417
  acp.removeHandler(handlerEntry);
399
418
  state.done = true;
400
419
  endIterator();
@@ -448,36 +467,29 @@ function createAcpQueryHandle(acp, queryOpts) {
448
467
  }; });
449
468
  },
450
469
  abort: function() {
470
+ if (state.aborted || iteratorDone) return;
451
471
  state.aborted = true;
452
- driverRuntime.call(driver, "cancel", driverContext(), function() {
453
- if (state.sessionId && acp.started) acp.notify("session/cancel", { sessionId: state.sessionId });
454
- });
455
- endIterator();
456
- },
457
- close: function() {
458
- messageQueueEnded = true;
459
- if (messageWaiting) {
460
- var resolve = messageWaiting;
461
- messageWaiting = null;
462
- resolve(null);
463
- }
464
- endIterator();
472
+ cancellation.stop();
473
+ handle.endInput();
474
+ messageQueue = [];
475
+ pendingPermissions.forEach(function (cancel) { try { cancel(); } catch (e) {} });
476
+ stopDelivery = driverRuntime.callAsync(driver, "cancel", driverContext(), function() {
477
+ if (state.sessionId && (!acp.started || acp.notify("session/cancel", { sessionId: state.sessionId }) === false)) {
478
+ throw new Error("ACP transport is disconnected");
479
+ }
480
+ }).catch(function (error) {
481
+ pushEvent({ yokeType: "error", text: "Cancellation could not be delivered: " + error.message });
482
+ }).finally(endIterator);
483
+ return stopDelivery;
465
484
  },
485
+ close: function() { handle.endInput(); endIterator(); },
466
486
  endInput: function() {
467
487
  messageQueueEnded = true;
468
- if (messageWaiting) {
469
- var resolve = messageWaiting;
470
- messageWaiting = null;
471
- resolve(null);
472
- }
488
+ if (messageWaiting) { var resolve = messageWaiting; messageWaiting = null; resolve(null); }
473
489
  },
474
490
  };
475
491
 
476
- if (abortController && abortController.signal) {
477
- abortController.signal.addEventListener("abort", function() {
478
- if (!state.aborted) handle.abort();
479
- }, { once: true });
480
- }
492
+ removeAbort = queryAbort.bindAbort(abortController, function () { handle.abort(); });
481
493
  return handle;
482
494
  }
483
495
 
@@ -1,3 +1,4 @@
1
+ var bindAbort = require("../query-abort").bindAbort;
1
2
  var childProcess = require("child_process");
2
3
  var readline = require("readline");
3
4
  var skillDiscovery = require("../skill-discovery");
@@ -75,6 +76,9 @@ function toolName(name) {
75
76
 
76
77
  function createAntigravityQueryHandle(binaryPath, queryOpts, onFinished) {
77
78
  var processHandle = null;
79
+ var removeAbort = function () {};
80
+ var killTimer = null;
81
+ var processExited = false;
78
82
  var outputReader = null;
79
83
  var events = [];
80
84
  var eventWaiter = null;
@@ -115,6 +119,7 @@ function createAntigravityQueryHandle(binaryPath, queryOpts, onFinished) {
115
119
  function endEvents() {
116
120
  if (ended) return;
117
121
  ended = true;
122
+ removeAbort();
118
123
  if (eventWaiter) {
119
124
  var resolve = eventWaiter;
120
125
  eventWaiter = null;
@@ -252,6 +257,8 @@ function createAntigravityQueryHandle(binaryPath, queryOpts, onFinished) {
252
257
  endEvents();
253
258
  });
254
259
  processHandle.on("exit", function(code, signal) {
260
+ processExited = true;
261
+ if (killTimer) clearTimeout(killTimer);
255
262
  if (!ended && activeTurn) {
256
263
  var message = stderr.trim() || "Antigravity CLI exited before completing the turn";
257
264
  pushEvent(isAuthError(message)
@@ -309,8 +316,24 @@ function createAntigravityQueryHandle(binaryPath, queryOpts, onFinished) {
309
316
  },
310
317
  abort: function() {
311
318
  if (ended) return;
319
+ inputEnded = true;
320
+ messages = [];
312
321
  pushEvent({ yokeType: "interrupted" });
313
- if (processHandle) processHandle.kill("SIGINT");
322
+ if (processHandle && !processExited) {
323
+ try { processHandle.kill("SIGINT"); } catch (error) {
324
+ pushEvent({ yokeType: "error", text: "Could not interrupt Antigravity: " + error.message });
325
+ }
326
+ if (!processExited) {
327
+ killTimer = setTimeout(function () {
328
+ if (!processExited) {
329
+ try { processHandle.kill("SIGKILL"); } catch (error) {
330
+ console.error("[yoke/antigravity] Could not terminate stopped query:", error.message);
331
+ }
332
+ }
333
+ }, 5000);
334
+ killTimer.unref();
335
+ }
336
+ }
314
337
  endEvents();
315
338
  },
316
339
  close: function() {
@@ -319,6 +342,7 @@ function createAntigravityQueryHandle(binaryPath, queryOpts, onFinished) {
319
342
  endEvents();
320
343
  },
321
344
  };
345
+ removeAbort = bindAbort(queryOpts.abortController, function () { handle.abort(); });
322
346
  return handle;
323
347
  }
324
348
 
@@ -4,6 +4,7 @@
4
4
  // This is the ONLY file (besides claude-worker.js) that imports the SDK.
5
5
  // Also manages worker processes for OS-level user isolation.
6
6
 
7
+ var bindQueryAbort = require("../query-abort").bindAbort;
7
8
  var path = require("path");
8
9
  var fs = require("fs");
9
10
  var os = require("os");
@@ -576,6 +577,7 @@ function normalizeBackgroundTasks(tasks) {
576
577
  function createQueryHandle(rawQuery, messageQueue, abortController) {
577
578
  var correlation = claudeCorrelation.createCorrelation();
578
579
  var closed = false;
580
+ var removeAbort = function () {};
579
581
  var handle = {
580
582
  // Opaque adapter state (null for in-process queries)
581
583
  _adapterState: null,
@@ -586,11 +588,12 @@ function createQueryHandle(rawQuery, messageQueue, abortController) {
586
588
  return {
587
589
  next: function() {
588
590
  return rawIter.next().then(function(result) {
589
- if (result.done) closed = true;
591
+ if (result.done) { closed = true; removeAbort(); }
590
592
  if (result.done) return result;
591
593
  return { value: correlation.result(result.value, flattenEvent(result.value)), done: false };
592
594
  }, function(err) {
593
595
  closed = true;
596
+ removeAbort();
594
597
  throw err;
595
598
  });
596
599
  },
@@ -684,6 +687,7 @@ function createQueryHandle(rawQuery, messageQueue, abortController) {
684
687
 
685
688
  abort: function() {
686
689
  closed = true;
690
+ try { messageQueue.end(); } catch (e) {}
687
691
  if (abortController) {
688
692
  try { abortController.abort(); } catch (e) {}
689
693
  }
@@ -691,6 +695,7 @@ function createQueryHandle(rawQuery, messageQueue, abortController) {
691
695
 
692
696
  close: function() {
693
697
  closed = true;
698
+ removeAbort();
694
699
  try { messageQueue.end(); } catch (e) {}
695
700
  if (rawQuery && typeof rawQuery.close === "function") {
696
701
  try { rawQuery.close(); } catch (e) {}
@@ -712,6 +717,10 @@ function createQueryHandle(rawQuery, messageQueue, abortController) {
712
717
  },
713
718
  };
714
719
 
720
+ removeAbort = bindQueryAbort(abortController, function () {
721
+ closed = true;
722
+ try { messageQueue.end(); } catch (e) {}
723
+ });
715
724
  return handle;
716
725
  }
717
726
 
@@ -1043,6 +1052,7 @@ function serializeWorkerValue(value, seen) {
1043
1052
  // query identically to an in-process query.
1044
1053
 
1045
1054
  function createWorkerQueryHandle(worker, canUseTool, onElicitation, callMcpTool, onUserDialog, onUserInputRequest, onModelSwitch, preToolUsePolicy) {
1055
+ var interactionAbort = new AbortController();
1046
1056
  var contextRequests = claudeContextUsage.createRequests(worker);
1047
1057
  var correlation = claudeCorrelation.createCorrelation();
1048
1058
  // Async iterable state
@@ -1085,6 +1095,7 @@ function createWorkerQueryHandle(worker, canUseTool, onElicitation, callMcpTool,
1085
1095
  }
1086
1096
 
1087
1097
  function endIter() {
1098
+ interactionAbort.abort();
1088
1099
  contextRequests.close();
1089
1100
  if (iterEnded) return;
1090
1101
  iterEnded = true;
@@ -1096,6 +1107,7 @@ function createWorkerQueryHandle(worker, canUseTool, onElicitation, callMcpTool,
1096
1107
  }
1097
1108
 
1098
1109
  function errorIter(err) {
1110
+ interactionAbort.abort();
1099
1111
  contextRequests.close();
1100
1112
  if (iterEnded) return;
1101
1113
  iterEnded = true;
@@ -1122,8 +1134,9 @@ function createWorkerQueryHandle(worker, canUseTool, onElicitation, callMcpTool,
1122
1134
  decisionReason: msg.decisionReason,
1123
1135
  defaultToNo: msg.defaultToNo === true,
1124
1136
  suppressAlwaysAllowRule: msg.suppressAlwaysAllowRule === true,
1125
- signal: { addEventListener: function() {} },
1137
+ signal: interactionAbort.signal,
1126
1138
  }).then(function(result) {
1139
+ if (interactionAbort.signal.aborted) result = { behavior: "deny", message: "Query cancelled" };
1127
1140
  worker.send({ type: "permission_response", requestId: msg.requestId, result: result });
1128
1141
  }).catch(function(e) {
1129
1142
  console.error("[yoke/claude] permission_response send failed:", e.message || e);
@@ -1149,8 +1162,9 @@ function createWorkerQueryHandle(worker, canUseTool, onElicitation, callMcpTool,
1149
1162
  if (onUserInputRequest) {
1150
1163
  claudeUserInput.ask(onUserInputRequest, msg.input, {
1151
1164
  toolUseID: msg.toolUseId,
1152
- signal: { addEventListener: function() {} },
1165
+ signal: interactionAbort.signal,
1153
1166
  }).then(function(result) {
1167
+ if (interactionAbort.signal.aborted) return;
1154
1168
  worker.send({ type: "ask_user_response", toolUseId: msg.toolUseId, result: result });
1155
1169
  }).catch(function(e) {
1156
1170
  console.error("[yoke/claude] ask_user_response send failed:", e.message || e);
@@ -1171,12 +1185,13 @@ function createWorkerQueryHandle(worker, canUseTool, onElicitation, callMcpTool,
1171
1185
  var workerElicitation = onUserInputRequest
1172
1186
  ? claudeUserInput.elicitation(onUserInputRequest, workerElicitationRequest, {
1173
1187
  requestId: msg.requestId,
1174
- signal: { addEventListener: function() {} },
1188
+ signal: interactionAbort.signal,
1175
1189
  })
1176
1190
  : onElicitation(workerElicitationRequest, {
1177
- signal: { addEventListener: function() {} },
1191
+ signal: interactionAbort.signal,
1178
1192
  });
1179
1193
  workerElicitation.then(function(result) {
1194
+ if (interactionAbort.signal.aborted) return;
1180
1195
  worker.send({ type: "elicitation_response", requestId: msg.requestId, result: result });
1181
1196
  }).catch(function(e) {
1182
1197
  console.error("[yoke/claude] elicitation_response send failed:", e.message || e);
@@ -1191,8 +1206,9 @@ function createWorkerQueryHandle(worker, canUseTool, onElicitation, callMcpTool,
1191
1206
  payload: msg.payload || {},
1192
1207
  toolUseID: msg.toolUseId || undefined,
1193
1208
  }, {
1194
- signal: { addEventListener: function() {} },
1209
+ signal: interactionAbort.signal,
1195
1210
  }).then(function(result) {
1211
+ if (interactionAbort.signal.aborted) return;
1196
1212
  worker.send({ type: "user_dialog_response", requestId: msg.requestId, result: result });
1197
1213
  }).catch(function(e) {
1198
1214
  console.error("[yoke/claude] user_dialog_response send failed:", e.message || e);
@@ -1327,7 +1343,7 @@ function createWorkerQueryHandle(worker, canUseTool, onElicitation, callMcpTool,
1327
1343
  getSubmittedMessageCount: correlation.submitted,
1328
1344
 
1329
1345
  pushMessage: function(text, images) {
1330
- if (iterEnded || (worker.process && (worker.process.killed || worker.process.exitCode != null))) return false;
1346
+ if (iterEnded || worker._abortSent || (worker.process && (worker.process.killed || worker.process.exitCode != null))) return false;
1331
1347
  var content = [];
1332
1348
  if (images && images.length > 0) {
1333
1349
  for (var i = 0; i < images.length; i++) {
@@ -1417,8 +1433,10 @@ function createWorkerQueryHandle(worker, canUseTool, onElicitation, callMcpTool,
1417
1433
  },
1418
1434
 
1419
1435
  abort: function() {
1436
+ if (iterEnded || worker._abortSent) return;
1420
1437
  console.log("[yoke/claude] ABORT sent to worker pid=" + (worker.process ? worker.process.pid : "?"));
1421
1438
  worker._abortSent = true;
1439
+ interactionAbort.abort();
1422
1440
  try { worker.send({ type: "abort" }); } catch (e) {}
1423
1441
  // If the worker doesn't finish within 5s (e.g. subagent stuck), force-kill it.
1424
1442
  // The worker exit handler will dispatch a fallback query_error and send done.
@@ -3,6 +3,8 @@
3
3
  // Implements the YOKE interface using codex app-server protocol.
4
4
  // Bidirectional JSON-RPC over stdin/stdout enables interactive approval flows.
5
5
 
6
+ var createTurnEvents = require("../codex-turn-events").createTurnEvents;
7
+ var bindQueryAbort = require("../query-abort").bindAbort;
6
8
  var path = require("path");
7
9
  var fs = require("fs");
8
10
  var { CodexAppServer } = require("../codex-app-server");
@@ -11,6 +13,7 @@ var skillDiscovery = require("../skill-discovery");
11
13
  var { resolveOsUserInfo } = require("../../os-users");
12
14
  var backgroundTasks = require("../codex-background-tasks");
13
15
  var codexUserInput = require("../codex-user-input");
16
+ var createTurnStop = require("../codex-turn-stop").createTurnStop;
14
17
 
15
18
  // --- Reasoning defaults ---
16
19
  // Reasoning summaries are the only readable thinking channel Codex offers:
@@ -812,6 +815,7 @@ function createEventState(model) {
812
815
 
813
816
  function createCodexQueryHandle(appServer, queryOpts) {
814
817
  var abortController = queryOpts.abortController;
818
+ var removeAbort = function () {};
815
819
  var promptParts = [queryOpts.systemPrompt, queryOpts.appendSystemPrompt].filter(function (part) { return !!part; });
816
820
  var systemPrompt = promptParts.join("\n\n");
817
821
  var canUseTool = queryOpts.canUseTool || null;
@@ -826,6 +830,11 @@ function createCodexQueryHandle(appServer, queryOpts) {
826
830
  }
827
831
 
828
832
  var state = createEventState(queryOpts.model);
833
+ var turnEvents = createTurnEvents(handleServerEvent);
834
+ var turnStop = createTurnStop(appServer, function (error) {
835
+ console.error("[yoke/codex] turn interrupt failed:", error.message || error);
836
+ pushEvent({ yokeType: "error", text: "Codex cancellation could not be confirmed. The remote turn may still be running." });
837
+ });
829
838
 
830
839
  // Internal event buffer for async iterator
831
840
  var eventBuffer = [];
@@ -864,6 +873,7 @@ function createCodexQueryHandle(appServer, queryOpts) {
864
873
  }
865
874
 
866
875
  function endIterator() {
876
+ removeAbort();
867
877
  iteratorDone = true;
868
878
  if (eventWaiting) {
869
879
  var resolve = eventWaiting;
@@ -912,6 +922,13 @@ function createCodexQueryHandle(appServer, queryOpts) {
912
922
  // Ignore events from other threads (app-server is shared across sessions)
913
923
  if (params.threadId && state.threadId && params.threadId !== state.threadId) return;
914
924
 
925
+ if (!turnEvents.accept(msg)) return;
926
+
927
+ // Capture cancellation identity even if Stop arrived before turn/start replied.
928
+ var eventTurnId = params.turnId || params.turn_id || (params.turn && params.turn.id);
929
+ if (method === "turn/started") turnStop.started(eventTurnId);
930
+ if (method === "turn/completed" || method === "turn/failed") turnStop.completed(eventTurnId);
931
+
915
932
  // After abort, only let turn-ending events through
916
933
  if (isCancelled() && method !== "turn/completed" && method !== "turn/failed" && method !== "serverRequest/resolved" && method !== "thread/status/changed") return;
917
934
 
@@ -1089,6 +1106,7 @@ function createCodexQueryHandle(appServer, queryOpts) {
1089
1106
 
1090
1107
  // Resolve turn promise when turn ends
1091
1108
  if (method === "turn/completed" || method === "turn/failed") {
1109
+ turnEvents.finish();
1092
1110
  if (turnResolve) {
1093
1111
  var resolve = turnResolve;
1094
1112
  turnResolve = null;
@@ -1204,6 +1222,7 @@ function createCodexQueryHandle(appServer, queryOpts) {
1204
1222
  while (!isCancelled()) {
1205
1223
  // Reset per-turn state
1206
1224
  state.turnStarted = false;
1225
+ state.currentTurnId = null;
1207
1226
  state.textBlocks = {};
1208
1227
  state.textLengths = {};
1209
1228
  state.thinkingBlocks = {};
@@ -1243,11 +1262,22 @@ function createCodexQueryHandle(appServer, queryOpts) {
1243
1262
  input = input.concat(skillItemsToInject);
1244
1263
  }
1245
1264
 
1246
- await appServer.send("turn/start", {
1247
- threadId: state.threadId,
1248
- input: input,
1249
- model: state.model,
1250
- }, 60000);
1265
+ turnStop.begin(state.threadId);
1266
+ turnEvents.begin();
1267
+ try {
1268
+ var turnResult = await appServer.send("turn/start", {
1269
+ threadId: state.threadId,
1270
+ input: input,
1271
+ model: state.model,
1272
+ }, 60000);
1273
+ var startedTurnId = turnResult && turnResult.turn && turnResult.turn.id;
1274
+ turnStop.startSettled(startedTurnId);
1275
+ turnEvents.ready(startedTurnId);
1276
+ } catch (startError) {
1277
+ turnEvents.finish();
1278
+ turnStop.startSettled(null);
1279
+ throw startError;
1280
+ }
1251
1281
 
1252
1282
  // Wait for turn to complete
1253
1283
  await turnPromise;
@@ -1348,19 +1378,23 @@ function createCodexQueryHandle(appServer, queryOpts) {
1348
1378
  abort: function() {
1349
1379
  console.log("[yoke/codex] handle.abort() called, threadId=" + state.threadId + " already aborted=" + state.aborted);
1350
1380
  state.aborted = true;
1351
- // Send turn/interrupt to stop the server-side turn
1352
- if (state.threadId && appServer.started) {
1353
- appServer.send("turn/interrupt", { threadId: state.threadId }, 5000).catch(function() {});
1354
- }
1355
- // End iterator immediately. sdk-bridge's post-loop code checks
1356
- // session.taskStopRequested and sends the interrupted message + done.
1357
- // This matches Claude's abort pattern.
1358
- if (turnResolve) {
1359
- var resolve = turnResolve;
1360
- turnResolve = null;
1361
- resolve();
1362
- }
1363
- endIterator();
1381
+ messageQueueEnded = true;
1382
+ messageQueue = [];
1383
+ if (messageWaiting) {
1384
+ var releaseMessage = messageWaiting;
1385
+ messageWaiting = null;
1386
+ releaseMessage(null);
1387
+ }
1388
+ // Keep the iterator alive until the exact interrupt settles so failures
1389
+ // reach the transcript instead of being silently reported as a stop.
1390
+ return turnStop.abort().then(function () {
1391
+ if (turnResolve) {
1392
+ var resolve = turnResolve;
1393
+ turnResolve = null;
1394
+ resolve();
1395
+ }
1396
+ endIterator();
1397
+ });
1364
1398
  },
1365
1399
 
1366
1400
  close: function() {
@@ -1383,12 +1417,9 @@ function createCodexQueryHandle(appServer, queryOpts) {
1383
1417
  },
1384
1418
  };
1385
1419
 
1386
- // Listen for external abort (sdk-bridge's stopTask calls session.abortController.abort())
1387
- if (abortController && abortController.signal) {
1388
- abortController.signal.addEventListener("abort", function() {
1389
- if (!state.aborted) handle.abort();
1390
- }, { once: true });
1391
- }
1420
+ removeAbort = bindQueryAbort(abortController, function () {
1421
+ if (!state.aborted) handle.abort();
1422
+ });
1392
1423
 
1393
1424
  return handle;
1394
1425
  }