gsd-pi 2.33.1 → 2.34.0-dev.bbb5216

package/dist/resources/extensions/gsd/tests/requirements.test.ts

@@ -1,106 +0,0 @@
-import { parseRequirementCounts } from "../files.ts";
-import { mkdtempSync, mkdirSync, rmSync, writeFileSync } from "node:fs";
-import { join } from "node:path";
-import { tmpdir } from "node:os";
-import { deriveState } from "../state.ts";
-import { runGSDDoctor } from "../doctor.ts";
-import { createTestContext } from './test-helpers.ts';
-
-const { assertEq, assertTrue, report } = createTestContext();
-console.log("\n=== requirement counts parser ===");
-{
-  const counts = parseRequirementCounts(`# Requirements
-
-## Active
-
-### R001 — Foo
-- Status: active
-
-### R002 — Bar
-- Status: blocked
-
-## Validated
-
-### R010 — Baz
-- Status: validated
-
-## Deferred
-
-### R020 — Qux
-- Status: deferred
-
-## Out of Scope
-
-### R030 — No
-- Status: out-of-scope
-`);
-  assertEq(counts.active, 2, "counts active requirements by section");
-  assertEq(counts.validated, 1, "counts validated requirements");
-  assertEq(counts.deferred, 1, "counts deferred requirements");
-  assertEq(counts.outOfScope, 1, "counts out of scope requirements");
-  assertEq(counts.blocked, 1, "counts blocked statuses");
-}
-
-const base = mkdtempSync(join(tmpdir(), "gsd-requirements-test-"));
-const gsd = join(base, ".gsd");
-const mDir = join(gsd, "milestones", "M001");
-const sDir = join(mDir, "slices", "S01");
-const tDir = join(sDir, "tasks");
-mkdirSync(tDir, { recursive: true });
-writeFileSync(join(gsd, "REQUIREMENTS.md"), `# Requirements
-
-## Active
-
-### R001 — Missing owner
-- Class: core-capability
-- Status: active
-- Description: thing
-- Why it matters: thing
-- Source: user
-- Primary owning slice: none yet
-- Supporting slices: none
-- Validation: unmapped
-- Notes: none
-
-## Validated
-
-## Deferred
-
-## Out of Scope
-
-## Traceability
-`, "utf-8");
-writeFileSync(join(mDir, "M001-ROADMAP.md"), `# M001: Demo
-
-## Slices
-- [ ] **S01: Demo Slice** \`risk:low\` \`depends:[]\`
-  > After this: demo works
-`, "utf-8");
-writeFileSync(join(sDir, "S01-PLAN.md"), `# S01: Demo Slice
-
-**Goal:** Demo
-**Demo:** Demo
-
-## Must-Haves
-- done
-
-## Tasks
-- [ ] **T01: Implement thing** \`est:10m\`
-  Task is in progress.
-`, "utf-8");
-
-console.log("\n=== deriveState includes requirements counts ===");
-{
-  const state = await deriveState(base);
-  assertTrue(state.requirements !== undefined, "state includes requirements summary");
-  assertEq(state.requirements?.active, 1, "state reports active requirement count");
-}
-
-console.log("\n=== doctor flags orphaned active requirement ===");
-{
-  const report = await runGSDDoctor(base);
-  assertTrue(report.issues.some(issue => issue.code === "active_requirement_missing_owner"), "doctor flags missing owner");
-}
-
-rmSync(base, { recursive: true, force: true });
-report();

package/dist/resources/extensions/gsd/tests/roadmap-parse-regression.test.ts

