@polderlabs/bizar-plugin 0.5.4 → 0.6.1

package/src/tools/bg-spawn.ts

@@ -92,6 +92,19 @@ export function createBgSpawnTool(deps: BgSpawnDeps) {
         .positive()
         .optional()
         .describe("Collect-time timeout in ms (1s..30min, default 5min)."),
+      persistent: z
+        .boolean()
+        .optional()
+        .default(false)
+        .describe("When true, auto-restart on terminal failure (up to maxRestarts)."),
+      maxRestarts: z
+        .number()
+        .int()
+        .min(1)
+        .max(10)
+        .optional()
+        .default(3)
+        .describe("Number of auto-restart attempts before giving up."),
     },
     execute: async (rawArgs, ctx) => {
       // 1. Odin-only (MEDIUM-26).
@@ -109,6 +122,8 @@ export function createBgSpawnTool(deps: BgSpawnDeps) {
         prompt: string;
         model?: string;
         timeoutMs?: number;
+        persistent?: boolean;
+        maxRestarts?: number;
       };
 
       // 2. Validate the model parameter (LOW-34 / §1.4).
@@ -118,7 +133,7 @@ export function createBgSpawnTool(deps: BgSpawnDeps) {
         if (m === null) {
           return {
             output: JSON.stringify({
-              error: `model must be in "providerID/modelID" format (e.g. "minimax/MiniMax-M3"). Omit to use the agent's default.`,
+              error: `model must be in "providerID/modelID" format (e.g. "openrouter/minimax-m3"). Omit to use the agent's default.`,
             }),
           };
         }
@@ -146,10 +161,15 @@ export function createBgSpawnTool(deps: BgSpawnDeps) {
           ? `${modelOverride.providerID}/${modelOverride.modelID}`
           : "agent-default",
         promptPreview: args.prompt.slice(0, 200),
+        prompt: args.prompt, // store full prompt for restart support
         parentAgent: ctx.agent,
         logPath: buildLogPath(deps.worktree, instanceId),
         timeoutMs,
         toolCallCount: 0,
+        // v0.5.5 — persistent auto-restart
+        persistent: args.persistent ?? false,
+        maxRestarts: args.maxRestarts ?? 3,
+        restartCount: 0,
       };
       const addRes = await deps.instanceManager.add(draft);
       if (addRes === "cap_reached") {

package/tests/attach-handler-bug.test.ts

@@ -14,6 +14,8 @@
  */
 
 import { describe, it, expect, beforeEach } from "bun:test";
+import os from "node:os";
+import path from "node:path";
 
 // --- Real InstanceManager (the one under test) ----------------------------
 
@@ -76,8 +78,8 @@ class InMemoryStateStore {
 
 // We import the real InstanceManager after the stubs are defined so the
 // test file fails fast if the real signature changes.
-import { InstanceManager } from "../src/background.ts";
-import type { BackgroundState } from "../src/background-state.ts";
+import { InstanceManager } from "../src/background.js";
+import type { BackgroundState } from "../src/background-state.js";
 
 function makeDraft(overrides: Partial<BackgroundState> = {}): BackgroundState {
   return {
@@ -86,14 +88,14 @@ function makeDraft(overrides: Partial<BackgroundState> = {}): BackgroundState {
     agent: "mimir",
     status: "pending",
     startedAt: Date.now(),
-    model: "minimax/MiniMax-M3",
+    model: "openrouter/minimax-m3",
     promptPreview: "test",
     resultPreview: undefined,
     resultMessageIds: [],
     error: undefined,
     parentAgent: "odin",
     parentInstanceId: undefined,
-    logPath: "/tmp/test.log",
+    logPath: path.join(os.tmpdir(), "test.log"),
     timeoutMs: 300_000,
     toolCallCount: 0,
     loopGuardTool: undefined,
@@ -122,7 +124,7 @@ describe("InstanceManager.add — empty sessionId (BUGFIX v0.5.1)", () => {
         warn: () => {},
         error: () => {},
       } as never,
-      serve: { worktree: "/tmp" } as never,
+      serve: { worktree: os.tmpdir() } as never,
       http: {} as never,
       stream: stream as never,
       stallTimeoutMs: 180_000,

package/tests/background-state.test.ts

@@ -40,7 +40,7 @@ function makeState(overrides: Partial<BackgroundState> = {}): BackgroundState {
     agent: "mimir",
     status: "running",
     startedAt: Date.now(),
-    model: "minimax/MiniMax-M3",
+    model: "openrouter/minimax-m3",
     promptPreview: "Do the thing",
     resultPreview: undefined,
     resultMessageIds: [],

package/tests/background.test.ts

@@ -22,7 +22,7 @@ function makeBgState(overrides: Partial<BackgroundState> = {}): BackgroundState
     agent: "mimir",
     status: "pending",
     startedAt: Date.now(),
-    model: "minimax/MiniMax-M3",
+    model: "openrouter/minimax-m3",
     promptPreview: "Do the thing",
     resultPreview: undefined,
     resultMessageIds: [],

package/tests/block.test.ts

@@ -17,6 +17,8 @@
  */
 
 import { describe, test, expect } from "bun:test";
+import os from "node:os";
+import path from "node:path";
 
 import { decide } from "../src/loop.js";
 import {
@@ -58,7 +60,7 @@ function emptyState(): SessionState {
 
 const FP = "fp:read:loop";
 const TOOL = "read";
-const ARGS = { path: "/tmp/example.txt" };
+const ARGS = { path: path.join(os.tmpdir(), "example.txt") };
 const NOW = 1_700_000_500_000;
 
 // For the block tests we need a window size that is at least as large as

package/tests/canonical-key-order.test.ts

@@ -8,25 +8,29 @@
 
 import { describe, test, expect } from "bun:test";
 import { fingerprint } from "../src/fingerprint";
+import os from "node:os";
+import path from "node:path";
+
+const TMP = path.join(os.tmpdir(), "canonical-key-order-test");
 
 describe("fingerprint — canonical key order", () => {
   test("flat object: same keys/values in different insertion order produce the same fingerprint", () => {
     const a = {
       tool: "read",
-      args: { path: "/tmp/foo.ts", recursive: false, limit: 10 },
+      args: { path: path.join(os.tmpdir(), "foo.ts"), recursive: false, limit: 10 },
     };
     const b = {
       tool: "read",
-      args: { limit: 10, recursive: false, path: "/tmp/foo.ts" },
+      args: { limit: 10, recursive: false, path: path.join(os.tmpdir(), "foo.ts") },
     };
     // Same keys, same values, different insertion order — must match.
-    expect(fingerprint(a.tool, a.args, "/tmp")).toBe(fingerprint(b.tool, b.args, "/tmp"));
+    expect(fingerprint(a.tool, a.args, TMP)).toBe(fingerprint(b.tool, b.args, TMP));
   });
 
   test("nested objects: different insertion order at both levels also match", () => {
     const a = { tool: "edit", args: { meta: { z: 1, a: 2 }, path: "/x" } };
     const b = { tool: "edit", args: { path: "/x", meta: { a: 2, z: 1 } } };
-    expect(fingerprint(a.tool, a.args, "/tmp")).toBe(fingerprint(b.tool, b.args, "/tmp"));
+    expect(fingerprint(a.tool, a.args, TMP)).toBe(fingerprint(b.tool, b.args, TMP));
   });
 
   test("deeply nested: three levels of differing key order all resolve to same fingerprint", () => {
@@ -54,18 +58,18 @@ describe("fingerprint — canonical key order", () => {
         },
       },
     };
-    expect(fingerprint(a.tool, a.args, "/tmp")).toBe(fingerprint(b.tool, b.args, "/tmp"));
+    expect(fingerprint(a.tool, a.args, TMP)).toBe(fingerprint(b.tool, b.args, TMP));
   });
 
   test("array order is preserved (arrays of same values in same order match)", () => {
     const a = { tool: "bash", args: { commands: ["echo a", "echo b"] } };
     const b = { tool: "bash", args: { commands: ["echo a", "echo b"] } };
-    expect(fingerprint(a.tool, a.args, "/tmp")).toBe(fingerprint(b.tool, b.args, "/tmp"));
+    expect(fingerprint(a.tool, a.args, TMP)).toBe(fingerprint(b.tool, b.args, TMP));
   });
 
   test("array with different order produces different fingerprint", () => {
     const a = { tool: "bash", args: { commands: ["echo a", "echo b"] } };
     const b = { tool: "bash", args: { commands: ["echo b", "echo a"] } };
-    expect(fingerprint(a.tool, a.args, "/tmp")).not.toBe(fingerprint(b.tool, b.args, "/tmp"));
+    expect(fingerprint(a.tool, a.args, TMP)).not.toBe(fingerprint(b.tool, b.args, TMP));
   });
 });

package/tests/event-stream.test.ts

@@ -65,7 +65,7 @@ type Event = EventSessionIdle | EventSessionError | EventMessagePartUpdated;
 class FakeEventStream {
   private handlers = new Map<string, Array<(event: Event) => void>>();
   private sessions = new Map<string, string>(); // instanceId → sessionId
-  private instances = new Map<string, BackgroundState>();
+  public instances = new Map<string, BackgroundState>();
   private closed = false;
 
   /** Register an instance's sessionId for event routing */
@@ -406,4 +406,4 @@ describe("EventStream interface contract", () => {
     expect(typeof stream.onSessionEvent).toBe("function");
     expect(typeof stream.close).toBe("function");
   });
-});
+});

package/tests/event.test.ts

@@ -125,7 +125,7 @@ class MockPlugin {
 
 // ── Test setup ───────────────────────────────────────────────────────────────
 
-const TEST_DIR = "/tmp/bizar-event-test";
+const TEST_DIR = path.join(os.tmpdir(), "bizar-event-test");
 const TEST_SESSION = "session-evt-001";
 const TEST_SESSION_2 = "session-evt-002";
 

package/tests/fingerprint.test.ts

@@ -8,10 +8,11 @@ import { describe, test, expect } from "bun:test";
 import { fingerprint } from "../src/fingerprint";
 import { mkdirSync, rmSync, writeFileSync } from "node:fs";
 import path from "node:path";
+import os from "node:os";
 
 /** Use a real temp directory as the worktree for path normalization tests */
-const WORKTREE = "/tmp/bizar-fingerprint-test-worktree";
-const OUTSIDE_TREE = "/tmp/bizar-outside-worktree";
+const WORKTREE = path.join(os.tmpdir(), "bizar-fingerprint-test-worktree");
+const OUTSIDE_TREE = path.join(os.tmpdir(), "bizar-outside-worktree");
 
 function setupWorktree() {
   try {
@@ -33,22 +34,22 @@ setupWorktree();
 
 describe("fingerprint — stable hash", () => {
   test("same args produce the same fingerprint", () => {
-    const args = { path: "/tmp/foo.ts", recursive: false };
+      const args = { path: path.join(os.tmpdir(), "foo.ts"), recursive: false };
     const a = fingerprint("read", args, WORKTREE);
     const b = fingerprint("read", args, WORKTREE);
     expect(a).toBe(b);
   });
 
   test("different tool name produces different fingerprint", () => {
-    const args = { path: "/tmp/foo.ts" };
+      const args = { path: path.join(os.tmpdir(), "foo.ts") };
     const a = fingerprint("read", args, WORKTREE);
     const b = fingerprint("edit", args, WORKTREE);
     expect(a).not.toBe(b);
   });
 
   test("different args produce different fingerprint", () => {
-    const a = fingerprint("read", { path: "/tmp/foo.ts" }, WORKTREE);
-    const b = fingerprint("read", { path: "/tmp/bar.ts" }, WORKTREE);
+    const a = fingerprint("read", { path: path.join(os.tmpdir(), "foo.ts") }, WORKTREE);
+    const b = fingerprint("read", { path: path.join(os.tmpdir(), "bar.ts") }, WORKTREE);
     expect(a).not.toBe(b);
   });
 
@@ -74,7 +75,7 @@ describe("fingerprint — path normalization", () => {
     const fp2 = fingerprint("read", { path: outside }, WORKTREE);
     expect(fp1).toBe(fp2);
     // Must NOT collide with a different outside path
-    const different = "/tmp/different/path/file.txt";
+    const different = path.join(os.tmpdir(), "different/path/file.txt");
     const fp3 = fingerprint("read", { path: different }, WORKTREE);
     expect(fp1).not.toBe(fp3);
   });
@@ -90,38 +91,38 @@ describe("fingerprint — path normalization", () => {
 
 describe("fingerprint — noise field stripping", () => {
   test("strips timestamp fields", () => {
-    const a = fingerprint("read", { path: "/tmp/foo.ts", createdAt: 1234567890 }, WORKTREE);
-    const b = fingerprint("read", { path: "/tmp/foo.ts", createdAt: 9999999999 }, WORKTREE);
+    const a = fingerprint("read", { path: path.join(os.tmpdir(), "foo.ts"), createdAt: 1234567890 }, WORKTREE);
+    const b = fingerprint("read", { path: path.join(os.tmpdir(), "foo.ts"), createdAt: 9999999999 }, WORKTREE);
     expect(a).toBe(b);
   });
 
   test("strips updatedAt timestamp fields", () => {
-    const a = fingerprint("read", { path: "/tmp/foo.ts", updatedAt: "2024-01-01T00:00:00Z" }, WORKTREE);
-    const b = fingerprint("read", { path: "/tmp/foo.ts", updatedAt: "2025-12-31T23:59:59Z" }, WORKTREE);
+    const a = fingerprint("read", { path: path.join(os.tmpdir(), "foo.ts"), updatedAt: "2024-01-01T00:00:00Z" }, WORKTREE);
+    const b = fingerprint("read", { path: path.join(os.tmpdir(), "foo.ts"), updatedAt: "2025-12-31T23:59:59Z" }, WORKTREE);
     expect(a).toBe(b);
   });
 
   test("strips id field", () => {
-    const a = fingerprint("read", { path: "/tmp/foo.ts", id: "abc123" }, WORKTREE);
-    const b = fingerprint("read", { path: "/tmp/foo.ts", id: "xyz789" }, WORKTREE);
+    const a = fingerprint("read", { path: path.join(os.tmpdir(), "foo.ts"), id: "abc123" }, WORKTREE);
+    const b = fingerprint("read", { path: path.join(os.tmpdir(), "foo.ts"), id: "xyz789" }, WORKTREE);
     expect(a).toBe(b);
   });
 
   test("strips uuid field", () => {
-    const a = fingerprint("read", { path: "/tmp/foo.ts", uuid: "550e8400-e29b-41d4-a716-446655440000" }, WORKTREE);
-    const b = fingerprint("read", { path: "/tmp/foo.ts", uuid: "6ba7b810-9dad-11d1-80b4-00c04fd430c8" }, WORKTREE);
+    const a = fingerprint("read", { path: path.join(os.tmpdir(), "foo.ts"), uuid: "550e8400-e29b-41d4-a716-446655440000" }, WORKTREE);
+    const b = fingerprint("read", { path: path.join(os.tmpdir(), "foo.ts"), uuid: "6ba7b810-9dad-11d1-80b4-00c04fd430c8" }, WORKTREE);
     expect(a).toBe(b);
   });
 
   test("strips nonce field", () => {
-    const a = fingerprint("read", { path: "/tmp/foo.ts", nonce: "random1" }, WORKTREE);
-    const b = fingerprint("read", { path: "/tmp/foo.ts", nonce: "random2" }, WORKTREE);
+    const a = fingerprint("read", { path: path.join(os.tmpdir(), "foo.ts"), nonce: "random1" }, WORKTREE);
+    const b = fingerprint("read", { path: path.join(os.tmpdir(), "foo.ts"), nonce: "random2" }, WORKTREE);
     expect(a).toBe(b);
   });
 
   test("strips cwd field entirely", () => {
-    const a = fingerprint("read", { path: "/tmp/foo.ts", cwd: "/home/user" }, WORKTREE);
-    const b = fingerprint("read", { path: "/tmp/foo.ts", cwd: "/completely/different" }, WORKTREE);
+    const a = fingerprint("read", { path: path.join(os.tmpdir(), "foo.ts"), cwd: "/home/user" }, WORKTREE);
+    const b = fingerprint("read", { path: path.join(os.tmpdir(), "foo.ts"), cwd: "/completely/different" }, WORKTREE);
     expect(a).toBe(b);
   });
 });
@@ -130,10 +131,10 @@ describe("fingerprint — nested objects", () => {
   test("nested objects are normalized recursively", () => {
     const a = fingerprint("edit", {
       meta: { author: "Alice", timestamp: 1000 },
-      path: "/tmp/foo.ts",
+      path: path.join(os.tmpdir(), "foo.ts"),
     }, WORKTREE);
     const b = fingerprint("edit", {
-      path: "/tmp/foo.ts",
+      path: path.join(os.tmpdir(), "foo.ts"),
       meta: { timestamp: 9999, author: "Alice" },
     }, WORKTREE);
     expect(a).toBe(b);

package/tests/http-client.test.ts

@@ -6,6 +6,8 @@
  */
 
 import { describe, it, expect } from "bun:test";
+import os from "node:os";
+import path from "node:path";
 
 // ---------------------------------------------------------------------------
 // Types that mirror the expected HttpClient API from §1 / §2
@@ -33,10 +35,7 @@ interface Message {
   parts: MessagePart[];
 }
 
-interface CollectedMessage {
-  info: Message;
-  parts: MessagePart[];
-}
+type CollectedMessage = Message;
 
 // ---------------------------------------------------------------------------
 // Fake HttpClient matching the expected interface
@@ -129,12 +128,12 @@ describe("HttpClient.createSession", () => {
       parentID: "parent_sess",
       title: "bgr:mimir:bgr_01",
       agent: "mimir",
-      directory: "/tmp/worktree",
+      directory: path.join(os.tmpdir(), "worktree"),
     });
 
     expect(session.id).toBeTruthy();
     expect(session.projectID).toBe("proj_test");
-    expect(session.directory).toBe("/tmp/worktree");
+    expect(session.directory).toBe(path.join(os.tmpdir(), "worktree"));
     expect(session.parentID).toBe("parent_sess");
     expect(session.title).toBe("bgr:mimir:bgr_01");
   });
@@ -185,8 +184,10 @@ describe("HttpClient.sendPrompt", () => {
     const msgs = await client.listMessages(session.id, "/tmp");
     const userMsg = msgs.find((m) => m.info.role === "user");
     expect(userMsg).toBeDefined();
-    expect(userMsg!.parts[0].type).toBe("text");
-    expect(userMsg!.parts[0].text).toBe("Do the research on X");
+    const firstPart = userMsg?.parts[0];
+    expect(firstPart).toBeDefined();
+    expect(firstPart?.type).toBe("text");
+    expect(firstPart?.text).toBe("Do the research on X");
   });
 
   it("generates a unique messageID per call (HIGH-2)", async () => {
@@ -261,7 +262,7 @@ describe("HttpClient.sendPrompt", () => {
       messageID: "msg_model_test",
       prompt: "Use a specific model",
       agent: "mimir",
-      model: { providerID: "minimax", modelID: "MiniMax-M3" },
+      model: { providerID: "openrouter", modelID: "minimax-m3" },
       directory: "/tmp",
     });
 
@@ -400,4 +401,4 @@ describe("HTTP timeout handling (HIGH-36)", () => {
     // This is ensured by the implementation not writing on AbortError
     expect(true).toBe(true);
   });
-});
+});

package/tests/init-helpers.test.ts

@@ -182,10 +182,10 @@ describe("readValidSessionIds — postmortem Layer 1 fix (v0.5.2)", () => {
       // no-op `.catch(() => undefined)` to suppress the unhandled-
       // rejection. We can't directly observe unhandled rejections in
       // bun:test, but we can verify the function returns cleanly.
-      let rejectFn: ((err: Error) => void) | null = null;
+      let rejectFn: ((err: Error) => void) | undefined;
       const input = makeInput({
         session: {
-          list: () => new Promise((_, reject) => {
+          list: () => new Promise((_, reject: (err: Error) => void) => {
             rejectFn = reject;
           }),
         },
@@ -195,7 +195,7 @@ describe("readValidSessionIds — postmortem Layer 1 fix (v0.5.2)", () => {
       // Now reject the hanging promise AFTER the function returned.
       // If the no-op catch is missing, this would be an unhandled
       // rejection. With it, the process keeps running.
-      if (rejectFn) rejectFn(new Error("late rejection"));
+      rejectFn?.(new Error("late rejection"));
       // Give the microtask queue a chance to run.
       await new Promise((r) => setTimeout(r, 10));
     },

package/tests/options.test.ts

@@ -1,4 +1,6 @@
 import { describe, expect, test } from "bun:test";
+import os from "node:os";
+import path from "node:path";
 import {
   DEFAULT_OPTIONS,
   expandHome,
@@ -126,9 +128,9 @@ describe("normalizeOptions", () => {
   });
 
   test("custom logDir and stateDir are preserved", () => {
-    const { options } = normalizeOptions({ logDir: "/tmp/my-logs", stateDir: "/tmp/my-state" });
-    expect(options.logDir).toBe("/tmp/my-logs");
-    expect(options.stateDir).toBe("/tmp/my-state");
+    const { options } = normalizeOptions({ logDir: path.join(os.tmpdir(), "my-logs"), stateDir: path.join(os.tmpdir(), "my-state") });
+    expect(options.logDir).toBe(path.join(os.tmpdir(), "my-logs"));
+    expect(options.stateDir).toBe(path.join(os.tmpdir(), "my-state"));
   });
 });
 
@@ -138,7 +140,7 @@ describe("findSecretDirMatch", () => {
   const home = process.env.HOME ?? "/home/test";
 
   test("returns null for safe paths", () => {
-    expect(findSecretDirMatch("/tmp/foo")).toBeNull();
+    expect(findSecretDirMatch(path.join(os.tmpdir(), "foo"))).toBeNull();
     expect(findSecretDirMatch("/home/user/project")).toBeNull();
   });
 
@@ -181,8 +183,8 @@ describe("findOffendingPath", () => {
   test("returns null when both paths are safe", () => {
     const opts: NormalizedOptions = {
       ...D,
-      logDir: "/tmp/bizar-logs",
-      stateDir: "/tmp/bizar-state",
+      logDir: path.join(os.tmpdir(), "bizar-logs"),
+      stateDir: path.join(os.tmpdir(), "bizar-state"),
     };
     expect(findOffendingPath(opts)).toBeNull();
   });
@@ -192,7 +194,7 @@ describe("findOffendingPath", () => {
     const opts: NormalizedOptions = {
       ...D,
       logDir: `${home}/.ssh/evil`,
-      stateDir: "/tmp/bizar-state",
+      stateDir: path.join(os.tmpdir(), "bizar-state"),
     };
     const result = findOffendingPath(opts);
     expect(result).not.toBeNull();
@@ -204,7 +206,7 @@ describe("findOffendingPath", () => {
     const home = process.env.HOME ?? "/home/test";
     const opts: NormalizedOptions = {
       ...D,
-      logDir: "/tmp/bizar-logs",
+      logDir: path.join(os.tmpdir(), "bizar-logs"),
       stateDir: `${home}/.aws/creds`,
     };
     const result = findOffendingPath(opts);

package/tests/serve.test.ts

@@ -155,7 +155,7 @@ describe("ENOENT and EACCES handling", () => {
 describe("unexpected exit", () => {
   it("marks all running instances failed when serve child exits unexpectedly", async () => {
     // Simulate: proc.exited resolves with non-zero code (not intentional shutdown)
-    const exitCode = 1;
+    const exitCode: number = 1;
     const runningInstances = [
       { instanceId: "bgr_01", status: "running", error: undefined as string | undefined },
       { instanceId: "bgr_02", status: "pending", error: undefined as string | undefined },
@@ -171,14 +171,18 @@ describe("unexpected exit", () => {
       }
     }
 
-    expect(runningInstances[0].status).toBe("failed");
-    expect(runningInstances[0].error).toBe("serve child exited unexpectedly");
-    expect(runningInstances[1].status).toBe("failed");
-    expect(runningInstances[1].error).toBe("serve child exited unexpectedly");
+    const first = runningInstances[0];
+    const second = runningInstances[1];
+    expect(first).toBeDefined();
+    expect(second).toBeDefined();
+    expect(first?.status).toBe("failed");
+    expect(first?.error).toBe("serve child exited unexpectedly");
+    expect(second?.status).toBe("failed");
+    expect(second?.error).toBe("serve child exited unexpectedly");
   });
 
   it("clean exit (code 0) does not mark instances failed", async () => {
-    const exitCode = 0;
+    const exitCode: number = 0;
     const runningInstances = [
       { instanceId: "bgr_01", status: "running" },
     ];
@@ -189,11 +193,11 @@ describe("unexpected exit", () => {
       }
     }
 
-    expect(runningInstances[0].status).toBe("running"); // unchanged
+    expect(runningInstances[0]?.status).toBe("running"); // unchanged
   });
 
   it("intentional shutdown does not trigger unexpected-exit path", async () => {
-    const exitCode = 1;
+    const exitCode: number = 1;
     const intentionalShutdown = true;
 
     const runningInstances = [{ instanceId: "bgr_01", status: "running" }];
@@ -204,7 +208,7 @@ describe("unexpected exit", () => {
       for (const inst of runningInstances) inst.status = "failed";
     }
 
-    expect(runningInstances[0].status).toBe("running");
+    expect(runningInstances[0]?.status).toBe("running");
   });
 
   it("servePID is cleared after unexpected exit", async () => {
@@ -332,4 +336,4 @@ describe("ServeLifecycle interface contract", () => {
     expect(typeof fake.password).toBe("string");
     expect(typeof fake.baseUrl).toBe("string");
   });
-});
+});

package/tests/settings.test.ts

@@ -44,7 +44,7 @@ class MockLogger {
   error(m: string) { this.messages.push({ level: "error", message: m }); }
 }
 
-const TEST_DIR = "/tmp/bizar-settings-test";
+const TEST_DIR = path.join(os.tmpdir(), "bizar-settings-test");
 
 beforeEach(() => {
   try { rmSync(TEST_DIR, { recursive: true, force: true }); } catch { /* ok */ }
@@ -252,7 +252,7 @@ describe("SettingsStore — file path", () => {
   });
 
   test("settings written with ~ path land in the expanded location", async () => {
-    const tmp = "/tmp/bizar-settings-expansion-test";
+    const tmp = path.join(os.tmpdir(), "bizar-settings-expansion-test");
     rmSync(tmp, { recursive: true, force: true });
     try {
       const logger = makeLogger();

package/tests/stall-think.test.ts

@@ -16,6 +16,7 @@
 import { describe, it, expect, beforeEach, afterEach } from "bun:test";
 import { writeFileSync, mkdirSync, unlinkSync, rmSync } from "node:fs";
 import path from "node:path";
+import os from "node:os";
 
 // ---------------------------------------------------------------------------
 // Group 1 — researchInterventionPrompt
@@ -130,7 +131,7 @@ const silentLogger = {
 };
 
 function makeTempDir(prefix: string): string {
-  const dir = `/tmp/bizar-stall-test-${prefix}-${process.pid}`;
+  const dir = path.join(os.tmpdir(), `bizar-stall-test-${prefix}-${process.pid}`);
   mkdirSync(dir, { recursive: true });
   return dir;
 }
@@ -161,11 +162,11 @@ describe("BackgroundState schema backfill", () => {
       agent: "mimir",
       status: "running",
       startedAt,
-      model: "minimax/MiniMax-M3",
+      model: "openrouter/minimax-m3",
       promptPreview: "Do the thing",
       toolCallCount: 0,
       parentAgent: "odin",
-      logPath: "/tmp/test.log",
+      logPath: path.join(os.tmpdir(), "test.log"),
       timeoutMs: 300_000,
       // lastEventAt is intentionally absent
       lastToolOrTextAt: startedAt,
@@ -188,11 +189,11 @@ describe("BackgroundState schema backfill", () => {
       agent: "mimir",
       status: "running",
       startedAt,
-      model: "minimax/MiniMax-M3",
+      model: "openrouter/minimax-m3",
       promptPreview: "Do the thing",
       toolCallCount: 0,
       parentAgent: "odin",
-      logPath: "/tmp/test.log",
+      logPath: path.join(os.tmpdir(), "test.log"),
       timeoutMs: 300_000,
       lastEventAt: startedAt,
       // lastToolOrTextAt is intentionally absent
@@ -215,11 +216,11 @@ describe("BackgroundState schema backfill", () => {
       agent: "mimir",
       status: "running",
       startedAt,
-      model: "minimax/MiniMax-M3",
+      model: "openrouter/minimax-m3",
       promptPreview: "Do the thing",
       toolCallCount: 0,
       parentAgent: "odin",
-      logPath: "/tmp/test.log",
+      logPath: path.join(os.tmpdir(), "test.log"),
       timeoutMs: 300_000,
       lastEventAt: startedAt,
       lastToolOrTextAt: startedAt,
@@ -436,7 +437,7 @@ function makeBgState(overrides: Partial<BackgroundState> = {}): BackgroundState
     agent: "mimir",
     status: "running",
     startedAt: now,
-    model: "minimax/MiniMax-M3",
+    model: "openrouter/minimax-m3",
     promptPreview: "Do the thing",
     resultPreview: undefined,
     resultMessageIds: [],
@@ -699,11 +700,11 @@ describe("bg-status toView — v0.3.0 fields", () => {
       agent: "mimir",
       status: "running",
       startedAt: now - 600_000,
-      model: "minimax/MiniMax-M3",
+      model: "openrouter/minimax-m3",
       promptPreview: "Research X",
       toolCallCount: 0,
       parentAgent: "odin",
-      logPath: "/tmp/test.log",
+      logPath: path.join(os.tmpdir(), "test.log"),
       timeoutMs: 300_000,
       lastEventAt: now - 60_000,
       lastToolOrTextAt: now - 60_000,
@@ -728,11 +729,11 @@ describe("bg-status toView — v0.3.0 fields", () => {
       agent: "mimir",
       status: "running",
       startedAt: now,
-      model: "minimax/MiniMax-M3",
+      model: "openrouter/minimax-m3",
       promptPreview: "Research Y",
       toolCallCount: 0,
       parentAgent: "odin",
-      logPath: "/tmp/test.log",
+      logPath: path.join(os.tmpdir(), "test.log"),
       timeoutMs: 300_000,
       // v0.3.0 fields are absent (no intervention yet)
     };