@opengsd/gsd-pi 1.0.1 → 1.0.2

package/src/resources/extensions/gsd/tests/headless-query.test.ts

@@ -1,184 +0,0 @@
-/**
- * Tests for `gsd headless query` — single JSON snapshot command.
- *
- * Validates that the snapshot contains state, next dispatch preview,
- * and parallel worker costs in one response.
- */
-
-import { describe, it, beforeEach, afterEach } from 'node:test'
-import assert from 'node:assert/strict'
-import { mkdtempSync, mkdirSync, rmSync, writeFileSync } from 'node:fs'
-import { join } from 'node:path'
-import { tmpdir } from 'node:os'
-
-import { handleQuery } from '../../../../headless-query.ts'
-import type { QuerySnapshot } from '../../../../headless-query.ts'
-import { invalidateStateCache } from '../state.ts'
-
-// ─── Fixture Helpers ────────────────────────────────────────────────────────
-
-function createFixture(): string {
-  const base = mkdtempSync(join(tmpdir(), 'gsd-query-test-'))
-  mkdirSync(join(base, '.gsd', 'milestones'), { recursive: true })
-  return base
-}
-
-function writeRoadmap(base: string, mid: string, content: string): void {
-  const dir = join(base, '.gsd', 'milestones', mid)
-  mkdirSync(dir, { recursive: true })
-  writeFileSync(join(dir, `${mid}-ROADMAP.md`), content)
-}
-
-function writeContext(base: string, mid: string): void {
-  const dir = join(base, '.gsd', 'milestones', mid)
-  mkdirSync(dir, { recursive: true })
-  writeFileSync(join(dir, `${mid}-CONTEXT.md`), `---\ntitle: Test Milestone\n---\n\n# Context\nTest.`)
-}
-
-function writeSlicePlan(base: string, mid: string, sid: string, content: string): void {
-  const dir = join(base, '.gsd', 'milestones', mid, 'slices', sid)
-  mkdirSync(join(dir, 'tasks'), { recursive: true })
-  writeFileSync(join(dir, `${sid}-PLAN.md`), content)
-}
-
-function writeTaskPlan(base: string, mid: string, sid: string, tid: string): void {
-  const dir = join(base, '.gsd', 'milestones', mid, 'slices', sid, 'tasks')
-  mkdirSync(dir, { recursive: true })
-  writeFileSync(join(dir, `${tid}-PLAN.md`), `---\nestimated_steps: 3\nestimated_files: 2\n---\n\n# ${tid}: Test Task\nDo something.`)
-}
-
-function writeParallelStatus(base: string, mid: string, cost: number): void {
-  const dir = join(base, '.gsd', 'parallel')
-  mkdirSync(dir, { recursive: true })
-  writeFileSync(join(dir, `${mid}.status.json`), JSON.stringify({
-    milestoneId: mid,
-    pid: process.pid,
-    state: 'running',
-    currentUnit: { type: 'execute-task', id: `${mid}/S01/T01`, startedAt: Date.now() },
-    completedUnits: 2,
-    cost,
-    lastHeartbeat: Date.now(),
-    startedAt: Date.now() - 60_000,
-    worktreePath: `/tmp/worktrees/${mid}`,
-  }))
-}
-
-function createExecutingFixture(base: string): void {
-  writeContext(base, 'M001')
-  writeRoadmap(base, 'M001', `# M001: Test Milestone
-
-**Vision:** Build something.
-
-## Slices
-
-- [ ] **S01: First Slice** \`risk:low\` \`depends:[]\`
-  > After this: The first slice works.
-`)
-  writeSlicePlan(base, 'M001', 'S01', `# S01: First Slice
-
-**Goal:** Implement something.
-**Demo:** It works.
-
-## Tasks
-
-- [ ] **T01: First Task** — Do the first thing
-  - Files: foo.ts
-  - Verify: run tests
-- [ ] **T02: Second Task** — Do the second thing
-  - Files: bar.ts
-`)
-  writeTaskPlan(base, 'M001', 'S01', 'T01')
-}
-
-// ─── Tests ──────────────────────────────────────────────────────────────────
-
-describe('headless query', () => {
-  let base: string
-
-  beforeEach(() => {
-    base = createFixture()
-    invalidateStateCache()
-  })
-
-  afterEach(() => {
-    rmSync(base, { recursive: true, force: true })
-  })
-
-  it('returns snapshot with state, next, and cost', async () => {
-    createExecutingFixture(base)
-    const result = await handleQuery(base)
-    const snap = result.data as QuerySnapshot
-
-    assert.equal(result.exitCode, 0)
-    // state
-    assert.equal(snap.state.phase, 'executing')
-    assert.equal(snap.state.activeMilestone!.id, 'M001')
-    assert.equal(snap.state.activeSlice!.id, 'S01')
-    assert.equal(snap.state.activeTask!.id, 'T01')
-    assert.ok(Array.isArray(snap.state.registry))
-    assert.ok(snap.state.progress)
-    // next
-    assert.equal(snap.next.action, 'dispatch')
-    assert.equal(snap.next.unitType, 'execute-task')
-    assert.ok(snap.next.unitId)
-    // cost (no parallel workers)
-    assert.equal(snap.cost.workers.length, 0)
-    assert.equal(snap.cost.total, 0)
-  })
-
-  it('returns stop when no milestones exist', async () => {
-    const result = await handleQuery(base)
-    const snap = result.data as QuerySnapshot
-
-    assert.equal(result.exitCode, 0)
-    assert.equal(snap.state.phase, 'pre-planning')
-    assert.equal(snap.state.activeMilestone, null)
-    assert.equal(snap.next.action, 'stop')
-    assert.ok(snap.next.reason)
-  })
-
-  it('aggregates parallel worker costs', async () => {
-    createExecutingFixture(base)
-    writeParallelStatus(base, 'M001', 1.50)
-    writeParallelStatus(base, 'M002', 2.75)
-    const result = await handleQuery(base)
-    const snap = result.data as QuerySnapshot
-
-    assert.equal(snap.cost.workers.length, 2)
-    assert.equal(snap.cost.total, 4.25)
-    assert.ok(snap.cost.workers.some(w => w.milestoneId === 'M001' && w.cost === 1.50))
-    assert.ok(snap.cost.workers.some(w => w.milestoneId === 'M002' && w.cost === 2.75))
-  })
-
-  it('shows dispatch preview for pre-planning with context', async () => {
-    writeContext(base, 'M001')
-    const result = await handleQuery(base)
-    const snap = result.data as QuerySnapshot
-
-    assert.equal(snap.state.phase, 'pre-planning')
-    assert.equal(snap.state.activeMilestone!.id, 'M001')
-    assert.equal(snap.next.action, 'dispatch')
-  })
-
-  it('reports all milestones complete with a clean stop reason', async () => {
-    writeRoadmap(base, 'M001', `# M001: Test Milestone
-
-## Slices
-
-- [x] **S01: First Slice** \`risk:low\` \`depends:[]\`
-  > Done.
-`)
-    writeFileSync(
-      join(base, '.gsd', 'milestones', 'M001', 'M001-SUMMARY.md'),
-      '# M001 Summary\n\nComplete.',
-    )
-
-    const result = await handleQuery(base)
-    const snap = result.data as QuerySnapshot
-
-    assert.equal(result.exitCode, 0)
-    assert.equal(snap.state.phase, 'complete')
-    assert.equal(snap.next.action, 'stop')
-    assert.equal(snap.next.reason, 'All milestones complete.')
-  })
-})