@@ -1,358 +0,0 @@
-/**
- * roadmap-parse-regression.test.ts — Regression tests for roadmap parsing.
- *
- * Exercises parseRoadmapSlices() and the prose fallback parser against
- * every known LLM-generated roadmap variant that has caused production bugs.
- *
- * Regression coverage for:
- *   #807   Prose slice headers not parsed → "No slice eligible" block
- *   #1248  Prose header regex only matched H2 with colon separator
- *   #1243  Same root cause as #1248
- *
- * Also covers dependency expansion (range syntax) and edge cases.
- */
-
-import { parseRoadmapSlices, expandDependencies } from '../roadmap-slices.ts';
-import { createTestContext } from './test-helpers.ts';
-
-const { assertEq, assertTrue, report } = createTestContext();
-
-async function main(): Promise<void> {
-
-  // ═══════════════════════════════════════════════════════════════════════
-  // A. Standard machine-readable format (should always work)
-  // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== A. Standard checkbox format ===');
-
-  {
-    const content = [
-      '# M001: Test Project',
-      '',
-      '## Slices',
-      '',
-      '- [ ] **S01: First Slice** `risk:low` `depends:[]`',
-      '- [ ] **S02: Second Slice** `risk:medium` `depends:[S01]`',
-      '- [x] **S03: Third Slice** `risk:high` `depends:[S01,S02]`',
-      '',
-      '## Boundary Map',
-      '',
-    ].join('\n');
-
-    const slices = parseRoadmapSlices(content);
-    assertEq(slices.length, 3, 'standard format: 3 slices');
-    assertEq(slices[0].id, 'S01', 'S01 id');
-    assertEq(slices[0].title, 'First Slice', 'S01 title');
-    assertEq(slices[0].done, false, 'S01 not done');
-    assertEq(slices[0].risk, 'low', 'S01 risk');
-    assertEq(slices[0].depends.length, 0, 'S01 no deps');
-
-    assertEq(slices[1].id, 'S02', 'S02 id');
-    assertEq(slices[1].depends.length, 1, 'S02 has 1 dep');
-    assertEq(slices[1].depends[0], 'S01', 'S02 depends on S01');
-
-    assertEq(slices[2].id, 'S03', 'S03 id');
-    assertEq(slices[2].done, true, 'S03 is done');
-    assertEq(slices[2].risk, 'high', 'S03 risk');
-    assertEq(slices[2].depends.length, 2, 'S03 has 2 deps');
-  }
-
-  // ═══════════════════════════════════════════════════════════════════════
-  // B. Prose fallback: H2 with colon (the only format the old regex matched)
-  // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== B. Prose fallback: H2 with colon ===');
-
-  {
-    const content = [
-      '# M001: Test',
-      '',
-      '## S01: Setup Foundation',
-      '',
-      'Do the setup work.',
-      '',
-      '## S02: Core Features',
-      '',
-      'Build the features.',
-      '',
-    ].join('\n');
-
-    const slices = parseRoadmapSlices(content);
-    assertEq(slices.length, 2, 'prose H2 colon: 2 slices');
-    assertEq(slices[0].id, 'S01', 'S01 id');
-    assertEq(slices[0].title, 'Setup Foundation', 'S01 title');
-    assertEq(slices[1].id, 'S02', 'S02 id');
-    assertEq(slices[1].title, 'Core Features', 'S02 title');
-  }
-
-  // ═══════════════════════════════════════════════════════════════════════
-  // C. Regression #1248: H3 headers (the old regex only matched ##)
-  // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== C. #1248: H3 headers ===');
-
-  {
-    const content = [
-      '# M001: Test',
-      '',
-      '### S01: Setup Foundation',
-      '',
-      'Do the setup work.',
-      '',
-      '### S02: Core Features',
-      '',
-      'Build the features.',
-      '',
-    ].join('\n');
-
-    const slices = parseRoadmapSlices(content);
-    assertEq(slices.length, 2, '#1248 H3: 2 slices parsed');
-    assertEq(slices[0].id, 'S01', 'S01 from H3');
-    assertEq(slices[1].id, 'S02', 'S02 from H3');
-  }
-
-  // ═══════════════════════════════════════════════════════════════════════
-  // D. Regression #1248: H4 headers
-  // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== D. #1248: H4 headers ===');
-
-  {
-    const content = [
-      '# M001: Test',
-      '',
-      '#### S01: Setup Foundation',
-      '',
-      '#### S02: Core Features',
-      '',
-    ].join('\n');
-
-    const slices = parseRoadmapSlices(content);
-    assertEq(slices.length, 2, '#1248 H4: 2 slices parsed');
-  }
-
-  // ═══════════════════════════════════════════════════════════════════════
-  // E. Regression #1248: H1 header (unusual but LLMs produce it)
-  // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== E. #1248: H1 headers ===');
-
-  {
-    const content = [
-      '# S01: Setup Foundation',
-      '',
-      'Setup stuff.',
-      '',
-      '# S02: Core Features',
-      '',
-      'Build stuff.',
-      '',
-    ].join('\n');
-
-    const slices = parseRoadmapSlices(content);
-    assertEq(slices.length, 2, '#1248 H1: 2 slices parsed');
-  }
-
-  // ═══════════════════════════════════════════════════════════════════════
-  // F. Regression #1248: Bold-wrapped IDs
-  // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== F. #1248: Bold-wrapped ===');
-
-  {
-    const content1 = '## **S01: Setup Foundation**\n\nDo stuff.\n\n## **S02: Features**\n\nMore stuff.\n';
-    const slices1 = parseRoadmapSlices(content1);
-    assertEq(slices1.length, 2, 'bold-wrapped: 2 slices');
-    assertEq(slices1[0].title, 'Setup Foundation', 'bold-wrapped: title extracted without bold');
-
-    const content2 = '## **S01**: Setup Foundation\n\n## **S02**: Features\n';
-    const slices2 = parseRoadmapSlices(content2);
-    assertEq(slices2.length, 2, 'bold ID only: 2 slices');
-  }
-
-  // ═══════════════════════════════════════════════════════════════════════
-  // G. Regression #1248: Dot separator
-  // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== G. #1248: Dot separator ===');
-
-  {
-    const content = '## S01. Setup Foundation\n\n## S02. Core Features\n';
-    const slices = parseRoadmapSlices(content);
-    assertEq(slices.length, 2, 'dot separator: 2 slices');
-    assertEq(slices[0].title, 'Setup Foundation', 'dot separator: title');
-  }
-
-  // ═══════════════════════════════════════════════════════════════════════
-  // H. Regression #1248: Em dash separator
-  // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== H. #1248: Em/en dash separators ===');
-
-  {
-    const content = '## S01 — Setup Foundation\n\n## S02 – Core Features\n';
-    const slices = parseRoadmapSlices(content);
-    assertEq(slices.length, 2, 'em/en dash: 2 slices');
-  }
-
-  // ═══════════════════════════════════════════════════════════════════════
-  // I. Regression #1248: Space-only separator (no punctuation)
-  // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== I. #1248: Space-only separator ===');
-
-  {
-    const content = '## S01 Setup Foundation\n\n## S02 Core Features\n';
-    const slices = parseRoadmapSlices(content);
-    assertEq(slices.length, 2, 'space-only: 2 slices');
-    assertEq(slices[0].title, 'Setup Foundation', 'space-only: title');
-  }
-
-  // ═══════════════════════════════════════════════════════════════════════
-  // J. Regression #1248: Non-zero-padded IDs
-  // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== J. #1248: Non-zero-padded IDs ===');
-
-  {
-    const content = '## S1: Setup\n\n## S2: Features\n';
-    const slices = parseRoadmapSlices(content);
-    assertEq(slices.length, 2, 'non-padded: 2 slices');
-    assertEq(slices[0].id, 'S1', 'non-padded: S1');
-  }
-
-  // ═══════════════════════════════════════════════════════════════════════
-  // K. Regression #1248: "Slice" prefix
-  // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== K. #1248: "Slice" prefix ===');
-
-  {
-    const content = '## Slice S01: Setup Foundation\n\n## Slice S02: Core Features\n';
-    const slices = parseRoadmapSlices(content);
-    assertEq(slices.length, 2, 'Slice prefix: 2 slices');
-    assertEq(slices[0].id, 'S01', 'Slice prefix: S01');
-  }
-
-  // ═══════════════════════════════════════════════════════════════════════
-  // L. Prose with "Depends on:" line
-  // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== L. Prose with Depends on: ===');
-
-  {
-    const content = [
-      '## S01: Foundation',
-      '',
-      'Build the base.',
-      '',
-      '## S02: Features',
-      '',
-      '**Depends on:** S01',
-      '',
-      'Build features.',
-    ].join('\n');
-
-    const slices = parseRoadmapSlices(content);
-    assertEq(slices.length, 2, 'prose deps: 2 slices');
-    assertEq(slices[1].depends.length, 1, 'S02 has 1 dep');
-    assertEq(slices[1].depends[0], 'S01', 'S02 depends on S01');
-  }
-
-  // ═══════════════════════════════════════════════════════════════════════
-  // M. Empty / edge cases
-  // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== M. Edge cases ===');
-
-  {
-    assertEq(parseRoadmapSlices('').length, 0, 'empty content → 0 slices');
-    assertEq(parseRoadmapSlices('# Just a title\n\nSome text.').length, 0, 'no slices at all → 0');
-
-    // Mixed format: ## Slices section with one checkbox + prose below
-    const mixed = [
-      '## Slices',
-      '',
-      '- [ ] **S01: Foundation** `risk:low` `depends:[]`',
-      '',
-      '## S02: Features',
-      '',
-      'Prose content.',
-    ].join('\n');
-    const mixedSlices = parseRoadmapSlices(mixed);
-    // The ## Slices section takes priority — prose headers outside it aren't picked up
-    assertEq(mixedSlices.length, 1, 'mixed: only 1 slice from ## Slices section');
-    assertEq(mixedSlices[0].id, 'S01', 'mixed: S01 from checkbox');
-  }
-
-  // ═══════════════════════════════════════════════════════════════════════
-  // N. Dependency range expansion
-  // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== N. Dependency range expansion ===');
-
-  {
-    assertEq(
-      expandDependencies(['S01-S04']),
-      ['S01', 'S02', 'S03', 'S04'],
-      'S01-S04 → 4 individual deps',
-    );
-
-    assertEq(
-      expandDependencies(['S01..S03']),
-      ['S01', 'S02', 'S03'],
-      'S01..S03 → 3 individual deps',
-    );
-
-    assertEq(
-      expandDependencies(['S01']),
-      ['S01'],
-      'single dep passes through',
-    );
-
-    assertEq(
-      expandDependencies(['S01', 'S03-S05']),
-      ['S01', 'S03', 'S04', 'S05'],
-      'mixed single + range',
-    );
-
-    assertEq(
-      expandDependencies(['']),
-      [],
-      'empty string filtered out',
-    );
-  }
-
-  // ═══════════════════════════════════════════════════════════════════════
-  // O. No-separator colon-less: "S01:Title" (no space after colon)
-  // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== O. No space after colon ===');
-
-  {
-    const content = '## S01:Foundation\n\n## S02:Features\n';
-    const slices = parseRoadmapSlices(content);
-    // The regex uses [:\s.—–-]* which allows colon with no space
-    assertEq(slices.length, 2, 'no-space-colon: 2 slices');
-  }
-
-  // ═══════════════════════════════════════════════════════════════════════
-  // P. Three-digit padded IDs
-  // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== P. Three-digit padded IDs ===');
-
-  {
-    const content = '## S001: Foundation\n\n## S002: Features\n';
-    const slices = parseRoadmapSlices(content);
-    assertEq(slices.length, 2, 'three-digit: 2 slices');
-    assertEq(slices[0].id, 'S001', 'three-digit: S001');
-  }
-
-  report();
-}
-
-main().catch((error) => {
-  console.error(error);
-  process.exit(1);
-});

