@gethmy/agent 1.2.0 → 1.4.1

package/dist/__tests__/process-group.test.js

@@ -1,88 +0,0 @@
-import { describe, expect, it } from "vitest";
-import { signalGroup, spawnInGroup, terminateGroup } from "../process-group.js";
-/**
- * Wait until the child writes a "ready" line to stdout. This is the only
- * reliable way to know a Node child has actually installed its signal
- * handlers — time-based waits flake under CI/test-suite load because the
- * handler may not be registered before the test sends SIGTERM, in which case
- * Node's default handler terminates the process and the escalation test
- * can't observe SIGKILL. Child scripts in this file print "ready\n" after
- * calling process.on(...).
- */
-function waitForReady(proc, timeoutMs = 3000) {
-    return new Promise((resolve, reject) => {
-        const timer = setTimeout(() => reject(new Error("child never reported ready")), timeoutMs);
-        let buf = "";
-        proc.stdout?.on("data", (d) => {
-            buf += d.toString();
-            if (buf.includes("ready")) {
-                clearTimeout(timer);
-                resolve();
-            }
-        });
-    });
-}
-describe("process-group", () => {
-    it("places the child in its own process group (pid === pgid)", async () => {
-        if (process.platform === "win32")
-            return;
-        const proc = spawnInGroup("sh", ["-c", "ps -o pgid= -p $$ | tr -d ' '"]);
-        let stdout = "";
-        proc.stdout?.on("data", (d) => {
-            stdout += d.toString();
-        });
-        await new Promise((resolve) => {
-            proc.on("exit", () => resolve());
-        });
-        const pgid = parseInt(stdout.trim(), 10);
-        expect(pgid).toBe(proc.pid);
-    });
-    it("signalGroup handles already-dead processes silently", () => {
-        const fakeProc = {
-            pid: 999_999_999,
-            killed: false,
-            kill: () => true,
-        };
-        expect(() => signalGroup(fakeProc, "SIGTERM")).not.toThrow();
-    });
-    it("terminateGroup exits a cooperative child on SIGINT", async () => {
-        if (process.platform === "win32")
-            return;
-        const proc = spawnInGroup(process.execPath, [
-            "-e",
-            "process.on('SIGINT', () => process.exit(0)); process.stdout.write('ready\\n'); setInterval(()=>{}, 1000);",
-        ]);
-        // Capture the exit state up front so we never miss it.
-        const exited = new Promise((resolve) => {
-            proc.once("exit", (code, signal) => resolve({ code, signal }));
-        });
-        await waitForReady(proc);
-        await terminateGroup(proc, {
-            sigintTimeoutMs: 2000,
-            sigtermTimeoutMs: 500,
-        });
-        const result = await exited;
-        // Either the handler fired (code=0) or we escalated past it — in
-        // both cases the termination contract is kept. We really only want
-        // to prove the process group was reached.
-        expect(result.code !== null || result.signal !== null).toBe(true);
-    });
-    it("terminateGroup escalates to SIGKILL when the group ignores signals", async () => {
-        if (process.platform === "win32")
-            return;
-        const proc = spawnInGroup(process.execPath, [
-            "-e",
-            "process.on('SIGINT', () => {}); process.on('SIGTERM', () => {}); process.stdout.write('ready\\n'); setInterval(()=>{}, 1000);",
-        ]);
-        const exited = new Promise((resolve) => {
-            proc.once("exit", (code, signal) => resolve({ code, signal }));
-        });
-        await waitForReady(proc);
-        await terminateGroup(proc, {
-            sigintTimeoutMs: 200,
-            sigtermTimeoutMs: 200,
-        });
-        const result = await exited;
-        expect(result.signal).toBe("SIGKILL");
-    });
-});

package/dist/__tests__/progress-tracker.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/__tests__/progress-tracker.test.js