package/src/resources/extensions/gsd/tests/integration/auto-preflight.test.ts

@@ -1,38 +0,0 @@
-import test from "node:test";
-import assert from "node:assert/strict";
-import { mkdtempSync, mkdirSync, rmSync, writeFileSync } from "node:fs";
-import { join } from "node:path";
-import { tmpdir } from "node:os";
-
-import { runGSDDoctor, selectDoctorScope, filterDoctorIssues } from "../../doctor.js";
-
-test("auto-preflight scopes to active milestone, ignoring historical", async (t) => {
-  const tmpBase = mkdtempSync(join(tmpdir(), "gsd-auto-preflight-test-"));
-  const gsd = join(tmpBase, ".gsd");
-
-  mkdirSync(join(gsd, "milestones", "M001", "slices", "S01", "tasks"), { recursive: true });
-  mkdirSync(join(gsd, "milestones", "M009", "slices", "S01", "tasks"), { recursive: true });
-
-  writeFileSync(join(gsd, "milestones", "M001", "M001-ROADMAP.md"), `# M001: Historical\n\n## Slices\n- [x] **S01: Old Slice** \`risk:low\` \`depends:[]\`\n  > After this: old done\n`);
-  writeFileSync(join(gsd, "milestones", "M001", "slices", "S01", "S01-PLAN.md"), `# S01: Old Slice\n\n**Goal:** Old\n**Demo:** Old\n\n## Must-Haves\n- done\n\n## Tasks\n- [x] **T01: Old Task** \`est:5m\`\n  done\n`);
-  writeFileSync(join(gsd, "milestones", "M001", "slices", "S01", "tasks", "T01-SUMMARY.md"), `---\nid: T01\nparent: S01\nmilestone: M001\nprovides: []\nrequires: []\naffects: []\nkey_files: []\nkey_decisions: []\npatterns_established: []\nobservability_surfaces: []\ndrill_down_paths: []\nduration: 5m\nverification_result: passed\ncompleted_at: 2026-03-09T00:00:00Z\n---\n\n# T01: Old Task\n\n**Done**\n\n## What Happened\nDone.\n\n## Diagnostics\n- log\n`);
-  writeFileSync(join(gsd, "milestones", "M001", "slices", "S01", "S01-SUMMARY.md"), `---\nid: S01\nparent: M001\nmilestone: M001\nprovides: []\nrequires: []\naffects: []\nkey_files: []\nkey_decisions: []\npatterns_established: []\nobservability_surfaces: []\ndrill_down_paths: []\nduration: 5m\nverification_result: passed\ncompleted_at: 2026-03-09T00:00:00Z\n---\n\n# S01: Old Slice\n\n**Done**\n\n## What Happened\nDone.\n\n## Verification\nDone.\n\n## Deviations\nNone\n\n## Known Limitations\nNone\n\n## Follow-ups\nNone\n\n## Files Created/Modified\n- \`x\` — x\n\n## Forward Intelligence\n\n### What the next slice should know\n- x\n\n### What's fragile\n- x\n\n### Authoritative diagnostics\n- x\n\n### What assumptions changed\n- x\n`);
-  writeFileSync(join(gsd, "milestones", "M001", "M001-VALIDATION.md"), `---\nverdict: pass\nremediation_round: 0\n---\n\n# Validation\nPassed.\n`);
-  writeFileSync(join(gsd, "milestones", "M001", "M001-SUMMARY.md"), `---\nid: M001\nstatus: complete\ncompleted_at: 2026-03-09T00:00:00Z\n---\n\n# M001: Historical\n\nComplete.\n`);
-
-  writeFileSync(join(gsd, "milestones", "M009", "M009-ROADMAP.md"), `# M009: Active\n\n## Slices\n- [ ] **S01: Active Slice** \`risk:low\` \`depends:[]\`\n  > After this: active works\n`);
-  writeFileSync(join(gsd, "milestones", "M009", "slices", "S01", "S01-PLAN.md"), `# S01: Active Slice\n\n**Goal:** Active\n**Demo:** Active\n\n## Must-Haves\n- done\n\n## Tasks\n- [ ] **T01: Active Task** \`est:5m\`\n  todo\n`);
-
-  t.after(() => rmSync(tmpBase, { recursive: true, force: true }));
-
-  const scope = await selectDoctorScope(tmpBase);
-  assert.equal(scope, "M009/S01", "active scope selected instead of historical milestone");
-
-  const scopedReport = await runGSDDoctor(tmpBase, { fix: false, scope });
-  const scopedBlocking = filterDoctorIssues(scopedReport.issues, { scope, includeWarnings: false });
-  assert.equal(scopedBlocking.length, 0, "no blocking issues in active scope");
-
-  const historicalReport = await runGSDDoctor(tmpBase, { fix: false });
-  const historicalWarnings = historicalReport.issues.filter(issue => issue.unitId.startsWith("M001/S01") && issue.severity === "warning");
-  assert.equal(historicalWarnings.length, 0, "completed historical milestone produces no checkbox/file-mismatch warnings");
-});

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

