byollm 0.1.0-alpha.4 → 0.1.0-alpha.6

package/README.md

@@ -1,5 +1,5 @@
 > [!WARNING]
-> **Alpha (`0.1.0-alpha.4`) — under active development. Don't use this yet.**
+> **Alpha (`0.1.0-alpha.6`) — 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-CKWS6NEW.js";
+} from "./chunk-EU4PO36M.js";
 
 // src/bin.ts
 process.exitCode = await main(process.argv.slice(2));

package/dist/{chunk-CKWS6NEW.js → chunk-EU4PO36M.js}

@@ -220,7 +220,7 @@ var ClientError = class extends Error {
   name = "ClientError";
   /** Is retrying this same call plausibly useful? */
   get retryable() {
-    return this.kind === "unreachable" || this.kind === "rate-limited" || this.kind === "server-error";
+    return this.kind === "unreachable" || this.kind === "not-ready" || this.kind === "rate-limited" || this.kind === "server-error";
   }
 };
 var DEFAULT_TIMEOUT_MS = 3e4;
@@ -383,6 +383,8 @@ var ProtocolClient = class _ProtocolClient {
         return new ClientError("unauthorized", message, retryAfter);
       case 403:
         return new ClientError("revoked", message, retryAfter);
+      case 409:
+        return new ClientError("not-ready", message, retryAfter);
       case 429:
         return new ClientError("rate-limited", message, retryAfter);
       case 400:
@@ -2141,11 +2143,8 @@ var Runner = class {
       );
       return;
     }
-    const fetched = await this.#options.client.fetch({
-      runnerId: this.#options.runnerId,
-      jobId: job.id,
-      leaseId: job.lease.id
-    });
+    const fetched = await this.#fetchWhenSealed(job);
+    if (!fetched) return;
     const payload = await this.#openPayload(job, fetched.envelope);
     const route = this.#routeFor(job.kind);
     const outcome = await this.runJob({ ...job, payload });
@@ -2194,6 +2193,49 @@ var Runner = class {
       }
     });
   }
+  /**
+   * Collect the payload, waiting if the upstream does not have it yet.
+   *
+   * On the direct plane this always succeeds first time: the site *is* the
+   * upstream, so it seals when asked. Through a relay the two are different
+   * parties — the site must be told which device claimed before it can seal to
+   * it — and `not-ready` is a normal answer for as long as that takes.
+   *
+   * Discovered by the skeleton relay, which is the reason it exists: the
+   * original claim-then-fetch had no wait here, so the first relayed job was
+   * claimed, refused with a 409 the daemon treated as a rejection, and
+   * abandoned while still holding a perfectly good lease.
+   *
+   * Bounded, and by the lease rather than by a retry count: waiting past our
+   * own lease means racing whoever gets the job next. Returning `null` gives
+   * the job up quietly — the upstream's `awaiting-payload` timer will requeue
+   * it, and the stub was never lost.
+   */
+  async #fetchWhenSealed(job) {
+    const deadline = Math.min(job.lease.expiresAt, this.#now() + 3e4);
+    let delay = 50;
+    for (; ; ) {
+      try {
+        return await this.#options.client.fetch({
+          runnerId: this.#options.runnerId,
+          jobId: job.id,
+          leaseId: job.lease.id
+        });
+      } catch (error) {
+        const notReady = error instanceof ClientError && error.kind === "not-ready";
+        if (!notReady) throw error;
+        if (this.#now() + delay >= deadline) {
+          this.#options.onEvent?.({
+            type: "error",
+            message: `gave up waiting for the payload of ${job.id}`
+          });
+          return null;
+        }
+        await new Promise((resolve) => setTimeout(resolve, delay));
+        delay = Math.min(delay * 2, 1e3);
+      }
+    }
+  }
   /**
    * Open work sealed to this machine, or refuse to run it.
    *
@@ -2991,7 +3033,7 @@ async function main(argv) {
 }
 
 // src/index.ts
-var DAEMON_VERSION = "0.1.0-alpha.4";
+var DAEMON_VERSION = "0.1.0-alpha.6";
 function daemonVersion() {
   return {
     daemon: DAEMON_VERSION,
@@ -3044,4 +3086,4 @@ export {
   runCli,
   main
 };
-//# sourceMappingURL=chunk-CKWS6NEW.js.map
+//# sourceMappingURL=chunk-EU4PO36M.js.map