gsd-pi 2.43.0 → 2.44.0-dev.0b97ffd

package/src/resources/extensions/gsd/tests/memory-store.test.ts

@@ -1,4 +1,3 @@
-import { createTestContext } from './test-helpers.ts';
 import {
   openDatabase,
   closeDatabase,
@@ -21,94 +20,90 @@ import {
   formatMemoriesForPrompt,
 } from '../memory-store.ts';
 import type { MemoryAction } from '../memory-store.ts';
-
-const { assertEq, assertTrue, assertMatch, report } = createTestContext();
+import { describe, test, beforeEach, afterEach } from 'node:test';
+import assert from 'node:assert/strict';
 
 // ═══════════════════════════════════════════════════════════════════════════
 // memory-store: fallback when DB not open
 // ═══════════════════════════════════════════════════════════════════════════
 
-console.log('\n=== memory-store: fallback returns empty when DB not open ===');
-{
+test('memory-store: fallback returns empty when DB not open', () => {
   closeDatabase();
-  assertTrue(!isDbAvailable(), 'DB should not be available');
+  assert.ok(!isDbAvailable(), 'DB should not be available');
 
-  assertEq(getActiveMemories(), [], 'getActiveMemories returns [] when DB closed');
-  assertEq(getActiveMemoriesRanked(), [], 'getActiveMemoriesRanked returns [] when DB closed');
-  assertEq(nextMemoryId(), 'MEM001', 'nextMemoryId returns MEM001 when DB closed');
-  assertEq(createMemory({ category: 'test', content: 'test' }), null, 'createMemory returns null when DB closed');
-  assertTrue(!reinforceMemory('MEM001'), 'reinforceMemory returns false when DB closed');
-  assertTrue(!isUnitProcessed('test/key'), 'isUnitProcessed returns false when DB closed');
-}
+  assert.deepStrictEqual(getActiveMemories(), [], 'getActiveMemories returns [] when DB closed');
+  assert.deepStrictEqual(getActiveMemoriesRanked(), [], 'getActiveMemoriesRanked returns [] when DB closed');
+  assert.deepStrictEqual(nextMemoryId(), 'MEM001', 'nextMemoryId returns MEM001 when DB closed');
+  assert.deepStrictEqual(createMemory({ category: 'test', content: 'test' }), null, 'createMemory returns null when DB closed');
+  assert.ok(!reinforceMemory('MEM001'), 'reinforceMemory returns false when DB closed');
+  assert.ok(!isUnitProcessed('test/key'), 'isUnitProcessed returns false when DB closed');
+});
 
 // ═══════════════════════════════════════════════════════════════════════════
 // memory-store: CRUD operations
 // ═══════════════════════════════════════════════════════════════════════════
 
-console.log('\n=== memory-store: create and query memories ===');
-{
+test('memory-store: create and query memories', () => {
   openDatabase(':memory:');
 
   // Create memories
   const id1 = createMemory({ category: 'gotcha', content: 'esbuild drops .node binaries' });
-  assertTrue(id1 !== null, 'createMemory should return an ID');
-  assertEq(id1, 'MEM001', 'first memory ID should be MEM001');
+  assert.ok(id1 !== null, 'createMemory should return an ID');
+  assert.deepStrictEqual(id1, 'MEM001', 'first memory ID should be MEM001');
 
   const id2 = createMemory({ category: 'convention', content: 'use :memory: for tests', confidence: 0.9 });
-  assertEq(id2, 'MEM002', 'second memory ID should be MEM002');
+  assert.deepStrictEqual(id2, 'MEM002', 'second memory ID should be MEM002');
 
   const id3 = createMemory({ category: 'architecture', content: 'extensions discovered from src/resources/' });
-  assertEq(id3, 'MEM003', 'third memory ID should be MEM003');
+  assert.deepStrictEqual(id3, 'MEM003', 'third memory ID should be MEM003');
 
   // Query all active
   const active = getActiveMemories();
-  assertEq(active.length, 3, 'should have 3 active memories');
-  assertEq(active[0].category, 'gotcha', 'first memory category');
-  assertEq(active[0].content, 'esbuild drops .node binaries', 'first memory content');
-  assertEq(active[1].confidence, 0.9, 'second memory confidence');
+  assert.deepStrictEqual(active.length, 3, 'should have 3 active memories');
+  assert.deepStrictEqual(active[0].category, 'gotcha', 'first memory category');
+  assert.deepStrictEqual(active[0].content, 'esbuild drops .node binaries', 'first memory content');
+  assert.deepStrictEqual(active[1].confidence, 0.9, 'second memory confidence');
 
   closeDatabase();
-}
+});
 
 // ═══════════════════════════════════════════════════════════════════════════
 // memory-store: update and reinforce
 // ═══════════════════════════════════════════════════════════════════════════
 
-console.log('\n=== memory-store: update and reinforce ===');
-{
+test('memory-store: update and reinforce', () => {
   openDatabase(':memory:');
 
   createMemory({ category: 'gotcha', content: 'original content' });
 
   // Update content
   const updated = updateMemoryContent('MEM001', 'revised content', 0.95);
-  assertTrue(updated, 'updateMemoryContent should return true');
+  assert.ok(updated, 'updateMemoryContent should return true');
 
   const active = getActiveMemories();
-  assertEq(active[0].content, 'revised content', 'content should be updated');
-  assertEq(active[0].confidence, 0.95, 'confidence should be updated');
+  assert.deepStrictEqual(active[0].content, 'revised content', 'content should be updated');
+  assert.deepStrictEqual(active[0].confidence, 0.95, 'confidence should be updated');
 
   // Reinforce
   const reinforced = reinforceMemory('MEM001');
-  assertTrue(reinforced, 'reinforceMemory should return true');
+  assert.ok(reinforced, 'reinforceMemory should return true');
 
   const after = getActiveMemories();
-  assertEq(after[0].hit_count, 1, 'hit_count should be 1 after reinforce');
+  assert.deepStrictEqual(after[0].hit_count, 1, 'hit_count should be 1 after reinforce');
 
   // Reinforce again
   reinforceMemory('MEM001');
   const after2 = getActiveMemories();
-  assertEq(after2[0].hit_count, 2, 'hit_count should be 2 after second reinforce');
+  assert.deepStrictEqual(after2[0].hit_count, 2, 'hit_count should be 2 after second reinforce');
 
   closeDatabase();
-}
+});
 
 // ═══════════════════════════════════════════════════════════════════════════
 // memory-store: supersede
 // ═══════════════════════════════════════════════════════════════════════════
 
-console.log('\n=== memory-store: supersede ===');
-{
+test('memory-store: supersede', () => {
   openDatabase(':memory:');
 
   createMemory({ category: 'convention', content: 'old convention' });
@@ -117,18 +112,17 @@ console.log('\n=== memory-store: supersede ===');
   supersedeMemory('MEM001', 'MEM002');
 
   const active = getActiveMemories();
-  assertEq(active.length, 1, 'should have 1 active memory after supersede');
-  assertEq(active[0].id, 'MEM002', 'active memory should be MEM002');
+  assert.deepStrictEqual(active.length, 1, 'should have 1 active memory after supersede');
+  assert.deepStrictEqual(active[0].id, 'MEM002', 'active memory should be MEM002');
 
   closeDatabase();
-}
+});
 
 // ═══════════════════════════════════════════════════════════════════════════
 // memory-store: ranked query ordering
 // ═══════════════════════════════════════════════════════════════════════════
 
-console.log('\n=== memory-store: ranked query ordering ===');
-{
+test('memory-store: ranked query ordering', () => {
   openDatabase(':memory:');
 
   // Low confidence, no hits
@@ -142,45 +136,43 @@ console.log('\n=== memory-store: ranked query ordering ===');
   for (let i = 0; i < 10; i++) reinforceMemory('MEM003');
 
   const ranked = getActiveMemoriesRanked(10);
-  assertEq(ranked.length, 3, 'should have 3 ranked memories');
+  assert.deepStrictEqual(ranked.length, 3, 'should have 3 ranked memories');
   // MEM003: 0.7 * (1 + 10*0.1) = 0.7 * 2.0 = 1.4
   // MEM002: 0.95 * (1 + 0*0.1) = 0.95
   // MEM001: 0.5 * (1 + 0*0.1) = 0.5
-  assertEq(ranked[0].id, 'MEM003', 'highest ranked should be MEM003 (reinforced)');
-  assertEq(ranked[1].id, 'MEM002', 'second ranked should be MEM002 (high confidence)');
-  assertEq(ranked[2].id, 'MEM001', 'lowest ranked should be MEM001');
+  assert.deepStrictEqual(ranked[0].id, 'MEM003', 'highest ranked should be MEM003 (reinforced)');
+  assert.deepStrictEqual(ranked[1].id, 'MEM002', 'second ranked should be MEM002 (high confidence)');
+  assert.deepStrictEqual(ranked[2].id, 'MEM001', 'lowest ranked should be MEM001');
 
   // Test limit
   const limited = getActiveMemoriesRanked(2);
-  assertEq(limited.length, 2, 'limit should cap results');
+  assert.deepStrictEqual(limited.length, 2, 'limit should cap results');
 
   closeDatabase();
-}
+});
 
 // ═══════════════════════════════════════════════════════════════════════════
 // memory-store: processed unit tracking
 // ═══════════════════════════════════════════════════════════════════════════
 
-console.log('\n=== memory-store: processed unit tracking ===');
-{
+test('memory-store: processed unit tracking', () => {
   openDatabase(':memory:');
 
-  assertTrue(!isUnitProcessed('execute-task/M001/S01/T01'), 'should not be processed initially');
+  assert.ok(!isUnitProcessed('execute-task/M001/S01/T01'), 'should not be processed initially');
 
   markUnitProcessed('execute-task/M001/S01/T01', '/path/to/activity.jsonl');
 
-  assertTrue(isUnitProcessed('execute-task/M001/S01/T01'), 'should be processed after marking');
-  assertTrue(!isUnitProcessed('execute-task/M001/S01/T02'), 'different key should not be processed');
+  assert.ok(isUnitProcessed('execute-task/M001/S01/T01'), 'should be processed after marking');
+  assert.ok(!isUnitProcessed('execute-task/M001/S01/T02'), 'different key should not be processed');
 
   closeDatabase();
-}
+});
 
 // ═══════════════════════════════════════════════════════════════════════════
 // memory-store: enforce memory cap
 // ═══════════════════════════════════════════════════════════════════════════
 
-console.log('\n=== memory-store: enforce memory cap ===');
-{
+test('memory-store: enforce memory cap', () => {
   openDatabase(':memory:');
 
   // Create 5 memories with varying confidence
@@ -194,23 +186,22 @@ console.log('\n=== memory-store: enforce memory cap ===');
   enforceMemoryCap(3);
 
   const active = getActiveMemories();
-  assertEq(active.length, 3, 'should have 3 active memories after cap enforcement');
+  assert.deepStrictEqual(active.length, 3, 'should have 3 active memories after cap enforcement');
 
   // The 2 lowest-ranked (MEM003=0.3 and MEM002=0.5) should be superseded
   const ids = active.map(m => m.id).sort();
-  assertTrue(ids.includes('MEM001'), 'MEM001 (0.9) should survive');
-  assertTrue(ids.includes('MEM004'), 'MEM004 (0.95) should survive');
-  assertTrue(ids.includes('MEM005'), 'MEM005 (0.7) should survive');
+  assert.ok(ids.includes('MEM001'), 'MEM001 (0.9) should survive');
+  assert.ok(ids.includes('MEM004'), 'MEM004 (0.95) should survive');
+  assert.ok(ids.includes('MEM005'), 'MEM005 (0.7) should survive');
 
   closeDatabase();
-}
+});
 
 // ═══════════════════════════════════════════════════════════════════════════
 // memory-store: applyMemoryActions transaction
 // ═══════════════════════════════════════════════════════════════════════════
 
-console.log('\n=== memory-store: applyMemoryActions ===');
-{
+test('memory-store: applyMemoryActions', () => {
   openDatabase(':memory:');
 
   const actions: MemoryAction[] = [
@@ -221,7 +212,7 @@ console.log('\n=== memory-store: applyMemoryActions ===');
   applyMemoryActions(actions, 'execute-task', 'M001/S01/T01');
 
   let active = getActiveMemories();
-  assertEq(active.length, 2, 'should have 2 memories after CREATE actions');
+  assert.deepStrictEqual(active.length, 2, 'should have 2 memories after CREATE actions');
 
   // Now apply UPDATE + REINFORCE
   const updateActions: MemoryAction[] = [
@@ -232,8 +223,8 @@ console.log('\n=== memory-store: applyMemoryActions ===');
   applyMemoryActions(updateActions, 'execute-task', 'M001/S01/T02');
 
   active = getActiveMemories();
-  assertEq(active.find(m => m.id === 'MEM001')?.content, 'updated gotcha', 'MEM001 should be updated');
-  assertEq(active.find(m => m.id === 'MEM002')?.hit_count, 1, 'MEM002 should be reinforced');
+  assert.deepStrictEqual(active.find(m => m.id === 'MEM001')?.content, 'updated gotcha', 'MEM001 should be updated');
+  assert.deepStrictEqual(active.find(m => m.id === 'MEM002')?.hit_count, 1, 'MEM002 should be reinforced');
 
   // SUPERSEDE
   const supersedeActions: MemoryAction[] = [
@@ -244,19 +235,18 @@ console.log('\n=== memory-store: applyMemoryActions ===');
   applyMemoryActions(supersedeActions, 'execute-task', 'M001/S01/T03');
 
   active = getActiveMemories();
-  assertEq(active.length, 2, 'should have 2 active after supersede');
-  assertTrue(!active.find(m => m.id === 'MEM001'), 'MEM001 should be superseded');
-  assertTrue(!!active.find(m => m.id === 'MEM003'), 'MEM003 should be active');
+  assert.deepStrictEqual(active.length, 2, 'should have 2 active after supersede');
+  assert.ok(!active.find(m => m.id === 'MEM001'), 'MEM001 should be superseded');
+  assert.ok(!!active.find(m => m.id === 'MEM003'), 'MEM003 should be active');
 
   closeDatabase();
-}
+});
 
 // ═══════════════════════════════════════════════════════════════════════════
 // memory-store: formatMemoriesForPrompt
 // ═══════════════════════════════════════════════════════════════════════════
 
-console.log('\n=== memory-store: formatMemoriesForPrompt ===');
-{
+test('memory-store: formatMemoriesForPrompt', () => {
   openDatabase(':memory:');
 
   createMemory({ category: 'gotcha', content: 'esbuild drops .node binaries' });
@@ -267,18 +257,18 @@ console.log('\n=== memory-store: formatMemoriesForPrompt ===');
   const memories = getActiveMemoriesRanked(30);
   const formatted = formatMemoriesForPrompt(memories);
 
-  assertTrue(formatted.includes('## Project Memory (auto-learned)'), 'should have header');
-  assertTrue(formatted.includes('### Gotcha'), 'should have gotcha category');
-  assertTrue(formatted.includes('### Convention'), 'should have convention category');
-  assertTrue(formatted.includes('### Architecture'), 'should have architecture category');
-  assertTrue(formatted.includes('- esbuild drops .node binaries'), 'should have gotcha content');
-  assertTrue(formatted.includes('- use :memory: for tests'), 'should have convention content');
+  assert.ok(formatted.includes('## Project Memory (auto-learned)'), 'should have header');
+  assert.ok(formatted.includes('### Gotcha'), 'should have gotcha category');
+  assert.ok(formatted.includes('### Convention'), 'should have convention category');
+  assert.ok(formatted.includes('### Architecture'), 'should have architecture category');
+  assert.ok(formatted.includes('- esbuild drops .node binaries'), 'should have gotcha content');
+  assert.ok(formatted.includes('- use :memory: for tests'), 'should have convention content');
 
   // Test empty memories
   closeDatabase();
   openDatabase(':memory:');
   const emptyFormatted = formatMemoriesForPrompt([]);
-  assertEq(emptyFormatted, '', 'empty memories should return empty string');
+  assert.deepStrictEqual(emptyFormatted, '', 'empty memories should return empty string');
 
   // Test token budget truncation
   closeDatabase();
@@ -288,58 +278,55 @@ console.log('\n=== memory-store: formatMemoriesForPrompt ===');
   }
   const budgetMemories = getActiveMemoriesRanked(30);
   const truncated = formatMemoriesForPrompt(budgetMemories, 500);
-  assertTrue(truncated.length < 2500, `formatted length ${truncated.length} should be under budget`);
+  assert.ok(truncated.length < 2500, `formatted length ${truncated.length} should be under budget`);
 
   closeDatabase();
-}
+});
 
 // ═══════════════════════════════════════════════════════════════════════════
 // memory-store: ID generation
 // ═══════════════════════════════════════════════════════════════════════════
 
-console.log('\n=== memory-store: ID generation ===');
-{
+test('memory-store: ID generation', () => {
   openDatabase(':memory:');
 
-  assertEq(nextMemoryId(), 'MEM001', 'first ID should be MEM001');
+  assert.deepStrictEqual(nextMemoryId(), 'MEM001', 'first ID should be MEM001');
 
   createMemory({ category: 'test', content: 'test' });
-  assertEq(nextMemoryId(), 'MEM002', 'after first create, next should be MEM002');
+  assert.deepStrictEqual(nextMemoryId(), 'MEM002', 'after first create, next should be MEM002');
 
   // Create several more
   for (let i = 0; i < 98; i++) createMemory({ category: 'test', content: `test ${i}` });
-  assertEq(nextMemoryId(), 'MEM100', 'after 99 creates, next should be MEM100');
+  assert.deepStrictEqual(nextMemoryId(), 'MEM100', 'after 99 creates, next should be MEM100');
 
   closeDatabase();
-}
+});
 
 // ═══════════════════════════════════════════════════════════════════════════
 // memory-store: schema migration (v2 → v3)
 // ═══════════════════════════════════════════════════════════════════════════
 
-console.log('\n=== memory-store: schema includes memories table ===');
-{
+test('memory-store: schema includes memories table', () => {
   openDatabase(':memory:');
 
   const adapter = _getAdapter()!;
 
   // Verify memories table exists
   const memCount = adapter.prepare('SELECT count(*) as cnt FROM memories').get();
-  assertEq(memCount?.['cnt'], 0, 'memories table should exist and be empty');
+  assert.deepStrictEqual(memCount?.['cnt'], 0, 'memories table should exist and be empty');
 
   // Verify memory_processed_units table exists
   const procCount = adapter.prepare('SELECT count(*) as cnt FROM memory_processed_units').get();
-  assertEq(procCount?.['cnt'], 0, 'memory_processed_units table should exist and be empty');
+  assert.deepStrictEqual(procCount?.['cnt'], 0, 'memory_processed_units table should exist and be empty');
 
   // Verify active_memories view exists
   const viewCount = adapter.prepare('SELECT count(*) as cnt FROM active_memories').get();
-  assertEq(viewCount?.['cnt'], 0, 'active_memories view should exist');
+  assert.deepStrictEqual(viewCount?.['cnt'], 0, 'active_memories view should exist');
 
-  // Verify schema version is 4
+  // Verify schema version is 10 (after M001 planning migrations)
   const version = adapter.prepare('SELECT MAX(version) as v FROM schema_version').get();
-  assertEq(version?.['v'], 4, 'schema version should be 4');
+  assert.deepStrictEqual(version?.['v'], 10, 'schema version should be 10');
 
   closeDatabase();
-}
+});
 
-report();

package/src/resources/extensions/gsd/tests/migrate-command.test.ts

@@ -15,9 +15,9 @@ import {
   writeGSDDirectory,
 } from '../migrate/index.ts';
 import { deriveState } from '../state.ts';
-import { createTestContext } from './test-helpers.ts';
+import { describe, test, beforeEach, afterEach } from 'node:test';
+import assert from 'node:assert/strict';
 
-const { assertEq, assertTrue, report } = createTestContext();
 // ─── Fixture Helpers ───────────────────────────────────────────────────────
 
 const SAMPLE_PROJECT = `# Integration Test Project
@@ -195,11 +195,9 @@ function createCompleteFixture(): string {
 // Tests
 // ═══════════════════════════════════════════════════════════════════════════
 
-async function main(): Promise<void> {
-
   // ─── Test 1: Path resolution — .planning appended when missing ─────────
-  console.log('\n=== Path resolution: .planning appended when source path lacks it ===');
-  {
+
+test('Path resolution: .planning appended when source path lacks it', () => {
     const base = createCompleteFixture();
     try {
       // Simulate the command's path resolution logic
@@ -207,16 +205,16 @@ async function main(): Promise<void> {
       if (!sourcePath.endsWith('.planning')) {
         sourcePath = join(sourcePath, '.planning');
       }
-      assertTrue(sourcePath.endsWith('.planning'), 'path-resolution: .planning appended');
-      assertTrue(existsSync(sourcePath), 'path-resolution: appended path exists');
+      assert.ok(sourcePath.endsWith('.planning'), 'path-resolution: .planning appended');
+      assert.ok(existsSync(sourcePath), 'path-resolution: appended path exists');
     } finally {
       rmSync(base, { recursive: true, force: true });
     }
-  }
+});
 
   // ─── Test 2: Path resolution — .planning used as-is ────────────────────
-  console.log('\n=== Path resolution: .planning used as-is when already present ===');
-  {
+
+test('Path resolution: .planning used as-is when already present', () => {
     const base = createCompleteFixture();
     try {
       const planningPath = join(base, '.planning');
@@ -224,39 +222,39 @@ async function main(): Promise<void> {
       if (!sourcePath.endsWith('.planning')) {
         sourcePath = join(sourcePath, '.planning');
       }
-      assertEq(sourcePath, resolve(planningPath), 'path-resolution: .planning not double-appended');
-      assertTrue(existsSync(sourcePath), 'path-resolution: direct path exists');
+      assert.deepStrictEqual(sourcePath, resolve(planningPath), 'path-resolution: .planning not double-appended');
+      assert.ok(existsSync(sourcePath), 'path-resolution: direct path exists');
     } finally {
       rmSync(base, { recursive: true, force: true });
     }
-  }
+});
 
   // ─── Test 3: Validation gating — non-existent path ─────────────────────
-  console.log('\n=== Validation gating: non-existent path returns invalid ===');
-  {
+
+test('Validation gating: non-existent path returns invalid', async () => {
     const fakePath = join(tmpdir(), 'gsd-cmd-nonexistent-' + Date.now(), '.planning');
     const result = await validatePlanningDirectory(fakePath);
-    assertEq(result.valid, false, 'validation: non-existent path is invalid');
-    assertTrue(result.issues.length > 0, 'validation: has issues for non-existent path');
+    assert.deepStrictEqual(result.valid, false, 'validation: non-existent path is invalid');
+    assert.ok(result.issues.length > 0, 'validation: has issues for non-existent path');
     const hasFatal = result.issues.some(i => i.severity === 'fatal');
-    assertTrue(hasFatal, 'validation: non-existent path has fatal issue');
-  }
+    assert.ok(hasFatal, 'validation: non-existent path has fatal issue');
+});
 
   // ─── Test 4: Validation gating — valid fixture passes ──────────────────
-  console.log('\n=== Validation gating: valid fixture passes validation ===');
-  {
+
+test('Validation gating: valid fixture passes validation', async () => {
     const base = createCompleteFixture();
     try {
       const result = await validatePlanningDirectory(join(base, '.planning'));
-      assertTrue(result.valid === true, 'validation: valid fixture passes');
+      assert.ok(result.valid === true, 'validation: valid fixture passes');
     } finally {
       rmSync(base, { recursive: true, force: true });
     }
-  }
+});
 
   // ─── Test 5: Full pipeline round-trip ──────────────────────────────────
-  console.log('\n=== Full pipeline: parse → transform → preview → write → deriveState ===');
-  {
+
+test('Full pipeline: parse → transform → preview → write → deriveState', async () => {
     const base = createCompleteFixture();
     const writeTarget = mkdtempSync(join(tmpdir(), 'gsd-cmd-write-'));
     try {
@@ -264,17 +262,17 @@ async function main(): Promise<void> {
 
       // (a) Validate
       const validation = await validatePlanningDirectory(planningPath);
-      assertTrue(validation.valid === true, 'pipeline: validation passes');
+      assert.ok(validation.valid === true, 'pipeline: validation passes');
 
       // (b) Parse
       const parsed = await parsePlanningDirectory(planningPath);
-      assertTrue(parsed.roadmap !== null, 'pipeline: roadmap parsed');
-      assertTrue(Object.keys(parsed.phases).length >= 2, 'pipeline: phases parsed');
+      assert.ok(parsed.roadmap !== null, 'pipeline: roadmap parsed');
+      assert.ok(Object.keys(parsed.phases).length >= 2, 'pipeline: phases parsed');
 
       // (c) Transform
       const project = transformToGSD(parsed);
-      assertTrue(project.milestones.length >= 1, 'pipeline: has milestones');
-      assertTrue(project.milestones[0].slices.length >= 1, 'pipeline: has slices');
+      assert.ok(project.milestones.length >= 1, 'pipeline: has milestones');
+      assert.ok(project.milestones[0].slices.length >= 1, 'pipeline: has slices');
 
       // Count totals for preview verification
       let totalTasks = 0;
@@ -294,76 +292,69 @@ async function main(): Promise<void> {
 
       // (d) Preview — verify counts match project data
       const preview = generatePreview(project);
-      assertEq(preview.milestoneCount, project.milestones.length, 'pipeline: preview milestoneCount');
-      assertEq(preview.totalSlices, totalSlices, 'pipeline: preview totalSlices');
-      assertEq(preview.totalTasks, totalTasks, 'pipeline: preview totalTasks');
-      assertEq(preview.doneSlices, doneSlices, 'pipeline: preview doneSlices');
-      assertEq(preview.doneTasks, doneTasks, 'pipeline: preview doneTasks');
+      assert.deepStrictEqual(preview.milestoneCount, project.milestones.length, 'pipeline: preview milestoneCount');
+      assert.deepStrictEqual(preview.totalSlices, totalSlices, 'pipeline: preview totalSlices');
+      assert.deepStrictEqual(preview.totalTasks, totalTasks, 'pipeline: preview totalTasks');
+      assert.deepStrictEqual(preview.doneSlices, doneSlices, 'pipeline: preview doneSlices');
+      assert.deepStrictEqual(preview.doneTasks, doneTasks, 'pipeline: preview doneTasks');
 
       // Completion percentages
       const expectedSlicePct = totalSlices > 0 ? Math.round((doneSlices / totalSlices) * 100) : 0;
       const expectedTaskPct = totalTasks > 0 ? Math.round((doneTasks / totalTasks) * 100) : 0;
-      assertEq(preview.sliceCompletionPct, expectedSlicePct, 'pipeline: preview sliceCompletionPct');
-      assertEq(preview.taskCompletionPct, expectedTaskPct, 'pipeline: preview taskCompletionPct');
+      assert.deepStrictEqual(preview.sliceCompletionPct, expectedSlicePct, 'pipeline: preview sliceCompletionPct');
+      assert.deepStrictEqual(preview.taskCompletionPct, expectedTaskPct, 'pipeline: preview taskCompletionPct');
 
       // Requirements in preview
-      assertEq(preview.requirements.active, 1, 'pipeline: preview requirements active');
-      assertEq(preview.requirements.validated, 1, 'pipeline: preview requirements validated');
-      assertEq(preview.requirements.total, 2, 'pipeline: preview requirements total');
+      assert.deepStrictEqual(preview.requirements.active, 1, 'pipeline: preview requirements active');
+      assert.deepStrictEqual(preview.requirements.validated, 1, 'pipeline: preview requirements validated');
+      assert.deepStrictEqual(preview.requirements.total, 2, 'pipeline: preview requirements total');
 
       // (e) Write
       const result = await writeGSDDirectory(project, writeTarget);
-      assertTrue(result.paths.length > 0, 'pipeline: files written');
+      assert.ok(result.paths.length > 0, 'pipeline: files written');
 
       // Key files exist
       const gsd = join(writeTarget, '.gsd');
-      assertTrue(existsSync(join(gsd, 'PROJECT.md')), 'pipeline: PROJECT.md written');
-      assertTrue(existsSync(join(gsd, 'STATE.md')), 'pipeline: STATE.md written');
-      assertTrue(existsSync(join(gsd, 'REQUIREMENTS.md')), 'pipeline: REQUIREMENTS.md written');
+      assert.ok(existsSync(join(gsd, 'PROJECT.md')), 'pipeline: PROJECT.md written');
+      assert.ok(existsSync(join(gsd, 'STATE.md')), 'pipeline: STATE.md written');
+      assert.ok(existsSync(join(gsd, 'REQUIREMENTS.md')), 'pipeline: REQUIREMENTS.md written');
 
       const m001 = join(gsd, 'milestones', 'M001');
-      assertTrue(existsSync(join(m001, 'M001-ROADMAP.md')), 'pipeline: M001-ROADMAP.md written');
-      assertTrue(existsSync(join(m001, 'M001-CONTEXT.md')), 'pipeline: M001-CONTEXT.md written');
+      assert.ok(existsSync(join(m001, 'M001-ROADMAP.md')), 'pipeline: M001-ROADMAP.md written');
+      assert.ok(existsSync(join(m001, 'M001-CONTEXT.md')), 'pipeline: M001-CONTEXT.md written');
 
       // At least one slice plan exists
       const s01Plan = join(m001, 'slices', 'S01', 'S01-PLAN.md');
-      assertTrue(existsSync(s01Plan), 'pipeline: S01-PLAN.md written');
+      assert.ok(existsSync(s01Plan), 'pipeline: S01-PLAN.md written');
 
       // (f) deriveState — coherent state from written output
       console.log('  --- deriveState ---');
       const state = await deriveState(writeTarget);
-      assertTrue(state.phase !== undefined, 'pipeline: deriveState returns phase');
-      assertTrue(state.activeMilestone !== null, 'pipeline: deriveState has activeMilestone');
-      assertEq(state.activeMilestone!.id, 'M001', 'pipeline: deriveState activeMilestone is M001');
-      assertTrue(state.progress!.slices !== undefined, 'pipeline: deriveState has slices progress');
-      assertTrue(state.progress!.tasks !== undefined, 'pipeline: deriveState has tasks progress');
+      assert.ok(state.phase !== undefined, 'pipeline: deriveState returns phase');
+      assert.ok(state.activeMilestone !== null, 'pipeline: deriveState has activeMilestone');
+      assert.deepStrictEqual(state.activeMilestone!.id, 'M001', 'pipeline: deriveState activeMilestone is M001');
+      assert.ok(state.progress!.slices !== undefined, 'pipeline: deriveState has slices progress');
+      assert.ok(state.progress!.tasks !== undefined, 'pipeline: deriveState has tasks progress');
 
     } finally {
       rmSync(base, { recursive: true, force: true });
       rmSync(writeTarget, { recursive: true, force: true });
     }
-  }
+});
 
   // ─── Test 6: .gsd/ exists detection ────────────────────────────────────
-  console.log('\n=== .gsd/ exists detection ===');
-  {
+
+test('.gsd/ exists detection', () => {
     const base = mkdtempSync(join(tmpdir(), 'gsd-cmd-exists-'));
     try {
       // No .gsd/ yet
-      assertTrue(!existsSync(join(base, '.gsd')), 'exists-detection: .gsd absent initially');
+      assert.ok(!existsSync(join(base, '.gsd')), 'exists-detection: .gsd absent initially');
 
       // Create .gsd/
       mkdirSync(join(base, '.gsd'), { recursive: true });
-      assertTrue(existsSync(join(base, '.gsd')), 'exists-detection: .gsd detected after creation');
+      assert.ok(existsSync(join(base, '.gsd')), 'exists-detection: .gsd detected after creation');
     } finally {
       rmSync(base, { recursive: true, force: true });
     }
-  }
-
-  report();
-}
-
-main().catch((err) => {
-  console.error('Unhandled error:', err);
-  process.exit(1);
 });
+