@@ -1,247 +0,0 @@
-import { EventEmitter } from "node:events";
-import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
-// Dynamic import so we can test the module without triggering side effects
-// from other imports (log, etc.)
-let ProgressTracker;
-// Minimal mock of HarmonyApiClient
-function makeMockClient() {
-    return {
-        updateAgentProgress: vi.fn().mockResolvedValue(undefined),
-        flushActivityLog: vi.fn().mockResolvedValue(undefined),
-        startAgentSession: vi.fn().mockResolvedValue(undefined),
-        endAgentSession: vi.fn().mockResolvedValue(undefined),
-    };
-}
-// Mock StreamParser — just an EventEmitter with the same typed interface
-class MockParser extends EventEmitter {
-    emitToolStart(name, input) {
-        this.emit("tool_start", name, input);
-    }
-    emitToolEnd(name, toolUseId, content) {
-        this.emit("tool_end", name, toolUseId, content);
-    }
-    emitText(content) {
-        this.emit("text", content);
-    }
-}
-// Access private fields via type cast
-function getPrivate(tracker) {
-    return tracker;
-}
-beforeEach(async () => {
-    // Reset module to get fresh class each test (avoids timer leakage between tests)
-    const mod = await import("../progress-tracker.js");
-    ProgressTracker = mod.ProgressTracker;
-});
-afterEach(() => {
-    vi.restoreAllMocks();
-});
-describe("ProgressTracker", () => {
-    describe("setSessionId", () => {
-        it("stores the session ID", () => {
-            const client = makeMockClient();
-            const tracker = new ProgressTracker(client, "card-1", 0, []);
-            tracker.stop(); // prevent heartbeat
-            expect(getPrivate(tracker).sessionId).toBeNull();
-            tracker.setSessionId("session-abc");
-            expect(getPrivate(tracker).sessionId).toBe("session-abc");
-        });
-        it("triggers flush with correct sessionId when sendUpdate is called", async () => {
-            const client = makeMockClient();
-            const tracker = new ProgressTracker(client, "card-1", 0, []);
-            tracker.stop();
-            tracker.setSessionId("session-xyz");
-            // Push an entry manually so flush has something to send
-            getPrivate(tracker).logBuffer.push({
-                phase: "exploring",
-                eventType: "tool_start",
-                toolName: "Read",
-                description: "Reading src/foo.ts",
-                metadata: {},
-                createdAt: new Date().toISOString(),
-            });
-            // Trigger sendUpdate by calling the private method indirectly
-            // (Force lastUpdateAt to 0 so throttle passes)
-            tracker.lastUpdateAt = 0;
-            tracker.sendUpdate("test task");
-            await vi.waitFor(() => {
-                expect(client.flushActivityLog).toHaveBeenCalledWith("card-1", expect.objectContaining({ sessionId: "session-xyz" }));
-            });
-        });
-    });
-    describe("tool_start events", () => {
-        it("creates a log entry in the buffer when tool has a description", () => {
-            const client = makeMockClient();
-            const tracker = new ProgressTracker(client, "card-2", 0, []);
-            tracker.stop();
-            const parser = new MockParser();
-            tracker.attach(parser);
-            parser.emitToolStart("Read", { file_path: "/src/foo.ts" });
-            const buf = getPrivate(tracker).logBuffer;
-            const entry = buf.find((e) => e.eventType === "tool_start");
-            expect(entry).toBeDefined();
-            expect(entry?.toolName).toBe("Read");
-            expect(entry?.description).toMatch(/Reading/);
-            expect(entry?.createdAt).toBeDefined();
-        });
-        it("does not create a log entry for tools with no description", () => {
-            const client = makeMockClient();
-            const tracker = new ProgressTracker(client, "card-2", 0, []);
-            tracker.stop();
-            const parser = new MockParser();
-            tracker.attach(parser);
-            // Unknown tool with no special description logic
-            parser.emitToolStart("SomeObscureUnknownTool", {});
-            const buf = getPrivate(tracker).logBuffer;
-            const entry = buf.find((e) => e.eventType === "tool_start" &&
-                e.toolName === "SomeObscureUnknownTool");
-            expect(entry).toBeUndefined();
-        });
-    });
-    describe("tool_end events", () => {
-        it("creates a log entry for every tool_end event", () => {
-            const client = makeMockClient();
-            const tracker = new ProgressTracker(client, "card-3", 0, []);
-            tracker.stop();
-            const parser = new MockParser();
-            tracker.attach(parser);
-            parser.emitToolEnd("Bash", "tool-use-1", "exit 0");
-            const buf = getPrivate(tracker).logBuffer;
-            const entry = buf.find((e) => e.eventType === "tool_end" && e.toolName === "Bash");
-            expect(entry).toBeDefined();
-            expect(entry?.description).toContain("Completed");
-        });
-    });
-    describe("buffer cap", () => {
-        it("caps buffer at 500 entries after pushing 510", () => {
-            const client = makeMockClient();
-            const tracker = new ProgressTracker(client, "card-4", 0, []);
-            tracker.stop();
-            const parser = new MockParser();
-            tracker.attach(parser);
-            // Emit 510 tool_end events (tool_end always creates an entry)
-            for (let i = 0; i < 510; i++) {
-                parser.emitToolEnd("Read", `id-${i}`, undefined);
-            }
-            const buf = getPrivate(tracker).logBuffer;
-            expect(buf.length).toBeLessThanOrEqual(500);
-        });
-        it("keeps the most recent entries when the buffer overflows", () => {
-            const client = makeMockClient();
-            const tracker = new ProgressTracker(client, "card-4b", 0, []);
-            tracker.stop();
-            const parser = new MockParser();
-            tracker.attach(parser);
-            for (let i = 0; i < 510; i++) {
-                parser.emitToolEnd(`Tool${i}`, `id-${i}`, undefined);
-            }
-            const buf = getPrivate(tracker).logBuffer;
-            // The oldest entries (0–9) should have been shifted out
-            expect(buf[0].toolName).not.toBe("Tool0");
-        });
-    });
-    describe("flushActivityLog", () => {
-        it("clears the buffer on successful flush", async () => {
-            const client = makeMockClient();
-            const tracker = new ProgressTracker(client, "card-5", 0, []);
-            tracker.stop();
-            tracker.setSessionId("session-flush");
-            getPrivate(tracker).logBuffer.push({
-                phase: "exploring",
-                eventType: "tool_end",
-                toolName: "Read",
-                description: "Completed: Read",
-                metadata: {},
-                createdAt: new Date().toISOString(),
-            });
-            // Invoke the private method
-            tracker.flushActivityLog();
-            // Wait for the async client call to resolve
-            await vi.waitFor(() => {
-                expect(client.flushActivityLog).toHaveBeenCalled();
-            });
-            // Buffer should be empty after successful flush
-            expect(getPrivate(tracker).logBuffer).toHaveLength(0);
-        });
-        it("retains buffer entries on flush error", async () => {
-            const client = makeMockClient();
-            client.flushActivityLog.mockRejectedValue(new Error("network error"));
-            const tracker = new ProgressTracker(client, "card-6", 0, []);
-            tracker.stop();
-            tracker.setSessionId("session-retry");
-            getPrivate(tracker).logBuffer.push({
-                phase: "exploring",
-                eventType: "tool_end",
-                toolName: "Read",
-                description: "Completed: Read",
-                metadata: {},
-                createdAt: new Date().toISOString(),
-            });
-            tracker.flushActivityLog();
-            await vi.waitFor(() => {
-                // After the rejected promise is caught, entries should be back in buffer
-                expect(getPrivate(tracker).logBuffer.length).toBeGreaterThan(0);
-            });
-        });
-        it("skips flush when no session ID is set", () => {
-            const client = makeMockClient();
-            const tracker = new ProgressTracker(client, "card-7", 0, []);
-            tracker.stop();
-            // No setSessionId called
-            getPrivate(tracker).logBuffer.push({
-                phase: "exploring",
-                eventType: "tool_end",
-                toolName: "Read",
-                description: "Completed: Read",
-                metadata: {},
-                createdAt: new Date().toISOString(),
-            });
-            tracker.flushActivityLog();
-            // Should not have called the client
-            expect(client.flushActivityLog).not.toHaveBeenCalled();
-            // Buffer should still contain the entry
-            expect(getPrivate(tracker).logBuffer).toHaveLength(1);
-        });
-        it("skips flush when buffer is empty", () => {
-            const client = makeMockClient();
-            const tracker = new ProgressTracker(client, "card-8", 0, []);
-            tracker.stop();
-            tracker.setSessionId("session-empty");
-            // logBuffer is empty by default
-            tracker.flushActivityLog();
-            expect(client.flushActivityLog).not.toHaveBeenCalled();
-        });
-    });
-    describe("sendUpdate payload", () => {
-        it("does not include recentActions in the updateAgentProgress payload", async () => {
-            const client = makeMockClient();
-            const tracker = new ProgressTracker(client, "card-9", 0, []);
-            tracker.stop();
-            tracker.lastUpdateAt = 0;
-            tracker.sendUpdate("doing stuff");
-            await vi.waitFor(() => {
-                expect(client.updateAgentProgress).toHaveBeenCalled();
-            });
-            const [, payload] = client.updateAgentProgress.mock.calls[0];
-            expect(payload).not.toHaveProperty("recentActions");
-        });
-        it("includes expected fields in updateAgentProgress payload", async () => {
-            const client = makeMockClient();
-            const tracker = new ProgressTracker(client, "card-10", 0, []);
-            tracker.stop();
-            tracker.lastUpdateAt = 0;
-            tracker.sendUpdate("building feature");
-            await vi.waitFor(() => {
-                expect(client.updateAgentProgress).toHaveBeenCalled();
-            });
-            const [cardId, payload] = client.updateAgentProgress.mock.calls[0];
-            expect(cardId).toBe("card-10");
-            expect(payload).toMatchObject({
-                status: "working",
-                currentTask: expect.any(String),
-                progressPercent: expect.any(Number),
-                phase: expect.any(String),
-            });
-        });
-    });
-});

