@unerr-ai/unerr 0.1.6 → 0.1.7

package/dist/__tests__/daemon-bridge.test.js

@@ -1,222 +0,0 @@
-/**
- * DM-3: MCP Bridge Integration tests.
- *
- * Tests cover:
- *   - client.ts: sendRequest, sendFireAndForget, probeDaemon
- *   - bootstrap.ts: waitForDaemonReady (poll-only, no spawn)
- *   - mcpBoot socket discovery order (repo sock → unerrd if running + registered)
- *   - Explicit error exits when no process / repo not registered
- *   - Bridge lifecycle: connect/disconnect through daemon
- *   - Activity throttling
- *   - Module isolation (no intelligence imports)
- */
-import { readFileSync, } from "node:fs";
-import { resolve } from "node:path";
-import { describe, expect, it } from "vitest";
-// ── Client module tests ────────────────────────────────────────────
-describe("Daemon client (client.ts)", () => {
-    it("exports all required methods", async () => {
-        const client = await import("../daemon/client.js");
-        expect(typeof client.sendRequest).toBe("function");
-        expect(typeof client.sendFireAndForget).toBe("function");
-        expect(typeof client.ensureRepo).toBe("function");
-        expect(typeof client.connectRepo).toBe("function");
-        expect(typeof client.disconnectRepo).toBe("function");
-        expect(typeof client.sendActivity).toBe("function");
-        expect(typeof client.getStatus).toBe("function");
-        expect(typeof client.probeDaemon).toBe("function");
-        expect(typeof client.daemonSockPath).toBe("function");
-    });
-    it("daemonSockPath returns path under ~/.unerr/", async () => {
-        const { daemonSockPath } = await import("../daemon/client.js");
-        const p = daemonSockPath();
-        expect(p).toContain(".unerr");
-        expect(p).toContain("unerrd.sock");
-    });
-    it("probeDaemon returns false for nonexistent socket", async () => {
-        const { probeDaemon } = await import("../daemon/client.js");
-        const result = await probeDaemon("/tmp/nonexistent-unerr-test.sock");
-        expect(result).toBe(false);
-    });
-    it("sendRequest rejects on nonexistent socket", async () => {
-        const { sendRequest } = await import("../daemon/client.js");
-        await expect(sendRequest("/tmp/nonexistent-unerr-test.sock", { cmd: "status" }, 1000)).rejects.toThrow();
-    });
-    it("sendFireAndForget does not throw on nonexistent socket", async () => {
-        const { sendFireAndForget } = await import("../daemon/client.js");
-        // Should not throw — it's fire-and-forget
-        expect(() => sendFireAndForget("/tmp/nonexistent-unerr-test.sock", {
-            cmd: "activity",
-            repo: "/tmp/fake",
-        })).not.toThrow();
-    });
-});
-// ── Bootstrap module tests ─────────────────────────────────────────
-describe("Daemon bootstrap (bootstrap.ts)", () => {
-    it("exports waitForDaemonReady and ensureDaemonRunning alias", async () => {
-        const bootstrap = await import("../daemon/bootstrap.js");
-        expect(typeof bootstrap.waitForDaemonReady).toBe("function");
-        expect(typeof bootstrap.ensureDaemonRunning).toBe("function");
-        expect(bootstrap.ensureDaemonRunning).toBe(bootstrap.waitForDaemonReady);
-    });
-});
-// ── Module isolation tests ─────────────────────────────────────────
-describe("DM-3 module isolation", () => {
-    it("client.ts imports only from daemon/ and node builtins", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/daemon/client.ts"), "utf-8");
-        const forbidden = [
-            /from\s+["']\.\.\/intelligence\//,
-            /from\s+["']\.\.\/behaviors\//,
-            /from\s+["']\.\.\/tracking\//,
-            /from\s+["']\.\.\/proxy\//,
-        ];
-        for (const pattern of forbidden) {
-            expect(content).not.toMatch(pattern);
-        }
-        // Must import from daemon/
-        expect(content).toMatch(/from\s+["']\.\/registry/);
-        expect(content).toMatch(/from\s+["']\.\/protocol/);
-    });
-    it("bootstrap.ts imports only from daemon/ and node builtins", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/daemon/bootstrap.ts"), "utf-8");
-        const forbidden = [
-            /from\s+["']\.\.\/intelligence\//,
-            /from\s+["']\.\.\/behaviors\//,
-            /from\s+["']\.\.\/tracking\//,
-            /from\s+["']\.\.\/proxy\//,
-        ];
-        for (const pattern of forbidden) {
-            expect(content).not.toMatch(pattern);
-        }
-        expect(content).toMatch(/from\s+["']\.\/client/);
-    });
-    it("bridge.ts still imports nothing from intelligence/", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/proxy/bridge.ts"), "utf-8");
-        const forbidden = [
-            /from\s+["']\.\.\/intelligence\//,
-            /from\s+["']\.\.\/behaviors\//,
-            /from\s+["']\.\.\/tracking\//,
-        ];
-        for (const pattern of forbidden) {
-            expect(content).not.toMatch(pattern);
-        }
-    });
-});
-// ── Socket discovery flow tests ────────────────────────────────────
-describe("mcpBoot socket discovery", () => {
-    it("cli.ts mcpBoot checks per-repo sock first", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/entrypoints/cli.ts"), "utf-8");
-        // Step 1: per-repo proxy sock
-        expect(content).toContain("proxy.sock");
-        expect(content).toContain("probeResult.alive");
-        // Step 2: unerrd (no auto-spawn, just probe + bridge)
-        expect(content).toContain("probeDaemon");
-        expect(content).toContain("ensureRepo");
-        expect(content).toContain("connectRepo");
-        expect(content).toContain("disconnectRepo");
-    });
-    it("mcpBoot does NOT auto-register repos (explicit registration only)", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/entrypoints/cli.ts"), "utf-8");
-        // mcpBoot checks findRepo but never calls addRepo
-        expect(content).toContain("findRepo(cwd)");
-        expect(content).not.toContain("addRepo(cwd, {})");
-    });
-    it("mcpBoot includes activity throttle at 60s", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/entrypoints/cli.ts"), "utf-8");
-        expect(content).toContain("ACTIVITY_THROTTLE_MS = 60_000");
-        expect(content).toContain("sendActivity(daemonSock, cwd)");
-    });
-});
-// ── Bridge lifecycle tests ─────────────────────────────────────────
-describe("Bridge connect/disconnect lifecycle", () => {
-    it("mcpBoot calls connectRepo before bridging and disconnectRepo after", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/entrypoints/cli.ts"), "utf-8");
-        // Ordering: connectRepo comes before startUdsBridge, disconnectRepo after
-        const connectIdx = content.indexOf("await connectRepo(daemonSock, cwd)");
-        const bridgeIdx = content.indexOf("await startUdsBridge(repoSockViaEnsure)");
-        const disconnectIdx = content.indexOf("await disconnectRepo(daemonSock, cwd)");
-        expect(connectIdx).toBeGreaterThan(-1);
-        expect(bridgeIdx).toBeGreaterThan(-1);
-        expect(disconnectIdx).toBeGreaterThan(-1);
-        expect(connectIdx).toBeLessThan(bridgeIdx);
-        expect(bridgeIdx).toBeLessThan(disconnectIdx);
-    });
-});
-// ── Readiness polling design tests (bootstrap.ts) ─────────────────
-describe("Readiness polling (bootstrap.ts)", () => {
-    it("does NOT spawn processes (poll-only, no child_process)", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/daemon/bootstrap.ts"), "utf-8");
-        expect(content).not.toContain("detached: true");
-        expect(content).not.toContain("child_process");
-        expect(content).not.toContain("spawn(");
-    });
-    it("polls at 100ms intervals with 5s timeout", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/daemon/bootstrap.ts"), "utf-8");
-        expect(content).toContain("POLL_INTERVAL_MS = 100");
-        expect(content).toContain("WAIT_TIMEOUT_MS = 5_000");
-    });
-    it("uses probeDaemon for fast-path check", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/daemon/bootstrap.ts"), "utf-8");
-        expect(content).toContain("probeDaemon(sock)");
-    });
-    it("exports waitForDaemonReady as primary + ensureDaemonRunning alias", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/daemon/bootstrap.ts"), "utf-8");
-        expect(content).toContain("export async function waitForDaemonReady");
-        expect(content).toContain("export const ensureDaemonRunning = waitForDaemonReady");
-    });
-});
-// ── Protocol integration tests ─────────────────────────────────────
-describe("Client protocol integration", () => {
-    it("client methods use correct cmd values", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/daemon/client.ts"), "utf-8");
-        expect(content).toContain('cmd: "ensure"');
-        expect(content).toContain('cmd: "connect"');
-        expect(content).toContain('cmd: "disconnect"');
-        expect(content).toContain('cmd: "activity"');
-        expect(content).toContain('cmd: "status"');
-    });
-    it("sendRequest uses newline-delimited JSON", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/daemon/client.ts"), "utf-8");
-        // Sends JSON with newline delimiter
-        expect(content).toContain("JSON.stringify(request)}\\n");
-        // Parses response up to newline
-        expect(content).toContain('buffer.indexOf("\\n")');
-    });
-    it("sendRequest has configurable timeout", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/daemon/client.ts"), "utf-8");
-        expect(content).toContain("timeoutMs = 30_000");
-    });
-});
-// ── Race safety tests ──────────────────────────────────────────────
-describe("Race safety", () => {
-    it("bootstrap.ts is poll-only (no file locks, no spawn)", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/daemon/bootstrap.ts"), "utf-8");
-        expect(content).not.toContain("lockFile");
-        expect(content).not.toContain("flock");
-        expect(content).not.toContain("spawn(");
-        expect(content).toContain("probeDaemon(sock)");
-    });
-});
-// ── Error handling tests ───────────────────────────────────────────
-describe("Error handling", () => {
-    it("mcpBoot exits 1 when no process found (no auto-spawn)", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/entrypoints/cli.ts"), "utf-8");
-        expect(content).toContain("No unerr process found");
-        expect(content).toContain("unerr daemon initialize");
-        expect(content).toContain("process.exit(1)");
-    });
-    it("mcpBoot exits 1 when repo not registered", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/entrypoints/cli.ts"), "utf-8");
-        expect(content).toContain("Repo not registered with unerrd");
-        expect(content).toContain("unerr install <agent>");
-    });
-    it("mcpBoot exits 1 on ensure failure", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/entrypoints/cli.ts"), "utf-8");
-        expect(content).toContain("Failed to ensure repo process");
-    });
-    it("mcpBoot handles daemon_dead from bridge", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/entrypoints/cli.ts"), "utf-8");
-        expect(content).toContain('"daemon_dead"');
-        expect(content).toContain("idle-stopped or crashed");
-    });
-});