@@ -1,83 +0,0 @@
-/**
- * Test: Doctor auto-fix for delimiter_in_title
- *
- * Verifies that `runGSDDoctor({ fix: true })` sanitizes em/en dashes
- * in milestone H1 titles by replacing them with ASCII hyphens.
- */
-
-import test from "node:test";
-import assert from "node:assert/strict";
-import { mkdtempSync, mkdirSync, readFileSync, rmSync, writeFileSync } from "node:fs";
-import { join } from "node:path";
-import { tmpdir } from "node:os";
-import { runGSDDoctor } from "../../doctor.js";
-
-test("doctor fix=true sanitizes em-dash in milestone title", async (t) => {
-  const tmpBase = mkdtempSync(join(tmpdir(), "gsd-doctor-delim-"));
-  const gsd = join(tmpBase, ".gsd");
-  const mDir = join(gsd, "milestones", "M001");
-  const sDir = join(mDir, "slices", "S01");
-  const tDir = join(sDir, "tasks");
-  mkdirSync(tDir, { recursive: true });
-
-  const roadmapWithEmDash = `# M001: Cockpit Foundation \u2014 Daemon + State Bridge
-
-## Success Criteria
-- HTTP server runs
-
-## Slices
-- [ ] **S01: Initial Setup** \`risk:low\` \`depends:[]\`
-  > After this: setup works
-`;
-
-  writeFileSync(join(mDir, "M001-ROADMAP.md"), roadmapWithEmDash);
-  writeFileSync(join(sDir, "S01-PLAN.md"), `# S01: Initial Setup\n\n## Tasks\n- [ ] **T01: Scaffold** \`est:15m\`\n`);
-  writeFileSync(join(tDir, "T01-PLAN.md"), "# T01: Scaffold\n");
-
-  t.after(() => rmSync(tmpBase, { recursive: true, force: true }));
-
-  // Run doctor with fix=true
-  const report = await runGSDDoctor(tmpBase, { fix: true });
-
-  // The em-dash should have been replaced
-  const fixed = readFileSync(join(mDir, "M001-ROADMAP.md"), "utf-8");
-  const h1 = fixed.split("\n").find(l => l.startsWith("# "))!;
-  assert.ok(h1, "H1 line should exist");
-  assert.ok(!h1.includes("\u2014"), "em-dash should be replaced");
-  assert.ok(!h1.includes("\u2013"), "en-dash should be replaced");
-  assert.ok(h1.includes("-"), "should contain ASCII hyphen as replacement");
-
-  // Should have recorded the fix
-  assert.ok(
-    report.fixesApplied.some(f => f.includes("sanitized")),
-    `fixesApplied should mention sanitization, got: ${JSON.stringify(report.fixesApplied)}`,
-  );
-
-  // The issue should NOT appear in the report (it was fixed)
-  const delimIssues = report.issues.filter(i => i.code === "delimiter_in_title" && i.unitId === "M001");
-  assert.equal(delimIssues.length, 0, "fixed issue should not appear in issues list");
-});
-
-test("doctor fix=false still reports delimiter_in_title as warning", async (t) => {
-  const tmpBase = mkdtempSync(join(tmpdir(), "gsd-doctor-delim-nf-"));
-  const gsd = join(tmpBase, ".gsd");
-  const mDir = join(gsd, "milestones", "M001");
-  const sDir = join(mDir, "slices", "S01");
-  const tDir = join(sDir, "tasks");
-  mkdirSync(tDir, { recursive: true });
-
-  writeFileSync(join(mDir, "M001-ROADMAP.md"), `# M001: Foundation \u2014 Core\n\n## Slices\n- [ ] **S01: Setup** \`risk:low\` \`depends:[]\`\n  > After: done\n`);
-  writeFileSync(join(sDir, "S01-PLAN.md"), `# S01: Setup\n\n## Tasks\n- [ ] **T01: Init** \`est:10m\`\n`);
-  writeFileSync(join(tDir, "T01-PLAN.md"), "# T01: Init\n");
-
-  t.after(() => rmSync(tmpBase, { recursive: true, force: true }));
-
-  const report = await runGSDDoctor(tmpBase, { fix: false });
-  const delimIssues = report.issues.filter(i => i.code === "delimiter_in_title");
-  assert.ok(delimIssues.length > 0, "should report delimiter_in_title as issue when fix=false");
-  assert.equal(delimIssues[0].severity, "warning");
-
-  // File should be unchanged
-  const content = readFileSync(join(mDir, "M001-ROADMAP.md"), "utf-8");
-  assert.ok(content.includes("\u2014"), "file should not be modified when fix=false");
-});

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

