gsd-pi 2.43.0 → 2.44.0-dev.0b97ffd

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

@@ -12,20 +12,16 @@
  * Also covers dependency expansion (range syntax) and edge cases.
  */
 
+import { describe, test } from 'node:test';
+import assert from 'node:assert/strict';
 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 ===');
 
-  {
+describe('roadmap-parse-regression', () => {
+test('A. Standard checkbox format', () => {
     const content = [
       '# M001: Test Project',
       '',
@@ -40,30 +36,27 @@ async function main(): Promise<void> {
     ].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');
-  }
+    assert.deepStrictEqual(slices.length, 3, 'standard format: 3 slices');
+    assert.deepStrictEqual(slices[0].id, 'S01', 'S01 id');
+    assert.deepStrictEqual(slices[0].title, 'First Slice', 'S01 title');
+    assert.deepStrictEqual(slices[0].done, false, 'S01 not done');
+    assert.deepStrictEqual(slices[0].risk, 'low', 'S01 risk');
+    assert.deepStrictEqual(slices[0].depends.length, 0, 'S01 no deps');
+
+    assert.deepStrictEqual(slices[1].id, 'S02', 'S02 id');
+    assert.deepStrictEqual(slices[1].depends.length, 1, 'S02 has 1 dep');
+    assert.deepStrictEqual(slices[1].depends[0], 'S01', 'S02 depends on S01');
+
+    assert.deepStrictEqual(slices[2].id, 'S03', 'S03 id');
+    assert.deepStrictEqual(slices[2].done, true, 'S03 is done');
+    assert.deepStrictEqual(slices[2].risk, 'high', 'S03 risk');
+    assert.deepStrictEqual(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 ===');
-
-  {
+test('B. Prose fallback: H2 with colon', () => {
     const content = [
       '# M001: Test',
       '',
@@ -78,20 +71,17 @@ async function main(): Promise<void> {
     ].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');
-  }
+    assert.deepStrictEqual(slices.length, 2, 'prose H2 colon: 2 slices');
+    assert.deepStrictEqual(slices[0].id, 'S01', 'S01 id');
+    assert.deepStrictEqual(slices[0].title, 'Setup Foundation', 'S01 title');
+    assert.deepStrictEqual(slices[1].id, 'S02', 'S02 id');
+    assert.deepStrictEqual(slices[1].title, 'Core Features', 'S02 title');
+});
 
   // ═══════════════════════════════════════════════════════════════════════
   // C. Regression #1248: H3 headers (the old regex only matched ##)
   // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== C. #1248: H3 headers ===');
-
-  {
+test('C. #1248: H3 headers', () => {
     const content = [
       '# M001: Test',
       '',
@@ -106,18 +96,15 @@ async function main(): Promise<void> {
     ].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');
-  }
+    assert.deepStrictEqual(slices.length, 2, '#1248 H3: 2 slices parsed');
+    assert.deepStrictEqual(slices[0].id, 'S01', 'S01 from H3');
+    assert.deepStrictEqual(slices[1].id, 'S02', 'S02 from H3');
+});
 
   // ═══════════════════════════════════════════════════════════════════════
   // D. Regression #1248: H4 headers
   // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== D. #1248: H4 headers ===');
-
-  {
+test('D. #1248: H4 headers', () => {
     const content = [
       '# M001: Test',
       '',
@@ -128,16 +115,13 @@ async function main(): Promise<void> {
     ].join('\n');
 
     const slices = parseRoadmapSlices(content);
-    assertEq(slices.length, 2, '#1248 H4: 2 slices parsed');
-  }
+    assert.deepStrictEqual(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 ===');
-
-  {
+test('E. #1248: H1 headers', () => {
     const content = [
       '# S01: Setup Foundation',
       '',
@@ -150,97 +134,76 @@ async function main(): Promise<void> {
     ].join('\n');
 
     const slices = parseRoadmapSlices(content);
-    assertEq(slices.length, 2, '#1248 H1: 2 slices parsed');
-  }
+    assert.deepStrictEqual(slices.length, 2, '#1248 H1: 2 slices parsed');
+});
 
   // ═══════════════════════════════════════════════════════════════════════
   // F. Regression #1248: Bold-wrapped IDs
   // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== F. #1248: Bold-wrapped ===');
-
-  {
+test('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');
+    assert.deepStrictEqual(slices1.length, 2, 'bold-wrapped: 2 slices');
+    assert.deepStrictEqual(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');
-  }
+    assert.deepStrictEqual(slices2.length, 2, 'bold ID only: 2 slices');
+});
 
   // ═══════════════════════════════════════════════════════════════════════
   // G. Regression #1248: Dot separator
   // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== G. #1248: Dot separator ===');
-
-  {
+test('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');
-  }
+    assert.deepStrictEqual(slices.length, 2, 'dot separator: 2 slices');
+    assert.deepStrictEqual(slices[0].title, 'Setup Foundation', 'dot separator: title');
+});
 
   // ═══════════════════════════════════════════════════════════════════════
   // H. Regression #1248: Em dash separator
   // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== H. #1248: Em/en dash separators ===');
-
-  {
+test('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');
-  }
+    assert.deepStrictEqual(slices.length, 2, 'em/en dash: 2 slices');
+});
 
   // ═══════════════════════════════════════════════════════════════════════
   // I. Regression #1248: Space-only separator (no punctuation)
   // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== I. #1248: Space-only separator ===');
-
-  {
+test('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');
-  }
+    assert.deepStrictEqual(slices.length, 2, 'space-only: 2 slices');
+    assert.deepStrictEqual(slices[0].title, 'Setup Foundation', 'space-only: title');
+});
 
   // ═══════════════════════════════════════════════════════════════════════
   // J. Regression #1248: Non-zero-padded IDs
   // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== J. #1248: Non-zero-padded IDs ===');
-
-  {
+test('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');
-  }
+    assert.deepStrictEqual(slices.length, 2, 'non-padded: 2 slices');
+    assert.deepStrictEqual(slices[0].id, 'S1', 'non-padded: S1');
+});
 
   // ═══════════════════════════════════════════════════════════════════════
   // K. Regression #1248: "Slice" prefix
   // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== K. #1248: "Slice" prefix ===');
-
-  {
+test('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');
-  }
+    assert.deepStrictEqual(slices.length, 2, 'Slice prefix: 2 slices');
+    assert.deepStrictEqual(slices[0].id, 'S01', 'Slice prefix: S01');
+});
 
   // ═══════════════════════════════════════════════════════════════════════
   // L. Prose with "Depends on:" line
   // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== L. Prose with Depends on: ===');
-
-  {
+test('L. Prose with Depends on:', () => {
     const content = [
       '## S01: Foundation',
       '',
@@ -254,20 +217,17 @@ async function main(): Promise<void> {
     ].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');
-  }
+    assert.deepStrictEqual(slices.length, 2, 'prose deps: 2 slices');
+    assert.deepStrictEqual(slices[1].depends.length, 1, 'S02 has 1 dep');
+    assert.deepStrictEqual(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');
+test('M. Edge cases', () => {
+    assert.deepStrictEqual(parseRoadmapSlices('').length, 0, 'empty content → 0 slices');
+    assert.deepStrictEqual(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 = [
@@ -281,81 +241,69 @@ async function main(): Promise<void> {
     ].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');
-  }
+    assert.deepStrictEqual(mixedSlices.length, 1, 'mixed: only 1 slice from ## Slices section');
+    assert.deepStrictEqual(mixedSlices[0].id, 'S01', 'mixed: S01 from checkbox');
+});
 
   // ═══════════════════════════════════════════════════════════════════════
   // N. Dependency range expansion
   // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== N. Dependency range expansion ===');
-
-  {
-    assertEq(
+test('N. Dependency range expansion', () => {
+    assert.deepStrictEqual(
       expandDependencies(['S01-S04']),
       ['S01', 'S02', 'S03', 'S04'],
       'S01-S04 → 4 individual deps',
     );
 
-    assertEq(
+    assert.deepStrictEqual(
       expandDependencies(['S01..S03']),
       ['S01', 'S02', 'S03'],
       'S01..S03 → 3 individual deps',
     );
 
-    assertEq(
+    assert.deepStrictEqual(
       expandDependencies(['S01']),
       ['S01'],
       'single dep passes through',
     );
 
-    assertEq(
+    assert.deepStrictEqual(
       expandDependencies(['S01', 'S03-S05']),
       ['S01', 'S03', 'S04', 'S05'],
       'mixed single + range',
     );
 
-    assertEq(
+    assert.deepStrictEqual(
       expandDependencies(['']),
       [],
       'empty string filtered out',
     );
-  }
+});
 
   // ═══════════════════════════════════════════════════════════════════════
   // O. No-separator colon-less: "S01:Title" (no space after colon)
   // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== O. No space after colon ===');
-
-  {
+test('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');
-  }
+    assert.deepStrictEqual(slices.length, 2, 'no-space-colon: 2 slices');
+});
 
   // ═══════════════════════════════════════════════════════════════════════
   // P. Three-digit padded IDs
   // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== P. Three-digit padded IDs ===');
-
-  {
+test('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');
-  }
+    assert.deepStrictEqual(slices.length, 2, 'three-digit: 2 slices');
+    assert.deepStrictEqual(slices[0].id, 'S001', 'three-digit: S001');
+});
 
   // ═══════════════════════════════════════════════════════════════════════
   // Q. Regression #1736: Table format under ## Slices
   // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== Q. #1736: Table format under ## Slices ===');
-
-  {
+test('Q. #1736: Table format under ## Slices', () => {
     const content = [
       '# M001: Test',
       '',
@@ -371,22 +319,19 @@ async function main(): Promise<void> {
     ].join('\n');
 
     const slices = parseRoadmapSlices(content);
-    assertEq(slices.length, 3, '#1736 table: 3 slices');
-    assertEq(slices[0].id, 'S01', '#1736 table: S01 id');
-    assertEq(slices[0].title, 'Setup Foundation', '#1736 table: S01 title');
-    assertEq(slices[0].done, true, '#1736 table: S01 done');
-    assertEq(slices[0].risk, 'low', '#1736 table: S01 risk');
-    assertEq(slices[1].done, false, '#1736 table: S02 not done');
-    assertEq(slices[2].done, true, '#1736 table: S03 done');
-  }
+    assert.deepStrictEqual(slices.length, 3, '#1736 table: 3 slices');
+    assert.deepStrictEqual(slices[0].id, 'S01', '#1736 table: S01 id');
+    assert.deepStrictEqual(slices[0].title, 'Setup Foundation', '#1736 table: S01 title');
+    assert.deepStrictEqual(slices[0].done, true, '#1736 table: S01 done');
+    assert.deepStrictEqual(slices[0].risk, 'low', '#1736 table: S01 risk');
+    assert.deepStrictEqual(slices[1].done, false, '#1736 table: S02 not done');
+    assert.deepStrictEqual(slices[2].done, true, '#1736 table: S03 done');
+});
 
   // ═══════════════════════════════════════════════════════════════════════
   // R. Regression #1736: Table format under ## Slice Overview
   // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== R. #1736: Table format under ## Slice Overview ===');
-
-  {
+test('R. #1736: Table format under ## Slice Overview', () => {
     const content = [
       '# M002: Overview Heading',
       '',
@@ -400,18 +345,15 @@ async function main(): Promise<void> {
     ].join('\n');
 
     const slices = parseRoadmapSlices(content);
-    assertEq(slices.length, 2, '#1736 overview: 2 slices');
-    assertEq(slices[0].done, true, '#1736 overview: S01 done');
-    assertEq(slices[1].done, false, '#1736 overview: S02 not done');
-  }
+    assert.deepStrictEqual(slices.length, 2, '#1736 overview: 2 slices');
+    assert.deepStrictEqual(slices[0].done, true, '#1736 overview: S01 done');
+    assert.deepStrictEqual(slices[1].done, false, '#1736 overview: S02 not done');
+});
 
   // ═══════════════════════════════════════════════════════════════════════
   // S. Regression #1736: Table with Done/Complete text status
   // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== S. #1736: Table with text status ===');
-
-  {
+test('S. #1736: Table with text status', () => {
     const content = [
       '# M003: Status Text',
       '',
@@ -426,19 +368,16 @@ async function main(): Promise<void> {
     ].join('\n');
 
     const slices = parseRoadmapSlices(content);
-    assertEq(slices.length, 3, '#1736 text status: 3 slices');
-    assertTrue(slices[0].done, '#1736 text status: Done = true');
-    assertTrue(!slices[1].done, '#1736 text status: Pending = false');
-    assertTrue(slices[2].done, '#1736 text status: Completed = true');
-  }
+    assert.deepStrictEqual(slices.length, 3, '#1736 text status: 3 slices');
+    assert.ok(slices[0].done, '#1736 text status: Done = true');
+    assert.ok(!slices[1].done, '#1736 text status: Pending = false');
+    assert.ok(slices[2].done, '#1736 text status: Completed = true');
+});
 
   // ═══════════════════════════════════════════════════════════════════════
   // T. Regression #1736: Checkbox format still works after table support
   // ═══════════════════════════════════════════════════════════════════════
-
-  console.log('\n=== T. #1736: Checkbox format unchanged ===');
-
-  {
+test('T. #1736: Checkbox format unchanged', () => {
     const content = [
       '# M005: Unchanged',
       '',
@@ -451,16 +390,10 @@ async function main(): Promise<void> {
     ].join('\n');
 
     const slices = parseRoadmapSlices(content);
-    assertEq(slices.length, 2, '#1736 checkbox compat: 2 slices');
-    assertEq(slices[0].done, true, '#1736 checkbox compat: S01 done');
-    assertEq(slices[0].demo, 'demo works.', '#1736 checkbox compat: demo');
-    assertEq(slices[1].done, false, '#1736 checkbox compat: S02 not done');
-  }
-
-  report();
-}
-
-main().catch((error) => {
-  console.error(error);
-  process.exit(1);
+    assert.deepStrictEqual(slices.length, 2, '#1736 checkbox compat: 2 slices');
+    assert.deepStrictEqual(slices[0].done, true, '#1736 checkbox compat: S01 done');
+    assert.deepStrictEqual(slices[0].demo, 'demo works.', '#1736 checkbox compat: demo');
+    assert.deepStrictEqual(slices[1].done, false, '#1736 checkbox compat: S02 not done');
+});
+
 });

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

@@ -1,6 +1,6 @@
 import test from "node:test";
 import assert from "node:assert/strict";
-import { parseRoadmap } from "../files.ts";
+import { parseRoadmap } from "../parsers-legacy.ts";
 import { parseRoadmapSlices, expandDependencies } from "../roadmap-slices.ts";
 
 const content = `# M003: Current