package/dist/__tests__/daemon-dashboard.test.js

@@ -1,202 +0,0 @@
-/**
- * DM-4: Unified Dashboard tests.
- *
- * Tests cover:
- *   - Daemon API module (api.ts): exports, route structure, isolation
- *   - Router: global routes, repo-prefixed routes, backward compatibility
- *   - Repo context provider
- *   - UI pages: AllReposPage, DaemonPage exist and export components
- *   - AppShell: daemon mode + standalone mode
- *   - daemon.ts integration: HTTP server startup
- */
-import { readFileSync } from "node:fs";
-import { resolve } from "node:path";
-import { describe, expect, it } from "vitest";
-// ── Daemon API module tests ────────────────────────────────────────
-describe("Daemon API (api.ts)", () => {
-    it("exports startDaemonApi", async () => {
-        const api = await import("../daemon/api.js");
-        expect(typeof api.startDaemonApi).toBe("function");
-    });
-    it("imports only from daemon/, server/ and node builtins + hono", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/daemon/api.ts"), "utf-8");
-        const forbidden = [
-            /from\s+["']\.\.\/intelligence\//,
-            /from\s+["']\.\.\/behaviors\//,
-            /from\s+["']\.\.\/tracking\//,
-        ];
-        for (const pattern of forbidden) {
-            expect(content).not.toMatch(pattern);
-        }
-        expect(content).toContain('from "hono"');
-        expect(content).toContain("process-manager");
-        expect(content).toContain("registry");
-    });
-    it("defines all required API routes", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/daemon/api.ts"), "utf-8");
-        expect(content).toContain('"/api/daemon"');
-        expect(content).toContain('"/api/repos"');
-        expect(content).toContain('"/api/repos/aggregate"');
-        expect(content).toContain('"/api/repo/:label/*"');
-    });
-    it("serves SPA for non-API routes", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/daemon/api.ts"), "utf-8");
-        expect(content).toContain("serveStatic");
-        expect(content).toContain("spaIndex");
-        expect(content).toContain("c.html(spaHtml)");
-    });
-    it("uses port 9847 for daemon dashboard", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/daemon/api.ts"), "utf-8");
-        expect(content).toContain("DAEMON_PORT = 9847");
-    });
-});
-// ── Router tests ───────────────────────────────────────────────────
-describe("Router (router.ts)", () => {
-    it("defines GlobalRouteId and RepoRouteId types", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/ui/lib/router.ts"), "utf-8");
-        expect(content).toContain("GlobalRouteId");
-        expect(content).toContain("RepoRouteId");
-        expect(content).toContain('"all-repos"');
-        expect(content).toContain('"daemon"');
-    });
-    it("exports useParsedRoute and useRepoLabel hooks", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/ui/lib/router.ts"), "utf-8");
-        expect(content).toContain("export function useParsedRoute");
-        expect(content).toContain("export function useRepoLabel");
-    });
-    it("parseHash handles repo/<label>/<route> format", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/ui/lib/router.ts"), "utf-8");
-        expect(content).toContain('first === "repo"');
-        expect(content).toContain("segments[1]");
-        expect(content).toContain("segments[2]");
-    });
-    it("navigateRoute generates repo-prefixed hashes", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/ui/lib/router.ts"), "utf-8");
-        expect(content).toContain("`#/repo/${repoLabel}`");
-        expect(content).toContain("`#/repo/${repoLabel}/${next}`");
-    });
-    it("preserves backward compatibility with bare routes", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/ui/lib/router.ts"), "utf-8");
-        // Bare routes still work (standalone mode)
-        expect(content).toContain("matchRepoRoute(first)");
-        expect(content).toContain("repoLabel: null");
-    });
-});
-// ── Repo context provider tests ────────────────────────────────────
-describe("Repo context (repo-context.ts)", () => {
-    it("exports RepoContext and useRepoContext", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/ui/lib/repo-context.ts"), "utf-8");
-        expect(content).toContain("export const RepoContext");
-        expect(content).toContain("export function useRepoContext");
-        expect(content).toContain("createContext");
-    });
-    it("includes isDaemonMode and apiBase fields", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/ui/lib/repo-context.ts"), "utf-8");
-        expect(content).toContain("isDaemonMode: boolean");
-        expect(content).toContain("apiBase: string");
-        expect(content).toContain("label: string | null");
-    });
-});
-// ── UI pages exist ─────────────────────────────────────────────────
-describe("UI pages", () => {
-    it("AllReposPage.tsx exists and exports component", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/ui/pages/AllReposPage.tsx"), "utf-8");
-        expect(content).toContain("export function AllReposPage");
-        expect(content).toContain("/api/repos");
-        expect(content).toContain("/api/repos/aggregate");
-    });
-    it("AllReposPage shows aggregated metrics", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/ui/pages/AllReposPage.tsx"), "utf-8");
-        expect(content).toContain("totalEntities");
-        expect(content).toContain("totalMemoryMb");
-        expect(content).toContain("Tokens Saved");
-        expect(content).toContain("Tool Calls");
-    });
-    it("DaemonPage.tsx exists and exports component", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/ui/pages/DaemonPage.tsx"), "utf-8");
-        expect(content).toContain("export function DaemonPage");
-        expect(content).toContain("/api/daemon");
-        expect(content).toContain("Supervisor");
-    });
-    it("DaemonPage shows process table", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/ui/pages/DaemonPage.tsx"), "utf-8");
-        expect(content).toContain("Managed Processes");
-        expect(content).toContain("repo.pid");
-        expect(content).toContain("repo.memory");
-    });
-});
-// ── AppShell updates ───────────────────────────────────────────────
-describe("AppShell (layout)", () => {
-    it("supports daemon mode with repo switcher", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/ui/components/layout/AppShell.tsx"), "utf-8");
-        expect(content).toContain("isDaemonMode");
-        expect(content).toContain("GLOBAL_NAV");
-        expect(content).toContain("REPO_NAV");
-        expect(content).toContain("repoLabel");
-    });
-    it("shows status dots for repo list", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/ui/components/layout/AppShell.tsx"), "utf-8");
-        expect(content).toContain("bg-success");
-        expect(content).toContain("bg-error");
-        expect(content).toContain("r.status");
-    });
-    it("falls back to standalone nav when not in daemon mode", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/ui/components/layout/AppShell.tsx"), "utf-8");
-        expect(content).toContain("{!isDaemonMode && (");
-    });
-});
-// ── App.tsx integration ────────────────────────────────────────────
-describe("App.tsx integration", () => {
-    it("detects daemon mode via /api/daemon", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/ui/App.tsx"), "utf-8");
-        expect(content).toContain("isDaemonMode");
-        expect(content).toContain('["daemon", "info"]');
-        expect(content).toContain("/api/daemon");
-    });
-    it("renders global pages (AllReposPage, DaemonPage)", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/ui/App.tsx"), "utf-8");
-        expect(content).toContain("<AllReposPage");
-        expect(content).toContain("<DaemonPage");
-        expect(content).toContain('case "all-repos"');
-        expect(content).toContain('case "daemon"');
-    });
-    it("provides RepoContext", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/ui/App.tsx"), "utf-8");
-        expect(content).toContain("RepoContext.Provider");
-        expect(content).toContain("repoCtx");
-        expect(content).toContain("apiBase");
-    });
-    it("passes repos list and repoLabel to AppShell", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/ui/App.tsx"), "utf-8");
-        expect(content).toContain("repos={repos}");
-        expect(content).toContain("repoLabel={parsed.repoLabel}");
-        expect(content).toContain("isDaemonMode={isDaemonMode}");
-    });
-});
-// ── daemon.ts HTTP integration ─────────────────────────────────────
-describe("daemon.ts HTTP integration", () => {
-    it("starts the dashboard API server", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/entrypoints/daemon.ts"), "utf-8");
-        expect(content).toContain("startDaemonApi");
-        expect(content).toContain("apiHandle");
-        expect(content).toContain("http://localhost");
-    });
-    it("closes HTTP server on shutdown", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/entrypoints/daemon.ts"), "utf-8");
-        expect(content).toContain("apiHandle?.close()");
-    });
-});
-// ── Proxy route design ─────────────────────────────────────────────
-describe("Proxy route design", () => {
-    it("api.ts reads server.json for per-repo HTTP port", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/daemon/api.ts"), "utf-8");
-        expect(content).toContain("server.json");
-        expect(content).toContain("repoPort");
-        expect(content).toContain("proxyToRepoHttp");
-    });
-    it("strips /api/repo/:label prefix before proxying", () => {
-        const content = readFileSync(resolve(process.cwd(), "src/daemon/api.ts"), "utf-8");
-        expect(content).toContain("c.req.path.replace");
-        expect(content).toContain("remaining");
-    });
-});

