gsd-pi 2.81.0 → 2.82.0

package/src/resources/extensions/gsd/tests/worktree-resolver.test.ts

@@ -1,1544 +0,0 @@
-// Project/App: GSD-2
-// File Purpose: WorktreeLifecycle merge/exit unit and regression tests.
-//
-// These tests previously exercised WorktreeResolver, which retired in slice 7
-// step E of ADR-016. The merge bodies now live on WorktreeLifecycle. The
-// `makeResolver()` shim below preserves the old void/throw caller shape so the
-// existing test bodies and `assert.throws(...)` assertions migrate verbatim
-// without a rewrite of every call site.
-import test from "node:test";
-import assert from "node:assert/strict";
-import { mkdtempSync, rmSync, mkdirSync, realpathSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import {
-  WorktreeLifecycle,
-  type WorktreeLifecycleDeps,
-  type NotifyCtx,
-} from "../worktree-lifecycle.js";
-import { WorktreeStateProjection } from "../worktree-state-projection.js";
-import { resolveWorktreeProjectRoot } from "../worktree-root.js";
-import { AutoSession } from "../auto/session.js";
-
-/**
- * Test-local type that extends WorktreeLifecycleDeps with the three fields
- * that lived on the retired LegacyTestDeps but never made it into
- * Lifecycle's narrower dep set. Tests can still record/override them; the
- * Lifecycle constructor ignores them via structural typing.
- */
-type LegacyTestDeps = WorktreeLifecycleDeps & {
-  shouldUseWorktreeIsolation?: () => boolean;
-  syncWorktreeStateBack?: (
-    mainBasePath: string,
-    worktreePath: string,
-    milestoneId: string,
-  ) => { synced: string[] };
-  captureIntegrationBranch?: (basePath: string, mid: string | undefined) => void;
-};
-
-/**
- * Shim factory preserving the legacy WorktreeResolver public shape for tests.
- * Wraps a fresh WorktreeLifecycle and converts the typed-result API back to
- * the old `void` / throw shape so test bodies migrate verbatim.
- */
-function makeResolver(s: AutoSession, deps: LegacyTestDeps) {
-  const lifecycle = new WorktreeLifecycle(s, deps);
-  return {
-    get workPath(): string {
-      return s.basePath;
-    },
-    get projectRoot(): string {
-      return resolveWorktreeProjectRoot(s.basePath, s.originalBasePath);
-    },
-    get lockPath(): string {
-      return resolveWorktreeProjectRoot(s.basePath, s.originalBasePath);
-    },
-    enterMilestone: (mid: string, ctx: NotifyCtx) =>
-      lifecycle.enterMilestone(mid, ctx),
-    exitMilestone: (
-      mid: string,
-      ctx: NotifyCtx,
-      opts?: { preserveBranch?: boolean },
-    ): void => {
-      const r = lifecycle.exitMilestone(
-        mid,
-        { merge: false, preserveBranch: opts?.preserveBranch },
-        ctx,
-      );
-      if (!r.ok && r.cause instanceof Error) throw r.cause;
-    },
-    mergeAndExit: (mid: string, ctx: NotifyCtx): void => {
-      const r = lifecycle.exitMilestone(mid, { merge: true }, ctx);
-      if (!r.ok && r.cause instanceof Error) throw r.cause;
-    },
-    mergeAndEnterNext: (
-      currentMilestoneId: string,
-      nextMilestoneId: string,
-      ctx: NotifyCtx,
-    ): void => {
-      lifecycle.mergeAndEnterNext(currentMilestoneId, nextMilestoneId, ctx);
-    },
-  };
-}
-import {
-  closeDatabase,
-  insertMilestone,
-  openDatabase,
-} from "../gsd-db.js";
-import { registerAutoWorker } from "../db/auto-workers.js";
-import {
-  claimMilestoneLease,
-  getMilestoneLease,
-  releaseMilestoneLease,
-} from "../db/milestone-leases.js";
-
-// ─── Helpers ─────────────────────────────────────────────────────────────────
-
-/** Track calls to mock deps for assertion. */
-interface CallLog {
-  fn: string;
-  args: unknown[];
-}
-
-function makeSession(
-  overrides?: Partial<AutoSession>,
-): AutoSession {
-  const s = new AutoSession();
-  s.basePath = overrides?.basePath ?? "/project";
-  s.originalBasePath = overrides?.originalBasePath ?? "/project";
-  Object.assign(s, overrides);
-  return s;
-}
-
-function makeDeps(
-  overrides?: Partial<LegacyTestDeps>,
-): LegacyTestDeps & { calls: CallLog[] } {
-  const calls: CallLog[] = [];
-
-  const deps: LegacyTestDeps & { calls: CallLog[] } = {
-    calls,
-    isInAutoWorktree: (basePath: string) => {
-      calls.push({ fn: "isInAutoWorktree", args: [basePath] });
-      return false;
-    },
-    shouldUseWorktreeIsolation: () => {
-      calls.push({ fn: "shouldUseWorktreeIsolation", args: [] });
-      return true;
-    },
-    getIsolationMode: () => {
-      calls.push({ fn: "getIsolationMode", args: [] });
-      return "worktree";
-    },
-    mergeMilestoneToMain: (
-      basePath: string,
-      milestoneId: string,
-      roadmapContent: string,
-    ) => {
-      calls.push({
-        fn: "mergeMilestoneToMain",
-        args: [basePath, milestoneId, roadmapContent],
-      });
-      return { pushed: false, codeFilesChanged: true };
-    },
-    syncWorktreeStateBack: (
-      mainBasePath: string,
-      worktreePath: string,
-      milestoneId: string,
-    ) => {
-      calls.push({
-        fn: "syncWorktreeStateBack",
-        args: [mainBasePath, worktreePath, milestoneId],
-      });
-      return { synced: [] };
-    },
-    teardownAutoWorktree: (
-      basePath: string,
-      milestoneId: string,
-      opts?: { preserveBranch?: boolean },
-    ) => {
-      calls.push({
-        fn: "teardownAutoWorktree",
-        args: [basePath, milestoneId, opts],
-      });
-    },
-    createAutoWorktree: (basePath: string, milestoneId: string) => {
-      calls.push({ fn: "createAutoWorktree", args: [basePath, milestoneId] });
-      return `/project/.gsd/worktrees/${milestoneId}`;
-    },
-    enterAutoWorktree: (basePath: string, milestoneId: string) => {
-      calls.push({ fn: "enterAutoWorktree", args: [basePath, milestoneId] });
-      return `/project/.gsd/worktrees/${milestoneId}`;
-    },
-    getAutoWorktreePath: (basePath: string, milestoneId: string) => {
-      calls.push({ fn: "getAutoWorktreePath", args: [basePath, milestoneId] });
-      return null;
-    },
-    autoCommitCurrentBranch: (
-      basePath: string,
-      reason: string,
-      milestoneId: string,
-    ) => {
-      calls.push({
-        fn: "autoCommitCurrentBranch",
-        args: [basePath, reason, milestoneId],
-      });
-    },
-    getCurrentBranch: (basePath: string) => {
-      calls.push({ fn: "getCurrentBranch", args: [basePath] });
-      return "main";
-    },
-    checkoutBranch: (basePath: string, branch: string) => {
-      calls.push({ fn: "checkoutBranch", args: [basePath, branch] });
-    },
-    autoWorktreeBranch: (milestoneId: string) => {
-      calls.push({ fn: "autoWorktreeBranch", args: [milestoneId] });
-      return `milestone/${milestoneId}`;
-    },
-    resolveMilestoneFile: (
-      basePath: string,
-      milestoneId: string,
-      fileType: string,
-    ) => {
-      calls.push({
-        fn: "resolveMilestoneFile",
-        args: [basePath, milestoneId, fileType],
-      });
-      return `/project/.gsd/milestones/${milestoneId}/${milestoneId}-ROADMAP.md`;
-    },
-    readFileSync: (path: string, _encoding: string) => {
-      calls.push({ fn: "readFileSync", args: [path] });
-      return "# Roadmap\n- [x] S01: Slice one\n";
-    },
-    GitServiceImpl: class MockGitServiceImpl {
-      basePath: string;
-      gitConfig: unknown;
-      constructor(basePath: string, gitConfig: unknown) {
-        calls.push({ fn: "GitServiceImpl", args: [basePath, gitConfig] });
-        this.basePath = basePath;
-        this.gitConfig = gitConfig;
-      }
-    } as unknown as LegacyTestDeps["GitServiceImpl"],
-    loadEffectiveGSDPreferences: () => {
-      calls.push({ fn: "loadEffectiveGSDPreferences", args: [] });
-      return { preferences: { git: {} } };
-    },
-    invalidateAllCaches: () => {
-      calls.push({ fn: "invalidateAllCaches", args: [] });
-    },
-    captureIntegrationBranch: (
-      basePath: string,
-      mid: string | undefined,
-    ) => {
-      calls.push({
-        fn: "captureIntegrationBranch",
-        args: [basePath, mid],
-      });
-    },
-    enterBranchModeForMilestone: (basePath: string, milestoneId: string) => {
-      calls.push({ fn: "enterBranchModeForMilestone", args: [basePath, milestoneId] });
-    },
-    worktreeProjection: new WorktreeStateProjection(),
-    ...overrides,
-  };
-
-  // Re-apply overrides that add the call tracking
-  if (overrides) {
-    for (const [key, val] of Object.entries(overrides)) {
-      if (key !== "calls") {
-        (deps as unknown as Record<string, unknown>)[key] = val;
-      }
-    }
-  }
-
-  return deps;
-}
-
-function makeNotifyCtx(): NotifyCtx & {
-  messages: Array<{ msg: string; level?: string }>;
-} {
-  const messages: Array<{ msg: string; level?: string }> = [];
-  return {
-    messages,
-    notify: (msg: string, level?: "info" | "warning" | "error" | "success") => {
-      messages.push({ msg, level });
-    },
-  };
-}
-
-function findCalls(calls: CallLog[], fn: string): CallLog[] {
-  return calls.filter((c) => c.fn === fn);
-}
-
-function makeDbBase(): string {
-  const base = mkdtempSync(join(tmpdir(), "gsd-worktree-resolver-"));
-  mkdirSync(join(base, ".gsd"), { recursive: true });
-  return base;
-}
-
-function cleanupDbBase(base: string): void {
-  try { closeDatabase(); } catch { /* noop */ }
-  try { rmSync(base, { recursive: true, force: true }); } catch { /* noop */ }
-}
-
-// ─── Getter Tests ────────────────────────────────────────────────────────────
-
-test("workPath returns s.basePath", () => {
-  const s = makeSession({ basePath: "/project/.gsd/worktrees/M001" });
-  const resolver = makeResolver(s,makeDeps());
-  assert.equal(resolver.workPath, "/project/.gsd/worktrees/M001");
-});
-
-test("projectRoot returns originalBasePath when set", () => {
-  const s = makeSession({
-    basePath: "/project/.gsd/worktrees/M001",
-    originalBasePath: "/project",
-  });
-  const resolver = makeResolver(s,makeDeps());
-  assert.equal(resolver.projectRoot, "/project");
-});
-
-test("projectRoot falls back to basePath when originalBasePath is empty", () => {
-  const s = makeSession({ basePath: "/project", originalBasePath: "" });
-  const resolver = makeResolver(s,makeDeps());
-  assert.equal(resolver.projectRoot, "/project");
-});
-
-test("lockPath returns originalBasePath when set (same as lockBase)", () => {
-  const s = makeSession({
-    basePath: "/project/.gsd/worktrees/M001",
-    originalBasePath: "/project",
-  });
-  const resolver = makeResolver(s,makeDeps());
-  assert.equal(resolver.lockPath, "/project");
-});
-
-test("lockPath falls back to basePath when originalBasePath is empty", () => {
-  const s = makeSession({ basePath: "/project", originalBasePath: "" });
-  const resolver = makeResolver(s,makeDeps());
-  assert.equal(resolver.lockPath, "/project");
-});
-
-// ─── enterMilestone Tests ────────────────────────────────────────────────────
-
-test("enterMilestone creates new worktree when none exists", () => {
-  const s = makeSession();
-  const deps = makeDeps({
-    getAutoWorktreePath: () => null,
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  new WorktreeLifecycle(s, deps).enterMilestone("M001", ctx);
-
-  assert.equal(s.basePath, "/project/.gsd/worktrees/M001");
-  assert.equal(findCalls(deps.calls, "createAutoWorktree").length, 1);
-  assert.equal(findCalls(deps.calls, "enterAutoWorktree").length, 0);
-  assert.equal(findCalls(deps.calls, "GitServiceImpl").length, 1);
-  assert.ok(
-    ctx.messages.some(
-      (m) => m.level === "info" && m.msg.includes("Entered worktree"),
-    ),
-  );
-});
-
-test("enterMilestone enters existing worktree instead of creating", () => {
-  const s = makeSession();
-  const deps = makeDeps({
-    getAutoWorktreePath: () => "/project/.gsd/worktrees/M001",
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  new WorktreeLifecycle(s, deps).enterMilestone("M001", ctx);
-
-  assert.equal(s.basePath, "/project/.gsd/worktrees/M001");
-  assert.equal(findCalls(deps.calls, "enterAutoWorktree").length, 1);
-  assert.equal(findCalls(deps.calls, "createAutoWorktree").length, 0);
-});
-
-test("enterMilestone is no-op when isolation mode is none", () => {
-  const s = makeSession();
-  const deps = makeDeps({
-    getIsolationMode: () => "none",
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  new WorktreeLifecycle(s, deps).enterMilestone("M001", ctx);
-
-  assert.equal(s.basePath, "/project"); // unchanged
-  assert.equal(findCalls(deps.calls, "createAutoWorktree").length, 0);
-  assert.equal(findCalls(deps.calls, "enterAutoWorktree").length, 0);
-  assert.equal(findCalls(deps.calls, "enterBranchModeForMilestone").length, 0);
-});
-
-test("enterMilestone passes project root to isolation mode guard", () => {
-  const s = makeSession({ basePath: "/project", originalBasePath: "/project" });
-  let checkedBasePath: string | undefined;
-  const deps = makeDeps({
-    getIsolationMode: (basePath?: string) => {
-      checkedBasePath = basePath;
-      return "none";
-    },
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  new WorktreeLifecycle(s, deps).enterMilestone("M001", ctx);
-
-  assert.equal(checkedBasePath, "/project");
-  assert.equal(findCalls(deps.calls, "createAutoWorktree").length, 0);
-});
-
-test("enterMilestone does NOT update basePath on creation failure", () => {
-  const s = makeSession();
-  const deps = makeDeps({
-    getAutoWorktreePath: () => null,
-    createAutoWorktree: () => {
-      throw new Error("disk full");
-    },
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  new WorktreeLifecycle(s, deps).enterMilestone("M001", ctx);
-
-  assert.equal(s.basePath, "/project"); // unchanged — error recovery
-  assert.ok(
-    ctx.messages.some(
-      (m) => m.level === "warning" && m.msg.includes("disk full"),
-    ),
-  );
-});
-
-test("enterMilestone uses originalBasePath as base for worktree ops", () => {
-  const s = makeSession({
-    basePath: "/project/.gsd/worktrees/M001",
-    originalBasePath: "/project",
-  });
-  let createdFrom = "";
-  const deps = makeDeps({
-    getAutoWorktreePath: () => null,
-    createAutoWorktree: (basePath: string, _mid: string) => {
-      createdFrom = basePath;
-      return "/project/.gsd/worktrees/M002";
-    },
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  new WorktreeLifecycle(s, deps).enterMilestone("M002", ctx);
-
-  assert.equal(createdFrom, "/project"); // uses originalBasePath, not current basePath
-});
-
-test("enterMilestone does not create double-nested worktree when originalBasePath is empty and basePath is a worktree path", () => {
-  // Regression test for #3729: when s.originalBasePath is "" (falsy) and
-  // s.basePath is already a worktree path, the expression
-  // `this.s.originalBasePath || this.s.basePath` evaluates to the worktree
-  // path. Passing that to createAutoWorktree produces a doubly-nested path
-  // like /project/.gsd/worktrees/M001/.gsd/worktrees/M002.
-  const wtPath = "/project/.gsd/worktrees/M001";
-  const s = makeSession({
-    basePath: wtPath,
-    originalBasePath: "/project", // will be overwritten below to simulate the bug
-  });
-  // Simulate the real bug: originalBasePath is "" (falsy) as it is when AutoSession
-  // is constructed fresh or reset() is called without auto-start re-setting it.
-  s.originalBasePath = "";
-
-  let createdFromPath = "";
-  const deps = makeDeps({
-    getAutoWorktreePath: () => null,
-    createAutoWorktree: (basePath: string, _mid: string) => {
-      createdFromPath = basePath;
-      return `/project/.gsd/worktrees/M002`;
-    },
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  new WorktreeLifecycle(s, deps).enterMilestone("M002", ctx);
-
-  // The path passed to createAutoWorktree must be the project root, NOT the
-  // worktree path. If it equals wtPath the worktree would be created at
-  // /project/.gsd/worktrees/M001/.gsd/worktrees/M002 (double-nesting).
-  assert.ok(
-    !createdFromPath.includes("/.gsd/worktrees/"),
-    `createAutoWorktree must be called with project root, got: "${createdFromPath}"`,
-  );
-});
-
-test("enterMilestone reacquires a released same-milestone lease before worktree entry", (t) => {
-  const base = makeDbBase();
-  t.after(() => cleanupDbBase(base));
-  openDatabase(join(base, ".gsd", "gsd.db"));
-  insertMilestone({ id: "M001", title: "Test milestone", status: "active" });
-
-  const workerId = registerAutoWorker({ projectRootRealpath: base });
-  const originalClaim = claimMilestoneLease(workerId, "M001");
-  assert.equal(originalClaim.ok, true);
-  if (!originalClaim.ok) throw new Error("expected test lease claim");
-  assert.equal(releaseMilestoneLease(workerId, "M001", originalClaim.token), true);
-
-  const s = makeSession({
-    basePath: base,
-    originalBasePath: base,
-    workerId,
-    currentMilestoneId: "M001",
-    milestoneLeaseToken: originalClaim.token,
-  });
-  const deps = makeDeps({
-    createAutoWorktree: (basePath: string, milestoneId: string) => join(basePath, ".gsd", "worktrees", milestoneId),
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  new WorktreeLifecycle(s, deps).enterMilestone("M001", ctx);
-
-  const row = getMilestoneLease("M001");
-  assert.ok(row);
-  assert.equal(row.worker_id, workerId);
-  assert.equal(row.status, "held");
-  assert.equal(row.fencing_token, originalClaim.token + 1);
-  assert.equal(s.milestoneLeaseToken, originalClaim.token + 1);
-  assert.equal(s.basePath, join(base, ".gsd", "worktrees", "M001"));
-  assert.equal(ctx.messages.some((m) => m.level === "error"), false);
-});
-
-// ─── enterMilestone Tests (branch mode) ──────────────────────────────────────
-
-test("enterMilestone in branch mode calls enterBranchModeForMilestone and rebuilds GitService", () => {
-  const s = makeSession();
-  const deps = makeDeps({
-    getIsolationMode: () => "branch",
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  new WorktreeLifecycle(s, deps).enterMilestone("M001", ctx);
-
-  // Branch mode: no worktree created, basePath unchanged
-  assert.equal(s.basePath, "/project");
-  assert.equal(findCalls(deps.calls, "enterBranchModeForMilestone").length, 1);
-  assert.equal(findCalls(deps.calls, "createAutoWorktree").length, 0);
-  assert.equal(findCalls(deps.calls, "enterAutoWorktree").length, 0);
-  assert.equal(findCalls(deps.calls, "GitServiceImpl").length, 1);
-  assert.ok(ctx.messages.some((m) => m.level === "info" && m.msg.includes("milestone/M001")));
-});
-
-test("enterMilestone in branch mode uses originalBasePath as base", () => {
-  const s = makeSession({ basePath: "/project", originalBasePath: "/project" });
-  let calledWith = "";
-  const deps = makeDeps({
-    getIsolationMode: () => "branch",
-    enterBranchModeForMilestone: (basePath: string, _mid: string) => {
-      calledWith = basePath;
-    },
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  new WorktreeLifecycle(s, deps).enterMilestone("M001", ctx);
-
-  assert.equal(calledWith, "/project");
-});
-
-test("enterMilestone in branch mode degrades isolation on failure", () => {
-  const s = makeSession();
-  const deps = makeDeps({
-    getIsolationMode: () => "branch",
-    enterBranchModeForMilestone: () => {
-      throw new Error("checkout failed");
-    },
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  new WorktreeLifecycle(s, deps).enterMilestone("M001", ctx);
-
-  assert.equal(s.basePath, "/project"); // unchanged
-  assert.ok(s.isolationDegraded);
-  assert.ok(ctx.messages.some((m) => m.level === "warning" && m.msg.includes("checkout failed")));
-});
-
-test("enterMilestone branch mode is skipped when isolationDegraded", () => {
-  const s = makeSession();
-  s.isolationDegraded = true;
-  const deps = makeDeps({
-    getIsolationMode: () => "branch",
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  new WorktreeLifecycle(s, deps).enterMilestone("M001", ctx);
-
-  assert.equal(findCalls(deps.calls, "enterBranchModeForMilestone").length, 0);
-  assert.equal(findCalls(deps.calls, "createAutoWorktree").length, 0);
-});
-
-// ─── exitMilestone Tests ─────────────────────────────────────────────────────
-
-test("exitMilestone commits, tears down, and resets basePath", () => {
-  const s = makeSession({
-    basePath: "/project/.gsd/worktrees/M001",
-    originalBasePath: "/project",
-  });
-  const deps = makeDeps({
-    isInAutoWorktree: () => true,
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  resolver.exitMilestone("M001", ctx);
-
-  assert.equal(s.basePath, "/project"); // reset to originalBasePath
-  assert.equal(findCalls(deps.calls, "autoCommitCurrentBranch").length, 1);
-  assert.equal(findCalls(deps.calls, "teardownAutoWorktree").length, 1);
-  assert.equal(findCalls(deps.calls, "GitServiceImpl").length, 1); // rebuilt
-  assert.equal(findCalls(deps.calls, "invalidateAllCaches").length, 1);
-});
-
-test("exitMilestone moves cwd to project root before teardown", (t) => {
-  const originalCwd = process.cwd();
-  const base = realpathSync(mkdtempSync(join(tmpdir(), "gsd-exit-cwd-")));
-  const wtPath = join(base, ".gsd", "worktrees", "M001");
-  mkdirSync(wtPath, { recursive: true });
-  t.after(() => {
-    process.chdir(originalCwd);
-    rmSync(base, { recursive: true, force: true });
-  });
-
-  process.chdir(wtPath);
-  const s = makeSession({
-    basePath: wtPath,
-    originalBasePath: base,
-  });
-  const deps = makeDeps({
-    isInAutoWorktree: () => true,
-  });
-  deps.teardownAutoWorktree = (
-    teardownBasePath: string,
-    milestoneId: string,
-    opts?: { preserveBranch?: boolean },
-  ) => {
-    deps.calls.push({ fn: "teardownAutoWorktree", args: [teardownBasePath, milestoneId, opts] });
-    assert.equal(process.cwd(), base);
-    rmSync(wtPath, { recursive: true, force: true });
-  };
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  resolver.exitMilestone("M001", ctx);
-
-  assert.equal(process.cwd(), base);
-  assert.equal(s.basePath, base);
-});
-
-test("exitMilestone is no-op when not in worktree", () => {
-  const s = makeSession();
-  const deps = makeDeps({
-    isInAutoWorktree: () => false,
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  resolver.exitMilestone("M001", ctx);
-
-  assert.equal(s.basePath, "/project"); // unchanged
-  assert.equal(findCalls(deps.calls, "autoCommitCurrentBranch").length, 0);
-  assert.equal(findCalls(deps.calls, "teardownAutoWorktree").length, 0);
-});
-
-test("exitMilestone passes preserveBranch option", () => {
-  const s = makeSession({
-    basePath: "/project/.gsd/worktrees/M001",
-    originalBasePath: "/project",
-  });
-  let preserveOpts: unknown = null;
-  const deps = makeDeps({
-    isInAutoWorktree: () => true,
-    teardownAutoWorktree: (
-      _basePath: string,
-      _mid: string,
-      opts?: { preserveBranch?: boolean },
-    ) => {
-      preserveOpts = opts;
-    },
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  resolver.exitMilestone("M001", ctx, { preserveBranch: true });
-
-  assert.deepEqual(preserveOpts, { preserveBranch: true });
-});
-
-test("exitMilestone still resets basePath even if auto-commit fails", () => {
-  const s = makeSession({
-    basePath: "/project/.gsd/worktrees/M001",
-    originalBasePath: "/project",
-  });
-  const deps = makeDeps({
-    isInAutoWorktree: () => true,
-    autoCommitCurrentBranch: () => {
-      throw new Error("commit error");
-    },
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  resolver.exitMilestone("M001", ctx);
-
-  // Should still complete: reset basePath, rebuild git service
-  assert.equal(s.basePath, "/project");
-  assert.equal(findCalls(deps.calls, "GitServiceImpl").length, 1);
-});
-
-// ─── mergeAndExit Tests (worktree mode) ──────────────────────────────────────
-
-test("mergeAndExit in worktree mode reads roadmap and merges", () => {
-  const s = makeSession({
-    basePath: "/project/.gsd/worktrees/M001",
-    originalBasePath: "/project",
-  });
-  const deps = makeDeps({
-    isInAutoWorktree: () => true,
-    getIsolationMode: () => "worktree",
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  resolver.mergeAndExit("M001", ctx);
-
-  // ADR-016 / slice 7 step D: the worktree → root state flow moved from the
-  // injected deps.syncWorktreeStateBack to WorktreeStateProjection
-  // .finalizeProjectionForMerge inside WorktreeLifecycle. The remaining
-  // assertions still cover the merge behaviour end-to-end.
-  assert.equal(findCalls(deps.calls, "resolveMilestoneFile").length, 1);
-  assert.equal(findCalls(deps.calls, "readFileSync").length, 1);
-  assert.equal(findCalls(deps.calls, "mergeMilestoneToMain").length, 1);
-  assert.equal(s.basePath, "/project"); // restored
-  assert.ok(ctx.messages.some((m) => m.msg.includes("merged to main")));
-});
-
-test("mergeAndExit in worktree mode shows pushed status", () => {
-  const s = makeSession({
-    basePath: "/project/.gsd/worktrees/M001",
-    originalBasePath: "/project",
-  });
-  const deps = makeDeps({
-    isInAutoWorktree: () => true,
-    getIsolationMode: () => "worktree",
-    mergeMilestoneToMain: () => ({ pushed: true, codeFilesChanged: true }),
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  resolver.mergeAndExit("M001", ctx);
-
-  assert.ok(ctx.messages.some((m) => m.msg.includes("Pushed to remote")));
-});
-
-test("mergeAndExit falls back to teardown with preserveBranch when roadmap is missing (#1573)", () => {
-  const s = makeSession({
-    basePath: "/project/.gsd/worktrees/M001",
-    originalBasePath: "/project",
-  });
-  const deps = makeDeps({
-    isInAutoWorktree: () => true,
-    getIsolationMode: () => "worktree",
-    resolveMilestoneFile: () => null,
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  resolver.mergeAndExit("M001", ctx);
-
-  const teardownCalls = findCalls(deps.calls, "teardownAutoWorktree");
-  assert.equal(teardownCalls.length, 1);
-  // Branch must be preserved so commits are not orphaned (#1573)
-  assert.deepEqual(teardownCalls[0].args[2], { preserveBranch: true });
-  assert.equal(findCalls(deps.calls, "mergeMilestoneToMain").length, 0);
-  assert.equal(s.basePath, "/project"); // restored
-  assert.ok(ctx.messages.some((m) => m.msg.includes("branch preserved")));
-});
-
-test("mergeAndExit resolves roadmap from worktree when missing at project root (#1573)", () => {
-  const s = makeSession({
-    basePath: "/project/.gsd/worktrees/M001",
-    originalBasePath: "/project",
-  });
-  // resolveMilestoneFile returns null for project root, returns path for worktree
-  const deps = makeDeps({
-    isInAutoWorktree: () => true,
-    getIsolationMode: () => "worktree",
-    resolveMilestoneFile: (basePath: string) => {
-      if (basePath === "/project") return null; // missing at project root
-      if (basePath === "/project/.gsd/worktrees/M001") {
-        return "/project/.gsd/worktrees/M001/.gsd/milestones/M001/M001-ROADMAP.md";
-      }
-      return null;
-    },
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  resolver.mergeAndExit("M001", ctx);
-
-  // Should have called mergeMilestoneToMain, not bare teardown
-  assert.equal(findCalls(deps.calls, "mergeMilestoneToMain").length, 1);
-  // #2945 Bug 3: secondary teardown is now called after merge for cleanup
-  assert.equal(findCalls(deps.calls, "teardownAutoWorktree").length, 1);
-  assert.equal(s.basePath, "/project"); // restored
-  assert.ok(ctx.messages.some((m) => m.msg.includes("merged to main")));
-});
-
-test("mergeAndExit in worktree mode restores to project root on merge failure", () => {
-  const s = makeSession({
-    basePath: "/project/.gsd/worktrees/M001",
-    originalBasePath: "/project",
-  });
-  const deps = makeDeps({
-    isInAutoWorktree: () => true,
-    getIsolationMode: () => "worktree",
-    mergeMilestoneToMain: () => {
-      throw new Error("conflict in main");
-    },
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  // Error propagates (#4380) — callers handle recovery. restoreToProjectRoot()
-  // still runs before re-throw so state is consistent for the caller.
-  assert.throws(() => resolver.mergeAndExit("M001", ctx), /conflict in main/);
-
-  assert.equal(s.basePath, "/project"); // error recovery — restored before re-throw
-  assert.ok(
-    ctx.messages.some(
-      (m) => m.level === "warning" && m.msg.includes("conflict in main"),
-    ),
-  );
-  assert.equal(findCalls(deps.calls, "GitServiceImpl").length, 1); // rebuilt after recovery
-});
-
-test("mergeAndExit failure message tells user worktree and branch are preserved (#1668)", () => {
-  // Regression test: before the fix, the failure message was a bare
-  // "Milestone merge failed: <reason>" with no recovery guidance. Users were
-  // left confused about whether their code had been deleted. The new message
-  // explicitly states that the worktree and branch are preserved and what to do.
-  const s = makeSession({
-    basePath: "/project/.gsd/worktrees/M001",
-    originalBasePath: "/project",
-  });
-  const deps = makeDeps({
-    isInAutoWorktree: () => true,
-    getIsolationMode: () => "worktree",
-    mergeMilestoneToMain: () => {
-      throw new Error("pathspec 'main' did not match any file(s) known to git");
-    },
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  // Error propagates (#4380) — notification is still emitted before re-throw
-  assert.throws(() => resolver.mergeAndExit("M001", ctx), /pathspec 'main' did not match/);
-
-  const warning = ctx.messages.find((m) => m.level === "warning");
-  assert.ok(warning, "a warning message is emitted");
-  // Must contain the original error
-  assert.ok(warning!.msg.includes("pathspec 'main' did not match"), "warning includes the original error");
-  // Must tell the user their work is safe
-  assert.ok(
-    warning!.msg.includes("preserved"),
-    "warning tells user the worktree and branch are preserved",
-  );
-  // Must suggest a recovery action
-  assert.ok(
-    warning!.msg.includes("retry") || warning!.msg.includes("manually"),
-    "warning suggests a recovery action",
-  );
-});
-
-test("mergeAndExit failure message references /gsd dispatch complete-milestone, not /complete-milestone (#1891)", () => {
-  // Regression test: the failure notification previously told users to
-  // "retry /complete-milestone" — a command that does not exist. The correct
-  // recovery command is "/gsd dispatch complete-milestone".
-  const s = makeSession({
-    basePath: "/project/.gsd/worktrees/M001",
-    originalBasePath: "/project",
-  });
-  const deps = makeDeps({
-    isInAutoWorktree: () => true,
-    getIsolationMode: () => "worktree",
-    mergeMilestoneToMain: () => {
-      throw new Error("dirty working tree");
-    },
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  // Error propagates (#4380) — notification is still emitted before re-throw
-  assert.throws(() => resolver.mergeAndExit("M001", ctx), /dirty working tree/);
-
-  const warning = ctx.messages.find((m) => m.level === "warning");
-  assert.ok(warning, "a warning message is emitted");
-  // Must reference the correct dispatch command
-  assert.ok(
-    warning!.msg.includes("/gsd dispatch complete-milestone"),
-    "warning references /gsd dispatch complete-milestone, not bare /complete-milestone",
-  );
-  // Must NOT contain the bare (incorrect) command without the dispatch prefix
-  assert.ok(
-    !warning!.msg.match(/retry\s+\/complete-milestone(?!\S)/),
-    "warning must not reference the non-existent /complete-milestone command",
-  );
-});
-
-// ─── mergeAndExit Tests (branch mode) ────────────────────────────────────────
-
-test("mergeAndExit in branch mode merges when on milestone branch", () => {
-  const s = makeSession({ basePath: "/project", originalBasePath: "/project" });
-  const deps = makeDeps({
-    isInAutoWorktree: () => false,
-    getIsolationMode: () => "branch",
-    getCurrentBranch: () => "milestone/M001",
-    autoWorktreeBranch: () => "milestone/M001",
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  resolver.mergeAndExit("M001", ctx);
-
-  assert.equal(findCalls(deps.calls, "mergeMilestoneToMain").length, 1);
-  assert.ok(ctx.messages.some((m) => m.msg.includes("branch mode")));
-});
-
-test("mergeAndExit in branch mode checks out the milestone branch and merges (#5538-followup)", () => {
-  // Regression: previously this case silently returned without merging,
-  // stranding the milestone's commits on the branch (the test12345 repro).
-  // The fix forces a checkout first; merge proceeds when checkout succeeds.
-  const s = makeSession({ basePath: "/project", originalBasePath: "/project" });
-  let currentBranch = "main";
-  const checkoutInvocations: Array<{ basePath: string; branch: string }> = [];
-  const deps = makeDeps({
-    isInAutoWorktree: () => false,
-    getIsolationMode: () => "branch",
-    getCurrentBranch: () => currentBranch,
-    autoWorktreeBranch: () => "milestone/M001",
-    checkoutBranch: (basePath: string, branch: string) => {
-      checkoutInvocations.push({ basePath, branch });
-      currentBranch = branch;
-    },
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  resolver.mergeAndExit("M001", ctx);
-
-  assert.equal(checkoutInvocations.length, 1, "must attempt checkout when on wrong branch");
-  assert.deepEqual(checkoutInvocations[0], { basePath: "/project", branch: "milestone/M001" });
-  assert.equal(findCalls(deps.calls, "mergeMilestoneToMain").length, 1);
-});
-
-test("mergeAndExit in branch mode throws when checkout fails", () => {
-  // Regression for the silent-skip bug: if the working tree is on the wrong
-  // branch and checkout fails, we must throw so the caller pauses auto-mode
-  // — never silently advance with the milestone unmerged.
-  const s = makeSession({ basePath: "/project", originalBasePath: "/project" });
-  const deps = makeDeps({
-    isInAutoWorktree: () => false,
-    getIsolationMode: () => "branch",
-    getCurrentBranch: () => "main",
-    autoWorktreeBranch: () => "milestone/M001",
-    checkoutBranch: () => {
-      throw new Error("dirty working tree blocks checkout");
-    },
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  assert.throws(
-    () => resolver.mergeAndExit("M001", ctx),
-    /dirty working tree blocks checkout/,
-  );
-  assert.equal(
-    findCalls(deps.calls, "mergeMilestoneToMain").length,
-    0,
-    "merge must not run when checkout failed",
-  );
-  const errorNotify = ctx.messages.find((m) => m.level === "error");
-  assert.ok(errorNotify, "an error notification must be emitted");
-  assert.match(errorNotify!.msg, /milestone\/M001 failed/);
-  assert.match(errorNotify!.msg, /Resolve manually/);
-  assert.equal(
-    ctx.messages.some((m) => m.level === "warning" && m.msg.includes("Milestone merge failed")),
-    false,
-    "checkout failures with explicit recovery guidance must not emit a duplicate warning",
-  );
-});
-
-test("mergeAndExit in branch mode throws when checkout reports success but HEAD is still wrong", () => {
-  // Defense in depth: even if checkoutBranch returns without throwing, we
-  // re-verify and throw if HEAD didn't actually move. Prevents merging on
-  // top of the wrong branch on platforms where the checkout is a no-op.
-  const s = makeSession({ basePath: "/project", originalBasePath: "/project" });
-  const deps = makeDeps({
-    isInAutoWorktree: () => false,
-    getIsolationMode: () => "branch",
-    getCurrentBranch: () => "main", // never changes — simulates no-op checkout
-    autoWorktreeBranch: () => "milestone/M001",
-    checkoutBranch: () => {
-      // Pretend success — but getCurrentBranch will still return "main".
-    },
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  assert.throws(
-    () => resolver.mergeAndExit("M001", ctx),
-    /reported success but current branch is main/,
-  );
-  assert.equal(findCalls(deps.calls, "mergeMilestoneToMain").length, 0);
-});
-
-test("mergeAndExit in branch mode handles merge failure gracefully", () => {
-  const s = makeSession({ basePath: "/project", originalBasePath: "/project" });
-  const deps = makeDeps({
-    isInAutoWorktree: () => false,
-    getIsolationMode: () => "branch",
-    getCurrentBranch: () => "milestone/M001",
-    autoWorktreeBranch: () => "milestone/M001",
-    mergeMilestoneToMain: () => {
-      throw new Error("branch merge conflict");
-    },
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  // Error propagates (#4380) — notification is still emitted before re-throw
-  assert.throws(() => resolver.mergeAndExit("M001", ctx), /branch merge conflict/);
-
-  assert.ok(
-    ctx.messages.some(
-      (m) => m.level === "warning" && m.msg.includes("branch merge conflict"),
-    ),
-  );
-});
-
-test("mergeAndExit in branch mode skips when no roadmap", () => {
-  const s = makeSession({ basePath: "/project", originalBasePath: "/project" });
-  const deps = makeDeps({
-    isInAutoWorktree: () => false,
-    getIsolationMode: () => "branch",
-    getCurrentBranch: () => "milestone/M001",
-    autoWorktreeBranch: () => "milestone/M001",
-    resolveMilestoneFile: () => null,
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  resolver.mergeAndExit("M001", ctx);
-
-  assert.equal(findCalls(deps.calls, "mergeMilestoneToMain").length, 0);
-});
-
-test("mergeAndExit in branch mode rebuilds GitService after merge", () => {
-  const s = makeSession({ basePath: "/project", originalBasePath: "/project" });
-  const deps = makeDeps({
-    isInAutoWorktree: () => false,
-    getIsolationMode: () => "branch",
-    getCurrentBranch: () => "milestone/M001",
-    autoWorktreeBranch: () => "milestone/M001",
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  resolver.mergeAndExit("M001", ctx);
-
-  assert.equal(findCalls(deps.calls, "GitServiceImpl").length, 1);
-});
-
-// ─── mergeAndExit Tests (none mode) ──────────────────────────────────────────
-
-test("mergeAndExit in none mode is a no-op", () => {
-  const s = makeSession();
-  const deps = makeDeps({
-    getIsolationMode: () => "none",
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  resolver.mergeAndExit("M001", ctx);
-
-  assert.equal(findCalls(deps.calls, "mergeMilestoneToMain").length, 0);
-  assert.equal(findCalls(deps.calls, "teardownAutoWorktree").length, 0);
-  assert.equal(ctx.messages.length, 0);
-});
-
-// ─── #1906 — metadata-only merge warning ────────────────────────────────────
-
-test("mergeAndExit warns when merge contains no code changes (#1906)", () => {
-  const s = makeSession({
-    basePath: "/project/.gsd/worktrees/M001",
-    originalBasePath: "/project",
-  });
-  const deps = makeDeps({
-    isInAutoWorktree: () => true,
-    getIsolationMode: () => "worktree",
-    mergeMilestoneToMain: () => ({ pushed: false, codeFilesChanged: false }),
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  resolver.mergeAndExit("M001", ctx);
-
-  assert.ok(
-    ctx.messages.some((m) => m.msg.includes("NO code changes") && m.level === "warning"),
-    "must emit warning when only .gsd/ metadata was merged",
-  );
-  assert.ok(
-    !ctx.messages.some((m) => m.msg.includes("merged to main") && m.level === "info"),
-    "must NOT emit success-style info notification for metadata-only merge",
-  );
-});
-
-test("mergeAndExit emits info when merge contains code changes (#1906)", () => {
-  const s = makeSession({
-    basePath: "/project/.gsd/worktrees/M001",
-    originalBasePath: "/project",
-  });
-  const deps = makeDeps({
-    isInAutoWorktree: () => true,
-    getIsolationMode: () => "worktree",
-    mergeMilestoneToMain: () => ({ pushed: false, codeFilesChanged: true }),
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  resolver.mergeAndExit("M001", ctx);
-
-  assert.ok(
-    ctx.messages.some((m) => m.msg.includes("merged to main") && m.level === "info"),
-    "must emit info notification when code files were merged",
-  );
-  assert.ok(
-    !ctx.messages.some((m) => m.msg.includes("NO code changes")),
-    "must NOT emit metadata-only warning when code files were merged",
-  );
-});
-
-test("mergeAndExit branch mode warns when merge contains no code changes (#1906)", () => {
-  const s = makeSession({
-    basePath: "/project",
-    originalBasePath: "/project",
-  });
-  const deps = makeDeps({
-    isInAutoWorktree: () => false,
-    getIsolationMode: () => "branch",
-    getCurrentBranch: () => "milestone/M001",
-    autoWorktreeBranch: () => "milestone/M001",
-    mergeMilestoneToMain: () => ({ pushed: false, codeFilesChanged: false }),
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  resolver.mergeAndExit("M001", ctx);
-
-  assert.ok(
-    ctx.messages.some((m) => m.msg.includes("NO code changes") && m.level === "warning"),
-    "branch mode must emit warning when only .gsd/ metadata was merged",
-  );
-});
-
-// ─── mergeAndEnterNext Tests ─────────────────────────────────────────────────
-
-test("mergeAndEnterNext calls mergeAndExit then enterMilestone", () => {
-  const s = makeSession({
-    basePath: "/project/.gsd/worktrees/M001",
-    originalBasePath: "/project",
-  });
-  const callOrder: string[] = [];
-  const deps = makeDeps({
-    isInAutoWorktree: () => true,
-    getIsolationMode: () => "worktree",
-    shouldUseWorktreeIsolation: () => true,
-    mergeMilestoneToMain: (
-      basePath: string,
-      milestoneId: string,
-      _roadmap: string,
-    ) => {
-      callOrder.push(`merge:${milestoneId}`);
-      return { pushed: false, codeFilesChanged: true };
-    },
-    getAutoWorktreePath: () => null,
-    createAutoWorktree: (basePath: string, milestoneId: string) => {
-      callOrder.push(`create:${milestoneId}`);
-      return `/project/.gsd/worktrees/${milestoneId}`;
-    },
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  resolver.mergeAndEnterNext("M001", "M002", ctx);
-
-  assert.deepEqual(callOrder, ["merge:M001", "create:M002"]);
-  assert.equal(s.basePath, "/project/.gsd/worktrees/M002");
-});
-
-test("mergeAndEnterNext enters next milestone even if merge fails", () => {
-  const s = makeSession({
-    basePath: "/project/.gsd/worktrees/M001",
-    originalBasePath: "/project",
-  });
-  const deps = makeDeps({
-    isInAutoWorktree: (basePath: string) => basePath.includes("worktrees"),
-    getIsolationMode: () => "worktree",
-    shouldUseWorktreeIsolation: () => true,
-    mergeMilestoneToMain: () => {
-      throw new Error("merge failed");
-    },
-    getAutoWorktreePath: () => null,
-    createAutoWorktree: (_basePath: string, milestoneId: string) => {
-      return `/project/.gsd/worktrees/${milestoneId}`;
-    },
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  resolver.mergeAndEnterNext("M001", "M002", ctx);
-
-  // Merge failed but enter should still happen
-  assert.equal(s.basePath, "/project/.gsd/worktrees/M002");
-  assert.ok(
-    ctx.messages.some(
-      (m) => m.level === "warning" && m.msg.includes("merge failed"),
-    ),
-  );
-  assert.ok(
-    ctx.messages.some(
-      (m) => m.level === "info" && m.msg.includes("Entered worktree"),
-    ),
-  );
-});
-
-test("mergeAndEnterNext halts when mergeAndExit preserves branch without merging", () => {
-  const s = makeSession({
-    basePath: "/project/.gsd/worktrees/M001",
-    originalBasePath: "/project",
-  });
-  const deps = makeDeps({
-    isInAutoWorktree: () => true,
-    getIsolationMode: () => "worktree",
-    shouldUseWorktreeIsolation: () => true,
-    resolveMilestoneFile: () => null,
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s, deps);
-
-  assert.throws(
-    () => resolver.mergeAndEnterNext("M001", "M002", ctx),
-    /Cannot enter milestone M002 because M001 was not merged/,
-  );
-  assert.equal(findCalls(deps.calls, "mergeMilestoneToMain").length, 0);
-  assert.equal(findCalls(deps.calls, "createAutoWorktree").length, 0);
-  assert.equal(findCalls(deps.calls, "enterAutoWorktree").length, 0);
-  assert.equal(s.basePath, "/project");
-  assert.ok(ctx.messages.some((m) => m.msg.includes("branch preserved")));
-});
-
-test("mergeAndEnterNext halts after branch-mode user-notified checkout failure", () => {
-  const s = makeSession({ basePath: "/project", originalBasePath: "/project" });
-  const deps = makeDeps({
-    isInAutoWorktree: () => false,
-    getIsolationMode: () => "branch",
-    getCurrentBranch: () => "main",
-    autoWorktreeBranch: () => "milestone/M001",
-    checkoutBranch: () => {
-      throw new Error("dirty working tree blocks checkout");
-    },
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  assert.throws(
-    () => resolver.mergeAndEnterNext("M001", "M002", ctx),
-    /dirty working tree blocks checkout/,
-  );
-  assert.equal(
-    findCalls(deps.calls, "enterBranchModeForMilestone").length,
-    0,
-    "must not enter the next milestone after a user-notified branch-mode failure",
-  );
-  assert.equal(findCalls(deps.calls, "mergeMilestoneToMain").length, 0);
-  assert.ok(ctx.messages.some((m) => m.level === "error" && m.msg.includes("Resolve manually")));
-});
-
-// ─── GitService Rebuild Atomicity ────────────────────────────────────────────
-
-test("GitService is rebuilt with the NEW basePath after enterMilestone", () => {
-  const s = makeSession();
-  let gitServiceBasePath = "";
-  const deps = makeDeps({
-    getAutoWorktreePath: () => null,
-    GitServiceImpl: class {
-      constructor(basePath: string, _config: unknown) {
-        gitServiceBasePath = basePath;
-      }
-    } as unknown as LegacyTestDeps["GitServiceImpl"],
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  new WorktreeLifecycle(s, deps).enterMilestone("M001", ctx);
-
-  assert.equal(gitServiceBasePath, "/project/.gsd/worktrees/M001"); // new path, not old
-});
-
-test("GitService is rebuilt with originalBasePath after exitMilestone", () => {
-  const s = makeSession({
-    basePath: "/project/.gsd/worktrees/M001",
-    originalBasePath: "/project",
-  });
-  let gitServiceBasePath = "";
-  const deps = makeDeps({
-    isInAutoWorktree: () => true,
-    GitServiceImpl: class {
-      constructor(basePath: string, _config: unknown) {
-        gitServiceBasePath = basePath;
-      }
-    } as unknown as LegacyTestDeps["GitServiceImpl"],
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  resolver.exitMilestone("M001", ctx);
-
-  assert.equal(gitServiceBasePath, "/project"); // project root, not worktree
-});
-
-// ─── Isolation Degradation Tests (#2483) ──────────────────────────────────
-
-test("enterMilestone sets isolationDegraded when worktree creation throws (#2483)", () => {
-  const s = makeSession();
-  const deps = makeDeps({
-    getAutoWorktreePath: () => null,
-    createAutoWorktree: () => {
-      throw new Error("empty repo");
-    },
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  new WorktreeLifecycle(s, deps).enterMilestone("M001", ctx);
-
-  assert.equal(s.isolationDegraded, true);
-  assert.equal(s.basePath, "/project"); // unchanged — error recovery
-});
-
-test("enterMilestone is no-op when isolationDegraded is true (#2483)", () => {
-  const s = makeSession();
-  s.isolationDegraded = true;
-  const deps = makeDeps();
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  new WorktreeLifecycle(s, deps).enterMilestone("M001", ctx);
-
-  assert.equal(s.basePath, "/project"); // unchanged
-  assert.equal(findCalls(deps.calls, "createAutoWorktree").length, 0);
-  assert.equal(findCalls(deps.calls, "enterAutoWorktree").length, 0);
-  assert.equal(findCalls(deps.calls, "shouldUseWorktreeIsolation").length, 0);
-});
-
-test("mergeAndExit is no-op when isolationDegraded is true (#2483)", () => {
-  const s = makeSession({
-    basePath: "/project",
-    originalBasePath: "/project",
-  });
-  s.isolationDegraded = true;
-  const deps = makeDeps({
-    getIsolationMode: () => "worktree",
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  resolver.mergeAndExit("M001", ctx);
-
-  assert.equal(findCalls(deps.calls, "mergeMilestoneToMain").length, 0);
-  assert.equal(findCalls(deps.calls, "teardownAutoWorktree").length, 0);
-  assert.equal(findCalls(deps.calls, "getIsolationMode").length, 0);
-  assert.ok(
-    ctx.messages.some(
-      (m) => m.level === "info" && m.msg.includes("isolation was degraded"),
-    ),
-  );
-});
-
-test("isolationDegraded is reset by session.reset() (#2483)", () => {
-  const s = new AutoSession();
-  s.isolationDegraded = true;
-
-  s.reset();
-
-  assert.equal(s.isolationDegraded, false);
-});
-
-// ─── #2625 — Default isolation mode change must not orphan worktree commits ──
-
-test("mergeAndExit still merges when mode is 'none' but session is in a worktree (#2625)", () => {
-  // Scenario: user upgraded from a version where default was "worktree" to one
-  // where default is "none". They have an active worktree with committed work.
-  // mergeAndExit must detect the active worktree and merge regardless of config.
-  const s = makeSession({
-    basePath: "/project/.gsd/worktrees/M001",
-    originalBasePath: "/project",
-  });
-  const deps = makeDeps({
-    isInAutoWorktree: () => true,
-    getIsolationMode: () => "none", // config says "none" — but we ARE in a worktree
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  resolver.mergeAndExit("M001", ctx);
-
-  // Must still merge — not skip silently
-  assert.equal(findCalls(deps.calls, "mergeMilestoneToMain").length, 1,
-    "must call mergeMilestoneToMain even when isolation mode is 'none' but we are in a worktree");
-  assert.equal(s.basePath, "/project", "basePath must be restored to project root");
-  assert.ok(ctx.messages.some((m) => m.msg.includes("merged to main")),
-    "must notify about the merge");
-});
-
-test("mergeAndExit in none mode remains a no-op when NOT in a worktree (#2625)", () => {
-  // When mode is "none" and we are genuinely not in a worktree, it should still be a no-op.
-  const s = makeSession({
-    basePath: "/project",
-    originalBasePath: "/project",
-  });
-  const deps = makeDeps({
-    isInAutoWorktree: () => false,
-    getIsolationMode: () => "none",
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  resolver.mergeAndExit("M001", ctx);
-
-  assert.equal(findCalls(deps.calls, "mergeMilestoneToMain").length, 0,
-    "must NOT merge when not in a worktree and mode is none");
-});
-
-// ─── #4380 — Non-MergeConflictError must not be swallowed ────────────────────
-
-test("mergeAndExit propagates non-MergeConflictError to caller (#4380)", () => {
-  // Regression test: previously the catch block in _mergeWorktreeMode only
-  // re-threw MergeConflictError. Permission errors, filesystem errors, and other
-  // non-conflict failures were swallowed silently, making broken states impossible
-  // to diagnose and preventing callers (phases.ts) from applying their own
-  // error-recovery logic.
-  const permissionError = new Error("EACCES: permission denied, open '/project/.git/SQUASH_MSG'");
-  const s = makeSession({
-    basePath: "/project/.gsd/worktrees/M001",
-    originalBasePath: "/project",
-  });
-  const deps = makeDeps({
-    isInAutoWorktree: () => true,
-    getIsolationMode: () => "worktree",
-    mergeMilestoneToMain: () => {
-      throw permissionError;
-    },
-  });
-  const ctx = makeNotifyCtx();
-  const resolver = makeResolver(s,deps);
-
-  // The error must propagate — callers need it to apply their own recovery logic
-  assert.throws(
-    () => resolver.mergeAndExit("M001", ctx),
-    (err: unknown) => err === permissionError,
-    "non-MergeConflictError must propagate to the caller, not be swallowed",
-  );
-});
-
-// ─── Regression: mergeAndExit anchors cwd at project root before merge work ─
-// (de73fb43d headless `gsd auto` exits-on-task regression)
-//
-// Background: the auto loop runs tasks inside the milestone worktree
-// (process.cwd() === worktreePath). When the milestone completes, the
-// worktree dir is torn down. If cwd was still inside it at that moment,
-// every subsequent process.cwd() throws ENOENT — and after de73fb43d
-// auto/run-unit.ts:50 turns that ENOENT into a session-failed cancel,
-// which in headless mode bubbles up to a "Auto-mode stopped" notify
-// and process.exit(0). mergeAndExit must therefore guarantee cwd is
-// anchored at the project root regardless of which merge path runs.
-
-test("mergeAndExit chdirs to project root before merge work (regression: headless gsd auto exit)", () => {
-  // Set up real dirs so process.chdir actually succeeds. realpathSync
-  // canonicalizes the macOS /var → /private/var symlink so equality holds.
-  const projectRoot = realpathSync(mkdtempSync(join(tmpdir(), "gsd-resolver-cwd-")));
-  const worktreePath = join(projectRoot, ".gsd/worktrees/M001");
-  mkdirSync(worktreePath, { recursive: true });
-  const previousCwd = process.cwd();
-
-  try {
-    process.chdir(worktreePath);
-    assert.equal(process.cwd(), worktreePath, "precondition: cwd is in worktree");
-
-    const s = makeSession({
-      basePath: worktreePath,
-      originalBasePath: projectRoot,
-    });
-    const deps = makeDeps({
-      isInAutoWorktree: () => true,
-      getIsolationMode: () => "worktree",
-    });
-    const ctx = makeNotifyCtx();
-    const resolver = makeResolver(s,deps);
-
-    resolver.mergeAndExit("M001", ctx);
-
-    assert.equal(
-      process.cwd(),
-      projectRoot,
-      "mergeAndExit must leave cwd at the project root, not the (about-to-be-removed) worktree",
-    );
-  } finally {
-    try { process.chdir(previousCwd); } catch { /* best-effort */ }
-    rmSync(projectRoot, { recursive: true, force: true });
-  }
-});
-
-test("mergeAndExit anchors cwd even on isolation-degraded skip path", () => {
-  // The skip paths (isolation-degraded, mode-none, missing-original-base)
-  // bypass the per-mode merge helpers entirely. They must still leave cwd
-  // at the project root so a subsequent worktree teardown elsewhere does
-  // not strand cwd in a deleted dir.
-  const projectRoot = realpathSync(mkdtempSync(join(tmpdir(), "gsd-resolver-cwd-degraded-")));
-  const worktreePath = join(projectRoot, ".gsd/worktrees/M001");
-  mkdirSync(worktreePath, { recursive: true });
-  const previousCwd = process.cwd();
-
-  try {
-    process.chdir(worktreePath);
-    const s = makeSession({
-      basePath: worktreePath,
-      originalBasePath: projectRoot,
-    });
-    s.isolationDegraded = true;
-    const deps = makeDeps({ getIsolationMode: () => "worktree" });
-    const ctx = makeNotifyCtx();
-    const resolver = makeResolver(s,deps);
-
-    resolver.mergeAndExit("M001", ctx);
-
-    assert.equal(
-      process.cwd(),
-      projectRoot,
-      "isolation-degraded skip must still anchor cwd at project root",
-    );
-  } finally {
-    try { process.chdir(previousCwd); } catch { /* best-effort */ }
-    rmSync(projectRoot, { recursive: true, force: true });
-  }
-});