byollm 0.1.0-alpha.75 → 0.1.0-alpha.76

package/README.md

@@ -1,5 +1,5 @@
 > [!WARNING]
-> **Alpha (`0.1.0-alpha.75`) — under active development. Don't use this yet.**
+> **Alpha (`0.1.0-alpha.76`) — under active development. Don't use this yet.**
 >
 > Install it deliberately: `npx byollm@alpha`, or `npm install byollm@alpha`.
 >

package/dist/bin.js

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 import {
   main
-} from "./chunk-ID6QL5NR.js";
+} from "./chunk-MSRBBLRQ.js";
 
 // src/bin.ts
 process.exitCode = await main(process.argv.slice(2));

package/dist/{chunk-ID6QL5NR.js → chunk-MSRBBLRQ.js}

@@ -53,6 +53,32 @@ function asHttpOrigin(candidate) {
 // src/cli.ts
 import { access } from "fs/promises";
 
+// src/emphasis.ts
+var REVERSE = "\x1B[7m";
+var BOLD = "\x1B[1m";
+var RESET = "\x1B[0m";
+function emphasisAllowed(context2) {
+  if (context2.env["NO_COLOR"] !== void 0) return false;
+  if (context2.env["FORCE_COLOR"] !== void 0) return true;
+  return context2.tty;
+}
+function emphasise(value, context2) {
+  return emphasisAllowed(context2) ? `${REVERSE}${BOLD} ${value} ${RESET}` : value;
+}
+function terminalContext() {
+  return {
+    // `@types/node` declares `isTTY` as `boolean`, and Node sets it to
+    // `undefined` when the stream is not a terminal. So this comparison is
+    // load-bearing even though the type says it cannot be — the linter is
+    // reasoning from a declaration that is wrong about its own runtime, and
+    // deleting it puts `undefined` into a field typed `boolean`. The same
+    // note stands over `setup.ts`, which met this first.
+    // eslint-disable-next-line @typescript-eslint/no-unnecessary-boolean-literal-compare
+    tty: process.stdout.isTTY === true,
+    env: process.env
+  };
+}
+
 // src/model.ts
 import { readFile, writeFile } from "fs/promises";
 
@@ -218,8 +244,8 @@ function until(message, now) {
   if (Number.isNaN(parsed)) return void 0;
   return parsed > now ? parsed : void 0;
 }
-function quotaBlock(message, now) {
-  if (!OBSERVED.some((seen) => seen.pattern.test(message))) return void 0;
+function quotaBlock(message, now, corpus = OBSERVED) {
+  if (!corpus.some((seen) => seen.pattern.test(message))) return void 0;
   const at = until(message, now);
   return { detail: message, ...at === void 0 ? {} : { until: at } };
 }
@@ -234,7 +260,6 @@ async function runProcessJob(job) {
       ok: false,
       code: "backend-error",
       message: "no time limit was set for this call",
-      retryable: false,
       durationMs: 0
     };
   }
@@ -253,7 +278,6 @@ function spawnIn(job, scratch) {
         ok: false,
         code: "canceled",
         message: "the job was canceled before it started",
-        retryable: false,
         durationMs: Date.now() - started
       });
       return;
@@ -316,7 +340,6 @@ function spawnIn(job, scratch) {
         ok: false,
         code: "backend-unreachable",
         message: `could not start ${displayName}: ${error.message}`,
-        retryable: false,
         durationMs: Date.now() - started
       });
     });
@@ -328,7 +351,6 @@ function spawnIn(job, scratch) {
           ok: false,
           code: "canceled",
           message: "the job was canceled",
-          retryable: false,
           durationMs
         });
         return;
@@ -338,7 +360,6 @@ function spawnIn(job, scratch) {
           ok: false,
           code: "timeout",
           message: `the model did not answer within ${String(request.timeoutMs)}ms`,
-          retryable: true,
           durationMs
         });
         return;
@@ -348,7 +369,6 @@ function spawnIn(job, scratch) {
           ok: false,
           code: "output-too-large",
           message: `the model produced more than ${String(request.maxOutputBytes)} bytes`,
-          retryable: false,
           durationMs
         });
         return;
