lsd-pi 1.0.0 → 1.1.1

package/src/resources/extensions/gsd/tests/integration/doctor-environment.test.ts

@@ -1,403 +0,0 @@
-import { describe, test } from 'node:test';
-import assert from 'node:assert/strict';
-/**
- * doctor-environment.test.ts — Tests for environment health checks (#1221).
- *
- * Tests:
- *   - Node version detection
- *   - Dependencies installed check
- *   - Env file detection
- *   - Port conflict detection
- *   - Disk space check
- *   - Docker detection
- *   - Project tool detection
- *   - Doctor issue conversion
- *   - Report formatting
- */
-
-import { mkdtempSync, mkdirSync, writeFileSync, rmSync, utimesSync } from "node:fs";
-import { join, dirname } from "node:path";
-import { tmpdir } from "node:os";
-
-import {
-  runEnvironmentChecks,
-  runFullEnvironmentChecks,
-  environmentResultsToDoctorIssues,
-  formatEnvironmentReport,
-  checkEnvironmentHealth,
-  type EnvironmentCheckResult,
-} from "../../doctor-environment.ts";
-function createProjectDir(files: Record<string, string> = {}): string {
-  const dir = mkdtempSync(join(tmpdir(), "gsd-env-test-"));
-  for (const [name, content] of Object.entries(files)) {
-    const filePath = join(dir, name);
-    mkdirSync(dirname(filePath), { recursive: true });
-    writeFileSync(filePath, content);
-  }
-  return dir;
-}
-
-describe('doctor-environment', async () => {
-  const cleanups: string[] = [];
-
-  try {
-    // ── Node Version Check ─────────────────────────────────────────────
-    test('env: no package.json returns empty', () => {
-      const dir = createProjectDir();
-      cleanups.push(dir);
-      const results = runEnvironmentChecks(dir);
-      // No package.json → no node checks
-      const nodeCheck = results.find(r => r.name === "node_version");
-      assert.deepStrictEqual(nodeCheck, undefined, "no node version check without package.json");
-    });
-
-    test('env: package.json without engines returns no node check', () => {
-      const dir = createProjectDir({
-        "package.json": JSON.stringify({ name: "test", version: "1.0.0" }),
-      });
-      cleanups.push(dir);
-      const results = runEnvironmentChecks(dir);
-      const nodeCheck = results.find(r => r.name === "node_version");
-      assert.deepStrictEqual(nodeCheck, undefined, "no node version check without engines field");
-    });
-
-    test('env: package.json with engines returns node check', () => {
-      const dir = createProjectDir({
-        "package.json": JSON.stringify({
-          name: "test",
-          version: "1.0.0",
-          engines: { node: ">=18.0.0" },
-        }),
-      });
-      cleanups.push(dir);
-      const results = runEnvironmentChecks(dir);
-      const nodeCheck = results.find(r => r.name === "node_version");
-      assert.ok(nodeCheck !== undefined, "node version check runs with engines field");
-      // Current node should be >= 18 in CI
-      assert.deepStrictEqual(nodeCheck!.status, "ok", "node version meets requirement");
-    });
-
-    // ── Dependencies Check ─────────────────────────────────────────────
-    test('env: missing node_modules detected', () => {
-      const dir = createProjectDir({
-        "package.json": JSON.stringify({ name: "test" }),
-      });
-      cleanups.push(dir);
-      const results = runEnvironmentChecks(dir);
-      const depsCheck = results.find(r => r.name === "dependencies");
-      assert.ok(depsCheck !== undefined, "dependencies check runs");
-      assert.deepStrictEqual(depsCheck!.status, "error", "missing node_modules is an error");
-      assert.ok(depsCheck!.message.includes("node_modules missing"), "reports missing node_modules");
-    });
-
-    test('env: existing node_modules detected', () => {
-      const dir = createProjectDir({
-        "package.json": JSON.stringify({ name: "test" }),
-      });
-      mkdirSync(join(dir, "node_modules"), { recursive: true });
-      cleanups.push(dir);
-      const results = runEnvironmentChecks(dir);
-      const depsCheck = results.find(r => r.name === "dependencies");
-      assert.ok(depsCheck !== undefined, "dependencies check runs");
-      assert.deepStrictEqual(depsCheck!.status, "ok", "existing node_modules is ok");
-    });
-
-    // ── Stale Dependencies: marker file check (#1974) ──────────────────
-    console.log("\n=== env: npm marker file newer than lockfile → ok (#1974) ===");
-    {
-      // Simulate the exact bug scenario:
-      // 1. node_modules dir mtime is old (no entries added/removed recently)
-      // 2. package-lock.json mtime is recent (npm rewrote it)
-      // 3. node_modules/.package-lock.json mtime is between dir and lockfile
-      //    (npm wrote it during the same install that rewrote the lockfile)
-      //
-      // The bug: code compares lockfile mtime vs dir mtime → false positive warning
-      // The fix: compare lockfile mtime vs marker file mtime → correctly ok
-      const dir = createProjectDir({
-        "package.json": JSON.stringify({ name: "test" }),
-      });
-      mkdirSync(join(dir, "node_modules"), { recursive: true });
-
-      // Simulate the exact bug: npm install with "up to date" rewrites the
-      // lockfile and the marker, but no packages are added/removed so the
-      // directory mtime should be old. We write the marker first (which
-      // bumps dir mtime), then force the dir mtime back to the past.
-      //
-      // Timeline: dir(T-120s) < lockfile(T-5s) ≈ marker(T-5s)
-      // Bug: code compares lockfile vs dir → false positive stale warning
-      // Fix: code compares lockfile vs marker → correctly reports ok
-      const dirTime = new Date(Date.now() - 120_000);
-      const installTime = new Date(Date.now() - 5_000);
-
-      // Write marker file (this bumps dir mtime as a side effect)
-      writeFileSync(join(dir, "node_modules", ".package-lock.json"), "{}");
-      utimesSync(join(dir, "node_modules", ".package-lock.json"), installTime, installTime);
-
-      // Force dir mtime back to the past — simulates no top-level entries changed
-      utimesSync(join(dir, "node_modules"), dirTime, dirTime);
-
-      // Lockfile written at install time (same as marker, or slightly after)
-      writeFileSync(join(dir, "package-lock.json"), "{}");
-      utimesSync(join(dir, "package-lock.json"), installTime, installTime);
-
-      cleanups.push(dir);
-      const results = runEnvironmentChecks(dir);
-      const depsCheck = results.find(r => r.name === "dependencies");
-      assert.ok(depsCheck !== undefined, "dependencies check runs");
-      assert.equal(depsCheck!.status, "ok", "npm marker newer than lockfile → not stale");
-    }
-
-    console.log("\n=== env: yarn marker file newer than lockfile → ok (#1974) ===");
-    {
-      const dir = createProjectDir({
-        "package.json": JSON.stringify({ name: "test" }),
-      });
-      mkdirSync(join(dir, "node_modules"), { recursive: true });
-
-      const dirTime = new Date(Date.now() - 120_000);
-      const installTime = new Date(Date.now() - 5_000);
-
-      writeFileSync(join(dir, "node_modules", ".yarn-integrity"), "{}");
-      utimesSync(join(dir, "node_modules", ".yarn-integrity"), installTime, installTime);
-      utimesSync(join(dir, "node_modules"), dirTime, dirTime);
-
-      writeFileSync(join(dir, "yarn.lock"), "");
-      utimesSync(join(dir, "yarn.lock"), installTime, installTime);
-
-      cleanups.push(dir);
-      const results = runEnvironmentChecks(dir);
-      const depsCheck = results.find(r => r.name === "dependencies");
-      assert.ok(depsCheck !== undefined, "dependencies check runs");
-      assert.equal(depsCheck!.status, "ok", "yarn marker newer than lockfile → not stale");
-    }
-
-    console.log("\n=== env: pnpm marker file newer than lockfile → ok (#1974) ===");
-    {
-      const dir = createProjectDir({
-        "package.json": JSON.stringify({ name: "test" }),
-      });
-      mkdirSync(join(dir, "node_modules"), { recursive: true });
-
-      const dirTime = new Date(Date.now() - 120_000);
-      const installTime = new Date(Date.now() - 5_000);
-
-      writeFileSync(join(dir, "node_modules", ".modules.yaml"), "{}");
-      utimesSync(join(dir, "node_modules", ".modules.yaml"), installTime, installTime);
-      utimesSync(join(dir, "node_modules"), dirTime, dirTime);
-
-      writeFileSync(join(dir, "pnpm-lock.yaml"), "");
-      utimesSync(join(dir, "pnpm-lock.yaml"), installTime, installTime);
-
-      cleanups.push(dir);
-      const results = runEnvironmentChecks(dir);
-      const depsCheck = results.find(r => r.name === "dependencies");
-      assert.ok(depsCheck !== undefined, "dependencies check runs");
-      assert.equal(depsCheck!.status, "ok", "pnpm marker newer than lockfile → not stale");
-    }
-
-    console.log("\n=== env: no marker file falls back to dir mtime → stale warning (#1974) ===");
-    {
-      // No marker file exists, lockfile newer than dir → should still warn
-      const dir = createProjectDir({
-        "package.json": JSON.stringify({ name: "test" }),
-      });
-      mkdirSync(join(dir, "node_modules"), { recursive: true });
-
-      const past = new Date(Date.now() - 60_000);
-      utimesSync(join(dir, "node_modules"), past, past);
-
-      writeFileSync(join(dir, "package-lock.json"), "{}");
-      // No marker file written — fallback to dir mtime comparison
-
-      cleanups.push(dir);
-      const results = runEnvironmentChecks(dir);
-      const depsCheck = results.find(r => r.name === "dependencies");
-      assert.ok(depsCheck !== undefined, "dependencies check runs");
-      assert.equal(depsCheck!.status, "warning", "no marker + lockfile newer → stale warning");
-    }
-
-    // ── Env File Check ─────────────────────────────────────────────────
-    test('env: .env.example without .env detected', () => {
-      const dir = createProjectDir({
-        ".env.example": "DB_URL=xxx\nAPI_KEY=xxx\n",
-      });
-      cleanups.push(dir);
-      const results = runEnvironmentChecks(dir);
-      const envCheck = results.find(r => r.name === "env_file");
-      assert.ok(envCheck !== undefined, "env file check runs");
-      assert.deepStrictEqual(envCheck!.status, "warning", "missing .env is a warning");
-    });
-
-    test('env: .env.example with .env is ok', () => {
-      const dir = createProjectDir({
-        ".env.example": "DB_URL=xxx\n",
-        ".env": "DB_URL=postgres://localhost/test\n",
-      });
-      cleanups.push(dir);
-      const results = runEnvironmentChecks(dir);
-      const envCheck = results.find(r => r.name === "env_file");
-      assert.ok(envCheck !== undefined, "env file check runs");
-      assert.deepStrictEqual(envCheck!.status, "ok", "present .env is ok");
-    });
-
-    test('env: .env.example with .env.local is ok', () => {
-      const dir = createProjectDir({
-        ".env.example": "DB_URL=xxx\n",
-        ".env.local": "DB_URL=postgres://localhost/test\n",
-      });
-      cleanups.push(dir);
-      const results = runEnvironmentChecks(dir);
-      const envCheck = results.find(r => r.name === "env_file");
-      assert.ok(envCheck !== undefined, "env file check runs");
-      assert.deepStrictEqual(envCheck!.status, "ok", ".env.local counts as present");
-    });
-
-    // ── Disk Space Check ───────────────────────────────────────────────
-    if (process.platform !== "win32") {
-      const dir = createProjectDir();
-      cleanups.push(dir);
-      const results = runEnvironmentChecks(dir);
-      const diskCheck = results.find(r => r.name === "disk_space");
-      assert.ok(diskCheck !== undefined, "disk space check runs on unix");
-      // Should be ok on dev machines with reasonable disk
-      assert.ok(diskCheck!.status === "ok" || diskCheck!.status === "warning", "disk check returns valid status");
-    }
-
-    // ── Project Tools Check ────────────────────────────────────────────
-    test('env: detects missing python when pyproject.toml exists', () => {
-      const dir = createProjectDir({
-        "package.json": JSON.stringify({ name: "test" }),
-        "pyproject.toml": "[build-system]\nrequires = ['setuptools']\n",
-      });
-      mkdirSync(join(dir, "node_modules"), { recursive: true });
-      cleanups.push(dir);
-      const results = runEnvironmentChecks(dir);
-      const pythonCheck = results.find(r => r.name === "python");
-      // Python is likely installed on CI/dev machines, so just verify the check runs
-      // without error — the result depends on the system
-      assert.ok(true, "python check runs without error");
-    });
-
-    test('env: detects Cargo.toml', () => {
-      const dir = createProjectDir({
-        "package.json": JSON.stringify({ name: "test" }),
-        "Cargo.toml": "[package]\nname = 'test'\n",
-      });
-      mkdirSync(join(dir, "node_modules"), { recursive: true });
-      cleanups.push(dir);
-      const results = runEnvironmentChecks(dir);
-      // Just verify it runs without error
-      assert.ok(true, "cargo check runs without error");
-    });
-
-    // ── Docker Check ───────────────────────────────────────────────────
-    test('env: no docker check without Dockerfile', () => {
-      const dir = createProjectDir({
-        "package.json": JSON.stringify({ name: "test" }),
-      });
-      mkdirSync(join(dir, "node_modules"), { recursive: true });
-      cleanups.push(dir);
-      const results = runEnvironmentChecks(dir);
-      const dockerCheck = results.find(r => r.name === "docker");
-      assert.deepStrictEqual(dockerCheck, undefined, "no docker check without Dockerfile");
-    });
-
-    test('env: docker check with Dockerfile', () => {
-      const dir = createProjectDir({
-        "package.json": JSON.stringify({ name: "test" }),
-        "Dockerfile": "FROM node:22\n",
-      });
-      mkdirSync(join(dir, "node_modules"), { recursive: true });
-      cleanups.push(dir);
-      const results = runEnvironmentChecks(dir);
-      const dockerCheck = results.find(r => r.name === "docker");
-      // Docker may or may not be installed on the test machine
-      assert.ok(dockerCheck !== undefined, "docker check runs when Dockerfile present");
-    });
-
-    // ── Doctor Issue Conversion ────────────────────────────────────────
-    test('env: converts results to doctor issues', () => {
-      const results: EnvironmentCheckResult[] = [
-        { name: "node_version", status: "ok", message: "Node.js v22.0.0" },
-        { name: "dependencies", status: "error", message: "node_modules missing" },
-        { name: "env_file", status: "warning", message: ".env missing", detail: "Copy .env.example" },
-      ];
-
-      const issues = environmentResultsToDoctorIssues(results);
-      assert.deepStrictEqual(issues.length, 2, "only non-ok results converted");
-      assert.deepStrictEqual(issues[0]!.severity, "error", "error severity preserved");
-      assert.deepStrictEqual(issues[0]!.code, "env_dependencies", "code prefixed with env_");
-      assert.deepStrictEqual(issues[1]!.severity, "warning", "warning severity preserved");
-      assert.ok(issues[1]!.message.includes("Copy .env.example"), "detail included in message");
-    });
-
-    // ── checkEnvironmentHealth integration ──────────────────────────────
-    test('env: checkEnvironmentHealth adds issues to array', async () => {
-      const dir = createProjectDir({
-        "package.json": JSON.stringify({ name: "test" }),
-      });
-      cleanups.push(dir);
-
-      const issues: any[] = [];
-      await checkEnvironmentHealth(dir, issues);
-      // Should have at least the missing node_modules issue
-      assert.ok(issues.some(i => i.code === "env_dependencies"), "environment issues added to array");
-    });
-
-    // ── Report Formatting ──────────────────────────────────────────────
-    test('env: formatEnvironmentReport', () => {
-      const results: EnvironmentCheckResult[] = [
-        { name: "node_version", status: "ok", message: "Node.js v22.0.0" },
-        { name: "dependencies", status: "error", message: "node_modules missing", detail: "Run npm install" },
-        { name: "disk_space", status: "ok", message: "50.2GB free" },
-      ];
-
-      const report = formatEnvironmentReport(results);
-      assert.ok(report.includes("Environment Health:"), "has header");
-      assert.ok(report.includes("Node.js v22.0.0"), "includes ok result");
-      assert.ok(report.includes("node_modules missing"), "includes error result");
-      assert.ok(report.includes("Run npm install"), "includes detail for errors");
-    });
-
-    test('env: formatEnvironmentReport empty', () => {
-      const report = formatEnvironmentReport([]);
-      assert.deepStrictEqual(report, "No environment checks applicable.", "empty report message");
-    });
-
-    // ── Full environment checks include git remote ─────────────────────
-    test('env: runFullEnvironmentChecks includes git remote', () => {
-      // runFullEnvironmentChecks adds git remote check
-      // We can't easily test this without a real git repo, but verify it doesn't throw
-      const dir = createProjectDir();
-      cleanups.push(dir);
-      const results = runFullEnvironmentChecks(dir);
-      // No git repo → no remote check, but should not throw
-      assert.ok(true, "runFullEnvironmentChecks does not throw on non-git dir");
-    });
-
-    // ── Port Detection from package.json ───────────────────────────────
-    if (process.platform !== "win32") {
-      const dir = createProjectDir({
-        "package.json": JSON.stringify({
-          name: "test",
-          scripts: {
-            dev: "next dev --port 3456",
-            start: "node server.js",
-          },
-        }),
-      });
-      mkdirSync(join(dir, "node_modules"), { recursive: true });
-      cleanups.push(dir);
-      const results = runEnvironmentChecks(dir);
-      // Port 3456 is unlikely to be in use, so no conflicts expected
-      const portConflicts = results.filter(r => r.name === "port_conflict");
-      // Just verify it ran without error
-      assert.ok(true, "port check with script-detected ports runs without error");
-    }
-
-  } finally {
-    for (const dir of cleanups) {
-      try { rmSync(dir, { recursive: true, force: true }); } catch { /* ignore */ }
-    }
-  }
-});

