@evident-ai/cli 3.1.1-dev.3d68bee → 3.1.1-dev.3ec6e7f
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 +695 -26
- package/dist/index.js.map +1 -1
- package/package.json +1 -1
package/dist/index.js
CHANGED
|
@@ -285,14 +285,14 @@ function blank() {
|
|
|
285
285
|
console.log();
|
|
286
286
|
}
|
|
287
287
|
function waitForEnter(prompt = "Press Enter to continue...") {
|
|
288
|
-
return new Promise((
|
|
288
|
+
return new Promise((resolve3) => {
|
|
289
289
|
process.stdout.write(chalk.dim(prompt));
|
|
290
290
|
const handler = () => {
|
|
291
291
|
process.stdin.removeListener("data", handler);
|
|
292
292
|
process.stdin.setRawMode?.(false);
|
|
293
293
|
process.stdin.pause();
|
|
294
294
|
console.log();
|
|
295
|
-
|
|
295
|
+
resolve3();
|
|
296
296
|
};
|
|
297
297
|
if (process.stdin.isTTY) {
|
|
298
298
|
process.stdin.setRawMode?.(true);
|
|
@@ -302,7 +302,7 @@ function waitForEnter(prompt = "Press Enter to continue...") {
|
|
|
302
302
|
});
|
|
303
303
|
}
|
|
304
304
|
function sleep(ms) {
|
|
305
|
-
return new Promise((
|
|
305
|
+
return new Promise((resolve3) => setTimeout(resolve3, ms));
|
|
306
306
|
}
|
|
307
307
|
|
|
308
308
|
// src/commands/login.ts
|
|
@@ -373,22 +373,23 @@ async function deviceFlowLogin(options) {
|
|
|
373
373
|
}
|
|
374
374
|
async function tokenLogin() {
|
|
375
375
|
console.log("Token login mode.");
|
|
376
|
-
console.log("
|
|
376
|
+
console.log("Run `evident login` on a machine with a browser to get a token.");
|
|
377
|
+
console.log("Manage or revoke existing tokens under Settings \u2192 CLI tokens.");
|
|
377
378
|
blank();
|
|
378
379
|
process.stdout.write("Paste token: ");
|
|
379
|
-
const token = await new Promise((
|
|
380
|
+
const token = await new Promise((resolve3) => {
|
|
380
381
|
let data = "";
|
|
381
382
|
process.stdin.setEncoding("utf8");
|
|
382
383
|
process.stdin.on("data", (chunk) => {
|
|
383
384
|
data += chunk;
|
|
384
385
|
});
|
|
385
386
|
process.stdin.on("end", () => {
|
|
386
|
-
|
|
387
|
+
resolve3(data.trim());
|
|
387
388
|
});
|
|
388
389
|
if (process.stdin.isTTY) {
|
|
389
390
|
process.stdin.once("data", (chunk) => {
|
|
390
391
|
process.stdin.pause();
|
|
391
|
-
|
|
392
|
+
resolve3(chunk.toString().trim());
|
|
392
393
|
});
|
|
393
394
|
process.stdin.resume();
|
|
394
395
|
}
|
|
@@ -467,9 +468,9 @@ async function whoami() {
|
|
|
467
468
|
}
|
|
468
469
|
|
|
469
470
|
// src/commands/run.ts
|
|
471
|
+
import { homedir as homedir3 } from "os";
|
|
472
|
+
import { isAbsolute as isAbsolute2, join as join3, parse, resolve as resolvePath } from "path";
|
|
470
473
|
import chalk6 from "chalk";
|
|
471
|
-
import ora3 from "ora";
|
|
472
|
-
import { select as select3 } from "@inquirer/prompts";
|
|
473
474
|
|
|
474
475
|
// ../../packages/types/src/telemetry/index.ts
|
|
475
476
|
var TelemetryEventTypes = {
|
|
@@ -485,6 +486,10 @@ var TelemetryEventTypes = {
|
|
|
485
486
|
var MAX_FRAME_BYTES = 256 * 1024;
|
|
486
487
|
var TUNNEL_DRAIN_PING_PATH = "/__evident/drain";
|
|
487
488
|
|
|
489
|
+
// ../../packages/types/src/runner-files.ts
|
|
490
|
+
var MAX_FILE_PUSH_BYTES = 64 * 1024;
|
|
491
|
+
var MAX_FILE_SYNC_DIRECTORIES = 16;
|
|
492
|
+
|
|
488
493
|
// ../../packages/types/src/logging/index.ts
|
|
489
494
|
var CORRELATION_ID_HEADER = "x-evident-correlation-id";
|
|
490
495
|
function log(level, event, fields) {
|
|
@@ -514,6 +519,10 @@ function stripQuery(url) {
|
|
|
514
519
|
}
|
|
515
520
|
}
|
|
516
521
|
|
|
522
|
+
// src/commands/run.ts
|
|
523
|
+
import ora3 from "ora";
|
|
524
|
+
import { select as select3 } from "@inquirer/prompts";
|
|
525
|
+
|
|
517
526
|
// src/lib/telemetry.ts
|
|
518
527
|
var CLI_VERSION = (true ? "3.0.0" : void 0) ?? process.env.npm_package_version ?? "unknown";
|
|
519
528
|
function getCliVersion() {
|
|
@@ -725,7 +734,7 @@ async function waitForOpenCodeHealth(port, timeoutMs = 3e4) {
|
|
|
725
734
|
if (health.healthy) {
|
|
726
735
|
return health;
|
|
727
736
|
}
|
|
728
|
-
await new Promise((
|
|
737
|
+
await new Promise((resolve3) => setTimeout(resolve3, 1e3));
|
|
729
738
|
}
|
|
730
739
|
return { healthy: false, error: "Timeout waiting for OpenCode to be healthy" };
|
|
731
740
|
}
|
|
@@ -1364,7 +1373,7 @@ async function sendPromptAsync(port, sessionId, content, options, attachments) {
|
|
|
1364
1373
|
}
|
|
1365
1374
|
}
|
|
1366
1375
|
if (attempt < READ_BACK_ATTEMPTS - 1) {
|
|
1367
|
-
await new Promise((
|
|
1376
|
+
await new Promise((resolve3) => setTimeout(resolve3, READ_BACK_DELAY_MS));
|
|
1368
1377
|
}
|
|
1369
1378
|
}
|
|
1370
1379
|
return null;
|
|
@@ -1738,12 +1747,12 @@ var StreamForwarder = class {
|
|
|
1738
1747
|
let endBody;
|
|
1739
1748
|
if (has_body) {
|
|
1740
1749
|
const chunks = [];
|
|
1741
|
-
bodyPromise = new Promise((
|
|
1750
|
+
bodyPromise = new Promise((resolve3) => {
|
|
1742
1751
|
pushBody = (buf) => {
|
|
1743
1752
|
chunks.push(buf);
|
|
1744
1753
|
};
|
|
1745
1754
|
endBody = () => {
|
|
1746
|
-
|
|
1755
|
+
resolve3(Buffer.concat(chunks));
|
|
1747
1756
|
};
|
|
1748
1757
|
});
|
|
1749
1758
|
}
|
|
@@ -1860,7 +1869,7 @@ function connectTunnel(options) {
|
|
|
1860
1869
|
} = options;
|
|
1861
1870
|
const tunnelUrl = getTunnelUrlConfig();
|
|
1862
1871
|
const url = `${tunnelUrl}/tunnel/${agentId}/connect`;
|
|
1863
|
-
return new Promise((
|
|
1872
|
+
return new Promise((resolve3, reject) => {
|
|
1864
1873
|
const ws = new WebSocket2(url, {
|
|
1865
1874
|
headers: {
|
|
1866
1875
|
Authorization: authHeader
|
|
@@ -1915,7 +1924,7 @@ function connectTunnel(options) {
|
|
|
1915
1924
|
clearTimeout(connectionTimeout);
|
|
1916
1925
|
const connectedAgentId = message.agent_id ?? agentId;
|
|
1917
1926
|
onConnected?.(connectedAgentId);
|
|
1918
|
-
|
|
1927
|
+
resolve3({
|
|
1919
1928
|
ws,
|
|
1920
1929
|
close: () => ws.close(1e3, "CLI shutdown")
|
|
1921
1930
|
});
|
|
@@ -2033,6 +2042,404 @@ var RunnerConnection = class {
|
|
|
2033
2042
|
}
|
|
2034
2043
|
};
|
|
2035
2044
|
|
|
2045
|
+
// src/lib/channels/driver.ts
|
|
2046
|
+
import { homedir as homedir2 } from "os";
|
|
2047
|
+
|
|
2048
|
+
// src/lib/file-push.ts
|
|
2049
|
+
import { randomUUID } from "crypto";
|
|
2050
|
+
import { chmod, mkdir, open as open2, realpath, rename, unlink } from "fs/promises";
|
|
2051
|
+
import { basename, dirname, isAbsolute, join as join2, relative, resolve as resolve2, sep } from "path";
|
|
2052
|
+
var FILE_MODE = 384;
|
|
2053
|
+
var DIRECTORY_MODE = 448;
|
|
2054
|
+
async function writePushedFile(request) {
|
|
2055
|
+
const { requestedPath, content, allowedDirectories, homeDir } = request;
|
|
2056
|
+
const bytes = content.byteLength;
|
|
2057
|
+
if (allowedDirectories.length === 0) {
|
|
2058
|
+
return refuse("file_sync_disabled", "File sync is not enabled on this runner.", {
|
|
2059
|
+
path: requestedPath,
|
|
2060
|
+
bytes
|
|
2061
|
+
});
|
|
2062
|
+
}
|
|
2063
|
+
if (bytes > MAX_FILE_PUSH_BYTES) {
|
|
2064
|
+
return refuse(
|
|
2065
|
+
"file_too_large",
|
|
2066
|
+
`File is ${bytes} bytes; the limit is ${MAX_FILE_PUSH_BYTES}.`,
|
|
2067
|
+
{
|
|
2068
|
+
path: requestedPath,
|
|
2069
|
+
bytes
|
|
2070
|
+
}
|
|
2071
|
+
);
|
|
2072
|
+
}
|
|
2073
|
+
const candidate = expandAndValidate(requestedPath, homeDir);
|
|
2074
|
+
if (candidate === null) {
|
|
2075
|
+
return refuse("invalid_path", "The requested path is not a valid absolute file path.", {
|
|
2076
|
+
path: requestedPath,
|
|
2077
|
+
bytes
|
|
2078
|
+
});
|
|
2079
|
+
}
|
|
2080
|
+
try {
|
|
2081
|
+
const { existingAncestor, missingSegments } = await resolveNearestExistingAncestor(
|
|
2082
|
+
dirname(candidate)
|
|
2083
|
+
);
|
|
2084
|
+
const realTarget = join2(existingAncestor, ...missingSegments, basename(candidate));
|
|
2085
|
+
const allowedDirectory = await findContainingAllowedDirectory(allowedDirectories, realTarget);
|
|
2086
|
+
if (allowedDirectory === null) {
|
|
2087
|
+
return refuse("path_not_allowed", "The runner does not allow writing to that location.", {
|
|
2088
|
+
path: realTarget,
|
|
2089
|
+
bytes
|
|
2090
|
+
});
|
|
2091
|
+
}
|
|
2092
|
+
if (missingSegments.length > 0) {
|
|
2093
|
+
await createMissingDirectories(existingAncestor, missingSegments);
|
|
2094
|
+
const realParent = await realpath(dirname(realTarget));
|
|
2095
|
+
if (realParent !== dirname(realTarget) || !contains(allowedDirectory, realTarget)) {
|
|
2096
|
+
return refuse("path_not_allowed", "The runner does not allow writing to that location.", {
|
|
2097
|
+
path: realTarget,
|
|
2098
|
+
bytes,
|
|
2099
|
+
reason: "parent_changed_after_create"
|
|
2100
|
+
});
|
|
2101
|
+
}
|
|
2102
|
+
}
|
|
2103
|
+
await writeAtomically(realTarget, content);
|
|
2104
|
+
log("info", "file_push_written", { path: realTarget, bytes });
|
|
2105
|
+
return { ok: true, path: realTarget };
|
|
2106
|
+
} catch (err) {
|
|
2107
|
+
const errno = err.code ?? "UNKNOWN";
|
|
2108
|
+
return refuse("write_failed", `The runner could not write the file (${errno}).`, {
|
|
2109
|
+
path: candidate,
|
|
2110
|
+
bytes,
|
|
2111
|
+
errno,
|
|
2112
|
+
...errorFields(err)
|
|
2113
|
+
});
|
|
2114
|
+
}
|
|
2115
|
+
}
|
|
2116
|
+
function expandAndValidate(requestedPath, homeDir) {
|
|
2117
|
+
if (requestedPath.trim() === "" || requestedPath.includes("\0")) {
|
|
2118
|
+
return null;
|
|
2119
|
+
}
|
|
2120
|
+
const expanded = requestedPath === "~" ? homeDir : requestedPath.startsWith("~/") ? join2(homeDir, requestedPath.slice(2)) : requestedPath;
|
|
2121
|
+
if (expanded.split(/[/\\]/).includes("..")) {
|
|
2122
|
+
return null;
|
|
2123
|
+
}
|
|
2124
|
+
if (!isAbsolute(expanded)) {
|
|
2125
|
+
return null;
|
|
2126
|
+
}
|
|
2127
|
+
const candidate = resolve2(expanded);
|
|
2128
|
+
const name = basename(candidate);
|
|
2129
|
+
return name === "" || name === "." || name === ".." ? null : candidate;
|
|
2130
|
+
}
|
|
2131
|
+
async function resolveNearestExistingAncestor(directory) {
|
|
2132
|
+
const missingSegments = [];
|
|
2133
|
+
let current = directory;
|
|
2134
|
+
for (; ; ) {
|
|
2135
|
+
try {
|
|
2136
|
+
return { existingAncestor: await realpath(current), missingSegments };
|
|
2137
|
+
} catch (err) {
|
|
2138
|
+
const parent = dirname(current);
|
|
2139
|
+
if (err.code !== "ENOENT" || parent === current) {
|
|
2140
|
+
throw err;
|
|
2141
|
+
}
|
|
2142
|
+
missingSegments.unshift(basename(current));
|
|
2143
|
+
current = parent;
|
|
2144
|
+
}
|
|
2145
|
+
}
|
|
2146
|
+
}
|
|
2147
|
+
async function findContainingAllowedDirectory(allowedDirectories, realTarget) {
|
|
2148
|
+
for (const directory of allowedDirectories) {
|
|
2149
|
+
if (!isAbsolute(directory)) {
|
|
2150
|
+
log("warn", "file_push_allowed_directory_skipped", { directory, reason: "not_absolute" });
|
|
2151
|
+
continue;
|
|
2152
|
+
}
|
|
2153
|
+
const realDirectory = await realpathCreatingIfMissing(directory);
|
|
2154
|
+
if (realDirectory !== null && contains(realDirectory, realTarget)) {
|
|
2155
|
+
return realDirectory;
|
|
2156
|
+
}
|
|
2157
|
+
}
|
|
2158
|
+
return null;
|
|
2159
|
+
}
|
|
2160
|
+
async function realpathCreatingIfMissing(directory) {
|
|
2161
|
+
try {
|
|
2162
|
+
return await realpath(directory);
|
|
2163
|
+
} catch (err) {
|
|
2164
|
+
if (err.code !== "ENOENT") {
|
|
2165
|
+
log("warn", "file_push_allowed_directory_skipped", {
|
|
2166
|
+
directory,
|
|
2167
|
+
reason: "unresolvable",
|
|
2168
|
+
...errorFields(err)
|
|
2169
|
+
});
|
|
2170
|
+
return null;
|
|
2171
|
+
}
|
|
2172
|
+
}
|
|
2173
|
+
try {
|
|
2174
|
+
await mkdir(directory, { recursive: true, mode: DIRECTORY_MODE });
|
|
2175
|
+
await chmod(directory, DIRECTORY_MODE);
|
|
2176
|
+
return await realpath(directory);
|
|
2177
|
+
} catch (err) {
|
|
2178
|
+
log("warn", "file_push_allowed_directory_skipped", {
|
|
2179
|
+
directory,
|
|
2180
|
+
reason: "create_failed",
|
|
2181
|
+
...errorFields(err)
|
|
2182
|
+
});
|
|
2183
|
+
return null;
|
|
2184
|
+
}
|
|
2185
|
+
}
|
|
2186
|
+
function contains(realDirectory, realTarget) {
|
|
2187
|
+
const rel = relative(realDirectory, realTarget);
|
|
2188
|
+
return rel !== "" && rel !== ".." && !rel.startsWith(`..${sep}`) && !isAbsolute(rel);
|
|
2189
|
+
}
|
|
2190
|
+
async function createMissingDirectories(existingAncestor, missingSegments) {
|
|
2191
|
+
let current = existingAncestor;
|
|
2192
|
+
for (const segment of missingSegments) {
|
|
2193
|
+
current = join2(current, segment);
|
|
2194
|
+
await mkdir(current, { recursive: true, mode: DIRECTORY_MODE });
|
|
2195
|
+
await chmod(current, DIRECTORY_MODE);
|
|
2196
|
+
}
|
|
2197
|
+
}
|
|
2198
|
+
async function writeAtomically(realTarget, content) {
|
|
2199
|
+
const temporaryPath = join2(dirname(realTarget), `.evident-push-${randomUUID()}.tmp`);
|
|
2200
|
+
let handle;
|
|
2201
|
+
try {
|
|
2202
|
+
handle = await open2(temporaryPath, "wx", FILE_MODE);
|
|
2203
|
+
await handle.writeFile(content);
|
|
2204
|
+
await handle.chmod(FILE_MODE);
|
|
2205
|
+
await handle.close();
|
|
2206
|
+
handle = void 0;
|
|
2207
|
+
await rename(temporaryPath, realTarget);
|
|
2208
|
+
} catch (err) {
|
|
2209
|
+
await discardTemporaryFile(temporaryPath, handle);
|
|
2210
|
+
throw err;
|
|
2211
|
+
}
|
|
2212
|
+
}
|
|
2213
|
+
async function discardTemporaryFile(temporaryPath, handle) {
|
|
2214
|
+
try {
|
|
2215
|
+
await handle?.close();
|
|
2216
|
+
} catch (err) {
|
|
2217
|
+
log("warn", "file_push_temp_close_failed", { path: temporaryPath, ...errorFields(err) });
|
|
2218
|
+
}
|
|
2219
|
+
try {
|
|
2220
|
+
await unlink(temporaryPath);
|
|
2221
|
+
} catch (err) {
|
|
2222
|
+
const errno = err.code;
|
|
2223
|
+
if (errno !== "ENOENT" && errno !== "ENOTDIR") {
|
|
2224
|
+
log("warn", "file_push_temp_cleanup_failed", { path: temporaryPath, ...errorFields(err) });
|
|
2225
|
+
}
|
|
2226
|
+
}
|
|
2227
|
+
}
|
|
2228
|
+
function refuse(code, message, fields) {
|
|
2229
|
+
log(code === "write_failed" ? "error" : "warn", "file_push_refused", { code, ...fields });
|
|
2230
|
+
return { ok: false, code, message };
|
|
2231
|
+
}
|
|
2232
|
+
|
|
2233
|
+
// src/lib/runner-file-sync.ts
|
|
2234
|
+
var MAX_ACK_ATTEMPTS = 5;
|
|
2235
|
+
async function syncPendingRunnerFiles(options) {
|
|
2236
|
+
const pending = await listPendingFiles(options);
|
|
2237
|
+
const pendingIds = new Set(pending.map((file) => file.id));
|
|
2238
|
+
for (const id of options.ackFailures.keys()) {
|
|
2239
|
+
if (!pendingIds.has(id)) options.ackFailures.delete(id);
|
|
2240
|
+
}
|
|
2241
|
+
if (pending.length === 0) return 0;
|
|
2242
|
+
options.log({
|
|
2243
|
+
level: "info",
|
|
2244
|
+
message: `Runner file sync: ${pending.length} file(s) queued for this runner`
|
|
2245
|
+
});
|
|
2246
|
+
let applied = 0;
|
|
2247
|
+
for (const file of pending) {
|
|
2248
|
+
if ((options.ackFailures.get(file.id) ?? 0) >= MAX_ACK_ATTEMPTS) continue;
|
|
2249
|
+
if (await applyOne(options, file)) applied += 1;
|
|
2250
|
+
}
|
|
2251
|
+
return applied;
|
|
2252
|
+
}
|
|
2253
|
+
async function listPendingFiles(options) {
|
|
2254
|
+
let res;
|
|
2255
|
+
try {
|
|
2256
|
+
res = await options.fetchImpl(`${options.apiUrl}/runners/${options.agentId}/files/pending`, {
|
|
2257
|
+
headers: { Authorization: options.getAuthHeader() }
|
|
2258
|
+
});
|
|
2259
|
+
} catch (err) {
|
|
2260
|
+
options.log({
|
|
2261
|
+
level: "warn",
|
|
2262
|
+
message: `Could not list pending runner files \u2014 retrying on the next drain: ${describe(err)}`
|
|
2263
|
+
});
|
|
2264
|
+
return [];
|
|
2265
|
+
}
|
|
2266
|
+
if (!res.ok) {
|
|
2267
|
+
options.log({
|
|
2268
|
+
level: res.status === 404 ? "debug" : "warn",
|
|
2269
|
+
message: `Listing pending runner files returned HTTP ${res.status} \u2014 retrying on the next drain`
|
|
2270
|
+
});
|
|
2271
|
+
return [];
|
|
2272
|
+
}
|
|
2273
|
+
let body;
|
|
2274
|
+
try {
|
|
2275
|
+
body = await res.json();
|
|
2276
|
+
} catch (err) {
|
|
2277
|
+
options.log({
|
|
2278
|
+
level: "warn",
|
|
2279
|
+
message: `Pending runner file list was not readable JSON \u2014 retrying on the next drain: ${describe(err)}`
|
|
2280
|
+
});
|
|
2281
|
+
return [];
|
|
2282
|
+
}
|
|
2283
|
+
if (!Array.isArray(body)) {
|
|
2284
|
+
options.log({
|
|
2285
|
+
level: "warn",
|
|
2286
|
+
message: "Pending runner file list was not an array \u2014 ignoring it for this drain"
|
|
2287
|
+
});
|
|
2288
|
+
return [];
|
|
2289
|
+
}
|
|
2290
|
+
const files = [];
|
|
2291
|
+
for (const entry of body) {
|
|
2292
|
+
const file = asPendingFile(entry);
|
|
2293
|
+
if (file === null) {
|
|
2294
|
+
options.log({
|
|
2295
|
+
level: "warn",
|
|
2296
|
+
message: "Ignoring a malformed pending runner file entry (expected id, path and size)"
|
|
2297
|
+
});
|
|
2298
|
+
continue;
|
|
2299
|
+
}
|
|
2300
|
+
files.push(file);
|
|
2301
|
+
}
|
|
2302
|
+
return files;
|
|
2303
|
+
}
|
|
2304
|
+
function asPendingFile(entry) {
|
|
2305
|
+
if (entry === null || typeof entry !== "object") return null;
|
|
2306
|
+
const { id, path, size } = entry;
|
|
2307
|
+
if (typeof id !== "string" || id === "") return null;
|
|
2308
|
+
if (typeof path !== "string" || path === "") return null;
|
|
2309
|
+
if (typeof size !== "number" || !Number.isFinite(size) || size < 0) return null;
|
|
2310
|
+
return { id, path, size };
|
|
2311
|
+
}
|
|
2312
|
+
async function applyOne(options, file) {
|
|
2313
|
+
const label = `${file.id.slice(0, 8)} (${file.path})`;
|
|
2314
|
+
if (options.allowedDirectories.length === 0) {
|
|
2315
|
+
options.log({
|
|
2316
|
+
level: "warn",
|
|
2317
|
+
message: `Runner file ${label} rejected: file sync is not enabled on this runner (start it with --enable-file-sync-to)`
|
|
2318
|
+
});
|
|
2319
|
+
await ack(options, file, "rejected", "file_sync_disabled");
|
|
2320
|
+
return false;
|
|
2321
|
+
}
|
|
2322
|
+
if (file.size > MAX_FILE_PUSH_BYTES) {
|
|
2323
|
+
options.log({
|
|
2324
|
+
level: "warn",
|
|
2325
|
+
message: `Runner file ${label} rejected: declared ${file.size} bytes, the limit is ${MAX_FILE_PUSH_BYTES}`
|
|
2326
|
+
});
|
|
2327
|
+
await ack(options, file, "rejected", "file_too_large");
|
|
2328
|
+
return false;
|
|
2329
|
+
}
|
|
2330
|
+
const download = await downloadContent(options, file, label);
|
|
2331
|
+
if (!download.ok) {
|
|
2332
|
+
if (download.terminal) await ack(options, file, "rejected", download.code);
|
|
2333
|
+
return false;
|
|
2334
|
+
}
|
|
2335
|
+
let outcome;
|
|
2336
|
+
try {
|
|
2337
|
+
outcome = await writePushedFile({
|
|
2338
|
+
requestedPath: file.path,
|
|
2339
|
+
content: download.content,
|
|
2340
|
+
allowedDirectories: options.allowedDirectories,
|
|
2341
|
+
homeDir: options.homeDir
|
|
2342
|
+
});
|
|
2343
|
+
} catch (err) {
|
|
2344
|
+
options.log({
|
|
2345
|
+
level: "error",
|
|
2346
|
+
message: `Runner file ${label} could not be written: ${describe(err)}`
|
|
2347
|
+
});
|
|
2348
|
+
await ack(options, file, "rejected", "write_failed");
|
|
2349
|
+
return false;
|
|
2350
|
+
}
|
|
2351
|
+
if (!outcome.ok) {
|
|
2352
|
+
options.log({
|
|
2353
|
+
level: "warn",
|
|
2354
|
+
message: `Runner file ${label} rejected (${outcome.code}): ${outcome.message}`
|
|
2355
|
+
});
|
|
2356
|
+
await ack(options, file, "rejected", outcome.code);
|
|
2357
|
+
return false;
|
|
2358
|
+
}
|
|
2359
|
+
options.log({
|
|
2360
|
+
level: "info",
|
|
2361
|
+
message: `Runner file ${label} applied (${download.content.byteLength} bytes)`
|
|
2362
|
+
});
|
|
2363
|
+
await ack(options, file, "applied");
|
|
2364
|
+
return true;
|
|
2365
|
+
}
|
|
2366
|
+
function durableDownloadCode(status) {
|
|
2367
|
+
return status === 413 ? "file_too_large" : "write_failed";
|
|
2368
|
+
}
|
|
2369
|
+
async function downloadContent(options, file, label) {
|
|
2370
|
+
try {
|
|
2371
|
+
const res = await options.fetchImpl(
|
|
2372
|
+
`${options.apiUrl}/runners/${options.agentId}/files/${file.id}/content`,
|
|
2373
|
+
{ headers: { Authorization: options.getAuthHeader() } }
|
|
2374
|
+
);
|
|
2375
|
+
if (!res.ok) {
|
|
2376
|
+
const terminal = res.status >= 400 && res.status < 500 && res.status !== 401 && res.status !== 403 && res.status !== 408 && res.status !== 429;
|
|
2377
|
+
if (!terminal) {
|
|
2378
|
+
options.log({
|
|
2379
|
+
level: "warn",
|
|
2380
|
+
message: `Downloading runner file ${label} returned HTTP ${res.status} \u2014 retrying on the next drain`
|
|
2381
|
+
});
|
|
2382
|
+
return { ok: false, terminal: false };
|
|
2383
|
+
}
|
|
2384
|
+
const code = durableDownloadCode(res.status);
|
|
2385
|
+
options.log({
|
|
2386
|
+
level: "error",
|
|
2387
|
+
message: `Downloading runner file ${label} returned HTTP ${res.status} \u2014 rejecting it as ${code} (the bytes never reached the writer)`
|
|
2388
|
+
});
|
|
2389
|
+
return { ok: false, terminal: true, code };
|
|
2390
|
+
}
|
|
2391
|
+
return { ok: true, content: Buffer.from(await res.arrayBuffer()) };
|
|
2392
|
+
} catch (err) {
|
|
2393
|
+
options.log({
|
|
2394
|
+
level: "warn",
|
|
2395
|
+
message: `Downloading runner file ${label} failed \u2014 retrying on the next drain: ${describe(err)}`
|
|
2396
|
+
});
|
|
2397
|
+
return { ok: false, terminal: false };
|
|
2398
|
+
}
|
|
2399
|
+
}
|
|
2400
|
+
async function ack(options, file, status, reason) {
|
|
2401
|
+
const outcome = `${status}${reason ? ` (${reason})` : ""}`;
|
|
2402
|
+
try {
|
|
2403
|
+
const res = await options.fetchImpl(
|
|
2404
|
+
`${options.apiUrl}/runners/${options.agentId}/files/${file.id}/ack`,
|
|
2405
|
+
{
|
|
2406
|
+
method: "POST",
|
|
2407
|
+
headers: {
|
|
2408
|
+
Authorization: options.getAuthHeader(),
|
|
2409
|
+
"Content-Type": "application/json"
|
|
2410
|
+
},
|
|
2411
|
+
body: JSON.stringify(reason ? { status, reason } : { status })
|
|
2412
|
+
}
|
|
2413
|
+
);
|
|
2414
|
+
if (!res.ok) {
|
|
2415
|
+
recordAckFailure(
|
|
2416
|
+
options,
|
|
2417
|
+
file,
|
|
2418
|
+
`Acking runner file ${file.id.slice(0, 8)} as ${outcome} returned HTTP ${res.status}`
|
|
2419
|
+
);
|
|
2420
|
+
return;
|
|
2421
|
+
}
|
|
2422
|
+
options.ackFailures.delete(file.id);
|
|
2423
|
+
} catch (err) {
|
|
2424
|
+
recordAckFailure(
|
|
2425
|
+
options,
|
|
2426
|
+
file,
|
|
2427
|
+
`Acking runner file ${file.id.slice(0, 8)} as ${outcome} failed: ${describe(err)}`
|
|
2428
|
+
);
|
|
2429
|
+
}
|
|
2430
|
+
}
|
|
2431
|
+
function recordAckFailure(options, file, what) {
|
|
2432
|
+
const attempts = (options.ackFailures.get(file.id) ?? 0) + 1;
|
|
2433
|
+
options.ackFailures.set(file.id, attempts);
|
|
2434
|
+
options.log({
|
|
2435
|
+
level: "error",
|
|
2436
|
+
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})`
|
|
2437
|
+
});
|
|
2438
|
+
}
|
|
2439
|
+
function describe(err) {
|
|
2440
|
+
return err instanceof Error ? err.message : String(err);
|
|
2441
|
+
}
|
|
2442
|
+
|
|
2036
2443
|
// src/lib/channels/driver.ts
|
|
2037
2444
|
function messageIdOf(m) {
|
|
2038
2445
|
if (!m || typeof m !== "object") return void 0;
|
|
@@ -2062,6 +2469,7 @@ var DEFAULT_STUCK_QUEUED_MS = 6e4;
|
|
|
2062
2469
|
var HEARTBEAT_MS = 6e4;
|
|
2063
2470
|
var ABSOLUTE_MAX_PROCESSING_MS = 6 * 60 * 60 * 1e3;
|
|
2064
2471
|
var POLL_MISS_GRACE_MS = HEARTBEAT_MS;
|
|
2472
|
+
var MAX_SUPERSEDED_CONVERSATIONS = 256;
|
|
2065
2473
|
var ChannelAuthError = class extends Error {
|
|
2066
2474
|
constructor(message) {
|
|
2067
2475
|
super(message);
|
|
@@ -2098,8 +2506,38 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
2098
2506
|
pausedMaxWaitMs;
|
|
2099
2507
|
stuckQueuedMs;
|
|
2100
2508
|
now;
|
|
2509
|
+
fileSyncDirectories;
|
|
2510
|
+
homeDir;
|
|
2101
2511
|
/** Cache of conversationId → opencode sessionId. */
|
|
2102
2512
|
sessions = /* @__PURE__ */ new Map();
|
|
2513
|
+
/**
|
|
2514
|
+
* conversationId → the opencode session this runner has ABANDONED as that
|
|
2515
|
+
* conversation's binding (#553), after a genuine (`sessionExists === true`)
|
|
2516
|
+
* dispatch failure: the session still exists but is wedged, so #485's self-heal
|
|
2517
|
+
* must bind a fresh one.
|
|
2518
|
+
*
|
|
2519
|
+
* Dropping the local binding + clearing the server row is not enough on its own:
|
|
2520
|
+
* a SIBLING message dispatched earlier in the same drain is still in-flight under
|
|
2521
|
+
* the same session, and its watcher's routine status writes carry
|
|
2522
|
+
* `opencode_session_id`, RESURRECTING the wedged id server-side after the clear —
|
|
2523
|
+
* and `ensureSession`'s persisted-id fallback then reuses it, defeating the
|
|
2524
|
+
* self-heal. This map makes the runner authoritative instead of racing those
|
|
2525
|
+
* writes: *`ensureSession` never reuses an abandoned id for that conversation,
|
|
2526
|
+
* whatever the server row says* — which holds even when the resurrecting write
|
|
2527
|
+
* is one we deliberately keep (see `markDone`).
|
|
2528
|
+
*
|
|
2529
|
+
* Bounded by construction, on both axes: keyed by CONVERSATION, so N failures on
|
|
2530
|
+
* one conversation hold ONE entry (the newest abandonment replaces the older), and
|
|
2531
|
+
* hard-capped at `MAX_SUPERSEDED_CONVERSATIONS` with FIFO eviction. Only the
|
|
2532
|
+
* NEWEST abandoned id per conversation is guarded: after a second abandonment a
|
|
2533
|
+
* late sibling of the FIRST session can write that id back and `ensureSession`
|
|
2534
|
+
* will reuse it — costing ONE repeat failure, which re-supersedes it. Deliberately
|
|
2535
|
+
* NOT dropped when the session's watcher tears down: `markDone` still writes the
|
|
2536
|
+
* abandoned id back (it must, or the reply is lost), so the guard has to outlive
|
|
2537
|
+
* the turn that resurrects it. In-memory only — a restart forgets it, at the same
|
|
2538
|
+
* bounded cost.
|
|
2539
|
+
*/
|
|
2540
|
+
supersededSessions = /* @__PURE__ */ new Map();
|
|
2103
2541
|
/**
|
|
2104
2542
|
* Per-opencode-session dispatch lock (Task 2.1a). `sendPromptAsync` is no
|
|
2105
2543
|
* longer idempotent (no caller-supplied `messageID`), and its read-back picks
|
|
@@ -2222,6 +2660,24 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
2222
2660
|
sessionTitles = /* @__PURE__ */ new Map();
|
|
2223
2661
|
/** Serialises drains so a reconnect during a drain doesn't double-process. */
|
|
2224
2662
|
draining = false;
|
|
2663
|
+
/**
|
|
2664
|
+
* Serialises runner-file syncs (#559) so the ~2s poll tick and a concurrent
|
|
2665
|
+
* drain ping don't download, write and ack the same file twice.
|
|
2666
|
+
*/
|
|
2667
|
+
syncingFiles = false;
|
|
2668
|
+
/**
|
|
2669
|
+
* Consecutive failed acks per pending file (#559). Lives on the driver so it
|
|
2670
|
+
* survives across drains — without it, a file whose ack keeps failing is
|
|
2671
|
+
* re-downloaded and re-written every ~2s until the server expires it.
|
|
2672
|
+
*/
|
|
2673
|
+
fileAckFailures = /* @__PURE__ */ new Map();
|
|
2674
|
+
/**
|
|
2675
|
+
* Monotonic count of files this runner has pulled and written (#559). Only
|
|
2676
|
+
* ever increases, so `run.ts` detects work by comparing it against the value
|
|
2677
|
+
* it saw on the previous cycle — including work that landed mid-sleep, the
|
|
2678
|
+
* same trick `lastProxiedActivityAt` uses.
|
|
2679
|
+
*/
|
|
2680
|
+
appliedFileCount = 0;
|
|
2225
2681
|
/**
|
|
2226
2682
|
* The currently-executing `drainPending()` promise, or null when idle. Lets a
|
|
2227
2683
|
* graceful shutdown (`waitForInFlight`) await an in-progress drain so a turn it
|
|
@@ -2252,6 +2708,8 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
2252
2708
|
this.pausedMaxWaitMs = config2.pausedMaxWaitMs ?? DEFAULT_PAUSED_MAX_WAIT_MS;
|
|
2253
2709
|
this.stuckQueuedMs = config2.stuckQueuedMs ?? DEFAULT_STUCK_QUEUED_MS;
|
|
2254
2710
|
this.now = config2.now ?? (() => Date.now());
|
|
2711
|
+
this.fileSyncDirectories = config2.fileSyncDirectories ?? [];
|
|
2712
|
+
this.homeDir = config2.homeDir ?? homedir2();
|
|
2255
2713
|
}
|
|
2256
2714
|
/** The IPv4-loopback base URL for the local `opencode serve`. */
|
|
2257
2715
|
get opencodeBase() {
|
|
@@ -2279,6 +2737,47 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
2279
2737
|
);
|
|
2280
2738
|
return run2;
|
|
2281
2739
|
}
|
|
2740
|
+
/**
|
|
2741
|
+
* Pull-and-apply any files Evident has queued for this runner (#559), riding
|
|
2742
|
+
* the EXISTING drain cycle — `run.ts` calls it from the same ~2s channel poll
|
|
2743
|
+
* and drain ping that call `drainPending()`. There is deliberately no channel,
|
|
2744
|
+
* control frame or poll loop of its own: worst-case latency is one poll tick.
|
|
2745
|
+
*
|
|
2746
|
+
* NEVER throws and never surfaces a `ChannelAuthError`: a file failure must not
|
|
2747
|
+
* cost a conversation turn. Failures are logged and either acked as a terminal
|
|
2748
|
+
* outcome or left pending for the next drain (see `runner-file-sync.ts`).
|
|
2749
|
+
*
|
|
2750
|
+
* Re-entrant calls are skipped (the poll tick and a drain ping can overlap).
|
|
2751
|
+
*
|
|
2752
|
+
* @returns the number of files written to disk.
|
|
2753
|
+
*/
|
|
2754
|
+
async syncPendingFiles() {
|
|
2755
|
+
if (this.stopped) return 0;
|
|
2756
|
+
if (this.syncingFiles) return 0;
|
|
2757
|
+
this.syncingFiles = true;
|
|
2758
|
+
try {
|
|
2759
|
+
const applied = await syncPendingRunnerFiles({
|
|
2760
|
+
agentId: this.agentId,
|
|
2761
|
+
apiUrl: this.apiUrl,
|
|
2762
|
+
getAuthHeader: this.getAuthHeader,
|
|
2763
|
+
fetchImpl: this.fetchImpl,
|
|
2764
|
+
allowedDirectories: this.fileSyncDirectories,
|
|
2765
|
+
homeDir: this.homeDir,
|
|
2766
|
+
ackFailures: this.fileAckFailures,
|
|
2767
|
+
log: this.log
|
|
2768
|
+
});
|
|
2769
|
+
this.appliedFileCount += applied;
|
|
2770
|
+
return applied;
|
|
2771
|
+
} catch (err) {
|
|
2772
|
+
this.log({
|
|
2773
|
+
level: "error",
|
|
2774
|
+
message: `Runner file sync failed unexpectedly (message processing is unaffected): ${err instanceof Error ? err.message : String(err)}`
|
|
2775
|
+
});
|
|
2776
|
+
return 0;
|
|
2777
|
+
} finally {
|
|
2778
|
+
this.syncingFiles = false;
|
|
2779
|
+
}
|
|
2780
|
+
}
|
|
2282
2781
|
async runDrain() {
|
|
2283
2782
|
let dispatched = 0;
|
|
2284
2783
|
try {
|
|
@@ -2312,6 +2811,28 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
2312
2811
|
}
|
|
2313
2812
|
return false;
|
|
2314
2813
|
}
|
|
2814
|
+
/**
|
|
2815
|
+
* File-pull work, for `run.ts`'s idle accounting (#559).
|
|
2816
|
+
*
|
|
2817
|
+
* Pulling a file is real work that `drainPending()` knows nothing about, so
|
|
2818
|
+
* without this a near-idle runner counts a credential pull as an empty tick
|
|
2819
|
+
* and `--idle-timeout` can `process.exit` mid-pull — leaving a
|
|
2820
|
+
* `.evident-push-*.tmp` behind — or immediately after the write, before the
|
|
2821
|
+
* browser has run the authorize/callback that activates it (the user then sees
|
|
2822
|
+
* `saved_not_activated` for a runner that was fine).
|
|
2823
|
+
*
|
|
2824
|
+
* Two signals because one cannot cover both cases: `inFlight` is the pull
|
|
2825
|
+
* happening RIGHT NOW (it may outlive the tick that started it), and
|
|
2826
|
+
* `appliedFiles` is monotonic so a pull that started AND finished between two
|
|
2827
|
+
* idle checks still shows up as an advance.
|
|
2828
|
+
*
|
|
2829
|
+
* CALLER CONTRACT: sample `inFlight` BEFORE calling `syncPendingFiles()` for
|
|
2830
|
+
* the cycle. `syncPendingFiles` sets the flag synchronously, so a caller that
|
|
2831
|
+
* samples afterwards reads `true` every single cycle and can never idle out.
|
|
2832
|
+
*/
|
|
2833
|
+
fileSyncActivity() {
|
|
2834
|
+
return { appliedFiles: this.appliedFileCount, inFlight: this.syncingFiles };
|
|
2835
|
+
}
|
|
2315
2836
|
/**
|
|
2316
2837
|
* OpenCode session ids the session-cleanup sweep (issue #190) must NOT delete:
|
|
2317
2838
|
* exactly those with a live (dispatched-but-not-done / paused) turn, i.e. a
|
|
@@ -2373,7 +2894,7 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
2373
2894
|
await this.sleep(step);
|
|
2374
2895
|
}
|
|
2375
2896
|
}
|
|
2376
|
-
while (this.hasInFlightWatchers()) {
|
|
2897
|
+
while (this.hasInFlightWatchers() || this.syncingFiles) {
|
|
2377
2898
|
if (this.now() >= deadline) return false;
|
|
2378
2899
|
await this.sleep(step);
|
|
2379
2900
|
}
|
|
@@ -2408,10 +2929,15 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
2408
2929
|
* @returns the count of messages NEWLY dispatched (not already in-flight).
|
|
2409
2930
|
*/
|
|
2410
2931
|
async processConversation(conv) {
|
|
2411
|
-
const sessionId = await this.ensureSession(conv);
|
|
2932
|
+
const { sessionId, refusedSessionId } = await this.ensureSession(conv);
|
|
2412
2933
|
const messages = await this.getPendingMessages(conv.id);
|
|
2413
2934
|
let dispatched = 0;
|
|
2414
2935
|
let skippedAlreadyDispatched = 0;
|
|
2936
|
+
if (refusedSessionId && messages.length > 0) {
|
|
2937
|
+
void this.postSignal(conv.id, messages[0].id, "session_superseded", {
|
|
2938
|
+
superseded_session_id: refusedSessionId
|
|
2939
|
+
});
|
|
2940
|
+
}
|
|
2415
2941
|
for (const message of messages) {
|
|
2416
2942
|
if (this.stopped) break;
|
|
2417
2943
|
if (this.dispatched.has(message.id)) {
|
|
@@ -2460,6 +2986,13 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
2460
2986
|
}
|
|
2461
2987
|
const errorMessage = err instanceof Error ? err.message : String(err);
|
|
2462
2988
|
this.sessions.delete(conv.id);
|
|
2989
|
+
this.supersede(conv.id, sessionId);
|
|
2990
|
+
this.log({
|
|
2991
|
+
level: "warn",
|
|
2992
|
+
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.`,
|
|
2993
|
+
conversation_id: conv.id,
|
|
2994
|
+
message_id: message.id
|
|
2995
|
+
});
|
|
2463
2996
|
await this.markFailed(conv.id, message.id, null, errorMessage).catch((markErr) => {
|
|
2464
2997
|
this.log({
|
|
2465
2998
|
level: "warn",
|
|
@@ -2500,8 +3033,42 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
2500
3033
|
this.ensureWatcherRunning(sessionId);
|
|
2501
3034
|
return dispatched;
|
|
2502
3035
|
}
|
|
3036
|
+
/**
|
|
3037
|
+
* Record that `sessionId` is no longer a valid binding for `conversationId`
|
|
3038
|
+
* (#553). Keyed by conversation and hard-capped, so it cannot grow with the
|
|
3039
|
+
* number of failures — see the `supersededSessions` field doc.
|
|
3040
|
+
*/
|
|
3041
|
+
supersede(conversationId, sessionId) {
|
|
3042
|
+
this.supersededSessions.delete(conversationId);
|
|
3043
|
+
this.supersededSessions.set(conversationId, sessionId);
|
|
3044
|
+
while (this.supersededSessions.size > MAX_SUPERSEDED_CONVERSATIONS) {
|
|
3045
|
+
const oldest = this.supersededSessions.keys().next().value;
|
|
3046
|
+
if (oldest === void 0) return;
|
|
3047
|
+
this.supersededSessions.delete(oldest);
|
|
3048
|
+
}
|
|
3049
|
+
}
|
|
3050
|
+
/** Whether `sessionId` is the session this conversation has abandoned (#553). */
|
|
3051
|
+
isSuperseded(conversationId, sessionId) {
|
|
3052
|
+
return this.supersededSessions.get(conversationId) === sessionId;
|
|
3053
|
+
}
|
|
3054
|
+
/**
|
|
3055
|
+
* Resolve the opencode session to run this conversation's turns in.
|
|
3056
|
+
*
|
|
3057
|
+
* `refusedSessionId` is set when the #553 guard fired — i.e. the persisted
|
|
3058
|
+
* binding was an id this runner had abandoned, so a resurrection genuinely
|
|
3059
|
+
* happened and a fresh session was bound instead. The caller reports it.
|
|
3060
|
+
*/
|
|
2503
3061
|
async ensureSession(conv) {
|
|
2504
3062
|
const bound = this.sessions.get(conv.id) ?? conv.opencode_session_id ?? null;
|
|
3063
|
+
if (bound && this.isSuperseded(conv.id, bound)) {
|
|
3064
|
+
this.log({
|
|
3065
|
+
level: "warn",
|
|
3066
|
+
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.`,
|
|
3067
|
+
conversation_id: conv.id
|
|
3068
|
+
});
|
|
3069
|
+
this.sessions.delete(conv.id);
|
|
3070
|
+
return { sessionId: await this.createAndBindSession(conv.id), refusedSessionId: bound };
|
|
3071
|
+
}
|
|
2505
3072
|
if (bound) {
|
|
2506
3073
|
const exists = await sessionExists(this.port, bound);
|
|
2507
3074
|
if (exists === false) {
|
|
@@ -2511,12 +3078,12 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
2511
3078
|
conversation_id: conv.id
|
|
2512
3079
|
});
|
|
2513
3080
|
this.sessions.delete(conv.id);
|
|
2514
|
-
return this.createAndBindSession(conv.id);
|
|
3081
|
+
return { sessionId: await this.createAndBindSession(conv.id) };
|
|
2515
3082
|
}
|
|
2516
3083
|
this.sessions.set(conv.id, bound);
|
|
2517
|
-
return bound;
|
|
3084
|
+
return { sessionId: bound };
|
|
2518
3085
|
}
|
|
2519
|
-
return this.createAndBindSession(conv.id);
|
|
3086
|
+
return { sessionId: await this.createAndBindSession(conv.id) };
|
|
2520
3087
|
}
|
|
2521
3088
|
/**
|
|
2522
3089
|
* Create a fresh OpenCode session for a conversation, cache the binding, and
|
|
@@ -3975,6 +4542,32 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
3975
4542
|
}
|
|
3976
4543
|
return messages;
|
|
3977
4544
|
}
|
|
4545
|
+
/**
|
|
4546
|
+
* The `opencode_session_id` fragment of a status PATCH body — `{}` when this
|
|
4547
|
+
* conversation has ABANDONED that session (#553). The field is optional
|
|
4548
|
+
* server-side and an absent one leaves the persisted binding untouched, so
|
|
4549
|
+
* omitting it is how a routine status write stops resurrecting it.
|
|
4550
|
+
*
|
|
4551
|
+
* ONLY for writes whose sole cost is a lost deep link. The `processing` notice
|
|
4552
|
+
* degrades to no "View in Evident" link (the reaction swap still fires) and the
|
|
4553
|
+
* turn-failure notice is built from the PATCH's own `error` text with a link off
|
|
4554
|
+
* the persisted row — neither loses content the user came for. `markDone`
|
|
4555
|
+
* deliberately does NOT use this helper: the server fetches the reply text
|
|
4556
|
+
* THROUGH the session id it is given, so suppressing there would replace the
|
|
4557
|
+
* agent's answer with a bare "✅ Done!" (the #183/#187 failure). The
|
|
4558
|
+
* `ensureSession` guard, not this suppression, is what makes the self-heal
|
|
4559
|
+
* stick.
|
|
4560
|
+
*/
|
|
4561
|
+
sessionIdBody(sessionId, conversationId, messageId, status) {
|
|
4562
|
+
if (!this.isSuperseded(conversationId, sessionId)) return { opencode_session_id: sessionId };
|
|
4563
|
+
this.log({
|
|
4564
|
+
level: "debug",
|
|
4565
|
+
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)}`,
|
|
4566
|
+
conversation_id: conversationId,
|
|
4567
|
+
message_id: messageId
|
|
4568
|
+
});
|
|
4569
|
+
return {};
|
|
4570
|
+
}
|
|
3978
4571
|
/**
|
|
3979
4572
|
* EXISTING combinedAuth route — now fired by the watcher on queued→running
|
|
3980
4573
|
* (Task 3.3), NOT at dispatch/claim time. `{status:'processing',
|
|
@@ -4004,7 +4597,7 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
4004
4597
|
headers: { Authorization: this.getAuthHeader(), "Content-Type": "application/json" },
|
|
4005
4598
|
body: JSON.stringify({
|
|
4006
4599
|
status: "processing",
|
|
4007
|
-
|
|
4600
|
+
...this.sessionIdBody(sessionId, conversationId, messageId, "processing"),
|
|
4008
4601
|
...opencodeMessageId ? { opencode_message_id: opencodeMessageId } : {},
|
|
4009
4602
|
...title ? { title } : {}
|
|
4010
4603
|
})
|
|
@@ -4053,6 +4646,11 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
4053
4646
|
headers: { Authorization: this.getAuthHeader(), "Content-Type": "application/json" },
|
|
4054
4647
|
body: JSON.stringify({
|
|
4055
4648
|
status: "done",
|
|
4649
|
+
// ALWAYS sent, even for a session this conversation has abandoned
|
|
4650
|
+
// (#553): the server reads the reply text back out of THIS session id
|
|
4651
|
+
// to deliver it. Omitting it would leave the user with "✅ Done!"
|
|
4652
|
+
// instead of the answer — a worse regression than the resurrection it
|
|
4653
|
+
// would prevent, which `ensureSession`'s guard handles anyway.
|
|
4056
4654
|
opencode_session_id: sessionId,
|
|
4057
4655
|
...opencodeMessageId ? { opencode_message_id: opencodeMessageId } : {},
|
|
4058
4656
|
...title ? { title } : {},
|
|
@@ -4081,7 +4679,11 @@ var ChannelDriver = class _ChannelDriver {
|
|
|
4081
4679
|
*/
|
|
4082
4680
|
async markFailed(conversationId, messageId, sessionId, error2, usage) {
|
|
4083
4681
|
const body = { status: "failed" };
|
|
4084
|
-
if (sessionId
|
|
4682
|
+
if (sessionId === null) {
|
|
4683
|
+
body.opencode_session_id = null;
|
|
4684
|
+
} else if (sessionId !== void 0) {
|
|
4685
|
+
Object.assign(body, this.sessionIdBody(sessionId, conversationId, messageId, "failed"));
|
|
4686
|
+
}
|
|
4085
4687
|
if (error2 !== void 0) body.error = error2;
|
|
4086
4688
|
if (usage) Object.assign(body, usage);
|
|
4087
4689
|
await this.callWithRetry(
|
|
@@ -4498,6 +5100,34 @@ function resolveLogLevel(options) {
|
|
|
4498
5100
|
}
|
|
4499
5101
|
return "info";
|
|
4500
5102
|
}
|
|
5103
|
+
function resolveFileSyncDirectories(raw, homeDir) {
|
|
5104
|
+
const directories = [];
|
|
5105
|
+
for (const entry of raw ?? []) {
|
|
5106
|
+
const trimmed = entry.trim();
|
|
5107
|
+
if (trimmed === "") {
|
|
5108
|
+
throw new Error("--enable-file-sync-to requires a directory path (got an empty value)");
|
|
5109
|
+
}
|
|
5110
|
+
const expanded = trimmed === "~" ? homeDir : trimmed.startsWith("~/") ? join3(homeDir, trimmed.slice(2)) : trimmed;
|
|
5111
|
+
if (!isAbsolute2(expanded)) {
|
|
5112
|
+
throw new Error(`--enable-file-sync-to requires an absolute directory path; got "${entry}"`);
|
|
5113
|
+
}
|
|
5114
|
+
const normalized = resolvePath(expanded);
|
|
5115
|
+
if (parse(normalized).root === normalized) {
|
|
5116
|
+
throw new Error(
|
|
5117
|
+
`--enable-file-sync-to will not allow-list the filesystem root ("${entry}"); name the specific directory the credentials belong in (for example ~/.claude)`
|
|
5118
|
+
);
|
|
5119
|
+
}
|
|
5120
|
+
if (!directories.includes(normalized)) {
|
|
5121
|
+
directories.push(normalized);
|
|
5122
|
+
}
|
|
5123
|
+
}
|
|
5124
|
+
if (directories.length > MAX_FILE_SYNC_DIRECTORIES) {
|
|
5125
|
+
throw new Error(
|
|
5126
|
+
`--enable-file-sync-to accepts at most ${MAX_FILE_SYNC_DIRECTORIES} directories; got ${directories.length}`
|
|
5127
|
+
);
|
|
5128
|
+
}
|
|
5129
|
+
return directories;
|
|
5130
|
+
}
|
|
4501
5131
|
function meetsThreshold(state, level) {
|
|
4502
5132
|
return LOG_LEVELS[level] >= LOG_LEVELS[state.logLevel];
|
|
4503
5133
|
}
|
|
@@ -4619,18 +5249,29 @@ async function handleAuthError(state, error2) {
|
|
|
4619
5249
|
async function driveChannels(state, driver) {
|
|
4620
5250
|
let idlePolls = 0;
|
|
4621
5251
|
let lastSeenProxiedActivityAt = state.lastProxiedActivityAt;
|
|
5252
|
+
let lastSeenAppliedFiles = driver.fileSyncActivity().appliedFiles;
|
|
4622
5253
|
while (state.running) {
|
|
4623
5254
|
if (state.connection?.reconnecting && state.connection.reconnectPromise) {
|
|
4624
5255
|
logActivity(state, { type: "info", message: "Waiting for tunnel reconnection..." });
|
|
4625
5256
|
if (state.interactive) displayStatus(state);
|
|
4626
5257
|
await state.connection.reconnectPromise;
|
|
4627
5258
|
}
|
|
5259
|
+
const carriedOverFileSync = driver.fileSyncActivity().inFlight;
|
|
5260
|
+
void driver.syncPendingFiles().catch(
|
|
5261
|
+
(error2) => logActivity(state, {
|
|
5262
|
+
type: "error",
|
|
5263
|
+
error: `Runner file sync failed: ${error2 instanceof Error ? error2.message : String(error2)}`
|
|
5264
|
+
})
|
|
5265
|
+
);
|
|
4628
5266
|
try {
|
|
4629
5267
|
const processed = await driver.drainPending();
|
|
4630
5268
|
state.messageCount += processed;
|
|
4631
5269
|
const proxiedActivity = state.lastProxiedActivityAt !== lastSeenProxiedActivityAt;
|
|
4632
5270
|
lastSeenProxiedActivityAt = state.lastProxiedActivityAt;
|
|
4633
|
-
|
|
5271
|
+
const appliedFiles = driver.fileSyncActivity().appliedFiles;
|
|
5272
|
+
const fileActivity = carriedOverFileSync || appliedFiles !== lastSeenAppliedFiles;
|
|
5273
|
+
lastSeenAppliedFiles = appliedFiles;
|
|
5274
|
+
if (processed > 0 || driver.hasInFlightWatchers() || proxiedActivity || fileActivity) {
|
|
4634
5275
|
idlePolls = 0;
|
|
4635
5276
|
if (processed > 0 && state.interactive) displayStatus(state);
|
|
4636
5277
|
} else if (state.idleTimeout !== null) {
|
|
@@ -4659,7 +5300,7 @@ async function driveChannels(state, driver) {
|
|
|
4659
5300
|
logActivity(state, { type: "error", error: `Channel processing error: ${errorMessage}` });
|
|
4660
5301
|
if (state.interactive) displayStatus(state);
|
|
4661
5302
|
}
|
|
4662
|
-
await new Promise((
|
|
5303
|
+
await new Promise((resolve3) => setTimeout(resolve3, CHANNEL_POLL_INTERVAL_MS));
|
|
4663
5304
|
if (state.idleTimeout !== null && idlePolls >= 2) {
|
|
4664
5305
|
const idleMs = idlePolls * CHANNEL_POLL_INTERVAL_MS;
|
|
4665
5306
|
if (idleMs > state.idleTimeout * 1e3) {
|
|
@@ -4804,8 +5445,10 @@ async function cleanup(state, opts = {}) {
|
|
|
4804
5445
|
async function run(options) {
|
|
4805
5446
|
const interactive = isInteractive(options.json);
|
|
4806
5447
|
let logLevel;
|
|
5448
|
+
let fileSyncDirectories;
|
|
4807
5449
|
try {
|
|
4808
5450
|
logLevel = resolveLogLevel(options);
|
|
5451
|
+
fileSyncDirectories = resolveFileSyncDirectories(options.enableFileSyncTo, homedir3());
|
|
4809
5452
|
} catch (error2) {
|
|
4810
5453
|
const message = error2 instanceof Error ? error2.message : String(error2);
|
|
4811
5454
|
if (options.json) {
|
|
@@ -4840,6 +5483,11 @@ async function run(options) {
|
|
|
4840
5483
|
sessionCleanupTimers: [],
|
|
4841
5484
|
authHeader: ""
|
|
4842
5485
|
};
|
|
5486
|
+
if (fileSyncDirectories.length > 0) {
|
|
5487
|
+
log2(state, `File sync enabled for: ${fileSyncDirectories.join(", ")}`);
|
|
5488
|
+
} else {
|
|
5489
|
+
log2(state, "File sync is disabled (no --enable-file-sync-to given)", "debug");
|
|
5490
|
+
}
|
|
4843
5491
|
if (!options.runner && options.agent) {
|
|
4844
5492
|
telemetry.info(
|
|
4845
5493
|
EventTypes.DEPRECATED_AGENT_FLAG_USED,
|
|
@@ -5032,6 +5680,10 @@ async function run(options) {
|
|
|
5032
5680
|
getAuthHeader: () => state.authHeader,
|
|
5033
5681
|
conversationFilter: state.conversationFilter,
|
|
5034
5682
|
stuckQueuedMs: CHANNEL_STUCK_QUEUED_MS,
|
|
5683
|
+
// #559: the `--enable-file-sync-to` allow-list. Empty ⇒ pending files are
|
|
5684
|
+
// REJECTED with `file_sync_disabled` on the ack, not silently ignored.
|
|
5685
|
+
fileSyncDirectories,
|
|
5686
|
+
homeDir: homedir3(),
|
|
5035
5687
|
log: (entry) => (
|
|
5036
5688
|
// Thread the driver's real level straight through so `debug`/`warn`
|
|
5037
5689
|
// survive the sink filter (they no longer collapse to info). `type`
|
|
@@ -5113,6 +5765,12 @@ async function run(options) {
|
|
|
5113
5765
|
onDrainPing: () => {
|
|
5114
5766
|
if (!state.running) return;
|
|
5115
5767
|
logActivity(state, { type: "info", message: "Drain ping received \u2014 draining" });
|
|
5768
|
+
void channelDriver.syncPendingFiles().catch(
|
|
5769
|
+
(error2) => logActivity(state, {
|
|
5770
|
+
type: "error",
|
|
5771
|
+
error: `Runner file sync failed on ping: ${error2 instanceof Error ? error2.message : String(error2)}`
|
|
5772
|
+
})
|
|
5773
|
+
);
|
|
5116
5774
|
channelDriver.drainPending().then((processed) => {
|
|
5117
5775
|
if (processed > 0) {
|
|
5118
5776
|
state.messageCount += processed;
|
|
@@ -5196,7 +5854,10 @@ program.name("evident").description("Run OpenCode locally and connect it to Evid
|
|
|
5196
5854
|
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);
|
|
5197
5855
|
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 }));
|
|
5198
5856
|
program.command("whoami").description("Show the currently logged in user").action(whoami);
|
|
5199
|
-
program.command("run").description("Connect to Evident and process messages").option("
|
|
5857
|
+
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(
|
|
5858
|
+
"-a, --agent [id]",
|
|
5859
|
+
"Deprecated alias for --runner (still supported; --runner wins if both are given)"
|
|
5860
|
+
).option("-p, --port <port>", "OpenCode port (default: 4096)", "4096").option(
|
|
5200
5861
|
"--log-level <level>",
|
|
5201
5862
|
"Log verbosity: debug | info | warn | error (default: info). Env: EVIDENT_LOG_LEVEL"
|
|
5202
5863
|
).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(
|
|
@@ -5208,6 +5869,11 @@ program.command("run").description("Connect to Evident and process messages").op
|
|
|
5208
5869
|
).option(
|
|
5209
5870
|
"--session-cleanup-interval <duration>",
|
|
5210
5871
|
"How often the cleanup sweep runs (default: 1h). Env: EVIDENT_SESSION_CLEANUP_INTERVAL"
|
|
5872
|
+
).option(
|
|
5873
|
+
"--enable-file-sync-to <dir>",
|
|
5874
|
+
"Allow Evident to write files into this directory (repeatable). Omit to disable file sync entirely.",
|
|
5875
|
+
(value, previous) => previous.concat([value]),
|
|
5876
|
+
[]
|
|
5211
5877
|
).action(
|
|
5212
5878
|
(options) => {
|
|
5213
5879
|
run({
|
|
@@ -5224,7 +5890,10 @@ program.command("run").description("Connect to Evident and process messages").op
|
|
|
5224
5890
|
// Raw strings — the resolver in run.ts single-sources parsing (M1).
|
|
5225
5891
|
sessionCleanupMaxAge: options.sessionCleanupMaxAge,
|
|
5226
5892
|
sessionCleanupMaxCount: options.sessionCleanupMaxCount,
|
|
5227
|
-
sessionCleanupInterval: options.sessionCleanupInterval
|
|
5893
|
+
sessionCleanupInterval: options.sessionCleanupInterval,
|
|
5894
|
+
// Raw values — expansion/validation is single-sourced in run.ts's
|
|
5895
|
+
// resolveFileSyncDirectories.
|
|
5896
|
+
enableFileSyncTo: options.enableFileSyncTo
|
|
5228
5897
|
});
|
|
5229
5898
|
}
|
|
5230
5899
|
);
|