gsd-pi 2.10.12 → 2.12.0

package/dist/resources/extensions/gsd/tests/migrate-parser.test.ts

@@ -0,0 +1,757 @@
+// Migration parser test suite
+// Tests for parsing old .planning directories into typed PlanningProject structures.
+// Uses synthetic fixture directories — no real .planning dirs needed.
+
+import { mkdtempSync, mkdirSync, rmSync, writeFileSync } from 'node:fs';
+import { join } from 'node:path';
+import { tmpdir } from 'node:os';
+
+import { parsePlanningDirectory } from '../migrate/parser.ts';
+import { validatePlanningDirectory } from '../migrate/validator.ts';
+
+import type { PlanningProject, ValidationResult } from '../migrate/types.ts';
+import { createTestContext } from './test-helpers.ts';
+
+const { assertEq, assertTrue, report } = createTestContext();
+// ─── Fixture Helpers ───────────────────────────────────────────────────────
+
+function createFixtureBase(): string {
+  return mkdtempSync(join(tmpdir(), 'gsd-migrate-test-'));
+}
+
+function createPlanningDir(base: string): string {
+  const dir = join(base, '.planning');
+  mkdirSync(dir, { recursive: true });
+  return dir;
+}
+
+function writeFile(dir: string, ...pathParts: string[]): (content: string) => void {
+  return (content: string) => {
+    const filePath = join(dir, ...pathParts);
+    mkdirSync(join(filePath, '..'), { recursive: true });
+    writeFileSync(filePath, content);
+  };
+}
+
+function cleanup(base: string): void {
+  rmSync(base, { recursive: true, force: true });
+}
+
+// ─── Sample Fixtures ───────────────────────────────────────────────────────
+
+const SAMPLE_ROADMAP = `# Project Roadmap
+
+## Phases
+
+- [x] 29 — Auth System
+- [ ] 30 — Dashboard
+- [ ] 31 — Notifications
+`;
+
+const SAMPLE_PROJECT = `# My Project
+
+A sample project for testing the migration parser.
+
+## Goals
+
+- Build a thing
+- Ship it
+`;
+
+const SAMPLE_REQUIREMENTS = `# Requirements
+
+## Active
+
+### R001 — User Authentication
+- Status: active
+- Description: Users must be able to log in.
+
+### R002 — Dashboard View
+- Status: active
+- Description: Main dashboard page.
+
+## Validated
+
+### R003 — Session Management
+- Status: validated
+- Description: Sessions expire after 24h.
+
+## Deferred
+
+### R004 — OAuth Support
+- Status: deferred
+- Description: Third-party login.
+`;
+
+const SAMPLE_STATE = `# State
+
+**Current Phase:** 30-dashboard
+**Status:** in-progress
+`;
+
+const SAMPLE_CONFIG = JSON.stringify({
+  projectName: 'test-project',
+  version: '1.0',
+});
+
+const SAMPLE_PLAN_XML = `---
+phase: "29-auth-system"
+plan: "01"
+type: "implementation"
+wave: 1
+depends_on: []
+files_modified: [src/auth.ts, src/login.ts]
+autonomous: true
+must_haves:
+  truths:
+    - Users can log in
+  artifacts:
+    - src/auth.ts
+  key_links: []
+---
+
+# 29-01: Implement Auth
+
+<objective>
+Build the authentication system with JWT tokens and session management.
+</objective>
+
+<tasks>
+<task>Create auth middleware</task>
+<task>Add login endpoint</task>
+<task>Add logout endpoint</task>
+</tasks>
+
+<context>
+The project needs authentication before any other features can be built.
+Auth tokens use JWT with RS256 signing.
+</context>
+
+<verification>
+- Login returns valid JWT
+- Middleware rejects invalid tokens
+- Logout invalidates session
+</verification>
+
+<success_criteria>
+All auth endpoints respond correctly and tokens are validated.
+</success_criteria>
+`;
+
+const SAMPLE_SUMMARY = `---
+phase: "29-auth-system"
+plan: "01"
+subsystem: "auth"
+tags:
+  - authentication
+  - security
+requires: []
+provides:
+  - auth-middleware
+  - jwt-validation
+affects:
+  - api-routes
+tech-stack:
+  - jsonwebtoken
+  - express
+key-files:
+  - src/auth.ts
+  - src/middleware/auth.ts
+key-decisions:
+  - Use RS256 for JWT signing
+  - Store refresh tokens in DB
+patterns-established:
+  - Middleware-based auth
+duration: "2h"
+completed: "2026-01-15"
+---
+
+# 29-01: Auth Implementation Summary
+
+Authentication system implemented with JWT tokens.
+
+## What Happened
+
+Built the auth middleware and login/logout endpoints.
+
+## Files Modified
+
+- \`src/auth.ts\` — Core auth logic
+- \`src/middleware/auth.ts\` — Express middleware
+`;
+
+const SAMPLE_RESEARCH = `# Auth Research
+
+## JWT vs Session Tokens
+
+JWT tokens are stateless and work well for microservices.
+Session tokens require server-side storage but are easier to revoke.
+
+## Decision
+
+Use JWT with short expiry + refresh tokens.
+`;
+
+const SAMPLE_MILESTONE_ROADMAP = `# Milestone v2.2 Roadmap
+
+## Phases
+
+- [x] 29 — Auth System
+- [x] 30 — Dashboard
+`;
+
+const SAMPLE_MILESTONE_SECTIONED_ROADMAP = `# Project Roadmap
+
+## v2.0 — Foundation
+
+<details>
+<summary>Completed</summary>
+
+- [x] 01 — Project Setup
+- [x] 02 — Database Schema
+
+</details>
+
+## v2.5 — Features
+
+- [x] 29 — Auth System
+- [ ] 30 — Dashboard
+- [ ] 31 — Notifications
+`;
+
+const SAMPLE_QUICK_PLAN = `# 001: Fix Login Bug
+
+## Description
+
+Fix the login button not responding on mobile.
+
+## Steps
+
+1. Debug click handler
+2. Fix event propagation
+3. Test on mobile
+`;
+
+const SAMPLE_QUICK_SUMMARY = `# 001: Fix Login Bug — Summary
+
+Fixed the login button by correcting the touch event handler.
+`;
+
+// ═══════════════════════════════════════════════════════════════════════════
+// Test Groups
+// ═══════════════════════════════════════════════════════════════════════════
+
+async function main(): Promise<void> {
+
+  // ─── Test 1: Complete .planning directory ──────────────────────────────
+  console.log('\n=== Complete .planning directory with all file types ===');
+  {
+    const base = createFixtureBase();
+    try {
+      const planning = createPlanningDir(base);
+
+      // Root files
+      writeFileSync(join(planning, 'PROJECT.md'), SAMPLE_PROJECT);
+      writeFileSync(join(planning, 'ROADMAP.md'), SAMPLE_ROADMAP);
+      writeFileSync(join(planning, 'REQUIREMENTS.md'), SAMPLE_REQUIREMENTS);
+      writeFileSync(join(planning, 'STATE.md'), SAMPLE_STATE);
+      writeFileSync(join(planning, 'config.json'), SAMPLE_CONFIG);
+
+      // Phase directory with plan, summary, research
+      const phaseDir = join(planning, 'phases', '29-auth-system');
+      mkdirSync(phaseDir, { recursive: true });
+      writeFileSync(join(phaseDir, '29-01-PLAN.md'), SAMPLE_PLAN_XML);
+      writeFileSync(join(phaseDir, '29-01-SUMMARY.md'), SAMPLE_SUMMARY);
+      writeFileSync(join(phaseDir, '29-RESEARCH.md'), SAMPLE_RESEARCH);
+
+      // Second phase directory
+      const phase2Dir = join(planning, 'phases', '30-dashboard');
+      mkdirSync(phase2Dir, { recursive: true });
+      writeFileSync(join(phase2Dir, '30-01-PLAN.md'), `---
+phase: "30-dashboard"
+plan: "01"
+type: "implementation"
+wave: 1
+depends_on: [29-01]
+files_modified: []
+autonomous: false
+---
+
+# 30-01: Build Dashboard
+
+<objective>
+Create the main dashboard view.
+</objective>
+
+<tasks>
+<task>Create dashboard component</task>
+<task>Add data fetching</task>
+</tasks>
+
+<context>
+Dashboard needs auth to be complete first.
+</context>
+`);
+
+      // Quick tasks
+      const quickDir = join(planning, 'quick', '001-fix-login');
+      mkdirSync(quickDir, { recursive: true });
+      writeFileSync(join(quickDir, '001-PLAN.md'), SAMPLE_QUICK_PLAN);
+      writeFileSync(join(quickDir, '001-SUMMARY.md'), SAMPLE_QUICK_SUMMARY);
+
+      // Milestones
+      const msDir = join(planning, 'milestones');
+      mkdirSync(msDir, { recursive: true });
+      writeFileSync(join(msDir, 'v2.2-ROADMAP.md'), SAMPLE_MILESTONE_ROADMAP);
+      writeFileSync(join(msDir, 'v2.2-REQUIREMENTS.md'), 'Milestone requirements here.');
+
+      // Research at root
+      const researchDir = join(planning, 'research');
+      mkdirSync(researchDir, { recursive: true });
+      writeFileSync(join(researchDir, 'architecture.md'), '# Architecture Research\n\nNotes.');
+
+      const project = await parsePlanningDirectory(planning);
+
+      // Top-level structure
+      assertEq(project.path, planning, 'project.path matches');
+      assertTrue(project.project !== null, 'PROJECT.md parsed');
+      assertTrue(project.roadmap !== null, 'ROADMAP.md parsed');
+      assertTrue(project.requirements.length > 0, 'requirements parsed');
+      assertTrue(project.state !== null, 'STATE.md parsed');
+      assertTrue(project.config !== null, 'config.json parsed');
+
+      // Phases
+      assertTrue('29-auth-system' in project.phases, 'phase 29 present');
+      assertTrue('30-dashboard' in project.phases, 'phase 30 present');
+
+      const phase29 = project.phases['29-auth-system'];
+      assertEq(phase29?.number, 29, 'phase 29 number');
+      assertEq(phase29?.slug, 'auth-system', 'phase 29 slug');
+      assertTrue('01' in (phase29?.plans ?? {}), 'phase 29 has plan 01');
+      assertTrue('01' in (phase29?.summaries ?? {}), 'phase 29 has summary 01');
+      assertTrue((phase29?.research?.length ?? 0) > 0, 'phase 29 has research');
+
+      // Plan content (XML-in-markdown)
+      const plan29 = phase29?.plans?.['01'];
+      assertTrue(plan29 !== undefined, 'plan 29-01 exists');
+      assertTrue(plan29?.objective?.includes('authentication') ?? false, 'plan objective extracted');
+      assertTrue((plan29?.tasks?.length ?? 0) >= 3, 'plan tasks extracted');
+      assertTrue(plan29?.context?.includes('JWT') ?? false, 'plan context extracted');
+      assertTrue(plan29?.verification !== '', 'plan verification extracted');
+      assertTrue(plan29?.successCriteria !== '', 'plan success criteria extracted');
+
+      // Plan frontmatter
+      assertEq(plan29?.frontmatter?.phase, '29-auth-system', 'plan frontmatter phase');
+      assertEq(plan29?.frontmatter?.plan, '01', 'plan frontmatter plan');
+      assertEq(plan29?.frontmatter?.type, 'implementation', 'plan frontmatter type');
+      assertEq(plan29?.frontmatter?.wave, 1, 'plan frontmatter wave');
+      assertEq(plan29?.frontmatter?.autonomous, true, 'plan frontmatter autonomous');
+
+      // Summary content
+      const summary29 = phase29?.summaries?.['01'];
+      assertTrue(summary29 !== undefined, 'summary 29-01 exists');
+      assertEq(summary29?.frontmatter?.phase, '29-auth-system', 'summary frontmatter phase');
+      assertEq(summary29?.frontmatter?.plan, '01', 'summary frontmatter plan');
+      assertEq(summary29?.frontmatter?.subsystem, 'auth', 'summary frontmatter subsystem');
+      assertTrue((summary29?.frontmatter?.tags?.length ?? 0) >= 2, 'summary frontmatter tags');
+      assertTrue((summary29?.frontmatter?.provides?.length ?? 0) >= 2, 'summary frontmatter provides');
+      assertTrue((summary29?.frontmatter?.affects?.length ?? 0) >= 1, 'summary frontmatter affects');
+      assertTrue((summary29?.frontmatter?.['tech-stack']?.length ?? 0) >= 2, 'summary frontmatter tech-stack');
+      assertTrue((summary29?.frontmatter?.['key-files']?.length ?? 0) >= 2, 'summary frontmatter key-files');
+      assertTrue((summary29?.frontmatter?.['key-decisions']?.length ?? 0) >= 2, 'summary frontmatter key-decisions');
+      assertTrue((summary29?.frontmatter?.['patterns-established']?.length ?? 0) >= 1, 'summary frontmatter patterns-established');
+      assertEq(summary29?.frontmatter?.duration, '2h', 'summary frontmatter duration');
+      assertEq(summary29?.frontmatter?.completed, '2026-01-15', 'summary frontmatter completed');
+
+      // Quick tasks
+      assertTrue(project.quickTasks.length >= 1, 'quick tasks parsed');
+      assertEq(project.quickTasks[0]?.number, 1, 'quick task number');
+      assertTrue(project.quickTasks[0]?.plan !== null, 'quick task has plan');
+      assertTrue(project.quickTasks[0]?.summary !== null, 'quick task has summary');
+
+      // Milestones
+      assertTrue(project.milestones.length >= 1, 'milestones parsed');
+
+      // Root research
+      assertTrue(project.research.length >= 1, 'root research parsed');
+
+      // Config
+      assertEq(project.config?.projectName, 'test-project', 'config projectName');
+
+      // State
+      assertTrue(project.state?.currentPhase?.includes('30') ?? false, 'state current phase');
+      assertEq(project.state?.status, 'in-progress', 'state status');
+
+      // Validation
+      assertEq(project.validation.valid, true, 'validation passes for complete dir');
+      assertEq(project.validation.issues.length, 0, 'no validation issues');
+    } finally {
+      cleanup(base);
+    }
+  }
+
+  // ─── Test 2: Minimal .planning directory (only ROADMAP.md) ─────────────
+  console.log('\n=== Minimal .planning directory (only ROADMAP.md) ===');
+  {
+    const base = createFixtureBase();
+    try {
+      const planning = createPlanningDir(base);
+      writeFileSync(join(planning, 'ROADMAP.md'), SAMPLE_ROADMAP);
+
+      const project = await parsePlanningDirectory(planning);
+
+      assertEq(project.project, null, 'minimal: PROJECT.md is null');
+      assertTrue(project.roadmap !== null, 'minimal: ROADMAP.md parsed');
+      assertEq(project.requirements.length, 0, 'minimal: no requirements');
+      assertEq(project.state, null, 'minimal: no state');
+      assertEq(project.config, null, 'minimal: no config');
+      assertEq(Object.keys(project.phases).length, 0, 'minimal: no phases');
+      assertEq(project.quickTasks.length, 0, 'minimal: no quick tasks');
+      assertEq(project.milestones.length, 0, 'minimal: no milestones');
+      assertEq(project.research.length, 0, 'minimal: no research');
+      assertEq(project.validation.valid, true, 'minimal: validation passes');
+    } finally {
+      cleanup(base);
+    }
+  }
+
+  // ─── Test 3: Missing directory → validation fatal error ────────────────
+  console.log('\n=== Missing directory → validation returns fatal error ===');
+  {
+    const base = createFixtureBase();
+    try {
+      const result = await validatePlanningDirectory(join(base, 'nonexistent'));
+
+      assertEq(result.valid, false, 'missing dir: validation fails');
+      assertTrue(result.issues.length > 0, 'missing dir: has issues');
+      assertTrue(
+        result.issues.some(i => i.severity === 'fatal'),
+        'missing dir: has fatal issue'
+      );
+    } finally {
+      cleanup(base);
+    }
+  }
+
+  // ─── Test 4: Duplicate phase numbers ───────────────────────────────────
+  console.log('\n=== Phase directory with duplicate numbers ===');
+  {
+    const base = createFixtureBase();
+    try {
+      const planning = createPlanningDir(base);
+      writeFileSync(join(planning, 'ROADMAP.md'), SAMPLE_ROADMAP);
+
+      const phasesDir = join(planning, 'phases');
+      mkdirSync(join(phasesDir, '45-core-infrastructure'), { recursive: true });
+      mkdirSync(join(phasesDir, '45-logging-config'), { recursive: true });
+
+      writeFileSync(
+        join(phasesDir, '45-core-infrastructure', '45-01-PLAN.md'),
+        '# Core Plan\n\n<objective>Core infra</objective>'
+      );
+      writeFileSync(
+        join(phasesDir, '45-logging-config', '45-01-PLAN.md'),
+        '# Logging Plan\n\n<objective>Logging config</objective>'
+      );
+
+      const project = await parsePlanningDirectory(planning);
+
+      assertTrue('45-core-infrastructure' in project.phases, 'dup nums: core-infrastructure phase present');
+      assertTrue('45-logging-config' in project.phases, 'dup nums: logging-config phase present');
+      assertEq(project.phases['45-core-infrastructure']?.number, 45, 'dup nums: both have number 45 (a)');
+      assertEq(project.phases['45-logging-config']?.number, 45, 'dup nums: both have number 45 (b)');
+    } finally {
+      cleanup(base);
+    }
+  }
+
+  // ─── Test 5: XML-in-markdown plan parsing ──────────────────────────────
+  console.log('\n=== Plan file with XML-in-markdown ===');
+  {
+    const base = createFixtureBase();
+    try {
+      const planning = createPlanningDir(base);
+      writeFileSync(join(planning, 'ROADMAP.md'), SAMPLE_ROADMAP);
+
+      const phaseDir = join(planning, 'phases', '29-auth-system');
+      mkdirSync(phaseDir, { recursive: true });
+      writeFileSync(join(phaseDir, '29-01-PLAN.md'), SAMPLE_PLAN_XML);
+
+      const project = await parsePlanningDirectory(planning);
+      const plan = project.phases['29-auth-system']?.plans?.['01'];
+
+      assertTrue(plan !== undefined, 'xml plan: plan exists');
+      assertTrue(plan?.objective?.includes('authentication') ?? false, 'xml plan: objective extracted');
+      assertTrue((plan?.tasks?.length ?? 0) === 3, 'xml plan: 3 tasks extracted');
+      assertTrue(plan?.tasks?.[0]?.includes('auth middleware') ?? false, 'xml plan: first task content');
+      assertTrue(plan?.context?.includes('JWT') ?? false, 'xml plan: context extracted');
+      assertTrue(plan?.verification?.includes('Login returns') ?? false, 'xml plan: verification extracted');
+      assertTrue(plan?.successCriteria?.includes('endpoints respond') ?? false, 'xml plan: success criteria extracted');
+    } finally {
+      cleanup(base);
+    }
+  }
+
+  // ─── Test 6: Summary file with YAML frontmatter ───────────────────────
+  console.log('\n=== Summary file with YAML frontmatter ===');
+  {
+    const base = createFixtureBase();
+    try {
+      const planning = createPlanningDir(base);
+      writeFileSync(join(planning, 'ROADMAP.md'), SAMPLE_ROADMAP);
+
+      const phaseDir = join(planning, 'phases', '29-auth-system');
+      mkdirSync(phaseDir, { recursive: true });
+      writeFileSync(join(phaseDir, '29-01-SUMMARY.md'), SAMPLE_SUMMARY);
+
+      const project = await parsePlanningDirectory(planning);
+      const summary = project.phases['29-auth-system']?.summaries?.['01'];
+
+      assertTrue(summary !== undefined, 'summary fm: summary exists');
+      assertEq(summary?.frontmatter?.phase, '29-auth-system', 'summary fm: phase');
+      assertEq(summary?.frontmatter?.plan, '01', 'summary fm: plan');
+      assertEq(summary?.frontmatter?.subsystem, 'auth', 'summary fm: subsystem');
+      assertEq(summary?.frontmatter?.tags, ['authentication', 'security'], 'summary fm: tags');
+      assertEq(summary?.frontmatter?.provides, ['auth-middleware', 'jwt-validation'], 'summary fm: provides');
+      assertEq(summary?.frontmatter?.affects, ['api-routes'], 'summary fm: affects');
+      assertEq(summary?.frontmatter?.['tech-stack'], ['jsonwebtoken', 'express'], 'summary fm: tech-stack');
+      assertEq(summary?.frontmatter?.['key-files'], ['src/auth.ts', 'src/middleware/auth.ts'], 'summary fm: key-files');
+      assertEq(summary?.frontmatter?.['key-decisions'], ['Use RS256 for JWT signing', 'Store refresh tokens in DB'], 'summary fm: key-decisions');
+      assertEq(summary?.frontmatter?.['patterns-established'], ['Middleware-based auth'], 'summary fm: patterns-established');
+      assertEq(summary?.frontmatter?.duration, '2h', 'summary fm: duration');
+      assertEq(summary?.frontmatter?.completed, '2026-01-15', 'summary fm: completed');
+    } finally {
+      cleanup(base);
+    }
+  }
+
+  // ─── Test 7: Orphan summaries (no matching plan) ──────────────────────
+  console.log('\n=== Orphan summaries (no matching plan) ===');
+  {
+    const base = createFixtureBase();
+    try {
+      const planning = createPlanningDir(base);
+      writeFileSync(join(planning, 'ROADMAP.md'), SAMPLE_ROADMAP);
+
+      const phaseDir = join(planning, 'phases', '45-logging-config');
+      mkdirSync(phaseDir, { recursive: true });
+
+      // Summaries without corresponding plans
+      writeFileSync(join(phaseDir, '45-04-SUMMARY.md'), `---
+phase: "45-logging-config"
+plan: "04"
+subsystem: "logging"
+---
+
+# 45-04 Summary
+
+Orphan summary content.
+`);
+      writeFileSync(join(phaseDir, '45-05-SUMMARY.md'), `---
+phase: "45-logging-config"
+plan: "05"
+subsystem: "logging"
+---
+
+# 45-05 Summary
+
+Another orphan.
+`);
+
+      const project = await parsePlanningDirectory(planning);
+      const phase = project.phases['45-logging-config'];
+
+      assertTrue(phase !== undefined, 'orphan: phase exists');
+      assertEq(Object.keys(phase?.plans ?? {}).length, 0, 'orphan: no plans');
+      assertTrue(Object.keys(phase?.summaries ?? {}).length >= 2, 'orphan: summaries preserved');
+      assertTrue('04' in (phase?.summaries ?? {}), 'orphan: summary 04 present');
+      assertTrue('05' in (phase?.summaries ?? {}), 'orphan: summary 05 present');
+    } finally {
+      cleanup(base);
+    }
+  }
+
+  // ─── Test 8: .archive/ directory skipped ──────────────────────────────
+  console.log('\n=== .archive/ directory → skipped by default ===');
+  {
+    const base = createFixtureBase();
+    try {
+      const planning = createPlanningDir(base);
+      writeFileSync(join(planning, 'ROADMAP.md'), SAMPLE_ROADMAP);
+
+      // Normal phase
+      const phaseDir = join(planning, 'phases', '29-auth-system');
+      mkdirSync(phaseDir, { recursive: true });
+      writeFileSync(join(phaseDir, '29-01-PLAN.md'), SAMPLE_PLAN_XML);
+
+      // Archived phase (should be skipped)
+      const archiveDir = join(planning, '.archive', 'v2.5-deploy', '29-old-auth');
+      mkdirSync(archiveDir, { recursive: true });
+      writeFileSync(join(archiveDir, '29-01-PLAN.md'), '# Archived plan');
+
+      const project = await parsePlanningDirectory(planning);
+
+      assertTrue('29-auth-system' in project.phases, 'archive: normal phase present');
+      // Archive phases should not appear in the phases map
+      assertTrue(!Object.keys(project.phases).some(k => k.includes('old-auth')), 'archive: archived phase not present');
+    } finally {
+      cleanup(base);
+    }
+  }
+
+  // ─── Test 9: Quick tasks ──────────────────────────────────────────────
+  console.log('\n=== Quick tasks parsed ===');
+  {
+    const base = createFixtureBase();
+    try {
+      const planning = createPlanningDir(base);
+      writeFileSync(join(planning, 'ROADMAP.md'), SAMPLE_ROADMAP);
+
+      // Quick task 1
+      const qt1 = join(planning, 'quick', '001-fix-login');
+      mkdirSync(qt1, { recursive: true });
+      writeFileSync(join(qt1, '001-PLAN.md'), SAMPLE_QUICK_PLAN);
+      writeFileSync(join(qt1, '001-SUMMARY.md'), SAMPLE_QUICK_SUMMARY);
+
+      // Quick task 2 (plan only, no summary)
+      const qt2 = join(planning, 'quick', '002-update-deps');
+      mkdirSync(qt2, { recursive: true });
+      writeFileSync(join(qt2, '002-PLAN.md'), '# 002: Update Dependencies\n\nUpdate all deps.');
+
+      const project = await parsePlanningDirectory(planning);
+
+      assertEq(project.quickTasks.length, 2, 'quick: 2 quick tasks');
+      assertEq(project.quickTasks[0]?.number, 1, 'quick: first task number');
+      assertEq(project.quickTasks[0]?.slug, 'fix-login', 'quick: first task slug');
+      assertTrue(project.quickTasks[0]?.plan !== null, 'quick: first task has plan');
+      assertTrue(project.quickTasks[0]?.summary !== null, 'quick: first task has summary');
+      assertEq(project.quickTasks[1]?.number, 2, 'quick: second task number');
+      assertTrue(project.quickTasks[1]?.plan !== null, 'quick: second task has plan');
+      assertEq(project.quickTasks[1]?.summary, null, 'quick: second task has no summary');
+    } finally {
+      cleanup(base);
+    }
+  }
+
+  // ─── Test 10: Roadmap with milestone sections and <details> ────────────
+  console.log('\n=== Roadmap with milestone sections and <details> blocks ===');
+  {
+    const base = createFixtureBase();
+    try {
+      const planning = createPlanningDir(base);
+      writeFileSync(join(planning, 'ROADMAP.md'), SAMPLE_MILESTONE_SECTIONED_ROADMAP);
+
+      const project = await parsePlanningDirectory(planning);
+
+      assertTrue(project.roadmap !== null, 'ms roadmap: roadmap parsed');
+      assertTrue((project.roadmap?.milestones?.length ?? 0) >= 2, 'ms roadmap: has milestone sections');
+
+      // Check collapsed milestone
+      const v20 = project.roadmap?.milestones?.find(m => m.id.includes('2.0'));
+      assertTrue(v20 !== undefined, 'ms roadmap: v2.0 milestone found');
+      assertEq(v20?.collapsed, true, 'ms roadmap: v2.0 is collapsed');
+      assertTrue((v20?.phases?.length ?? 0) >= 2, 'ms roadmap: v2.0 has phases');
+      assertTrue(v20?.phases?.every(p => p.done) ?? false, 'ms roadmap: v2.0 phases all done');
+
+      // Check active milestone
+      const v25 = project.roadmap?.milestones?.find(m => m.id.includes('2.5'));
+      assertTrue(v25 !== undefined, 'ms roadmap: v2.5 milestone found');
+      assertEq(v25?.collapsed, false, 'ms roadmap: v2.5 is not collapsed');
+      assertTrue((v25?.phases?.length ?? 0) >= 3, 'ms roadmap: v2.5 has phases');
+
+      // Check completion state
+      const phase29 = v25?.phases?.find(p => p.number === 29);
+      assertTrue(phase29?.done === true, 'ms roadmap: phase 29 is done');
+      const phase30 = v25?.phases?.find(p => p.number === 30);
+      assertTrue(phase30?.done === false, 'ms roadmap: phase 30 is not done');
+    } finally {
+      cleanup(base);
+    }
+  }
+
+  // ─── Test 11: Non-standard phase files → extra files ──────────────────
+  console.log('\n=== Non-standard phase files → collected as extra files ===');
+  {
+    const base = createFixtureBase();
+    try {
+      const planning = createPlanningDir(base);
+      writeFileSync(join(planning, 'ROADMAP.md'), SAMPLE_ROADMAP);
+
+      const phaseDir = join(planning, 'phases', '36-attachment-system');
+      mkdirSync(phaseDir, { recursive: true });
+      writeFileSync(join(phaseDir, '36-01-PLAN.md'), '<objective>Attachments</objective>');
+      writeFileSync(join(phaseDir, 'BASELINE.md'), '# Baseline\n\nBaseline measurements.');
+      writeFileSync(join(phaseDir, 'BUNDLE-ANALYSIS.md'), '# Bundle Analysis\n\nResults.');
+      writeFileSync(join(phaseDir, 'depcheck-results.txt'), 'unused: pkg-a, pkg-b');
+
+      const project = await parsePlanningDirectory(planning);
+      const phase = project.phases['36-attachment-system'];
+
+      assertTrue(phase !== undefined, 'extra: phase exists');
+      assertTrue((phase?.extraFiles?.length ?? 0) >= 3, 'extra: non-standard files collected');
+      assertTrue(
+        phase?.extraFiles?.some(f => f.fileName === 'BASELINE.md') ?? false,
+        'extra: BASELINE.md collected'
+      );
+      assertTrue(
+        phase?.extraFiles?.some(f => f.fileName === 'BUNDLE-ANALYSIS.md') ?? false,
+        'extra: BUNDLE-ANALYSIS.md collected'
+      );
+      assertTrue(
+        phase?.extraFiles?.some(f => f.fileName === 'depcheck-results.txt') ?? false,
+        'extra: depcheck-results.txt collected'
+      );
+    } finally {
+      cleanup(base);
+    }
+  }
+
+  // ─── Test 12: Validation — missing ROADMAP.md → warning (not fatal) ───
+  console.log('\n=== Validation: missing ROADMAP.md → warning (not fatal) ===');
+  {
+    const base = createFixtureBase();
+    try {
+      const planning = createPlanningDir(base);
+      // Only PROJECT.md, no ROADMAP.md
+      writeFileSync(join(planning, 'PROJECT.md'), SAMPLE_PROJECT);
+
+      const result = await validatePlanningDirectory(planning);
+
+      assertEq(result.valid, true, 'no roadmap: validation still passes');
+      assertTrue(
+        result.issues.some(i => i.severity === 'warning' && i.file.includes('ROADMAP')),
+        'no roadmap: warning issue mentions ROADMAP'
+      );
+    } finally {
+      cleanup(base);
+    }
+  }
+
+  // ─── Test 13: Validation — missing PROJECT.md → warning ───────────────
+  console.log('\n=== Validation: missing PROJECT.md → warning ===');
+  {
+    const base = createFixtureBase();
+    try {
+      const planning = createPlanningDir(base);
+      writeFileSync(join(planning, 'ROADMAP.md'), SAMPLE_ROADMAP);
+      // No PROJECT.md
+
+      const result = await validatePlanningDirectory(planning);
+
+      assertEq(result.valid, true, 'no project: validation passes (warning only)');
+      assertTrue(
+        result.issues.some(i => i.severity === 'warning' && i.file.includes('PROJECT')),
+        'no project: warning issue mentions PROJECT'
+      );
+    } finally {
+      cleanup(base);
+    }
+  }
+
+  report();
+}
+
+main().catch((error) => {
+  console.error(error);
+  process.exit(1);
+});