@@ -1,243 +0,0 @@
-import { describe, test } from 'node:test';
-import assert from 'node:assert/strict';
-import { mkdtempSync, mkdirSync, rmSync, writeFileSync, existsSync } from "node:fs";
-import { join } from "node:path";
-import { tmpdir } from "node:os";
-
-import { runGSDDoctor } from "../../doctor.js";
-import { formatDoctorReportJson } from "../../doctor-format.js";
-// ── Helpers ─────────────────────────────────────────────────────────────────
-
-function makeBase(): { base: string; gsd: string; mDir: string } {
-  const base = mkdtempSync(join(tmpdir(), "gsd-doctor-enh-"));
-  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-enhancements', async () => {
-  // ── 1. Circular dependency detection ──────────────────────────────────────
-  test('circular dependency detection', async () => {
-    const { base, mDir } = makeBase();
-    writeRoadmap(mDir, `# M001: Circular Test\n\n## Slices\n- [ ] **S01: Slice A** \`risk:low\` \`depends:[S02]\`\n  > After this: done\n- [ ] **S02: Slice B** \`risk:low\` \`depends:[S01]\`\n  > After this: done\n`);
-    writeSlice(mDir, "S01", "# S01: Slice A\n\n**Goal:** A\n**Demo:** A\n\n## Tasks\n- [ ] **T01: Task** `est:10m`\n  Pending.\n");
-    writeSlice(mDir, "S02", "# S02: Slice B\n\n**Goal:** B\n**Demo:** B\n\n## Tasks\n- [ ] **T01: Task** `est:10m`\n  Pending.\n");
-
-    const result = await runGSDDoctor(base, { fix: false });
-    assert.ok(
-      result.issues.some(i => i.code === "circular_slice_dependency"),
-      "detects circular dependency S01 → S02 → S01",
-    );
-    rmSync(base, { recursive: true, force: true });
-  });
-
-  // ── 2. Duplicate task IDs ──────────────────────────────────────────────────
-  test('duplicate task IDs', async () => {
-    const { base, mDir } = makeBase();
-    writeRoadmap(mDir, `# M001: Dup Test\n\n## Slices\n- [ ] **S01: Slice** \`risk:low\` \`depends:[]\`\n  > After this: done\n`);
-    writeSlice(mDir, "S01", "# S01: Slice\n\n**Goal:** G\n**Demo:** D\n\n## Tasks\n- [ ] **T01: First** `est:10m`\n  Task one.\n- [ ] **T01: Duplicate** `est:10m`\n  Task dup.\n");
-
-    const result = await runGSDDoctor(base, { fix: false });
-    assert.ok(
-      result.issues.some(i => i.code === "duplicate_task_id"),
-      "detects duplicate task ID T01",
-    );
-    rmSync(base, { recursive: true, force: true });
-  });
-
-  // ── 3. Orphaned slice directory ──────────────────────────────────────────
-  test('orphaned slice directory', async () => {
-    const { base, mDir } = makeBase();
-    writeRoadmap(mDir, `# M001: Orphan Test\n\n## Slices\n- [ ] **S01: Slice** \`risk:low\` \`depends:[]\`\n  > After this: done\n`);
-    writeSlice(mDir, "S01", "# S01: Slice\n\n**Goal:** G\n**Demo:** D\n\n## Tasks\n- [ ] **T01: Task** `est:10m`\n  Pending.\n");
-    // Create an extra slice directory not in roadmap
-    mkdirSync(join(mDir, "slices", "S99"), { recursive: true });
-
-    const result = await runGSDDoctor(base, { fix: false });
-    assert.ok(
-      result.issues.some(i => i.code === "orphaned_slice_directory" && i.message.includes("S99")),
-      "detects orphaned slice directory S99",
-    );
-    rmSync(base, { recursive: true, force: true });
-  });
-
-  // ── 4. Task file not in plan ───────────────────────────────────────────────
-  test('task file not in plan', async () => {
-    const { base, mDir } = makeBase();
-    writeRoadmap(mDir, `# M001: Extra Task 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");
-    // T01 summary (matches plan)
-    writeFileSync(join(sDir, "tasks", "T01-SUMMARY.md"), "---\nstatus: done\n---\n# T01\nDone.\n");
-    // T99 summary (NOT in plan)
-    writeFileSync(join(sDir, "tasks", "T99-SUMMARY.md"), "---\nstatus: done\n---\n# T99\nExtra.\n");
-
-    const result = await runGSDDoctor(base, { fix: false });
-    assert.ok(
-      result.issues.some(i => i.code === "task_file_not_in_plan" && i.message.includes("T99")),
-      "detects task summary T99 not in plan",
-    );
-    rmSync(base, { recursive: true, force: true });
-  });
-
-  // ── 5. Stale REPLAN file ────────────────────────────────────────────────────
-  test('stale REPLAN detection', async () => {
-    const { base, mDir } = makeBase();
-    writeRoadmap(mDir, `# M001: Replan 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"), "---\nstatus: done\ncompleted_at: 2026-01-01T00:00:00Z\n---\n# T01\nDone.\n");
-    // Add a REPLAN file even though all tasks are done
-    writeFileSync(join(sDir, "S01-REPLAN.md"), "# S01 REPLAN\nSomething changed.\n");
-
-    const result = await runGSDDoctor(base, { fix: false });
-    assert.ok(
-      result.issues.some(i => i.code === "stale_replan_file"),
-      "detects stale REPLAN when all tasks are done",
-    );
-    rmSync(base, { recursive: true, force: true });
-  });
-
-  // ── 6. Metrics ledger corrupt ───────────────────────────────────────────────
-  test('metrics ledger corrupt', async () => {
-    const { base, gsd, mDir } = makeBase();
-    writeRoadmap(mDir, `# M001: Metrics Test\n\n## Slices\n- [ ] **S01: Slice** \`risk:low\` \`depends:[]\`\n  > After this: done\n`);
-    writeSlice(mDir, "S01", "# S01: Slice\n\n**Goal:** G\n**Demo:** D\n\n## Tasks\n- [ ] **T01: Task** `est:10m`\n  Pending.\n");
-    // Write invalid metrics.json
-    writeFileSync(join(gsd, "metrics.json"), '{"version":2,"data":[]}');
-
-    const result = await runGSDDoctor(base, { fix: false });
-    assert.ok(
-      result.issues.some(i => i.code === "metrics_ledger_corrupt"),
-      "detects corrupt metrics ledger (version != 1)",
-    );
-    rmSync(base, { recursive: true, force: true });
-  });
-
-  // ── 7. Large planning file ──────────────────────────────────────────────────
-  test('large planning file', async () => {
-    const { base, mDir } = makeBase();
-    writeRoadmap(mDir, `# M001: Large File 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- [ ] **T01: Task** `est:10m`\n  Pending.\n");
-    // Write a 101KB .md file
-    const bigContent = "# Big File\n" + "x".repeat(101 * 1024);
-    writeFileSync(join(sDir, "BIGFILE.md"), bigContent);
-
-    const result = await runGSDDoctor(base, { fix: false });
-    assert.ok(
-      result.issues.some(i => i.code === "large_planning_file"),
-      "detects large planning file over 100KB",
-    );
-    rmSync(base, { recursive: true, force: true });
-  });
-
-  // ── 8. Future timestamp ─────────────────────────────────────────────────────
-  test('future timestamp', async () => {
-    const { base, mDir } = makeBase();
-    writeRoadmap(mDir, `# M001: Timestamp 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");
-    // completed_at is 2 days in the future
-    const futureDate = new Date(Date.now() + 2 * 24 * 60 * 60 * 1000).toISOString();
-    writeFileSync(
-      join(sDir, "tasks", "T01-SUMMARY.md"),
-      `---\nstatus: done\ncompleted_at: ${futureDate}\n---\n# T01\nDone.\n`,
-    );
-
-    const result = await runGSDDoctor(base, { fix: false });
-    assert.ok(
-      result.issues.some(i => i.code === "future_timestamp"),
-      "detects future completed_at timestamp",
-    );
-    rmSync(base, { recursive: true, force: true });
-  });
-
-  // ── 9. JSON output format ───────────────────────────────────────────────────
-  test('JSON output format', async () => {
-    const { base, mDir } = makeBase();
-    writeRoadmap(mDir, `# M001: JSON Test\n\n## Slices\n- [ ] **S01: Slice** \`risk:low\` \`depends:[]\`\n  > After this: done\n`);
-    writeSlice(mDir, "S01", "# S01: Slice\n\n**Goal:** G\n**Demo:** D\n\n## Tasks\n- [ ] **T01: Task** `est:10m`\n  Pending.\n");
-
-    const result = await runGSDDoctor(base, { fix: false });
-    const json = formatDoctorReportJson(result);
-
-    let parsed: unknown;
-    try {
-      parsed = JSON.parse(json);
-    } catch {
-      parsed = null;
-    }
-
-    assert.ok(parsed !== null, "formatDoctorReportJson produces valid JSON");
-    assert.ok(typeof (parsed as Record<string, unknown>)?.ok === "boolean", "JSON has ok field");
-    assert.ok(Array.isArray((parsed as Record<string, unknown>)?.issues), "JSON has issues array");
-    assert.ok(Array.isArray((parsed as Record<string, unknown>)?.fixesApplied), "JSON has fixesApplied array");
-    assert.ok(typeof (parsed as Record<string, unknown>)?.generatedAt === "string", "JSON has generatedAt field");
-    assert.ok(typeof (parsed as Record<string, unknown>)?.summary === "object", "JSON has summary object");
-
-    rmSync(base, { recursive: true, force: true });
-  });
-
-  // ── 10. Dry-run mode ────────────────────────────────────────────────────────
-  test('dry-run mode', async () => {
-    const { base, mDir } = makeBase();
-    writeRoadmap(mDir, `# M001: Dry Run Test\n\n## Slices\n- [ ] **S01: Slice** \`risk:low\` \`depends:[]\`\n  > After this: done\n`);
-    writeSlice(mDir, "S01", "# S01: Slice\n\n**Goal:** G\n**Demo:** D\n\n## Tasks\n- [ ] **T01: Task** `est:10m`\n  Pending.\n");
-
-    const result = await runGSDDoctor(base, { fix: true, dryRun: true });
-    // dry-run with fix:true still runs the doctor; shouldFix() returns false
-    // so no reconciliation fixes are applied through that path
-    assert.ok(result.issues !== undefined, "dry-run still produces issue list");
-    assert.ok(Array.isArray(result.fixesApplied), "dry-run report has fixesApplied array");
-
-    rmSync(base, { recursive: true, force: true });
-  });
-
-  // ── 11. Per-check timing ─────────────────────────────────────────────────────
-  test('per-check timing', async () => {
-    const { base, mDir } = makeBase();
-    writeRoadmap(mDir, `# M001: Timing Test\n\n## Slices\n- [ ] **S01: Slice** \`risk:low\` \`depends:[]\`\n  > After this: done\n`);
-    writeSlice(mDir, "S01", "# S01: Slice\n\n**Goal:** G\n**Demo:** D\n\n## Tasks\n- [ ] **T01: Task** `est:10m`\n  Pending.\n");
-
-    const result = await runGSDDoctor(base, { fix: false });
-    assert.ok(result.timing !== undefined, "report includes timing");
-    assert.ok(typeof result.timing?.git === "number", "timing.git is a number");
-    assert.ok(typeof result.timing?.runtime === "number", "timing.runtime is a number");
-    assert.ok(typeof result.timing?.environment === "number", "timing.environment is a number");
-    assert.ok(typeof result.timing?.gsdState === "number", "timing.gsdState is a number");
-
-    rmSync(base, { recursive: true, force: true });
-  });
-
-  // ── 12. Doctor history ───────────────────────────────────────────────────────
-  test('doctor history', async () => {
-    const { base, gsd, mDir } = makeBase();
-    writeRoadmap(mDir, `# M001: History Test\n\n## Slices\n- [ ] **S01: Slice** \`risk:low\` \`depends:[]\`\n  > After this: done\n`);
-    writeSlice(mDir, "S01", "# S01: Slice\n\n**Goal:** G\n**Demo:** D\n\n## Tasks\n- [ ] **T01: Task** `est:10m`\n  Pending.\n");
-
-    await runGSDDoctor(base, { fix: false });
-
-    const historyPath = join(gsd, "doctor-history.jsonl");
-    assert.ok(existsSync(historyPath), "doctor-history.jsonl is created after run");
-
-    const { readDoctorHistory } = await import("../../doctor.js");
-    const history = await readDoctorHistory(base);
-    assert.ok(history.length >= 1, "history has at least one entry");
-    assert.ok(typeof history[0]?.ts === "string", "history entry has ts field");
-    assert.ok(typeof history[0]?.ok === "boolean", "history entry has ok field");
-    assert.ok(typeof history[0]?.errors === "number", "history entry has errors count");
-    assert.ok(Array.isArray(history[0]?.codes), "history entry has codes array");
-
-    rmSync(base, { recursive: true, force: true });
-  });
-});