package/src/resources/extensions/gsd/tests/integration/doctor-false-positives.test.ts

@@ -1,243 +0,0 @@
-import { describe, test } from 'node:test';
-import assert from 'node:assert/strict';
-import { mkdtempSync, mkdirSync, rmSync, writeFileSync, existsSync, readdirSync } from "node:fs";
-import { join } from "node:path";
-import { tmpdir } from "node:os";
-
-import { runGSDDoctor } from "../../doctor.js";
-import { parsePlan } from "../../parsers-legacy.js";
-
-// ── Helpers ─────────────────────────────────────────────────────────────────
-
-function makeBase(): { base: string; gsd: string; mDir: string } {
-  const base = mkdtempSync(join(tmpdir(), "gsd-doctor-fp-"));
-  const gsd = join(base, ".gsd");
-  const mDir = join(gsd, "milestones", "M001");
-  mkdirSync(join(mDir, "slices"), { recursive: true });
-  return { base, gsd, mDir };
-}
-
-function writeRoadmap(mDir: string, content: string): void {
-  writeFileSync(join(mDir, "M001-ROADMAP.md"), content);
-}
-
-function writeSlice(mDir: string, sliceId: string, planContent: string): string {
-  const sDir = join(mDir, "slices", sliceId);
-  const tDir = join(sDir, "tasks");
-  mkdirSync(tDir, { recursive: true });
-  writeFileSync(join(sDir, `${sliceId}-PLAN.md`), planContent);
-  return sDir;
-}
-
-describe('doctor false-positives (#3105)', async () => {
-
-  // ═══════════════════════════════════════════════════════════════════════════
-  // Bug 1: Orphaned worktree directory recreated by appendDoctorHistory
-  // ═══════════════════════════════════════════════════════════════════════════
-
-  test('Bug 1: orphaned worktree check ignores dirs containing only .gsd/doctor-history.jsonl', async () => {
-    // Simulate: a worktree dir that only contains .gsd/doctor-history.jsonl
-    // (created by appendDoctorHistory writing to the worktree-scoped path).
-    // The orphan check should NOT warn about this directory.
-    const { base, gsd } = makeBase();
-    writeRoadmap(join(gsd, "milestones", "M001"), `# M001: Test\n\n## Slices\n- [ ] **S01: Slice** \`risk:low\` \`depends:[]\`\n  > After this: done\n`);
-    writeSlice(join(gsd, "milestones", "M001"), "S01", "# S01: Slice\n\n**Goal:** G\n**Demo:** D\n\n## Tasks\n- [ ] **T01: Task** `est:10m`\n  Pending.\n");
-
-    // Create a worktree directory that only has .gsd/doctor-history.jsonl
-    const wtDir = join(gsd, "worktrees", "M042");
-    const wtGsdDir = join(wtDir, ".gsd");
-    mkdirSync(wtGsdDir, { recursive: true });
-    writeFileSync(join(wtGsdDir, "doctor-history.jsonl"), '{"ts":"2026-01-01","ok":true}\n');
-
-    const result = await runGSDDoctor(base, { fix: false });
-
-    // Should NOT produce worktree_directory_orphaned for a dir that only has doctor history
-    const orphanIssues = result.issues.filter(
-      i => i.code === "worktree_directory_orphaned" && i.unitId === "M042"
-    );
-    assert.equal(orphanIssues.length, 0,
-      "should not warn about worktree dir that only contains .gsd/doctor-history.jsonl");
-
-    rmSync(base, { recursive: true, force: true });
-  });
-
-  // ═══════════════════════════════════════════════════════════════════════════
-  // Bug 2: blocker_discovered + all tasks done = unfixable deadlock
-  // ═══════════════════════════════════════════════════════════════════════════
-
-  test('Bug 2: blocker_discovered with all tasks done should not warn (implicitly resolved)', async () => {
-    // Scenario: blocker was discovered and resolved within the same task.
-    // blocker_discovered: true, no REPLAN, but all tasks are done.
-    // Neither blocker_discovered_no_replan nor stale_replan_file should fire.
-    const { base, mDir } = makeBase();
-    writeRoadmap(mDir, `# M001: Blocker Test\n\n## Slices\n- [ ] **S01: Slice** \`risk:low\` \`depends:[]\`\n  > After this: done\n`);
-    const sDir = writeSlice(mDir, "S01",
-      "# S01: Slice\n\n**Goal:** G\n**Demo:** D\n\n## Tasks\n- [x] **T01: Task** `est:10m`\n  Done.\n");
-    writeFileSync(join(sDir, "tasks", "T01-SUMMARY.md"), `---
-id: T01
-parent: S01
-milestone: M001
-provides: []
-requires: []
-affects: []
-key_files: []
-key_decisions: []
-patterns_established: []
-observability_surfaces: []
-drill_down_paths: []
-duration: 10m
-verification_result: passed
-completed_at: 2026-01-01T00:00:00Z
-blocker_discovered: true
----
-
-# T01: Task
-
-**Done**
-
-## What Happened
-Found a blocker, resolved it in-task.
-
-## Diagnostics
-- log
-`);
-
-    const result = await runGSDDoctor(base, { fix: false });
-
-    // Should NOT produce blocker_discovered_no_replan when all tasks are done
-    const blockerIssues = result.issues.filter(i => i.code === "blocker_discovered_no_replan");
-    assert.equal(blockerIssues.length, 0,
-      "should not warn about blocker_discovered when all tasks are done (blocker was implicitly resolved)");
-
-    // Also should NOT produce stale_replan_file (no REPLAN exists, so this shouldn't fire anyway)
-    const staleReplanIssues = result.issues.filter(i => i.code === "stale_replan_file");
-    assert.equal(staleReplanIssues.length, 0,
-      "should not produce stale_replan_file when no REPLAN exists");
-
-    rmSync(base, { recursive: true, force: true });
-  });
-
-  test('Bug 2: blocker_discovered with incomplete tasks should still warn', async () => {
-    // Sanity check: when there IS an incomplete task and blocker_discovered, warn as before.
-    const { base, mDir } = makeBase();
-    writeRoadmap(mDir, `# M001: Blocker Warn Test\n\n## Slices\n- [ ] **S01: Slice** \`risk:low\` \`depends:[]\`\n  > After this: done\n`);
-    const sDir = writeSlice(mDir, "S01",
-      "# S01: Slice\n\n**Goal:** G\n**Demo:** D\n\n## Tasks\n- [x] **T01: Task A** `est:10m`\n  Done.\n- [ ] **T02: Task B** `est:10m`\n  Pending.\n");
-    writeFileSync(join(sDir, "tasks", "T01-SUMMARY.md"), `---
-id: T01
-parent: S01
-milestone: M001
-provides: []
-requires: []
-affects: []
-key_files: []
-key_decisions: []
-patterns_established: []
-observability_surfaces: []
-drill_down_paths: []
-duration: 10m
-verification_result: passed
-completed_at: 2026-01-01T00:00:00Z
-blocker_discovered: true
----
-
-# T01: Task A
-
-**Done**
-
-## What Happened
-Found blocker, but T02 is still pending.
-
-## Diagnostics
-- log
-`);
-
-    const result = await runGSDDoctor(base, { fix: false });
-
-    const blockerIssues = result.issues.filter(i => i.code === "blocker_discovered_no_replan");
-    assert.ok(blockerIssues.length > 0,
-      "should still warn about blocker_discovered when some tasks are not done");
-
-    rmSync(base, { recursive: true, force: true });
-  });
-
-  // ═══════════════════════════════════════════════════════════════════════════
-  // Bug 3: Multi-task plan — T02+ outside ## Tasks section
-  // ═══════════════════════════════════════════════════════════════════════════
-
-  test('Bug 3: parsePlan finds all tasks even when interleaved with detail sections', () => {
-    // Multi-task plan where T02 checkbox appears after T01's ## Steps heading,
-    // which ends the ## Tasks section for extractSection().
-    const planContent = `# S01: Demo Slice
-
-**Goal:** Build the demo
-**Demo:** Run it
-
-## Must-Haves
-- Feature A
-
-## Tasks
-- [x] **T01: First task** \`est:30m\`
-  Implement the first thing.
-## Steps
-1. Step one
-2. Step two
-## Must-Haves
-- Requirement A
-- [x] **T02: Second task** \`est:1h\`
-  Implement the second thing.
-## Steps
-1. Step one
-2. Step two
-`;
-
-    const plan = parsePlan(planContent);
-    const taskIds = plan.tasks.map(t => t.id);
-
-    assert.ok(taskIds.includes("T01"), "should find T01");
-    assert.ok(taskIds.includes("T02"), "should find T02 even when after T01 detail headings");
-    assert.equal(plan.tasks.length, 2, "should find exactly 2 tasks");
-  });
-
-  test('Bug 3: task_file_not_in_plan should not fire for T02 in multi-task plan', async () => {
-    const { base, mDir } = makeBase();
-    writeRoadmap(mDir, `# M001: Multi-Task Test\n\n## Slices\n- [ ] **S01: Slice** \`risk:low\` \`depends:[]\`\n  > After this: done\n`);
-
-    // Plan with interleaved headings (the problematic format)
-    const sDir = writeSlice(mDir, "S01", `# S01: Demo Slice
-
-**Goal:** Build the demo
-**Demo:** Run it
-
-## Must-Haves
-- Feature A
-
-## Tasks
-- [x] **T01: First task** \`est:30m\`
-  Implement the first thing.
-## Steps
-1. Step one
-## Must-Haves
-- Req A
-- [x] **T02: Second task** \`est:1h\`
-  Implement the second thing.
-## Steps
-1. Step one
-`);
-
-    // Both tasks have summaries on disk
-    writeFileSync(join(sDir, "tasks", "T01-SUMMARY.md"), "---\nstatus: done\ncompleted_at: 2026-01-01T00:00:00Z\n---\n# T01\nDone.\n");
-    writeFileSync(join(sDir, "tasks", "T02-SUMMARY.md"), "---\nstatus: done\ncompleted_at: 2026-01-01T00:00:00Z\n---\n# T02\nDone.\n");
-
-    const result = await runGSDDoctor(base, { fix: false });
-
-    // T02 should NOT be flagged as "not in plan"
-    const notInPlan = result.issues.filter(
-      i => i.code === "task_file_not_in_plan" && i.message.includes("T02")
-    );
-    assert.equal(notInPlan.length, 0,
-      "should not report T02 as 'not in plan' when it exists in the interleaved plan format");
-
-    rmSync(base, { recursive: true, force: true });
-  });
-});