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.
- package/lib/project-pair-result-outbox.js +1 -0
- package/lib/project-pair-result-recovery.js +15 -2
- package/lib/public/app.js +2 -0
- package/lib/public/css/input.css +27 -0
- package/lib/public/css/messages.css +15 -8
- package/lib/public/index.html +3 -0
- package/lib/public/modules/app-connection.js +23 -2
- package/lib/public/modules/app-favicon.js +1 -0
- package/lib/public/modules/app-messages.js +2 -1
- package/lib/public/modules/app-rendering.js +16 -10
- package/lib/public/modules/generation-stop.js +24 -0
- package/lib/public/modules/pair-result-status.js +27 -2
- package/lib/sdk-bridge.js +17 -3
- package/lib/sessions.js +2 -0
- package/lib/yoke/acp-process-manager.js +25 -15
- package/lib/yoke/acp-query-handle.js +52 -40
- package/lib/yoke/adapters/antigravity.js +25 -1
- package/lib/yoke/adapters/claude.js +25 -7
- package/lib/yoke/adapters/codex.js +55 -24
- package/lib/yoke/adapters/kiro.js +30 -19
- package/lib/yoke/codex-turn-events.js +32 -0
- package/lib/yoke/codex-turn-stop.js +59 -0
- package/lib/yoke/query-abort.js +40 -0
- package/lib/yoke/user-input-guidance.js +11 -0
- package/lib/yoke/user-input.js +1 -1
- package/package.json +1 -1
|
@@ -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
|
|
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
|
|
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
|
|
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
|
|
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
|
|
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
|
|
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
|
|
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
|
-
|
|
453
|
-
|
|
454
|
-
|
|
455
|
-
|
|
456
|
-
|
|
457
|
-
|
|
458
|
-
|
|
459
|
-
|
|
460
|
-
|
|
461
|
-
|
|
462
|
-
|
|
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
|
-
|
|
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)
|
|
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:
|
|
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:
|
|
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:
|
|
1188
|
+
signal: interactionAbort.signal,
|
|
1175
1189
|
})
|
|
1176
1190
|
: onElicitation(workerElicitationRequest, {
|
|
1177
|
-
signal:
|
|
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:
|
|
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
|
-
|
|
1247
|
-
|
|
1248
|
-
|
|
1249
|
-
|
|
1250
|
-
|
|
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
|
-
|
|
1352
|
-
|
|
1353
|
-
|
|
1354
|
-
|
|
1355
|
-
|
|
1356
|
-
|
|
1357
|
-
|
|
1358
|
-
|
|
1359
|
-
|
|
1360
|
-
|
|
1361
|
-
|
|
1362
|
-
|
|
1363
|
-
|
|
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
|
-
|
|
1387
|
-
|
|
1388
|
-
|
|
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
|
}
|