@nanobpm/nano-workforce 0.36.0 → 0.37.0

package/app/reviewWait.test.ts

@@ -1,8 +1,9 @@
 // Contract for the review-wait liveness policy — the ISO-8601 timeout validation handed to the
 // process's timer catch, and the Copilot re-request nudge cooldown. Both are env-driven, so a
 // blank/malformed operator value must fall back to a sane default rather than deploy an
-// uninterpretable timer or a runaway nudge interval. Run with `deno test -A`.
-import { assertEquals } from "jsr:@std/assert@1";
+// uninterpretable timer or a runaway nudge interval. Run with `node --test`.
+import { test } from "node:test";
+import { assertEquals } from "#test-assert";
 import {
   clampNudgeMinutes,
   DEFAULT_REVIEW_NUDGE_MINUTES,
@@ -11,7 +12,7 @@ import {
   reviewWaitTimeout,
 } from "./reviewWait.ts";
 
-Deno.test("reviewWaitTimeout: blank / absent / malformed → default", () => {
+test("reviewWaitTimeout: blank / absent / malformed → default", () => {
   assertEquals(reviewWaitTimeout(undefined), DEFAULT_REVIEW_WAIT_TIMEOUT);
   assertEquals(reviewWaitTimeout(""), DEFAULT_REVIEW_WAIT_TIMEOUT);
   assertEquals(reviewWaitTimeout("   "), DEFAULT_REVIEW_WAIT_TIMEOUT);
@@ -21,7 +22,7 @@ Deno.test("reviewWaitTimeout: blank / absent / malformed → default", () => {
   assertEquals(reviewWaitTimeout("garbage"), DEFAULT_REVIEW_WAIT_TIMEOUT);
 });
 
-Deno.test("reviewWaitTimeout: a valid ISO-8601 duration is honoured and upper-cased", () => {
+test("reviewWaitTimeout: a valid ISO-8601 duration is honoured and upper-cased", () => {
   assertEquals(reviewWaitTimeout("PT30M"), "PT30M");
   assertEquals(reviewWaitTimeout("pt30m"), "PT30M");
   assertEquals(reviewWaitTimeout(" PT1H30M "), "PT1H30M");
@@ -30,12 +31,12 @@ Deno.test("reviewWaitTimeout: a valid ISO-8601 duration is honoured and upper-ca
   assertEquals(reviewWaitTimeout("P1DT2H"), "P1DT2H");
 });
 
-Deno.test("reviewWaitTimeout: a custom fallback is used when the value is invalid", () => {
+test("reviewWaitTimeout: a custom fallback is used when the value is invalid", () => {
   assertEquals(reviewWaitTimeout("nope", "PT10M"), "PT10M");
   assertEquals(reviewWaitTimeout(undefined, "PT10M"), "PT10M");
 });
 
-Deno.test("clampNudgeMinutes: blank / absent / non-numeric → fallback", () => {
+test("clampNudgeMinutes: blank / absent / non-numeric → fallback", () => {
   assertEquals(clampNudgeMinutes(""), DEFAULT_REVIEW_NUDGE_MINUTES);
   assertEquals(clampNudgeMinutes("   "), DEFAULT_REVIEW_NUDGE_MINUTES);
   assertEquals(clampNudgeMinutes(undefined), DEFAULT_REVIEW_NUDGE_MINUTES);
@@ -44,27 +45,27 @@ Deno.test("clampNudgeMinutes: blank / absent / non-numeric → fallback", () =>
   assertEquals(clampNudgeMinutes(NaN), DEFAULT_REVIEW_NUDGE_MINUTES);
 });
 
-Deno.test("clampNudgeMinutes: a valid positive value (string or number) is honoured", () => {
+test("clampNudgeMinutes: a valid positive value (string or number) is honoured", () => {
   assertEquals(clampNudgeMinutes("10"), 10);
   assertEquals(clampNudgeMinutes(15), 15);
   assertEquals(clampNudgeMinutes(" 7 "), 7);
 });
 
-Deno.test("clampNudgeMinutes: fractional values are truncated", () => {
+test("clampNudgeMinutes: fractional values are truncated", () => {
   assertEquals(clampNudgeMinutes("5.9"), 5);
   assertEquals(clampNudgeMinutes(3.2), 3);
 });
 
-Deno.test("clampNudgeMinutes: zero and negatives fall back (a 0 cooldown would hammer the API)", () => {
+test("clampNudgeMinutes: zero and negatives fall back (a 0 cooldown would hammer the API)", () => {
   assertEquals(clampNudgeMinutes(0), DEFAULT_REVIEW_NUDGE_MINUTES);
   assertEquals(clampNudgeMinutes("-4"), DEFAULT_REVIEW_NUDGE_MINUTES);
 });
 
-Deno.test("clampNudgeMinutes: above the ceiling is clamped, not rejected", () => {
+test("clampNudgeMinutes: above the ceiling is clamped, not rejected", () => {
   assertEquals(clampNudgeMinutes(100_000), MAX_REVIEW_NUDGE_MINUTES);
   assertEquals(clampNudgeMinutes(String(MAX_REVIEW_NUDGE_MINUTES + 1)), MAX_REVIEW_NUDGE_MINUTES);
 });
 
-Deno.test("clampNudgeMinutes: an oversized fallback is itself clamped to the ceiling", () => {
+test("clampNudgeMinutes: an oversized fallback is itself clamped to the ceiling", () => {
   assertEquals(clampNudgeMinutes("", MAX_REVIEW_NUDGE_MINUTES + 50), MAX_REVIEW_NUDGE_MINUTES);
 });

package/app/rounds.test.ts

@@ -1,10 +1,11 @@
 // Contract for clampRounds — the per-submit round-cap coercion. The submit form renders every
 // field as a text input (the runtime hardcodes type="text"), so the override arrives as a string
-// and a blank field must fall back to the fleet default. Run with `deno test`.
-import { assertEquals } from "jsr:@std/assert@1";
+// and a blank field must fall back to the fleet default. Run with `node --test`.
+import { test } from "node:test";
+import { assertEquals } from "#test-assert";
 import { clampCiFixBudget, clampRounds, MAX_CI_FIX_CEILING, MAX_ROUNDS_CEILING } from "./rounds.ts";
 
-Deno.test("blank / absent / non-numeric → fallback", () => {
+test("blank / absent / non-numeric → fallback", () => {
   assertEquals(clampRounds("", 20), 20);
   assertEquals(clampRounds("   ", 20), 20);
   assertEquals(clampRounds(undefined, 20), 20);
@@ -13,28 +14,28 @@ Deno.test("blank / absent / non-numeric → fallback", () => {
   assertEquals(clampRounds(NaN, 20), 20);
 });
 
-Deno.test("a valid positive value (string or number) is honoured", () => {
+test("a valid positive value (string or number) is honoured", () => {
   assertEquals(clampRounds("5", 20), 5);
   assertEquals(clampRounds(30, 20), 30);
   assertEquals(clampRounds(" 12 ", 20), 12);
 });
 
-Deno.test("fractional values are truncated to a whole round", () => {
+test("fractional values are truncated to a whole round", () => {
   assertEquals(clampRounds("7.9", 20), 7);
   assertEquals(clampRounds(3.2, 20), 3);
 });
 
-Deno.test("zero and negatives fall back (a cap < 1 would escalate before round 1)", () => {
+test("zero and negatives fall back (a cap < 1 would escalate before round 1)", () => {
   assertEquals(clampRounds(0, 20), 20);
   assertEquals(clampRounds("-4", 20), 20);
 });
 
-Deno.test("above the ceiling is clamped, not rejected", () => {
+test("above the ceiling is clamped, not rejected", () => {
   assertEquals(clampRounds(1000, 20), MAX_ROUNDS_CEILING);
   assertEquals(clampRounds(String(MAX_ROUNDS_CEILING + 1), 20), MAX_ROUNDS_CEILING);
 });
 
-Deno.test("an oversized fallback is itself clamped to the ceiling", () => {
+test("an oversized fallback is itself clamped to the ceiling", () => {
   // A blank/invalid input falls back — but the fallback must not bypass the ceiling.
   assertEquals(clampRounds("", MAX_ROUNDS_CEILING + 50), MAX_ROUNDS_CEILING);
   assertEquals(clampRounds("   ", MAX_ROUNDS_CEILING + 1), MAX_ROUNDS_CEILING);
@@ -45,7 +46,7 @@ Deno.test("an oversized fallback is itself clamped to the ceiling", () => {
 // clampCiFixBudget — the merge stage's CI-fix attempt budget (NANO_PR_MAX_CI_FIX_ROUNDS). Unlike
 // clampRounds it ALLOWS 0, because 0 is a meaningful setting: disable auto-fix so a blocked PR
 // escalates to a human immediately.
-Deno.test("ci-fix budget: blank / non-numeric / negative → fallback", () => {
+test("ci-fix budget: blank / non-numeric / negative → fallback", () => {
   assertEquals(clampCiFixBudget("", 3), 3);
   assertEquals(clampCiFixBudget("   ", 3), 3);
   assertEquals(clampCiFixBudget(undefined, 3), 3);
@@ -55,20 +56,20 @@ Deno.test("ci-fix budget: blank / non-numeric / negative → fallback", () => {
   assertEquals(clampCiFixBudget("-1", 3), 3);
 });
 
-Deno.test("ci-fix budget: 0 is honoured (disables auto-fix), unlike clampRounds", () => {
+test("ci-fix budget: 0 is honoured (disables auto-fix), unlike clampRounds", () => {
   assertEquals(clampCiFixBudget(0, 3), 0);
   assertEquals(clampCiFixBudget("0", 3), 0);
   // Contrast: the round cap treats 0 as invalid and falls back.
   assertEquals(clampRounds(0, 3), 3);
 });
 
-Deno.test("ci-fix budget: a valid positive value is honoured and truncated", () => {
+test("ci-fix budget: a valid positive value is honoured and truncated", () => {
   assertEquals(clampCiFixBudget("5", 3), 5);
   assertEquals(clampCiFixBudget(2, 3), 2);
   assertEquals(clampCiFixBudget("4.8", 3), 4);
 });
 
-Deno.test("ci-fix budget: above the ceiling is clamped; oversized fallback too", () => {
+test("ci-fix budget: above the ceiling is clamped; oversized fallback too", () => {
   assertEquals(clampCiFixBudget(1000, 3), MAX_CI_FIX_CEILING);
   assertEquals(clampCiFixBudget("", MAX_CI_FIX_CEILING + 5), MAX_CI_FIX_CEILING);
 });

package/app/service.test.ts

@@ -5,30 +5,25 @@
 // dead "(no question provided)" question on the re-opened PR (the same stale-row class the plan
 // loop already guards in `startPlan`). Drives `submitPr` against an in-memory data layer with the
 // GitHub transport forced off so it is hermetic.
-import { assertEquals } from "jsr:@std/assert@1";
+import { test } from "node:test";
+import { assertEquals } from "#test-assert";
 import { pollIncidentsImpl, submitPr } from "./service.ts";
 
-// deno-lint-ignore no-explicit-any
 function memTable(rows: any[], key: string) {
   return {
-    // deno-lint-ignore no-explicit-any
     get: (k: any) => Promise.resolve(rows.find((r) => r[key] === k) ?? null),
     all: () => Promise.resolve([...rows]),
-    // deno-lint-ignore no-explicit-any
     find: (q: any) =>
       Promise.resolve(rows.filter((r) => Object.entries(q).every(([f, v]) => r[f] === v))),
-    // deno-lint-ignore no-explicit-any
     insert: (r: any) => {
       rows.push(r);
       return Promise.resolve(r);
     },
-    // deno-lint-ignore no-explicit-any
     update: (k: any, patch: any) => {
       const r = rows.find((x) => x[key] === k);
       if (r) Object.assign(r, patch);
       return Promise.resolve(r);
     },
-    // deno-lint-ignore no-explicit-any
     delete: (k: any) => {
       for (let i = rows.length - 1; i >= 0; i--) if (rows[i][key] === k) rows.splice(i, 1);
       return Promise.resolve();
@@ -37,18 +32,18 @@ function memTable(rows: any[], key: string) {
 }
 
 function withGithubOff(run: () => Promise<void>): Promise<void> {
-  const prevMode = Deno.env.get("NANO_PR_GITHUB_TRANSPORT");
-  const prevTok = Deno.env.get("GITHUB_TOKEN");
-  Deno.env.set("NANO_PR_GITHUB_TRANSPORT", "token"); // no token below -> fetchPrMeta returns null
-  Deno.env.delete("GITHUB_TOKEN");
+  const prevMode = process.env["NANO_PR_GITHUB_TRANSPORT"];
+  const prevTok = process.env["GITHUB_TOKEN"];
+  process.env["NANO_PR_GITHUB_TRANSPORT"] = "token"; // no token below -> fetchPrMeta returns null
+  delete process.env["GITHUB_TOKEN"];
   return run().finally(() => {
-    if (prevMode !== undefined) Deno.env.set("NANO_PR_GITHUB_TRANSPORT", prevMode);
-    else Deno.env.delete("NANO_PR_GITHUB_TRANSPORT");
-    if (prevTok !== undefined) Deno.env.set("GITHUB_TOKEN", prevTok);
+    if (prevMode !== undefined) process.env["NANO_PR_GITHUB_TRANSPORT"] = prevMode;
+    else delete process.env["NANO_PR_GITHUB_TRANSPORT"];
+    if (prevTok !== undefined) process.env["GITHUB_TOKEN"] = prevTok;
   });
 }
 
-Deno.test("re-submit of a cancelled PR clears stale open escalations + the denormalised pointer", async () => {
+test("re-submit of a cancelled PR clears stale open escalations + the denormalised pointer", async () => {
   await withGithubOff(async () => {
     const PR_KEY = "owner/repo#42";
     const stores: Record<string, { rows: unknown[]; key: string }> = {
@@ -74,11 +69,9 @@ Deno.test("re-submit of a cancelled PR clears stale open escalations + the denor
     };
     const data = {
       table: (name: string, key: string) => memTable(stores[name]?.rows ?? [], stores[name]?.key ?? key),
-      // deno-lint-ignore no-explicit-any
     } as any;
     const engine = {
       createInstance: () => Promise.resolve({ processInstanceKey: "PI-9" }),
-      // deno-lint-ignore no-explicit-any
     } as any;
 
     await submitPr(data, engine, {
@@ -125,7 +118,7 @@ function incidentFetch(byInstance: Record<string, unknown[]>) {
   };
 }
 
-Deno.test("pollIncidents mirrors an ACTIVE incident onto the PR row, then clears it, leaving status untouched", async () => {
+test("pollIncidents mirrors an ACTIVE incident onto the PR row, then clears it, leaving status untouched", async () => {
   const row = {
     pr_key: "owner/repo#7",
     repo: "owner/repo",
@@ -141,7 +134,6 @@ Deno.test("pollIncidents mirrors an ACTIVE incident onto the PR row, then clears
   };
   const data = {
     table: (name: string, key: string) => memTable(stores[name]?.rows ?? [], stores[name]?.key ?? key),
-    // deno-lint-ignore no-explicit-any
   } as any;
   const headers = { "content-type": "application/json" };
 
@@ -173,7 +165,7 @@ Deno.test("pollIncidents mirrors an ACTIVE incident onto the PR row, then clears
   assertEquals(row.status, "converging");
 });
 
-Deno.test("pollIncidents never queries a PR with no live instance and clears any stale incident", async () => {
+test("pollIncidents never queries a PR with no live instance and clears any stale incident", async () => {
   const noKey = {
     pr_key: "owner/repo#8",
     status: "converging",
@@ -195,7 +187,6 @@ Deno.test("pollIncidents never queries a PR with no live instance and clears any
   };
   const data = {
     table: (name: string, key: string) => memTable(stores[name]?.rows ?? [], stores[name]?.key ?? key),
-    // deno-lint-ignore no-explicit-any
   } as any;
   const headers = { "content-type": "application/json" };
 
@@ -215,7 +206,7 @@ Deno.test("pollIncidents never queries a PR with no live instance and clears any
   assertEquals(terminal.incident_message, null);
 });
 
-Deno.test("pollIncidents picks the oldest incident by creationTime, sorting a missing timestamp last", async () => {
+test("pollIncidents picks the oldest incident by creationTime, sorting a missing timestamp last", async () => {
   const row = {
     pr_key: "owner/repo#11",
     status: "converging",
@@ -229,7 +220,6 @@ Deno.test("pollIncidents picks the oldest incident by creationTime, sorting a mi
   };
   const data = {
     table: (name: string, key: string) => memTable(stores[name]?.rows ?? [], stores[name]?.key ?? key),
-    // deno-lint-ignore no-explicit-any
   } as any;
   const headers = { "content-type": "application/json" };
 
@@ -259,7 +249,7 @@ Deno.test("pollIncidents picks the oldest incident by creationTime, sorting a mi
 // source also keeps the returned key aligned with the DB-persisted `String(...)` value and dodges
 // JS 53-bit precision limits for large 64-bit keys (which is why keys travel as strings in
 // practice). `submitPr` must stringify it both in the returned body and the persisted row.
-Deno.test("submitPr stringifies a numeric processInstanceKey (contract: string | null)", async () => {
+test("submitPr stringifies a numeric processInstanceKey (contract: string | null)", async () => {
   await withGithubOff(async () => {
     const PR_KEY = "owner/repo#7";
     const stores: Record<string, { rows: unknown[]; key: string }> = {
@@ -269,14 +259,12 @@ Deno.test("submitPr stringifies a numeric processInstanceKey (contract: string |
     };
     const data = {
       table: (name: string, key: string) => memTable(stores[name]?.rows ?? [], stores[name]?.key ?? key),
-      // deno-lint-ignore no-explicit-any
     } as any;
     const engine = {
       // A large key delivered as a JS number — the exact case that breaks dev response validation
       // (number vs the `string | null` contract). Kept within MAX_SAFE_INTEGER so the fixture
       // itself is exact; true 64-bit keys travel as strings for the same precision reason.
       createInstance: () => Promise.resolve({ processInstanceKey: 2251799813685249 }),
-      // deno-lint-ignore no-explicit-any
     } as any;
 
     const res = await submitPr(data, engine, {
@@ -286,7 +274,6 @@ Deno.test("submitPr stringifies a numeric processInstanceKey (contract: string |
       prKey: PR_KEY,
     });
 
-    // deno-lint-ignore no-explicit-any
     const processKey = (res as any).processKey;
     assertEquals(typeof processKey, "string");
     assertEquals(processKey, "2251799813685249");

package/app/taskDelta.test.ts

@@ -1,5 +1,6 @@
 // Unit tests for the structured scope/impl-change report (D5, issue #55 / #49).
-import { assert, assertEquals } from "jsr:@std/assert@1";
+import { test } from "node:test";
+import { assert, assertEquals } from "#test-assert";
 import type { DataLayer } from "@nanobpm/urban";
 import {
   aggregateEpicDeltas,
@@ -11,49 +12,39 @@ import {
 
 // A tiny in-memory stand-in for the record gateway (insert/find/findOne/update/delete), mirroring
 // the fake-app style used across the app tests (see app/blackboard.test.ts).
-// deno-lint-ignore no-explicit-any
 function memData(): { data: DataLayer; stores: Record<string, any[]> } {
-  // deno-lint-ignore no-explicit-any
   const stores: Record<string, any[]> = {};
   const seq: Record<string, number> = {};
   function tbl(name: string, pk = "id") {
-    // deno-lint-ignore no-explicit-any
     const rows = (stores[name] ??= [] as any[]);
-    // deno-lint-ignore no-explicit-any
     const match = (r: any, where: any) => Object.entries(where).every(([k, v]) => r[k] === v);
     return {
-      // deno-lint-ignore no-explicit-any require-await
       async insert(row: any) {
         const id = (seq[name] = (seq[name] ?? 0) + 1);
         rows.push(pk === "id" ? { id, ...row } : { ...row });
         return pk === "id" ? id : row[pk];
       },
-      // deno-lint-ignore no-explicit-any require-await
       async find(where: any = {}) {
         return rows.filter((r) => match(r, where));
       },
-      // deno-lint-ignore no-explicit-any require-await
       async findOne(where: any = {}) {
         return rows.find((r) => match(r, where));
       },
-      // deno-lint-ignore no-explicit-any require-await
       async update(id: any, patch: any) {
         const r = rows.find((row) => row[pk] === id);
         if (r) Object.assign(r, patch);
       },
-      // deno-lint-ignore no-explicit-any require-await
       async delete(id: any) {
         const i = rows.findIndex((row) => row[pk] === id);
         if (i >= 0) rows.splice(i, 1);
       },
     };
   }
-  // deno-lint-ignore no-explicit-any
   const data = { table: (n: string, pk?: string) => tbl(n, pk) } as any as DataLayer;
   return { data, stores };
 }
 
-Deno.test("parseTaskDelta: trims, dedupes arrays, and drops empties", () => {
+test("parseTaskDelta: trims, dedupes arrays, and drops empties", () => {
   const d = parseTaskDelta({
     contractChange: "  new signature  ",
     newlyTouches: ["a.rs", " a.rs ", "", "b.rs"],
@@ -67,14 +58,14 @@ Deno.test("parseTaskDelta: trims, dedupes arrays, and drops empties", () => {
   assertEquals(d.constraint, undefined, "a blank constraint is dropped");
 });
 
-Deno.test("parseTaskDelta: a delta with nothing actionable is null", () => {
+test("parseTaskDelta: a delta with nothing actionable is null", () => {
   assertEquals(parseTaskDelta(undefined), null);
   assertEquals(parseTaskDelta({}), null);
   assertEquals(parseTaskDelta({ newlyTouches: [], affectsTasks: [], contractChange: "  " }), null);
   assertEquals(parseTaskDelta("not an object"), null);
 });
 
-Deno.test("recordTaskDelta: upserts per (plan, task) — a resume overwrites, not duplicates", async () => {
+test("recordTaskDelta: upserts per (plan, task) — a resume overwrites, not duplicates", async () => {
   const { data, stores } = memData();
   const first = await recordTaskDelta(data, "o/r#1", "gap-2", {
     newlyTouches: ["a.rs"],
@@ -99,7 +90,7 @@ Deno.test("recordTaskDelta: upserts per (plan, task) — a resume overwrites, no
   assertEquals(entry.wave, 1);
 });
 
-Deno.test("readTaskDeltas: scoped to a plan, in write order, arrays decoded", async () => {
+test("readTaskDeltas: scoped to a plan, in write order, arrays decoded", async () => {
   const { data } = memData();
   await recordTaskDelta(data, "o/r#1", "gap-2", { newlyTouches: ["a.rs"], affectsTasks: [] });
   await recordTaskDelta(data, "o/r#1", "gap-8", { newlyTouches: [], affectsTasks: ["gap-2"], constraint: "x" });
@@ -111,7 +102,7 @@ Deno.test("readTaskDeltas: scoped to a plan, in write order, arrays decoded", as
   assertEquals(entries[1].affectsTasks, ["gap-2"]);
 });
 
-Deno.test("aggregateEpicDeltas: unions touched files + affected tasks, lists changes/constraints", async () => {
+test("aggregateEpicDeltas: unions touched files + affected tasks, lists changes/constraints", async () => {
   const { data } = memData();
   await recordTaskDelta(data, "p", "gap-2", {
     newlyTouches: ["engine/state.rs"],
@@ -132,7 +123,7 @@ Deno.test("aggregateEpicDeltas: unions touched files + affected tasks, lists cha
   assertEquals(report.deltas.length, 2);
 });
 
-Deno.test("clearTaskDeltas: drops a plan's whole set (re-plan cleanup), leaving other plans intact", async () => {
+test("clearTaskDeltas: drops a plan's whole set (re-plan cleanup), leaving other plans intact", async () => {
   const { data, stores } = memData();
   await recordTaskDelta(data, "p", "gap-2", { newlyTouches: ["a.rs"], affectsTasks: [] });
   await recordTaskDelta(data, "p", "gap-8", { newlyTouches: ["b.rs"], affectsTasks: [] });

package/app/trialMerge.test.ts

@@ -1,13 +1,14 @@
-import { assertEquals } from "jsr:@std/assert@1";
+import { test } from "node:test";
+import { assertEquals } from "#test-assert";
 import { shouldRunTrialMerge, trialMergeDecision } from "./trialMerge.ts";
 
-Deno.test("trialMergeDecision only escalates clean-merge suite failures", () => {
+test("trialMergeDecision only escalates clean-merge suite failures", () => {
   assertEquals(trialMergeDecision("clean"), "proceed");
   assertEquals(trialMergeDecision("merge-conflict"), "proceed");
   assertEquals(trialMergeDecision("suite-failed"), "escalate");
 });
 
-Deno.test("shouldRunTrialMerge skips lone heads and mergify queues", () => {
+test("shouldRunTrialMerge skips lone heads and mergify queues", () => {
   assertEquals(shouldRunTrialMerge(0, { land: { method: "gh-merge" } }), false);
   assertEquals(shouldRunTrialMerge(1, { land: { method: "gh-merge" } }), false);
   assertEquals(shouldRunTrialMerge(2, { land: { method: "mergify-queue" } }), false);

package/app/version.ts

@@ -5,7 +5,7 @@
 // inspecting the working tree at runtime. This module gathers that identity — the app's package
 // version, the resolved `@nanobpm/urban` version, the git commit (read from `.git`, handling both
 // an ordinary `.git` directory and the `gitdir:` file pointer used by worktrees/submodules, with
-// an env override for detached deploys), plus the Node/Deno runtime, pid and start time — so an operator
+// an env override for detached deploys), plus the Node runtime, pid and start time — so an operator
 // debugging a stuck instance can confirm the process is on the code they think it is.
 //
 // Every probe is best-effort: a missing file or unavailable `.git` yields `null` for that field
@@ -39,20 +39,11 @@ function readJson(path: string): Record<string, unknown> | null {
 }
 
 /**
- * Read an env var across runtimes: Node exposes `process.env`; Deno may not populate it, so fall
- * back to `Deno.env.get` (guarded — reading env can throw without `--allow-env`).
+ * Read an env var from `process.env`, trimmed; returns null when unset or blank.
  */
 export function envVar(name: string): string | null {
-  const fromProcess = globalThis.process?.env?.[name];
+  const fromProcess = process.env[name];
   if (typeof fromProcess === "string" && fromProcess.trim()) return fromProcess.trim();
-  // biome-ignore lint/plugin: runtime/framework contract boundary for external data shape
-  const deno = (globalThis as { Deno?: { env?: { get?(k: string): string | undefined } } }).Deno;
-  try {
-    const fromDeno = deno?.env?.get?.(name);
-    if (typeof fromDeno === "string" && fromDeno.trim()) return fromDeno.trim();
-  } catch {
-    // Env access denied — treat as unset.
-  }
   return null;
 }
 
@@ -159,13 +150,8 @@ function gitBranch(): string | null {
 }
 
 function runtime(): string {
-  const proc = globalThis.process;
-  // Deno exposes `Deno.version.deno`; Node exposes `process.version` (e.g. "v24.15.0").
-  // biome-ignore lint/plugin: runtime/framework contract boundary for external data shape
-  const deno = (globalThis as { Deno?: { version?: { deno?: string } } }).Deno;
-  if (deno?.version?.deno) return `deno ${deno.version.deno}`;
-  if (proc?.version) return `node ${proc.version}`;
-  return "unknown";
+  // Node exposes `process.version` (e.g. "v24.15.0").
+  return process.version ? `node ${process.version}` : "unknown";
 }
 
 export interface VersionInfo {
@@ -185,7 +171,6 @@ export interface VersionInfo {
  * tree ONCE at module load rather than re-reading files (`.git`, package.jsons) on every request.
  */
 const STATIC: Omit<VersionInfo, "uptimeSeconds"> = (() => {
-  const proc = globalThis.process;
   const pkg = appPackage();
   return Object.freeze({
     name: appName(pkg),
@@ -194,7 +179,7 @@ const STATIC: Omit<VersionInfo, "uptimeSeconds"> = (() => {
     gitSha: gitSha(),
     gitBranch: gitBranch(),
     runtime: runtime(),
-    pid: typeof proc?.pid === "number" ? proc.pid : null,
+    pid: typeof process.pid === "number" ? process.pid : null,
     startedAt: STARTED_AT.toISOString(),
   });
 })();

package/app/waves.test.ts

@@ -1,20 +1,21 @@
-// Red/green regression for the plan levelizer (issue #20). Run with `deno test`.
+// Red/green regression for the plan levelizer (issue #20). Run with `node --test`.
 //
-// One `Deno.test` = one named property of computeWaves. These encode the wave
+// One `test` = one named property of computeWaves. These encode the wave
 // contract: independent tasks share wave 0 (all-parallel), a chain steps 0,1,2…
 // (all-sequential), a diamond re-converges, and a malformed graph is rejected
 // rather than silently mis-levelized.
-import { assertEquals, assertThrows } from "jsr:@std/assert@1";
+import { test } from "node:test";
+import { assertEquals, assertThrows } from "#test-assert";
 import { computeWaves, WaveError, type WaveGateTask, type WaveTask, waveMergeTargets } from "./waves.ts";
 
-Deno.test("no dependencies → every task in wave 0 (fully parallel)", () => {
+test("no dependencies → every task in wave 0 (fully parallel)", () => {
   const tasks: WaveTask[] = [{ id: "a" }, { id: "b" }, { id: "c" }];
   const { waves, waveCount } = computeWaves(tasks);
   assertEquals(waveCount, 1);
   assertEquals(waves, [["a", "b", "c"]]);
 });
 
-Deno.test("linear chain → one task per wave (fully sequential)", () => {
+test("linear chain → one task per wave (fully sequential)", () => {
   const tasks: WaveTask[] = [
     { id: "a" },
     { id: "b", dependsOn: ["a"] },
@@ -26,7 +27,7 @@ Deno.test("linear chain → one task per wave (fully sequential)", () => {
   assertEquals([waveOf.get("a"), waveOf.get("b"), waveOf.get("c")], [0, 1, 2]);
 });
 
-Deno.test("diamond → longest-path level; join waits for both arms", () => {
+test("diamond → longest-path level; join waits for both arms", () => {
   const tasks: WaveTask[] = [
     { id: "a" },
     { id: "b", dependsOn: ["a"] },
@@ -38,7 +39,7 @@ Deno.test("diamond → longest-path level; join waits for both arms", () => {
   assertEquals(waves, [["a"], ["b", "c"], ["d"]]);
 });
 
-Deno.test("mixed graph → level is 1 + max(dep level), not 1 + min", () => {
+test("mixed graph → level is 1 + max(dep level), not 1 + min", () => {
   // e depends on a (wave 0) and d (wave 2) → must land in wave 3, behind the deeper dep.
   const tasks: WaveTask[] = [
     { id: "a" },
@@ -52,20 +53,20 @@ Deno.test("mixed graph → level is 1 + max(dep level), not 1 + min", () => {
   assertEquals(waveOf.get("e"), 4);
 });
 
-Deno.test("empty plan → zero waves", () => {
+test("empty plan → zero waves", () => {
   const { waves, waveCount } = computeWaves([]);
   assertEquals(waveCount, 0);
   assertEquals(waves, []);
 });
 
-Deno.test("blank / whitespace dependsOn entries are ignored", () => {
+test("blank / whitespace dependsOn entries are ignored", () => {
   const tasks: WaveTask[] = [{ id: "a", dependsOn: ["", "  "] }];
   const { waves, waveCount } = computeWaves(tasks);
   assertEquals(waveCount, 1);
   assertEquals(waves, [["a"]]);
 });
 
-Deno.test("dependency cycle → WaveError", () => {
+test("dependency cycle → WaveError", () => {
   const tasks: WaveTask[] = [
     { id: "a", dependsOn: ["b"] },
     { id: "b", dependsOn: ["a"] },
@@ -73,11 +74,11 @@ Deno.test("dependency cycle → WaveError", () => {
   assertThrows(() => computeWaves(tasks), WaveError, "cycle");
 });
 
-Deno.test("self-dependency → WaveError", () => {
+test("self-dependency → WaveError", () => {
   assertThrows(() => computeWaves([{ id: "a", dependsOn: ["a"] }]), WaveError, "itself");
 });
 
-Deno.test("unknown dependency id → WaveError", () => {
+test("unknown dependency id → WaveError", () => {
   assertThrows(
     () => computeWaves([{ id: "a", dependsOn: ["ghost"] }]),
     WaveError,
@@ -85,7 +86,7 @@ Deno.test("unknown dependency id → WaveError", () => {
   );
 });
 
-Deno.test("duplicate task id → WaveError", () => {
+test("duplicate task id → WaveError", () => {
   assertThrows(
     () => computeWaves([{ id: "a" }, { id: "a" }]),
     WaveError,
@@ -98,7 +99,7 @@ Deno.test("duplicate task id → WaveError", () => {
 // opened/waiting-with-a-PR tasks of the gate wave, ignore other waves, and treat blocked/skipped
 // and keyless tasks as nothing-to-wait-on so no non-mergeable PR state can wedge the barrier.
 
-Deno.test("waveMergeTargets → only opened PRs of the gate wave", () => {
+test("waveMergeTargets → only opened PRs of the gate wave", () => {
   const tasks: WaveGateTask[] = [
     { wave: 0, status: "opened", pr_key: "o/r#1" },
     { wave: 0, status: "opened", pr_key: "o/r#2" },
@@ -107,7 +108,7 @@ Deno.test("waveMergeTargets → only opened PRs of the gate wave", () => {
   assertEquals(waveMergeTargets(tasks, 0), ["o/r#1", "o/r#2"]);
 });
 
-Deno.test("waveMergeTargets → mergeable waiting tasks are waited on; failed/keyless tasks are not", () => {
+test("waveMergeTargets → mergeable waiting tasks are waited on; failed/keyless tasks are not", () => {
   const tasks: WaveGateTask[] = [
     { wave: 0, status: "opened", pr_key: "o/r#1" },
     { wave: 0, status: "blocked", pr_key: null },
@@ -119,7 +120,7 @@ Deno.test("waveMergeTargets → mergeable waiting tasks are waited on; failed/ke
   assertEquals(waveMergeTargets(tasks, 0), ["o/r#1", "o/r#2"]);
 });
 
-Deno.test("waveMergeTargets → a wave with no opened PRs clears vacuously (empty)", () => {
+test("waveMergeTargets → a wave with no opened PRs clears vacuously (empty)", () => {
   const tasks: WaveGateTask[] = [
     { wave: 0, status: "blocked", pr_key: null },
     { wave: 0, status: "skipped", pr_key: null },