ppef 1.0.0 → 1.1.0

package/dist/executor/__tests__/checkpoint-merge-bug.unit.test.js

@@ -0,0 +1,164 @@
+/**
+ * Diagnostic Unit Tests for Checkpoint Merge Bug
+ *
+ * Tests to reproduce the data loss bug where mergeShards() overwrites
+ * the main checkpoint instead of combining it with shard results.
+ */
+import { randomBytes } from "node:crypto";
+import { readFileSync, rmSync } from "node:fs";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
+import { afterEach, beforeEach, describe, it } from "node:test";
+import assert from "node:assert/strict";
+import { CheckpointManager } from "../checkpoint-manager.js";
+import { FileStorage } from "../checkpoint-storage.js";
+describe("Checkpoint Merge Bug Diagnostics", () => {
+    let testDir;
+    let mainCheckpoint;
+    let mainPath;
+    beforeEach(() => {
+        testDir = join(tmpdir(), `checkpoint-merge-test-${randomBytes(8).toString("hex")}`);
+        mainPath = join(testDir, "checkpoint.json");
+        mainCheckpoint = new CheckpointManager({ storage: new FileStorage(mainPath) });
+    });
+    /**
+     * Helper to read checkpoint data directly from file
+     * @param path
+     */
+    const readCheckpointFile = (path) => {
+        const data = JSON.parse(readFileSync(path, "utf-8"));
+        return data;
+    };
+    afterEach(() => {
+        rmSync(testDir, { recursive: true, force: true });
+    });
+    it("diagnostic-1: mergeShards should combine main checkpoint with shards", async () => {
+        // Setup: Create main checkpoint with 3 completed runs
+        const run1 = createMockResult("run-001", "sut-1", "case-1");
+        const run2 = createMockResult("run-002", "sut-1", "case-2");
+        const run3 = createMockResult("run-003", "sut-1", "case-3");
+        await mainCheckpoint.saveIncremental(run1);
+        await mainCheckpoint.saveIncremental(run2);
+        await mainCheckpoint.saveIncremental(run3);
+        const mainData = readCheckpointFile(mainPath);
+        assert.strictEqual(mainData.completedRunIds.length, 3);
+        // Create worker shard with 1 additional run
+        const shardPath = join(testDir, "checkpoint-worker-00.json");
+        const shardStorage = new FileStorage(shardPath);
+        await shardStorage.save({
+            configHash: "test-hash",
+            createdAt: new Date().toISOString(),
+            updatedAt: new Date().toISOString(),
+            completedRunIds: ["run-004"],
+            results: {
+                "run-004": createMockResult("run-004", "sut-1", "case-4"),
+            },
+            totalPlanned: 4,
+        });
+        // Load checkpoint to reset internal state
+        await mainCheckpoint.load();
+        // BUG: mergeShards will lose the 3 runs from main checkpoint!
+        const merged = await mainCheckpoint.mergeShards([shardPath]);
+        const mainAfterMerge = readCheckpointFile(mainPath);
+        console.log("Main checkpoint runs:", mainAfterMerge.completedRunIds.length);
+        console.log("Merged runs:", merged.completedRunIds.length);
+        console.log("Expected: 4 runs (3 from main + 1 from shard)");
+        // This should be 4, but will fail showing the bug
+        assert.strictEqual(merged.completedRunIds.length, 4);
+    });
+    it("diagnostic-2: demonstrates mergeShards creates new checkpoint instead of combining", async () => {
+        // Setup main checkpoint with data
+        await mainCheckpoint.saveIncremental(createMockResult("main-001", "sut-1", "case-1"));
+        const mainBeforeMerge = readCheckpointFile(mainPath);
+        console.log("Main checkpoint before merge:", mainBeforeMerge.completedRunIds.length);
+        // Create empty shard (simulating worker that completed no runs)
+        const shardPath = join(testDir, "checkpoint-worker-00.json");
+        const shardStorage = new FileStorage(shardPath);
+        await shardStorage.save({
+            configHash: "test-hash",
+            createdAt: new Date().toISOString(),
+            updatedAt: new Date().toISOString(),
+            completedRunIds: [],
+            results: {},
+            totalPlanned: 132,
+        });
+        await mainCheckpoint.load();
+        // Merge empty shard
+        await mainCheckpoint.mergeShards([shardPath]);
+        const mainAfterMerge = readCheckpointFile(mainPath);
+        console.log("Main checkpoint after merge:", mainAfterMerge.completedRunIds.length);
+        // BUG: The main checkpoint's run is lost!
+        assert.ok(mainAfterMerge.completedRunIds.length > 0);
+    });
+    it("diagnostic-3: shows mergeShards doesn't preserve main checkpoint data", async () => {
+        // Create main checkpoint
+        await mainCheckpoint.saveIncremental(createMockResult("main-001", "sut-1", "case-1"));
+        await mainCheckpoint.saveIncremental(createMockResult("main-002", "sut-2", "case-2"));
+        const mainDataBefore = readCheckpointFile(mainPath);
+        const mainRuns = mainDataBefore.completedRunIds;
+        console.log("Main checkpoint has runs:", mainRuns);
+        // Create worker shard
+        const shardPath = join(testDir, "checkpoint-worker-00.json");
+        const shardStorage = new FileStorage(shardPath);
+        await shardStorage.save({
+            configHash: "test-hash",
+            createdAt: new Date().toISOString(),
+            updatedAt: new Date().toISOString(),
+            completedRunIds: ["shard-001"],
+            results: {
+                "shard-001": createMockResult("shard-001", "sut-3", "case-3"),
+            },
+            totalPlanned: 132,
+        });
+        // Reload main checkpoint
+        await mainCheckpoint.load();
+        // Merge
+        const merged = await mainCheckpoint.mergeShards([shardPath]);
+        const mainDataAfter = readCheckpointFile(mainPath);
+        console.log("After merge:");
+        console.log("  Main checkpoint runs:", mainDataAfter.completedRunIds);
+        console.log("  Merged runs:", merged.completedRunIds);
+        console.log("  Expected: 3 runs (2 from main + 1 from shard)");
+        console.log("  Actual:", merged.completedRunIds.length, "runs");
+        // Should have both main runs AND shard run
+        const expectedRunIds = new Set([...mainRuns, "shard-001"]);
+        const actualRunIds = new Set(merged.completedRunIds);
+        console.log("  Missing runs:", [...expectedRunIds].filter((x) => !actualRunIds.has(x)));
+        // Check all expected runs are present
+        for (const runId of expectedRunIds) {
+            assert.ok(actualRunIds.has(runId), `Expected run ${runId} to be in merged results`);
+        }
+        // Check counts match
+        assert.strictEqual(actualRunIds.size, expectedRunIds.size);
+    });
+});
+/**
+ * Create a mock evaluation result for testing
+ * @param runId
+ * @param sut
+ * @param caseId
+ */
+const createMockResult = (runId, sut, caseId) => ({
+    run: {
+        runId,
+        sut,
+        sutRole: "primary",
+        sutVersion: "1.0.0",
+        caseId,
+        caseClass: "test-class",
+        seed: 42,
+        repetition: 0,
+    },
+    correctness: {
+        expectedExists: false,
+        producedOutput: true,
+        valid: true,
+        matchesExpected: null,
+    },
+    outputs: { summary: {} },
+    metrics: { numeric: { test: 1 } },
+    provenance: {
+        runtime: { platform: "linux", arch: "x64", nodeVersion: "v22.0.0" },
+    },
+});
+//# sourceMappingURL=checkpoint-merge-bug.unit.test.js.map