package/dist/__tests__/daemon-registry.test.js

@@ -1,240 +0,0 @@
-/**
- * Tests for the daemon registry (DM-1 Tasks 2-5, 9).
- *
- * Uses temp dirs for both ~/.unerr (global) and fake repo paths.
- * Mocks `os.homedir()` to isolate from real user state.
- */
-import { existsSync, mkdirSync, readFileSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
-// Mock homedir before importing registry (module-level side effect via globalDir)
-const testHome = join(tmpdir(), `unerr-test-home-${process.pid}-${Date.now()}`);
-vi.mock("node:os", async () => {
-    const actual = await vi.importActual("node:os");
-    return { ...actual, homedir: () => testHome };
-});
-import { addRepo, deriveLabel, detectChildConflicts, detectParentConflict, findRepo, listRepos, readNeedsInput, readRegistry, removeRepo, updateRepoSettings, writeNeedsInput, writeRegistry, } from "../daemon/registry.js";
-let testNum = 0;
-function makeRepo(name) {
-    testNum++;
-    const p = join(testHome, `test-${testNum}`, name);
-    mkdirSync(p, { recursive: true });
-    return p;
-}
-beforeEach(() => {
-    mkdirSync(testHome, { recursive: true });
-    // Reset registry for test isolation
-    writeRegistry({ version: 1, repos: [] });
-});
-afterEach(() => {
-    vi.restoreAllMocks();
-});
-describe("Registry CRUD", () => {
-    it("adds a new repo and writes repos.json", () => {
-        const repoPath = makeRepo("my-app");
-        const result = addRepo(repoPath);
-        expect(result.ok).toBe(true);
-        if (!result.ok)
-            return;
-        expect(result.created).toBe(true);
-        expect(result.entry.path).toBe(repoPath);
-        expect(result.entry.label).toBe("my-app");
-        expect(result.entry.idleTimeout).toBe(1800);
-        const reg = readRegistry();
-        expect(reg.repos).toHaveLength(1);
-        expect(reg.repos[0].path).toBe(repoPath);
-    });
-    it("is idempotent — adding the same path twice returns existing entry", () => {
-        const repoPath = makeRepo("idempotent");
-        const first = addRepo(repoPath);
-        expect(first.ok).toBe(true);
-        if (!first.ok)
-            return;
-        expect(first.created).toBe(true);
-        const second = addRepo(repoPath);
-        expect(second.ok).toBe(true);
-        if (!second.ok)
-            return;
-        expect(second.created).toBe(false);
-        expect(second.entry.path).toBe(repoPath);
-    });
-    it("removes a repo", () => {
-        const repoPath = makeRepo("to-remove");
-        addRepo(repoPath);
-        expect(listRepos()).toHaveLength(1);
-        const removed = removeRepo(repoPath);
-        expect(removed).toBe(true);
-        expect(listRepos()).toHaveLength(0);
-    });
-    it("returns false when removing non-existent repo", () => {
-        expect(removeRepo("/nonexistent")).toBe(false);
-    });
-    it("findRepo by path", () => {
-        const repoPath = makeRepo("findable");
-        addRepo(repoPath);
-        const found = findRepo(repoPath);
-        expect(found).toBeDefined();
-        expect(found.path).toBe(repoPath);
-    });
-    it("findRepo by label", () => {
-        const repoPath = makeRepo("by-label");
-        addRepo(repoPath);
-        const found = findRepo("by-label");
-        expect(found).toBeDefined();
-        expect(found.path).toBe(repoPath);
-    });
-    it("listRepos returns all registered repos", () => {
-        addRepo(makeRepo("repo-a"));
-        addRepo(makeRepo("repo-b"));
-        addRepo(makeRepo("repo-c"));
-        expect(listRepos()).toHaveLength(3);
-    });
-});
-describe("Settings", () => {
-    it("add with settings writes to registry and local config", () => {
-        const repoPath = makeRepo("with-settings");
-        const result = addRepo(repoPath, {
-            idleTimeout: 60,
-            javaBuildTool: "Gradle",
-        });
-        expect(result.ok).toBe(true);
-        if (!result.ok)
-            return;
-        expect(result.entry.idleTimeout).toBe(60);
-        expect(result.entry.settings.javaBuildTool).toBe("Gradle");
-        // Verify local config mirror
-        const localConfig = JSON.parse(readFileSync(join(repoPath, ".unerr", "config.json"), "utf-8"));
-        expect(localConfig.javaBuildTool).toBe("Gradle");
-    });
-    it("updateRepoSettings patches and mirrors", () => {
-        const repoPath = makeRepo("update-settings");
-        addRepo(repoPath, { javaBuildTool: "Maven" });
-        const updated = updateRepoSettings(repoPath, {
-            javaBuildTool: "Gradle",
-            idleTimeout: 120,
-        });
-        expect(updated).not.toBeNull();
-        expect(updated.settings.javaBuildTool).toBe("Gradle");
-        expect(updated.idleTimeout).toBe(120);
-        // Verify persisted
-        const reg = readRegistry();
-        const persisted = reg.repos.find((r) => r.path === repoPath);
-        expect(persisted.settings.javaBuildTool).toBe("Gradle");
-        expect(persisted.idleTimeout).toBe(120);
-        // Verify local mirror
-        const localConfig = JSON.parse(readFileSync(join(repoPath, ".unerr", "config.json"), "utf-8"));
-        expect(localConfig.javaBuildTool).toBe("Gradle");
-    });
-    it("updateRepoSettings returns null for unknown repo", () => {
-        expect(updateRepoSettings("/nonexistent", { idleTimeout: 60 })).toBeNull();
-    });
-});
-describe("Parent detection", () => {
-    it("detects registered parent directory", () => {
-        const parent = makeRepo("monorepo");
-        const child = join(parent, "packages", "frontend");
-        mkdirSync(child, { recursive: true });
-        addRepo(parent);
-        const repos = listRepos();
-        const conflict = detectParentConflict(child, repos);
-        expect(conflict).toBe(parent);
-    });
-    it("returns null when no parent conflict", () => {
-        const a = makeRepo("standalone-a");
-        const b = makeRepo("standalone-b");
-        addRepo(a);
-        const repos = listRepos();
-        expect(detectParentConflict(b, repos)).toBeNull();
-    });
-    it("addRepo blocks registration when parent exists", () => {
-        const parent = makeRepo("parent-block");
-        const child = join(parent, "sub", "project");
-        mkdirSync(child, { recursive: true });
-        addRepo(parent);
-        const result = addRepo(child);
-        expect(result.ok).toBe(false);
-        if (result.ok)
-            return;
-        expect(result.parentConflict).toBe(parent);
-    });
-    it("addRepo allows with skipParentCheck", () => {
-        const parent = makeRepo("parent-allow");
-        const child = join(parent, "sub", "project");
-        mkdirSync(child, { recursive: true });
-        addRepo(parent);
-        const result = addRepo(child, {}, { skipParentCheck: true });
-        expect(result.ok).toBe(true);
-    });
-});
-describe("Child detection", () => {
-    it("detects registered child directories", () => {
-        const parent = makeRepo("new-parent");
-        const childA = join(parent, "packages", "a");
-        const childB = join(parent, "packages", "b");
-        mkdirSync(childA, { recursive: true });
-        mkdirSync(childB, { recursive: true });
-        addRepo(childA, {}, { skipParentCheck: true });
-        addRepo(childB, {}, { skipParentCheck: true });
-        const repos = listRepos();
-        const conflicts = detectChildConflicts(parent, repos);
-        expect(conflicts).toHaveLength(2);
-        expect(conflicts).toContain(childA);
-        expect(conflicts).toContain(childB);
-    });
-    it("returns empty when no child conflicts", () => {
-        const a = makeRepo("no-child-a");
-        addRepo(a);
-        const b = makeRepo("no-child-b");
-        const repos = listRepos();
-        expect(detectChildConflicts(b, repos)).toHaveLength(0);
-    });
-});
-describe("Label generation", () => {
-    it("uses basename for first repo", () => {
-        expect(deriveLabel("/users/dev/my-app", [])).toBe("my-app");
-    });
-    it("uses parent-basename on collision", () => {
-        const existing = [{ label: "my-app" }];
-        expect(deriveLabel("/users/work/my-app", existing)).toBe("work-my-app");
-    });
-    it("uses numeric suffix on double collision", () => {
-        const existing = [
-            { label: "my-app" },
-            { label: "work-my-app" },
-        ];
-        expect(deriveLabel("/other/work/my-app", existing)).toBe("my-app-2");
-    });
-});
-describe("Needs-input signals", () => {
-    it("write and read round-trip", () => {
-        const repoPath = makeRepo("needs-input-test");
-        const signals = [
-            {
-                type: "needs_input",
-                key: "javaBuildTool",
-                auto: "Gradle",
-                alternatives: ["Maven"],
-                reason: "gradlew wrapper present",
-            },
-        ];
-        writeNeedsInput(repoPath, signals);
-        const read = readNeedsInput(repoPath);
-        expect(read).toEqual(signals);
-    });
-    it("returns empty array when no signals file", () => {
-        expect(readNeedsInput("/nonexistent")).toEqual([]);
-    });
-});
-describe("Ensures .unerr/ directory", () => {
-    it("creates .unerr/ on add if missing", () => {
-        const repoPath = makeRepo("ensure-dir");
-        addRepo(repoPath);
-        expect(existsSync(join(repoPath, ".unerr"))).toBe(true);
-    });
-    it("creates .unerr/config.json with settings", () => {
-        const repoPath = makeRepo("config-file");
-        addRepo(repoPath, { javaBuildTool: "Maven" });
-        expect(existsSync(join(repoPath, ".unerr", "config.json"))).toBe(true);
-    });
-});