@evident-ai/cli 3.1.1-dev.c64b404 → 3.1.1-dev.caa3277
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/README.md +19 -11
- package/dist/index.js +1636 -171
- package/dist/index.js.map +1 -1
- package/package.json +1 -1
package/dist/index.js
CHANGED
|
@@ -11,8 +11,8 @@ import chalk2 from "chalk";
|
|
|
11
11
|
|
|
12
12
|
// src/lib/config.ts
|
|
13
13
|
import Conf from "conf";
|
|
14
|
-
import {
|
|
15
|
-
import {
|
|
14
|
+
import { chmodSync, existsSync, statSync } from "fs";
|
|
15
|
+
import { dirname } from "path";
|
|
16
16
|
var PRODUCTION_API_URL = "https://api.production.evident.run/v1";
|
|
17
17
|
var PRODUCTION_TUNNEL_URL = "wss://tunnel.production.evident.run";
|
|
18
18
|
var defaults = {
|
|
@@ -47,8 +47,35 @@ var credentials = new Conf({
|
|
|
47
47
|
projectName: "evident",
|
|
48
48
|
projectSuffix: "",
|
|
49
49
|
configName: "credentials",
|
|
50
|
-
defaults: {}
|
|
50
|
+
defaults: {},
|
|
51
|
+
configFileMode: 384
|
|
51
52
|
});
|
|
53
|
+
var CREDENTIALS_FILE_MODE = 384;
|
|
54
|
+
var CREDENTIALS_DIR_MODE = 448;
|
|
55
|
+
var permissionWarningEmitted = false;
|
|
56
|
+
function hardenCredentialsPermissions() {
|
|
57
|
+
if (process.platform === "win32") {
|
|
58
|
+
return;
|
|
59
|
+
}
|
|
60
|
+
const file = credentials.path;
|
|
61
|
+
for (const [path, mode] of [
|
|
62
|
+
[file, CREDENTIALS_FILE_MODE],
|
|
63
|
+
[dirname(file), CREDENTIALS_DIR_MODE]
|
|
64
|
+
]) {
|
|
65
|
+
try {
|
|
66
|
+
if (existsSync(path) && (statSync(path).mode & 511) !== mode) {
|
|
67
|
+
chmodSync(path, mode);
|
|
68
|
+
}
|
|
69
|
+
} catch (err) {
|
|
70
|
+
if (!permissionWarningEmitted) {
|
|
71
|
+
permissionWarningEmitted = true;
|
|
72
|
+
console.error(
|
|
73
|
+
`[config] could not restrict permissions on ${path}; the credentials file may be readable by other users on this machine: ${err instanceof Error ? err.message : String(err)}`
|
|
74
|
+
);
|
|
75
|
+
}
|
|
76
|
+
}
|
|
77
|
+
}
|
|
78
|
+
}
|
|
52
79
|
function getApiUrlConfig() {
|
|
53
80
|
return getApiUrl();
|
|
54
81
|
}
|
|
@@ -59,6 +86,7 @@ function credentialsKey() {
|
|
|
59
86
|
return getApiUrl();
|
|
60
87
|
}
|
|
61
88
|
function getCredentials() {
|
|
89
|
+
hardenCredentialsPermissions();
|
|
62
90
|
const byEndpoint = credentials.get("byEndpoint") ?? {};
|
|
63
91
|
return byEndpoint[credentialsKey()] ?? {};
|
|
64
92
|
}
|
|
@@ -70,14 +98,17 @@ function setCredentials(creds) {
|
|
|
70
98
|
expiresAt: creds.expiresAt
|
|
71
99
|
};
|
|
72
100
|
credentials.set("byEndpoint", byEndpoint);
|
|
101
|
+
hardenCredentialsPermissions();
|
|
73
102
|
}
|
|
74
103
|
function clearCredentials() {
|
|
75
104
|
const byEndpoint = credentials.get("byEndpoint") ?? {};
|
|
76
105
|
delete byEndpoint[credentialsKey()];
|
|
77
106
|
credentials.set("byEndpoint", byEndpoint);
|
|
107
|
+
hardenCredentialsPermissions();
|
|
78
108
|
}
|
|
79
109
|
function clearAllCredentials() {
|
|
80
110
|
credentials.clear();
|
|
111
|
+
hardenCredentialsPermissions();
|
|
81
112
|
}
|
|
82
113
|
function getCliName() {
|
|
83
114
|
const argv1 = process.argv[1] || "";
|
|
@@ -236,16 +267,28 @@ async function getToken() {
|
|
|
236
267
|
}
|
|
237
268
|
return null;
|
|
238
269
|
}
|
|
270
|
+
function toError(err) {
|
|
271
|
+
return err instanceof Error ? err : new Error(String(err));
|
|
272
|
+
}
|
|
239
273
|
async function deleteToken(options = {}) {
|
|
240
274
|
const keytar = await getKeytar();
|
|
275
|
+
const failures = [];
|
|
241
276
|
if (keytar) {
|
|
242
277
|
if (options.all) {
|
|
243
|
-
|
|
278
|
+
let accounts = [];
|
|
279
|
+
try {
|
|
280
|
+
accounts = await keytar.findCredentials(SERVICE_NAME);
|
|
281
|
+
} catch (err) {
|
|
282
|
+
failures.push({ type: "enumerate", error: toError(err) });
|
|
283
|
+
}
|
|
244
284
|
await Promise.all(
|
|
245
|
-
|
|
246
|
-
|
|
247
|
-
|
|
248
|
-
|
|
285
|
+
accounts.map(async (entry) => {
|
|
286
|
+
try {
|
|
287
|
+
await keytar.deletePassword(SERVICE_NAME, entry.account);
|
|
288
|
+
} catch (err) {
|
|
289
|
+
failures.push({ type: "delete", account: entry.account, error: toError(err) });
|
|
290
|
+
}
|
|
291
|
+
})
|
|
249
292
|
);
|
|
250
293
|
} else {
|
|
251
294
|
await keytar.deletePassword(SERVICE_NAME, keychainAccount());
|
|
@@ -256,6 +299,7 @@ async function deleteToken(options = {}) {
|
|
|
256
299
|
} else {
|
|
257
300
|
clearCredentials();
|
|
258
301
|
}
|
|
302
|
+
return { failures };
|
|
259
303
|
}
|
|
260
304
|
|
|
261
305
|
// src/utils/ui.ts
|
|
@@ -285,14 +329,14 @@ function blank() {
|
|
|
285
329
|
console.log();
|
|
286
330
|
}
|
|
287
331
|
function waitForEnter(prompt = "Press Enter to continue...") {
|
|
288
|
-
return new Promise((
|
|
332
|
+
return new Promise((resolve3) => {
|
|
289
333
|
process.stdout.write(chalk.dim(prompt));
|
|
290
334
|
const handler = () => {
|
|
291
335
|
process.stdin.removeListener("data", handler);
|
|
292
336
|
process.stdin.setRawMode?.(false);
|
|
293
337
|
process.stdin.pause();
|
|
294
338
|
console.log();
|
|
295
|
-
|
|
339
|
+
resolve3();
|
|
296
340
|
};
|
|
297
341
|
if (process.stdin.isTTY) {
|
|
298
342
|
process.stdin.setRawMode?.(true);
|
|
@@ -302,7 +346,7 @@ function waitForEnter(prompt = "Press Enter to continue...") {
|
|
|
302
346
|
});
|
|
303
347
|
}
|
|
304
348
|
function sleep(ms) {
|
|
305
|
-
return new Promise((
|
|
349
|
+
return new Promise((resolve3) => setTimeout(resolve3, ms));
|
|
306
350
|
}
|
|
307
351
|
|
|
308
352
|
// src/commands/login.ts
|
|
@@ -373,22 +417,25 @@ async function deviceFlowLogin(options) {
|
|
|
373
417
|
}
|
|
374
418
|
async function tokenLogin() {
|
|
375
419
|
console.log("Token login mode.");
|
|
376
|
-
console.log("
|
|
420
|
+
console.log("Create a token under Settings \u2192 CLI tokens in the dashboard, then paste it below.");
|
|
421
|
+
console.log(
|
|
422
|
+
"(Alternatively, run `evident login` on a machine with a browser, or set EVIDENT_TOKEN for CI.)"
|
|
423
|
+
);
|
|
377
424
|
blank();
|
|
378
425
|
process.stdout.write("Paste token: ");
|
|
379
|
-
const token = await new Promise((
|
|
426
|
+
const token = await new Promise((resolve3) => {
|
|
380
427
|
let data = "";
|
|
381
428
|
process.stdin.setEncoding("utf8");
|
|
382
429
|
process.stdin.on("data", (chunk) => {
|
|
383
430
|
data += chunk;
|
|
384
431
|
});
|
|
385
432
|
process.stdin.on("end", () => {
|
|
386
|
-
|
|
433
|
+
resolve3(data.trim());
|
|
387
434
|
});
|
|
388
435
|
if (process.stdin.isTTY) {
|
|
389
436
|
process.stdin.once("data", (chunk) => {
|
|
390
437
|
process.stdin.pause();
|
|
391
|
-
|
|
438
|
+
resolve3(chunk.toString().trim());
|
|
392
439
|
});
|
|
393
440
|
process.stdin.resume();
|
|
394
441
|
}
|
|
@@ -397,13 +444,22 @@ async function tokenLogin() {
|
|
|
397
444
|
printError("No token provided.");
|
|
398
445
|
process.exit(1);
|
|
399
446
|
}
|
|
447
|
+
await validateAndStoreToken(token);
|
|
448
|
+
}
|
|
449
|
+
async function validateAndStoreToken(token) {
|
|
400
450
|
const spinner = ora("Validating token...").start();
|
|
401
451
|
try {
|
|
402
|
-
const result = await api.
|
|
452
|
+
const result = await api.get("/me", {
|
|
453
|
+
headers: { Authorization: `Bearer ${token}` }
|
|
454
|
+
});
|
|
455
|
+
if (!result.user) {
|
|
456
|
+
throw new Error(
|
|
457
|
+
"This token is not a user login (e.g. a runner key). Paste a CLI token instead."
|
|
458
|
+
);
|
|
459
|
+
}
|
|
403
460
|
await storeToken({
|
|
404
461
|
token,
|
|
405
|
-
user: result.user
|
|
406
|
-
expiresAt: result.expires_at
|
|
462
|
+
user: { email: result.user.email }
|
|
407
463
|
});
|
|
408
464
|
spinner.stop();
|
|
409
465
|
printSuccess(`Logged in as ${chalk2.bold(result.user.email)}`);
|
|
@@ -423,9 +479,22 @@ async function login(options) {
|
|
|
423
479
|
}
|
|
424
480
|
|
|
425
481
|
// src/commands/logout.ts
|
|
482
|
+
function describeFailure(failure) {
|
|
483
|
+
if (failure.type === "enumerate") {
|
|
484
|
+
return `could not list stored keychain entries (${failure.error.message})`;
|
|
485
|
+
}
|
|
486
|
+
return `${failure.account} (${failure.error.message})`;
|
|
487
|
+
}
|
|
426
488
|
async function logout(options = {}) {
|
|
427
489
|
if (options.all) {
|
|
428
|
-
await deleteToken({ all: true });
|
|
490
|
+
const result = await deleteToken({ all: true });
|
|
491
|
+
if (result.failures.length > 0) {
|
|
492
|
+
printError(
|
|
493
|
+
`Failed to fully clear your keychain: ${result.failures.map(describeFailure).join("; ")}. Your local credentials file was cleared, but stale keychain entries may remain \u2014 run \`evident logout --all\` again, or remove them manually from your OS keychain / credential manager.`
|
|
494
|
+
);
|
|
495
|
+
process.exitCode = 1;
|
|
496
|
+
return;
|
|
497
|
+
}
|
|
429
498
|
printSuccess("Logged out of all endpoints.");
|
|
430
499
|
return;
|
|
431
500
|
}
|
|
@@ -450,7 +519,9 @@ async function whoami() {
|
|
|
450
519
|
blank();
|
|
451
520
|
console.log(keyValue("Endpoint", apiUrl));
|
|
452
521
|
console.log(keyValue("User", chalk3.bold(credentials2.user.email)));
|
|
453
|
-
|
|
522
|
+
if (credentials2.user.id) {
|
|
523
|
+
console.log(keyValue("User ID", credentials2.user.id));
|
|
524
|
+
}
|
|
454
525
|
if (credentials2.expiresAt) {
|
|
455
526
|
const expiresAt = new Date(credentials2.expiresAt);
|
|
456
527
|
const now = /* @__PURE__ */ new Date();
|
|
@@ -467,9 +538,9 @@ async function whoami() {
|
|
|
467
538
|
}
|
|
468
539
|
|
|
469
540
|
// src/commands/run.ts
|
|
541
|
+
import { homedir as homedir2 } from "os";
|
|
542
|
+
import { isAbsolute as isAbsolute2, join as join2, parse, resolve as resolvePath } from "path";
|
|
470
543
|
import chalk6 from "chalk";
|
|
471
|
-
import ora3 from "ora";
|
|
472
|
-
import { select as select3 } from "@inquirer/prompts";
|
|
473
544
|
|
|
474
545
|
// ../../packages/types/src/telemetry/index.ts
|
|
475
546
|
var TelemetryEventTypes = {
|
|
@@ -478,13 +549,20 @@ var TelemetryEventTypes = {
|
|
|
478
549
|
AGENT_DISCONNECTED: "agent.disconnected",
|
|
479
550
|
AGENT_MESSAGE_PROCESSING: "agent.message_processing",
|
|
480
551
|
AGENT_MESSAGE_DONE: "agent.message_done",
|
|
481
|
-
AGENT_MESSAGE_FAILED: "agent.message_failed"
|
|
552
|
+
AGENT_MESSAGE_FAILED: "agent.message_failed",
|
|
553
|
+
// A `warn`/`error` runner-side log line forwarded server-side for
|
|
554
|
+
// observability (issue #916) — see `apps/cli/src/lib/runner-activity-telemetry.ts`.
|
|
555
|
+
RUNNER_ACTIVITY: "runner.activity"
|
|
482
556
|
};
|
|
483
557
|
|
|
484
558
|
// ../../packages/types/src/tunnel/index.ts
|
|
485
559
|
var MAX_FRAME_BYTES = 256 * 1024;
|
|
486
560
|
var TUNNEL_DRAIN_PING_PATH = "/__evident/drain";
|
|
487
561
|
|
|
562
|
+
// ../../packages/types/src/runner-files.ts
|
|
563
|
+
var MAX_FILE_PUSH_BYTES = 64 * 1024;
|
|
564
|
+
var MAX_FILE_SYNC_DIRECTORIES = 16;
|
|
565
|
+
|
|
488
566
|
// ../../packages/types/src/logging/index.ts
|
|
489
567
|
var CORRELATION_ID_HEADER = "x-evident-correlation-id";
|
|
490
568
|
function log(level, event, fields) {
|
|
@@ -499,6 +577,12 @@ function log(level, event, fields) {
|
|
|
499
577
|
);
|
|
500
578
|
}
|
|
501
579
|
}
|
|
580
|
+
function errorFields(err) {
|
|
581
|
+
if (err instanceof Error) {
|
|
582
|
+
return { error: err.message, error_name: err.name };
|
|
583
|
+
}
|
|
584
|
+
return { error: String(err) };
|
|
585
|
+
}
|
|
502
586
|
function stripQuery(url) {
|
|
503
587
|
try {
|
|
504
588
|
return new URL(url).pathname;
|
|
@@ -508,6 +592,10 @@ function stripQuery(url) {
|
|
|
508
592
|
}
|
|
509
593
|
}
|
|
510
594
|
|
|
595
|
+
// src/commands/run.ts
|
|
596
|
+
import ora3 from "ora";
|
|
597
|
+
import { select as select3 } from "@inquirer/prompts";
|
|
598
|
+
|
|
511
599
|
// src/lib/telemetry.ts
|
|
512
600
|
var CLI_VERSION = (true ? "3.0.0" : void 0) ?? process.env.npm_package_version ?? "unknown";
|
|
513
601
|
function getCliVersion() {
|
|
@@ -519,6 +607,13 @@ var isShuttingDown = false;
|
|
|
519
607
|
var FLUSH_INTERVAL_MS = 5e3;
|
|
520
608
|
var MAX_BUFFER_SIZE = 50;
|
|
521
609
|
var FLUSH_TIMEOUT_MS = 3e3;
|
|
610
|
+
var authProvider = null;
|
|
611
|
+
function setTelemetryAuthProvider(provider) {
|
|
612
|
+
authProvider = provider;
|
|
613
|
+
}
|
|
614
|
+
var FLUSH_FAILURE_LOG_INTERVAL_MS = 6e4;
|
|
615
|
+
var lastFlushFailureLoggedAt = 0;
|
|
616
|
+
var suppressedFlushFailureCount = 0;
|
|
522
617
|
function logEvent(eventType, options = {}) {
|
|
523
618
|
const event = {
|
|
524
619
|
event_type: eventType,
|
|
@@ -553,9 +648,16 @@ async function flushEvents() {
|
|
|
553
648
|
flushTimeout = null;
|
|
554
649
|
}
|
|
555
650
|
try {
|
|
556
|
-
const
|
|
557
|
-
|
|
558
|
-
|
|
651
|
+
const providerContext = authProvider?.();
|
|
652
|
+
let authHeader;
|
|
653
|
+
if (providerContext?.authHeader) {
|
|
654
|
+
authHeader = providerContext.authHeader;
|
|
655
|
+
} else {
|
|
656
|
+
const credentials2 = await getToken();
|
|
657
|
+
if (!credentials2) {
|
|
658
|
+
return;
|
|
659
|
+
}
|
|
660
|
+
authHeader = `Bearer ${credentials2.token}`;
|
|
559
661
|
}
|
|
560
662
|
const apiUrl = getApiUrlConfig();
|
|
561
663
|
const controller = new AbortController();
|
|
@@ -570,7 +672,7 @@ async function flushEvents() {
|
|
|
570
672
|
method: "POST",
|
|
571
673
|
headers: {
|
|
572
674
|
"Content-Type": "application/json",
|
|
573
|
-
Authorization:
|
|
675
|
+
Authorization: authHeader
|
|
574
676
|
},
|
|
575
677
|
body: JSON.stringify(request),
|
|
576
678
|
signal: controller.signal
|
|
@@ -582,8 +684,15 @@ async function flushEvents() {
|
|
|
582
684
|
clearTimeout(timeout);
|
|
583
685
|
}
|
|
584
686
|
} catch (error2) {
|
|
585
|
-
|
|
586
|
-
|
|
687
|
+
const now = Date.now();
|
|
688
|
+
if (now - lastFlushFailureLoggedAt >= FLUSH_FAILURE_LOG_INTERVAL_MS) {
|
|
689
|
+
const message = error2 instanceof Error ? error2.message : String(error2);
|
|
690
|
+
const suffix = suppressedFlushFailureCount > 0 ? ` (${suppressedFlushFailureCount} more suppressed in the last ${FLUSH_FAILURE_LOG_INTERVAL_MS / 1e3}s)` : "";
|
|
691
|
+
console.error(`Telemetry flush error: ${message}${suffix}`);
|
|
692
|
+
lastFlushFailureLoggedAt = now;
|
|
693
|
+
suppressedFlushFailureCount = 0;
|
|
694
|
+
} else {
|
|
695
|
+
suppressedFlushFailureCount++;
|
|
587
696
|
}
|
|
588
697
|
}
|
|
589
698
|
}
|
|
@@ -652,6 +761,69 @@ var EventTypes = {
|
|
|
652
761
|
DEPRECATED_AGENT_KEY_ENV_USED: "cli.deprecated_agent_key_env_used"
|
|
653
762
|
};
|
|
654
763
|
|
|
764
|
+
// src/lib/runner-activity-telemetry.ts
|
|
765
|
+
var FORWARDED_LEVELS = /* @__PURE__ */ new Set(["warn", "error"]);
|
|
766
|
+
var SEVERITY_BY_LEVEL = {
|
|
767
|
+
warn: "warning",
|
|
768
|
+
error: "error"
|
|
769
|
+
};
|
|
770
|
+
var MAX_MESSAGE_LENGTH = 500;
|
|
771
|
+
var TRUNCATION_MARKER = "\u2026";
|
|
772
|
+
function redact(message) {
|
|
773
|
+
return message.replace(/esk_[A-Za-z0-9_-]+/g, "esk_***").replace(/ct_[A-Za-z0-9_-]+/g, "ct_***").replace(/https?:\/\/\S+/g, "<url>");
|
|
774
|
+
}
|
|
775
|
+
function truncate(message) {
|
|
776
|
+
if (message.length <= MAX_MESSAGE_LENGTH) return message;
|
|
777
|
+
return message.slice(0, MAX_MESSAGE_LENGTH - TRUNCATION_MARKER.length) + TRUNCATION_MARKER;
|
|
778
|
+
}
|
|
779
|
+
var RATE_LIMIT_WINDOW_MS = 6e4;
|
|
780
|
+
var RATE_LIMIT_MAX_EVENTS = 30;
|
|
781
|
+
var windowStartedAt = 0;
|
|
782
|
+
var windowCount = 0;
|
|
783
|
+
var windowDroppedCount = 0;
|
|
784
|
+
function admitUnderRateLimit(now) {
|
|
785
|
+
if (now - windowStartedAt >= RATE_LIMIT_WINDOW_MS) {
|
|
786
|
+
if (windowDroppedCount > 0) {
|
|
787
|
+
console.error(
|
|
788
|
+
`[runner-activity-telemetry] rate cap reached: dropped ${windowDroppedCount} ${windowDroppedCount === 1 ? "entry" : "entries"} in the last ${RATE_LIMIT_WINDOW_MS / 1e3}s (cap ${RATE_LIMIT_MAX_EVENTS}/min)`
|
|
789
|
+
);
|
|
790
|
+
}
|
|
791
|
+
windowStartedAt = now;
|
|
792
|
+
windowCount = 0;
|
|
793
|
+
windowDroppedCount = 0;
|
|
794
|
+
}
|
|
795
|
+
if (windowCount >= RATE_LIMIT_MAX_EVENTS) {
|
|
796
|
+
windowDroppedCount++;
|
|
797
|
+
if (windowDroppedCount === 1) {
|
|
798
|
+
console.error(
|
|
799
|
+
`[runner-activity-telemetry] rate cap reached (${RATE_LIMIT_MAX_EVENTS} per ${RATE_LIMIT_WINDOW_MS / 1e3}s) \u2014 dropping further entries this window`
|
|
800
|
+
);
|
|
801
|
+
}
|
|
802
|
+
return false;
|
|
803
|
+
}
|
|
804
|
+
windowCount++;
|
|
805
|
+
return true;
|
|
806
|
+
}
|
|
807
|
+
function forwardRunnerActivity(entry, context) {
|
|
808
|
+
try {
|
|
809
|
+
if (!FORWARDED_LEVELS.has(entry.level)) return;
|
|
810
|
+
if (!context.agentId || !context.authHeader) return;
|
|
811
|
+
if (!admitUnderRateLimit(Date.now())) return;
|
|
812
|
+
const rawMessage = entry.error ?? entry.message ?? "";
|
|
813
|
+
const message = truncate(redact(rawMessage));
|
|
814
|
+
logEvent(TelemetryEventTypes.RUNNER_ACTIVITY, {
|
|
815
|
+
severity: SEVERITY_BY_LEVEL[entry.level],
|
|
816
|
+
message,
|
|
817
|
+
metadata: { source: "cli.run" },
|
|
818
|
+
agentId: context.agentId
|
|
819
|
+
});
|
|
820
|
+
} catch (err) {
|
|
821
|
+
console.error(
|
|
822
|
+
`[runner-activity-telemetry] failed to forward runner activity: ${err instanceof Error ? err.message : String(err)}`
|
|
823
|
+
);
|
|
824
|
+
}
|
|
825
|
+
}
|
|
826
|
+
|
|
655
827
|
// src/lib/auth.ts
|
|
656
828
|
async function getAuthCredentials() {
|
|
657
829
|
const runnerKey = process.env.EVIDENT_RUNNER_KEY;
|
|
@@ -719,7 +891,7 @@ async function waitForOpenCodeHealth(port, timeoutMs = 3e4) {
|
|
|
719
891
|
if (health.healthy) {
|
|
720
892
|
return health;
|
|
721
893
|
}
|
|
722
|
-
await new Promise((
|
|
894
|
+
await new Promise((resolve3) => setTimeout(resolve3, 1e3));
|
|
723
895
|
}
|
|
724
896
|
return { healthy: false, error: "Timeout waiting for OpenCode to be healthy" };
|
|
725
897
|
}
|
|
@@ -734,7 +906,7 @@ function buildOpenCodeVersionWarning(version2) {
|
|
|
734
906
|
if (isQueueValidatedVersion(version2)) return null;
|
|
735
907
|
const detected = version2 ? `v${version2}` : "unknown";
|
|
736
908
|
const validated = QUEUE_VALIDATED_OPENCODE_VERSIONS.map((v) => `v${v}`).join(", ");
|
|
737
|
-
return `Warning: opencode ${detected} is not a queue-validated version (validated: ${validated}). Native message queuing \u2014 which channel (Slack
|
|
909
|
+
return `Warning: opencode ${detected} is not a queue-validated version (validated: ${validated}). Native message queuing \u2014 which channel (Slack) message handling relies on \u2014 is unverified on this version; queued/follow-up messages may behave unexpectedly. Continuing anyway. Bumping the validated set requires re-running the queue validation.`;
|
|
738
910
|
}
|
|
739
911
|
|
|
740
912
|
// src/lib/opencode/process.ts
|
|
@@ -1026,6 +1198,12 @@ async function promptOpenCodeInstall(interactive) {
|
|
|
1026
1198
|
return action;
|
|
1027
1199
|
}
|
|
1028
1200
|
|
|
1201
|
+
// src/lib/opencode/provider-check.ts
|
|
1202
|
+
function buildNoProviderWarning(hasProvider) {
|
|
1203
|
+
if (hasProvider !== false) return null;
|
|
1204
|
+
return "Warning: opencode has no authenticated model provider configured, so it won't be able to answer prompts. Run `opencode auth login` to set one up (see https://opencode.ai for details).";
|
|
1205
|
+
}
|
|
1206
|
+
|
|
1029
1207
|
// src/lib/opencode/session.ts
|
|
1030
1208
|
function opencodeBase(port) {
|
|
1031
1209
|
return `http://127.0.0.1:${port}`;
|
|
@@ -1228,6 +1406,11 @@ async function getModelAttachmentCapability(port, model) {
|
|
|
1228
1406
|
}
|
|
1229
1407
|
const entry = provider.models[modelId];
|
|
1230
1408
|
if (!entry || typeof entry !== "object") return null;
|
|
1409
|
+
if (entry.capabilities && typeof entry.capabilities === "object") {
|
|
1410
|
+
if (typeof entry.capabilities.attachment === "boolean") {
|
|
1411
|
+
return entry.capabilities.attachment;
|
|
1412
|
+
}
|
|
1413
|
+
}
|
|
1231
1414
|
return typeof entry.attachment === "boolean" ? entry.attachment : null;
|
|
1232
1415
|
} catch (err) {
|
|
1233
1416
|
console.error(
|
|
@@ -1256,6 +1439,16 @@ async function buildFileParts(attachments, capable) {
|
|
|
1256
1439
|
);
|
|
1257
1440
|
dataUrl = null;
|
|
1258
1441
|
}
|
|
1442
|
+
if (dataUrl !== null && typeof dataUrl === "object") {
|
|
1443
|
+
outcomes.push({
|
|
1444
|
+
index: a.index,
|
|
1445
|
+
mime: a.mime,
|
|
1446
|
+
filename: a.filename,
|
|
1447
|
+
status: "failed",
|
|
1448
|
+
reason: "needs_reauth"
|
|
1449
|
+
});
|
|
1450
|
+
continue;
|
|
1451
|
+
}
|
|
1259
1452
|
if (dataUrl == null) {
|
|
1260
1453
|
outcomes.push({ index: a.index, mime: a.mime, filename: a.filename, status: "failed" });
|
|
1261
1454
|
continue;
|
|
@@ -1337,7 +1530,7 @@ async function sendPromptAsync(port, sessionId, content, options, attachments) {
|
|
|
1337
1530
|
}
|
|
1338
1531
|
}
|
|
1339
1532
|
if (attempt < READ_BACK_ATTEMPTS - 1) {
|
|
1340
|
-
await new Promise((
|
|
1533
|
+
await new Promise((resolve3) => setTimeout(resolve3, READ_BACK_DELAY_MS));
|
|
1341
1534
|
}
|
|
1342
1535
|
}
|
|
1343
1536
|
return null;
|
|
@@ -1465,6 +1658,9 @@ function isPreamblePinnedRunning(messages, userMessageId) {
|
|
|
1465
1658
|
const reply = findLastAssistantReplyFor(messages, userMessageId);
|
|
1466
1659
|
return completedOf(reply) != null && finishOf(reply) === "tool-calls";
|
|
1467
1660
|
}
|
|
1661
|
+
function isB2AbandonmentConfirmed(params) {
|
|
1662
|
+
return params.pinnedForMs >= params.minPinnedMs && params.descendantOngoing === false;
|
|
1663
|
+
}
|
|
1468
1664
|
function messageError(messages, userMessageId) {
|
|
1469
1665
|
const reply = findLastAssistantReplyFor(messages, userMessageId);
|
|
1470
1666
|
const error2 = errorOf(reply);
|
|
@@ -1478,12 +1674,79 @@ function messageError(messages, userMessageId) {
|
|
|
1478
1674
|
}
|
|
1479
1675
|
return "The agent run failed.";
|
|
1480
1676
|
}
|
|
1677
|
+
function messageFailure(messages, userMessageId) {
|
|
1678
|
+
const reply = findLastAssistantReplyFor(messages, userMessageId);
|
|
1679
|
+
const error2 = errorOf(reply);
|
|
1680
|
+
if (error2 == null || typeof error2 !== "object") return null;
|
|
1681
|
+
const e = error2;
|
|
1682
|
+
const replyProviderId = reply?.info?.providerID ?? null;
|
|
1683
|
+
const replyModelId = reply?.info?.modelID ?? null;
|
|
1684
|
+
if (e.name === "ProviderAuthError") {
|
|
1685
|
+
const data = e.data;
|
|
1686
|
+
const providerId = typeof data?.providerID === "string" && data.providerID || replyProviderId;
|
|
1687
|
+
return { kind: "model_auth", providerId, modelId: replyModelId, reason: "missing" };
|
|
1688
|
+
}
|
|
1689
|
+
if (e.name === "APIError") {
|
|
1690
|
+
const data = e.data;
|
|
1691
|
+
const statusCode = data?.statusCode;
|
|
1692
|
+
if (statusCode === 401 || statusCode === 403) {
|
|
1693
|
+
return {
|
|
1694
|
+
kind: "model_auth",
|
|
1695
|
+
providerId: replyProviderId,
|
|
1696
|
+
modelId: replyModelId,
|
|
1697
|
+
reason: "rejected"
|
|
1698
|
+
};
|
|
1699
|
+
}
|
|
1700
|
+
}
|
|
1701
|
+
return null;
|
|
1702
|
+
}
|
|
1703
|
+
function applyZeroProviderFallback(classified, hasConfiguredProvider, replyProviderId, replyModelId = null) {
|
|
1704
|
+
if (classified != null) return classified;
|
|
1705
|
+
if (hasConfiguredProvider !== false) return null;
|
|
1706
|
+
return {
|
|
1707
|
+
kind: "model_auth",
|
|
1708
|
+
providerId: replyProviderId,
|
|
1709
|
+
modelId: replyModelId,
|
|
1710
|
+
reason: "missing"
|
|
1711
|
+
};
|
|
1712
|
+
}
|
|
1481
1713
|
function hasRunningAssistantExcept(messages, exceptUserMessageId) {
|
|
1482
1714
|
if (!messages || messages.length === 0) return false;
|
|
1483
1715
|
return messages.some(
|
|
1484
1716
|
(m) => roleOf(m) === "assistant" && parentIdOf(m) !== exceptUserMessageId && isAssistantInFlight(m)
|
|
1485
1717
|
);
|
|
1486
1718
|
}
|
|
1719
|
+
async function hasAnyConfiguredProvider(port) {
|
|
1720
|
+
try {
|
|
1721
|
+
const res = await fetch(`${opencodeBase(port)}/config/providers`);
|
|
1722
|
+
if (!res.ok) {
|
|
1723
|
+
console.error(
|
|
1724
|
+
`[hasAnyConfiguredProvider] GET /config/providers returned HTTP ${res.status} (port ${port})`
|
|
1725
|
+
);
|
|
1726
|
+
return null;
|
|
1727
|
+
}
|
|
1728
|
+
const body = await res.json();
|
|
1729
|
+
if (!body || typeof body !== "object" || Array.isArray(body)) {
|
|
1730
|
+
console.error(
|
|
1731
|
+
`[hasAnyConfiguredProvider] GET /config/providers body was not a plain object (port ${port})`
|
|
1732
|
+
);
|
|
1733
|
+
return null;
|
|
1734
|
+
}
|
|
1735
|
+
const defaults2 = body.default;
|
|
1736
|
+
if (!defaults2 || typeof defaults2 !== "object" || Array.isArray(defaults2)) {
|
|
1737
|
+
console.error(
|
|
1738
|
+
`[hasAnyConfiguredProvider] GET /config/providers body had no \`default\` object (port ${port})`
|
|
1739
|
+
);
|
|
1740
|
+
return null;
|
|
1741
|
+
}
|
|
1742
|
+
return Object.keys(defaults2).length > 0;
|
|
1743
|
+
} catch (err) {
|
|
1744
|
+
console.error(
|
|
1745
|
+
`[hasAnyConfiguredProvider] GET /config/providers failed (port ${port}): ${err instanceof Error ? err.message : String(err)}`
|
|
1746
|
+
);
|
|
1747
|
+
return null;
|
|
1748
|
+
}
|
|
1749
|
+
}
|
|
1487
1750
|
|
|
1488
1751
|
// src/lib/opencode/session-cleanup.ts
|
|
1489
1752
|
var DURATION_UNIT_MS = {
|
|
@@ -1642,10 +1905,11 @@ var StreamForwarder = class {
|
|
|
1642
1905
|
* Abort every in-flight stream (e.g. on WebSocket close).
|
|
1643
1906
|
*/
|
|
1644
1907
|
abortAll() {
|
|
1645
|
-
for (const stream of this.inflight.
|
|
1908
|
+
for (const [sid, stream] of this.inflight.entries()) {
|
|
1646
1909
|
try {
|
|
1647
1910
|
stream.abort();
|
|
1648
|
-
} catch {
|
|
1911
|
+
} catch (err) {
|
|
1912
|
+
log("error", "forwarder_abort_failed", { sid, ...errorFields(err) });
|
|
1649
1913
|
}
|
|
1650
1914
|
}
|
|
1651
1915
|
this.inflight.clear();
|
|
@@ -1679,12 +1943,12 @@ var StreamForwarder = class {
|
|
|
1679
1943
|
let endBody;
|
|
1680
1944
|
if (has_body) {
|
|
1681
1945
|
const chunks = [];
|
|
1682
|
-
bodyPromise = new Promise((
|
|
1946
|
+
bodyPromise = new Promise((resolve3) => {
|
|
1683
1947
|
pushBody = (buf) => {
|
|
1684
1948
|
chunks.push(buf);
|
|
1685
1949
|
};
|
|
1686
1950
|
endBody = () => {
|
|
1687
|
-
|
|
1951
|
+
resolve3(Buffer.concat(chunks));
|
|
1688
1952
|
};
|
|
1689
1953
|
});
|
|
1690
1954
|
}
|
|
@@ -1795,31 +2059,20 @@ function connectTunnel(options) {
|
|
|
1795
2059
|
onConnected,
|
|
1796
2060
|
onDisconnected,
|
|
1797
2061
|
onError,
|
|
1798
|
-
onRequest,
|
|
1799
2062
|
onResponse,
|
|
1800
2063
|
onInfo,
|
|
1801
2064
|
onDrainPing
|
|
1802
2065
|
} = options;
|
|
1803
2066
|
const tunnelUrl = getTunnelUrlConfig();
|
|
1804
2067
|
const url = `${tunnelUrl}/tunnel/${agentId}/connect`;
|
|
1805
|
-
return new Promise((
|
|
2068
|
+
return new Promise((resolve3, reject) => {
|
|
1806
2069
|
const ws = new WebSocket2(url, {
|
|
1807
2070
|
headers: {
|
|
1808
2071
|
Authorization: authHeader
|
|
1809
2072
|
}
|
|
1810
2073
|
});
|
|
1811
|
-
const streamStartTimes = /* @__PURE__ */ new Map();
|
|
1812
2074
|
const forwarder = new StreamForwarder(ws, port, {
|
|
1813
|
-
|
|
1814
|
-
if (path === TUNNEL_DRAIN_PING_PATH) return;
|
|
1815
|
-
streamStartTimes.set(sid, Date.now());
|
|
1816
|
-
onRequest?.(method, path, sid);
|
|
1817
|
-
},
|
|
1818
|
-
onHead: (sid, status) => {
|
|
1819
|
-
const startedAt = streamStartTimes.get(sid);
|
|
1820
|
-
streamStartTimes.delete(sid);
|
|
1821
|
-
onResponse?.(status, startedAt ? Date.now() - startedAt : 0, sid);
|
|
1822
|
-
},
|
|
2075
|
+
onHead: () => onResponse?.(),
|
|
1823
2076
|
onDrainPing: () => onDrainPing?.()
|
|
1824
2077
|
});
|
|
1825
2078
|
const connectionTimeout = setTimeout(() => {
|
|
@@ -1867,7 +2120,7 @@ function connectTunnel(options) {
|
|
|
1867
2120
|
clearTimeout(connectionTimeout);
|
|
1868
2121
|
const connectedAgentId = message.agent_id ?? agentId;
|
|
1869
2122
|
onConnected?.(connectedAgentId);
|
|
1870
|
-
|
|
2123
|
+
resolve3({
|
|
1871
2124
|
ws,
|
|
1872
2125
|
close: () => ws.close(1e3, "CLI shutdown")
|
|
1873
2126
|
});
|
|
@@ -1895,7 +2148,6 @@ function connectTunnel(options) {
|
|
|
1895
2148
|
ws.on("close", (code, reason) => {
|
|
1896
2149
|
const reasonStr = reason.toString() || upgradeRejection || (code === 1006 ? "abnormal closure" : "No reason provided");
|
|
1897
2150
|
forwarder.abortAll();
|
|
1898
|
-
streamStartTimes.clear();
|
|
1899
2151
|
onDisconnected?.(code, reasonStr);
|
|
1900
2152
|
});
|
|
1901
2153
|
});
|
|
@@ -1930,7 +2182,11 @@ var RunnerConnection = class {
|
|
|
1930
2182
|
if (this.connection) {
|
|
1931
2183
|
try {
|
|
1932
2184
|
this.connection.close();
|
|
1933
|
-
} catch {
|
|
2185
|
+
} catch (err) {
|
|
2186
|
+
log("error", "runner_connection_close_failed", {
|
|
2187
|
+
agent_id: this.resolvedAgentId,
|
|
2188
|
+
...errorFields(err)
|
|
2189
|
+
});
|
|
1934
2190
|
}
|
|
1935
2191
|
this.connection = null;
|
|
1936
2192
|
}
|
|
@@ -1982,6 +2238,416 @@ var RunnerConnection = class {
|
|
|
1982
2238
|
}
|
|
1983
2239
|
};
|
|
1984
2240
|
|
|
2241
|
+
// src/lib/tunnel/ready-marker.ts
|
|
2242
|
+
import { writeFileSync } from "fs";
|
|
2243
|
+
function writeTunnelReadyMarker(path, agentId) {
|
|
2244
|
+
try {
|
|
2245
|
+
writeFileSync(path, `${agentId}
|
|
2246
|
+
`);
|
|
2247
|
+
return { ok: true };
|
|
2248
|
+
} catch (error2) {
|
|
2249
|
+
return { ok: false, error: error2 instanceof Error ? error2.message : String(error2) };
|
|
2250
|
+
}
|
|
2251
|
+
}
|
|
2252
|
+
|
|
2253
|
+
// src/lib/channels/driver.ts
|
|
2254
|
+
import { homedir } from "os";
|
|
2255
|
+
|
|
2256
|
+
// src/lib/file-push.ts
|
|
2257
|
+
import { randomUUID } from "crypto";
|
|
2258
|
+
import { chmod, mkdir, open as open2, realpath, rename, unlink } from "fs/promises";
|
|
2259
|
+
import { basename, dirname as dirname2, isAbsolute, join, relative, resolve as resolve2, sep } from "path";
|
|
2260
|
+
var FILE_MODE = 384;
|
|
2261
|
+
var DIRECTORY_MODE = 448;
|
|
2262
|
+
async function writePushedFile(request) {
|
|
2263
|
+
const { requestedPath, content, allowedDirectories, homeDir } = request;
|
|
2264
|
+
const bytes = content.byteLength;
|
|
2265
|
+
if (allowedDirectories.length === 0) {
|
|
2266
|
+
return refuse("file_sync_disabled", "File sync is not enabled on this runner.", {
|
|
2267
|
+
path: requestedPath,
|
|
2268
|
+
bytes
|
|
2269
|
+
});
|
|
2270
|
+
}
|
|
2271
|
+
if (bytes > MAX_FILE_PUSH_BYTES) {
|
|
2272
|
+
return refuse(
|
|
2273
|
+
"file_too_large",
|
|
2274
|
+
`File is ${bytes} bytes; the limit is ${MAX_FILE_PUSH_BYTES}.`,
|
|
2275
|
+
{
|
|
2276
|
+
path: requestedPath,
|
|
2277
|
+
bytes
|
|
2278
|
+
}
|
|
2279
|
+
);
|
|
2280
|
+
}
|
|
2281
|
+
const candidate = expandAndValidate(requestedPath, homeDir);
|
|
2282
|
+
if (candidate === null) {
|
|
2283
|
+
return refuse("invalid_path", "The requested path is not a valid absolute file path.", {
|
|
2284
|
+
path: requestedPath,
|
|
2285
|
+
bytes
|
|
2286
|
+
});
|
|
2287
|
+
}
|
|
2288
|
+
try {
|
|
2289
|
+
const { existingAncestor, missingSegments } = await resolveNearestExistingAncestor(
|
|
2290
|
+
dirname2(candidate)
|
|
2291
|
+
);
|
|
2292
|
+
const realTarget = join(existingAncestor, ...missingSegments, basename(candidate));
|
|
2293
|
+
const allowedDirectory = await findContainingAllowedDirectory(allowedDirectories, realTarget);
|
|
2294
|
+
if (allowedDirectory === null) {
|
|
2295
|
+
return refuse("path_not_allowed", "The runner does not allow writing to that location.", {
|
|
2296
|
+
path: realTarget,
|
|
2297
|
+
bytes
|
|
2298
|
+
});
|
|
2299
|
+
}
|
|
2300
|
+
if (missingSegments.length > 0) {
|
|
2301
|
+
await createMissingDirectories(existingAncestor, missingSegments);
|
|
2302
|
+
const realParent = await realpath(dirname2(realTarget));
|
|
2303
|
+
if (realParent !== dirname2(realTarget) || !contains(allowedDirectory, realTarget)) {
|
|
2304
|
+
return refuse("path_not_allowed", "The runner does not allow writing to that location.", {
|
|
2305
|
+
path: realTarget,
|
|
2306
|
+
bytes,
|
|
2307
|
+
reason: "parent_changed_after_create"
|
|
2308
|
+
});
|
|
2309
|
+
}
|
|
2310
|
+
}
|
|
2311
|
+
await writeAtomically(realTarget, content);
|
|
2312
|
+
log("info", "file_push_written", { path: realTarget, bytes });
|
|
2313
|
+
return { ok: true, path: realTarget };
|
|
2314
|
+
} catch (err) {
|
|
2315
|
+
const errno = err.code ?? "UNKNOWN";
|
|
2316
|
+
return refuse("write_failed", `The runner could not write the file (${errno}).`, {
|
|
2317
|
+
path: candidate,
|
|
2318
|
+
bytes,
|
|
2319
|
+
errno,
|
|
2320
|
+
...errorFields(err)
|
|
2321
|
+
});
|
|
2322
|
+
}
|
|
2323
|
+
}
|
|
2324
|
+
function expandAndValidate(requestedPath, homeDir) {
|
|
2325
|
+
if (requestedPath.trim() === "" || requestedPath.includes("\0")) {
|
|
2326
|
+
return null;
|
|
2327
|
+
}
|
|
2328
|
+
const expanded = requestedPath === "~" ? homeDir : requestedPath.startsWith("~/") ? join(homeDir, requestedPath.slice(2)) : requestedPath;
|
|
2329
|
+
if (expanded.split(/[/\\]/).includes("..")) {
|
|
2330
|
+
return null;
|
|
2331
|
+
}
|
|
2332
|
+
if (!isAbsolute(expanded)) {
|
|
2333
|
+
return null;
|
|
2334
|
+
}
|
|
2335
|
+
const candidate = resolve2(expanded);
|
|
2336
|
+
const name = basename(candidate);
|
|
2337
|
+
return name === "" || name === "." || name === ".." ? null : candidate;
|
|
2338
|
+
}
|
|
2339
|
+
async function resolveNearestExistingAncestor(directory) {
|
|
2340
|
+
const missingSegments = [];
|
|
2341
|
+
let current = directory;
|
|
2342
|
+
for (; ; ) {
|
|
2343
|
+
try {
|
|
2344
|
+
return { existingAncestor: await realpath(current), missingSegments };
|
|
2345
|
+
} catch (err) {
|
|
2346
|
+
const parent = dirname2(current);
|
|
2347
|
+
if (err.code !== "ENOENT" || parent === current) {
|
|
2348
|
+
throw err;
|
|
2349
|
+
}
|
|
2350
|
+
missingSegments.unshift(basename(current));
|
|
2351
|
+
current = parent;
|
|
2352
|
+
}
|
|
2353
|
+
}
|
|
2354
|
+
}
|
|
2355
|
+
async function findContainingAllowedDirectory(allowedDirectories, realTarget) {
|
|
2356
|
+
for (const directory of allowedDirectories) {
|
|
2357
|
+
if (!isAbsolute(directory)) {
|
|
2358
|
+
log("warn", "file_push_allowed_directory_skipped", { directory, reason: "not_absolute" });
|
|
2359
|
+
continue;
|
|
2360
|
+
}
|
|
2361
|
+
const realDirectory = await realpathCreatingIfMissing(directory);
|
|
2362
|
+
if (realDirectory !== null && contains(realDirectory, realTarget)) {
|
|
2363
|
+
return realDirectory;
|
|
2364
|
+
}
|
|
2365
|
+
}
|
|
2366
|
+
return null;
|
|
2367
|
+
}
|
|
2368
|
+
async function realpathCreatingIfMissing(directory) {
|
|
2369
|
+
try {
|
|
2370
|
+
return await realpath(directory);
|
|
2371
|
+
} catch (err) {
|
|
2372
|
+
if (err.code !== "ENOENT") {
|
|
2373
|
+
log("warn", "file_push_allowed_directory_skipped", {
|
|
2374
|
+
directory,
|
|
2375
|
+
reason: "unresolvable",
|
|
2376
|
+
...errorFields(err)
|
|
2377
|
+
});
|
|
2378
|
+
return null;
|
|
2379
|
+
}
|
|
2380
|
+
}
|
|
2381
|
+
try {
|
|
2382
|
+
await mkdir(directory, { recursive: true, mode: DIRECTORY_MODE });
|
|
2383
|
+
await chmod(directory, DIRECTORY_MODE);
|
|
2384
|
+
return await realpath(directory);
|
|
2385
|
+
} catch (err) {
|
|
2386
|
+
log("warn", "file_push_allowed_directory_skipped", {
|
|
2387
|
+
directory,
|
|
2388
|
+
reason: "create_failed",
|
|
2389
|
+
...errorFields(err)
|
|
2390
|
+
});
|
|
2391
|
+
return null;
|
|
2392
|
+
}
|
|
2393
|
+
}
|
|
2394
|
+
function contains(realDirectory, realTarget) {
|
|
2395
|
+
const rel = relative(realDirectory, realTarget);
|
|
2396
|
+
return rel !== "" && rel !== ".." && !rel.startsWith(`..${sep}`) && !isAbsolute(rel);
|
|
2397
|
+
}
|
|
2398
|
+
async function createMissingDirectories(existingAncestor, missingSegments) {
|
|
2399
|
+
let current = existingAncestor;
|
|
2400
|
+
for (const segment of missingSegments) {
|
|
2401
|
+
current = join(current, segment);
|
|
2402
|
+
await mkdir(current, { recursive: true, mode: DIRECTORY_MODE });
|
|
2403
|
+
await chmod(current, DIRECTORY_MODE);
|
|
2404
|
+
}
|
|
2405
|
+
}
|
|
2406
|
+
async function writeAtomically(realTarget, content) {
|
|
2407
|
+
const temporaryPath = join(dirname2(realTarget), `.evident-push-${randomUUID()}.tmp`);
|
|
2408
|
+
let handle;
|
|
2409
|
+
try {
|
|
2410
|
+
handle = await open2(temporaryPath, "wx", FILE_MODE);
|
|
2411
|
+
await handle.writeFile(content);
|
|
2412
|
+
await handle.chmod(FILE_MODE);
|
|
2413
|
+
await handle.close();
|
|
2414
|
+
handle = void 0;
|
|
2415
|
+
await rename(temporaryPath, realTarget);
|
|
2416
|
+
} catch (err) {
|
|
2417
|
+
await discardTemporaryFile(temporaryPath, handle);
|
|
2418
|
+
throw err;
|
|
2419
|
+
}
|
|
2420
|
+
}
|
|
2421
|
+
async function discardTemporaryFile(temporaryPath, handle) {
|
|
2422
|
+
try {
|
|
2423
|
+
await handle?.close();
|
|
2424
|
+
} catch (err) {
|
|
2425
|
+
log("warn", "file_push_temp_close_failed", { path: temporaryPath, ...errorFields(err) });
|
|
2426
|
+
}
|
|
2427
|
+
try {
|
|
2428
|
+
await unlink(temporaryPath);
|
|
2429
|
+
} catch (err) {
|
|
2430
|
+
const errno = err.code;
|
|
2431
|
+
if (errno !== "ENOENT" && errno !== "ENOTDIR") {
|
|
2432
|
+
log("warn", "file_push_temp_cleanup_failed", { path: temporaryPath, ...errorFields(err) });
|
|
2433
|
+
}
|
|
2434
|
+
}
|
|
2435
|
+
}
|
|
2436
|
+
function refuse(code, message, fields) {
|
|
2437
|
+
log(code === "write_failed" ? "error" : "warn", "file_push_refused", { code, ...fields });
|
|
2438
|
+
return { ok: false, code, message };
|
|
2439
|
+
}
|
|
2440
|
+
|
|
2441
|
+
// src/lib/runner-file-sync.ts
|
|
2442
|
+
var MAX_ACK_ATTEMPTS = 5;
|
|
2443
|
+
async function syncPendingRunnerFiles(options) {
|
|
2444
|
+
const pending = await listPendingFiles(options);
|
|
2445
|
+
const pendingIds = new Set(pending.map((file) => file.id));
|
|
2446
|
+
for (const id of options.ackFailures.keys()) {
|
|
2447
|
+
if (!pendingIds.has(id)) options.ackFailures.delete(id);
|
|
2448
|
+
}
|
|
2449
|
+
if (pending.length === 0) return 0;
|
|
2450
|
+
options.log({
|
|
2451
|
+
level: "info",
|
|
2452
|
+
message: `Runner file sync: ${pending.length} file(s) queued for this runner`
|
|
2453
|
+
});
|
|
2454
|
+
let applied = 0;
|
|
2455
|
+
for (const file of pending) {
|
|
2456
|
+
if ((options.ackFailures.get(file.id) ?? 0) >= MAX_ACK_ATTEMPTS) continue;
|
|
2457
|
+
if (await applyOne(options, file)) applied += 1;
|
|
2458
|
+
}
|
|
2459
|
+
return applied;
|
|
2460
|
+
}
|
|
2461
|
+
async function listPendingFiles(options) {
|
|
2462
|
+
let res;
|
|
2463
|
+
try {
|
|
2464
|
+
res = await options.fetchImpl(`${options.apiUrl}/runners/${options.agentId}/files/pending`, {
|
|
2465
|
+
headers: { Authorization: options.getAuthHeader() }
|
|
2466
|
+
});
|
|
2467
|
+
} catch (err) {
|
|
2468
|
+
options.log({
|
|
2469
|
+
level: "warn",
|
|
2470
|
+
message: `Could not list pending runner files \u2014 retrying on the next drain: ${describe(err)}`
|
|
2471
|
+
});
|
|
2472
|
+
return [];
|
|
2473
|
+
}
|
|
2474
|
+
if (!res.ok) {
|
|
2475
|
+
options.log({
|
|
2476
|
+
level: res.status === 404 ? "debug" : "warn",
|
|
2477
|
+
message: `Listing pending runner files returned HTTP ${res.status} \u2014 retrying on the next drain`
|
|
2478
|
+
});
|
|
2479
|
+
return [];
|
|
2480
|
+
}
|
|
2481
|
+
let body;
|
|
2482
|
+
try {
|
|
2483
|
+
body = await res.json();
|
|
2484
|
+
} catch (err) {
|
|
2485
|
+
options.log({
|
|
2486
|
+
level: "warn",
|
|
2487
|
+
message: `Pending runner file list was not readable JSON \u2014 retrying on the next drain: ${describe(err)}`
|
|
2488
|
+
});
|
|
2489
|
+
return [];
|
|
2490
|
+
}
|
|
2491
|
+
if (!Array.isArray(body)) {
|
|
2492
|
+
options.log({
|
|
2493
|
+
level: "warn",
|
|
2494
|
+
message: "Pending runner file list was not an array \u2014 ignoring it for this drain"
|
|
2495
|
+
});
|
|
2496
|
+
return [];
|
|
2497
|
+
}
|
|
2498
|
+
const files = [];
|
|
2499
|
+
for (const entry of body) {
|
|
2500
|
+
const file = asPendingFile(entry);
|
|
2501
|
+
if (file === null) {
|
|
2502
|
+
options.log({
|
|
2503
|
+
level: "warn",
|
|
2504
|
+
message: "Ignoring a malformed pending runner file entry (expected id, path and size)"
|
|
2505
|
+
});
|
|
2506
|
+
continue;
|
|
2507
|
+
}
|
|
2508
|
+
files.push(file);
|
|
2509
|
+
}
|
|
2510
|
+
return files;
|
|
2511
|
+
}
|
|
2512
|
+
function asPendingFile(entry) {
|
|
2513
|
+
if (entry === null || typeof entry !== "object") return null;
|
|
2514
|
+
const { id, path, size } = entry;
|
|
2515
|
+
if (typeof id !== "string" || id === "") return null;
|
|
2516
|
+
if (typeof path !== "string" || path === "") return null;
|
|
2517
|
+
if (typeof size !== "number" || !Number.isFinite(size) || size < 0) return null;
|
|
2518
|
+
return { id, path, size };
|
|
2519
|
+
}
|
|
2520
|
+
async function applyOne(options, file) {
|
|
2521
|
+
const label = `${file.id.slice(0, 8)} (${file.path})`;
|
|
2522
|
+
if (options.allowedDirectories.length === 0) {
|
|
2523
|
+
options.log({
|
|
2524
|
+
level: "warn",
|
|
2525
|
+
message: `Runner file ${label} rejected: file sync is not enabled on this runner (start it with --enable-file-sync-to)`
|
|
2526
|
+
});
|
|
2527
|
+
await ack(options, file, "rejected", "file_sync_disabled");
|
|
2528
|
+
return false;
|
|
2529
|
+
}
|
|
2530
|
+
if (file.size > MAX_FILE_PUSH_BYTES) {
|
|
2531
|
+
options.log({
|
|
2532
|
+
level: "warn",
|
|
2533
|
+
message: `Runner file ${label} rejected: declared ${file.size} bytes, the limit is ${MAX_FILE_PUSH_BYTES}`
|
|
2534
|
+
});
|
|
2535
|
+
await ack(options, file, "rejected", "file_too_large");
|
|
2536
|
+
return false;
|
|
2537
|
+
}
|
|
2538
|
+
const download = await downloadContent(options, file, label);
|
|
2539
|
+
if (!download.ok) {
|
|
2540
|
+
if (download.terminal) await ack(options, file, "rejected", download.code);
|
|
2541
|
+
return false;
|
|
2542
|
+
}
|
|
2543
|
+
let outcome;
|
|
2544
|
+
try {
|
|
2545
|
+
outcome = await writePushedFile({
|
|
2546
|
+
requestedPath: file.path,
|
|
2547
|
+
content: download.content,
|
|
2548
|
+
allowedDirectories: options.allowedDirectories,
|
|
2549
|
+
homeDir: options.homeDir
|
|
2550
|
+
});
|
|
2551
|
+
} catch (err) {
|
|
2552
|
+
options.log({
|
|
2553
|
+
level: "error",
|
|
2554
|
+
message: `Runner file ${label} could not be written: ${describe(err)}`
|
|
2555
|
+
});
|
|
2556
|
+
await ack(options, file, "rejected", "write_failed");
|
|
2557
|
+
return false;
|
|
2558
|
+
}
|
|
2559
|
+
if (!outcome.ok) {
|
|
2560
|
+
options.log({
|
|
2561
|
+
level: "warn",
|
|
2562
|
+
message: `Runner file ${label} rejected (${outcome.code}): ${outcome.message}`
|
|
2563
|
+
});
|
|
2564
|
+
await ack(options, file, "rejected", outcome.code);
|
|
2565
|
+
return false;
|
|
2566
|
+
}
|
|
2567
|
+
options.log({
|
|
2568
|
+
level: "info",
|
|
2569
|
+
message: `Runner file ${label} applied (${download.content.byteLength} bytes)`
|
|
2570
|
+
});
|
|
2571
|
+
await ack(options, file, "applied");
|
|
2572
|
+
return true;
|
|
2573
|
+
}
|
|
2574
|
+
function durableDownloadCode(status) {
|
|
2575
|
+
return status === 413 ? "file_too_large" : "write_failed";
|
|
2576
|
+
}
|
|
2577
|
+
async function downloadContent(options, file, label) {
|
|
2578
|
+
try {
|
|
2579
|
+
const res = await options.fetchImpl(
|
|
2580
|
+
`${options.apiUrl}/runners/${options.agentId}/files/${file.id}/content`,
|
|
2581
|
+
{ headers: { Authorization: options.getAuthHeader() } }
|
|
2582
|
+
);
|
|
2583
|
+
if (!res.ok) {
|
|
2584
|
+
const terminal = res.status >= 400 && res.status < 500 && res.status !== 401 && res.status !== 403 && res.status !== 408 && res.status !== 429;
|
|
2585
|
+
if (!terminal) {
|
|
2586
|
+
options.log({
|
|
2587
|
+
level: "warn",
|
|
2588
|
+
message: `Downloading runner file ${label} returned HTTP ${res.status} \u2014 retrying on the next drain`
|
|
2589
|
+
});
|
|
2590
|
+
return { ok: false, terminal: false };
|
|
2591
|
+
}
|
|
2592
|
+
const code = durableDownloadCode(res.status);
|
|
2593
|
+
options.log({
|
|
2594
|
+
level: "error",
|
|
2595
|
+
message: `Downloading runner file ${label} returned HTTP ${res.status} \u2014 rejecting it as ${code} (the bytes never reached the writer)`
|
|
2596
|
+
});
|
|
2597
|
+
return { ok: false, terminal: true, code };
|
|
2598
|
+
}
|
|
2599
|
+
return { ok: true, content: Buffer.from(await res.arrayBuffer()) };
|
|
2600
|
+
} catch (err) {
|
|
2601
|
+
options.log({
|
|
2602
|
+
level: "warn",
|
|
2603
|
+
message: `Downloading runner file ${label} failed \u2014 retrying on the next drain: ${describe(err)}`
|
|
2604
|
+
});
|
|
2605
|
+
return { ok: false, terminal: false };
|
|
2606
|
+
}
|
|
2607
|
+
}
|
|
2608
|
+
async function ack(options, file, status, reason) {
|
|
2609
|
+
const outcome = `${status}${reason ? ` (${reason})` : ""}`;
|
|
2610
|
+
try {
|
|
2611
|
+
const res = await options.fetchImpl(
|
|
2612
|
+
`${options.apiUrl}/runners/${options.agentId}/files/${file.id}/ack`,
|
|
2613
|
+
{
|
|
2614
|
+
method: "POST",
|
|
2615
|
+
headers: {
|
|
2616
|
+
Authorization: options.getAuthHeader(),
|
|
2617
|
+
"Content-Type": "application/json"
|
|
2618
|
+
},
|
|
2619
|
+
body: JSON.stringify(reason ? { status, reason } : { status })
|
|
2620
|
+
}
|
|
2621
|
+
);
|
|
2622
|
+
if (!res.ok) {
|
|
2623
|
+
recordAckFailure(
|
|
2624
|
+
options,
|
|
2625
|
+
file,
|
|
2626
|
+
`Acking runner file ${file.id.slice(0, 8)} as ${outcome} returned HTTP ${res.status}`
|
|
2627
|
+
);
|
|
2628
|
+
return;
|
|
2629
|
+
}
|
|
2630
|
+
options.ackFailures.delete(file.id);
|
|
2631
|
+
} catch (err) {
|
|
2632
|
+
recordAckFailure(
|
|
2633
|
+
options,
|
|
2634
|
+
file,
|
|
2635
|
+
`Acking runner file ${file.id.slice(0, 8)} as ${outcome} failed: ${describe(err)}`
|
|
2636
|
+
);
|
|
2637
|
+
}
|
|
2638
|
+
}
|
|
2639
|
+
function recordAckFailure(options, file, what) {
|
|
2640
|
+
const attempts = (options.ackFailures.get(file.id) ?? 0) + 1;
|
|
2641
|
+
options.ackFailures.set(file.id, attempts);
|
|
2642
|
+
options.log({
|
|
2643
|
+
level: "error",
|
|
2644
|
+
message: attempts >= MAX_ACK_ATTEMPTS ? `${what} \u2014 giving up after ${attempts} attempts. It stays pending until the server expires it; restart the runner to retry.` : `${what} \u2014 it stays pending until a later drain re-acks it (attempt ${attempts} of ${MAX_ACK_ATTEMPTS})`
|
|
2645
|
+
});
|
|
2646
|
+
}
|
|
2647
|
+
function describe(err) {
|
|
2648
|
+
return err instanceof Error ? err.message : String(err);
|
|
2649
|
+
}
|
|
2650
|
+
|
|
1985
2651
|
// src/lib/channels/driver.ts
|
|
1986
2652
|
function messageIdOf(m) {
|
|
1987
2653
|
if (!m || typeof m !== "object") return void 0;
|
|
@@ -2010,7 +2676,10 @@ var DEFAULT_PAUSED_MAX_WAIT_MS = 10 * 60 * 1e3;
|
|
|
2010
2676
|
var DEFAULT_STUCK_QUEUED_MS = 6e4;
|
|
2011
2677
|
var HEARTBEAT_MS = 6e4;
|
|
2012
2678
|
var ABSOLUTE_MAX_PROCESSING_MS = 6 * 60 * 60 * 1e3;
|
|
2679
|
+
var B2_ABANDONMENT_MIN_PINNED_MS = 3 * 6e4;
|
|
2680
|
+
var B2_ABANDONMENT_RECHECK_MS = HEARTBEAT_MS;
|
|
2013
2681
|
var POLL_MISS_GRACE_MS = HEARTBEAT_MS;
|
|
2682
|
+
var MAX_SUPERSEDED_CONVERSATIONS = 256;
|
|
2014
2683
|
var ChannelAuthError = class extends Error {
|
|
2015
2684
|
constructor(message) {
|
|
2016
2685
|
super(message);
|
|
@@ -2033,7 +2702,7 @@ function backoffDelay(attempt, policy) {
|
|
|
2033
2702
|
function isRetryableStatus(status) {
|
|
2034
2703
|
return status === 429 || status >= 500 && status <= 599;
|
|
2035
2704
|
}
|
|
2036
|
-
var ChannelDriver = class {
|
|
2705
|
+
var ChannelDriver = class _ChannelDriver {
|
|
2037
2706
|
agentId;
|
|
2038
2707
|
port;
|
|
2039
2708
|
apiUrl;
|
|
@@ -2047,8 +2716,38 @@ var ChannelDriver = class {
|
|
|
2047
2716
|
pausedMaxWaitMs;
|
|
2048
2717
|
stuckQueuedMs;
|
|
2049
2718
|
now;
|
|
2719
|
+
fileSyncDirectories;
|
|
2720
|
+
homeDir;
|
|
2050
2721
|
/** Cache of conversationId → opencode sessionId. */
|
|
2051
2722
|
sessions = /* @__PURE__ */ new Map();
|
|
2723
|
+
/**
|
|
2724
|
+
* conversationId → the opencode session this runner has ABANDONED as that
|
|
2725
|
+
* conversation's binding (#553), after a genuine (`sessionExists === true`)
|
|
2726
|
+
* dispatch failure: the session still exists but is wedged, so #485's self-heal
|
|
2727
|
+
* must bind a fresh one.
|
|
2728
|
+
*
|
|
2729
|
+
* Dropping the local binding + clearing the server row is not enough on its own:
|
|
2730
|
+
* a SIBLING message dispatched earlier in the same drain is still in-flight under
|
|
2731
|
+
* the same session, and its watcher's routine status writes carry
|
|
2732
|
+
* `opencode_session_id`, RESURRECTING the wedged id server-side after the clear —
|
|
2733
|
+
* and `ensureSession`'s persisted-id fallback then reuses it, defeating the
|
|
2734
|
+
* self-heal. This map makes the runner authoritative instead of racing those
|
|
2735
|
+
* writes: *`ensureSession` never reuses an abandoned id for that conversation,
|
|
2736
|
+
* whatever the server row says* — which holds even when the resurrecting write
|
|
2737
|
+
* is one we deliberately keep (see `markDone`).
|
|
2738
|
+
*
|
|
2739
|
+
* Bounded by construction, on both axes: keyed by CONVERSATION, so N failures on
|
|
2740
|
+
* one conversation hold ONE entry (the newest abandonment replaces the older), and
|
|
2741
|
+
* hard-capped at `MAX_SUPERSEDED_CONVERSATIONS` with FIFO eviction. Only the
|
|
2742
|
+
* NEWEST abandoned id per conversation is guarded: after a second abandonment a
|
|
2743
|
+
* late sibling of the FIRST session can write that id back and `ensureSession`
|
|
2744
|
+
* will reuse it — costing ONE repeat failure, which re-supersedes it. Deliberately
|
|
2745
|
+
* NOT dropped when the session's watcher tears down: `markDone` still writes the
|
|
2746
|
+
* abandoned id back (it must, or the reply is lost), so the guard has to outlive
|
|
2747
|
+
* the turn that resurrects it. In-memory only — a restart forgets it, at the same
|
|
2748
|
+
* bounded cost.
|
|
2749
|
+
*/
|
|
2750
|
+
supersededSessions = /* @__PURE__ */ new Map();
|
|
2052
2751
|
/**
|
|
2053
2752
|
* Per-opencode-session dispatch lock (Task 2.1a). `sendPromptAsync` is no
|
|
2054
2753
|
* longer idempotent (no caller-supplied `messageID`), and its read-back picks
|
|
@@ -2158,9 +2857,12 @@ var ChannelDriver = class {
|
|
|
2158
2857
|
sessionParents = /* @__PURE__ */ new Map();
|
|
2159
2858
|
/**
|
|
2160
2859
|
* Per-session OpenCode title cache (#310), keyed by sessionId. Only a resolved
|
|
2161
|
-
* NON-EMPTY name is stored (terminal — a real session name
|
|
2162
|
-
* so we do NOT re-GET `/session/:id` every tick.
|
|
2163
|
-
*
|
|
2860
|
+
* NON-EMPTY, non-placeholder name is stored (terminal — a real session name
|
|
2861
|
+
* won't later un-name), so we do NOT re-GET `/session/:id` every tick. "Non-empty"
|
|
2862
|
+
* excludes OpenCode's synchronous default title (see
|
|
2863
|
+
* `OPENCODE_DEFAULT_TITLE_PREFIX`, #549) — that placeholder is treated the same
|
|
2864
|
+
* as an empty title so it never latches. A missing entry = not yet resolved OR
|
|
2865
|
+
* resolved-but-still-empty/placeholder → re-fetch on next need, since OpenCode
|
|
2164
2866
|
* names sessions asynchronously mid-turn. Driver-level (not per-watcher) so both
|
|
2165
2867
|
* the watcher completion path AND the restart-recovery re-adopt path (which has
|
|
2166
2868
|
* no watcher) can resolve the title.
|
|
@@ -2168,6 +2870,24 @@ var ChannelDriver = class {
|
|
|
2168
2870
|
sessionTitles = /* @__PURE__ */ new Map();
|
|
2169
2871
|
/** Serialises drains so a reconnect during a drain doesn't double-process. */
|
|
2170
2872
|
draining = false;
|
|
2873
|
+
/**
|
|
2874
|
+
* Serialises runner-file syncs (#559) so the ~2s poll tick and a concurrent
|
|
2875
|
+
* drain ping don't download, write and ack the same file twice.
|
|
2876
|
+
*/
|
|
2877
|
+
syncingFiles = false;
|
|
2878
|
+
/**
|
|
2879
|
+
* Consecutive failed acks per pending file (#559). Lives on the driver so it
|
|
2880
|
+
* survives across drains — without it, a file whose ack keeps failing is
|
|
2881
|
+
* re-downloaded and re-written every ~2s until the server expires it.
|
|
2882
|
+
*/
|
|
2883
|
+
fileAckFailures = /* @__PURE__ */ new Map();
|
|
2884
|
+
/**
|
|
2885
|
+
* Monotonic count of files this runner has pulled and written (#559). Only
|
|
2886
|
+
* ever increases, so `run.ts` detects work by comparing it against the value
|
|
2887
|
+
* it saw on the previous cycle — including work that landed mid-sleep, the
|
|
2888
|
+
* same trick `lastProxiedActivityAt` uses.
|
|
2889
|
+
*/
|
|
2890
|
+
appliedFileCount = 0;
|
|
2171
2891
|
/**
|
|
2172
2892
|
* The currently-executing `drainPending()` promise, or null when idle. Lets a
|
|
2173
2893
|
* graceful shutdown (`waitForInFlight`) await an in-progress drain so a turn it
|
|
@@ -2198,6 +2918,8 @@ var ChannelDriver = class {
|
|
|
2198
2918
|
this.pausedMaxWaitMs = config2.pausedMaxWaitMs ?? DEFAULT_PAUSED_MAX_WAIT_MS;
|
|
2199
2919
|
this.stuckQueuedMs = config2.stuckQueuedMs ?? DEFAULT_STUCK_QUEUED_MS;
|
|
2200
2920
|
this.now = config2.now ?? (() => Date.now());
|
|
2921
|
+
this.fileSyncDirectories = config2.fileSyncDirectories ?? [];
|
|
2922
|
+
this.homeDir = config2.homeDir ?? homedir();
|
|
2201
2923
|
}
|
|
2202
2924
|
/** The IPv4-loopback base URL for the local `opencode serve`. */
|
|
2203
2925
|
get opencodeBase() {
|
|
@@ -2225,6 +2947,47 @@ var ChannelDriver = class {
|
|
|
2225
2947
|
);
|
|
2226
2948
|
return run2;
|
|
2227
2949
|
}
|
|
2950
|
+
/**
|
|
2951
|
+
* Pull-and-apply any files Evident has queued for this runner (#559), riding
|
|
2952
|
+
* the EXISTING drain cycle — `run.ts` calls it from the same ~2s channel poll
|
|
2953
|
+
* and drain ping that call `drainPending()`. There is deliberately no channel,
|
|
2954
|
+
* control frame or poll loop of its own: worst-case latency is one poll tick.
|
|
2955
|
+
*
|
|
2956
|
+
* NEVER throws and never surfaces a `ChannelAuthError`: a file failure must not
|
|
2957
|
+
* cost a conversation turn. Failures are logged and either acked as a terminal
|
|
2958
|
+
* outcome or left pending for the next drain (see `runner-file-sync.ts`).
|
|
2959
|
+
*
|
|
2960
|
+
* Re-entrant calls are skipped (the poll tick and a drain ping can overlap).
|
|
2961
|
+
*
|
|
2962
|
+
* @returns the number of files written to disk.
|
|
2963
|
+
*/
|
|
2964
|
+
async syncPendingFiles() {
|
|
2965
|
+
if (this.stopped) return 0;
|
|
2966
|
+
if (this.syncingFiles) return 0;
|
|
2967
|
+
this.syncingFiles = true;
|
|
2968
|
+
try {
|
|
2969
|
+
const applied = await syncPendingRunnerFiles({
|
|
2970
|
+
agentId: this.agentId,
|
|
2971
|
+
apiUrl: this.apiUrl,
|
|
2972
|
+
getAuthHeader: this.getAuthHeader,
|
|
2973
|
+
fetchImpl: this.fetchImpl,
|
|
2974
|
+
allowedDirectories: this.fileSyncDirectories,
|
|
2975
|
+
homeDir: this.homeDir,
|
|
2976
|
+
ackFailures: this.fileAckFailures,
|
|
2977
|
+
log: this.log
|
|
2978
|
+
});
|
|
2979
|
+
this.appliedFileCount += applied;
|
|
2980
|
+
return applied;
|
|
2981
|
+
} catch (err) {
|
|
2982
|
+
this.log({
|
|
2983
|
+
level: "error",
|
|
2984
|
+
message: `Runner file sync failed unexpectedly (message processing is unaffected): ${err instanceof Error ? err.message : String(err)}`
|
|
2985
|
+
});
|
|
2986
|
+
return 0;
|
|
2987
|
+
} finally {
|
|
2988
|
+
this.syncingFiles = false;
|
|
2989
|
+
}
|
|
2990
|
+
}
|
|
2228
2991
|
async runDrain() {
|
|
2229
2992
|
let dispatched = 0;
|
|
2230
2993
|
try {
|
|
@@ -2258,6 +3021,28 @@ var ChannelDriver = class {
|
|
|
2258
3021
|
}
|
|
2259
3022
|
return false;
|
|
2260
3023
|
}
|
|
3024
|
+
/**
|
|
3025
|
+
* File-pull work, for `run.ts`'s idle accounting (#559).
|
|
3026
|
+
*
|
|
3027
|
+
* Pulling a file is real work that `drainPending()` knows nothing about, so
|
|
3028
|
+
* without this a near-idle runner counts a credential pull as an empty tick
|
|
3029
|
+
* and `--idle-timeout` can `process.exit` mid-pull — leaving a
|
|
3030
|
+
* `.evident-push-*.tmp` behind — or immediately after the write, before the
|
|
3031
|
+
* browser has run the authorize/callback that activates it (the user then sees
|
|
3032
|
+
* `saved_not_activated` for a runner that was fine).
|
|
3033
|
+
*
|
|
3034
|
+
* Two signals because one cannot cover both cases: `inFlight` is the pull
|
|
3035
|
+
* happening RIGHT NOW (it may outlive the tick that started it), and
|
|
3036
|
+
* `appliedFiles` is monotonic so a pull that started AND finished between two
|
|
3037
|
+
* idle checks still shows up as an advance.
|
|
3038
|
+
*
|
|
3039
|
+
* CALLER CONTRACT: sample `inFlight` BEFORE calling `syncPendingFiles()` for
|
|
3040
|
+
* the cycle. `syncPendingFiles` sets the flag synchronously, so a caller that
|
|
3041
|
+
* samples afterwards reads `true` every single cycle and can never idle out.
|
|
3042
|
+
*/
|
|
3043
|
+
fileSyncActivity() {
|
|
3044
|
+
return { appliedFiles: this.appliedFileCount, inFlight: this.syncingFiles };
|
|
3045
|
+
}
|
|
2261
3046
|
/**
|
|
2262
3047
|
* OpenCode session ids the session-cleanup sweep (issue #190) must NOT delete:
|
|
2263
3048
|
* exactly those with a live (dispatched-but-not-done / paused) turn, i.e. a
|
|
@@ -2319,7 +3104,7 @@ var ChannelDriver = class {
|
|
|
2319
3104
|
await this.sleep(step);
|
|
2320
3105
|
}
|
|
2321
3106
|
}
|
|
2322
|
-
while (this.hasInFlightWatchers()) {
|
|
3107
|
+
while (this.hasInFlightWatchers() || this.syncingFiles) {
|
|
2323
3108
|
if (this.now() >= deadline) return false;
|
|
2324
3109
|
await this.sleep(step);
|
|
2325
3110
|
}
|
|
@@ -2354,10 +3139,15 @@ var ChannelDriver = class {
|
|
|
2354
3139
|
* @returns the count of messages NEWLY dispatched (not already in-flight).
|
|
2355
3140
|
*/
|
|
2356
3141
|
async processConversation(conv) {
|
|
2357
|
-
const sessionId = await this.ensureSession(conv);
|
|
3142
|
+
const { sessionId, refusedSessionId } = await this.ensureSession(conv);
|
|
2358
3143
|
const messages = await this.getPendingMessages(conv.id);
|
|
2359
3144
|
let dispatched = 0;
|
|
2360
3145
|
let skippedAlreadyDispatched = 0;
|
|
3146
|
+
if (refusedSessionId && messages.length > 0) {
|
|
3147
|
+
void this.postSignal(conv.id, messages[0].id, "session_superseded", {
|
|
3148
|
+
superseded_session_id: refusedSessionId
|
|
3149
|
+
});
|
|
3150
|
+
}
|
|
2361
3151
|
for (const message of messages) {
|
|
2362
3152
|
if (this.stopped) break;
|
|
2363
3153
|
if (this.dispatched.has(message.id)) {
|
|
@@ -2384,7 +3174,8 @@ var ChannelDriver = class {
|
|
|
2384
3174
|
} catch (err) {
|
|
2385
3175
|
if (err instanceof ChannelAuthError) throw err;
|
|
2386
3176
|
this.dispatched.delete(message.id);
|
|
2387
|
-
|
|
3177
|
+
const exists = await sessionExists(this.port, sessionId);
|
|
3178
|
+
if (exists === false) {
|
|
2388
3179
|
this.sessions.delete(conv.id);
|
|
2389
3180
|
this.log({
|
|
2390
3181
|
level: "warn",
|
|
@@ -2394,15 +3185,39 @@ var ChannelDriver = class {
|
|
|
2394
3185
|
});
|
|
2395
3186
|
break;
|
|
2396
3187
|
}
|
|
2397
|
-
|
|
3188
|
+
if (exists === null) {
|
|
3189
|
+
this.log({
|
|
3190
|
+
level: "warn",
|
|
3191
|
+
message: `Message ${message.id.slice(0, 8)} dispatch failed and session (${sessionId.slice(0, 8)}) existence could not be confirmed (opencode momentarily unreachable) \u2014 deferring this and later messages for conversation ${conv.id.slice(0, 8)} to the next tick rather than treating it as a genuine failure.`,
|
|
3192
|
+
conversation_id: conv.id,
|
|
3193
|
+
message_id: message.id
|
|
3194
|
+
});
|
|
3195
|
+
break;
|
|
3196
|
+
}
|
|
3197
|
+
const errorMessage = err instanceof Error ? err.message : String(err);
|
|
3198
|
+
this.sessions.delete(conv.id);
|
|
3199
|
+
this.supersede(conv.id, sessionId);
|
|
3200
|
+
this.log({
|
|
3201
|
+
level: "warn",
|
|
3202
|
+
message: `Abandoning OpenCode session ${sessionId.slice(0, 8)} as the binding for conversation ${conv.id.slice(0, 8)} (it exists but failed to run a turn) \u2014 a fresh session is created on the next tick, whatever the persisted binding says by then.`,
|
|
3203
|
+
conversation_id: conv.id,
|
|
3204
|
+
message_id: message.id
|
|
3205
|
+
});
|
|
3206
|
+
await this.markFailed(conv.id, message.id, null, errorMessage).catch((markErr) => {
|
|
3207
|
+
this.log({
|
|
3208
|
+
level: "warn",
|
|
3209
|
+
message: `markFailed PATCH for message ${message.id.slice(0, 8)} (conversation ${conv.id.slice(0, 8)}) failed (best-effort, not retried): ${markErr instanceof Error ? markErr.message : String(markErr)}`,
|
|
3210
|
+
conversation_id: conv.id,
|
|
3211
|
+
message_id: message.id
|
|
3212
|
+
});
|
|
2398
3213
|
});
|
|
2399
3214
|
this.log({
|
|
2400
3215
|
level: "error",
|
|
2401
|
-
message: `Message ${message.id.slice(0, 8)} dispatch failed: ${
|
|
3216
|
+
message: `Message ${message.id.slice(0, 8)} dispatch failed: ${errorMessage}`,
|
|
2402
3217
|
conversation_id: conv.id,
|
|
2403
3218
|
message_id: message.id
|
|
2404
3219
|
});
|
|
2405
|
-
|
|
3220
|
+
break;
|
|
2406
3221
|
}
|
|
2407
3222
|
if (opencodeMessageId === null) {
|
|
2408
3223
|
this.log({
|
|
@@ -2428,8 +3243,42 @@ var ChannelDriver = class {
|
|
|
2428
3243
|
this.ensureWatcherRunning(sessionId);
|
|
2429
3244
|
return dispatched;
|
|
2430
3245
|
}
|
|
3246
|
+
/**
|
|
3247
|
+
* Record that `sessionId` is no longer a valid binding for `conversationId`
|
|
3248
|
+
* (#553). Keyed by conversation and hard-capped, so it cannot grow with the
|
|
3249
|
+
* number of failures — see the `supersededSessions` field doc.
|
|
3250
|
+
*/
|
|
3251
|
+
supersede(conversationId, sessionId) {
|
|
3252
|
+
this.supersededSessions.delete(conversationId);
|
|
3253
|
+
this.supersededSessions.set(conversationId, sessionId);
|
|
3254
|
+
while (this.supersededSessions.size > MAX_SUPERSEDED_CONVERSATIONS) {
|
|
3255
|
+
const oldest = this.supersededSessions.keys().next().value;
|
|
3256
|
+
if (oldest === void 0) return;
|
|
3257
|
+
this.supersededSessions.delete(oldest);
|
|
3258
|
+
}
|
|
3259
|
+
}
|
|
3260
|
+
/** Whether `sessionId` is the session this conversation has abandoned (#553). */
|
|
3261
|
+
isSuperseded(conversationId, sessionId) {
|
|
3262
|
+
return this.supersededSessions.get(conversationId) === sessionId;
|
|
3263
|
+
}
|
|
3264
|
+
/**
|
|
3265
|
+
* Resolve the opencode session to run this conversation's turns in.
|
|
3266
|
+
*
|
|
3267
|
+
* `refusedSessionId` is set when the #553 guard fired — i.e. the persisted
|
|
3268
|
+
* binding was an id this runner had abandoned, so a resurrection genuinely
|
|
3269
|
+
* happened and a fresh session was bound instead. The caller reports it.
|
|
3270
|
+
*/
|
|
2431
3271
|
async ensureSession(conv) {
|
|
2432
3272
|
const bound = this.sessions.get(conv.id) ?? conv.opencode_session_id ?? null;
|
|
3273
|
+
if (bound && this.isSuperseded(conv.id, bound)) {
|
|
3274
|
+
this.log({
|
|
3275
|
+
level: "warn",
|
|
3276
|
+
message: `OpenCode session ${bound.slice(0, 8)} was abandoned for conversation ${conv.id.slice(0, 8)} after a failed dispatch but is still bound to it (the persisted id was written back by a turn already in flight) \u2014 ignoring it and binding a fresh session.`,
|
|
3277
|
+
conversation_id: conv.id
|
|
3278
|
+
});
|
|
3279
|
+
this.sessions.delete(conv.id);
|
|
3280
|
+
return { sessionId: await this.createAndBindSession(conv.id), refusedSessionId: bound };
|
|
3281
|
+
}
|
|
2433
3282
|
if (bound) {
|
|
2434
3283
|
const exists = await sessionExists(this.port, bound);
|
|
2435
3284
|
if (exists === false) {
|
|
@@ -2439,12 +3288,12 @@ var ChannelDriver = class {
|
|
|
2439
3288
|
conversation_id: conv.id
|
|
2440
3289
|
});
|
|
2441
3290
|
this.sessions.delete(conv.id);
|
|
2442
|
-
return this.createAndBindSession(conv.id);
|
|
3291
|
+
return { sessionId: await this.createAndBindSession(conv.id) };
|
|
2443
3292
|
}
|
|
2444
3293
|
this.sessions.set(conv.id, bound);
|
|
2445
|
-
return bound;
|
|
3294
|
+
return { sessionId: bound };
|
|
2446
3295
|
}
|
|
2447
|
-
return this.createAndBindSession(conv.id);
|
|
3296
|
+
return { sessionId: await this.createAndBindSession(conv.id) };
|
|
2448
3297
|
}
|
|
2449
3298
|
/**
|
|
2450
3299
|
* Create a fresh OpenCode session for a conversation, cache the binding, and
|
|
@@ -2532,7 +3381,11 @@ var ChannelDriver = class {
|
|
|
2532
3381
|
* (not-owned / out-of-range / deleted-at-source / workspace gone) / 413
|
|
2533
3382
|
* (over-cap). On ANY non-2xx or thrown failure we return `null` so the caller
|
|
2534
3383
|
* OMITS that one image and the text turn still sends — NEVER throws the turn.
|
|
2535
|
-
*
|
|
3384
|
+
* A 404 body carrying `{ reason: 'needs_reauth' }` (#547 — the server CONFIRMED
|
|
3385
|
+
* a Slack `files:read` scope problem via `files.info`) instead resolves the
|
|
3386
|
+
* `AttachmentFetchNeedsReauth` sentinel, so the in-thread note can steer the
|
|
3387
|
+
* user to reconnect Slack instead of a generic "unavailable". Failures are
|
|
3388
|
+
* logged with context (no silent swallow).
|
|
2536
3389
|
*/
|
|
2537
3390
|
async fetchAttachmentDataUrl(messageId, index, mime) {
|
|
2538
3391
|
try {
|
|
@@ -2541,6 +3394,25 @@ var ChannelDriver = class {
|
|
|
2541
3394
|
{ headers: { Authorization: this.getAuthHeader() } }
|
|
2542
3395
|
);
|
|
2543
3396
|
if (!res.ok) {
|
|
3397
|
+
let reason;
|
|
3398
|
+
try {
|
|
3399
|
+
const body = await res.json();
|
|
3400
|
+
if (body && typeof body.reason === "string") reason = body.reason;
|
|
3401
|
+
} catch (parseErr) {
|
|
3402
|
+
this.log({
|
|
3403
|
+
level: "debug",
|
|
3404
|
+
message: `Attachment fetch for message ${messageId.slice(0, 8)} index ${index}: error body was not JSON (${parseErr instanceof Error ? parseErr.message : String(parseErr)}) \u2014 treating as a plain failure`,
|
|
3405
|
+
message_id: messageId
|
|
3406
|
+
});
|
|
3407
|
+
}
|
|
3408
|
+
if (reason === "needs_reauth") {
|
|
3409
|
+
this.log({
|
|
3410
|
+
level: "error",
|
|
3411
|
+
message: `Attachment fetch for message ${messageId.slice(0, 8)} index ${index} returned HTTP ${res.status} \u2014 server confirmed a Slack reauth/scope problem \u2014 omitting this image (text turn proceeds)`,
|
|
3412
|
+
message_id: messageId
|
|
3413
|
+
});
|
|
3414
|
+
return { needsReauth: true };
|
|
3415
|
+
}
|
|
2544
3416
|
this.log({
|
|
2545
3417
|
level: "error",
|
|
2546
3418
|
message: `Attachment fetch for message ${messageId.slice(0, 8)} index ${index} returned HTTP ${res.status} \u2014 omitting this image (text turn proceeds)`,
|
|
@@ -2580,6 +3452,9 @@ var ChannelDriver = class {
|
|
|
2580
3452
|
if (this.attachmentsSkippedSignalled.has(messageId)) return;
|
|
2581
3453
|
this.attachmentsSkippedSignalled.add(messageId);
|
|
2582
3454
|
const skippedReason = capabilityUnknown ? "unknown" : "unsupported";
|
|
3455
|
+
const failedReason = outcomes.some(
|
|
3456
|
+
(o) => o.status === "failed" && o.reason === "needs_reauth"
|
|
3457
|
+
) ? "needs_reauth" : void 0;
|
|
2583
3458
|
this.log({
|
|
2584
3459
|
level: "info",
|
|
2585
3460
|
message: `Message ${messageId.slice(0, 8)}: ${skipped} image(s) skipped (${capabilityUnknown ? "capability was unreadable \u2014 failed open to text-only" : "model not attachment-capable"}), ${failed} image(s) unavailable (deleted-at-source or fetch failure) \u2014 noting to Evident`,
|
|
@@ -2589,7 +3464,8 @@ var ChannelDriver = class {
|
|
|
2589
3464
|
void this.postSignal(conversationId, messageId, "attachments_skipped", {
|
|
2590
3465
|
skipped,
|
|
2591
3466
|
failed,
|
|
2592
|
-
...skipped > 0 ? { skipped_reason: skippedReason } : {}
|
|
3467
|
+
...skipped > 0 ? { skipped_reason: skippedReason } : {},
|
|
3468
|
+
...failedReason ? { failed_reason: failedReason } : {}
|
|
2593
3469
|
});
|
|
2594
3470
|
}
|
|
2595
3471
|
/** Register a freshly-dispatched message with its session's watcher state. */
|
|
@@ -2620,12 +3496,17 @@ var ChannelDriver = class {
|
|
|
2620
3496
|
stuckReported: false,
|
|
2621
3497
|
lastAliveAt: 0,
|
|
2622
3498
|
aliveInFlight: false,
|
|
3499
|
+
titleSynced: false,
|
|
3500
|
+
titleSyncInFlight: false,
|
|
2623
3501
|
awaitingHumanLatched: false,
|
|
2624
3502
|
pausedOnQuestion: false,
|
|
2625
3503
|
pausedOnPermission: false,
|
|
2626
3504
|
pausedClearConfirmed: false,
|
|
2627
3505
|
pausedInFlight: false,
|
|
2628
|
-
deliveryDeadlineAnchored: false
|
|
3506
|
+
deliveryDeadlineAnchored: false,
|
|
3507
|
+
b2PinnedSinceMs: 0,
|
|
3508
|
+
b2LastDescendantCheckMs: 0,
|
|
3509
|
+
b2AbandonedSignalled: false
|
|
2629
3510
|
});
|
|
2630
3511
|
}
|
|
2631
3512
|
/**
|
|
@@ -2693,12 +3574,17 @@ var ChannelDriver = class {
|
|
|
2693
3574
|
// with no extra `re_adopted` signal needed (folds old WI-6).
|
|
2694
3575
|
lastAliveAt: 0,
|
|
2695
3576
|
aliveInFlight: false,
|
|
3577
|
+
titleSynced: false,
|
|
3578
|
+
titleSyncInFlight: false,
|
|
2696
3579
|
awaitingHumanLatched: false,
|
|
2697
3580
|
pausedOnQuestion: false,
|
|
2698
3581
|
pausedOnPermission: false,
|
|
2699
3582
|
pausedClearConfirmed: false,
|
|
2700
3583
|
pausedInFlight: false,
|
|
2701
|
-
deliveryDeadlineAnchored: false
|
|
3584
|
+
deliveryDeadlineAnchored: false,
|
|
3585
|
+
b2PinnedSinceMs: 0,
|
|
3586
|
+
b2LastDescendantCheckMs: 0,
|
|
3587
|
+
b2AbandonedSignalled: false
|
|
2702
3588
|
});
|
|
2703
3589
|
}
|
|
2704
3590
|
/**
|
|
@@ -2853,65 +3739,14 @@ var ChannelDriver = class {
|
|
|
2853
3739
|
if (!claimed) {
|
|
2854
3740
|
this.log({
|
|
2855
3741
|
level: "debug",
|
|
2856
|
-
message: `Message ${inFlight.evidentMessageId.slice(0, 8)} already marked processing \u2014 continuing`,
|
|
2857
|
-
conversation_id: conv.id,
|
|
2858
|
-
message_id: inFlight.evidentMessageId
|
|
2859
|
-
});
|
|
2860
|
-
}
|
|
2861
|
-
}
|
|
2862
|
-
if (state === "done") {
|
|
2863
|
-
this.anchorDeliveryDeadline(inFlight);
|
|
2864
|
-
if (!inFlight.done) {
|
|
2865
|
-
this.log({
|
|
2866
|
-
level: "info",
|
|
2867
|
-
message: `Message ${inFlight.evidentMessageId.slice(0, 8)} completed \u2014 marking done`,
|
|
3742
|
+
message: `Message ${inFlight.evidentMessageId.slice(0, 8)} already marked processing \u2014 continuing`,
|
|
2868
3743
|
conversation_id: conv.id,
|
|
2869
3744
|
message_id: inFlight.evidentMessageId
|
|
2870
3745
|
});
|
|
2871
|
-
const title = await this.resolveSessionTitle(sessionId, watcher.conv.id);
|
|
2872
|
-
const usage = messageUsage(messages, inFlight.opencodeMessageId);
|
|
2873
|
-
try {
|
|
2874
|
-
await this.markDone(
|
|
2875
|
-
conv.id,
|
|
2876
|
-
inFlight.evidentMessageId,
|
|
2877
|
-
sessionId,
|
|
2878
|
-
inFlight.opencodeMessageId,
|
|
2879
|
-
title,
|
|
2880
|
-
usage
|
|
2881
|
-
);
|
|
2882
|
-
} catch (err) {
|
|
2883
|
-
if (err instanceof ChannelAuthError) throw err;
|
|
2884
|
-
if (err instanceof ChannelTerminalError) {
|
|
2885
|
-
this.log({
|
|
2886
|
-
level: "warn",
|
|
2887
|
-
message: `Failed to mark message ${inFlight.evidentMessageId.slice(0, 8)} done (terminal HTTP ${err.status}) \u2014 leaving for the cron safety net: ${err.message}`,
|
|
2888
|
-
conversation_id: conv.id,
|
|
2889
|
-
message_id: inFlight.evidentMessageId
|
|
2890
|
-
});
|
|
2891
|
-
this.removeInFlight(watcher, inFlight.evidentMessageId);
|
|
2892
|
-
return;
|
|
2893
|
-
}
|
|
2894
|
-
if (this.now() >= inFlight.deadline) {
|
|
2895
|
-
this.log({
|
|
2896
|
-
level: "warn",
|
|
2897
|
-
message: `Failed to mark message ${inFlight.evidentMessageId.slice(0, 8)} done within the watch window \u2014 leaving for the cron safety net: ${err instanceof Error ? err.message : String(err)}`,
|
|
2898
|
-
conversation_id: conv.id,
|
|
2899
|
-
message_id: inFlight.evidentMessageId
|
|
2900
|
-
});
|
|
2901
|
-
this.removeInFlight(watcher, inFlight.evidentMessageId);
|
|
2902
|
-
return;
|
|
2903
|
-
}
|
|
2904
|
-
this.log({
|
|
2905
|
-
level: "warn",
|
|
2906
|
-
message: `Failed to mark message ${inFlight.evidentMessageId.slice(0, 8)} done (will retry next tick): ${err instanceof Error ? err.message : String(err)}`,
|
|
2907
|
-
conversation_id: conv.id,
|
|
2908
|
-
message_id: inFlight.evidentMessageId
|
|
2909
|
-
});
|
|
2910
|
-
return;
|
|
2911
|
-
}
|
|
2912
|
-
inFlight.done = true;
|
|
2913
3746
|
}
|
|
2914
|
-
|
|
3747
|
+
}
|
|
3748
|
+
if (state === "done") {
|
|
3749
|
+
await this.settleMessageDone(sessionId, watcher, inFlight, messages);
|
|
2915
3750
|
return;
|
|
2916
3751
|
}
|
|
2917
3752
|
if (state === "failed") {
|
|
@@ -2925,8 +3760,16 @@ var ChannelDriver = class {
|
|
|
2925
3760
|
message_id: inFlight.evidentMessageId
|
|
2926
3761
|
});
|
|
2927
3762
|
const usage = messageUsage(messages, inFlight.opencodeMessageId);
|
|
3763
|
+
const failure = await this.classifyModelAuthFailure(messages, inFlight.opencodeMessageId);
|
|
2928
3764
|
try {
|
|
2929
|
-
await this.markFailed(
|
|
3765
|
+
await this.markFailed(
|
|
3766
|
+
conv.id,
|
|
3767
|
+
inFlight.evidentMessageId,
|
|
3768
|
+
sessionId,
|
|
3769
|
+
error2,
|
|
3770
|
+
usage,
|
|
3771
|
+
failure
|
|
3772
|
+
);
|
|
2930
3773
|
} catch (err) {
|
|
2931
3774
|
if (err instanceof ChannelAuthError) throw err;
|
|
2932
3775
|
if (err instanceof ChannelTerminalError) {
|
|
@@ -2971,6 +3814,44 @@ var ChannelDriver = class {
|
|
|
2971
3814
|
});
|
|
2972
3815
|
}
|
|
2973
3816
|
const activelyRunning = state === "running" && !awaitingHuman;
|
|
3817
|
+
const pinnedNow = activelyRunning && isPreamblePinnedRunning(messages, inFlight.opencodeMessageId);
|
|
3818
|
+
const snapshotReadable = messages != null && messages.length > 0;
|
|
3819
|
+
if (!pinnedNow) {
|
|
3820
|
+
if (snapshotReadable) {
|
|
3821
|
+
inFlight.b2PinnedSinceMs = 0;
|
|
3822
|
+
inFlight.b2LastDescendantCheckMs = 0;
|
|
3823
|
+
inFlight.b2AbandonedSignalled = false;
|
|
3824
|
+
}
|
|
3825
|
+
} else {
|
|
3826
|
+
if (inFlight.b2AbandonedSignalled) {
|
|
3827
|
+
await this.settleMessageDone(sessionId, watcher, inFlight, messages);
|
|
3828
|
+
return;
|
|
3829
|
+
}
|
|
3830
|
+
if (inFlight.b2PinnedSinceMs === 0) inFlight.b2PinnedSinceMs = this.now();
|
|
3831
|
+
const pinnedForMs = this.now() - inFlight.b2PinnedSinceMs;
|
|
3832
|
+
if (pinnedForMs >= B2_ABANDONMENT_MIN_PINNED_MS && this.now() - inFlight.b2LastDescendantCheckMs >= B2_ABANDONMENT_RECHECK_MS) {
|
|
3833
|
+
inFlight.b2LastDescendantCheckMs = this.now();
|
|
3834
|
+
const descendantOngoing = await this.isAnyDescendantSessionOngoing(sessionId);
|
|
3835
|
+
if (isB2AbandonmentConfirmed({
|
|
3836
|
+
pinnedForMs,
|
|
3837
|
+
minPinnedMs: B2_ABANDONMENT_MIN_PINNED_MS,
|
|
3838
|
+
descendantOngoing
|
|
3839
|
+
})) {
|
|
3840
|
+
inFlight.b2AbandonedSignalled = true;
|
|
3841
|
+
this.log({
|
|
3842
|
+
level: "warn",
|
|
3843
|
+
message: `Message ${id.slice(0, 8)} b2-pinned for ${Math.round(pinnedForMs / 1e3)}s with no ongoing descendant sub-agent session (status-map confirmed) \u2014 treating the delegated/tool turn as abandoned, resolving done`,
|
|
3844
|
+
conversation_id: conv.id,
|
|
3845
|
+
message_id: id
|
|
3846
|
+
});
|
|
3847
|
+
void this.postSignal(conv.id, id, "b2_abandoned_resolved", {
|
|
3848
|
+
watched_for_ms: pinnedForMs
|
|
3849
|
+
});
|
|
3850
|
+
await this.settleMessageDone(sessionId, watcher, inFlight, messages);
|
|
3851
|
+
return;
|
|
3852
|
+
}
|
|
3853
|
+
}
|
|
3854
|
+
}
|
|
2974
3855
|
if (activelyRunning && this.now() - inFlight.processingAnchorMs >= ABSOLUTE_MAX_PROCESSING_MS) {
|
|
2975
3856
|
this.log({
|
|
2976
3857
|
level: "warn",
|
|
@@ -2990,6 +3871,18 @@ var ChannelDriver = class {
|
|
|
2990
3871
|
inFlight.aliveInFlight = false;
|
|
2991
3872
|
if (ok) inFlight.lastAliveAt = this.now();
|
|
2992
3873
|
});
|
|
3874
|
+
if (!inFlight.titleSynced && !inFlight.titleSyncInFlight) {
|
|
3875
|
+
inFlight.titleSyncInFlight = true;
|
|
3876
|
+
void this.resolveSessionTitle(sessionId, conv.id).then(async (title) => {
|
|
3877
|
+
if (!title) {
|
|
3878
|
+
inFlight.titleSyncInFlight = false;
|
|
3879
|
+
return;
|
|
3880
|
+
}
|
|
3881
|
+
const ok = await this.patchConversationTitle(conv.id, title);
|
|
3882
|
+
inFlight.titleSyncInFlight = false;
|
|
3883
|
+
if (ok) inFlight.titleSynced = true;
|
|
3884
|
+
});
|
|
3885
|
+
}
|
|
2993
3886
|
}
|
|
2994
3887
|
if (awaitingHuman) {
|
|
2995
3888
|
if (!inFlight.awaitingHumanLatched) {
|
|
@@ -3027,6 +3920,70 @@ var ChannelDriver = class {
|
|
|
3027
3920
|
this.removeInFlight(watcher, inFlight.evidentMessageId);
|
|
3028
3921
|
}
|
|
3029
3922
|
}
|
|
3923
|
+
/**
|
|
3924
|
+
* Settle a message whose run-state has resolved `'done'` — extracted verbatim
|
|
3925
|
+
* (pure refactor, no behavior change) from `serviceInFlightMessage`'s former
|
|
3926
|
+
* inline `state === 'done'` branch body, so a SECOND caller (the #721
|
|
3927
|
+
* b2-abandonment resolution) can reach the exact same completion behavior
|
|
3928
|
+
* (delivery-deadline anchoring, title resolution, usage extraction, and
|
|
3929
|
+
* `markDone`'s auth/terminal/transient-retry discipline) without duplicating it
|
|
3930
|
+
* and risking the two copies silently drifting apart.
|
|
3931
|
+
*/
|
|
3932
|
+
async settleMessageDone(sessionId, watcher, inFlight, messages) {
|
|
3933
|
+
const conv = watcher.conv;
|
|
3934
|
+
this.anchorDeliveryDeadline(inFlight);
|
|
3935
|
+
if (!inFlight.done) {
|
|
3936
|
+
this.log({
|
|
3937
|
+
level: "info",
|
|
3938
|
+
message: `Message ${inFlight.evidentMessageId.slice(0, 8)} completed \u2014 marking done`,
|
|
3939
|
+
conversation_id: conv.id,
|
|
3940
|
+
message_id: inFlight.evidentMessageId
|
|
3941
|
+
});
|
|
3942
|
+
const title = await this.resolveSessionTitle(sessionId, watcher.conv.id);
|
|
3943
|
+
const usage = messageUsage(messages, inFlight.opencodeMessageId);
|
|
3944
|
+
try {
|
|
3945
|
+
await this.markDone(
|
|
3946
|
+
conv.id,
|
|
3947
|
+
inFlight.evidentMessageId,
|
|
3948
|
+
sessionId,
|
|
3949
|
+
inFlight.opencodeMessageId,
|
|
3950
|
+
title,
|
|
3951
|
+
usage
|
|
3952
|
+
);
|
|
3953
|
+
} catch (err) {
|
|
3954
|
+
if (err instanceof ChannelAuthError) throw err;
|
|
3955
|
+
if (err instanceof ChannelTerminalError) {
|
|
3956
|
+
this.log({
|
|
3957
|
+
level: "warn",
|
|
3958
|
+
message: `Failed to mark message ${inFlight.evidentMessageId.slice(0, 8)} done (terminal HTTP ${err.status}) \u2014 leaving for the cron safety net: ${err.message}`,
|
|
3959
|
+
conversation_id: conv.id,
|
|
3960
|
+
message_id: inFlight.evidentMessageId
|
|
3961
|
+
});
|
|
3962
|
+
this.removeInFlight(watcher, inFlight.evidentMessageId);
|
|
3963
|
+
return;
|
|
3964
|
+
}
|
|
3965
|
+
if (this.now() >= inFlight.deadline) {
|
|
3966
|
+
this.log({
|
|
3967
|
+
level: "warn",
|
|
3968
|
+
message: `Failed to mark message ${inFlight.evidentMessageId.slice(0, 8)} done within the watch window \u2014 leaving for the cron safety net: ${err instanceof Error ? err.message : String(err)}`,
|
|
3969
|
+
conversation_id: conv.id,
|
|
3970
|
+
message_id: inFlight.evidentMessageId
|
|
3971
|
+
});
|
|
3972
|
+
this.removeInFlight(watcher, inFlight.evidentMessageId);
|
|
3973
|
+
return;
|
|
3974
|
+
}
|
|
3975
|
+
this.log({
|
|
3976
|
+
level: "warn",
|
|
3977
|
+
message: `Failed to mark message ${inFlight.evidentMessageId.slice(0, 8)} done (will retry next tick): ${err instanceof Error ? err.message : String(err)}`,
|
|
3978
|
+
conversation_id: conv.id,
|
|
3979
|
+
message_id: inFlight.evidentMessageId
|
|
3980
|
+
});
|
|
3981
|
+
return;
|
|
3982
|
+
}
|
|
3983
|
+
inFlight.done = true;
|
|
3984
|
+
}
|
|
3985
|
+
this.removeInFlight(watcher, inFlight.evidentMessageId);
|
|
3986
|
+
}
|
|
3030
3987
|
// Restart recovery: re-adopt `processing` messages (ADR-0046, WI-3/4/5)
|
|
3031
3988
|
/**
|
|
3032
3989
|
* Re-adopt this agent's `processing` messages on drain (ADR-0046 Decision §1).
|
|
@@ -3193,6 +4150,7 @@ var ChannelDriver = class {
|
|
|
3193
4150
|
if (state === "failed") {
|
|
3194
4151
|
const error2 = messageError(messages, ocId ?? "") ?? void 0;
|
|
3195
4152
|
const usage = messageUsage(messages, ocId ?? "");
|
|
4153
|
+
const failure = await this.classifyModelAuthFailure(messages, ocId ?? "");
|
|
3196
4154
|
this.log({
|
|
3197
4155
|
level: "error",
|
|
3198
4156
|
message: `Re-adopt: message ${row.id.slice(0, 8)} errored while unwatched \u2014 marking failed: ${error2 ?? "(no error text)"}`,
|
|
@@ -3200,7 +4158,7 @@ var ChannelDriver = class {
|
|
|
3200
4158
|
message_id: row.id
|
|
3201
4159
|
});
|
|
3202
4160
|
try {
|
|
3203
|
-
await this.markFailed(row.conversation_id, row.id, sessionId, error2, usage);
|
|
4161
|
+
await this.markFailed(row.conversation_id, row.id, sessionId, error2, usage, failure);
|
|
3204
4162
|
} catch (err) {
|
|
3205
4163
|
if (err instanceof ChannelAuthError) throw err;
|
|
3206
4164
|
if (err instanceof ChannelTerminalError) {
|
|
@@ -3606,6 +4564,47 @@ var ChannelDriver = class {
|
|
|
3606
4564
|
}
|
|
3607
4565
|
return false;
|
|
3608
4566
|
}
|
|
4567
|
+
/**
|
|
4568
|
+
* Tri-state variant of the upward parentID membership walk (#721), used ONLY
|
|
4569
|
+
* by `isAnyDescendantSessionOngoing`. Walks the SAME cached
|
|
4570
|
+
* `resolveSessionParent` chain `sessionBelongsTo` uses above, but — unlike
|
|
4571
|
+
* `sessionBelongsTo`, which deliberately collapses "confirmed not a
|
|
4572
|
+
* descendant" and "the walk's fetch failed" into the same `false` (safe for
|
|
4573
|
+
* its OTHER callers: interaction attribution and the recovery-path
|
|
4574
|
+
* `isAnyDescendantSessionAlive`, both of which just retry next tick with no
|
|
4575
|
+
* safety consequence either way) — this variant keeps those two outcomes
|
|
4576
|
+
* SEPARATE, because `isAnyDescendantSessionOngoing`'s caller
|
|
4577
|
+
* (`isB2AbandonmentConfirmed`) must never treat "couldn't tell" as "confirmed
|
|
4578
|
+
* not ongoing".
|
|
4579
|
+
*
|
|
4580
|
+
* Return contract:
|
|
4581
|
+
* - `true` → the walk reached `rootSessionId` — `sessionId` IS a descendant.
|
|
4582
|
+
* - `false` → the walk reached a definitive, parent-less root session
|
|
4583
|
+
* WITHOUT ever matching `rootSessionId` — `sessionId` is
|
|
4584
|
+
* CONFIRMED NOT a descendant of it.
|
|
4585
|
+
* - `null` → INDETERMINATE: a `GET /session/:id` fetch failed partway
|
|
4586
|
+
* through the walk (`resolveSessionParent` returned `undefined`),
|
|
4587
|
+
* or the depth cap (32) was hit without a definitive answer (a
|
|
4588
|
+
* pathological/cyclic chain proves nothing either way). NEVER
|
|
4589
|
+
* treat this the same as `false` — see `sessionBelongsTo`'s own
|
|
4590
|
+
* doc comment above for why that collapse is safe THERE but not
|
|
4591
|
+
* here.
|
|
4592
|
+
*
|
|
4593
|
+
* `sessionBelongsTo` itself is UNCHANGED — this is an additive helper scoped
|
|
4594
|
+
* to the live-path descendant check, not a modification of shared code used
|
|
4595
|
+
* by interaction attribution or the recovery path.
|
|
4596
|
+
*/
|
|
4597
|
+
async resolveSessionMembership(sessionId, rootSessionId) {
|
|
4598
|
+
let current = sessionId;
|
|
4599
|
+
for (let depth = 0; current && depth < 32; depth++) {
|
|
4600
|
+
if (current === rootSessionId) return true;
|
|
4601
|
+
const parent = await this.resolveSessionParent(current);
|
|
4602
|
+
if (parent === void 0) return null;
|
|
4603
|
+
if (parent === null) return false;
|
|
4604
|
+
current = parent;
|
|
4605
|
+
}
|
|
4606
|
+
return null;
|
|
4607
|
+
}
|
|
3609
4608
|
/**
|
|
3610
4609
|
* Resolve (and cache) a session's `parentID` via `GET /session/:id`. Returns
|
|
3611
4610
|
* `null` for a root session (no parent) and `undefined` when opencode is
|
|
@@ -3628,19 +4627,36 @@ var ChannelDriver = class {
|
|
|
3628
4627
|
if (parent !== void 0) this.sessionParents.set(sessionId, parent);
|
|
3629
4628
|
return parent;
|
|
3630
4629
|
}
|
|
4630
|
+
/**
|
|
4631
|
+
* OpenCode's synchronous default session title (e.g.
|
|
4632
|
+
* `"New session - 1737800000000"`), assigned immediately when a session is
|
|
4633
|
+
* created — before OpenCode's async LLM-based auto-titling later renames it
|
|
4634
|
+
* mid-turn (#549). Matched by this literal, case-sensitive prefix only; the
|
|
4635
|
+
* timestamp suffix's exact format is deliberately NOT matched, since the prefix
|
|
4636
|
+
* alone is the stable, cheap signal and over-anchoring on the timestamp
|
|
4637
|
+
* representation risks silently breaking if OpenCode ever changes it. Accepted
|
|
4638
|
+
* trade-off: a genuine LLM-assigned title that happens to literally start with
|
|
4639
|
+
* this prefix would also fail to latch (see `resolveSessionTitle`) —
|
|
4640
|
+
* vanishingly unlikely in practice, and deliberately not engineered around.
|
|
4641
|
+
*/
|
|
4642
|
+
static OPENCODE_DEFAULT_TITLE_PREFIX = /^New session - /;
|
|
3631
4643
|
/**
|
|
3632
4644
|
* Resolve (and cache in `sessionTitles`) the OpenCode session TITLE (#310) so the
|
|
3633
4645
|
* status PATCH can carry it into the "Live sessions" list. Driver-level cache so
|
|
3634
4646
|
* BOTH the watcher completion path and the restart-recovery re-adopt path (which
|
|
3635
4647
|
* has no watcher) can use it. `conversationId` is passed only for log context.
|
|
3636
4648
|
* Best-effort:
|
|
3637
|
-
* - a resolved NON-EMPTY title
|
|
4649
|
+
* - a resolved NON-EMPTY title that does NOT match
|
|
4650
|
+
* `OPENCODE_DEFAULT_TITLE_PREFIX` is cached and terminal (a real session name
|
|
3638
4651
|
* won't later un-name), so we do NOT re-GET `/session/:id` every tick;
|
|
3639
|
-
* - while the title is still absent
|
|
3640
|
-
*
|
|
3641
|
-
*
|
|
3642
|
-
*
|
|
3643
|
-
*
|
|
4652
|
+
* - while the title is still absent, empty, or matches the OpenCode
|
|
4653
|
+
* placeholder prefix (#549) we do NOT latch it — OpenCode names sessions
|
|
4654
|
+
* asynchronously mid-turn, so an early call (e.g. at `processing`) must leave
|
|
4655
|
+
* the cache unresolved and re-fetch on the next need so a later call (e.g. at
|
|
4656
|
+
* `done`) picks up the name assigned in the meantime. Such a call returns
|
|
4657
|
+
* `null` (omit the title on THIS PATCH) without caching. If a session is
|
|
4658
|
+
* never renamed, the title is omitted forever rather than ever persisting
|
|
4659
|
+
* the placeholder as a last resort;
|
|
3644
4660
|
* - a failed request likewise leaves the cache unresolved (retry next need)
|
|
3645
4661
|
* and returns `null` — it must NEVER throw or block completion.
|
|
3646
4662
|
* A failure is logged with agent/session context (no silent catch).
|
|
@@ -3653,7 +4669,7 @@ var ChannelDriver = class {
|
|
|
3653
4669
|
if (res.ok) {
|
|
3654
4670
|
const body = await res.json();
|
|
3655
4671
|
const title = body && typeof body.title === "string" ? body.title.trim() : "";
|
|
3656
|
-
if (title.length > 0) {
|
|
4672
|
+
if (title.length > 0 && !_ChannelDriver.OPENCODE_DEFAULT_TITLE_PREFIX.test(title)) {
|
|
3657
4673
|
this.sessionTitles.set(sessionId, title);
|
|
3658
4674
|
return title;
|
|
3659
4675
|
}
|
|
@@ -3673,6 +4689,54 @@ var ChannelDriver = class {
|
|
|
3673
4689
|
}
|
|
3674
4690
|
return null;
|
|
3675
4691
|
}
|
|
4692
|
+
/**
|
|
4693
|
+
* Best-effort mid-turn title sync (#711 follow-up): PATCH a resolved OpenCode
|
|
4694
|
+
* session title onto the conversation via the PLAIN conversation-update
|
|
4695
|
+
* endpoint (`PATCH /runners/:agentId/conversations/:conversationId`) — NOT the
|
|
4696
|
+
* message-status endpoint `markProcessing`/`markDone` use. Deliberately a
|
|
4697
|
+
* separate, lighter call: it carries no `status`, so it cannot re-trigger the
|
|
4698
|
+
* `processing`/`done` transition side effects (Slack notices, activity-log
|
|
4699
|
+
* rows, delivery jobs) those PATCHes gate on `transitioned` — this call only
|
|
4700
|
+
* ever touches `conversations.title`. That route (`routes/conversations.ts`)
|
|
4701
|
+
* skips a title write matching the stored value, so a redundant call with the
|
|
4702
|
+
* same title is a real no-op — it does not bump `updated_at`, which the
|
|
4703
|
+
* conversation list sorts and paginates on. (Note this is a DIFFERENT guard
|
|
4704
|
+
* from `threads.ts`'s "non-empty AND changed" one, which only covers the
|
|
4705
|
+
* message-status PATCH; the non-empty half is enforced here instead, by
|
|
4706
|
+
* `resolveSessionTitle` never returning an empty/placeholder title.)
|
|
4707
|
+
*
|
|
4708
|
+
* Telemetry-only / never blocks the caller, mirroring `postSignal`: a failure
|
|
4709
|
+
* is logged and the title is simply retried on the next heartbeat tick (the
|
|
4710
|
+
* caller only latches `titleSynced` on `true`).
|
|
4711
|
+
*/
|
|
4712
|
+
async patchConversationTitle(conversationId, title) {
|
|
4713
|
+
try {
|
|
4714
|
+
const res = await this.fetchImpl(
|
|
4715
|
+
`${this.apiUrl}/runners/${this.agentId}/conversations/${conversationId}`,
|
|
4716
|
+
{
|
|
4717
|
+
method: "PATCH",
|
|
4718
|
+
headers: { Authorization: this.getAuthHeader(), "Content-Type": "application/json" },
|
|
4719
|
+
body: JSON.stringify({ title })
|
|
4720
|
+
}
|
|
4721
|
+
);
|
|
4722
|
+
if (!res.ok) {
|
|
4723
|
+
this.log({
|
|
4724
|
+
level: "debug",
|
|
4725
|
+
message: `Mid-turn title sync PATCH for conversation ${conversationId.slice(0, 8)} returned HTTP ${res.status} (best-effort, will retry next heartbeat)`,
|
|
4726
|
+
conversation_id: conversationId
|
|
4727
|
+
});
|
|
4728
|
+
return false;
|
|
4729
|
+
}
|
|
4730
|
+
return true;
|
|
4731
|
+
} catch (err) {
|
|
4732
|
+
this.log({
|
|
4733
|
+
level: "debug",
|
|
4734
|
+
message: `Best-effort mid-turn title sync PATCH failed for conversation ${conversationId.slice(0, 8)} (will retry next heartbeat): ${err instanceof Error ? err.message : String(err)}`,
|
|
4735
|
+
conversation_id: conversationId
|
|
4736
|
+
});
|
|
4737
|
+
return false;
|
|
4738
|
+
}
|
|
4739
|
+
}
|
|
3676
4740
|
/**
|
|
3677
4741
|
* DEFENSIVE cross-check for the restart-recovery path (WI-2): is any descendant
|
|
3678
4742
|
* (`task` sub-agent) session under `rootSessionId` still genuinely doing work?
|
|
@@ -3731,6 +4795,84 @@ var ChannelDriver = class {
|
|
|
3731
4795
|
}
|
|
3732
4796
|
return false;
|
|
3733
4797
|
}
|
|
4798
|
+
/**
|
|
4799
|
+
* LIVE-PATH descendant-liveness check (#721): is any descendant (`task`
|
|
4800
|
+
* sub-agent) session under `rootSessionId` currently ONGOING per OpenCode's own
|
|
4801
|
+
* in-memory status map (`isSessionOngoing` — `busy`/`retry`)?
|
|
4802
|
+
*
|
|
4803
|
+
* Deliberately NOT `isAnyDescendantSessionAlive` (the RECOVERY-path
|
|
4804
|
+
* cross-check above): that method judges liveness from the child's OWN
|
|
4805
|
+
* TRANSCRIPT (`isSessionActivelyGenerating`), which is the right (only) option
|
|
4806
|
+
* on the recovery path because a restart WIPES `SessionStatus`. On the LIVE
|
|
4807
|
+
* path the local opencode server IS running, so its in-memory status map is
|
|
4808
|
+
* live and authoritative — and per ADR-0047 §4a ("the child has its own entry
|
|
4809
|
+
* [in the map]"), a `task` descendant's OWN busy/retry entry reflects its
|
|
4810
|
+
* ENTIRE turn (including any tool call it is itself executing), not a
|
|
4811
|
+
* per-message transcript snapshot. This sidesteps the "child's own tool is
|
|
4812
|
+
* executing, between its step's completion and the next generation step"
|
|
4813
|
+
* transcript gap that a transcript-based check would need a second,
|
|
4814
|
+
* sustained-window bound to guard against — it is simply not derived from
|
|
4815
|
+
* message timestamps at all.
|
|
4816
|
+
*
|
|
4817
|
+
* Why not just check `isSessionOngoing(port, rootSessionId)` (the ROOT's own
|
|
4818
|
+
* status, as the recovery path does per §4a)? Because on the LIVE path the
|
|
4819
|
+
* root session can be shared: a SECOND, unrelated user message can land on the
|
|
4820
|
+
* SAME session (issue #721's own root cause) and keep the root `busy` for a
|
|
4821
|
+
* reason that has nothing to do with THIS message's delegation. A `task`
|
|
4822
|
+
* descendant session is spawned for exactly one delegated turn and never
|
|
4823
|
+
* reused, so its OWN status-map entry is unambiguous evidence about that one
|
|
4824
|
+
* delegation — which the root's status is not.
|
|
4825
|
+
*
|
|
4826
|
+
* Why membership is checked via `resolveSessionMembership`, NOT
|
|
4827
|
+
* `sessionBelongsTo`: `sessionBelongsTo` collapses a transient
|
|
4828
|
+
* `GET /session/:id` fetch failure into "not a descendant", which would
|
|
4829
|
+
* silently drop a genuinely-live candidate from consideration on the one
|
|
4830
|
+
* unlucky tick its membership-walk fetch hiccups (#721).
|
|
4831
|
+
* `resolveSessionMembership` keeps that failure mode as a distinct `null`
|
|
4832
|
+
* (indeterminate) so it is folded into THIS method's own `indeterminate` flag
|
|
4833
|
+
* instead.
|
|
4834
|
+
*
|
|
4835
|
+
* Return contract (note the DIFFERENT judge vs. `isAnyDescendantSessionAlive`):
|
|
4836
|
+
* - `true` → some descendant session is `busy`/`retry` (genuinely ongoing).
|
|
4837
|
+
* - `false` → enumeration succeeded, EVERY candidate's MEMBERSHIP was
|
|
4838
|
+
* confirmed either way (`resolveSessionMembership` never
|
|
4839
|
+
* returned `null`), and every CONFIRMED descendant's status read
|
|
4840
|
+
* succeeded and is not ongoing (includes "no descendant session
|
|
4841
|
+
* exists at all" — e.g. a plain, non-`task` tool call).
|
|
4842
|
+
* - `null` → INDETERMINATE: `listSessions` failed, OR at least one
|
|
4843
|
+
* candidate's MEMBERSHIP could not be confirmed
|
|
4844
|
+
* (`resolveSessionMembership` returned `null` — a fetch failure
|
|
4845
|
+
* or pathological chain partway through the parent walk), OR at
|
|
4846
|
+
* least one CONFIRMED descendant's `isSessionOngoing` read
|
|
4847
|
+
* failed — and no OTHER candidate was already confirmed `true`.
|
|
4848
|
+
* The caller MUST NOT treat `null` the same as `false` here
|
|
4849
|
+
* (unlike the recovery cross-check's contract) — see
|
|
4850
|
+
* `isB2AbandonmentConfirmed`.
|
|
4851
|
+
*/
|
|
4852
|
+
async isAnyDescendantSessionOngoing(rootSessionId) {
|
|
4853
|
+
const sessions = await listSessions(this.port);
|
|
4854
|
+
if (!sessions) {
|
|
4855
|
+
this.log({
|
|
4856
|
+
level: "warn",
|
|
4857
|
+
message: `Could not enumerate sessions to check descendant liveness for root ${rootSessionId} (listSessions failed) \u2014 treating descendant liveness as indeterminate`
|
|
4858
|
+
});
|
|
4859
|
+
return null;
|
|
4860
|
+
}
|
|
4861
|
+
let indeterminate = false;
|
|
4862
|
+
for (const candidate of sessions) {
|
|
4863
|
+
if (!candidate?.id || candidate.id === rootSessionId) continue;
|
|
4864
|
+
const membership = await this.resolveSessionMembership(candidate.id, rootSessionId);
|
|
4865
|
+
if (membership === null) {
|
|
4866
|
+
indeterminate = true;
|
|
4867
|
+
continue;
|
|
4868
|
+
}
|
|
4869
|
+
if (membership === false) continue;
|
|
4870
|
+
const ongoing = await isSessionOngoing(this.port, candidate.id);
|
|
4871
|
+
if (ongoing === true) return true;
|
|
4872
|
+
if (ongoing === null) indeterminate = true;
|
|
4873
|
+
}
|
|
4874
|
+
return indeterminate ? null : false;
|
|
4875
|
+
}
|
|
3734
4876
|
/**
|
|
3735
4877
|
* Cheap decision-telemetry label for a running row's LAST correlated reply
|
|
3736
4878
|
* (WI-2 Task 2.2): which running SHAPE it is, for the status-gated recovery log.
|
|
@@ -3859,6 +5001,32 @@ var ChannelDriver = class {
|
|
|
3859
5001
|
}
|
|
3860
5002
|
return messages;
|
|
3861
5003
|
}
|
|
5004
|
+
/**
|
|
5005
|
+
* The `opencode_session_id` fragment of a status PATCH body — `{}` when this
|
|
5006
|
+
* conversation has ABANDONED that session (#553). The field is optional
|
|
5007
|
+
* server-side and an absent one leaves the persisted binding untouched, so
|
|
5008
|
+
* omitting it is how a routine status write stops resurrecting it.
|
|
5009
|
+
*
|
|
5010
|
+
* ONLY for writes whose sole cost is a lost deep link. The `processing` notice
|
|
5011
|
+
* degrades to no "View in Evident" link (the reaction swap still fires) and the
|
|
5012
|
+
* turn-failure notice is built from the PATCH's own `error` text with a link off
|
|
5013
|
+
* the persisted row — neither loses content the user came for. `markDone`
|
|
5014
|
+
* deliberately does NOT use this helper: the server fetches the reply text
|
|
5015
|
+
* THROUGH the session id it is given, so suppressing there would replace the
|
|
5016
|
+
* agent's answer with a bare "✅ Done!" (the #183/#187 failure). The
|
|
5017
|
+
* `ensureSession` guard, not this suppression, is what makes the self-heal
|
|
5018
|
+
* stick.
|
|
5019
|
+
*/
|
|
5020
|
+
sessionIdBody(sessionId, conversationId, messageId, status) {
|
|
5021
|
+
if (!this.isSuperseded(conversationId, sessionId)) return { opencode_session_id: sessionId };
|
|
5022
|
+
this.log({
|
|
5023
|
+
level: "debug",
|
|
5024
|
+
message: `Omitting the abandoned OpenCode session ${sessionId.slice(0, 8)} from the '${status}' update for message ${messageId.slice(0, 8)} so it is not re-bound to conversation ${conversationId.slice(0, 8)}`,
|
|
5025
|
+
conversation_id: conversationId,
|
|
5026
|
+
message_id: messageId
|
|
5027
|
+
});
|
|
5028
|
+
return {};
|
|
5029
|
+
}
|
|
3862
5030
|
/**
|
|
3863
5031
|
* EXISTING combinedAuth route — now fired by the watcher on queued→running
|
|
3864
5032
|
* (Task 3.3), NOT at dispatch/claim time. `{status:'processing',
|
|
@@ -3888,7 +5056,7 @@ var ChannelDriver = class {
|
|
|
3888
5056
|
headers: { Authorization: this.getAuthHeader(), "Content-Type": "application/json" },
|
|
3889
5057
|
body: JSON.stringify({
|
|
3890
5058
|
status: "processing",
|
|
3891
|
-
|
|
5059
|
+
...this.sessionIdBody(sessionId, conversationId, messageId, "processing"),
|
|
3892
5060
|
...opencodeMessageId ? { opencode_message_id: opencodeMessageId } : {},
|
|
3893
5061
|
...title ? { title } : {}
|
|
3894
5062
|
})
|
|
@@ -3937,6 +5105,11 @@ var ChannelDriver = class {
|
|
|
3937
5105
|
headers: { Authorization: this.getAuthHeader(), "Content-Type": "application/json" },
|
|
3938
5106
|
body: JSON.stringify({
|
|
3939
5107
|
status: "done",
|
|
5108
|
+
// ALWAYS sent, even for a session this conversation has abandoned
|
|
5109
|
+
// (#553): the server reads the reply text back out of THIS session id
|
|
5110
|
+
// to deliver it. Omitting it would leave the user with "✅ Done!"
|
|
5111
|
+
// instead of the answer — a worse regression than the resurrection it
|
|
5112
|
+
// would prevent, which `ensureSession`'s guard handles anyway.
|
|
3940
5113
|
opencode_session_id: sessionId,
|
|
3941
5114
|
...opencodeMessageId ? { opencode_message_id: opencodeMessageId } : {},
|
|
3942
5115
|
...title ? { title } : {},
|
|
@@ -3953,16 +5126,31 @@ var ChannelDriver = class {
|
|
|
3953
5126
|
}
|
|
3954
5127
|
/**
|
|
3955
5128
|
* Mark a message `failed`. `sessionId` / `error` are threaded to the API ONLY
|
|
3956
|
-
* when provided (issue #182)
|
|
3957
|
-
* `
|
|
3958
|
-
*
|
|
3959
|
-
*
|
|
5129
|
+
* when provided (issue #182). Three states for `sessionId`:
|
|
5130
|
+
* - omitted (`undefined`) → don't send the field, leave the persisted
|
|
5131
|
+
* session untouched (unused today; kept for API symmetry).
|
|
5132
|
+
* - a real id (`string`) → send it, update the persisted session (the
|
|
5133
|
+
* turn-failure call sites: an errored OpenCode turn).
|
|
5134
|
+
* - explicit `null` → send it, CLEAR the persisted session (issue
|
|
5135
|
+
* #485's dispatch-handoff-failure call site: the session id still
|
|
5136
|
+
* exists but is wedged, so the next attempt must get a fresh one
|
|
5137
|
+
* instead of reusing it — see WI-1's server-side null-clearing PATCH).
|
|
3960
5138
|
*/
|
|
3961
|
-
async markFailed(conversationId, messageId, sessionId, error2, usage) {
|
|
5139
|
+
async markFailed(conversationId, messageId, sessionId, error2, usage, failure) {
|
|
3962
5140
|
const body = { status: "failed" };
|
|
3963
|
-
if (sessionId
|
|
5141
|
+
if (sessionId === null) {
|
|
5142
|
+
body.opencode_session_id = null;
|
|
5143
|
+
} else if (sessionId !== void 0) {
|
|
5144
|
+
Object.assign(body, this.sessionIdBody(sessionId, conversationId, messageId, "failed"));
|
|
5145
|
+
}
|
|
3964
5146
|
if (error2 !== void 0) body.error = error2;
|
|
3965
5147
|
if (usage) Object.assign(body, usage);
|
|
5148
|
+
if (failure) {
|
|
5149
|
+
body.failure_kind = failure.kind;
|
|
5150
|
+
body.failure_provider_id = failure.providerId;
|
|
5151
|
+
body.failure_model_id = failure.modelId;
|
|
5152
|
+
body.failure_reason = failure.reason;
|
|
5153
|
+
}
|
|
3966
5154
|
await this.callWithRetry(
|
|
3967
5155
|
"marking message as failed",
|
|
3968
5156
|
() => this.fetchImpl(
|
|
@@ -3975,6 +5163,29 @@ var ChannelDriver = class {
|
|
|
3975
5163
|
)
|
|
3976
5164
|
);
|
|
3977
5165
|
}
|
|
5166
|
+
/**
|
|
5167
|
+
* Classify an errored turn as a model-auth failure (#736 P1-4), for `markFailed`.
|
|
5168
|
+
*
|
|
5169
|
+
* `messageFailure` alone (structured OpenCode error → `model_auth`) covers
|
|
5170
|
+
* most cases; when it returns `null` on this ALREADY-FAILED turn, fall back
|
|
5171
|
+
* to the P1-2b zero-provider check — one extra loopback call to
|
|
5172
|
+
* `hasAnyConfiguredProvider`, only reached when the structured classifier
|
|
5173
|
+
* couldn't place it. Fails open (never throws): a fallback probe failure
|
|
5174
|
+
* (`null`/indeterminate) leaves the classification `null`, which produces
|
|
5175
|
+
* today's byte-identical PATCH body via `markFailed`'s `if (failure)` guard.
|
|
5176
|
+
*/
|
|
5177
|
+
async classifyModelAuthFailure(messages, userMessageId) {
|
|
5178
|
+
const classified = messageFailure(messages, userMessageId);
|
|
5179
|
+
if (classified != null) return classified;
|
|
5180
|
+
const reply = findLastAssistantReplyFor(messages, userMessageId);
|
|
5181
|
+
const hasProvider = await hasAnyConfiguredProvider(this.port);
|
|
5182
|
+
return applyZeroProviderFallback(
|
|
5183
|
+
classified,
|
|
5184
|
+
hasProvider,
|
|
5185
|
+
reply?.info?.providerID ?? null,
|
|
5186
|
+
reply?.info?.modelID ?? null
|
|
5187
|
+
);
|
|
5188
|
+
}
|
|
3978
5189
|
/**
|
|
3979
5190
|
* Best-effort, SINGLE-ATTEMPT runner→server telemetry ping
|
|
3980
5191
|
* (queued-followup-redrive). `POST .../messages/:id/signal {signal, ...extra}`
|
|
@@ -4127,10 +5338,16 @@ var ChannelDriver = class {
|
|
|
4127
5338
|
import chalk5 from "chalk";
|
|
4128
5339
|
import ora2 from "ora";
|
|
4129
5340
|
import { select as select2 } from "@inquirer/prompts";
|
|
5341
|
+
var INTERACTIVE_START_TIMEOUT_MS = 3e4;
|
|
4130
5342
|
async function ensureOpenCodeRunning(ctx) {
|
|
4131
5343
|
const healthCheck = await checkOpenCodeHealth(ctx.port);
|
|
4132
5344
|
if (healthCheck.healthy) {
|
|
4133
|
-
return {
|
|
5345
|
+
return {
|
|
5346
|
+
port: ctx.port,
|
|
5347
|
+
process: null,
|
|
5348
|
+
version: healthCheck.version ?? null,
|
|
5349
|
+
notReadyReason: null
|
|
5350
|
+
};
|
|
4134
5351
|
}
|
|
4135
5352
|
const runningInstances = await findHealthyOpenCodeInstances();
|
|
4136
5353
|
if (runningInstances.length > 0) {
|
|
@@ -4151,7 +5368,7 @@ async function ensureOpenCodeRunning(ctx) {
|
|
|
4151
5368
|
console.log(chalk5.yellow("Tip: Run with the correct port:"));
|
|
4152
5369
|
console.log(
|
|
4153
5370
|
chalk5.dim(
|
|
4154
|
-
` ${getCliName()} run --
|
|
5371
|
+
` ${getCliName()} run --runner ${ctx.agentId} --port ${runningInstances[0].port}`
|
|
4155
5372
|
)
|
|
4156
5373
|
);
|
|
4157
5374
|
}
|
|
@@ -4171,14 +5388,22 @@ async function ensureOpenCodeRunning(ctx) {
|
|
|
4171
5388
|
if (!ctx.interactive) {
|
|
4172
5389
|
ctx.log(`OpenCode is not running on port ${ctx.port}. Starting it automatically...`);
|
|
4173
5390
|
const proc = await startOpenCode(ctx.port);
|
|
4174
|
-
const health = await waitForOpenCodeHealth(ctx.port,
|
|
5391
|
+
const health = await waitForOpenCodeHealth(ctx.port, ctx.startTimeoutMs);
|
|
4175
5392
|
if (!health.healthy) {
|
|
4176
|
-
|
|
4177
|
-
|
|
4178
|
-
|
|
5393
|
+
return {
|
|
5394
|
+
port: ctx.port,
|
|
5395
|
+
process: proc,
|
|
5396
|
+
version: null,
|
|
5397
|
+
notReadyReason: `it did not become healthy within ${Math.round(ctx.startTimeoutMs / 1e3)}s`
|
|
5398
|
+
};
|
|
4179
5399
|
}
|
|
4180
5400
|
ctx.log(`OpenCode started on port ${ctx.port}${health.version ? ` (v${health.version})` : ""}`);
|
|
4181
|
-
return {
|
|
5401
|
+
return {
|
|
5402
|
+
port: ctx.port,
|
|
5403
|
+
process: proc,
|
|
5404
|
+
version: health.version ?? null,
|
|
5405
|
+
notReadyReason: null
|
|
5406
|
+
};
|
|
4182
5407
|
}
|
|
4183
5408
|
let port = ctx.port;
|
|
4184
5409
|
if (isPortInUse(port)) {
|
|
@@ -4231,15 +5456,15 @@ Port ${port} is already in use.`));
|
|
|
4231
5456
|
if (action === "start") {
|
|
4232
5457
|
const spinner = ora2("Starting OpenCode...").start();
|
|
4233
5458
|
const proc = await startOpenCode(port);
|
|
4234
|
-
const health = await waitForOpenCodeHealth(port,
|
|
5459
|
+
const health = await waitForOpenCodeHealth(port, INTERACTIVE_START_TIMEOUT_MS);
|
|
4235
5460
|
if (!health.healthy) {
|
|
4236
5461
|
spinner.fail("Failed to start OpenCode");
|
|
4237
5462
|
throw new Error("OpenCode failed to start");
|
|
4238
5463
|
}
|
|
4239
5464
|
spinner.stop();
|
|
4240
|
-
return { port, process: proc, version: health.version ?? null };
|
|
5465
|
+
return { port, process: proc, version: health.version ?? null, notReadyReason: null };
|
|
4241
5466
|
}
|
|
4242
|
-
return { port, process: null, version: null };
|
|
5467
|
+
return { port, process: null, version: null, notReadyReason: "you chose to continue without it" };
|
|
4243
5468
|
}
|
|
4244
5469
|
|
|
4245
5470
|
// src/commands/agent-lookup.ts
|
|
@@ -4281,19 +5506,21 @@ async function resolveAgentIdFromKey(authHeader) {
|
|
|
4281
5506
|
return { agent_id: data.agent_id };
|
|
4282
5507
|
}
|
|
4283
5508
|
return {
|
|
4284
|
-
error: "Cannot resolve runner ID: auth type is not agent_key. Please provide --
|
|
5509
|
+
error: "Cannot resolve runner ID: auth type is not agent_key. Please provide --runner explicitly."
|
|
4285
5510
|
};
|
|
4286
5511
|
} catch (error2) {
|
|
4287
5512
|
const message = error2 instanceof Error ? error2.message : "Unknown error";
|
|
4288
5513
|
return { error: `Failed to resolve runner from key: ${message}` };
|
|
4289
5514
|
}
|
|
4290
5515
|
}
|
|
5516
|
+
var BEST_EFFORT_NOTIFY_TIMEOUT_MS = 2e3;
|
|
4291
5517
|
async function notifyAgentDisconnected(agentId, authHeader) {
|
|
4292
5518
|
const apiUrl = getApiUrlConfig();
|
|
4293
5519
|
try {
|
|
4294
5520
|
const response = await fetch(`${apiUrl}/runners/${agentId}/disconnect`, {
|
|
4295
5521
|
method: "POST",
|
|
4296
|
-
headers: { Authorization: authHeader }
|
|
5522
|
+
headers: { Authorization: authHeader },
|
|
5523
|
+
signal: AbortSignal.timeout(BEST_EFFORT_NOTIFY_TIMEOUT_MS)
|
|
4297
5524
|
});
|
|
4298
5525
|
if (!response.ok) {
|
|
4299
5526
|
const serverMessage = await readErrorMessage(response);
|
|
@@ -4304,7 +5531,35 @@ async function notifyAgentDisconnected(agentId, authHeader) {
|
|
|
4304
5531
|
}
|
|
4305
5532
|
return { ok: true };
|
|
4306
5533
|
} catch (error2) {
|
|
4307
|
-
return { ok: false, error:
|
|
5534
|
+
return { ok: false, error: describeBestEffortError(error2) };
|
|
5535
|
+
}
|
|
5536
|
+
}
|
|
5537
|
+
function describeBestEffortError(error2) {
|
|
5538
|
+
const name = error2?.name;
|
|
5539
|
+
if (name === "TimeoutError" || name === "AbortError") {
|
|
5540
|
+
return `timed out after ${BEST_EFFORT_NOTIFY_TIMEOUT_MS}ms`;
|
|
5541
|
+
}
|
|
5542
|
+
return error2 instanceof Error ? error2.message : String(error2);
|
|
5543
|
+
}
|
|
5544
|
+
async function reportMicrovmId(agentId, authHeader, microvmId) {
|
|
5545
|
+
try {
|
|
5546
|
+
const apiUrl = getApiUrlConfig();
|
|
5547
|
+
const response = await fetch(`${apiUrl}/runners/${agentId}/microvm`, {
|
|
5548
|
+
method: "POST",
|
|
5549
|
+
headers: { Authorization: authHeader, "Content-Type": "application/json" },
|
|
5550
|
+
body: JSON.stringify({ microvm_id: microvmId }),
|
|
5551
|
+
signal: AbortSignal.timeout(BEST_EFFORT_NOTIFY_TIMEOUT_MS)
|
|
5552
|
+
});
|
|
5553
|
+
if (!response.ok) {
|
|
5554
|
+
const serverMessage = await readErrorMessage(response);
|
|
5555
|
+
return {
|
|
5556
|
+
ok: false,
|
|
5557
|
+
error: `HTTP ${response.status}${serverMessage ? `: ${serverMessage}` : ""}`
|
|
5558
|
+
};
|
|
5559
|
+
}
|
|
5560
|
+
return { ok: true };
|
|
5561
|
+
} catch (error2) {
|
|
5562
|
+
return { ok: false, error: describeBestEffortError(error2) };
|
|
4308
5563
|
}
|
|
4309
5564
|
}
|
|
4310
5565
|
async function getAgentInfo(agentId, authHeader) {
|
|
@@ -4354,6 +5609,7 @@ var MAX_ACTIVITY_LOG_ENTRIES = 10;
|
|
|
4354
5609
|
var CHANNEL_POLL_INTERVAL_MS = Number(process.env.EVIDENT_CHANNEL_POLL_INTERVAL_MS) || 2e3;
|
|
4355
5610
|
var CHANNEL_STUCK_QUEUED_MS = Number(process.env.EVIDENT_STUCK_QUEUED_MS) || void 0;
|
|
4356
5611
|
var SHUTDOWN_DRAIN_TIMEOUT_MS = Number(process.env.EVIDENT_SHUTDOWN_DRAIN_MS) || 25e3;
|
|
5612
|
+
var TELEMETRY_SHUTDOWN_TIMEOUT_MS = 5e3;
|
|
4357
5613
|
function resolveLogLevel(options) {
|
|
4358
5614
|
const accepted = Object.keys(LOG_LEVELS);
|
|
4359
5615
|
const validate = (value, source) => {
|
|
@@ -4377,6 +5633,63 @@ function resolveLogLevel(options) {
|
|
|
4377
5633
|
}
|
|
4378
5634
|
return "info";
|
|
4379
5635
|
}
|
|
5636
|
+
function resolveFileSyncDirectories(raw, homeDir) {
|
|
5637
|
+
const directories = [];
|
|
5638
|
+
for (const entry of raw ?? []) {
|
|
5639
|
+
const trimmed = entry.trim();
|
|
5640
|
+
if (trimmed === "") {
|
|
5641
|
+
throw new Error("--enable-file-sync-to requires a directory path (got an empty value)");
|
|
5642
|
+
}
|
|
5643
|
+
const expanded = trimmed === "~" ? homeDir : trimmed.startsWith("~/") ? join2(homeDir, trimmed.slice(2)) : trimmed;
|
|
5644
|
+
if (!isAbsolute2(expanded)) {
|
|
5645
|
+
throw new Error(`--enable-file-sync-to requires an absolute directory path; got "${entry}"`);
|
|
5646
|
+
}
|
|
5647
|
+
const normalized = resolvePath(expanded);
|
|
5648
|
+
if (parse(normalized).root === normalized) {
|
|
5649
|
+
throw new Error(
|
|
5650
|
+
`--enable-file-sync-to will not allow-list the filesystem root ("${entry}"); name the specific directory the credentials belong in (for example ~/.claude)`
|
|
5651
|
+
);
|
|
5652
|
+
}
|
|
5653
|
+
if (!directories.includes(normalized)) {
|
|
5654
|
+
directories.push(normalized);
|
|
5655
|
+
}
|
|
5656
|
+
}
|
|
5657
|
+
if (directories.length > MAX_FILE_SYNC_DIRECTORIES) {
|
|
5658
|
+
throw new Error(
|
|
5659
|
+
`--enable-file-sync-to accepts at most ${MAX_FILE_SYNC_DIRECTORIES} directories; got ${directories.length}`
|
|
5660
|
+
);
|
|
5661
|
+
}
|
|
5662
|
+
return directories;
|
|
5663
|
+
}
|
|
5664
|
+
var DEFAULT_OPENCODE_START_TIMEOUT_SECONDS = 180;
|
|
5665
|
+
var MAX_OPENCODE_START_TIMEOUT_SECONDS = 3600;
|
|
5666
|
+
var OPENCODE_START_TIMEOUT_ENV = "EVIDENT_OPENCODE_START_TIMEOUT";
|
|
5667
|
+
function resolveOpenCodeStartTimeoutMs(options, env = process.env) {
|
|
5668
|
+
const defaultMs = DEFAULT_OPENCODE_START_TIMEOUT_SECONDS * 1e3;
|
|
5669
|
+
let raw;
|
|
5670
|
+
let source;
|
|
5671
|
+
if (options.opencodeStartTimeout !== void 0) {
|
|
5672
|
+
raw = options.opencodeStartTimeout;
|
|
5673
|
+
source = "--opencode-start-timeout";
|
|
5674
|
+
} else if (env[OPENCODE_START_TIMEOUT_ENV] !== void 0 && env[OPENCODE_START_TIMEOUT_ENV] !== "") {
|
|
5675
|
+
raw = env[OPENCODE_START_TIMEOUT_ENV];
|
|
5676
|
+
source = OPENCODE_START_TIMEOUT_ENV;
|
|
5677
|
+
} else {
|
|
5678
|
+
return { timeoutMs: defaultMs, warnings: [] };
|
|
5679
|
+
}
|
|
5680
|
+
const trimmed = raw.trim();
|
|
5681
|
+
const seconds = Number(trimmed);
|
|
5682
|
+
const isPositiveInteger = /^\d+$/.test(trimmed) && Number.isInteger(seconds) && seconds > 0;
|
|
5683
|
+
if (!isPositiveInteger || seconds > MAX_OPENCODE_START_TIMEOUT_SECONDS) {
|
|
5684
|
+
return {
|
|
5685
|
+
timeoutMs: defaultMs,
|
|
5686
|
+
warnings: [
|
|
5687
|
+
`Ignoring invalid ${source} "${raw}": expected a positive integer number of seconds (at most ${MAX_OPENCODE_START_TIMEOUT_SECONDS}); using the default ${DEFAULT_OPENCODE_START_TIMEOUT_SECONDS}s`
|
|
5688
|
+
]
|
|
5689
|
+
};
|
|
5690
|
+
}
|
|
5691
|
+
return { timeoutMs: seconds * 1e3, warnings: [] };
|
|
5692
|
+
}
|
|
4380
5693
|
function meetsThreshold(state, level) {
|
|
4381
5694
|
return LOG_LEVELS[level] >= LOG_LEVELS[state.logLevel];
|
|
4382
5695
|
}
|
|
@@ -4398,6 +5711,10 @@ function log2(state, message, level = "info") {
|
|
|
4398
5711
|
function logActivity(state, entry) {
|
|
4399
5712
|
const level = entry.level ?? (entry.type === "error" ? "error" : "info");
|
|
4400
5713
|
if (!meetsThreshold(state, level)) return;
|
|
5714
|
+
forwardRunnerActivity(
|
|
5715
|
+
{ level, message: entry.message, error: entry.error },
|
|
5716
|
+
{ agentId: state.agentId, authHeader: state.authHeader }
|
|
5717
|
+
);
|
|
4401
5718
|
const fullEntry = {
|
|
4402
5719
|
...entry,
|
|
4403
5720
|
level,
|
|
@@ -4498,18 +5815,29 @@ async function handleAuthError(state, error2) {
|
|
|
4498
5815
|
async function driveChannels(state, driver) {
|
|
4499
5816
|
let idlePolls = 0;
|
|
4500
5817
|
let lastSeenProxiedActivityAt = state.lastProxiedActivityAt;
|
|
5818
|
+
let lastSeenAppliedFiles = driver.fileSyncActivity().appliedFiles;
|
|
4501
5819
|
while (state.running) {
|
|
4502
5820
|
if (state.connection?.reconnecting && state.connection.reconnectPromise) {
|
|
4503
5821
|
logActivity(state, { type: "info", message: "Waiting for tunnel reconnection..." });
|
|
4504
5822
|
if (state.interactive) displayStatus(state);
|
|
4505
5823
|
await state.connection.reconnectPromise;
|
|
4506
5824
|
}
|
|
5825
|
+
const carriedOverFileSync = driver.fileSyncActivity().inFlight;
|
|
5826
|
+
void driver.syncPendingFiles().catch(
|
|
5827
|
+
(error2) => logActivity(state, {
|
|
5828
|
+
type: "error",
|
|
5829
|
+
error: `Runner file sync failed: ${error2 instanceof Error ? error2.message : String(error2)}`
|
|
5830
|
+
})
|
|
5831
|
+
);
|
|
4507
5832
|
try {
|
|
4508
5833
|
const processed = await driver.drainPending();
|
|
4509
5834
|
state.messageCount += processed;
|
|
4510
5835
|
const proxiedActivity = state.lastProxiedActivityAt !== lastSeenProxiedActivityAt;
|
|
4511
5836
|
lastSeenProxiedActivityAt = state.lastProxiedActivityAt;
|
|
4512
|
-
|
|
5837
|
+
const appliedFiles = driver.fileSyncActivity().appliedFiles;
|
|
5838
|
+
const fileActivity = carriedOverFileSync || appliedFiles !== lastSeenAppliedFiles;
|
|
5839
|
+
lastSeenAppliedFiles = appliedFiles;
|
|
5840
|
+
if (processed > 0 || driver.hasInFlightWatchers() || proxiedActivity || fileActivity) {
|
|
4513
5841
|
idlePolls = 0;
|
|
4514
5842
|
if (processed > 0 && state.interactive) displayStatus(state);
|
|
4515
5843
|
} else if (state.idleTimeout !== null) {
|
|
@@ -4538,7 +5866,7 @@ async function driveChannels(state, driver) {
|
|
|
4538
5866
|
logActivity(state, { type: "error", error: `Channel processing error: ${errorMessage}` });
|
|
4539
5867
|
if (state.interactive) displayStatus(state);
|
|
4540
5868
|
}
|
|
4541
|
-
await new Promise((
|
|
5869
|
+
await new Promise((resolve3) => setTimeout(resolve3, CHANNEL_POLL_INTERVAL_MS));
|
|
4542
5870
|
if (state.idleTimeout !== null && idlePolls >= 2) {
|
|
4543
5871
|
const idleMs = idlePolls * CHANNEL_POLL_INTERVAL_MS;
|
|
4544
5872
|
if (idleMs > state.idleTimeout * 1e3) {
|
|
@@ -4641,7 +5969,18 @@ async function notifyOffline(state) {
|
|
|
4641
5969
|
if (state.interactive) displayStatus(state);
|
|
4642
5970
|
}
|
|
4643
5971
|
}
|
|
5972
|
+
async function timeShutdownPhase(state, durations, name, run2) {
|
|
5973
|
+
const startedAt = Date.now();
|
|
5974
|
+
try {
|
|
5975
|
+
return await run2();
|
|
5976
|
+
} finally {
|
|
5977
|
+
const elapsedMs = Date.now() - startedAt;
|
|
5978
|
+
durations[name] = elapsedMs;
|
|
5979
|
+
log2(state, `Shutdown phase ${name}: ${elapsedMs}ms`);
|
|
5980
|
+
}
|
|
5981
|
+
}
|
|
4644
5982
|
async function cleanup(state, opts = {}) {
|
|
5983
|
+
const durations = {};
|
|
4645
5984
|
state.running = false;
|
|
4646
5985
|
for (const timer of state.sessionCleanupTimers) {
|
|
4647
5986
|
clearInterval(timer);
|
|
@@ -4655,7 +5994,13 @@ async function cleanup(state, opts = {}) {
|
|
|
4655
5994
|
logActivity(state, { type: "info", message: "Draining in-flight work before shutdown..." });
|
|
4656
5995
|
displayStatus(state);
|
|
4657
5996
|
}
|
|
4658
|
-
const
|
|
5997
|
+
const driver = state.channelDriver;
|
|
5998
|
+
const settled = await timeShutdownPhase(
|
|
5999
|
+
state,
|
|
6000
|
+
durations,
|
|
6001
|
+
"drain",
|
|
6002
|
+
() => driver.waitForInFlight(SHUTDOWN_DRAIN_TIMEOUT_MS)
|
|
6003
|
+
);
|
|
4659
6004
|
if (!settled) {
|
|
4660
6005
|
logActivity(state, {
|
|
4661
6006
|
type: "info",
|
|
@@ -4664,13 +6009,15 @@ async function cleanup(state, opts = {}) {
|
|
|
4664
6009
|
if (state.interactive) displayStatus(state);
|
|
4665
6010
|
}
|
|
4666
6011
|
}
|
|
4667
|
-
await notifyOffline(state);
|
|
6012
|
+
await timeShutdownPhase(state, durations, "offline_notify", () => notifyOffline(state));
|
|
4668
6013
|
if (state.connection) {
|
|
4669
|
-
state.connection
|
|
6014
|
+
const connection = state.connection;
|
|
6015
|
+
await timeShutdownPhase(state, durations, "tunnel_close", () => connection.close());
|
|
4670
6016
|
state.connection = null;
|
|
4671
6017
|
}
|
|
4672
6018
|
if (state.opencodeProcess) {
|
|
4673
|
-
|
|
6019
|
+
const opencodeProcess = state.opencodeProcess;
|
|
6020
|
+
await timeShutdownPhase(state, durations, "opencode_stop", () => stopOpenCode(opencodeProcess));
|
|
4674
6021
|
if (state.interactive) {
|
|
4675
6022
|
logActivity(state, { type: "info", message: "Stopped OpenCode process" });
|
|
4676
6023
|
displayStatus(state);
|
|
@@ -4679,12 +6026,15 @@ async function cleanup(state, opts = {}) {
|
|
|
4679
6026
|
}
|
|
4680
6027
|
state.opencodeProcess = null;
|
|
4681
6028
|
}
|
|
6029
|
+
return durations;
|
|
4682
6030
|
}
|
|
4683
6031
|
async function run(options) {
|
|
4684
6032
|
const interactive = isInteractive(options.json);
|
|
4685
6033
|
let logLevel;
|
|
6034
|
+
let fileSyncDirectories;
|
|
4686
6035
|
try {
|
|
4687
6036
|
logLevel = resolveLogLevel(options);
|
|
6037
|
+
fileSyncDirectories = resolveFileSyncDirectories(options.enableFileSyncTo, homedir2());
|
|
4688
6038
|
} catch (error2) {
|
|
4689
6039
|
const message = error2 instanceof Error ? error2.message : String(error2);
|
|
4690
6040
|
if (options.json) {
|
|
@@ -4719,6 +6069,12 @@ async function run(options) {
|
|
|
4719
6069
|
sessionCleanupTimers: [],
|
|
4720
6070
|
authHeader: ""
|
|
4721
6071
|
};
|
|
6072
|
+
setTelemetryAuthProvider(() => ({ authHeader: state.authHeader, agentId: state.agentId }));
|
|
6073
|
+
if (fileSyncDirectories.length > 0) {
|
|
6074
|
+
log2(state, `File sync enabled for: ${fileSyncDirectories.join(", ")}`);
|
|
6075
|
+
} else {
|
|
6076
|
+
log2(state, "File sync is disabled (no --enable-file-sync-to given)", "debug");
|
|
6077
|
+
}
|
|
4722
6078
|
if (!options.runner && options.agent) {
|
|
4723
6079
|
telemetry.info(
|
|
4724
6080
|
EventTypes.DEPRECATED_AGENT_FLAG_USED,
|
|
@@ -4742,14 +6098,38 @@ async function run(options) {
|
|
|
4742
6098
|
const handleSignal = async () => {
|
|
4743
6099
|
if (state.shuttingDown) return;
|
|
4744
6100
|
state.shuttingDown = true;
|
|
6101
|
+
const shutdownStartedAt = Date.now();
|
|
4745
6102
|
if (state.interactive) {
|
|
4746
6103
|
logActivity(state, { type: "info", message: "Shutting down..." });
|
|
4747
6104
|
displayStatus(state);
|
|
4748
6105
|
} else {
|
|
4749
6106
|
log2(state, "Shutting down...");
|
|
4750
6107
|
}
|
|
4751
|
-
await cleanup(state, { graceful: true });
|
|
4752
|
-
|
|
6108
|
+
const durations = await cleanup(state, { graceful: true });
|
|
6109
|
+
const telemetryBudgetMs = Number(process.env.EVIDENT_TELEMETRY_SHUTDOWN_MS) || TELEMETRY_SHUTDOWN_TIMEOUT_MS;
|
|
6110
|
+
await timeShutdownPhase(state, durations, "telemetry_flush", async () => {
|
|
6111
|
+
let timer;
|
|
6112
|
+
const flushed = shutdownTelemetry().then(
|
|
6113
|
+
() => true,
|
|
6114
|
+
(error2) => {
|
|
6115
|
+
log2(
|
|
6116
|
+
state,
|
|
6117
|
+
`Telemetry flush failed during shutdown: ${error2 instanceof Error ? error2.message : String(error2)}`,
|
|
6118
|
+
"warn"
|
|
6119
|
+
);
|
|
6120
|
+
return true;
|
|
6121
|
+
}
|
|
6122
|
+
);
|
|
6123
|
+
const timedOut = new Promise((resolve3) => {
|
|
6124
|
+
timer = setTimeout(() => resolve3(false), telemetryBudgetMs);
|
|
6125
|
+
});
|
|
6126
|
+
if (!await Promise.race([flushed, timedOut])) {
|
|
6127
|
+
log2(state, `Telemetry flush exceeded ${telemetryBudgetMs}ms \u2014 exiting anyway`, "warn");
|
|
6128
|
+
}
|
|
6129
|
+
clearTimeout(timer);
|
|
6130
|
+
});
|
|
6131
|
+
const breakdown = Object.entries(durations).map(([phase, ms]) => `${phase}=${ms}ms`).join(" ");
|
|
6132
|
+
log2(state, `Shutdown complete in ${Date.now() - shutdownStartedAt}ms (${breakdown})`);
|
|
4753
6133
|
process.exit(0);
|
|
4754
6134
|
};
|
|
4755
6135
|
process.on("SIGINT", handleSignal);
|
|
@@ -4863,25 +6243,67 @@ async function run(options) {
|
|
|
4863
6243
|
}
|
|
4864
6244
|
spinner?.succeed(`Runner: ${validation.agent.name || state.agentId}`);
|
|
4865
6245
|
state.agentName = validation.agent.name;
|
|
6246
|
+
const microvmId = process.env.MICROVM_ID?.trim();
|
|
6247
|
+
if (microvmId) {
|
|
6248
|
+
const reported = await reportMicrovmId(state.agentId, state.authHeader, microvmId);
|
|
6249
|
+
if (reported.ok) {
|
|
6250
|
+
log2(state, "Reported MicroVM identity so this runner can be resumed rather than restarted");
|
|
6251
|
+
} else {
|
|
6252
|
+
const message = `Could not report MicroVM identity (future wakes will cold-start): ${reported.error}`;
|
|
6253
|
+
log2(state, message, "warn");
|
|
6254
|
+
if (state.interactive && !state.json) {
|
|
6255
|
+
logActivity(state, { type: "info", level: "warn", message });
|
|
6256
|
+
}
|
|
6257
|
+
}
|
|
6258
|
+
} else {
|
|
6259
|
+
log2(state, "Not running in a MicroVM (MICROVM_ID unset) \u2014 nothing to report", "debug");
|
|
6260
|
+
}
|
|
6261
|
+
const { timeoutMs: opencodeStartTimeoutMs, warnings: opencodeStartTimeoutWarnings } = resolveOpenCodeStartTimeoutMs(options, process.env);
|
|
6262
|
+
for (const warning2 of opencodeStartTimeoutWarnings) {
|
|
6263
|
+
logActivity(state, { type: "info", level: "warn", message: warning2 });
|
|
6264
|
+
}
|
|
4866
6265
|
const ocSpinner = interactive && !state.json ? ora3("Checking OpenCode...").start() : null;
|
|
4867
6266
|
try {
|
|
4868
6267
|
const oc = await ensureOpenCodeRunning({
|
|
4869
6268
|
port: state.port,
|
|
4870
6269
|
interactive: state.interactive,
|
|
4871
6270
|
agentId: state.agentId,
|
|
4872
|
-
log: (message) => log2(state, message)
|
|
6271
|
+
log: (message) => log2(state, message),
|
|
6272
|
+
startTimeoutMs: opencodeStartTimeoutMs
|
|
4873
6273
|
});
|
|
4874
6274
|
state.port = oc.port;
|
|
4875
6275
|
state.opencodeProcess = oc.process;
|
|
4876
6276
|
state.opencodeVersion = oc.version;
|
|
4877
|
-
state.opencodeConnected = oc.
|
|
6277
|
+
state.opencodeConnected = oc.notReadyReason === null;
|
|
4878
6278
|
const version2 = state.opencodeVersion ? ` (v${state.opencodeVersion})` : "";
|
|
4879
6279
|
ocSpinner?.succeed(`OpenCode running on port ${state.port}${version2}`);
|
|
4880
|
-
|
|
4881
|
-
|
|
4882
|
-
|
|
4883
|
-
|
|
4884
|
-
|
|
6280
|
+
if (!state.interactive && oc.notReadyReason !== null) {
|
|
6281
|
+
const message = `OpenCode is not ready on port ${state.port}: ${oc.notReadyReason}. The runner will still come online, but messages will fail until opencode answers \u2014 raise the wait with --opencode-start-timeout <seconds> (env ${OPENCODE_START_TIMEOUT_ENV}).`;
|
|
6282
|
+
logActivity(state, { type: "info", level: "warn", message });
|
|
6283
|
+
} else {
|
|
6284
|
+
const versionWarning = buildOpenCodeVersionWarning(state.opencodeVersion);
|
|
6285
|
+
if (versionWarning) {
|
|
6286
|
+
log2(state, versionWarning, "warn");
|
|
6287
|
+
if (state.interactive && !state.json) {
|
|
6288
|
+
logActivity(state, { type: "info", level: "warn", message: versionWarning });
|
|
6289
|
+
}
|
|
6290
|
+
}
|
|
6291
|
+
const noProviderWarning = buildNoProviderWarning(
|
|
6292
|
+
await hasAnyConfiguredProvider(state.port)
|
|
6293
|
+
);
|
|
6294
|
+
if (noProviderWarning) {
|
|
6295
|
+
log2(state, noProviderWarning, "warn");
|
|
6296
|
+
if (state.interactive && !state.json) {
|
|
6297
|
+
logActivity(state, { type: "info", level: "warn", message: noProviderWarning });
|
|
6298
|
+
blank();
|
|
6299
|
+
console.log(chalk6.yellow("\u26A0 No OpenCode model provider is configured."));
|
|
6300
|
+
console.log(
|
|
6301
|
+
chalk6.dim(
|
|
6302
|
+
`Run ${chalk6.cyan("opencode auth login")} to set one up \u2014 messages will fail until then.`
|
|
6303
|
+
)
|
|
6304
|
+
);
|
|
6305
|
+
blank();
|
|
6306
|
+
}
|
|
4885
6307
|
}
|
|
4886
6308
|
}
|
|
4887
6309
|
} catch (error2) {
|
|
@@ -4896,6 +6318,10 @@ async function run(options) {
|
|
|
4896
6318
|
getAuthHeader: () => state.authHeader,
|
|
4897
6319
|
conversationFilter: state.conversationFilter,
|
|
4898
6320
|
stuckQueuedMs: CHANNEL_STUCK_QUEUED_MS,
|
|
6321
|
+
// #559: the `--enable-file-sync-to` allow-list. Empty ⇒ pending files are
|
|
6322
|
+
// REJECTED with `file_sync_disabled` on the ack, not silently ignored.
|
|
6323
|
+
fileSyncDirectories,
|
|
6324
|
+
homeDir: homedir2(),
|
|
4899
6325
|
log: (entry) => (
|
|
4900
6326
|
// Thread the driver's real level straight through so `debug`/`warn`
|
|
4901
6327
|
// survive the sink filter (they no longer collapse to info). `type`
|
|
@@ -4922,6 +6348,18 @@ async function run(options) {
|
|
|
4922
6348
|
type: "info",
|
|
4923
6349
|
message: `Tunnel ${isReconnect ? "reconnected" : "connected"} (runner: ${agentId})`
|
|
4924
6350
|
});
|
|
6351
|
+
if (options.tunnelReadyFile) {
|
|
6352
|
+
const marker = writeTunnelReadyMarker(options.tunnelReadyFile, agentId);
|
|
6353
|
+
if (marker.ok) {
|
|
6354
|
+
log2(state, `Wrote tunnel readiness marker to ${options.tunnelReadyFile}`, "debug");
|
|
6355
|
+
} else {
|
|
6356
|
+
log2(
|
|
6357
|
+
state,
|
|
6358
|
+
`Failed to write tunnel readiness marker to ${options.tunnelReadyFile}: ${marker.error}`,
|
|
6359
|
+
"error"
|
|
6360
|
+
);
|
|
6361
|
+
}
|
|
6362
|
+
}
|
|
4925
6363
|
emitAgentConnected(state.agentId, {
|
|
4926
6364
|
port: state.port,
|
|
4927
6365
|
cli_version: getCliVersion(),
|
|
@@ -4977,6 +6415,12 @@ async function run(options) {
|
|
|
4977
6415
|
onDrainPing: () => {
|
|
4978
6416
|
if (!state.running) return;
|
|
4979
6417
|
logActivity(state, { type: "info", message: "Drain ping received \u2014 draining" });
|
|
6418
|
+
void channelDriver.syncPendingFiles().catch(
|
|
6419
|
+
(error2) => logActivity(state, {
|
|
6420
|
+
type: "error",
|
|
6421
|
+
error: `Runner file sync failed on ping: ${error2 instanceof Error ? error2.message : String(error2)}`
|
|
6422
|
+
})
|
|
6423
|
+
);
|
|
4980
6424
|
channelDriver.drainPending().then((processed) => {
|
|
4981
6425
|
if (processed > 0) {
|
|
4982
6426
|
state.messageCount += processed;
|
|
@@ -5060,10 +6504,16 @@ program.name("evident").description("Run OpenCode locally and connect it to Evid
|
|
|
5060
6504
|
program.command("login").description("Authenticate with Evident").option("--token", "Use token-based authentication (for CI/CD)").option("--no-browser", "Do not open the browser automatically").action(login);
|
|
5061
6505
|
program.command("logout").description("Remove stored credentials for the current endpoint").option("--all", "Remove stored credentials for all endpoints").action((options) => logout({ all: options.all }));
|
|
5062
6506
|
program.command("whoami").description("Show the currently logged in user").action(whoami);
|
|
5063
|
-
program.command("run").description("Connect to Evident and process messages").option("
|
|
6507
|
+
program.command("run").description("Connect to Evident and process messages").option("--runner [id]", "Runner ID to connect to (optional when EVIDENT_RUNNER_KEY is set)").option(
|
|
6508
|
+
"-a, --agent [id]",
|
|
6509
|
+
"Deprecated alias for --runner (still supported; --runner wins if both are given)"
|
|
6510
|
+
).option("-p, --port <port>", "OpenCode port (default: 4096)", "4096").option(
|
|
5064
6511
|
"--log-level <level>",
|
|
5065
6512
|
"Log verbosity: debug | info | warn | error (default: info). Env: EVIDENT_LOG_LEVEL"
|
|
5066
|
-
).option("-v, --verbose", "Alias for --log-level debug (ignored if --log-level is set)").option("-c, --conversation <id>", "Process only this specific conversation").option("--idle-timeout <seconds>", "Exit after N seconds idle").option(
|
|
6513
|
+
).option("-v, --verbose", "Alias for --log-level debug (ignored if --log-level is set)").option("-c, --conversation <id>", "Process only this specific conversation").option("--idle-timeout <seconds>", "Exit after N seconds idle").option(
|
|
6514
|
+
"--opencode-start-timeout <seconds>",
|
|
6515
|
+
"Seconds to wait for OpenCode to become healthy when the runner starts it (default: 180). Env: EVIDENT_OPENCODE_START_TIMEOUT"
|
|
6516
|
+
).option("--json", "Output in JSON format").option(
|
|
5067
6517
|
"--session-cleanup-max-age <duration>",
|
|
5068
6518
|
"Delete OpenCode sessions idle longer than this (e.g. 7d, 24h). Enables cleanup. Env: EVIDENT_SESSION_CLEANUP_MAX_AGE"
|
|
5069
6519
|
).option(
|
|
@@ -5072,6 +6522,14 @@ program.command("run").description("Connect to Evident and process messages").op
|
|
|
5072
6522
|
).option(
|
|
5073
6523
|
"--session-cleanup-interval <duration>",
|
|
5074
6524
|
"How often the cleanup sweep runs (default: 1h). Env: EVIDENT_SESSION_CLEANUP_INTERVAL"
|
|
6525
|
+
).option(
|
|
6526
|
+
"--enable-file-sync-to <dir>",
|
|
6527
|
+
"Allow Evident to write files into this directory (repeatable). Omit to disable file sync entirely.",
|
|
6528
|
+
(value, previous) => previous.concat([value]),
|
|
6529
|
+
[]
|
|
6530
|
+
).option(
|
|
6531
|
+
"--tunnel-ready-file <path>",
|
|
6532
|
+
"Path to write once the tunnel is connected (set by the MicroVM hooks; unused on a developer machine)"
|
|
5075
6533
|
).action(
|
|
5076
6534
|
(options) => {
|
|
5077
6535
|
run({
|
|
@@ -5084,11 +6542,18 @@ program.command("run").description("Connect to Evident and process messages").op
|
|
|
5084
6542
|
verbose: options.verbose,
|
|
5085
6543
|
conversation: options.conversation,
|
|
5086
6544
|
idleTimeout: options.idleTimeout ? parseInt(options.idleTimeout, 10) : void 0,
|
|
6545
|
+
// Raw string — validation/precedence is single-sourced in run.ts's
|
|
6546
|
+
// resolveOpenCodeStartTimeoutMs (flag > EVIDENT_OPENCODE_START_TIMEOUT > 180s default).
|
|
6547
|
+
opencodeStartTimeout: options.opencodeStartTimeout,
|
|
5087
6548
|
json: options.json,
|
|
5088
6549
|
// Raw strings — the resolver in run.ts single-sources parsing (M1).
|
|
5089
6550
|
sessionCleanupMaxAge: options.sessionCleanupMaxAge,
|
|
5090
6551
|
sessionCleanupMaxCount: options.sessionCleanupMaxCount,
|
|
5091
|
-
sessionCleanupInterval: options.sessionCleanupInterval
|
|
6552
|
+
sessionCleanupInterval: options.sessionCleanupInterval,
|
|
6553
|
+
// Raw values — expansion/validation is single-sourced in run.ts's
|
|
6554
|
+
// resolveFileSyncDirectories.
|
|
6555
|
+
enableFileSyncTo: options.enableFileSyncTo,
|
|
6556
|
+
tunnelReadyFile: options.tunnelReadyFile
|
|
5092
6557
|
});
|
|
5093
6558
|
}
|
|
5094
6559
|
);
|