@genroc/eval-node 0.0.0-edge.dcfec388 → 0.0.0-edge.df9d1f1d

package/README.md

@@ -109,20 +109,19 @@ match. Retrying does not help with any of them, because the same code fails the
 | code | when |
 |---|---|
 | `threw` | the script threw or rejected |
-| `timeout` | it ran past the task's `timeout` (5000 ms when the task has none) |
+| `timeout` | it ran past `timeout_ms` (default 5000) |
 | `compile_error` | the module does not parse, or has no default-export function |
 | `nonserializable` | the return value cannot be serialized to JSON (a cycle, a `BigInt`) |
 | `exited` | the script called `process.exit()` |
 
-`error.data` is `{name, stack?}`. `name` is the error's class name, so throwing your own
-`class LimitExceeded extends Error {}` tells your failures apart; setting `e.name` overrides it,
-and a thrown non-`Error` is `Thrown`. The stack is mapped back to your source lines.
+`error.data` is `{name, stack?}`. Set `e.name = "LimitExceeded"` in the script to tell your
+own failures apart. The stack is mapped back to your source lines.
 
 If the worker itself faults, it releases the task for another worker and reports no error.
 
-The task's `timeout` is the script's budget, waiting in the queue included. The worker stops
-the script just before it and answers `timeout`, so `external.timeout` means no worker answered
-at all. With `script-node`, set it through the `timeout_ms` input.
+Keep the task's `timeout` above `timeout_ms`. Otherwise a slow script ends with
+`external.timeout`, and you cannot tell it from having no worker at all. The `script-node`
+template already does this.
 
 ## Limits
 

package/dist/realm.js