@@ -356,7 +376,7 @@ function spawnIn(job, scratch) {
       if (code !== 0) {
         const said = `${stdout}
 ${stderr}`;
-        const blocked = quotaBlock(said, Date.now());
+        const blocked = quotaBlock(said, (job.now ?? Date.now)(), job.corpus);
         const authFailed = blocked === void 0 && isAuthFailure(said);
         finish({
           ok: false,
@@ -378,7 +398,6 @@ ${stderr}`;
             goes to the site on success. */
             stdout.trim() !== "" ? `${displayName} failed: ${firstLine(stdout)}` : `${displayName} exited with status ${String(code)}`
           ),
-          retryable: false,
           durationMs
         });
         return;
@@ -665,21 +684,20 @@ function failureMessage(event) {
   const error = record(event["error"]);
   return diagnostic(event["message"]) ?? diagnostic(error?.["message"]) ?? "codex reported an error";
 }
-function classifyFailure(message, now) {
-  const blocked = quotaBlock(message, now);
+function classifyFailure(message, now, corpus) {
+  const blocked = quotaBlock(message, now, corpus);
   if (blocked !== void 0) {
     return {
       code: "quota-exhausted",
-      retryable: true,
       ...blocked.until === void 0 ? {} : { until: blocked.until }
     };
   }
   if (isAuthFailure(message)) {
-    return { code: "unauthorized", retryable: false };
+    return { code: "unauthorized" };
   }
-  return { code: "backend-error", retryable: false };
+  return { code: "backend-error" };
 }
-function parseCodexOutput(output, now = Date.now()) {
+function parseCodexOutput(output, now = Date.now(), corpus) {
   const messages = [];
   for (const line of output.split(/\r?\n/u)) {
     if (line.trim() === "") continue;
@@ -701,7 +719,7 @@ function parseCodexOutput(output, now = Date.now()) {
     }
     if (type === "error" || type === "turn.failed") {
       const detail = failureMessage(event);
-      const classified = classifyFailure(detail, now);
+      const classified = classifyFailure(detail, now, corpus);
       return {
         ok: false,
         ...classified,
@@ -715,8 +733,7 @@ function parseCodexOutput(output, now = Date.now()) {
   return {
     ok: false,
     code: "backend-error",
-    message: "the codex CLI ended without a terminal event",
-    retryable: false
+    message: "the codex CLI ended without a terminal event"
   };
 }
 var CodexCliBackend = class {
@@ -965,7 +982,6 @@ var OpenAiHttpBackend = class {
         ok: false,
         code: "backend-error",
         message: "no time limit was set for this call",
-        retryable: false,
         durationMs: 0
       };
     }
@@ -990,7 +1006,6 @@ var OpenAiHttpBackend = class {
         return this.#fail(
           "unauthorized",
           "the model server rejected our credentials",
-          false,
           started
         );
       }
@@ -998,7 +1013,6 @@ var OpenAiHttpBackend = class {
         return this.#fail(
           "model-not-found",
           `the model server does not know "${request.model}"`,
-          false,
           started
         );
       }
@@ -1006,7 +1020,6 @@ var OpenAiHttpBackend = class {
         return this.#fail(
           "backend-error",
           `the model server returned HTTP ${String(response.status)}`,
-          response.status >= 500,
           started
         );
       }
@@ -1015,7 +1028,6 @@ var OpenAiHttpBackend = class {
         return this.#fail(
           "output-too-large",
           `the model produced more than ${String(request.maxOutputBytes)} bytes`,
-          false,
           started
         );
       }
@@ -1024,37 +1036,43 @@ var OpenAiHttpBackend = class {
         return this.#fail(
           "backend-error",
           "the model server's response was not in OpenAI chat-completion shape",
-          false,
           started
         );
       }
       return { ok: true, text: content, durationMs: Date.now() - started };
     } catch (error) {
       if (request.signal.aborted) {
-        return this.#fail("canceled", "the job was canceled", false, started);
+        return this.#fail("canceled", "the job was canceled", started);
       }
       if (isAbort(error)) {
         return this.#fail(
           "timeout",
           `the model did not answer within ${String(request.timeoutMs)}ms`,
-          true,
           started
         );
       }
       return this.#fail(
         "backend-unreachable",
         describeFetchError(error, this.#baseUrl.origin),
-        true,
         started
       );
     }
   }
-  #fail(code, message, retryable, started) {
+  /*
+   * No `retryable` argument any more — ruled 2026-09-04.
+   *
+   * This adapter used to decide it per failure, and it was the only one whose
+   * rule was defensible: `true` for unreachable and for a 5xx. That is
+   * exactly why removing it matters. The decision now lives once, in the
+   * site-facing class table, and it was the *disagreement* between three
+   * adapters that let a value only quota produced become readable as a fact
+   * about somebody's account.
+   */
+  #fail(code, message, started) {
     return {
       ok: false,
       code,
       message,
-      retryable,
       durationMs: Date.now() - started
     };
   }
@@ -3486,27 +3504,27 @@ var FOR_SITE = Object.freeze({
   "backend-unreachable": {
     code: "service_unavailable",
     message: SERVICE_UNAVAILABLE,
-    retryable: false
+    retryable: true
   },
   "backend-error": {
     code: "service_unavailable",
     message: SERVICE_UNAVAILABLE,
-    retryable: false
+    retryable: true
   },
   "model-not-found": {
     code: "service_unavailable",
     message: SERVICE_UNAVAILABLE,
-    retryable: false
+    retryable: true
   },
   "quota-exhausted": {
     code: "service_unavailable",
     message: SERVICE_UNAVAILABLE,
-    retryable: false
+    retryable: true
   },
   unauthorized: {
     code: "service_unavailable",
     message: SERVICE_UNAVAILABLE,
-    retryable: false
+    retryable: true
   },
   timeout: {
     code: "timeout",
@@ -3647,6 +3665,8 @@ var Runner = class {
    * service was blocked.
    */
   #blockedReport = /* @__PURE__ */ new Map();
+  /** What was last handed to the writer, so an unchanged pass writes nothing. */
+  #lastWritten;
   /**
    * The key grants are checked against — from the pairing, or adopted later.
    *
@@ -3769,6 +3789,7 @@ var Runner = class {
         }
         this.#blocked.delete(route.service);
         this.#blockedReport.delete(route.service);
+        this.serviceStates.delete(route.service);
       }
       let usable = probed.get(route.service);
       if (usable === void 0) {
@@ -3831,6 +3852,7 @@ var Runner = class {
       });
     }
     this.#capabilities = capabilities;
+    this.#writeServiceStates();
     return capabilities;
   }
   #backendFor(route) {
@@ -4166,6 +4188,9 @@ var Runner = class {
         outcome: "error",
         code: "no-capability",
         message: "this device has no route for that job kind",
+        /* Not retryable, and not from the class table: this is the runner's
+           own refusal rather than a backend failure. A device with no route
+           for a kind will not grow one by being asked again. */
         retryable: false
       };
     }
@@ -4210,6 +4235,12 @@ var Runner = class {
       });
       if (!result.ok && result.code === "unauthorized") {
         this.#unauthenticated.add(route.service);
+        this.#noteServiceState(route.service, {
+          state: {
+            kind: "signed-out",
+            detail: result.message
+          }
+        });
         this.#options.onEvent?.({
           type: "service-not-signed-in",
           service: route.service,
@@ -4218,18 +4249,15 @@ var Runner = class {
       }
       if (!result.ok && result.code === "quota-exhausted") {
         this.#blocked.set(route.service, result.until);
-        const report2 = {
+        const report2 = this.#noteServiceState(route.service, {
           ...this.serviceStates.get(route.service),
           state: {
             kind: "blocked",
             detail: result.message,
             ...result.until === void 0 ? {} : { until: result.until }
           }
-        };
-        this.serviceStates.set(route.service, report2);
+        });
         this.#blockedReport.set(route.service, report2);
-        const noted = this.#options.onServiceStates?.(this.serviceStates);
-        if (noted !== void 0) void noted.catch(() => void 0);
         this.#options.onEvent?.({
           type: "service-out-of-quota",
           service: route.service,
@@ -4339,6 +4367,34 @@ var Runner = class {
       await sleep(heartbeatMs * (0.85 + Math.random() * 0.3), signal);
     }
   }
+  /**
+   * Record a service's state, and tell whoever writes the file — S-2.
+   *
+   * One door, because the bug was two doors and only one of them wrote. The
+   * quota path wrote on block and not on lift; the sign-out path never wrote
+   * at all. Both are changes `byollm status` would report differently, and
+   * status is a different process whose only source is that file.
+   */
+  #noteServiceState(service, report2) {
+    this.serviceStates.set(service, report2);
+    this.#writeServiceStates();
+    return report2;
+  }
+  /**
+   * Hand the current map to whoever persists it.
+   *
+   * Not awaited, and the rejection is swallowed on purpose: a job must not
+   * fail — nor the runner die on an unhandled rejection — because a status
+   * note could not be written. The note is evidence about the job, not part
+   * of it.
+   */
+  #writeServiceStates() {
+    const now = JSON.stringify([...this.serviceStates]);
+    if (now === this.#lastWritten) return;
+    this.#lastWritten = now;
+    const noted = this.#options.onServiceStates?.(this.serviceStates);
+    if (noted !== void 0) void noted.catch(() => void 0);
+  }
   /**
    * Note a refusal that means the relationship is over.
    *
@@ -5907,7 +5963,7 @@ Connecting to ${origin}
           `
   Your steps:
     1) Open:       ${info.verificationUrl}
-    2) Enter code: ${info.userCode}   (expires in ${String(minutes)} minutes)
+    2) Enter code: ${emphasise(info.userCode, terminalContext())}   (expires in ${String(minutes)} minutes)
     3) Check the screen shows this device's fingerprint, then approve:
 
        ${fingerprint3(deviceIdentity.identity)}
@@ -6973,7 +7029,7 @@ async function exists(path) {
 }
 
 // src/index.ts
-var DAEMON_VERSION = "0.1.0-alpha.75";
+var DAEMON_VERSION = "0.1.0-alpha.76";
 function daemonVersion() {
   return {
     daemon: DAEMON_VERSION,
@@ -7025,4 +7081,4 @@ export {
   runCli,
   main
 };
-//# sourceMappingURL=chunk-ID6QL5NR.js.map
+//# sourceMappingURL=chunk-MSRBBLRQ.js.map