package/dist/__tests__/reconcile-heartbeat.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/__tests__/reconcile-heartbeat.test.js

@@ -1,116 +0,0 @@
-import { mkdtempSync, rmSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
-vi.mock("../worktree.js", () => ({
-    cleanupWorktree: vi.fn(),
-}));
-vi.mock("../board-helpers.js", () => ({
-    moveCardToColumn: vi.fn().mockResolvedValue(undefined),
-    addLabelByName: vi.fn().mockResolvedValue(undefined),
-    buildLabelMap: () => new Map(),
-    hasLabel: () => false,
-    resolveCardLabels: () => [],
-}));
-import { Reconciler } from "../reconcile.js";
-import { newRunId, StateStore } from "../state-store.js";
-import { DEFAULT_AGENT_CONFIG } from "../types.js";
-function makeRun(overrides = {}) {
-    return {
-        runId: newRunId(),
-        cardId: "card-z",
-        cardShortId: 7,
-        pipeline: "implement",
-        workerId: 0,
-        sessionId: null,
-        worktreePath: null,
-        branchName: null,
-        daemonPid: 999_999_999,
-        phase: "running",
-        startedAt: Date.now(),
-        lastHeartbeatAt: Date.now(),
-        endedAt: null,
-        status: "active",
-        costCents: 0,
-        ...overrides,
-    };
-}
-describe("Reconciler stale-heartbeat detection", () => {
-    let dir;
-    let store;
-    let pool;
-    let client;
-    beforeEach(() => {
-        dir = mkdtempSync(join(tmpdir(), "reconcile-"));
-        store = new StateStore(join(dir, "state.json"));
-        pool = {
-            knownCardIds: () => new Set(),
-            isCardActive: vi.fn().mockReturnValue(false),
-            isCardKnown: vi.fn().mockReturnValue(false),
-            enqueue: vi.fn().mockResolvedValue(undefined),
-            removeCard: vi.fn().mockResolvedValue(undefined),
-        };
-        client = {
-            getBoard: vi.fn().mockResolvedValue({
-                cards: [],
-                columns: [{ id: "col-todo", name: "To Do" }],
-                labels: [],
-            }),
-            endAgentSession: vi.fn().mockResolvedValue(undefined),
-            getCard: vi.fn().mockResolvedValue({
-                card: {
-                    id: "card-z",
-                    short_id: 7,
-                    project_id: "p",
-                    column_id: "col-todo",
-                    labelIds: [],
-                },
-            }),
-        };
-    });
-    afterEach(() => {
-        rmSync(dir, { recursive: true, force: true });
-    });
-    it("recovers foreign-daemon runs whose PID is dead", async () => {
-        await store.insertRun(makeRun({ daemonPid: 999_999_999 }));
-        const reconciler = new Reconciler(client, pool, "p", "agent-user", ["To Do"], [], "", 60_000, store, DEFAULT_AGENT_CONFIG);
-        // Exercise a single tick manually.
-        await reconciler.tick();
-        expect(client.endAgentSession).toHaveBeenCalledWith("card-z", expect.objectContaining({ status: "paused" }));
-        expect(store.getActiveRuns()).toHaveLength(0);
-    });
-    it("recovers our own runs whose worker lost track of the card", async () => {
-        // daemonPid === process.pid but pool does NOT have the card active
-        // and heartbeat is old. That's a zombie — should recover.
-        await store.insertRun(makeRun({
-            daemonPid: process.pid,
-            lastHeartbeatAt: Date.now() - 10 * 60_000, // 10 minutes ago
-        }));
-        const reconciler = new Reconciler(client, pool, "p", "agent-user", ["To Do"], [], "", 60_000, store, DEFAULT_AGENT_CONFIG);
-        await reconciler.tick();
-        expect(client.endAgentSession).toHaveBeenCalled();
-        expect(store.getActiveRuns()).toHaveLength(0);
-    });
-    it("does not touch our own runs that the pool still holds", async () => {
-        pool.isCardActive = vi.fn().mockReturnValue(true);
-        await store.insertRun(makeRun({
-            daemonPid: process.pid,
-            lastHeartbeatAt: Date.now() - 10 * 60_000,
-        }));
-        const reconciler = new Reconciler(client, pool, "p", "agent-user", ["To Do"], [], "", 60_000, store, DEFAULT_AGENT_CONFIG);
-        await reconciler.tick();
-        expect(client.endAgentSession).not.toHaveBeenCalled();
-        expect(store.getActiveRuns()).toHaveLength(1);
-    });
-    it("does not touch fresh runs even if foreign-pid checks match", async () => {
-        // Foreign daemon but pid is our own (alive) — not dead.
-        await store.insertRun(makeRun({
-            daemonPid: process.pid,
-            lastHeartbeatAt: Date.now(), // fresh
-        }));
-        pool.isCardActive = vi.fn().mockReturnValue(true);
-        const reconciler = new Reconciler(client, pool, "p", "agent-user", ["To Do"], [], "", 60_000, store, DEFAULT_AGENT_CONFIG);
-        await reconciler.tick();
-        expect(client.endAgentSession).not.toHaveBeenCalled();
-    });
-});

package/dist/__tests__/recovery.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/__tests__/recovery.test.js

@@ -1,126 +0,0 @@
-import { mkdtempSync, rmSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
-import { recoverOrphans } from "../recovery.js";
-import { newRunId, StateStore } from "../state-store.js";
-import { DEFAULT_AGENT_CONFIG } from "../types.js";
-// Mock worktree cleanup so tests don't shell out.
-vi.mock("../worktree.js", () => ({
-    cleanupWorktree: vi.fn(),
-}));
-// Mock board helpers so we can stub out network.
-vi.mock("../board-helpers.js", () => ({
-    moveCardToColumn: vi.fn().mockResolvedValue(undefined),
-    addLabelByName: vi.fn().mockResolvedValue(undefined),
-}));
-import { addLabelByName, moveCardToColumn } from "../board-helpers.js";
-import { cleanupWorktree } from "../worktree.js";
-function makeClient(overrides = {}) {
-    return {
-        endAgentSession: vi.fn().mockResolvedValue(undefined),
-        getCard: vi.fn().mockResolvedValue({
-            card: {
-                id: "card-1",
-                short_id: 42,
-                project_id: "proj-1",
-                title: "test",
-                column_id: "col-1",
-            },
-        }),
-        ...overrides,
-    };
-}
-function sampleRun(overrides = {}) {
-    return {
-        runId: newRunId(),
-        cardId: "card-1",
-        cardShortId: 42,
-        pipeline: "implement",
-        workerId: 0,
-        sessionId: null,
-        worktreePath: "/tmp/wt",
-        branchName: "agent/42-thing",
-        daemonPid: 999_999_999, // almost certainly not alive
-        phase: "running",
-        startedAt: Date.now(),
-        lastHeartbeatAt: Date.now(),
-        endedAt: null,
-        status: "active",
-        costCents: 0,
-        ...overrides,
-    };
-}
-describe("recoverOrphans", () => {
-    let dir;
-    let store;
-    beforeEach(() => {
-        dir = mkdtempSync(join(tmpdir(), "recovery-"));
-        store = new StateStore(join(dir, "state.json"));
-        vi.clearAllMocks();
-    });
-    afterEach(() => {
-        rmSync(dir, { recursive: true, force: true });
-    });
-    it("no-ops when there are no active runs", async () => {
-        const client = makeClient();
-        const outcomes = await recoverOrphans(store, client, DEFAULT_AGENT_CONFIG);
-        expect(outcomes).toEqual([]);
-        expect(client.endAgentSession).not.toHaveBeenCalled();
-    });
-    it("ends session, moves implement card, labels, and cleans worktree", async () => {
-        const run = sampleRun();
-        await store.insertRun(run);
-        const client = makeClient();
-        const outcomes = await recoverOrphans(store, client, DEFAULT_AGENT_CONFIG);
-        expect(outcomes).toHaveLength(1);
-        expect(outcomes[0].errors).toEqual([]);
-        expect(client.endAgentSession).toHaveBeenCalledWith("card-1", expect.objectContaining({ status: "paused" }));
-        expect(moveCardToColumn).toHaveBeenCalledWith(client, expect.objectContaining({ id: "card-1" }), DEFAULT_AGENT_CONFIG.pickupColumns[0]);
-        expect(addLabelByName).toHaveBeenCalledWith(client, expect.objectContaining({ id: "card-1" }), "agent-recovered", expect.any(String));
-        expect(cleanupWorktree).toHaveBeenCalledWith("/tmp/wt", "agent/42-thing");
-        expect(store.getRun(run.runId).status).toBe("orphaned");
-    });
-    it("does not move review cards back to a pickup column", async () => {
-        const run = sampleRun({ pipeline: "review" });
-        await store.insertRun(run);
-        const client = makeClient();
-        await recoverOrphans(store, client, DEFAULT_AGENT_CONFIG);
-        expect(moveCardToColumn).not.toHaveBeenCalled();
-        expect(addLabelByName).toHaveBeenCalled();
-        expect(cleanupWorktree).toHaveBeenCalled();
-    });
-    it("still closes out state when the card is not reachable", async () => {
-        const run = sampleRun();
-        await store.insertRun(run);
-        const client = makeClient({
-            getCard: vi.fn().mockRejectedValue(new Error("404")),
-        });
-        const outcomes = await recoverOrphans(store, client, DEFAULT_AGENT_CONFIG);
-        expect(outcomes[0].actions).toContain("card not reachable — local cleanup only");
-        expect(cleanupWorktree).toHaveBeenCalled();
-        expect(store.getRun(run.runId).status).toBe("orphaned");
-    });
-    it("skips runs that claim a live daemon pid", async () => {
-        const run = sampleRun({ daemonPid: process.pid });
-        await store.insertRun(run);
-        const client = makeClient();
-        const outcomes = await recoverOrphans(store, client, DEFAULT_AGENT_CONFIG);
-        expect(outcomes[0].actions).toContain("skipped: daemon pid still alive");
-        expect(client.endAgentSession).not.toHaveBeenCalled();
-        expect(store.getRun(run.runId).status).toBe("active");
-    });
-    it("continues past individual API failures and records them", async () => {
-        const run = sampleRun();
-        await store.insertRun(run);
-        const client = makeClient({
-            endAgentSession: vi.fn().mockRejectedValue(new Error("boom")),
-        });
-        const outcomes = await recoverOrphans(store, client, DEFAULT_AGENT_CONFIG);
-        expect(outcomes[0].errors).toEqual([
-            expect.stringContaining("endAgentSession: boom"),
-        ]);
-        expect(cleanupWorktree).toHaveBeenCalled();
-        expect(store.getRun(run.runId).status).toBe("orphaned");
-    });
-});

package/dist/__tests__/review-parser.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/__tests__/review-parser.test.js

@@ -1,65 +0,0 @@
-import { describe, expect, it } from "vitest";
-import { parseReviewOutput } from "../review-completion.js";
-describe("parseReviewOutput", () => {
-    it("parses verdict from ```json fenced block (strategy 1)", () => {
-        const out = `
-Some preamble text.
-
-\`\`\`json
-{
-  "verdict": "approved",
-  "summary": "All good",
-  "findings": []
-}
-\`\`\`
-`;
-        const r = parseReviewOutput(out);
-        expect(r.verdict).toBe("approved");
-        expect(r.summary).toBe("All good");
-    });
-    it("parses last fenced block when multiple are present", () => {
-        const out = `
-\`\`\`json
-{ "verdict": "rejected", "summary": "first" }
-\`\`\`
-
-Later correction:
-
-\`\`\`json
-{ "verdict": "approved", "summary": "final" }
-\`\`\`
-`;
-        const r = parseReviewOutput(out);
-        expect(r.verdict).toBe("approved");
-        expect(r.summary).toBe("final");
-    });
-    it("parses bare JSON object without fences (strategy 2)", () => {
-        const out = `Thinking...
-{ "unrelated": true }
-Result:
-{ "verdict": "rejected", "summary": "bugs", "findings": [{ "title": "x", "severity": "major" }] }
-`;
-        const r = parseReviewOutput(out);
-        expect(r.verdict).toBe("rejected");
-        expect(r.findings).toHaveLength(1);
-        expect(r.findings[0].title).toBe("x");
-    });
-    it("falls back to regex when JSON is malformed (strategy 3)", () => {
-        const out = `Claude rambling without proper JSON close
-    "verdict": "approved", "summary": "but missing braces everywhere
-    `;
-        const r = parseReviewOutput(out);
-        expect(r.verdict).toBe("approved");
-        expect(r.findings).toHaveLength(0);
-    });
-    it("returns error verdict when nothing parseable (strategy 4)", () => {
-        const out = "Just prose. No JSON. No verdict keyword.";
-        const r = parseReviewOutput(out);
-        expect(r.verdict).toBe("error");
-    });
-    it("handles empty output gracefully", () => {
-        const r = parseReviewOutput("");
-        expect(r.verdict).toBe("error");
-        expect(r.summary).toBe("");
-    });
-});

package/dist/__tests__/state-store.test.d.ts

@@ -1 +0,0 @@
-export {};