Fix CI: the budget timer never throws after its database is closed; Docker builds without git (#76)

* Fix CI: the budget timer never throws after its database is closed

The debounced check after a budget change ran outside its try/catch: in a test that closed its database (or while
shutting down) it threw "database not opened" and failed whatever test was running then (on Linux CI: the three
`godmode binary` runner tests). It is caught now, and the timer no longer keeps the process alive.

* Fix the Docker build: the build stamp works without git

Bun.spawnSync throws when git isn't installed (the image's build stage), so the stamp's own fallback to "unknown"
was never reached and the binary build failed.

Author: danielehrhardt

packages/core/scripts/build.ts

@@ -62,8 +62,13 @@ class BuildError extends Error {}
 /** "<commit>[+changes] <date>" of the checkout that is built; "unknown" outside a git checkout. */
 function buildStamp(): string {
   const git = (...args: string[]) => {
-    const res = Bun.spawnSync(["git", ...args], { cwd: repoDir, stdout: "pipe", stderr: "ignore" });
-    return res.exitCode === 0 ? res.stdout.toString().trim() : null;
+    // Without git installed (the Docker build stage) spawning throws instead of failing.
+    try {
+      const res = Bun.spawnSync(["git", ...args], { cwd: repoDir, stdout: "pipe", stderr: "ignore" });
+      return res.exitCode === 0 ? res.stdout.toString().trim() : null;
+    } catch {
+      return null;
+    }
   };
   const commit = git("rev-parse", "--short", "HEAD");
   if (!commit) return "unknown";

packages/core/src/services/budgets.ts

@@ -229,16 +229,18 @@ function budgetsMayHaveChanged(): void {
   if (releaseTimer) return;
   releaseTimer = setTimeout(() => {
     releaseTimer = null;
-    const now = amountsNow();
-    if (now === lastAmounts) return;
-    lastAmounts = now;
-    if (!get("SELECT 1 FROM paused_runs WHERE reason = 'budget'")) return;
+    // A timer outlives what started it (shutting down, a test that closed its database): it never throws.
     try {
+      const now = amountsNow();
+      if (now === lastAmounts) return;
+      lastAmounts = now;
+      if (!get("SELECT 1 FROM paused_runs WHERE reason = 'budget'")) return;
       releaseHeld("auto");
     } catch (err) {
       log.warn("could not continue held runs", err);
     }
   }, 250);
+  releaseTimer.unref?.();
 }
 
 export function startBudgets(): void {