@polderlabs/bizar 6.2.4 → 6.3.0

package/plugins/bizar/tests/serve.test.ts

@@ -1,339 +0,0 @@
-/**
- * ServeLifecycle tests.
- *
- * Tests: Bun.spawn args (MEDIUM-23), ENOENT/EACCES fallback (MEDIUM-23),
- * unexpected exit marks all instances failed (HIGH-9), health check timeout
- * (MEDIUM-42), BIZAR_SERVE_DISABLE=1 (LOW-43), restart with backoff (HIGH-9).
- *
- * We mock Bun.spawn to return a fake subprocess so tests are deterministic
- * and do not actually spawn cline serve.
- */
-
-import { describe, it, expect, beforeEach, vi } from "bun:test";
-import type { ServeLifecycle } from "../src/serve.ts";
-
-// ---------------------------------------------------------------------------
-// Fake subprocess helpers
-// ---------------------------------------------------------------------------
-
-/** Makes a fake Bun.spawn ExitedPromise that resolves immediately */
-function makeFakeProc(exitCode: number, signal?: string) {
-  return {
-    exited: Promise.resolve(exitCode),
-    kill: vi.fn(),
-    stdout: { readable: true } as unknown as ReadableStream & { pipe: () => void },
-    stderr: { readable: true } as unknown as ReadableStream & { pipe: () => void },
-  };
-}
-
-// ---------------------------------------------------------------------------
-// ServeLifecycle test doubles — mirrors the expected API from §5.1 / §5.2
-// ---------------------------------------------------------------------------
-
-interface FakeServeHooks {
-  start: () => Promise<void>;
-  stop: () => Promise<void>;
-  healthCheck: () => Promise<boolean>;
-  pid: number | null;
-  port: number;
-  password: string;
-  baseUrl: string;
-}
-
-/** In-memory fake ServeLifecycle for testing */
-class FakeServeLifecycle implements FakeServeHooks {
-  pid: number | null = null;
-  port = 0;
-  password = "";
-  baseUrl = "";
-  private _exitedPromise: Promise<number> = Promise.resolve(0);
-  private _intentionalShutdown = false;
-  private _onUnexpectedExit: ((exitCode: number) => void) | null = null;
-  private _spawnCalls: Array<{ args: string[]; env: Record<string, string> }> = [];
-  private _serveDisabled = false;
-
-  constructor(opts: { serveDisabled?: boolean; spawnCalls?: Array<{ args: string[]; env: Record<string, string> }> } = {}) {
-    this._serveDisabled = opts.serveDisabled ?? false;
-    this._spawnCalls = opts.spawnCalls ?? [];
-  }
-
-  get spawnCalls() {
-    return this._spawnCalls;
-  }
-
-  async start(): Promise<void> {
-    if (this._serveDisabled) return;
-    // Simulate finding the port from stdout
-    this.port = 4096;
-    this.baseUrl = `http://127.0.0.1:${this.port}`;
-    this.pid = 12345;
-    this.password = "fake-password";
-  }
-
-  async stop(): Promise<void> {
-    this.pid = null;
-  }
-
-  async healthCheck(): Promise<boolean> {
-    return this.port > 0;
-  }
-}
-
-// ---------------------------------------------------------------------------
-// Bun.spawn args verification tests (MEDIUM-23)
-// ---------------------------------------------------------------------------
-
-describe("ServeLifecycle spawn args", () => {
-  it("uses --port 0 by default (random OS-assigned port)", async () => {
-    const fake = new FakeServeLifecycle();
-    await fake.start();
-    // The real implementation would call Bun.spawn with ["cline", "serve", "--port", "0", "--hostname", "127.0.0.1"]
-    expect(fake.port).toBe(4096); // fake sets this; real impl reads from stdout
-  });
-
-  it("uses --hostname 127.0.0.1 (MEDIUM-28)", async () => {
-    const fake = new FakeServeLifecycle();
-    await fake.start();
-    expect(fake.baseUrl).toContain("127.0.0.1");
-  });
-
-  it("passes CLINE_SERVER_PASSWORD in env (HIGH-24)", async () => {
-    const fake = new FakeServeLifecycle();
-    await fake.start();
-    expect(fake.password).toBeTruthy();
-  });
-
-  it("does NOT include --dangerously-skip-permissions by default (MEDIUM-29)", async () => {
-    const fake = new FakeServeLifecycle();
-    await fake.start();
-    // Real impl checks env BIZAR_BACKGROUND_SKIP_PERMISSIONS=1 before adding flag
-    // This test verifies the default (no flag) is honoured
-    expect(fake.pid).not.toBeNull();
-  });
-
-  it("adds --dangerously-skip-permissions when BIZAR_BACKGROUND_SKIP_PERMISSIONS=1 (MEDIUM-29)", async () => {
-    // This would be verified by checking spawnCalls in a full mock
-    // The env var handling is in options.ts — tested there
-    expect(true).toBe(true);
-  });
-});
-
-// ---------------------------------------------------------------------------
-// ENOENT / EACCES fallback (MEDIUM-23)
-// ---------------------------------------------------------------------------
-
-describe("ENOENT and EACCES handling", () => {
-  it("ENOENT sets servePID to null and does not crash", async () => {
-    const fake = new FakeServeLifecycle();
-    // In real impl, ENOENT is caught and logged; servePID = null
-    fake.pid = null;
-    expect(fake.pid).toBeNull();
-  });
-
-  it("EACCES sets servePID to null and does not crash", async () => {
-    const fake = new FakeServeLifecycle();
-    fake.pid = null;
-    expect(fake.pid).toBeNull();
-  });
-
-  it("bizar_spawn_background returns error when servePID is null", async () => {
-    const fake = new FakeServeLifecycle();
-    await fake.start();
-    fake.pid = null;
-    // Real impl checks: if (servePID === null) return { error: "..." }
-    const result = fake.pid === null
-      ? { error: "Background agent serve is not available. See plugin logs." }
-      : { instanceId: "bgr_test" };
-    expect(result).toHaveProperty("error");
-  });
-});
-
-// ---------------------------------------------------------------------------
-// Unexpected exit handling (HIGH-9)
-// ---------------------------------------------------------------------------
-
-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: number = 1;
-    const runningInstances = [
-      { instanceId: "bgr_01", status: "running", error: undefined as string | undefined },
-      { instanceId: "bgr_02", status: "pending", error: undefined as string | undefined },
-    ];
-
-    // Simulate the HIGH-9 handler
-    if (exitCode !== 0) {
-      for (const inst of runningInstances) {
-        if (inst.status === "running" || inst.status === "pending") {
-          inst.status = "failed";
-          inst.error = "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: number = 0;
-    const runningInstances = [
-      { instanceId: "bgr_01", status: "running" },
-    ];
-
-    if (exitCode !== 0) {
-      for (const inst of runningInstances) {
-        inst.status = "failed";
-      }
-    }
-
-    expect(runningInstances[0]?.status).toBe("running"); // unchanged
-  });
-
-  it("intentional shutdown does not trigger unexpected-exit path", async () => {
-    const exitCode: number = 1;
-    const intentionalShutdown = true;
-
-    const runningInstances = [{ instanceId: "bgr_01", status: "running" }];
-
-    if (exitCode === 0 || intentionalShutdown) {
-      // clean path — no marking
-    } else {
-      for (const inst of runningInstances) inst.status = "failed";
-    }
-
-    expect(runningInstances[0]?.status).toBe("running");
-  });
-
-  it("servePID is cleared after unexpected exit", async () => {
-    const fake = new FakeServeLifecycle();
-    await fake.start();
-    expect(fake.pid).not.toBeNull();
-
-    // Simulate unexpected exit
-    fake.pid = null;
-    expect(fake.pid).toBeNull();
-  });
-});
-
-// ---------------------------------------------------------------------------
-// Restart with backoff (HIGH-9)
-// ---------------------------------------------------------------------------
-
-describe("restart with backoff", () => {
-  it("uses exponential backoff: 250ms, 500ms, 1s", async () => {
-    const delays: number[] = [];
-    let attempt = 0;
-
-    // Simulate retry loop with backoff
-    while (attempt < 3) {
-      const delay = 250 * Math.pow(2, attempt);
-      delays.push(delay);
-      attempt++;
-    }
-
-    expect(delays).toEqual([250, 500, 1000]);
-  });
-
-  it("gives up after 3 retries and returns error", async () => {
-    let success = false;
-    let attempts = 0;
-    const maxAttempts = 3;
-
-    while (attempts < maxAttempts) {
-      attempts++;
-      // Simulate failure
-      success = false;
-    }
-
-    expect(success).toBe(false);
-    expect(attempts).toBe(3);
-  });
-
-  it("succeeds on first retry", async () => {
-    let attempts = 0;
-
-    while (true) {
-      attempts++;
-      if (attempts >= 1) break; // simulate success on first retry
-    }
-
-    expect(attempts).toBe(1);
-  });
-});
-
-// ---------------------------------------------------------------------------
-// Health check timeout (MEDIUM-42)
-// ---------------------------------------------------------------------------
-
-describe("health check timeout", () => {
-  it("init logs timeout error and sets servePID to null when port never appears", async () => {
-    // Simulate: process stdout never emits the listening line within 5s
-    const fake = new FakeServeLifecycle();
-    await fake.start();
-
-    // If health check times out, servePID should be null
-    fake.pid = null;
-    expect(fake.pid).toBeNull();
-  });
-
-  it("health check polls GET /health with 100ms interval", async () => {
-    // The interval is a timing parameter; verified by integration test
-    // Unit test documents the expected interval
-    const interval = 100;
-    expect(interval).toBe(100);
-  });
-
-  it("health check times out after 5s", async () => {
-    const timeout = 5000;
-    expect(timeout).toBe(5000);
-  });
-});
-
-// ---------------------------------------------------------------------------
-// BIZAR_SERVE_DISABLE=1 (LOW-43)
-// ---------------------------------------------------------------------------
-
-describe("BIZAR_SERVE_DISABLE=1", () => {
-  it("when set, servePID is never set", async () => {
-    const fake = new FakeServeLifecycle({ serveDisabled: true });
-    await fake.start();
-    expect(fake.pid).toBeNull();
-  });
-
-  it("bizar_spawn_background returns 'background agents disabled' when serve is disabled", async () => {
-    const fake = new FakeServeLifecycle({ serveDisabled: true });
-    await fake.start();
-
-    const result = fake.pid === null
-      ? { error: "Background agents are disabled (BIZAR_SERVE_DISABLE=1)." }
-      : { instanceId: "bgr_test" };
-
-    expect(result).toHaveProperty("error");
-    expect((result as { error: string }).error).toContain("disabled");
-  });
-});
-
-// ---------------------------------------------------------------------------
-// ServeLifecycle interface contract verification
-// ---------------------------------------------------------------------------
-
-describe("ServeLifecycle interface contract", () => {
-  it("has start(), stop(), healthCheck(), pid, port, password, baseUrl", () => {
-    const fake = new FakeServeLifecycle();
-    expect(typeof fake.start).toBe("function");
-    expect(typeof fake.stop).toBe("function");
-    expect(typeof fake.healthCheck).toBe("function");
-    // pid is number | null; use the pattern that matches both
-    expect(fake.pid === null || typeof fake.pid === "number").toBe(true);
-    expect(typeof fake.port).toBe("number");
-    expect(typeof fake.password).toBe("string");
-    expect(typeof fake.baseUrl).toBe("string");
-  });
-});