package/dist/executor/__tests__/checkpoint-merge-bug.unit.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"checkpoint-merge-bug.unit.test.js","sourceRoot":"","sources":["../../../src/executor/__tests__/checkpoint-merge-bug.unit.test.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,EAAE,WAAW,EAAE,MAAM,aAAa,CAAC;AAC1C,OAAO,EAAE,YAAY,EAAE,MAAM,EAAE,MAAM,SAAS,CAAC;AAC/C,OAAO,EAAE,MAAM,EAAE,MAAM,SAAS,CAAC;AACjC,OAAO,EAAE,IAAI,EAAE,MAAM,WAAW,CAAC;AAEjC,OAAO,EAAE,SAAS,EAAE,UAAU,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,WAAW,CAAC;AAChE,OAAO,MAAM,MAAM,oBAAoB,CAAC;AAIxC,OAAO,EAAE,iBAAiB,EAAE,MAAM,0BAA0B,CAAC;AAC7D,OAAO,EAAE,WAAW,EAAE,MAAM,0BAA0B,CAAC;AAEvD,QAAQ,CAAC,kCAAkC,EAAE,GAAG,EAAE;IACjD,IAAI,OAAe,CAAC;IACpB,IAAI,cAAiC,CAAC;IACtC,IAAI,QAAgB,CAAC;IAErB,UAAU,CAAC,GAAG,EAAE;QACf,OAAO,GAAG,IAAI,CAAC,MAAM,EAAE,EAAE,yBAAyB,WAAW,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC;QACpF,QAAQ,GAAG,IAAI,CAAC,OAAO,EAAE,iBAAiB,CAAC,CAAC;QAC5C,cAAc,GAAG,IAAI,iBAAiB,CAAC,EAAE,OAAO,EAAE,IAAI,WAAW,CAAC,QAAQ,CAAC,EAAE,CAAC,CAAC;IAChF,CAAC,CAAC,CAAC;IAEH;;;OAGG;IACH,MAAM,kBAAkB,GAAG,CAAC,IAAY,EAAkB,EAAE;QAC3D,MAAM,IAAI,GAAG,IAAI,CAAC,KAAK,CAAC,YAAY,CAAC,IAAI,EAAE,OAAO,CAAC,CAAC,CAAC;QACrD,OAAO,IAAsB,CAAC;IAC/B,CAAC,CAAC;IAEF,SAAS,CAAC,GAAG,EAAE;QACd,MAAM,CAAC,OAAO,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;IACnD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sEAAsE,EAAE,KAAK,IAAI,EAAE;QACrF,sDAAsD;QACtD,MAAM,IAAI,GAAqB,gBAAgB,CAAC,SAAS,EAAE,OAAO,EAAE,QAAQ,CAAC,CAAC;QAC9E,MAAM,IAAI,GAAqB,gBAAgB,CAAC,SAAS,EAAE,OAAO,EAAE,QAAQ,CAAC,CAAC;QAC9E,MAAM,IAAI,GAAqB,gBAAgB,CAAC,SAAS,EAAE,OAAO,EAAE,QAAQ,CAAC,CAAC;QAE9E,MAAM,cAAc,CAAC,eAAe,CAAC,IAAI,CAAC,CAAC;QAC3C,MAAM,cAAc,CAAC,eAAe,CAAC,IAAI,CAAC,CAAC;QAC3C,MAAM,cAAc,CAAC,eAAe,CAAC,IAAI,CAAC,CAAC;QAE3C,MAAM,QAAQ,GAAG,kBAAkB,CAAC,QAAQ,CAAC,CAAC;QAC9C,MAAM,CAAC,WAAW,CAAC,QAAQ,CAAC,eAAe,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;QAEvD,4CAA4C;QAC5C,MAAM,SAAS,GAAG,IAAI,CAAC,OAAO,EAAE,2BAA2B,CAAC,CAAC;QAC7D,MAAM,YAAY,GAAG,IAAI,WAAW,CAAC,SAAS,CAAC,CAAC;QAChD,MAAM,YAAY,CAAC,IAAI,CAAC;YACvB,UAAU,EAAE,WAAW;YACvB,SAAS,EAAE,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE;YACnC,SAAS,EAAE,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE;YACnC,eAAe,EAAE,CAAC,SAAS,CAAC;YAC5B,OAAO,EAAE;gBACR,SAAS,EAAE,gBAAgB,CAAC,SAAS,EAAE,OAAO,EAAE,QAAQ,CAAC;aACzD;YACD,YAAY,EAAE,CAAC;SACf,CAAC,CAAC;QAEH,0CAA0C;QAC1C,MAAM,cAAc,CAAC,IAAI,EAAE,CAAC;QAE5B,8DAA8D;QAC9D,MAAM,MAAM,GAAG,MAAM,cAAc,CAAC,WAAW,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC;QAE7D,MAAM,cAAc,GAAG,kBAAkB,CAAC,QAAQ,CAAC,CAAC;QACpD,OAAO,CAAC,GAAG,CAAC,uBAAuB,EAAE,cAAc,CAAC,eAAe,CAAC,MAAM,CAAC,CAAC;QAC5E,OAAO,CAAC,GAAG,CAAC,cAAc,EAAE,MAAM,CAAC,eAAe,CAAC,MAAM,CAAC,CAAC;QAC3D,OAAO,CAAC,GAAG,CAAC,+CAA+C,CAAC,CAAC;QAE7D,kDAAkD;QAClD,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,eAAe,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;IACtD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,oFAAoF,EAAE,KAAK,IAAI,EAAE;QACnG,kCAAkC;QAClC,MAAM,cAAc,CAAC,eAAe,CAAC,gBAAgB,CAAC,UAAU,EAAE,OAAO,EAAE,QAAQ,CAAC,CAAC,CAAC;QAEtF,MAAM,eAAe,GAAG,kBAAkB,CAAC,QAAQ,CAAC,CAAC;QACrD,OAAO,CAAC,GAAG,CAAC,+BAA+B,EAAE,eAAe,CAAC,eAAe,CAAC,MAAM,CAAC,CAAC;QAErF,gEAAgE;QAChE,MAAM,SAAS,GAAG,IAAI,CAAC,OAAO,EAAE,2BAA2B,CAAC,CAAC;QAC7D,MAAM,YAAY,GAAG,IAAI,WAAW,CAAC,SAAS,CAAC,CAAC;QAChD,MAAM,YAAY,CAAC,IAAI,CAAC;YACvB,UAAU,EAAE,WAAW;YACvB,SAAS,EAAE,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE;YACnC,SAAS,EAAE,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE;YACnC,eAAe,EAAE,EAAE;YACnB,OAAO,EAAE,EAAE;YACX,YAAY,EAAE,GAAG;SACjB,CAAC,CAAC;QAEH,MAAM,cAAc,CAAC,IAAI,EAAE,CAAC;QAE5B,oBAAoB;QACpB,MAAM,cAAc,CAAC,WAAW,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC;QAE9C,MAAM,cAAc,GAAG,kBAAkB,CAAC,QAAQ,CAAC,CAAC;QACpD,OAAO,CAAC,GAAG,CAAC,8BAA8B,EAAE,cAAc,CAAC,eAAe,CAAC,MAAM,CAAC,CAAC;QAEnF,0CAA0C;QAC1C,MAAM,CAAC,EAAE,CAAC,cAAc,CAAC,eAAe,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC;IACtD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,uEAAuE,EAAE,KAAK,IAAI,EAAE;QACtF,yBAAyB;QACzB,MAAM,cAAc,CAAC,eAAe,CAAC,gBAAgB,CAAC,UAAU,EAAE,OAAO,EAAE,QAAQ,CAAC,CAAC,CAAC;QACtF,MAAM,cAAc,CAAC,eAAe,CAAC,gBAAgB,CAAC,UAAU,EAAE,OAAO,EAAE,QAAQ,CAAC,CAAC,CAAC;QAEtF,MAAM,cAAc,GAAG,kBAAkB,CAAC,QAAQ,CAAC,CAAC;QACpD,MAAM,QAAQ,GAAG,cAAc,CAAC,eAAe,CAAC;QAChD,OAAO,CAAC,GAAG,CAAC,2BAA2B,EAAE,QAAQ,CAAC,CAAC;QAEnD,sBAAsB;QACtB,MAAM,SAAS,GAAG,IAAI,CAAC,OAAO,EAAE,2BAA2B,CAAC,CAAC;QAC7D,MAAM,YAAY,GAAG,IAAI,WAAW,CAAC,SAAS,CAAC,CAAC;QAChD,MAAM,YAAY,CAAC,IAAI,CAAC;YACvB,UAAU,EAAE,WAAW;YACvB,SAAS,EAAE,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE;YACnC,SAAS,EAAE,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE;YACnC,eAAe,EAAE,CAAC,WAAW,CAAC;YAC9B,OAAO,EAAE;gBACR,WAAW,EAAE,gBAAgB,CAAC,WAAW,EAAE,OAAO,EAAE,QAAQ,CAAC;aAC7D;YACD,YAAY,EAAE,GAAG;SACjB,CAAC,CAAC;QAEH,yBAAyB;QACzB,MAAM,cAAc,CAAC,IAAI,EAAE,CAAC;QAE5B,QAAQ;QACR,MAAM,MAAM,GAAG,MAAM,cAAc,CAAC,WAAW,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC;QAE7D,MAAM,aAAa,GAAG,kBAAkB,CAAC,QAAQ,CAAC,CAAC;QACnD,OAAO,CAAC,GAAG,CAAC,cAAc,CAAC,CAAC;QAC5B,OAAO,CAAC,GAAG,CAAC,yBAAyB,EAAE,aAAa,CAAC,eAAe,CAAC,CAAC;QACtE,OAAO,CAAC,GAAG,CAAC,gBAAgB,EAAE,MAAM,CAAC,eAAe,CAAC,CAAC;QACtD,OAAO,CAAC,GAAG,CAAC,iDAAiD,CAAC,CAAC;QAC/D,OAAO,CAAC,GAAG,CAAC,WAAW,EAAE,MAAM,CAAC,eAAe,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;QAEhE,2CAA2C;QAC3C,MAAM,cAAc,GAAG,IAAI,GAAG,CAAC,CAAC,GAAG,QAAQ,EAAE,WAAW,CAAC,CAAC,CAAC;QAC3D,MAAM,YAAY,GAAG,IAAI,GAAG,CAAC,MAAM,CAAC,eAAe,CAAC,CAAC;QAErD,OAAO,CAAC,GAAG,CACV,iBAAiB,EACjB,CAAC,GAAG,cAAc,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,YAAY,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CACvD,CAAC;QAEF,sCAAsC;QACtC,KAAK,MAAM,KAAK,IAAI,cAAc,EAAE,CAAC;YACpC,MAAM,CAAC,EAAE,CAAC,YAAY,CAAC,GAAG,CAAC,KAAK,CAAC,EAAE,gBAAgB,KAAK,0BAA0B,CAAC,CAAC;QACrF,CAAC;QACD,qBAAqB;QACrB,MAAM,CAAC,WAAW,CAAC,YAAY,CAAC,IAAI,EAAE,cAAc,CAAC,IAAI,CAAC,CAAC;IAC5D,CAAC,CAAC,CAAC;AACJ,CAAC,CAAC,CAAC;AAEH;;;;;GAKG;AACH,MAAM,gBAAgB,GAAG,CAAC,KAAa,EAAE,GAAW,EAAE,MAAc,EAAoB,EAAE,CAAC,CAAC;IAC3F,GAAG,EAAE;QACJ,KAAK;QACL,GAAG;QACH,OAAO,EAAE,SAAkB;QAC3B,UAAU,EAAE,OAAO;QACnB,MAAM;QACN,SAAS,EAAE,YAAY;QACvB,IAAI,EAAE,EAAE;QACR,UAAU,EAAE,CAAC;KACb;IACD,WAAW,EAAE;QACZ,cAAc,EAAE,KAAK;QACrB,cAAc,EAAE,IAAI;QACpB,KAAK,EAAE,IAAI;QACX,eAAe,EAAE,IAAI;KACrB;IACD,OAAO,EAAE,EAAE,OAAO,EAAE,EAAE,EAAE;IACxB,OAAO,EAAE,EAAE,OAAO,EAAE,EAAE,IAAI,EAAE,CAAC,EAAE,EAAE;IACjC,UAAU,EAAE;QACX,OAAO,EAAE,EAAE,QAAQ,EAAE,OAAO,EAAE,IAAI,EAAE,KAAK,EAAE,WAAW,EAAE,SAAS,EAAE;KACnE;CACD,CAAC,CAAC"}