package/dist/resources/extensions/gsd/tests/roadmap-slices.test.ts

@@ -1,66 +0,0 @@
-import test from "node:test";
-import assert from "node:assert/strict";
-import { parseRoadmap } from "../files.ts";
-import { parseRoadmapSlices, expandDependencies } from "../roadmap-slices.ts";
-
-const content = `# M003: Current
-
-**Vision:** Build the thing.
-
-## Slices
-- [x] **S01: First Slice** \`risk:low\` \`depends:[]\`
-  > After this: First demo works.
-- [ ] **S02: Second Slice** \`risk:medium\` \`depends:[S01]\`
-- [x] **S03: Third Slice** \`depends:[S01, S02]\`
-  > After this: Third demo works.
-
-## Boundary Map
-### S01 → S02
-Produces:
-  foo.ts
-`;
-
-test("parseRoadmapSlices extracts slices with dependencies and risk", () => {
-  const slices = parseRoadmapSlices(content);
-  assert.equal(slices.length, 3);
-  assert.equal(slices[0]?.id, "S01");
-  assert.equal(slices[0]?.done, true);
-  assert.equal(slices[0]?.demo, "First demo works.");
-  assert.deepEqual(slices[1]?.depends, ["S01"]);
-  assert.equal(slices[1]?.risk, "medium");
-  assert.equal(slices[2]?.risk, "low");
-  assert.deepEqual(slices[2]?.depends, ["S01", "S02"]);
-});
-
-test("parseRoadmap integration: uses extracted slice parser", () => {
-  const roadmap = parseRoadmap(content);
-  assert.equal(roadmap.title, "M003: Current");
-  assert.equal(roadmap.vision, "Build the thing.");
-  assert.equal(roadmap.slices.length, 3);
-  assert.equal(roadmap.boundaryMap.length, 1);
-});
-
-test("expandDependencies: plain IDs, ranges, and edge cases", () => {
-  assert.deepEqual(expandDependencies([]), []);
-  assert.deepEqual(expandDependencies(["S01"]), ["S01"]);
-  assert.deepEqual(expandDependencies(["S01", "S03"]), ["S01", "S03"]);
-  assert.deepEqual(expandDependencies(["S01-S04"]), ["S01", "S02", "S03", "S04"]);
-  assert.deepEqual(expandDependencies(["S01-S01"]), ["S01"]);
-  assert.deepEqual(expandDependencies(["S01..S03"]), ["S01", "S02", "S03"]);
-  assert.deepEqual(expandDependencies(["S01-S03", "S05"]), ["S01", "S02", "S03", "S05"]);
-  assert.deepEqual(expandDependencies(["S04-S01"]), ["S04-S01"]);
-  assert.deepEqual(expandDependencies(["S01-T04"]), ["S01-T04"]);
-});
-
-test("parseRoadmapSlices: range syntax in depends expanded", () => {
-  const rangeContent = `# M016: Test\n\n## Slices\n- [x] **S01: A** \`risk:low\` \`depends:[]\`\n- [x] **S02: B** \`risk:low\` \`depends:[]\`\n- [x] **S03: C** \`risk:low\` \`depends:[]\`\n- [x] **S04: D** \`risk:low\` \`depends:[]\`\n- [ ] **S05: E** \`risk:low\` \`depends:[S01-S04]\`\n  > After this: all done\n`;
-  const slices = parseRoadmapSlices(rangeContent);
-  assert.equal(slices.length, 5);
-  assert.deepEqual(slices[4]?.depends, ["S01", "S02", "S03", "S04"]);
-});
-
-test("parseRoadmapSlices: comma-separated depends still works", () => {
-  const commaContent = `# M001: Test\n\n## Slices\n- [ ] **S05: E** \`risk:low\` \`depends:[S01,S02,S03,S04]\`\n  > After this: done\n`;
-  const slices = parseRoadmapSlices(commaContent);
-  assert.deepEqual(slices[0]?.depends, ["S01", "S02", "S03", "S04"]);
-});