package/plugins/bizar/tests/settings.test.ts

@@ -1,351 +0,0 @@
-/**
- * settings.test.ts
- *
- * Tests for SettingsStore — read/write round-trip, atomic writes,
- * corrupt-file fallback, validation, and concurrency.
- *
- * Groups:
- *   1. defaults-on-missing-file
- *   2. defaults-on-corrupt-file
- *   3. round-trip update()
- *   4. typed set() for each key
- *   5. invalid-value rejection (clamped to safe value)
- *   6. file path expansion (~ → home)
- *   7. concurrent update() serialization
- *   8. sequential update() round-trip
- */
-
-import { describe, test, expect, beforeEach, afterEach } from "bun:test";
-import {
-  rmSync,
-  mkdirSync,
-  writeFileSync,
-  existsSync,
-  readFileSync,
-} from "node:fs";
-import path from "node:path";
-
-import {
-  SettingsStore,
-  DEFAULT_PLAN_SETTINGS,
-  type PlanSettings,
-} from "../src/settings";
-import os from "node:os";
-
-// Minimal mock logger
-class MockLogger {
-  messages: Array<{ level: string; message: string }> = [];
-  log(opts: { level: string; message: string }) {
-    this.messages.push(opts);
-  }
-  debug(m: string) { this.messages.push({ level: "debug", message: m }); }
-  info(m: string) { this.messages.push({ level: "info", message: m }); }
-  warn(m: string) { this.messages.push({ level: "warn", message: m }); }
-  error(m: string) { this.messages.push({ level: "error", message: m }); }
-}
-
-const TEST_DIR = path.join(os.tmpdir(), "bizar-settings-test");
-
-beforeEach(() => {
-  try { rmSync(TEST_DIR, { recursive: true, force: true }); } catch { /* ok */ }
-  mkdirSync(TEST_DIR, { recursive: true });
-});
-
-afterEach(() => {
-  try { rmSync(TEST_DIR, { recursive: true, force: true }); } catch { /* ok */ }
-});
-
-function makeLogger(): MockLogger {
-  return new MockLogger();
-}
-
-// ===========================================================================
-// Group 1 — defaults on missing file
-// ===========================================================================
-
-describe("SettingsStore — missing file", () => {
-  test("returns DEFAULT_PLAN_SETTINGS when file does not exist", async () => {
-    const logger = makeLogger();
-    const store = new SettingsStore(TEST_DIR, logger);
-    const s = await store.get();
-    expect(s).toEqual(DEFAULT_PLAN_SETTINGS);
-  });
-
-  test("does not log a warning when file is simply missing", async () => {
-    const logger = makeLogger();
-    const store = new SettingsStore(TEST_DIR, logger);
-    await store.get();
-    // ENOENT is silent — only SyntaxError / schema errors log.
-    const warns = logger.messages.filter((m) => m.level === "warn");
-    expect(warns).toHaveLength(0);
-  });
-});
-
-// ===========================================================================
-// Group 2 — corrupt file fallback
-// ===========================================================================
-
-describe("SettingsStore — corrupt file", () => {
-  test("returns defaults + warning when JSON is invalid", async () => {
-    const logger = makeLogger();
-    const filePath = path.join(TEST_DIR, "plan-settings.json");
-    writeFileSync(filePath, "{ not valid json :: !!", "utf8");
-    const store = new SettingsStore(TEST_DIR, logger);
-    const s = await store.get();
-    expect(s).toEqual(DEFAULT_PLAN_SETTINGS);
-    expect(logger.messages.some((m) => m.level === "warn" && m.message.includes("corrupt"))).toBe(true);
-  });
-
-  test("returns defaults + warning when JSON is null", async () => {
-    const logger = makeLogger();
-    const filePath = path.join(TEST_DIR, "plan-settings.json");
-    writeFileSync(filePath, "null", "utf8");
-    const store = new SettingsStore(TEST_DIR, logger);
-    const s = await store.get();
-    expect(s).toEqual(DEFAULT_PLAN_SETTINGS);
-  });
-
-  test("returns defaults + warning when top-level is an array", async () => {
-    const logger = makeLogger();
-    const filePath = path.join(TEST_DIR, "plan-settings.json");
-    writeFileSync(filePath, "[]", "utf8");
-    const store = new SettingsStore(TEST_DIR, logger);
-    const s = await store.get();
-    expect(s).toEqual(DEFAULT_PLAN_SETTINGS);
-  });
-
-  test("clamps invalid values in valid JSON without losing the file", async () => {
-    const logger = makeLogger();
-    const filePath = path.join(TEST_DIR, "plan-settings.json");
-    writeFileSync(
-      filePath,
-      JSON.stringify({
-        visualPlanEnabled: "yes", // wrong type
-        defaultTemplate: "nonsense", // unknown
-        lastUsedSlug: 42, // wrong type
-      }),
-      "utf8",
-    );
-    const store = new SettingsStore(TEST_DIR, logger);
-    const s = await store.get();
-    // Invalid values clamped to defaults, but the file is preserved.
-    expect(s.visualPlanEnabled).toBe(false);
-    expect(s.defaultTemplate).toBe("blank");
-    expect(s.lastUsedSlug).toBeNull();
-    expect(existsSync(filePath)).toBe(true);
-  });
-});
-
-// ===========================================================================
-// Group 3 — round-trip update()
-// ===========================================================================
-
-describe("SettingsStore — update() round-trip", () => {
-  test("update() persists a single field and returns the merged state", async () => {
-    const logger = makeLogger();
-    const store = new SettingsStore(TEST_DIR, logger);
-    const after = await store.update({ visualPlanEnabled: true });
-    expect(after.visualPlanEnabled).toBe(true);
-    expect(after.defaultTemplate).toBe("blank"); // unchanged
-    // File written
-    const raw = JSON.parse(readFileSync(path.join(TEST_DIR, "plan-settings.json"), "utf8"));
-    expect(raw.visualPlanEnabled).toBe(true);
-  });
-
-  test("update() merges multiple fields in a single call", async () => {
-    const logger = makeLogger();
-    const store = new SettingsStore(TEST_DIR, logger);
-    await store.update({
-      visualPlanEnabled: true,
-      defaultTemplate: "feature-design",
-      lastUsedSlug: "my-feature",
-    });
-    const loaded = await store.get();
-    expect(loaded).toEqual({
-      visualPlanEnabled: true,
-      defaultTemplate: "feature-design",
-      lastUsedSlug: "my-feature",
-    });
-  });
-
-  test("update() preserves existing fields not in the patch", async () => {
-    const logger = makeLogger();
-    const store = new SettingsStore(TEST_DIR, logger);
-    await store.update({ visualPlanEnabled: true, lastUsedSlug: "alpha" });
-    await store.update({ defaultTemplate: "decision-record" });
-    const loaded = await store.get();
-    expect(loaded.visualPlanEnabled).toBe(true); // preserved
-    expect(loaded.lastUsedSlug).toBe("alpha"); // preserved
-    expect(loaded.defaultTemplate).toBe("decision-record"); // updated
-  });
-});
-
-// ===========================================================================
-// Group 4 — typed set() for each key
-// ===========================================================================
-
-describe("SettingsStore — set() with typed keys", () => {
-  test("set('visualPlanEnabled', true) persists correctly", async () => {
-    const logger = makeLogger();
-    const store = new SettingsStore(TEST_DIR, logger);
-    const after = await store.set("visualPlanEnabled", true);
-    expect(after.visualPlanEnabled).toBe(true);
-  });
-
-  test("set('defaultTemplate', 'feature-design') persists correctly", async () => {
-    const logger = makeLogger();
-    const store = new SettingsStore(TEST_DIR, logger);
-    const after = await store.set("defaultTemplate", "feature-design");
-    expect(after.defaultTemplate).toBe("feature-design");
-  });
-
-  test("set('lastUsedSlug', 'foo') persists correctly", async () => {
-    const logger = makeLogger();
-    const store = new SettingsStore(TEST_DIR, logger);
-    const after = await store.set("lastUsedSlug", "foo");
-    expect(after.lastUsedSlug).toBe("foo");
-  });
-
-  test("set('lastUsedSlug', null) clears the slug", async () => {
-    const logger = makeLogger();
-    const store = new SettingsStore(TEST_DIR, logger);
-    await store.set("lastUsedSlug", "foo");
-    const after = await store.set("lastUsedSlug", null);
-    expect(after.lastUsedSlug).toBeNull();
-  });
-});
-
-// ===========================================================================
-// Group 5 — invalid-value rejection
-// ===========================================================================
-
-describe("SettingsStore — invalid-value rejection", () => {
-  test("set('defaultTemplate', 'nonsense') is rejected, default preserved", async () => {
-    const logger = makeLogger();
-    const store = new SettingsStore(TEST_DIR, logger);
-    // Force a type-coercion via cast
-    const after = await store.set("defaultTemplate", "nonsense" as never);
-    expect(after.defaultTemplate).toBe("blank"); // unchanged
-    expect(logger.messages.some((m) => m.level === "warn")).toBe(true);
-  });
-
-  test("update() with invalid template is silently ignored", async () => {
-    const logger = makeLogger();
-    const store = new SettingsStore(TEST_DIR, logger);
-    await store.set("defaultTemplate", "feature-design");
-    await store.update({ defaultTemplate: "nonsense" as never });
-    const after = await store.get();
-    expect(after.defaultTemplate).toBe("feature-design"); // unchanged
-  });
-});
-
-// ===========================================================================
-// Group 6 — file path expansion
-// ===========================================================================
-
-describe("SettingsStore — file path", () => {
-  test("constructor expands ~ in settingsDir", () => {
-    const logger = makeLogger();
-    const store = new SettingsStore("~/.cache/test-bizar", logger);
-    const expected = path.join(os.homedir(), ".cache/test-bizar/plan-settings.json");
-    expect(store.getFilePath()).toBe(expected);
-  });
-
-  test("settings written with ~ path land in the expanded location", async () => {
-    const tmp = path.join(os.tmpdir(), "bizar-settings-expansion-test");
-    rmSync(tmp, { recursive: true, force: true });
-    try {
-      const logger = makeLogger();
-      const store = new SettingsStore(tmp, logger);
-      await store.update({ visualPlanEnabled: true });
-      expect(existsSync(path.join(tmp, "plan-settings.json"))).toBe(true);
-    } finally {
-      rmSync(tmp, { recursive: true, force: true });
-    }
-  });
-});
-
-// ===========================================================================
-// Group 7 — concurrent update() serialization
-// ===========================================================================
-
-describe("SettingsStore — concurrent updates", () => {
-  test("concurrent update() calls do not corrupt the file", async () => {
-    const logger = makeLogger();
-    const store = new SettingsStore(TEST_DIR, logger);
-
-    // Fire many updates in parallel that each flip the visualPlanEnabled
-    // flag. The mutex guarantees no write is lost or interleaved.
-    const updates = Array.from({ length: 30 }, (_, i) =>
-      store.update({ visualPlanEnabled: i % 2 === 0 }),
-    );
-    const results = await Promise.all(updates);
-
-    // All returned states are valid (either true or false).
-    for (const r of results) {
-      expect(typeof r.visualPlanEnabled).toBe("boolean");
-    }
-
-    // The on-disk file is valid JSON (parseable).
-    const filePath = path.join(TEST_DIR, "plan-settings.json");
-    expect(existsSync(filePath)).toBe(true);
-    const raw = readFileSync(filePath, "utf8");
-    const parsed = JSON.parse(raw) as PlanSettings;
-    expect(typeof parsed.visualPlanEnabled).toBe("boolean");
-  });
-
-  test("concurrent set() calls do not corrupt the file", async () => {
-    const logger = makeLogger();
-    const store = new SettingsStore(TEST_DIR, logger);
-    const sets = Array.from({ length: 20 }, (_, i) =>
-      store.set("lastUsedSlug", `slug-${i}`),
-    );
-    const results = await Promise.all(sets);
-    for (const r of results) {
-      expect(typeof r.lastUsedSlug).toBe("string");
-      expect(r.lastUsedSlug).toMatch(/^slug-\d+$/);
-    }
-    // File is valid JSON
-    const filePath = path.join(TEST_DIR, "plan-settings.json");
-    const parsed = JSON.parse(readFileSync(filePath, "utf8"));
-    expect(parsed.lastUsedSlug).toMatch(/^slug-\d+$/);
-  });
-});
-
-// ===========================================================================
-// Group 8 — atomic write (no .tmp file left behind)
-// ===========================================================================
-
-describe("SettingsStore — atomic write", () => {
-  test("update() uses atomic rename (no .tmp file on disk)", async () => {
-    const logger = makeLogger();
-    const store = new SettingsStore(TEST_DIR, logger);
-    await store.update({ visualPlanEnabled: true });
-    const filePath = path.join(TEST_DIR, "plan-settings.json");
-    expect(existsSync(filePath)).toBe(true);
-    expect(existsSync(`${filePath}.tmp`)).toBe(false);
-  });
-});
-
-// ===========================================================================
-// Group 9 — round-trip persistence
-// ===========================================================================
-
-describe("SettingsStore — full round-trip", () => {
-  test("write, then create a fresh store, reads back the same value", async () => {
-    const logger1 = makeLogger();
-    const store1 = new SettingsStore(TEST_DIR, logger1);
-    const written = await store1.update({
-      visualPlanEnabled: true,
-      defaultTemplate: "bug-investigation",
-      lastUsedSlug: "round-trip",
-    });
-
-    // Fresh store from the same dir
-    const logger2 = makeLogger();
-    const store2 = new SettingsStore(TEST_DIR, logger2);
-    const read = await store2.get();
-
-    expect(read).toEqual(written);
-  });
-});