@evident-ai/cli 3.1.1-dev.51bb5fa → 3.1.1-dev.5a3c6a5
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 +15 -11
- package/dist/index.js +813 -42
- 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] || "";
|
|
@@ -285,14 +316,14 @@ function blank() {
|
|
|
285
316
|
console.log();
|
|
286
317
|
}
|
|
287
318
|
function waitForEnter(prompt = "Press Enter to continue...") {
|
|
288
|
-
return new Promise((
|
|
319
|
+
return new Promise((resolve3) => {
|
|
289
320
|
process.stdout.write(chalk.dim(prompt));
|
|
290
321
|
const handler = () => {
|
|
291
322
|
process.stdin.removeListener("data", handler);
|
|
292
323
|
process.stdin.setRawMode?.(false);
|
|
293
324
|
process.stdin.pause();
|
|
294
325
|
console.log();
|
|
295
|
-
|
|
326
|
+
resolve3();
|
|
296
327
|
};
|
|
297
328
|
if (process.stdin.isTTY) {
|
|
298
329
|
process.stdin.setRawMode?.(true);
|
|
@@ -302,7 +333,7 @@ function waitForEnter(prompt = "Press Enter to continue...") {
|
|
|
302
333
|
});
|
|
303
334
|
}
|
|
304
335
|
function sleep(ms) {
|
|
305
|
-
return new Promise((
|
|
336
|
+
return new Promise((resolve3) => setTimeout(resolve3, ms));
|
|
306
337
|
}
|
|
307
338
|
|
|
308
339
|
// src/commands/login.ts
|
|
@@ -373,22 +404,23 @@ async function deviceFlowLogin(options) {
|
|
|
373
404
|
}
|
|
374
405
|
async function tokenLogin() {
|
|
375
406
|
console.log("Token login mode.");
|
|
376
|
-
console.log("
|
|
407
|
+
console.log("Run `evident login` on a machine with a browser to get a token.");
|
|
408
|
+
console.log("Manage or revoke existing tokens under Settings \u2192 CLI tokens.");
|
|
377
409
|
blank();
|
|
378
410
|
process.stdout.write("Paste token: ");
|
|
379
|
-
const token = await new Promise((
|
|
411
|
+
const token = await new Promise((resolve3) => {
|
|
380
412
|
let data = "";
|
|
381
413
|
process.stdin.setEncoding("utf8");
|
|
382
414
|
process.stdin.on("data", (chunk) => {
|
|
383
415
|
data += chunk;
|
|
384
416
|
});
|
|
385
417
|
process.stdin.on("end", () => {
|
|
386
|
-
|
|
418
|
+
resolve3(data.trim());
|
|
387
419
|
});
|
|
388
420
|
if (process.stdin.isTTY) {
|
|
389
421
|
process.stdin.once("data", (chunk) => {
|
|
390
422
|
process.stdin.pause();
|
|
391
|
-
|
|
423
|
+
resolve3(chunk.toString().trim());
|
|
392
424
|
});
|
|
393
425
|
process.stdin.resume();
|
|
394
426
|
}
|
|
@@ -467,9 +499,9 @@ async function whoami() {
|
|
|
467
499
|
}
|
|
468
500
|
|
|
469
501
|
// src/commands/run.ts
|
|
502
|
+
import { homedir as homedir2 } from "os";
|
|
503
|
+
import { isAbsolute as isAbsolute2, join as join2, parse, resolve as resolvePath } from "path";
|
|
470
504
|
import chalk6 from "chalk";
|
|
471
|
-
import ora3 from "ora";
|
|
472
|
-
import { select as select3 } from "@inquirer/prompts";
|
|
473
505
|
|
|
474
506
|
// ../../packages/types/src/telemetry/index.ts
|
|
475
507
|
var TelemetryEventTypes = {
|
|
@@ -485,6 +517,10 @@ var TelemetryEventTypes = {
|
|
|
485
517
|
var MAX_FRAME_BYTES = 256 * 1024;
|
|
486
518
|
var TUNNEL_DRAIN_PING_PATH = "/__evident/drain";
|
|
487
519
|
|
|
520
|
+
// ../../packages/types/src/runner-files.ts
|
|
521
|
+
var MAX_FILE_PUSH_BYTES = 64 * 1024;
|
|
522
|
+
var MAX_FILE_SYNC_DIRECTORIES = 16;
|
|
523
|
+
|
|
488
524
|
// ../../packages/types/src/logging/index.ts
|
|
489
525
|
var CORRELATION_ID_HEADER = "x-evident-correlation-id";
|
|
490
526
|
function log(level, event, fields) {
|
|
@@ -499,6 +535,12 @@ function log(level, event, fields) {
|
|
|
499
535
|
);
|
|
500
536
|
}
|
|
501
537
|
}
|
|
538
|
+
function errorFields(err) {
|
|
539
|
+
if (err instanceof Error) {
|
|
540
|
+
return { error: err.message, error_name: err.name };
|
|
541
|
+
}
|
|
542
|
+
return { error: String(err) };
|
|
543
|
+
}
|
|
502
544
|
function stripQuery(url) {
|
|
503
545
|
try {
|
|
504
546
|
return new URL(url).pathname;
|
|
@@ -508,6 +550,10 @@ function stripQuery(url) {
|
|
|
508
550
|
}
|
|
509
551
|
}
|
|
510
552
|
|
|
553
|
+
// src/commands/run.ts
|
|
554
|
+
import ora3 from "ora";
|
|
555
|
+
import { select as select3 } from "@inquirer/prompts";
|
|
556
|
+
|
|
511
557
|
// src/lib/telemetry.ts
|
|
512
558
|
var CLI_VERSION = (true ? "3.0.0" : void 0) ?? process.env.npm_package_version ?? "unknown";
|
|
513
559
|
function getCliVersion() {
|
|
@@ -719,7 +765,7 @@ async function waitForOpenCodeHealth(port, timeoutMs = 3e4) {
|
|
|
719
765
|
if (health.healthy) {
|
|
720
766
|
return health;
|
|
721
767
|
}
|
|
722
|
-
await new Promise((
|
|
768
|
+
await new Promise((resolve3) => setTimeout(resolve3, 1e3));
|
|
723
769
|
}
|
|
724
770
|
return { healthy: false, error: "Timeout waiting for OpenCode to be healthy" };
|
|
725
771
|
}
|
|
@@ -1267,6 +1313,16 @@ async function buildFileParts(attachments, capable) {
|
|
|
1267
1313
|
);
|
|
1268
1314
|
dataUrl = null;
|
|
1269
1315
|
}
|
|
1316
|
+
if (dataUrl !== null && typeof dataUrl === "object") {
|
|
1317
|
+
outcomes.push({
|
|
1318
|
+
index: a.index,
|
|
1319
|
+
mime: a.mime,
|
|
1320
|
+
filename: a.filename,
|
|
1321
|
+
status: "failed",
|
|
1322
|
+
reason: "needs_reauth"
|
|
1323
|
+
});
|
|
1324
|
+
continue;
|
|
1325
|
+
}
|
|
1270
1326
|
if (dataUrl == null) {
|
|
1271
1327
|
outcomes.push({ index: a.index, mime: a.mime, filename: a.filename, status: "failed" });
|
|
1272
1328
|
continue;
|
|
@@ -1348,7 +1404,7 @@ async function sendPromptAsync(port, sessionId, content, options, attachments) {
|
|
|
1348
1404
|
}
|
|
1349
1405
|
}
|
|
1350
1406
|
if (attempt < READ_BACK_ATTEMPTS - 1) {
|
|
1351
|
-
await new Promise((
|
|
1407
|
+
await new Promise((resolve3) => setTimeout(resolve3, READ_BACK_DELAY_MS));
|
|
1352
1408
|
}
|
|
1353
1409
|
}
|
|
1354
1410
|
return null;
|
|
@@ -1684,10 +1740,11 @@ var StreamForwarder = class {
|
|
|
1684
1740
|
* Abort every in-flight stream (e.g. on WebSocket close).
|
|
1685
1741
|
*/
|
|
1686
1742
|
abortAll() {
|
|
1687
|
-
for (const stream of this.inflight.
|
|
1743
|
+
for (const [sid, stream] of this.inflight.entries()) {
|
|
1688
1744
|
try {
|
|
1689
1745
|
stream.abort();
|
|
1690
|
-
} catch {
|
|
1746
|
+
} catch (err) {
|
|
1747
|
+
log("error", "forwarder_abort_failed", { sid, ...errorFields(err) });
|
|
1691
1748
|
}
|
|
1692
1749
|
}
|
|
1693
1750
|
this.inflight.clear();
|
|
@@ -1721,12 +1778,12 @@ var StreamForwarder = class {
|
|
|
1721
1778
|
let endBody;
|
|
1722
1779
|
if (has_body) {
|
|
1723
1780
|
const chunks = [];
|
|
1724
|
-
bodyPromise = new Promise((
|
|
1781
|
+
bodyPromise = new Promise((resolve3) => {
|
|
1725
1782
|
pushBody = (buf) => {
|
|
1726
1783
|
chunks.push(buf);
|
|
1727
1784
|
};
|
|
1728
1785
|
endBody = () => {
|
|
1729
|
-
|
|
1786
|
+
resolve3(Buffer.concat(chunks));
|
|
1730
1787
|
};
|
|
1731
1788
|
});
|
|
1732
1789
|
}
|
|
@@ -1843,7 +1900,7 @@ function connectTunnel(options) {
|
|
|
1843
1900
|
} = options;
|
|
1844
1901
|
const tunnelUrl = getTunnelUrlConfig();
|
|
1845
1902
|
const url = `${tunnelUrl}/tunnel/${agentId}/connect`;
|
|
1846
|
-
return new Promise((
|
|
1903
|
+
return new Promise((resolve3, reject) => {
|
|
1847
1904
|
const ws = new WebSocket2(url, {
|
|
1848
1905
|
headers: {
|
|
1849
1906
|
Authorization: authHeader
|
|
@@ -1898,7 +1955,7 @@ function connectTunnel(options) {
|
|
|
1898
1955
|
clearTimeout(connectionTimeout);
|
|
1899
1956
|
const connectedAgentId = message.agent_id ?? agentId;
|
|
1900
1957
|
onConnected?.(connectedAgentId);
|
|
1901
|
-
|
|
1958
|
+
resolve3({
|
|
1902
1959
|
ws,
|
|
1903
1960
|
close: () => ws.close(1e3, "CLI shutdown")
|
|
1904
1961
|
});
|
|
@@ -1960,7 +2017,11 @@ var RunnerConnection = class {
|
|
|
1960
2017
|
if (this.connection) {
|
|
1961
2018
|
try {
|
|
1962
2019
|
this.connection.close();
|
|
1963
|
-
} catch {
|
|
2020
|
+
} catch (err) {
|
|
2021
|
+
log("error", "runner_connection_close_failed", {
|
|
2022
|
+
agent_id: this.resolvedAgentId,
|
|
2023
|
+
...errorFields(err)
|
|
2024
|
+
});
|
|
1964
2025
|
}
|
|
1965
2026
|
this.connection = null;
|
|
1966
2027
|
}
|
|
@@ -2012,6 +2073,404 @@ var RunnerConnection = class {
|
|
|
2012
2073
|
}
|
|
2013
2074
|
};
|
|
2014
2075
|
|
|
2076
|
+
// src/lib/channels/driver.ts
|
|
2077
|
+
import { homedir } from "os";
|
|
2078
|
+
|
|
2079
|
+
// src/lib/file-push.ts
|
|
2080
|
+
import { randomUUID } from "crypto";
|
|
2081
|
+
import { chmod, mkdir, open as open2, realpath, rename, unlink } from "fs/promises";
|
|
2082
|
+
import { basename, dirname as dirname2, isAbsolute, join, relative, resolve as resolve2, sep } from "path";
|
|
2083
|
+
var FILE_MODE = 384;
|
|
2084
|
+
var DIRECTORY_MODE = 448;
|
|
2085
|
+
async function writePushedFile(request) {
|
|
2086
|
+
const { requestedPath, content, allowedDirectories, homeDir } = request;
|
|
2087
|
+
const bytes = content.byteLength;
|
|
2088
|
+
if (allowedDirectories.length === 0) {
|
|
2089
|
+
return refuse("file_sync_disabled", "File sync is not enabled on this runner.", {
|
|
2090
|
+
path: requestedPath,
|
|
2091
|
+
bytes
|
|
2092
|
+
});
|
|
2093
|
+
}
|
|
2094
|
+
if (bytes > MAX_FILE_PUSH_BYTES) {
|
|
2095
|
+
return refuse(
|
|
2096
|
+
"file_too_large",
|
|
2097
|
+
`File is ${bytes} bytes; the limit is ${MAX_FILE_PUSH_BYTES}.`,
|
|
2098
|
+
{
|
|
2099
|
+
path: requestedPath,
|
|
2100
|
+
bytes
|
|
2101
|
+
}
|
|
2102
|
+
);
|
|
2103
|
+
}
|
|
2104
|
+
const candidate = expandAndValidate(requestedPath, homeDir);
|
|
2105
|
+
if (candidate === null) {
|
|
2106
|
+
return refuse("invalid_path", "The requested path is not a valid absolute file path.", {
|
|
2107
|
+
path: requestedPath,
|
|
2108
|
+
bytes
|
|
2109
|
+
});
|
|
2110
|
+
}
|
|
2111
|
+
try {
|
|
2112
|
+
const { existingAncestor, missingSegments } = await resolveNearestExistingAncestor(
|
|
2113
|
+
dirname2(candidate)
|
|
2114
|
+
);
|
|
2115
|
+
const realTarget = join(existingAncestor, ...missingSegments, basename(candidate));
|
|
2116
|
+
const allowedDirectory = await findContainingAllowedDirectory(allowedDirectories, realTarget);
|
|
2117
|
+
if (allowedDirectory === null) {
|
|
2118
|
+
return refuse("path_not_allowed", "The runner does not allow writing to that location.", {
|
|
2119
|
+
path: realTarget,
|
|
2120
|
+
bytes
|
|
2121
|
+
});
|
|
2122
|
+
}
|
|
2123
|
+
if (missingSegments.length > 0) {
|
|
2124
|
+
await createMissingDirectories(existingAncestor, missingSegments);
|
|
2125
|
+
const realParent = await realpath(dirname2(realTarget));
|
|
2126
|
+
if (realParent !== dirname2(realTarget) || !contains(allowedDirectory, realTarget)) {
|
|
2127
|
+
return refuse("path_not_allowed", "The runner does not allow writing to that location.", {
|
|
2128
|
+
path: realTarget,
|
|
2129
|
+
bytes,
|
|
2130
|
+
reason: "parent_changed_after_create"
|
|
2131
|
+
});
|
|
2132
|
+
}
|
|
2133
|
+
}
|
|
2134
|
+
await writeAtomically(realTarget, content);
|
|
2135
|
+
log("info", "file_push_written", { path: realTarget, bytes });
|
|
2136
|
+
return { ok: true, path: realTarget };
|
|
2137
|
+
} catch (err) {
|
|
2138
|
+
const errno = err.code ?? "UNKNOWN";
|
|
2139
|
+
return refuse("write_failed", `The runner could not write the file (${errno}).`, {
|
|
2140
|
+
path: candidate,
|
|
2141
|
+
bytes,
|
|
2142
|
+
errno,
|
|
2143
|
+
...errorFields(err)
|
|
2144
|
+
});
|
|
2145
|
+
}
|
|
2146
|
+
}
|
|
2147
|
+
function expandAndValidate(requestedPath, homeDir) {
|
|
2148
|
+
if (requestedPath.trim() === "" || requestedPath.includes("\0")) {
|
|
2149
|
+
return null;
|
|
2150
|
+
}
|
|
2151
|
+
const expanded = requestedPath === "~" ? homeDir : requestedPath.startsWith("~/") ? join(homeDir, requestedPath.slice(2)) : requestedPath;
|
|
2152
|
+
if (expanded.split(/[/\\]/).includes("..")) {
|
|
2153
|
+
return null;
|
|
2154
|
+
}
|
|
2155
|
+
if (!isAbsolute(expanded)) {
|
|
2156
|
+
return null;
|
|
2157
|
+
}
|
|
2158
|
+
const candidate = resolve2(expanded);
|
|
2159
|
+
const name = basename(candidate);
|
|
2160
|
+
return name === "" || name === "." || name === ".." ? null : candidate;
|
|
2161
|
+
}
|
|
2162
|
+
async function resolveNearestExistingAncestor(directory) {
|
|
2163
|
+
const missingSegments = [];
|
|
2164
|
+
let current = directory;
|
|
2165
|
+
for (; ; ) {
|
|
2166
|
+
try {
|
|
2167
|
+
return { existingAncestor: await realpath(current), missingSegments };
|
|
2168
|
+
} catch (err) {
|
|
2169
|
+
const parent = dirname2(current);
|
|
2170
|
+
if (err.code !== "ENOENT" || parent === current) {
|
|
2171
|
+
throw err;
|
|
2172
|
+
}
|
|
2173
|
+
missingSegments.unshift(basename(current));
|
|
2174
|
+
current = parent;
|
|
2175
|
+
}
|
|
2176
|
+
}
|
|
2177
|
+
}
|
|
2178
|
+
async function findContainingAllowedDirectory(allowedDirectories, realTarget) {
|
|
2179
|
+
for (const directory of allowedDirectories) {
|
|
2180
|
+
if (!isAbsolute(directory)) {
|
|
2181
|
+
log("warn", "file_push_allowed_directory_skipped", { directory, reason: "not_absolute" });
|
|
2182
|
+
continue;
|
|
2183
|
+
}
|
|
2184
|
+
const realDirectory = await realpathCreatingIfMissing(directory);
|
|
2185
|
+
if (realDirectory !== null && contains(realDirectory, realTarget)) {
|
|
2186
|
+
return realDirectory;
|
|
2187
|
+
}
|
|
2188
|
+
}
|
|
2189
|
+
return null;
|
|
2190
|
+
}
|
|
2191
|
+
async function realpathCreatingIfMissing(directory) {
|
|
2192
|
+
try {
|
|
2193
|
+
return await realpath(directory);
|
|
2194
|
+
} catch (err) {
|
|
2195
|
+
if (err.code !== "ENOENT") {
|
|
2196
|
+
log("warn", "file_push_allowed_directory_skipped", {
|
|
2197
|
+
directory,
|
|
2198
|
+
reason: "unresolvable",
|
|
2199
|
+
...errorFields(err)
|
|
2200
|
+
});
|
|
2201
|
+
return null;
|
|
2202
|
+
}
|
|
2203
|
+
}
|
|
2204
|
+
try {
|
|
2205
|
+
await mkdir(directory, { recursive: true, mode: DIRECTORY_MODE });
|
|
2206
|
+
await chmod(directory, DIRECTORY_MODE);
|
|
2207
|
+
return await realpath(directory);
|
|
2208
|
+
} catch (err) {
|
|
2209
|
+
log("warn", "file_push_allowed_directory_skipped", {
|
|
2210
|
+
directory,
|
|
2211
|
+
reason: "create_failed",
|
|
2212
|
+
...errorFields(err)
|
|
2213
|
+
});
|
|
2214
|
+
return null;
|
|
2215
|
+
}
|
|
2216
|
+
}
|
|
2217
|
+
function contains(realDirectory, realTarget) {
|
|
2218
|
+
const rel = relative(realDirectory, realTarget);
|
|
2219
|
+
return rel !== "" && rel !== ".." && !rel.startsWith(`..${sep}`) && !isAbsolute(rel);
|
|
2220
|
+
}
|
|
2221
|
+
async function createMissingDirectories(existingAncestor, missingSegments) {
|
|
2222
|
+
let current = existingAncestor;
|
|
2223
|
+
for (const segment of missingSegments) {
|
|
2224
|
+
current = join(current, segment);
|
|
2225
|
+
await mkdir(current, { recursive: true, mode: DIRECTORY_MODE });
|
|
2226
|
+
await chmod(current, DIRECTORY_MODE);
|
|
2227
|
+
}
|
|
2228
|
+
}
|
|
2229
|
+
async function writeAtomically(realTarget, content) {
|
|
2230
|
+
const temporaryPath = join(dirname2(realTarget), `.evident-push-${randomUUID()}.tmp`);
|
|
2231
|
+
let handle;
|
|
2232
|
+
try {
|
|
2233
|
+
handle = await open2(temporaryPath, "wx", FILE_MODE);
|
|
2234
|
+
await handle.writeFile(content);
|
|
2235
|
+
await handle.chmod(FILE_MODE);
|
|
2236
|
+
await handle.close();
|
|
2237
|
+
handle = void 0;
|
|
2238
|
+
await rename(temporaryPath, realTarget);
|
|
2239
|
+
} catch (err) {
|
|
2240
|
+
await discardTemporaryFile(temporaryPath, handle);
|
|
2241
|
+
throw err;
|
|
2242
|
+
}
|
|
2243
|
+
}
|
|
2244
|
+
async function discardTemporaryFile(temporaryPath, handle) {
|
|
2245
|
+
try {
|
|
2246
|
+
await handle?.close();
|
|
2247
|
+
} catch (err) {
|
|
2248
|
+
log("warn", "file_push_temp_close_failed", { path: temporaryPath, ...errorFields(err) });
|
|
2249
|
+
}
|
|
2250
|
+
try {
|
|
2251
|
+
await unlink(temporaryPath);
|
|
2252
|
+
} catch (err) {
|
|
2253
|
+
const errno = err.code;
|
|
2254
|
+
if (errno !== "ENOENT" && errno !== "ENOTDIR") {
|
|
2255
|
+
log("warn", "file_push_temp_cleanup_failed", { path: temporaryPath, ...errorFields(err) });
|
|
2256
|
+
}
|
|
2257
|
+
}
|
|
2258
|
+
}
|
|
2259
|
+
function refuse(code, message, fields) {
|
|
2260
|
+
log(code === "write_failed" ? "error" : "warn", "file_push_refused", { code, ...fields });
|
|
2261
|
+
return { ok: false, code, message };
|
|
2262
|
+
}
|
|
2263
|
+
|
|
2264
|
+
// src/lib/runner-file-sync.ts
|
|
2265
|
+
var MAX_ACK_ATTEMPTS = 5;
|
|
2266
|
+
async function syncPendingRunnerFiles(options) {
|
|
2267
|
+
const pending = await listPendingFiles(options);
|
|
2268
|
+
const pendingIds = new Set(pending.map((file) => file.id));
|
|
2269
|
+
for (const id of options.ackFailures.keys()) {
|
|
2270
|
+
if (!pendingIds.has(id)) options.ackFailures.delete(id);
|
|
2271
|
+
}
|
|
2272
|
+
if (pending.length === 0) return 0;
|
|
2273
|
+
options.log({
|
|
2274
|
+
level: "info",
|
|
2275
|
+
message: `Runner file sync: ${pending.length} file(s) queued for this runner`
|
|
2276
|
+
});
|
|
2277
|
+
let applied = 0;
|
|
2278
|
+
for (const file of pending) {
|
|
2279
|
+
if ((options.ackFailures.get(file.id) ?? 0) >= MAX_ACK_ATTEMPTS) continue;
|
|
2280
|
+
if (await applyOne(options, file)) applied += 1;
|
|
2281
|
+
}
|
|
2282
|
+
return applied;
|
|
2283
|
+
}
|
|
2284
|
+
async function listPendingFiles(options) {
|
|
2285
|
+
let res;
|
|
2286
|
+
try {
|
|
2287
|
+
res = await options.fetchImpl(`${options.apiUrl}/runners/${options.agentId}/files/pending`, {
|
|
2288
|
+
headers: { Authorization: options.getAuthHeader() }
|
|
2289
|
+
});
|
|
2290
|
+
} catch (err) {
|
|
2291
|
+
options.log({
|
|
2292
|
+
level: "warn",
|
|
2293
|
+
message: `Could not list pending runner files \u2014 retrying on the next drain: ${describe(err)}`
|
|
2294
|
+
});
|
|
2295
|
+
return [];
|
|
2296
|
+
}
|
|
2297
|
+
if (!res.ok) {
|
|
2298
|
+
options.log({
|
|
2299
|
+
level: res.status === 404 ? "debug" : "warn",
|
|
2300
|
+
message: `Listing pending runner files returned HTTP ${res.status} \u2014 retrying on the next drain`
|
|
2301
|
+
});
|
|
2302
|
+
return [];
|
|
2303
|
+
}
|
|
2304
|
+
let body;
|
|
2305
|
+
try {
|
|
2306
|
+
body = await res.json();
|
|
2307
|
+
} catch (err) {
|
|
2308
|
+
options.log({
|
|
2309
|
+
level: "warn",
|
|
2310
|
+
message: `Pending runner file list was not readable JSON \u2014 retrying on the next drain: ${describe(err)}`
|
|
2311
|
+
});
|
|
2312
|
+
return [];
|
|
2313
|
+
}
|
|
2314
|
+
if (!Array.isArray(body)) {
|
|
2315
|
+
options.log({
|
|
2316
|
+
level: "warn",
|
|
2317
|
+
message: "Pending runner file list was not an array \u2014 ignoring it for this drain"
|
|
2318
|
+
});
|
|
2319
|
+
return [];
|
|
2320
|
+
}
|
|
2321
|
+
const files = [];
|
|
2322
|
+
for (const entry of body) {
|
|
2323
|
+
const file = asPendingFile(entry);
|
|
2324
|
+
if (file === null) {
|
|
2325
|
+
options.log({
|
|
2326
|
+
level: "warn",
|
|
2327
|
+
message: "Ignoring a malformed pending runner file entry (expected id, path and size)"
|
|
2328
|
+
});
|
|
2329
|
+
continue;
|
|
2330
|
+
}
|
|
2331
|
+
files.push(file);
|
|
2332
|
+
}
|
|
2333
|
+
return files;
|
|
2334
|
+
}
|
|
2335
|
+
function asPendingFile(entry) {
|
|
2336
|
+
if (entry === null || typeof entry !== "object") return null;
|
|
2337
|
+
const { id, path, size } = entry;
|
|
2338
|
+
if (typeof id !== "string" || id === "") return null;
|
|
2339
|
+
if (typeof path !== "string" || path === "") return null;
|
|
2340
|
+
if (typeof size !== "number" || !Number.isFinite(size) || size < 0) return null;
|
|
2341
|
+
return { id, path, size };
|
|
2342
|
+
}
|
|
2343
|
+
async function applyOne(options, file) {
|
|
2344
|
+
const label = `${file.id.slice(0, 8)} (${file.path})`;
|
|
2345
|
+
if (options.allowedDirectories.length === 0) {
|
|
2346
|
+
options.log({
|
|
2347
|
+
level: "warn",
|
|
2348
|
+
message: `Runner file ${label} rejected: file sync is not enabled on this runner (start it with --enable-file-sync-to)`
|
|
2349
|
+
});
|
|
2350
|
+
await ack(options, file, "rejected", "file_sync_disabled");
|
|
2351
|
+
return false;
|
|
2352
|
+
}
|
|
2353
|
+
if (file.size > MAX_FILE_PUSH_BYTES) {
|
|
2354
|
+
options.log({
|
|
2355
|
+
level: "warn",
|
|
2356
|
+
message: `Runner file ${label} rejected: declared ${file.size} bytes, the limit is ${MAX_FILE_PUSH_BYTES}`
|
|
2357
|
+
});
|
|
2358
|
+
await ack(options, file, "rejected", "file_too_large");
|
|
2359
|
+
return false;
|
|
2360
|
+
}
|
|
2361
|
+
const download = await downloadContent(options, file, label);
|
|
2362
|
+
if (!download.ok) {
|
|
2363
|
+
if (download.terminal) await ack(options, file, "rejected", download.code);
|
|
2364
|
+
return false;
|
|
2365
|
+
}
|
|
2366
|
+
let outcome;
|
|
2367
|
+
try {
|
|
2368
|
+
outcome = await writePushedFile({
|
|
2369
|
+
requestedPath: file.path,
|
|
2370
|
+
content: download.content,
|
|
2371
|
+
allowedDirectories: options.allowedDirectories,
|
|
2372
|
+
homeDir: options.homeDir
|
|
2373
|
+
});
|
|
2374
|
+
} catch (err) {
|
|
2375
|
+
options.log({
|
|
2376
|
+
level: "error",
|
|
2377
|
+
message: `Runner file ${label} could not be written: ${describe(err)}`
|
|
2378
|
+
});
|
|
2379
|
+
await ack(options, file, "rejected", "write_failed");
|
|
2380
|
+
return false;
|
|
2381
|
+
}
|
|
2382
|
+
if (!outcome.ok) {
|
|
2383
|
+
options.log({
|
|
2384
|
+
level: "warn",
|
|
2385
|
+
message: `Runner file ${label} rejected (${outcome.code}): ${outcome.message}`
|
|
2386
|
+
});
|
|
2387
|
+
await ack(options, file, "rejected", outcome.code);
|
|
2388
|
+
return false;
|
|
2389
|
+
}
|
|
2390
|
+
options.log({
|
|
2391
|
+
level: "info",
|
|
2392
|
+
message: `Runner file ${label} applied (${download.content.byteLength} bytes)`
|
|
2393
|
+
});
|
|
2394
|
+
await ack(options, file, "applied");
|
|
2395
|
+
return true;
|
|
2396
|
+
}
|
|
2397
|
+
function durableDownloadCode(status) {
|
|
2398
|
+
return status === 413 ? "file_too_large" : "write_failed";
|
|
2399
|
+
}
|
|
2400
|
+
async function downloadContent(options, file, label) {
|
|
2401
|
+
try {
|
|
2402
|
+
const res = await options.fetchImpl(
|
|
2403
|
+
`${options.apiUrl}/runners/${options.agentId}/files/${file.id}/content`,
|
|
2404
|
+
{ headers: { Authorization: options.getAuthHeader() } }
|
|
2405
|
+
);
|
|
2406
|
+
if (!res.ok) {
|
|
2407
|
+
const terminal = res.status >= 400 && res.status < 500 && res.status !== 401 && res.status !== 403 && res.status !== 408 && res.status !== 429;
|
|
2408
|
+
if (!terminal) {
|
|
2409
|
+
options.log({
|
|
2410
|
+
level: "warn",
|
|
2411
|
+
message: `Downloading runner file ${label} returned HTTP ${res.status} \u2014 retrying on the next drain`
|
|
2412
|
+
});
|
|
2413
|
+
return { ok: false, terminal: false };
|
|
2414
|
+
}
|
|
2415
|
+
const code = durableDownloadCode(res.status);
|
|
2416
|
+
options.log({
|
|
2417
|
+
level: "error",
|
|
2418
|
+
message: `Downloading runner file ${label} returned HTTP ${res.status} \u2014 rejecting it as ${code} (the bytes never reached the writer)`
|
|
2419
|
+
});
|
|
2420
|
+
return { ok: false, terminal: true, code };
|
|
2421
|
+
}
|
|
2422
|
+
return { ok: true, content: Buffer.from(await res.arrayBuffer()) };
|
|
2423
|
+
} catch (err) {
|
|
2424
|
+
options.log({
|
|
2425
|
+
level: "warn",
|
|
2426
|
+
message: `Downloading runner file ${label} failed \u2014 retrying on the next drain: ${describe(err)}`
|
|
2427
|
+
});
|
|
2428
|
+
return { ok: false, terminal: false };
|
|
2429
|
+
}
|
|
2430
|
+
}
|
|
2431
|
+
async function ack(options, file, status, reason) {
|
|
2432
|
+
const outcome = `${status}${reason ? ` (${reason})` : ""}`;
|
|
2433
|
+
try {
|
|
2434
|
+
const res = await options.fetchImpl(
|
|
2435
|
+
`${options.apiUrl}/runners/${options.agentId}/files/${file.id}/ack`,
|
|
2436
|
+
{
|
|
2437
|
+
method: "POST",
|
|
2438
|
+
headers: {
|
|
2439
|
+
Authorization: options.getAuthHeader(),
|
|
2440
|
+
"Content-Type": "application/json"
|
|
2441
|
+
},
|
|
2442
|
+
body: JSON.stringify(reason ? { status, reason } : { status })
|
|
2443
|
+
}
|
|
2444
|
+
);
|
|
2445
|
+
if (!res.ok) {
|
|
2446
|
+
recordAckFailure(
|
|
2447
|
+
options,
|
|
2448
|
+
file,
|
|
2449
|
+
`Acking runner file ${file.id.slice(0, 8)} as ${outcome} returned HTTP ${res.status}`
|
|
2450
|
+
);
|
|
2451
|
+
return;
|
|
2452
|
+
}
|
|
2453
|
+
options.ackFailures.delete(file.id);
|
|
2454
|
+
} catch (err) {
|
|
2455
|
+
recordAckFailure(
|
|
2456
|
+
options,
|
|
2457
|
+
file,
|
|
2458
|
+
`Acking runner file ${file.id.slice(0, 8)} as ${outcome} failed: ${describe(err)}`
|
|
2459
|
+
);
|
|
2460
|
+
}
|
|
2461
|
+
}
|
|
2462
|
+
function recordAckFailure(options, file, what) {
|
|
2463
|
+
const attempts = (options.ackFailures.get(file.id) ?? 0) + 1;
|
|
2464
|
+
options.ackFailures.set(file.id, attempts);
|
|
2465
|
+
options.log({
|
|
2466
|
+
level: "error",
|
|
2467
|
+
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})`
|
|
2468
|
+
});
|
|
2469
|
+
}
|
|
2470
|
+
function describe(err) {
|
|
2471
|
+
return err instanceof Error ? err.message : String(err);
|
|
2472
|
+
}
|
|
2473
|
+
|
|
2015
2474
|
// src/lib/channels/driver.ts
|
|
2016
2475
|
function messageIdOf(m) {
|
|
2017
2476
|
if (!m || typeof m !== "object") return void 0;
|
|
@@ -2041,6 +2500,7 @@ var DEFAULT_STUCK_QUEUED_MS = 6e4;
|
|
|
2041
2500
|
var HEARTBEAT_MS = 6e4;
|
|
2042
2501
|
var ABSOLUTE_MAX_PROCESSING_MS = 6 * 60 * 60 * 1e3;
|
|
2043
2502
|
var POLL_MISS_GRACE_MS = HEARTBEAT_MS;
|
|
2503
|
+
var MAX_SUPERSEDED_CONVERSATIONS = 256;
|
|
2044
2504
|
var ChannelAuthError = class extends Error {
|
|
2045
2505
|
constructor(message) {
|
|
2046
2506
|
super(message);
|
|
@@ -2077,8 +2537,38 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
2077
2537
|
pausedMaxWaitMs;
|
|
2078
2538
|
stuckQueuedMs;
|
|
2079
2539
|
now;
|
|
2540
|
+
fileSyncDirectories;
|
|
2541
|
+
homeDir;
|
|
2080
2542
|
/** Cache of conversationId → opencode sessionId. */
|
|
2081
2543
|
sessions = /* @__PURE__ */ new Map();
|
|
2544
|
+
/**
|
|
2545
|
+
* conversationId → the opencode session this runner has ABANDONED as that
|
|
2546
|
+
* conversation's binding (#553), after a genuine (`sessionExists === true`)
|
|
2547
|
+
* dispatch failure: the session still exists but is wedged, so #485's self-heal
|
|
2548
|
+
* must bind a fresh one.
|
|
2549
|
+
*
|
|
2550
|
+
* Dropping the local binding + clearing the server row is not enough on its own:
|
|
2551
|
+
* a SIBLING message dispatched earlier in the same drain is still in-flight under
|
|
2552
|
+
* the same session, and its watcher's routine status writes carry
|
|
2553
|
+
* `opencode_session_id`, RESURRECTING the wedged id server-side after the clear —
|
|
2554
|
+
* and `ensureSession`'s persisted-id fallback then reuses it, defeating the
|
|
2555
|
+
* self-heal. This map makes the runner authoritative instead of racing those
|
|
2556
|
+
* writes: *`ensureSession` never reuses an abandoned id for that conversation,
|
|
2557
|
+
* whatever the server row says* — which holds even when the resurrecting write
|
|
2558
|
+
* is one we deliberately keep (see `markDone`).
|
|
2559
|
+
*
|
|
2560
|
+
* Bounded by construction, on both axes: keyed by CONVERSATION, so N failures on
|
|
2561
|
+
* one conversation hold ONE entry (the newest abandonment replaces the older), and
|
|
2562
|
+
* hard-capped at `MAX_SUPERSEDED_CONVERSATIONS` with FIFO eviction. Only the
|
|
2563
|
+
* NEWEST abandoned id per conversation is guarded: after a second abandonment a
|
|
2564
|
+
* late sibling of the FIRST session can write that id back and `ensureSession`
|
|
2565
|
+
* will reuse it — costing ONE repeat failure, which re-supersedes it. Deliberately
|
|
2566
|
+
* NOT dropped when the session's watcher tears down: `markDone` still writes the
|
|
2567
|
+
* abandoned id back (it must, or the reply is lost), so the guard has to outlive
|
|
2568
|
+
* the turn that resurrects it. In-memory only — a restart forgets it, at the same
|
|
2569
|
+
* bounded cost.
|
|
2570
|
+
*/
|
|
2571
|
+
supersededSessions = /* @__PURE__ */ new Map();
|
|
2082
2572
|
/**
|
|
2083
2573
|
* Per-opencode-session dispatch lock (Task 2.1a). `sendPromptAsync` is no
|
|
2084
2574
|
* longer idempotent (no caller-supplied `messageID`), and its read-back picks
|
|
@@ -2201,6 +2691,24 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
2201
2691
|
sessionTitles = /* @__PURE__ */ new Map();
|
|
2202
2692
|
/** Serialises drains so a reconnect during a drain doesn't double-process. */
|
|
2203
2693
|
draining = false;
|
|
2694
|
+
/**
|
|
2695
|
+
* Serialises runner-file syncs (#559) so the ~2s poll tick and a concurrent
|
|
2696
|
+
* drain ping don't download, write and ack the same file twice.
|
|
2697
|
+
*/
|
|
2698
|
+
syncingFiles = false;
|
|
2699
|
+
/**
|
|
2700
|
+
* Consecutive failed acks per pending file (#559). Lives on the driver so it
|
|
2701
|
+
* survives across drains — without it, a file whose ack keeps failing is
|
|
2702
|
+
* re-downloaded and re-written every ~2s until the server expires it.
|
|
2703
|
+
*/
|
|
2704
|
+
fileAckFailures = /* @__PURE__ */ new Map();
|
|
2705
|
+
/**
|
|
2706
|
+
* Monotonic count of files this runner has pulled and written (#559). Only
|
|
2707
|
+
* ever increases, so `run.ts` detects work by comparing it against the value
|
|
2708
|
+
* it saw on the previous cycle — including work that landed mid-sleep, the
|
|
2709
|
+
* same trick `lastProxiedActivityAt` uses.
|
|
2710
|
+
*/
|
|
2711
|
+
appliedFileCount = 0;
|
|
2204
2712
|
/**
|
|
2205
2713
|
* The currently-executing `drainPending()` promise, or null when idle. Lets a
|
|
2206
2714
|
* graceful shutdown (`waitForInFlight`) await an in-progress drain so a turn it
|
|
@@ -2231,6 +2739,8 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
2231
2739
|
this.pausedMaxWaitMs = config2.pausedMaxWaitMs ?? DEFAULT_PAUSED_MAX_WAIT_MS;
|
|
2232
2740
|
this.stuckQueuedMs = config2.stuckQueuedMs ?? DEFAULT_STUCK_QUEUED_MS;
|
|
2233
2741
|
this.now = config2.now ?? (() => Date.now());
|
|
2742
|
+
this.fileSyncDirectories = config2.fileSyncDirectories ?? [];
|
|
2743
|
+
this.homeDir = config2.homeDir ?? homedir();
|
|
2234
2744
|
}
|
|
2235
2745
|
/** The IPv4-loopback base URL for the local `opencode serve`. */
|
|
2236
2746
|
get opencodeBase() {
|
|
@@ -2258,6 +2768,47 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
2258
2768
|
);
|
|
2259
2769
|
return run2;
|
|
2260
2770
|
}
|
|
2771
|
+
/**
|
|
2772
|
+
* Pull-and-apply any files Evident has queued for this runner (#559), riding
|
|
2773
|
+
* the EXISTING drain cycle — `run.ts` calls it from the same ~2s channel poll
|
|
2774
|
+
* and drain ping that call `drainPending()`. There is deliberately no channel,
|
|
2775
|
+
* control frame or poll loop of its own: worst-case latency is one poll tick.
|
|
2776
|
+
*
|
|
2777
|
+
* NEVER throws and never surfaces a `ChannelAuthError`: a file failure must not
|
|
2778
|
+
* cost a conversation turn. Failures are logged and either acked as a terminal
|
|
2779
|
+
* outcome or left pending for the next drain (see `runner-file-sync.ts`).
|
|
2780
|
+
*
|
|
2781
|
+
* Re-entrant calls are skipped (the poll tick and a drain ping can overlap).
|
|
2782
|
+
*
|
|
2783
|
+
* @returns the number of files written to disk.
|
|
2784
|
+
*/
|
|
2785
|
+
async syncPendingFiles() {
|
|
2786
|
+
if (this.stopped) return 0;
|
|
2787
|
+
if (this.syncingFiles) return 0;
|
|
2788
|
+
this.syncingFiles = true;
|
|
2789
|
+
try {
|
|
2790
|
+
const applied = await syncPendingRunnerFiles({
|
|
2791
|
+
agentId: this.agentId,
|
|
2792
|
+
apiUrl: this.apiUrl,
|
|
2793
|
+
getAuthHeader: this.getAuthHeader,
|
|
2794
|
+
fetchImpl: this.fetchImpl,
|
|
2795
|
+
allowedDirectories: this.fileSyncDirectories,
|
|
2796
|
+
homeDir: this.homeDir,
|
|
2797
|
+
ackFailures: this.fileAckFailures,
|
|
2798
|
+
log: this.log
|
|
2799
|
+
});
|
|
2800
|
+
this.appliedFileCount += applied;
|
|
2801
|
+
return applied;
|
|
2802
|
+
} catch (err) {
|
|
2803
|
+
this.log({
|
|
2804
|
+
level: "error",
|
|
2805
|
+
message: `Runner file sync failed unexpectedly (message processing is unaffected): ${err instanceof Error ? err.message : String(err)}`
|
|
2806
|
+
});
|
|
2807
|
+
return 0;
|
|
2808
|
+
} finally {
|
|
2809
|
+
this.syncingFiles = false;
|
|
2810
|
+
}
|
|
2811
|
+
}
|
|
2261
2812
|
async runDrain() {
|
|
2262
2813
|
let dispatched = 0;
|
|
2263
2814
|
try {
|
|
@@ -2291,6 +2842,28 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
2291
2842
|
}
|
|
2292
2843
|
return false;
|
|
2293
2844
|
}
|
|
2845
|
+
/**
|
|
2846
|
+
* File-pull work, for `run.ts`'s idle accounting (#559).
|
|
2847
|
+
*
|
|
2848
|
+
* Pulling a file is real work that `drainPending()` knows nothing about, so
|
|
2849
|
+
* without this a near-idle runner counts a credential pull as an empty tick
|
|
2850
|
+
* and `--idle-timeout` can `process.exit` mid-pull — leaving a
|
|
2851
|
+
* `.evident-push-*.tmp` behind — or immediately after the write, before the
|
|
2852
|
+
* browser has run the authorize/callback that activates it (the user then sees
|
|
2853
|
+
* `saved_not_activated` for a runner that was fine).
|
|
2854
|
+
*
|
|
2855
|
+
* Two signals because one cannot cover both cases: `inFlight` is the pull
|
|
2856
|
+
* happening RIGHT NOW (it may outlive the tick that started it), and
|
|
2857
|
+
* `appliedFiles` is monotonic so a pull that started AND finished between two
|
|
2858
|
+
* idle checks still shows up as an advance.
|
|
2859
|
+
*
|
|
2860
|
+
* CALLER CONTRACT: sample `inFlight` BEFORE calling `syncPendingFiles()` for
|
|
2861
|
+
* the cycle. `syncPendingFiles` sets the flag synchronously, so a caller that
|
|
2862
|
+
* samples afterwards reads `true` every single cycle and can never idle out.
|
|
2863
|
+
*/
|
|
2864
|
+
fileSyncActivity() {
|
|
2865
|
+
return { appliedFiles: this.appliedFileCount, inFlight: this.syncingFiles };
|
|
2866
|
+
}
|
|
2294
2867
|
/**
|
|
2295
2868
|
* OpenCode session ids the session-cleanup sweep (issue #190) must NOT delete:
|
|
2296
2869
|
* exactly those with a live (dispatched-but-not-done / paused) turn, i.e. a
|
|
@@ -2352,7 +2925,7 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
2352
2925
|
await this.sleep(step);
|
|
2353
2926
|
}
|
|
2354
2927
|
}
|
|
2355
|
-
while (this.hasInFlightWatchers()) {
|
|
2928
|
+
while (this.hasInFlightWatchers() || this.syncingFiles) {
|
|
2356
2929
|
if (this.now() >= deadline) return false;
|
|
2357
2930
|
await this.sleep(step);
|
|
2358
2931
|
}
|
|
@@ -2387,10 +2960,15 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
2387
2960
|
* @returns the count of messages NEWLY dispatched (not already in-flight).
|
|
2388
2961
|
*/
|
|
2389
2962
|
async processConversation(conv) {
|
|
2390
|
-
const sessionId = await this.ensureSession(conv);
|
|
2963
|
+
const { sessionId, refusedSessionId } = await this.ensureSession(conv);
|
|
2391
2964
|
const messages = await this.getPendingMessages(conv.id);
|
|
2392
2965
|
let dispatched = 0;
|
|
2393
2966
|
let skippedAlreadyDispatched = 0;
|
|
2967
|
+
if (refusedSessionId && messages.length > 0) {
|
|
2968
|
+
void this.postSignal(conv.id, messages[0].id, "session_superseded", {
|
|
2969
|
+
superseded_session_id: refusedSessionId
|
|
2970
|
+
});
|
|
2971
|
+
}
|
|
2394
2972
|
for (const message of messages) {
|
|
2395
2973
|
if (this.stopped) break;
|
|
2396
2974
|
if (this.dispatched.has(message.id)) {
|
|
@@ -2417,7 +2995,8 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
2417
2995
|
} catch (err) {
|
|
2418
2996
|
if (err instanceof ChannelAuthError) throw err;
|
|
2419
2997
|
this.dispatched.delete(message.id);
|
|
2420
|
-
|
|
2998
|
+
const exists = await sessionExists(this.port, sessionId);
|
|
2999
|
+
if (exists === false) {
|
|
2421
3000
|
this.sessions.delete(conv.id);
|
|
2422
3001
|
this.log({
|
|
2423
3002
|
level: "warn",
|
|
@@ -2427,15 +3006,39 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
2427
3006
|
});
|
|
2428
3007
|
break;
|
|
2429
3008
|
}
|
|
2430
|
-
|
|
3009
|
+
if (exists === null) {
|
|
3010
|
+
this.log({
|
|
3011
|
+
level: "warn",
|
|
3012
|
+
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.`,
|
|
3013
|
+
conversation_id: conv.id,
|
|
3014
|
+
message_id: message.id
|
|
3015
|
+
});
|
|
3016
|
+
break;
|
|
3017
|
+
}
|
|
3018
|
+
const errorMessage = err instanceof Error ? err.message : String(err);
|
|
3019
|
+
this.sessions.delete(conv.id);
|
|
3020
|
+
this.supersede(conv.id, sessionId);
|
|
3021
|
+
this.log({
|
|
3022
|
+
level: "warn",
|
|
3023
|
+
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.`,
|
|
3024
|
+
conversation_id: conv.id,
|
|
3025
|
+
message_id: message.id
|
|
3026
|
+
});
|
|
3027
|
+
await this.markFailed(conv.id, message.id, null, errorMessage).catch((markErr) => {
|
|
3028
|
+
this.log({
|
|
3029
|
+
level: "warn",
|
|
3030
|
+
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)}`,
|
|
3031
|
+
conversation_id: conv.id,
|
|
3032
|
+
message_id: message.id
|
|
3033
|
+
});
|
|
2431
3034
|
});
|
|
2432
3035
|
this.log({
|
|
2433
3036
|
level: "error",
|
|
2434
|
-
message: `Message ${message.id.slice(0, 8)} dispatch failed: ${
|
|
3037
|
+
message: `Message ${message.id.slice(0, 8)} dispatch failed: ${errorMessage}`,
|
|
2435
3038
|
conversation_id: conv.id,
|
|
2436
3039
|
message_id: message.id
|
|
2437
3040
|
});
|
|
2438
|
-
|
|
3041
|
+
break;
|
|
2439
3042
|
}
|
|
2440
3043
|
if (opencodeMessageId === null) {
|
|
2441
3044
|
this.log({
|
|
@@ -2461,8 +3064,42 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
2461
3064
|
this.ensureWatcherRunning(sessionId);
|
|
2462
3065
|
return dispatched;
|
|
2463
3066
|
}
|
|
3067
|
+
/**
|
|
3068
|
+
* Record that `sessionId` is no longer a valid binding for `conversationId`
|
|
3069
|
+
* (#553). Keyed by conversation and hard-capped, so it cannot grow with the
|
|
3070
|
+
* number of failures — see the `supersededSessions` field doc.
|
|
3071
|
+
*/
|
|
3072
|
+
supersede(conversationId, sessionId) {
|
|
3073
|
+
this.supersededSessions.delete(conversationId);
|
|
3074
|
+
this.supersededSessions.set(conversationId, sessionId);
|
|
3075
|
+
while (this.supersededSessions.size > MAX_SUPERSEDED_CONVERSATIONS) {
|
|
3076
|
+
const oldest = this.supersededSessions.keys().next().value;
|
|
3077
|
+
if (oldest === void 0) return;
|
|
3078
|
+
this.supersededSessions.delete(oldest);
|
|
3079
|
+
}
|
|
3080
|
+
}
|
|
3081
|
+
/** Whether `sessionId` is the session this conversation has abandoned (#553). */
|
|
3082
|
+
isSuperseded(conversationId, sessionId) {
|
|
3083
|
+
return this.supersededSessions.get(conversationId) === sessionId;
|
|
3084
|
+
}
|
|
3085
|
+
/**
|
|
3086
|
+
* Resolve the opencode session to run this conversation's turns in.
|
|
3087
|
+
*
|
|
3088
|
+
* `refusedSessionId` is set when the #553 guard fired — i.e. the persisted
|
|
3089
|
+
* binding was an id this runner had abandoned, so a resurrection genuinely
|
|
3090
|
+
* happened and a fresh session was bound instead. The caller reports it.
|
|
3091
|
+
*/
|
|
2464
3092
|
async ensureSession(conv) {
|
|
2465
3093
|
const bound = this.sessions.get(conv.id) ?? conv.opencode_session_id ?? null;
|
|
3094
|
+
if (bound && this.isSuperseded(conv.id, bound)) {
|
|
3095
|
+
this.log({
|
|
3096
|
+
level: "warn",
|
|
3097
|
+
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.`,
|
|
3098
|
+
conversation_id: conv.id
|
|
3099
|
+
});
|
|
3100
|
+
this.sessions.delete(conv.id);
|
|
3101
|
+
return { sessionId: await this.createAndBindSession(conv.id), refusedSessionId: bound };
|
|
3102
|
+
}
|
|
2466
3103
|
if (bound) {
|
|
2467
3104
|
const exists = await sessionExists(this.port, bound);
|
|
2468
3105
|
if (exists === false) {
|
|
@@ -2472,12 +3109,12 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
2472
3109
|
conversation_id: conv.id
|
|
2473
3110
|
});
|
|
2474
3111
|
this.sessions.delete(conv.id);
|
|
2475
|
-
return this.createAndBindSession(conv.id);
|
|
3112
|
+
return { sessionId: await this.createAndBindSession(conv.id) };
|
|
2476
3113
|
}
|
|
2477
3114
|
this.sessions.set(conv.id, bound);
|
|
2478
|
-
return bound;
|
|
3115
|
+
return { sessionId: bound };
|
|
2479
3116
|
}
|
|
2480
|
-
return this.createAndBindSession(conv.id);
|
|
3117
|
+
return { sessionId: await this.createAndBindSession(conv.id) };
|
|
2481
3118
|
}
|
|
2482
3119
|
/**
|
|
2483
3120
|
* Create a fresh OpenCode session for a conversation, cache the binding, and
|
|
@@ -2565,7 +3202,11 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
2565
3202
|
* (not-owned / out-of-range / deleted-at-source / workspace gone) / 413
|
|
2566
3203
|
* (over-cap). On ANY non-2xx or thrown failure we return `null` so the caller
|
|
2567
3204
|
* OMITS that one image and the text turn still sends — NEVER throws the turn.
|
|
2568
|
-
*
|
|
3205
|
+
* A 404 body carrying `{ reason: 'needs_reauth' }` (#547 — the server CONFIRMED
|
|
3206
|
+
* a Slack `files:read` scope problem via `files.info`) instead resolves the
|
|
3207
|
+
* `AttachmentFetchNeedsReauth` sentinel, so the in-thread note can steer the
|
|
3208
|
+
* user to reconnect Slack instead of a generic "unavailable". Failures are
|
|
3209
|
+
* logged with context (no silent swallow).
|
|
2569
3210
|
*/
|
|
2570
3211
|
async fetchAttachmentDataUrl(messageId, index, mime) {
|
|
2571
3212
|
try {
|
|
@@ -2574,6 +3215,25 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
2574
3215
|
{ headers: { Authorization: this.getAuthHeader() } }
|
|
2575
3216
|
);
|
|
2576
3217
|
if (!res.ok) {
|
|
3218
|
+
let reason;
|
|
3219
|
+
try {
|
|
3220
|
+
const body = await res.json();
|
|
3221
|
+
if (body && typeof body.reason === "string") reason = body.reason;
|
|
3222
|
+
} catch (parseErr) {
|
|
3223
|
+
this.log({
|
|
3224
|
+
level: "debug",
|
|
3225
|
+
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`,
|
|
3226
|
+
message_id: messageId
|
|
3227
|
+
});
|
|
3228
|
+
}
|
|
3229
|
+
if (reason === "needs_reauth") {
|
|
3230
|
+
this.log({
|
|
3231
|
+
level: "error",
|
|
3232
|
+
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)`,
|
|
3233
|
+
message_id: messageId
|
|
3234
|
+
});
|
|
3235
|
+
return { needsReauth: true };
|
|
3236
|
+
}
|
|
2577
3237
|
this.log({
|
|
2578
3238
|
level: "error",
|
|
2579
3239
|
message: `Attachment fetch for message ${messageId.slice(0, 8)} index ${index} returned HTTP ${res.status} \u2014 omitting this image (text turn proceeds)`,
|
|
@@ -2613,6 +3273,9 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
2613
3273
|
if (this.attachmentsSkippedSignalled.has(messageId)) return;
|
|
2614
3274
|
this.attachmentsSkippedSignalled.add(messageId);
|
|
2615
3275
|
const skippedReason = capabilityUnknown ? "unknown" : "unsupported";
|
|
3276
|
+
const failedReason = outcomes.some(
|
|
3277
|
+
(o) => o.status === "failed" && o.reason === "needs_reauth"
|
|
3278
|
+
) ? "needs_reauth" : void 0;
|
|
2616
3279
|
this.log({
|
|
2617
3280
|
level: "info",
|
|
2618
3281
|
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`,
|
|
@@ -2622,7 +3285,8 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
2622
3285
|
void this.postSignal(conversationId, messageId, "attachments_skipped", {
|
|
2623
3286
|
skipped,
|
|
2624
3287
|
failed,
|
|
2625
|
-
...skipped > 0 ? { skipped_reason: skippedReason } : {}
|
|
3288
|
+
...skipped > 0 ? { skipped_reason: skippedReason } : {},
|
|
3289
|
+
...failedReason ? { failed_reason: failedReason } : {}
|
|
2626
3290
|
});
|
|
2627
3291
|
}
|
|
2628
3292
|
/** Register a freshly-dispatched message with its session's watcher state. */
|
|
@@ -3909,6 +4573,32 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
3909
4573
|
}
|
|
3910
4574
|
return messages;
|
|
3911
4575
|
}
|
|
4576
|
+
/**
|
|
4577
|
+
* The `opencode_session_id` fragment of a status PATCH body — `{}` when this
|
|
4578
|
+
* conversation has ABANDONED that session (#553). The field is optional
|
|
4579
|
+
* server-side and an absent one leaves the persisted binding untouched, so
|
|
4580
|
+
* omitting it is how a routine status write stops resurrecting it.
|
|
4581
|
+
*
|
|
4582
|
+
* ONLY for writes whose sole cost is a lost deep link. The `processing` notice
|
|
4583
|
+
* degrades to no "View in Evident" link (the reaction swap still fires) and the
|
|
4584
|
+
* turn-failure notice is built from the PATCH's own `error` text with a link off
|
|
4585
|
+
* the persisted row — neither loses content the user came for. `markDone`
|
|
4586
|
+
* deliberately does NOT use this helper: the server fetches the reply text
|
|
4587
|
+
* THROUGH the session id it is given, so suppressing there would replace the
|
|
4588
|
+
* agent's answer with a bare "✅ Done!" (the #183/#187 failure). The
|
|
4589
|
+
* `ensureSession` guard, not this suppression, is what makes the self-heal
|
|
4590
|
+
* stick.
|
|
4591
|
+
*/
|
|
4592
|
+
sessionIdBody(sessionId, conversationId, messageId, status) {
|
|
4593
|
+
if (!this.isSuperseded(conversationId, sessionId)) return { opencode_session_id: sessionId };
|
|
4594
|
+
this.log({
|
|
4595
|
+
level: "debug",
|
|
4596
|
+
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)}`,
|
|
4597
|
+
conversation_id: conversationId,
|
|
4598
|
+
message_id: messageId
|
|
4599
|
+
});
|
|
4600
|
+
return {};
|
|
4601
|
+
}
|
|
3912
4602
|
/**
|
|
3913
4603
|
* EXISTING combinedAuth route — now fired by the watcher on queued→running
|
|
3914
4604
|
* (Task 3.3), NOT at dispatch/claim time. `{status:'processing',
|
|
@@ -3938,7 +4628,7 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
3938
4628
|
headers: { Authorization: this.getAuthHeader(), "Content-Type": "application/json" },
|
|
3939
4629
|
body: JSON.stringify({
|
|
3940
4630
|
status: "processing",
|
|
3941
|
-
|
|
4631
|
+
...this.sessionIdBody(sessionId, conversationId, messageId, "processing"),
|
|
3942
4632
|
...opencodeMessageId ? { opencode_message_id: opencodeMessageId } : {},
|
|
3943
4633
|
...title ? { title } : {}
|
|
3944
4634
|
})
|
|
@@ -3987,6 +4677,11 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
3987
4677
|
headers: { Authorization: this.getAuthHeader(), "Content-Type": "application/json" },
|
|
3988
4678
|
body: JSON.stringify({
|
|
3989
4679
|
status: "done",
|
|
4680
|
+
// ALWAYS sent, even for a session this conversation has abandoned
|
|
4681
|
+
// (#553): the server reads the reply text back out of THIS session id
|
|
4682
|
+
// to deliver it. Omitting it would leave the user with "✅ Done!"
|
|
4683
|
+
// instead of the answer — a worse regression than the resurrection it
|
|
4684
|
+
// would prevent, which `ensureSession`'s guard handles anyway.
|
|
3990
4685
|
opencode_session_id: sessionId,
|
|
3991
4686
|
...opencodeMessageId ? { opencode_message_id: opencodeMessageId } : {},
|
|
3992
4687
|
...title ? { title } : {},
|
|
@@ -4003,14 +4698,23 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
4003
4698
|
}
|
|
4004
4699
|
/**
|
|
4005
4700
|
* Mark a message `failed`. `sessionId` / `error` are threaded to the API ONLY
|
|
4006
|
-
* when provided (issue #182)
|
|
4007
|
-
* `
|
|
4008
|
-
*
|
|
4009
|
-
*
|
|
4701
|
+
* when provided (issue #182). Three states for `sessionId`:
|
|
4702
|
+
* - omitted (`undefined`) → don't send the field, leave the persisted
|
|
4703
|
+
* session untouched (unused today; kept for API symmetry).
|
|
4704
|
+
* - a real id (`string`) → send it, update the persisted session (the
|
|
4705
|
+
* turn-failure call sites: an errored OpenCode turn).
|
|
4706
|
+
* - explicit `null` → send it, CLEAR the persisted session (issue
|
|
4707
|
+
* #485's dispatch-handoff-failure call site: the session id still
|
|
4708
|
+
* exists but is wedged, so the next attempt must get a fresh one
|
|
4709
|
+
* instead of reusing it — see WI-1's server-side null-clearing PATCH).
|
|
4010
4710
|
*/
|
|
4011
4711
|
async markFailed(conversationId, messageId, sessionId, error2, usage) {
|
|
4012
4712
|
const body = { status: "failed" };
|
|
4013
|
-
if (sessionId
|
|
4713
|
+
if (sessionId === null) {
|
|
4714
|
+
body.opencode_session_id = null;
|
|
4715
|
+
} else if (sessionId !== void 0) {
|
|
4716
|
+
Object.assign(body, this.sessionIdBody(sessionId, conversationId, messageId, "failed"));
|
|
4717
|
+
}
|
|
4014
4718
|
if (error2 !== void 0) body.error = error2;
|
|
4015
4719
|
if (usage) Object.assign(body, usage);
|
|
4016
4720
|
await this.callWithRetry(
|
|
@@ -4427,6 +5131,34 @@ function resolveLogLevel(options) {
|
|
|
4427
5131
|
}
|
|
4428
5132
|
return "info";
|
|
4429
5133
|
}
|
|
5134
|
+
function resolveFileSyncDirectories(raw, homeDir) {
|
|
5135
|
+
const directories = [];
|
|
5136
|
+
for (const entry of raw ?? []) {
|
|
5137
|
+
const trimmed = entry.trim();
|
|
5138
|
+
if (trimmed === "") {
|
|
5139
|
+
throw new Error("--enable-file-sync-to requires a directory path (got an empty value)");
|
|
5140
|
+
}
|
|
5141
|
+
const expanded = trimmed === "~" ? homeDir : trimmed.startsWith("~/") ? join2(homeDir, trimmed.slice(2)) : trimmed;
|
|
5142
|
+
if (!isAbsolute2(expanded)) {
|
|
5143
|
+
throw new Error(`--enable-file-sync-to requires an absolute directory path; got "${entry}"`);
|
|
5144
|
+
}
|
|
5145
|
+
const normalized = resolvePath(expanded);
|
|
5146
|
+
if (parse(normalized).root === normalized) {
|
|
5147
|
+
throw new Error(
|
|
5148
|
+
`--enable-file-sync-to will not allow-list the filesystem root ("${entry}"); name the specific directory the credentials belong in (for example ~/.claude)`
|
|
5149
|
+
);
|
|
5150
|
+
}
|
|
5151
|
+
if (!directories.includes(normalized)) {
|
|
5152
|
+
directories.push(normalized);
|
|
5153
|
+
}
|
|
5154
|
+
}
|
|
5155
|
+
if (directories.length > MAX_FILE_SYNC_DIRECTORIES) {
|
|
5156
|
+
throw new Error(
|
|
5157
|
+
`--enable-file-sync-to accepts at most ${MAX_FILE_SYNC_DIRECTORIES} directories; got ${directories.length}`
|
|
5158
|
+
);
|
|
5159
|
+
}
|
|
5160
|
+
return directories;
|
|
5161
|
+
}
|
|
4430
5162
|
function meetsThreshold(state, level) {
|
|
4431
5163
|
return LOG_LEVELS[level] >= LOG_LEVELS[state.logLevel];
|
|
4432
5164
|
}
|
|
@@ -4548,18 +5280,29 @@ async function handleAuthError(state, error2) {
|
|
|
4548
5280
|
async function driveChannels(state, driver) {
|
|
4549
5281
|
let idlePolls = 0;
|
|
4550
5282
|
let lastSeenProxiedActivityAt = state.lastProxiedActivityAt;
|
|
5283
|
+
let lastSeenAppliedFiles = driver.fileSyncActivity().appliedFiles;
|
|
4551
5284
|
while (state.running) {
|
|
4552
5285
|
if (state.connection?.reconnecting && state.connection.reconnectPromise) {
|
|
4553
5286
|
logActivity(state, { type: "info", message: "Waiting for tunnel reconnection..." });
|
|
4554
5287
|
if (state.interactive) displayStatus(state);
|
|
4555
5288
|
await state.connection.reconnectPromise;
|
|
4556
5289
|
}
|
|
5290
|
+
const carriedOverFileSync = driver.fileSyncActivity().inFlight;
|
|
5291
|
+
void driver.syncPendingFiles().catch(
|
|
5292
|
+
(error2) => logActivity(state, {
|
|
5293
|
+
type: "error",
|
|
5294
|
+
error: `Runner file sync failed: ${error2 instanceof Error ? error2.message : String(error2)}`
|
|
5295
|
+
})
|
|
5296
|
+
);
|
|
4557
5297
|
try {
|
|
4558
5298
|
const processed = await driver.drainPending();
|
|
4559
5299
|
state.messageCount += processed;
|
|
4560
5300
|
const proxiedActivity = state.lastProxiedActivityAt !== lastSeenProxiedActivityAt;
|
|
4561
5301
|
lastSeenProxiedActivityAt = state.lastProxiedActivityAt;
|
|
4562
|
-
|
|
5302
|
+
const appliedFiles = driver.fileSyncActivity().appliedFiles;
|
|
5303
|
+
const fileActivity = carriedOverFileSync || appliedFiles !== lastSeenAppliedFiles;
|
|
5304
|
+
lastSeenAppliedFiles = appliedFiles;
|
|
5305
|
+
if (processed > 0 || driver.hasInFlightWatchers() || proxiedActivity || fileActivity) {
|
|
4563
5306
|
idlePolls = 0;
|
|
4564
5307
|
if (processed > 0 && state.interactive) displayStatus(state);
|
|
4565
5308
|
} else if (state.idleTimeout !== null) {
|
|
@@ -4588,7 +5331,7 @@ async function driveChannels(state, driver) {
|
|
|
4588
5331
|
logActivity(state, { type: "error", error: `Channel processing error: ${errorMessage}` });
|
|
4589
5332
|
if (state.interactive) displayStatus(state);
|
|
4590
5333
|
}
|
|
4591
|
-
await new Promise((
|
|
5334
|
+
await new Promise((resolve3) => setTimeout(resolve3, CHANNEL_POLL_INTERVAL_MS));
|
|
4592
5335
|
if (state.idleTimeout !== null && idlePolls >= 2) {
|
|
4593
5336
|
const idleMs = idlePolls * CHANNEL_POLL_INTERVAL_MS;
|
|
4594
5337
|
if (idleMs > state.idleTimeout * 1e3) {
|
|
@@ -4733,8 +5476,10 @@ async function cleanup(state, opts = {}) {
|
|
|
4733
5476
|
async function run(options) {
|
|
4734
5477
|
const interactive = isInteractive(options.json);
|
|
4735
5478
|
let logLevel;
|
|
5479
|
+
let fileSyncDirectories;
|
|
4736
5480
|
try {
|
|
4737
5481
|
logLevel = resolveLogLevel(options);
|
|
5482
|
+
fileSyncDirectories = resolveFileSyncDirectories(options.enableFileSyncTo, homedir2());
|
|
4738
5483
|
} catch (error2) {
|
|
4739
5484
|
const message = error2 instanceof Error ? error2.message : String(error2);
|
|
4740
5485
|
if (options.json) {
|
|
@@ -4769,6 +5514,11 @@ async function run(options) {
|
|
|
4769
5514
|
sessionCleanupTimers: [],
|
|
4770
5515
|
authHeader: ""
|
|
4771
5516
|
};
|
|
5517
|
+
if (fileSyncDirectories.length > 0) {
|
|
5518
|
+
log2(state, `File sync enabled for: ${fileSyncDirectories.join(", ")}`);
|
|
5519
|
+
} else {
|
|
5520
|
+
log2(state, "File sync is disabled (no --enable-file-sync-to given)", "debug");
|
|
5521
|
+
}
|
|
4772
5522
|
if (!options.runner && options.agent) {
|
|
4773
5523
|
telemetry.info(
|
|
4774
5524
|
EventTypes.DEPRECATED_AGENT_FLAG_USED,
|
|
@@ -4961,6 +5711,10 @@ async function run(options) {
|
|
|
4961
5711
|
getAuthHeader: () => state.authHeader,
|
|
4962
5712
|
conversationFilter: state.conversationFilter,
|
|
4963
5713
|
stuckQueuedMs: CHANNEL_STUCK_QUEUED_MS,
|
|
5714
|
+
// #559: the `--enable-file-sync-to` allow-list. Empty ⇒ pending files are
|
|
5715
|
+
// REJECTED with `file_sync_disabled` on the ack, not silently ignored.
|
|
5716
|
+
fileSyncDirectories,
|
|
5717
|
+
homeDir: homedir2(),
|
|
4964
5718
|
log: (entry) => (
|
|
4965
5719
|
// Thread the driver's real level straight through so `debug`/`warn`
|
|
4966
5720
|
// survive the sink filter (they no longer collapse to info). `type`
|
|
@@ -5042,6 +5796,12 @@ async function run(options) {
|
|
|
5042
5796
|
onDrainPing: () => {
|
|
5043
5797
|
if (!state.running) return;
|
|
5044
5798
|
logActivity(state, { type: "info", message: "Drain ping received \u2014 draining" });
|
|
5799
|
+
void channelDriver.syncPendingFiles().catch(
|
|
5800
|
+
(error2) => logActivity(state, {
|
|
5801
|
+
type: "error",
|
|
5802
|
+
error: `Runner file sync failed on ping: ${error2 instanceof Error ? error2.message : String(error2)}`
|
|
5803
|
+
})
|
|
5804
|
+
);
|
|
5045
5805
|
channelDriver.drainPending().then((processed) => {
|
|
5046
5806
|
if (processed > 0) {
|
|
5047
5807
|
state.messageCount += processed;
|
|
@@ -5125,7 +5885,10 @@ program.name("evident").description("Run OpenCode locally and connect it to Evid
|
|
|
5125
5885
|
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);
|
|
5126
5886
|
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 }));
|
|
5127
5887
|
program.command("whoami").description("Show the currently logged in user").action(whoami);
|
|
5128
|
-
program.command("run").description("Connect to Evident and process messages").option("
|
|
5888
|
+
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(
|
|
5889
|
+
"-a, --agent [id]",
|
|
5890
|
+
"Deprecated alias for --runner (still supported; --runner wins if both are given)"
|
|
5891
|
+
).option("-p, --port <port>", "OpenCode port (default: 4096)", "4096").option(
|
|
5129
5892
|
"--log-level <level>",
|
|
5130
5893
|
"Log verbosity: debug | info | warn | error (default: info). Env: EVIDENT_LOG_LEVEL"
|
|
5131
5894
|
).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("--json", "Output in JSON format").option(
|
|
@@ -5137,6 +5900,11 @@ program.command("run").description("Connect to Evident and process messages").op
|
|
|
5137
5900
|
).option(
|
|
5138
5901
|
"--session-cleanup-interval <duration>",
|
|
5139
5902
|
"How often the cleanup sweep runs (default: 1h). Env: EVIDENT_SESSION_CLEANUP_INTERVAL"
|
|
5903
|
+
).option(
|
|
5904
|
+
"--enable-file-sync-to <dir>",
|
|
5905
|
+
"Allow Evident to write files into this directory (repeatable). Omit to disable file sync entirely.",
|
|
5906
|
+
(value, previous) => previous.concat([value]),
|
|
5907
|
+
[]
|
|
5140
5908
|
).action(
|
|
5141
5909
|
(options) => {
|
|
5142
5910
|
run({
|
|
@@ -5153,7 +5921,10 @@ program.command("run").description("Connect to Evident and process messages").op
|
|
|
5153
5921
|
// Raw strings — the resolver in run.ts single-sources parsing (M1).
|
|
5154
5922
|
sessionCleanupMaxAge: options.sessionCleanupMaxAge,
|
|
5155
5923
|
sessionCleanupMaxCount: options.sessionCleanupMaxCount,
|
|
5156
|
-
sessionCleanupInterval: options.sessionCleanupInterval
|
|
5924
|
+
sessionCleanupInterval: options.sessionCleanupInterval,
|
|
5925
|
+
// Raw values — expansion/validation is single-sourced in run.ts's
|
|
5926
|
+
// resolveFileSyncDirectories.
|
|
5927
|
+
enableFileSyncTo: options.enableFileSyncTo
|
|
5157
5928
|
});
|
|
5158
5929
|
}
|
|
5159
5930
|
);
|