@@ -36,10 +36,7 @@ function scriptStack(err) {
 }
 function describe(err, kind) {
     if (err instanceof Error) {
-        // A subclass inherits `name` from Error.prototype, so `class LimitExceeded extends Error {}`
-        // would report "Error"; a name the script set still wins.
-        const name = err.name === "Error" && err.constructor.name ? err.constructor.name : err.name;
-        return { kind, name, message: err.message, stack: scriptStack(err) };
+        return { kind, name: err.name, message: err.message, stack: scriptStack(err) };
     }
     // A script may throw a non-Error (`throw {code: "x"}`), so name/message must not assume one.
     return { kind, name: "Thrown", message: safeText(err) };

package/dist/worker.js

@@ -19,9 +19,6 @@ const POLL_MS = Number(process.env.POLL_MS ?? 250);
 // should return its task quickly rather than holding it for the whole budget.
 const LEASE_MS = Number(process.env.LEASE_MS ?? 30_000);
 const RENEW_MS = Math.max(1_000, Math.floor(LEASE_MS / 3));
-// The answer has to land before the task's deadline fires, or a script that overran surfaces as
-// external.timeout ("nobody answered") rather than as its own `timeout`.
-const ANSWER_MARGIN_MS = 250;
 const PROCESS_FILTER = process.env.PROCESS ?? "";
 const TASK_FILTER = process.env.TASK ?? "";
 // Large values arrive as refs; a bundle is one object per definition version, fetched once.
@@ -97,13 +94,17 @@ async function call(path, body) {
 }
 /** A task input that is not an EvalRequest is the definition's fault, reported as compile_error:
  *  the nearest permanent kind, since no retry fixes the definition. */
-function asEvalRequest(input, timeoutMs) {
+function asEvalRequest(input) {
     if (typeof input !== "object" || input === null)
         return "the task input is not an object";
     const r = input;
     if (typeof r.code !== "string")
         return "the task input has no `code` string";
-    return { code: r.code, input: r.input, timeout_ms: timeoutMs };
+    return {
+        code: r.code,
+        input: r.input,
+        timeout_ms: typeof r.timeout_ms === "number" ? r.timeout_ms : undefined,
+    };
 }
 const inFlight = new Map();
 let running = true;
@@ -160,9 +161,6 @@ async function answer(token, outcome) {
     await release(token);
 }
 async function run(job, signal) {
-    // The task's own timeout is the script's budget. Taken on arrival, because fetching the
-    // objects below spends the same deadline.
-    const due = job.deadline_in_ms === undefined ? undefined : performance.now() + job.deadline_in_ms;
     let resolved;
     try {
         resolved = await resolveObjects(job);
@@ -174,16 +172,7 @@ async function run(job, signal) {
         await release(job.token);
         return;
     }
-    let timeoutMs;
-    if (due !== undefined) {
-        timeoutMs = Math.floor(due - performance.now() - ANSWER_MARGIN_MS);
-        if (timeoutMs <= 0) {
-            // Too late to run anything: the deadline fires as external.timeout, which is the truth.
-            await release(job.token);
-            return;
-        }
-    }
-    const req = asEvalRequest(resolved, timeoutMs);
+    const req = asEvalRequest(resolved);
     if (typeof req === "string") {
         await answer(job.token, {
             error: { code: "compile_error", message: req, data: { name: "BadTaskInput" } },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@genroc/eval-node",
-  "version": "0.0.0-edge.dcfec388",
+  "version": "0.0.0-edge.df9d1f1d",
   "type": "module",
   "engines": {
     "node": ">=24"

package/realm.ts

@@ -35,10 +35,7 @@ function scriptStack(err: unknown): string | undefined {
 
 function describe(err: unknown, kind: FailureKind): EvalFailure {
   if (err instanceof Error) {
-    // A subclass inherits `name` from Error.prototype, so `class LimitExceeded extends Error {}`
-    // would report "Error"; a name the script set still wins.
-    const name = err.name === "Error" && err.constructor.name ? err.constructor.name : err.name;
-    return { kind, name, message: err.message, stack: scriptStack(err) };
+    return { kind, name: err.name, message: err.message, stack: scriptStack(err) };
   }
   // A script may throw a non-Error (`throw {code: "x"}`), so name/message must not assume one.
   return { kind, name: "Thrown", message: safeText(err) };

package/worker.ts

@@ -22,9 +22,6 @@ const POLL_MS = Number(process.env.POLL_MS ?? 250);
 // should return its task quickly rather than holding it for the whole budget.
 const LEASE_MS = Number(process.env.LEASE_MS ?? 30_000);
 const RENEW_MS = Math.max(1_000, Math.floor(LEASE_MS / 3));
-// The answer has to land before the task's deadline fires, or a script that overran surfaces as
-// external.timeout ("nobody answered") rather than as its own `timeout`.
-const ANSWER_MARGIN_MS = 250;
 const PROCESS_FILTER = process.env.PROCESS ?? "";
 const TASK_FILTER = process.env.TASK ?? "";
 
@@ -37,7 +34,6 @@ type QueueTask = {
   external_input: unknown;
   objects?: ObjectEntry[];
   raises?: Record<string, unknown>;
-  deadline_in_ms?: number;
 };
 
 // Large values arrive as refs; a bundle is one object per definition version, fetched once.
@@ -110,11 +106,15 @@ async function call(path: string, body: unknown): Promise<{ ok: boolean; status:
 
 /** A task input that is not an EvalRequest is the definition's fault, reported as compile_error:
  *  the nearest permanent kind, since no retry fixes the definition. */
-function asEvalRequest(input: unknown, timeoutMs: number | undefined): EvalRequest | string {
+function asEvalRequest(input: unknown): EvalRequest | string {
   if (typeof input !== "object" || input === null) return "the task input is not an object";
   const r = input as Record<string, unknown>;
   if (typeof r.code !== "string") return "the task input has no `code` string";
-  return { code: r.code, input: r.input, timeout_ms: timeoutMs };
+  return {
+    code: r.code,
+    input: r.input,
+    timeout_ms: typeof r.timeout_ms === "number" ? r.timeout_ms : undefined,
+  };
 }
 
 /** genroc cannot call us, so a cancellation arrives on our own heartbeat and reaches the
@@ -183,9 +183,6 @@ async function answer(token: string, outcome: Record<string, unknown>): Promise<
 }
 
 async function run(job: QueueTask, signal: AbortSignal): Promise<void> {
-  // The task's own timeout is the script's budget. Taken on arrival, because fetching the
-  // objects below spends the same deadline.
-  const due = job.deadline_in_ms === undefined ? undefined : performance.now() + job.deadline_in_ms;
   let resolved: unknown;
   try {
     resolved = await resolveObjects(job);
@@ -196,16 +193,7 @@ async function run(job: QueueTask, signal: AbortSignal): Promise<void> {
     await release(job.token);
     return;
   }
-  let timeoutMs: number | undefined;
-  if (due !== undefined) {
-    timeoutMs = Math.floor(due - performance.now() - ANSWER_MARGIN_MS);
-    if (timeoutMs <= 0) {
-      // Too late to run anything: the deadline fires as external.timeout, which is the truth.
-      await release(job.token);
-      return;
-    }
-  }
-  const req = asEvalRequest(resolved, timeoutMs);
+  const req = asEvalRequest(resolved);
   if (typeof req === "string") {
     await answer(job.token, {
       error: { code: "compile_error", message: req, data: { name: "BadTaskInput" } },