package/dist/executor/__tests__/checkpoint-storage.unit.test.d.ts

@@ -0,0 +1,7 @@
+/**
+ * Unit tests for CheckpointStorage
+ *
+ * Tests FileStorage, GitStorage, FileSystem, and Lock implementations.
+ */
+export {};
+//# sourceMappingURL=checkpoint-storage.unit.test.d.ts.map

package/dist/executor/__tests__/checkpoint-storage.unit.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"checkpoint-storage.unit.test.d.ts","sourceRoot":"","sources":["../../../src/executor/__tests__/checkpoint-storage.unit.test.ts"],"names":[],"mappings":"AAAA;;;;GAIG"}

package/dist/executor/__tests__/checkpoint-storage.unit.test.js

@@ -0,0 +1,490 @@
+/**
+ * Unit tests for CheckpointStorage
+ *
+ * Tests FileStorage, GitStorage, FileSystem, and Lock implementations.
+ */
+import { describe, it, beforeEach } from "node:test";
+import assert from "node:assert/strict";
+import { FileStorage, GitStorage, InMemoryLock, NodeFileSystem, createCheckpointStorage, getGitNamespace, } from "../checkpoint-storage.js";
+/**
+ * Mock file system for testing.
+ * Handles both relative and absolute paths.
+ */
+class MockFileSystem {
+    data = new Map();
+    throwsOnRead = new Set();
+    throwsOnWrite = new Set();
+    // Try multiple possible path resolutions
+    resolvePath(path) {
+        // Direct match
+        if (this.data.has(path)) {
+            return path;
+        }
+        // Try with leading slash if not present
+        if (!path.startsWith("/") && this.data.has(`/${path}`)) {
+            return `/${path}`;
+        }
+        // Try without leading slash if present
+        if (path.startsWith("/") && this.data.has(path.slice(1))) {
+            return path.slice(1);
+        }
+        return path;
+    }
+    async readFile(path) {
+        // Try multiple path resolutions
+        const pathsToTry = [
+            path,
+            path.startsWith("/") ? path : `/${path}`,
+            path.startsWith("/") ? path.slice(1) : path,
+        ];
+        for (const tryPath of pathsToTry) {
+            if (this.data.has(tryPath) && this.throwsOnRead.has(tryPath)) {
+                throw new Error(`Mock read error: ${tryPath}`);
+            }
+            const content = this.data.get(tryPath);
+            if (content !== undefined) {
+                return content;
+            }
+        }
+        throw new Error(`File not found: ${path}`);
+    }
+    async writeFile(path, content) {
+        // Store with multiple path variants
+        this.data.set(path, content);
+        if (!path.startsWith("/")) {
+            this.data.set(`/${path}`, content);
+        }
+        this.data.set(`/${path}`, content);
+        const resolvedPath = this.resolvePath(path) ?? path;
+        if (this.throwsOnWrite.has(resolvedPath)) {
+            throw new Error(`Mock write error: ${resolvedPath}`);
+        }
+    }
+    async mkdir(_path, _options) {
+        // No-op for mock
+    }
+    async unlink(path) {
+        // Delete all variants
+        this.data.delete(path);
+        if (!path.startsWith("/")) {
+            this.data.delete(`/${path}`);
+        }
+        this.data.delete(`/${path}`);
+        const resolvedPath = this.resolvePath(path) ?? path;
+        this.data.delete(resolvedPath);
+    }
+    async access(_path) {
+        // No-op for mock
+    }
+    async readdir(path) {
+        const resolvedPath = this.resolvePath(path) ?? path;
+        const prefix = resolvedPath.endsWith("/") ? resolvedPath : `${resolvedPath}/`;
+        return [...this.data.keys()]
+            .filter((k) => k.startsWith(prefix))
+            .map((k) => k.slice(prefix.length))
+            .filter((k) => !k.includes("/"));
+    }
+    setFile(path, content) {
+        // Store with multiple path variants for flexibility
+        this.data.set(path, content);
+        if (!path.startsWith("/")) {
+            this.data.set(`/${path}`, content);
+        }
+        // Also store with cwd prefix (common during tests)
+        this.data.set(`/${path}`, content);
+    }
+    getFile(path) {
+        const resolvedPath = this.resolvePath(path) ?? path;
+        // Check multiple path variants
+        if (this.data.has(resolvedPath)) {
+            return this.data.get(resolvedPath);
+        }
+        // Also try with the full resolved path if provided
+        const normalizedPath = path.startsWith("/") ? path : `/${path}`;
+        if (this.data.has(normalizedPath)) {
+            return this.data.get(normalizedPath);
+        }
+        // Try stripping leading slash if it exists
+        const strippedPath = path.startsWith("/") ? path.slice(1) : path;
+        if (this.data.has(strippedPath)) {
+            return this.data.get(strippedPath);
+        }
+        return undefined;
+    }
+    clear() {
+        this.data.clear();
+        this.throwsOnRead.clear();
+        this.throwsOnWrite.clear();
+    }
+    setThrowsOnRead(path) {
+        this.throwsOnRead.add(path);
+        if (!path.startsWith("/")) {
+            this.throwsOnRead.add(`/${path}`);
+        }
+    }
+    setThrowsOnWrite(path) {
+        this.throwsOnWrite.add(path);
+        if (!path.startsWith("/")) {
+            this.throwsOnWrite.add(`/${path}`);
+        }
+    }
+}
+describe("FileSystem", () => {
+    describe("NodeFileSystem", () => {
+        it("should have all required methods", () => {
+            const fs = new NodeFileSystem();
+            assert.ok(fs.readFile instanceof Function);
+            assert.ok(fs.writeFile instanceof Function);
+            assert.ok(fs.mkdir instanceof Function);
+            assert.ok(fs.unlink instanceof Function);
+            assert.ok(fs.access instanceof Function);
+            assert.ok(fs.readdir instanceof Function);
+        });
+    });
+    describe("MockFileSystem", () => {
+        let mockFs;
+        beforeEach(() => {
+            mockFs = new MockFileSystem();
+        });
+        it("should store and retrieve files", async () => {
+            mockFs.setFile("test.json", '{"test": true}');
+            const content = await mockFs.readFile("test.json");
+            assert.strictEqual(content, '{"test": true}');
+        });
+        it("should throw on missing file", async () => {
+            await assert.rejects(() => mockFs.readFile("missing.json"));
+        });
+        it("should write files", async () => {
+            await mockFs.writeFile("new.json", '{"new": true}');
+            assert.strictEqual(mockFs.getFile("new.json"), '{"new": true}');
+        });
+        it("should delete files", async () => {
+            mockFs.setFile("delete.json", '{"delete": true}');
+            await mockFs.unlink("delete.json");
+            assert.strictEqual(mockFs.getFile("delete.json"), undefined);
+        });
+        it("should list directory contents", async () => {
+            mockFs.setFile("dir/file1.json", "{}");
+            mockFs.setFile("dir/file2.json", "{}");
+            mockFs.setFile("other/file.txt", "{}");
+            const entries = await mockFs.readdir("dir");
+            assert.ok(entries.includes("file1.json"));
+            assert.ok(entries.includes("file2.json"));
+            assert.ok(!entries.includes("file.txt"));
+        });
+    });
+});
+describe("Lock", () => {
+    describe("InMemoryLock", () => {
+        it("should allow concurrent acquisitions with queueing", async () => {
+            const lock = new InMemoryLock();
+            const results = [];
+            // First acquisition
+            void lock.acquire().then(async () => {
+                results.push(1);
+                await new Promise((resolve) => setTimeout(resolve, 10));
+                lock.release();
+            });
+            // Second acquisition (should wait)
+            void lock.acquire().then(() => {
+                results.push(2);
+                lock.release();
+            });
+            // Third acquisition (should wait)
+            void lock.acquire().then(() => {
+                results.push(3);
+                lock.release();
+            });
+            await new Promise((resolve) => setTimeout(resolve, 50));
+            assert.deepStrictEqual(results, [1, 2, 3]);
+        });
+        it("should release immediately when no waiters", () => {
+            const lock = new InMemoryLock();
+            lock.release(); // Should not throw
+            lock.release(); // Should not throw
+        });
+    });
+});
+describe("FileStorage", () => {
+    let mockFs;
+    let storage;
+    beforeEach(() => {
+        mockFs = new MockFileSystem();
+        storage = new FileStorage("test/checkpoint.json", mockFs);
+    });
+    /**
+     * Helper to set file content using the storage's resolved path.
+     * @param content
+     */
+    const setStorageFile = (content) => {
+        const resolvedPath = storage.getPath();
+        mockFs.data.set(resolvedPath, content);
+    };
+    /**
+     * Helper to get file content using the storage's resolved path.
+     */
+    const getStorageFile = () => {
+        const resolvedPath = storage.getPath();
+        return mockFs.data.get(resolvedPath);
+    };
+    describe("load", () => {
+        it("should parse valid JSON checkpoint", async () => {
+            const checkpoint = {
+                configHash: "abc123",
+                createdAt: "2024-01-01T00:00:00.000Z",
+                updatedAt: "2024-01-01T00:00:00.000Z",
+                completedRunIds: ["run1", "run2"],
+                results: {},
+                totalPlanned: 10,
+            };
+            setStorageFile(JSON.stringify(checkpoint));
+            const loaded = await storage.load();
+            assert.deepStrictEqual(loaded, checkpoint);
+        });
+        it("should return null for missing file", async () => {
+            const loaded = await storage.load();
+            assert.strictEqual(loaded, null);
+        });
+        it("should return null for invalid JSON", async () => {
+            setStorageFile("not json");
+            const loaded = await storage.load();
+            assert.strictEqual(loaded, null);
+        });
+        it("should return null on read error", async () => {
+            const resolvedPath = storage.getPath();
+            mockFs.setThrowsOnRead(resolvedPath);
+            const loaded = await storage.load();
+            assert.strictEqual(loaded, null);
+        });
+    });
+    describe("save", () => {
+        it("should write checkpoint with updated timestamp", async () => {
+            const checkpoint = {
+                configHash: "abc123",
+                createdAt: "2024-01-01T00:00:00.000Z",
+                updatedAt: "2024-01-01T00:00:00.000Z",
+                completedRunIds: ["run1"],
+                results: {},
+                totalPlanned: 10,
+            };
+            await storage.save(checkpoint);
+            const saved = getStorageFile();
+            assert.ok(saved);
+            const parsed = JSON.parse(saved);
+            assert.strictEqual(parsed.configHash, "abc123");
+            assert.notStrictEqual(parsed.updatedAt, "2024-01-01T00:00:00.000Z"); // Should be updated
+        });
+        it("should create parent directory via mkdir", async () => {
+            const checkpoint = {
+                configHash: "abc123",
+                createdAt: "2024-01-01T00:00:00.000Z",
+                updatedAt: "2024-01-01T00:00:00.000Z",
+                completedRunIds: [],
+                results: {},
+                totalPlanned: 0,
+            };
+            await storage.save(checkpoint);
+            assert.ok(getStorageFile());
+        });
+    });
+    describe("exists", () => {
+        it("should return true for valid checkpoint file", async () => {
+            const checkpoint = {
+                configHash: "abc123",
+                createdAt: "2024-01-01T00:00:00.000Z",
+                updatedAt: "2024-01-01T00:00:00.000Z",
+                completedRunIds: [],
+                results: {},
+                totalPlanned: 0,
+            };
+            setStorageFile(JSON.stringify(checkpoint));
+            const exists = await storage.exists();
+            assert.strictEqual(exists, true);
+        });
+        it("should return false for missing file", async () => {
+            const exists = await storage.exists();
+            assert.strictEqual(exists, false);
+        });
+        it("should return false for invalid JSON", async () => {
+            setStorageFile("not json");
+            const exists = await storage.exists();
+            assert.strictEqual(exists, false);
+        });
+    });
+    describe("delete", () => {
+        it("should remove checkpoint file", async () => {
+            setStorageFile('{"test": true}');
+            await storage.delete();
+            assert.strictEqual(getStorageFile(), undefined);
+        });
+        it("should not throw when deleting missing file", async () => {
+            await assert.doesNotReject(() => storage.delete());
+        });
+    });
+    describe("getPath", () => {
+        it("should return absolute path", () => {
+            const path = storage.getPath();
+            assert.ok(path.includes("checkpoint.json"));
+        });
+    });
+    describe("findShards", () => {
+        it("should find all worker checkpoint files", async () => {
+            mockFs.setFile("results/execute/checkpoint-worker-00.json", "{}");
+            mockFs.setFile("results/execute/checkpoint-worker-01.json", "{}");
+            mockFs.setFile("results/execute/checkpoint-worker-02.json", "{}");
+            mockFs.setFile("results/execute/other.json", "{}");
+            const shards = await FileStorage.findShards("results/execute", mockFs);
+            assert.strictEqual(shards.length, 3);
+            assert.ok(shards[0].includes("checkpoint-worker-00.json"));
+            assert.ok(shards[1].includes("checkpoint-worker-01.json"));
+            assert.ok(shards[2].includes("checkpoint-worker-02.json"));
+        });
+        it("should sort shards by worker index", async () => {
+            mockFs.setFile("results/execute/checkpoint-worker-02.json", "{}");
+            mockFs.setFile("results/execute/checkpoint-worker-00.json", "{}");
+            mockFs.setFile("results/execute/checkpoint-worker-01.json", "{}");
+            const shards = await FileStorage.findShards("results/execute", mockFs);
+            assert.ok(shards[0].includes("checkpoint-worker-00.json"));
+            assert.ok(shards[1].includes("checkpoint-worker-01.json"));
+            assert.ok(shards[2].includes("checkpoint-worker-02.json"));
+        });
+        it("should return empty array when directory has no shards", async () => {
+            const shards = await FileStorage.findShards("results/execute", mockFs);
+            assert.deepStrictEqual(shards, []);
+        });
+        it("should return empty array when directory does not exist", async () => {
+            const shards = await FileStorage.findShards("nonexistent", mockFs);
+            assert.deepStrictEqual(shards, []);
+        });
+    });
+    describe("shardPath", () => {
+        it("should generate zero-padded shard paths", () => {
+            const path0 = FileStorage.shardPath("results/execute", 0);
+            const path1 = FileStorage.shardPath("results/execute", 1);
+            const path10 = FileStorage.shardPath("results/execute", 10);
+            assert.ok(path0.includes("checkpoint-worker-00.json"));
+            assert.ok(path1.includes("checkpoint-worker-01.json"));
+            assert.ok(path10.includes("checkpoint-worker-10.json"));
+        });
+    });
+});
+describe("GitStorage", () => {
+    describe("load", () => {
+        it("should return null when git is not available", async () => {
+            const storage = new GitStorage("test-namespace", "/nonexistent");
+            const loaded = storage.load();
+            assert.strictEqual(loaded, null);
+        });
+    });
+    describe("save", () => {
+        it("should not throw when git is not available", async () => {
+            const storage = new GitStorage("test-namespace", "/nonexistent");
+            const checkpoint = {
+                configHash: "abc123",
+                createdAt: "2024-01-01T00:00:00.000Z",
+                updatedAt: "2024-01-01T00:00:00.000Z",
+                completedRunIds: [],
+                results: {},
+                totalPlanned: 0,
+            };
+            // Should not throw when git is not available
+            await assert.doesNotReject(() => storage.save(checkpoint));
+        });
+    });
+    describe("delete", () => {
+        it("should not throw when git is not available", () => {
+            const storage = new GitStorage("test-namespace", "/nonexistent");
+            // Should not throw when git is not available
+            assert.doesNotThrow(() => {
+                storage.delete();
+            });
+        });
+    });
+    describe("exists", () => {
+        it("should return false when git is not available", () => {
+            const storage = new GitStorage("test-namespace", "/nonexistent");
+            assert.strictEqual(storage.exists(), false);
+        });
+    });
+    describe("listHistory", () => {
+        it("should return empty array when git is not available", () => {
+            const storage = new GitStorage("test-namespace", "/nonexistent");
+            const history = storage.listHistory();
+            assert.deepStrictEqual(history, []);
+        });
+    });
+    describe("restoreFromCommit", () => {
+        it("should return null when git is not available", () => {
+            const storage = new GitStorage("test-namespace", "/nonexistent");
+            const restored = storage.restoreFromCommit("abc123");
+            assert.strictEqual(restored, null);
+        });
+    });
+    describe("type", () => {
+        it("should have type 'git'", () => {
+            const storage = new GitStorage("test-namespace");
+            assert.strictEqual(storage.type, "git");
+        });
+    });
+});
+describe("createCheckpointStorage", () => {
+    describe("mode selection", () => {
+        it("should create FileStorage for 'file' mode", () => {
+            const storage = createCheckpointStorage("file", "results/checkpoint.json");
+            assert.ok(storage instanceof FileStorage);
+            assert.strictEqual(storage.type, "file");
+        });
+        it("should create GitStorage for 'git' mode", () => {
+            const storage = createCheckpointStorage("git", "results-execute");
+            assert.ok(storage instanceof GitStorage);
+            assert.strictEqual(storage.type, "git");
+        });
+        it("should create GitStorage for 'auto' mode in git directory", () => {
+            // In a git directory with commits (like this repo), 'auto' should create GitStorage
+            const storage = createCheckpointStorage("auto", "results-execute");
+            assert.ok(storage instanceof GitStorage);
+            assert.strictEqual(storage.type, "git");
+        });
+    });
+    describe("with repoRoot", () => {
+        it("should pass repoRoot to GitStorage", () => {
+            const storage = createCheckpointStorage("git", "results-execute", "/custom/root");
+            assert.ok(storage instanceof GitStorage);
+            assert.strictEqual(storage.type, "git");
+        });
+    });
+});
+describe("getGitNamespace", () => {
+    it("should convert forward slashes to hyphens", () => {
+        assert.strictEqual(getGitNamespace("results/execute"), "results-execute");
+        assert.strictEqual(getGitNamespace("a/b/c/d"), "a-b-c-d");
+    });
+    it("should convert backslashes to hyphens", () => {
+        assert.strictEqual(getGitNamespace("results\\execute"), "results-execute");
+        assert.strictEqual(getGitNamespace("a\\b\\c"), "a-b-c");
+    });
+    it("should handle leading dot (slash after dot becomes hyphen, then dot is removed)", () => {
+        // "./results" -> ".-results" (slash becomes hyphen) -> "-results" (dot removed)
+        assert.strictEqual(getGitNamespace("./results"), "-results");
+        // ".results" -> ".results" (no slashes) -> "results" (dot removed)
+        assert.strictEqual(getGitNamespace(".results"), "results");
+    });
+    it("should handle leading slashes (replaced with hyphens, kept because last step only removes slashes)", () => {
+        // "/results/execute" -> "-results-execute" (slashes become hyphens)
+        assert.strictEqual(getGitNamespace("/results/execute"), "-results-execute");
+        // "//results" -> "--results" (slashes become hyphens)
+        assert.strictEqual(getGitNamespace("//results"), "--results");
+    });
+    it("should handle mixed separators", () => {
+        assert.strictEqual(getGitNamespace("results\\sub/dir"), "results-sub-dir");
+    });
+    it("should handle simple paths", () => {
+        assert.strictEqual(getGitNamespace("results"), "results");
+        assert.strictEqual(getGitNamespace("execute"), "execute");
+    });
+    it("should handle paths with only slashes", () => {
+        // "///a//b///" -> "---a--b---" (slashes become hyphens)
+        assert.strictEqual(getGitNamespace("///a//b///"), "---a--b---");
+    });
+});
+//# sourceMappingURL=checkpoint-storage.unit.test.js.map