pushwork 2.0.0-a.sub.1 → 2.0.0-preview.2

package/test/integration/in-memory-sync.test.ts

@@ -1,830 +0,0 @@
-/**
- * Sync Reliability Tests
- *
- * These tests verify sync reliability using the CLI subprocess pattern
- * (same as fuzzer.test.ts) but with convergence-based assertions.
- * 
- * Key difference from fuzzer tests: instead of fixed delays, we use
- * convergence detection to know when sync is complete.
- */
-
-import * as fs from "fs/promises";
-import * as path from "path";
-import * as tmp from "tmp";
-import { execFile } from "child_process";
-import { promisify } from "util";
-import * as crypto from "crypto";
-
-const execFilePromise = promisify(execFile);
-
-// Path to the pushwork CLI
-const PUSHWORK_CLI = path.join(__dirname, "../../dist/cli.js");
-
-/**
- * Execute pushwork CLI command
- */
-async function pushwork(
-  args: string[],
-  cwd: string
-): Promise<{ stdout: string; stderr: string }> {
-  try {
-    const result = await execFilePromise("node", [PUSHWORK_CLI, ...args], {
-      cwd,
-      env: { ...process.env, FORCE_COLOR: "0" },
-    });
-    return result;
-  } catch (error: any) {
-    throw new Error(
-      `pushwork ${args.join(" ")} failed: ${error.message}\nstdout: ${error.stdout}\nstderr: ${error.stderr}`
-    );
-  }
-}
-
-/**
- * Compute hash of all files in a directory (excluding .pushwork)
- */
-async function hashDirectory(dirPath: string): Promise<string> {
-  const files = await getAllFiles(dirPath);
-  const hash = crypto.createHash("sha256");
-
-  files.sort();
-
-  for (const file of files) {
-    if (file.includes(".pushwork")) continue;
-
-    const fullPath = path.join(dirPath, file);
-    const content = await fs.readFile(fullPath);
-
-    hash.update(file);
-    hash.update(content);
-  }
-
-  return hash.digest("hex");
-}
-
-/**
- * Recursively get all files in a directory
- */
-async function getAllFiles(
-  dirPath: string,
-  basePath: string = dirPath
-): Promise<string[]> {
-  const entries = await fs.readdir(dirPath, { withFileTypes: true });
-  const files: string[] = [];
-
-  for (const entry of entries) {
-    const fullPath = path.join(dirPath, entry.name);
-    const relativePath = path.relative(basePath, fullPath);
-
-    if (entry.isDirectory()) {
-      if (entry.name === ".pushwork") continue;
-      const subFiles = await getAllFiles(fullPath, basePath);
-      files.push(...subFiles);
-    } else if (entry.isFile()) {
-      files.push(relativePath);
-    }
-  }
-
-  return files;
-}
-
-/**
- * Check if a path exists
- */
-async function pathExists(filePath: string): Promise<boolean> {
-  try {
-    await fs.access(filePath);
-    return true;
-  } catch {
-    return false;
-  }
-}
-
-/**
- * Sync until repos converge or max rounds reached.
- * Returns the number of rounds it took to converge, or throws if it didn't.
- * 
- * This is the key helper - instead of fixed delays, we sync until convergence.
- */
-async function syncUntilConverged(
-  repoA: string,
-  repoB: string,
-  options: {
-    maxRounds?: number;
-    timeoutMs?: number;
-  } = {}
-): Promise<{ rounds: number; hashA: string; hashB: string }> {
-  const { maxRounds = 5, timeoutMs = 30000 } = options;
-  const startTime = Date.now();
-
-  for (let round = 1; round <= maxRounds; round++) {
-    if (Date.now() - startTime > timeoutMs) {
-      const hashA = await hashDirectory(repoA);
-      const hashB = await hashDirectory(repoB);
-      throw new Error(
-        `Sync timeout after ${round - 1} rounds and ${Date.now() - startTime}ms. ` +
-        `hashA=${hashA.slice(0, 8)}, hashB=${hashB.slice(0, 8)}`
-      );
-    }
-
-    // Sync both repos (use --gentle for incremental sync)
-    await pushwork(["sync", "--gentle"], repoA);
-    await pushwork(["sync", "--gentle"], repoB);
-
-    // Check if converged
-    const hashA = await hashDirectory(repoA);
-    const hashB = await hashDirectory(repoB);
-
-    if (hashA === hashB) {
-      return { rounds: round, hashA, hashB };
-    }
-  }
-
-  const hashA = await hashDirectory(repoA);
-  const hashB = await hashDirectory(repoB);
-  throw new Error(
-    `Failed to converge after ${maxRounds} sync rounds. ` +
-    `hashA=${hashA.slice(0, 8)}, hashB=${hashB.slice(0, 8)}`
-  );
-}
-
-describe("Sync Reliability Tests", () => {
-  let tmpDir: string;
-  let cleanup: () => void;
-
-  beforeEach(() => {
-    const tmpObj = tmp.dirSync({ unsafeCleanup: true });
-    tmpDir = tmpObj.name;
-    cleanup = tmpObj.removeCallback;
-  });
-
-  afterEach(() => {
-    cleanup();
-  });
-
-  describe("Basic Two-Repo Sync", () => {
-    /**
-     * STRICT TEST: Check state immediately after clone, no extra syncs.
-     * This should expose the same issues as the fuzzer.
-     */
-    it("should have matching state immediately after clone (strict)", async () => {
-      const repoA = path.join(tmpDir, "repo-a");
-      const repoB = path.join(tmpDir, "repo-b");
-      await fs.mkdir(repoA);
-      await fs.mkdir(repoB);
-
-      // Create file and init A
-      await fs.writeFile(path.join(repoA, "test.txt"), "Hello from A");
-      await pushwork(["init", "."], repoA);
-
-      // Clone to B (no extra syncs!)
-      const { stdout: rootUrl } = await pushwork(["url"], repoA);
-      await pushwork(["clone", rootUrl.trim(), repoB], tmpDir);
-
-      // STRICT: Check immediately, no syncUntilConverged
-      const hashA = await hashDirectory(repoA);
-      const hashB = await hashDirectory(repoB);
-
-      // Debug output if they don't match
-      if (hashA !== hashB) {
-        const filesA = await getAllFiles(repoA);
-        const filesB = await getAllFiles(repoB);
-        console.log("MISMATCH DETECTED:");
-        console.log("  repoA files:", filesA.filter(f => !f.includes(".pushwork")));
-        console.log("  repoB files:", filesB.filter(f => !f.includes(".pushwork")));
-        console.log("  hashA:", hashA.slice(0, 16));
-        console.log("  hashB:", hashB.slice(0, 16));
-      }
-
-      expect(hashA).toBe(hashB);
-
-      // Verify file exists in both
-      expect(await pathExists(path.join(repoA, "test.txt"))).toBe(true);
-      expect(await pathExists(path.join(repoB, "test.txt"))).toBe(true);
-
-      // Verify content matches
-      const contentA = await fs.readFile(path.join(repoA, "test.txt"), "utf-8");
-      const contentB = await fs.readFile(path.join(repoB, "test.txt"), "utf-8");
-      expect(contentA).toBe("Hello from A");
-      expect(contentB).toBe("Hello from A");
-    }, 30000);
-
-    it("should sync a file from A to B (with convergence)", async () => {
-      const repoA = path.join(tmpDir, "repo-a");
-      const repoB = path.join(tmpDir, "repo-b");
-      await fs.mkdir(repoA);
-      await fs.mkdir(repoB);
-
-      // Create file and init A
-      await fs.writeFile(path.join(repoA, "test.txt"), "Hello from A");
-      await pushwork(["init", "."], repoA);
-
-      // Clone to B
-      const { stdout: rootUrl } = await pushwork(["url"], repoA);
-      await pushwork(["clone", rootUrl.trim(), repoB], tmpDir);
-
-      // Verify convergence (allows retries)
-      const { rounds, hashA, hashB } = await syncUntilConverged(repoA, repoB);
-
-      expect(hashA).toBe(hashB);
-      expect(rounds).toBeLessThanOrEqual(2); // Should converge quickly
-
-      // Verify content
-      const contentA = await fs.readFile(path.join(repoA, "test.txt"), "utf-8");
-      const contentB = await fs.readFile(path.join(repoB, "test.txt"), "utf-8");
-      expect(contentA).toBe(contentB);
-      expect(contentA).toBe("Hello from A");
-    }, 30000);
-
-    it("should sync a new file added to B back to A", async () => {
-      const repoA = path.join(tmpDir, "repo-a");
-      const repoB = path.join(tmpDir, "repo-b");
-      await fs.mkdir(repoA);
-      await fs.mkdir(repoB);
-
-      // Init A with initial file
-      await fs.writeFile(path.join(repoA, "initial.txt"), "initial");
-      await pushwork(["init", "."], repoA);
-
-      // Clone to B
-      const { stdout: rootUrl } = await pushwork(["url"], repoA);
-      await pushwork(["clone", rootUrl.trim(), repoB], tmpDir);
-
-      // Initial convergence
-      await syncUntilConverged(repoA, repoB);
-
-      // B creates new file
-      await fs.writeFile(path.join(repoB, "from-b.txt"), "Created by B");
-
-      // Sync until converged
-      const { rounds } = await syncUntilConverged(repoA, repoB);
-
-      expect(rounds).toBeLessThanOrEqual(3);
-
-      // Verify A got B's file
-      expect(await pathExists(path.join(repoA, "from-b.txt"))).toBe(true);
-      const content = await fs.readFile(path.join(repoA, "from-b.txt"), "utf-8");
-      expect(content).toBe("Created by B");
-    }, 30000);
-
-    it("should sync subdirectories correctly", async () => {
-      const repoA = path.join(tmpDir, "repo-a");
-      const repoB = path.join(tmpDir, "repo-b");
-      await fs.mkdir(repoA);
-      await fs.mkdir(repoB);
-
-      // Create nested structure in A
-      await fs.mkdir(path.join(repoA, "subdir"), { recursive: true });
-      await fs.writeFile(path.join(repoA, "subdir", "nested.txt"), "Nested content");
-      await pushwork(["init", "."], repoA);
-
-      // Clone to B
-      const { stdout: rootUrl } = await pushwork(["url"], repoA);
-      await pushwork(["clone", rootUrl.trim(), repoB], tmpDir);
-
-      // Verify convergence
-      const { rounds } = await syncUntilConverged(repoA, repoB);
-
-      expect(rounds).toBeLessThanOrEqual(2);
-
-      // Verify B got the nested file
-      expect(await pathExists(path.join(repoB, "subdir", "nested.txt"))).toBe(true);
-      const content = await fs.readFile(path.join(repoB, "subdir", "nested.txt"), "utf-8");
-      expect(content).toBe("Nested content");
-    }, 30000);
-  });
-
-  describe("Concurrent Operations", () => {
-    it("should handle concurrent file creation on both sides", async () => {
-      const repoA = path.join(tmpDir, "repo-a");
-      const repoB = path.join(tmpDir, "repo-b");
-      await fs.mkdir(repoA);
-      await fs.mkdir(repoB);
-
-      // Init A
-      await fs.writeFile(path.join(repoA, "initial.txt"), "initial");
-      await pushwork(["init", "."], repoA);
-
-      // Clone to B
-      const { stdout: rootUrl } = await pushwork(["url"], repoA);
-      await pushwork(["clone", rootUrl.trim(), repoB], tmpDir);
-
-      // Initial convergence
-      await syncUntilConverged(repoA, repoB);
-
-      // Both create files concurrently (before syncing)
-      await fs.writeFile(path.join(repoA, "file-a.txt"), "From A");
-      await fs.writeFile(path.join(repoB, "file-b.txt"), "From B");
-
-      // Sync until converged
-      const { rounds } = await syncUntilConverged(repoA, repoB);
-
-      expect(rounds).toBeLessThanOrEqual(3);
-
-      // Both should have both files
-      expect(await pathExists(path.join(repoA, "file-a.txt"))).toBe(true);
-      expect(await pathExists(path.join(repoA, "file-b.txt"))).toBe(true);
-      expect(await pathExists(path.join(repoB, "file-a.txt"))).toBe(true);
-      expect(await pathExists(path.join(repoB, "file-b.txt"))).toBe(true);
-    }, 30000);
-
-    it("should handle file modification sync", async () => {
-      const repoA = path.join(tmpDir, "repo-a");
-      const repoB = path.join(tmpDir, "repo-b");
-      await fs.mkdir(repoA);
-      await fs.mkdir(repoB);
-
-      // Init A with file
-      await fs.writeFile(path.join(repoA, "shared.txt"), "Original");
-      await pushwork(["init", "."], repoA);
-
-      // Clone to B
-      const { stdout: rootUrl } = await pushwork(["url"], repoA);
-      await pushwork(["clone", rootUrl.trim(), repoB], tmpDir);
-
-      // Initial convergence
-      await syncUntilConverged(repoA, repoB);
-
-      // A modifies the file
-      await fs.writeFile(path.join(repoA, "shared.txt"), "Modified by A");
-
-      // Sync until converged
-      const { rounds } = await syncUntilConverged(repoA, repoB);
-
-      expect(rounds).toBeLessThanOrEqual(3);
-
-      // B should have the modification
-      const contentB = await fs.readFile(path.join(repoB, "shared.txt"), "utf-8");
-      expect(contentB).toBe("Modified by A");
-    }, 30000);
-
-    it("should handle file deletion sync", async () => {
-      const repoA = path.join(tmpDir, "repo-a");
-      const repoB = path.join(tmpDir, "repo-b");
-      await fs.mkdir(repoA);
-      await fs.mkdir(repoB);
-
-      // Init A with file
-      await fs.writeFile(path.join(repoA, "to-delete.txt"), "Will be deleted");
-      await pushwork(["init", "."], repoA);
-
-      // Clone to B
-      const { stdout: rootUrl } = await pushwork(["url"], repoA);
-      await pushwork(["clone", rootUrl.trim(), repoB], tmpDir);
-
-      // Initial convergence
-      await syncUntilConverged(repoA, repoB);
-
-      // Verify B has the file
-      expect(await pathExists(path.join(repoB, "to-delete.txt"))).toBe(true);
-
-      // A deletes the file
-      await fs.unlink(path.join(repoA, "to-delete.txt"));
-
-      // Sync until converged
-      const { rounds } = await syncUntilConverged(repoA, repoB);
-
-      expect(rounds).toBeLessThanOrEqual(3);
-
-      // File should be deleted in B
-      expect(await pathExists(path.join(repoB, "to-delete.txt"))).toBe(false);
-    }, 30000);
-  });
-
-  describe("Subdirectory File Deletion - Resurrection Bug", () => {
-    it("deleted file in artifact directory should not resurrect", async () => {
-      // Files in artifact directories (dist/ by default) resurrect after sync.
-      // Phase 1 (push) correctly removes the file entry from the directory doc,
-      // but the Automerge merge with the server's version re-introduces it.
-      // Phase 2 (pull) then sees it as a "new remote document" and re-creates it.
-      const repoA = path.join(tmpDir, "repo-a");
-      await fs.mkdir(repoA);
-
-      await fs.mkdir(path.join(repoA, "dist", "assets"), { recursive: true });
-      await fs.writeFile(path.join(repoA, "dist", "assets", "app.js"), "// build 1");
-      await pushwork(["init", "."], repoA);
-      await pushwork(["sync"], repoA);
-
-      // Delete the file
-      await fs.unlink(path.join(repoA, "dist", "assets", "app.js"));
-
-      // Sync - push deletion then pull
-      await pushwork(["sync"], repoA);
-
-      // File should stay deleted
-      expect(await pathExists(path.join(repoA, "dist", "assets", "app.js"))).toBe(false);
-
-      // Sync again - should NOT come back from server
-      await pushwork(["sync"], repoA);
-      expect(await pathExists(path.join(repoA, "dist", "assets", "app.js"))).toBe(false);
-    }, 60000);
-
-    it("deleted file in depth-1 subdirectory should not resurrect (control)", async () => {
-      // Control: depth-1 subdirectories work correctly
-      const repoA = path.join(tmpDir, "repo-a");
-      await fs.mkdir(repoA);
-
-      await fs.mkdir(path.join(repoA, "subdir"), { recursive: true });
-      await fs.writeFile(path.join(repoA, "subdir", "file.txt"), "content");
-      await pushwork(["init", "."], repoA);
-      await pushwork(["sync"], repoA);
-
-      await fs.unlink(path.join(repoA, "subdir", "file.txt"));
-
-      await pushwork(["sync"], repoA);
-      expect(await pathExists(path.join(repoA, "subdir", "file.txt"))).toBe(false);
-
-      await pushwork(["sync"], repoA);
-      expect(await pathExists(path.join(repoA, "subdir", "file.txt"))).toBe(false);
-    }, 60000);
-
-    it("deleted build artifacts should not resurrect after rebuild cycle", async () => {
-      // Real-world scenario: build step creates new hashed files and deletes
-      // old ones in dist/assets/. The deleted files come back from the server.
-      const repoA = path.join(tmpDir, "repo-a");
-      await fs.mkdir(repoA);
-
-      await fs.mkdir(path.join(repoA, "dist", "assets"), { recursive: true });
-      await fs.writeFile(path.join(repoA, "dist", "assets", "app-ABC123.js"), "// build 1");
-      await fs.writeFile(path.join(repoA, "dist", "assets", "vendor-DEF456.js"), "// vendor 1");
-      await fs.writeFile(path.join(repoA, "dist", "index.js"), "// index 1");
-      await pushwork(["init", "."], repoA);
-      await pushwork(["sync"], repoA);
-
-      // Simulate rebuild: new hashed files replace old ones
-      await fs.unlink(path.join(repoA, "dist", "assets", "app-ABC123.js"));
-      await fs.unlink(path.join(repoA, "dist", "assets", "vendor-DEF456.js"));
-      await fs.writeFile(path.join(repoA, "dist", "assets", "app-XYZ789.js"), "// build 2");
-      await fs.writeFile(path.join(repoA, "dist", "assets", "vendor-UVW012.js"), "// vendor 2");
-      await fs.writeFile(path.join(repoA, "dist", "index.js"), "// index 2");
-
-      await pushwork(["sync"], repoA);
-
-      // Old files should be gone, new files should exist
-      expect(await pathExists(path.join(repoA, "dist", "assets", "app-ABC123.js"))).toBe(false);
-      expect(await pathExists(path.join(repoA, "dist", "assets", "vendor-DEF456.js"))).toBe(false);
-      expect(await pathExists(path.join(repoA, "dist", "assets", "app-XYZ789.js"))).toBe(true);
-      expect(await pathExists(path.join(repoA, "dist", "assets", "vendor-UVW012.js"))).toBe(true);
-
-      // Sync again - old files should NOT come back from server
-      await pushwork(["sync"], repoA);
-
-      expect(await pathExists(path.join(repoA, "dist", "assets", "app-ABC123.js"))).toBe(false);
-      expect(await pathExists(path.join(repoA, "dist", "assets", "vendor-DEF456.js"))).toBe(false);
-    }, 60000);
-
-    it("deleted artifact files should not resurrect on clone", async () => {
-      // Two repos: A deletes files in an artifact directory, B should not
-      // see the deleted files after syncing.
-      const repoA = path.join(tmpDir, "repo-a");
-      const repoB = path.join(tmpDir, "repo-b");
-      await fs.mkdir(repoA);
-      await fs.mkdir(repoB);
-
-      await fs.mkdir(path.join(repoA, "dist", "assets"), { recursive: true });
-      await fs.writeFile(path.join(repoA, "dist", "assets", "app-ABC123.js"), "// build 1");
-      await fs.writeFile(path.join(repoA, "dist", "assets", "vendor-DEF456.js"), "// vendor 1");
-      await fs.writeFile(path.join(repoA, "dist", "index.js"), "// index 1");
-      await pushwork(["init", "."], repoA);
-
-      // Clone to B and converge
-      const { stdout: rootUrl } = await pushwork(["url"], repoA);
-      await pushwork(["clone", rootUrl.trim(), repoB], tmpDir);
-      await syncUntilConverged(repoA, repoB);
-
-      expect(await pathExists(path.join(repoB, "dist", "assets", "app-ABC123.js"))).toBe(true);
-
-      // A rebuilds
-      await fs.unlink(path.join(repoA, "dist", "assets", "app-ABC123.js"));
-      await fs.unlink(path.join(repoA, "dist", "assets", "vendor-DEF456.js"));
-      await fs.writeFile(path.join(repoA, "dist", "assets", "app-XYZ789.js"), "// build 2");
-      await fs.writeFile(path.join(repoA, "dist", "assets", "vendor-UVW012.js"), "// vendor 2");
-      await fs.writeFile(path.join(repoA, "dist", "index.js"), "// index 2");
-
-      // Sync A then B
-      await pushwork(["sync"], repoA);
-
-      // A should not have resurrected files
-      expect(await pathExists(path.join(repoA, "dist", "assets", "app-ABC123.js"))).toBe(false);
-      expect(await pathExists(path.join(repoA, "dist", "assets", "vendor-DEF456.js"))).toBe(false);
-
-      await pushwork(["sync"], repoB);
-
-      // B should have new files, NOT old files
-      expect(await pathExists(path.join(repoB, "dist", "assets", "app-ABC123.js"))).toBe(false);
-      expect(await pathExists(path.join(repoB, "dist", "assets", "vendor-DEF456.js"))).toBe(false);
-      expect(await pathExists(path.join(repoB, "dist", "assets", "app-XYZ789.js"))).toBe(true);
-    }, 90000);
-
-    it("deleted file in depth-3 subdirectory should not resurrect", async () => {
-      const repoA = path.join(tmpDir, "repo-a");
-      await fs.mkdir(repoA);
-
-      await fs.mkdir(path.join(repoA, "a", "b", "c"), { recursive: true });
-      await fs.writeFile(path.join(repoA, "a", "b", "c", "deep.txt"), "deep");
-      await pushwork(["init", "."], repoA);
-      await pushwork(["sync"], repoA);
-
-      await fs.unlink(path.join(repoA, "a", "b", "c", "deep.txt"));
-
-      await pushwork(["sync"], repoA);
-      expect(await pathExists(path.join(repoA, "a", "b", "c", "deep.txt"))).toBe(false);
-
-      await pushwork(["sync"], repoA);
-      expect(await pathExists(path.join(repoA, "a", "b", "c", "deep.txt"))).toBe(false);
-    }, 60000);
-
-    it("create+delete in same subdirectory should not resurrect deleted files", async () => {
-      // Regression guard: simultaneous create+delete in the same non-artifact
-      // subdirectory should work. This passes today but we don't want it to regress.
-      const repoA = path.join(tmpDir, "repo-a");
-      await fs.mkdir(repoA);
-
-      await fs.mkdir(path.join(repoA, "subdir"), { recursive: true });
-      await fs.writeFile(path.join(repoA, "subdir", "old.txt"), "old content");
-      await pushwork(["init", "."], repoA);
-      await pushwork(["sync"], repoA);
-
-      // Simultaneously create new file and delete old file in same dir
-      await fs.unlink(path.join(repoA, "subdir", "old.txt"));
-      await fs.writeFile(path.join(repoA, "subdir", "new.txt"), "new content");
-
-      await pushwork(["sync"], repoA);
-
-      expect(await pathExists(path.join(repoA, "subdir", "old.txt"))).toBe(false);
-      expect(await pathExists(path.join(repoA, "subdir", "new.txt"))).toBe(true);
-
-      // Sync again - old file should NOT come back
-      await pushwork(["sync"], repoA);
-
-      expect(await pathExists(path.join(repoA, "subdir", "old.txt"))).toBe(false);
-      expect(await pathExists(path.join(repoA, "subdir", "new.txt"))).toBe(true);
-    }, 60000);
-
-    it("deleted file in depth-2 with sibling dirs should not resurrect", async () => {
-      // The depth-3 test has intermediate dirs (a/b/c) with only one child each.
-      // The dist/assets test has dist/ containing both assets/ (subdir) and
-      // index.js (file). Test if having a file sibling alongside the subdir matters.
-      const repoA = path.join(tmpDir, "repo-a");
-      await fs.mkdir(repoA);
-
-      await fs.mkdir(path.join(repoA, "parent", "child"), { recursive: true });
-      await fs.writeFile(path.join(repoA, "parent", "sibling.txt"), "sibling at parent level");
-      await fs.writeFile(path.join(repoA, "parent", "child", "target.txt"), "will be deleted");
-      await pushwork(["init", "."], repoA);
-      await pushwork(["sync"], repoA);
-
-      await fs.unlink(path.join(repoA, "parent", "child", "target.txt"));
-
-      await pushwork(["sync"], repoA);
-      expect(await pathExists(path.join(repoA, "parent", "child", "target.txt"))).toBe(false);
-      expect(await pathExists(path.join(repoA, "parent", "sibling.txt"))).toBe(true);
-
-      await pushwork(["sync"], repoA);
-      expect(await pathExists(path.join(repoA, "parent", "child", "target.txt"))).toBe(false);
-    }, 60000);
-
-    it("deleted file in root directory should not resurrect", async () => {
-      const repoA = path.join(tmpDir, "repo-a");
-      await fs.mkdir(repoA);
-
-      await fs.writeFile(path.join(repoA, "root-file.txt"), "root content");
-      await fs.writeFile(path.join(repoA, "keep.txt"), "keep this");
-      await pushwork(["init", "."], repoA);
-      await pushwork(["sync"], repoA);
-
-      // Delete file in root
-      await fs.unlink(path.join(repoA, "root-file.txt"));
-
-      await pushwork(["sync"], repoA);
-      expect(await pathExists(path.join(repoA, "root-file.txt"))).toBe(false);
-      expect(await pathExists(path.join(repoA, "keep.txt"))).toBe(true);
-
-      // Sync again - should NOT come back
-      await pushwork(["sync"], repoA);
-      expect(await pathExists(path.join(repoA, "root-file.txt"))).toBe(false);
-    }, 60000);
-
-    it("deleted file in non-artifact subdirectory (src/) should not resurrect", async () => {
-      const repoA = path.join(tmpDir, "repo-a");
-      await fs.mkdir(repoA);
-
-      await fs.mkdir(path.join(repoA, "src"), { recursive: true });
-      await fs.writeFile(path.join(repoA, "src", "index.ts"), "export default 1");
-      await fs.writeFile(path.join(repoA, "src", "helper.ts"), "export function help() {}");
-      await pushwork(["init", "."], repoA);
-      await pushwork(["sync"], repoA);
-
-      // Delete one file in src/
-      await fs.unlink(path.join(repoA, "src", "helper.ts"));
-
-      await pushwork(["sync"], repoA);
-      expect(await pathExists(path.join(repoA, "src", "helper.ts"))).toBe(false);
-      expect(await pathExists(path.join(repoA, "src", "index.ts"))).toBe(true);
-
-      // Sync again - should NOT come back
-      await pushwork(["sync"], repoA);
-      expect(await pathExists(path.join(repoA, "src", "helper.ts"))).toBe(false);
-    }, 60000);
-
-    it("deleted files should not resurrect after multiple sync cycles", async () => {
-      // Simulate real-world usage: multiple syncs over time with deletions
-      const repoA = path.join(tmpDir, "repo-a");
-      await fs.mkdir(repoA);
-
-      await fs.mkdir(path.join(repoA, "src"), { recursive: true });
-      await fs.writeFile(path.join(repoA, "readme.txt"), "readme");
-      await fs.writeFile(path.join(repoA, "src", "app.ts"), "app");
-      await fs.writeFile(path.join(repoA, "src", "old.ts"), "old");
-      await pushwork(["init", "."], repoA);
-      await pushwork(["sync"], repoA);
-
-      // Cycle 1: delete root file
-      await fs.unlink(path.join(repoA, "readme.txt"));
-      await pushwork(["sync"], repoA);
-      expect(await pathExists(path.join(repoA, "readme.txt"))).toBe(false);
-
-      // Cycle 2: delete src file
-      await fs.unlink(path.join(repoA, "src", "old.ts"));
-      await pushwork(["sync"], repoA);
-      expect(await pathExists(path.join(repoA, "src", "old.ts"))).toBe(false);
-
-      // Cycle 3: just sync - nothing should come back
-      await pushwork(["sync"], repoA);
-      expect(await pathExists(path.join(repoA, "readme.txt"))).toBe(false);
-      expect(await pathExists(path.join(repoA, "src", "old.ts"))).toBe(false);
-      expect(await pathExists(path.join(repoA, "src", "app.ts"))).toBe(true);
-    }, 90000);
-
-    it("peer B should not see files deleted by peer A (root)", async () => {
-      const repoA = path.join(tmpDir, "repo-a");
-      const repoB = path.join(tmpDir, "repo-b");
-      await fs.mkdir(repoA);
-      await fs.mkdir(repoB);
-
-      await fs.writeFile(path.join(repoA, "keep.txt"), "keep");
-      await fs.writeFile(path.join(repoA, "delete-me.txt"), "gone");
-      await pushwork(["init", "."], repoA);
-
-      // Clone to B and converge
-      const { stdout: rootUrl } = await pushwork(["url"], repoA);
-      await pushwork(["clone", rootUrl.trim(), repoB], tmpDir);
-      await syncUntilConverged(repoA, repoB);
-
-      expect(await pathExists(path.join(repoB, "delete-me.txt"))).toBe(true);
-
-      // A deletes a root file
-      await fs.unlink(path.join(repoA, "delete-me.txt"));
-      await pushwork(["sync"], repoA);
-
-      // B syncs - should see the deletion
-      await pushwork(["sync"], repoB);
-      expect(await pathExists(path.join(repoB, "delete-me.txt"))).toBe(false);
-      expect(await pathExists(path.join(repoB, "keep.txt"))).toBe(true);
-
-      // B syncs again - should stay deleted
-      await pushwork(["sync"], repoB);
-      expect(await pathExists(path.join(repoB, "delete-me.txt"))).toBe(false);
-    }, 90000);
-
-    it("peer B should not see files deleted by peer A (src/)", async () => {
-      const repoA = path.join(tmpDir, "repo-a");
-      const repoB = path.join(tmpDir, "repo-b");
-      await fs.mkdir(repoA);
-      await fs.mkdir(repoB);
-
-      await fs.mkdir(path.join(repoA, "src"), { recursive: true });
-      await fs.writeFile(path.join(repoA, "src", "index.ts"), "export default 1");
-      await fs.writeFile(path.join(repoA, "src", "old.ts"), "old code");
-      await pushwork(["init", "."], repoA);
-
-      const { stdout: rootUrl } = await pushwork(["url"], repoA);
-      await pushwork(["clone", rootUrl.trim(), repoB], tmpDir);
-      await syncUntilConverged(repoA, repoB);
-
-      expect(await pathExists(path.join(repoB, "src", "old.ts"))).toBe(true);
-
-      // A deletes a file in src/
-      await fs.unlink(path.join(repoA, "src", "old.ts"));
-      await pushwork(["sync"], repoA);
-
-      // B syncs - should see the deletion
-      await pushwork(["sync"], repoB);
-      expect(await pathExists(path.join(repoB, "src", "old.ts"))).toBe(false);
-      expect(await pathExists(path.join(repoB, "src", "index.ts"))).toBe(true);
-
-      // B syncs again - should stay deleted
-      await pushwork(["sync"], repoB);
-      expect(await pathExists(path.join(repoB, "src", "old.ts"))).toBe(false);
-    }, 90000);
-
-    it("peer B should not see files deleted by peer A (dist/)", async () => {
-      const repoA = path.join(tmpDir, "repo-a");
-      const repoB = path.join(tmpDir, "repo-b");
-      await fs.mkdir(repoA);
-      await fs.mkdir(repoB);
-
-      await fs.mkdir(path.join(repoA, "dist", "assets"), { recursive: true });
-      await fs.writeFile(path.join(repoA, "dist", "index.js"), "// index");
-      await fs.writeFile(path.join(repoA, "dist", "assets", "app-ABC.js"), "// build 1");
-      await pushwork(["init", "."], repoA);
-
-      const { stdout: rootUrl } = await pushwork(["url"], repoA);
-      await pushwork(["clone", rootUrl.trim(), repoB], tmpDir);
-      await syncUntilConverged(repoA, repoB);
-
-      expect(await pathExists(path.join(repoB, "dist", "assets", "app-ABC.js"))).toBe(true);
-
-      // A rebuilds: delete old artifact, create new one
-      await fs.unlink(path.join(repoA, "dist", "assets", "app-ABC.js"));
-      await fs.writeFile(path.join(repoA, "dist", "assets", "app-XYZ.js"), "// build 2");
-      await pushwork(["sync"], repoA);
-
-      // A should not have resurrected
-      expect(await pathExists(path.join(repoA, "dist", "assets", "app-ABC.js"))).toBe(false);
-
-      // B syncs - should see new file, NOT old file
-      await pushwork(["sync"], repoB);
-      expect(await pathExists(path.join(repoB, "dist", "assets", "app-ABC.js"))).toBe(false);
-      expect(await pathExists(path.join(repoB, "dist", "assets", "app-XYZ.js"))).toBe(true);
-
-      // B syncs again - old file should stay gone
-      await pushwork(["sync"], repoB);
-      expect(await pathExists(path.join(repoB, "dist", "assets", "app-ABC.js"))).toBe(false);
-    }, 90000);
-
-    it("peer B should see artifact file content update after URL replacement", async () => {
-      // When peer A modifies an artifact file, the document is replaced entirely
-      // (new Automerge doc with a new URL). Peer B's snapshot still points to the
-      // old (now orphaned) URL. detectRemoteChanges sees no head change on the old
-      // doc, and detectNewRemoteDocuments skips paths already in the snapshot.
-      // Without URL replacement detection, B never sees the update.
-      const repoA = path.join(tmpDir, "repo-a");
-      const repoB = path.join(tmpDir, "repo-b");
-      await fs.mkdir(repoA);
-      await fs.mkdir(repoB);
-
-      await fs.mkdir(path.join(repoA, "dist"), { recursive: true });
-      await fs.writeFile(path.join(repoA, "dist", "app.js"), "// version 1");
-      await pushwork(["init", "."], repoA);
-
-      const { stdout: rootUrl } = await pushwork(["url"], repoA);
-      await pushwork(["clone", rootUrl.trim(), repoB], tmpDir);
-      await syncUntilConverged(repoA, repoB);
-
-      const bContentV1 = await fs.readFile(path.join(repoB, "dist", "app.js"), "utf-8");
-      expect(bContentV1).toBe("// version 1");
-
-      // A modifies the artifact file — this triggers nuclear replacement (new URL)
-      await fs.writeFile(path.join(repoA, "dist", "app.js"), "// version 2");
-      await pushwork(["sync"], repoA);
-
-      // B syncs — should pick up the new content despite the URL change
-      await pushwork(["sync"], repoB);
-      const bContentV2 = await fs.readFile(path.join(repoB, "dist", "app.js"), "utf-8");
-      expect(bContentV2).toBe("// version 2");
-    }, 90000);
-  });
-
-  describe("Move/Rename Detection", () => {
-    it("should handle file rename", async () => {
-      const repoA = path.join(tmpDir, "repo-a");
-      const repoB = path.join(tmpDir, "repo-b");
-      await fs.mkdir(repoA);
-      await fs.mkdir(repoB);
-
-      // Init A with file
-      const content = "This content will be used for similarity detection during move";
-      await fs.writeFile(path.join(repoA, "original.txt"), content);
-      await pushwork(["init", "."], repoA);
-
-      // Clone to B
-      const { stdout: rootUrl } = await pushwork(["url"], repoA);
-      await pushwork(["clone", rootUrl.trim(), repoB], tmpDir);
-
-      // Initial convergence
-      await syncUntilConverged(repoA, repoB);
-
-      // A renames the file
-      await fs.rename(
-        path.join(repoA, "original.txt"),
-        path.join(repoA, "renamed.txt")
-      );
-
-      // Sync until converged
-      const { rounds } = await syncUntilConverged(repoA, repoB);
-
-      expect(rounds).toBeLessThanOrEqual(3);
-
-      // Verify both repos have renamed.txt and not original.txt
-      expect(await pathExists(path.join(repoA, "original.txt"))).toBe(false);
-      expect(await pathExists(path.join(repoA, "renamed.txt"))).toBe(true);
-      expect(await pathExists(path.join(repoB, "original.txt"))).toBe(false);
-      expect(await pathExists(path.join(repoB, "renamed.txt"))).toBe(true);
-
-      // Verify content preserved
-      const contentB = await fs.readFile(path.join(repoB, "renamed.txt"), "utf-8");
-      expect(contentB).toBe(content);
-    }, 30000);
-  });
-});