gsd-pi 2.44.0-dev.848dd4c → 2.44.0

package/src/resources/extensions/gsd/tests/auto-secrets-gate.test.ts

@@ -43,36 +43,31 @@ function makeNoUICtx(cwd: string) {
 
 // ─── Scenario 1: No manifest exists ──────────────────────────────────────────
 
-test('secrets gate: no manifest exists — getManifestStatus returns null', async (t) => {
+test('secrets gate: no manifest exists — getManifestStatus returns null', async () => {
   const tmp = makeTempDir('gate-no-manifest');
-  t.after(() => rmSync(tmp, { recursive: true, force: true }));
-
-  // No .gsd directory at all
-  const result = await getManifestStatus(tmp, 'M001');
-  assert.strictEqual(result, null, 'should return null when no manifest file exists');
+  try {
+    // No .gsd directory at all
+    const result = await getManifestStatus(tmp, 'M001');
+    assert.strictEqual(result, null, 'should return null when no manifest file exists');
+  } finally {
+    rmSync(tmp, { recursive: true, force: true });
+  }
 });
 
 // ─── Scenario 2: Pending keys exist ─────────────────────────────────────────
 
-test('secrets gate: pending keys exist — gate triggers collection, manifest updated on disk', async (t) => {
+test('secrets gate: pending keys exist — gate triggers collection, manifest updated on disk', async () => {
   const tmp = makeTempDir('gate-pending');
   const savedA = process.env.GSD_GATE_TEST_EXISTING;
-  t.after(() => {
-    delete process.env.GSD_GATE_TEST_EXISTING;
-    if (savedA !== undefined) process.env.GSD_GATE_TEST_EXISTING = savedA;
+  try {
+    // Simulate one key already in env
+    process.env.GSD_GATE_TEST_EXISTING = 'already-here';
+
+    // Ensure pending keys are NOT in env
     delete process.env.GSD_GATE_TEST_PEND_A;
     delete process.env.GSD_GATE_TEST_PEND_B;
-    rmSync(tmp, { recursive: true, force: true });
-  });
-
-  // Simulate one key already in env
-  process.env.GSD_GATE_TEST_EXISTING = 'already-here';
-
-  // Ensure pending keys are NOT in env
-  delete process.env.GSD_GATE_TEST_PEND_A;
-  delete process.env.GSD_GATE_TEST_PEND_B;
 
-  writeManifest(tmp, `# Secrets Manifest
+    writeManifest(tmp, `# Secrets Manifest
 
 **Milestone:** M001
 **Generated:** 2025-06-20T10:00:00Z
@@ -102,60 +97,62 @@ test('secrets gate: pending keys exist — gate triggers collection, manifest up
 1. Already in env
 `);
 
-  // (a) Verify getManifestStatus shows pending keys
-  const status = await getManifestStatus(tmp, 'M001');
-  assert.notStrictEqual(status, null, 'manifest should exist');
-  assert.ok(status!.pending.length > 0, 'should have pending keys');
-  assert.deepStrictEqual(status!.pending, ['GSD_GATE_TEST_PEND_A', 'GSD_GATE_TEST_PEND_B'], 'pending keys');
-  assert.deepStrictEqual(status!.existing, ['GSD_GATE_TEST_EXISTING'], 'existing keys');
-
-  // (b) Call collectSecretsFromManifest with no-UI context
-  // With hasUI: false, collectOneSecret returns null → pending keys become "skipped"
-  const result = await collectSecretsFromManifest(tmp, 'M001', makeNoUICtx(tmp));
-
-  // (c) Verify return shape
-  assert.deepStrictEqual(result.applied, [], 'no keys applied (no UI to enter values)');
-  assert.ok(result.skipped.includes('GSD_GATE_TEST_PEND_A'), 'PEND_A should be skipped');
-  assert.ok(result.skipped.includes('GSD_GATE_TEST_PEND_B'), 'PEND_B should be skipped');
-  assert.deepStrictEqual(result.existingSkipped, ['GSD_GATE_TEST_EXISTING']);
-
-  // (d) Verify manifest on disk was updated — pending entries that went through
-  // collection are now "skipped". The existing-in-env entry retains its manifest
-  // status ("pending") because collectSecretsFromManifest only updates entries
-  // that flow through collectOneSecret. At runtime, getManifestStatus overrides
-  // env-present entries to "existing" regardless of manifest status.
-  const manifestPath = join(tmp, '.gsd', 'milestones', 'M001', 'M001-SECRETS.md');
-  const updatedContent = readFileSync(manifestPath, 'utf8');
-  assert.ok(
-    updatedContent.includes('**Status:** skipped'),
-    'formerly-pending entries should now have status "skipped" in the manifest file',
-  );
-  // Count: PEND_A → skipped, PEND_B → skipped, EXISTING stays pending on disk
-  const skippedMatches = updatedContent.match(/\*\*Status:\*\* skipped/g);
-  assert.strictEqual(skippedMatches?.length, 2, 'two entries should have status "skipped"');
-  const pendingMatches = updatedContent.match(/\*\*Status:\*\* pending/g);
-  assert.strictEqual(pendingMatches?.length, 1, 'one entry (existing-in-env) retains pending on disk');
-
-  // (e) Verify getManifestStatus now shows no pending
-  const statusAfter = await getManifestStatus(tmp, 'M001');
-  assert.notStrictEqual(statusAfter, null);
-  assert.deepStrictEqual(statusAfter!.pending, [], 'no pending keys after collection');
+    // (a) Verify getManifestStatus shows pending keys
+    const status = await getManifestStatus(tmp, 'M001');
+    assert.notStrictEqual(status, null, 'manifest should exist');
+    assert.ok(status!.pending.length > 0, 'should have pending keys');
+    assert.deepStrictEqual(status!.pending, ['GSD_GATE_TEST_PEND_A', 'GSD_GATE_TEST_PEND_B'], 'pending keys');
+    assert.deepStrictEqual(status!.existing, ['GSD_GATE_TEST_EXISTING'], 'existing keys');
+
+    // (b) Call collectSecretsFromManifest with no-UI context
+    // With hasUI: false, collectOneSecret returns null → pending keys become "skipped"
+    const result = await collectSecretsFromManifest(tmp, 'M001', makeNoUICtx(tmp));
+
+    // (c) Verify return shape
+    assert.deepStrictEqual(result.applied, [], 'no keys applied (no UI to enter values)');
+    assert.ok(result.skipped.includes('GSD_GATE_TEST_PEND_A'), 'PEND_A should be skipped');
+    assert.ok(result.skipped.includes('GSD_GATE_TEST_PEND_B'), 'PEND_B should be skipped');
+    assert.deepStrictEqual(result.existingSkipped, ['GSD_GATE_TEST_EXISTING']);
+
+    // (d) Verify manifest on disk was updated — pending entries that went through
+    // collection are now "skipped". The existing-in-env entry retains its manifest
+    // status ("pending") because collectSecretsFromManifest only updates entries
+    // that flow through collectOneSecret. At runtime, getManifestStatus overrides
+    // env-present entries to "existing" regardless of manifest status.
+    const manifestPath = join(tmp, '.gsd', 'milestones', 'M001', 'M001-SECRETS.md');
+    const updatedContent = readFileSync(manifestPath, 'utf8');
+    assert.ok(
+      updatedContent.includes('**Status:** skipped'),
+      'formerly-pending entries should now have status "skipped" in the manifest file',
+    );
+    // Count: PEND_A → skipped, PEND_B → skipped, EXISTING stays pending on disk
+    const skippedMatches = updatedContent.match(/\*\*Status:\*\* skipped/g);
+    assert.strictEqual(skippedMatches?.length, 2, 'two entries should have status "skipped"');
+    const pendingMatches = updatedContent.match(/\*\*Status:\*\* pending/g);
+    assert.strictEqual(pendingMatches?.length, 1, 'one entry (existing-in-env) retains pending on disk');
+
+    // (e) Verify getManifestStatus now shows no pending
+    const statusAfter = await getManifestStatus(tmp, 'M001');
+    assert.notStrictEqual(statusAfter, null);
+    assert.deepStrictEqual(statusAfter!.pending, [], 'no pending keys after collection');
+  } finally {
+    delete process.env.GSD_GATE_TEST_EXISTING;
+    if (savedA !== undefined) process.env.GSD_GATE_TEST_EXISTING = savedA;
+    delete process.env.GSD_GATE_TEST_PEND_A;
+    delete process.env.GSD_GATE_TEST_PEND_B;
+    rmSync(tmp, { recursive: true, force: true });
+  }
 });
 
 // ─── Scenario 3: No pending keys — all collected or in env ──────────────────
 
-test('secrets gate: no pending keys — getManifestStatus shows pending.length === 0', async (t) => {
+test('secrets gate: no pending keys — getManifestStatus shows pending.length === 0', async () => {
   const tmp = makeTempDir('gate-no-pending');
   const savedKey = process.env.GSD_GATE_TEST_ENVKEY;
-  t.after(() => {
-    delete process.env.GSD_GATE_TEST_ENVKEY;
-    if (savedKey !== undefined) process.env.GSD_GATE_TEST_ENVKEY = savedKey;
-    rmSync(tmp, { recursive: true, force: true });
-  });
-
-  process.env.GSD_GATE_TEST_ENVKEY = 'some-value';
+  try {
+    process.env.GSD_GATE_TEST_ENVKEY = 'some-value';
 
-  writeManifest(tmp, `# Secrets Manifest
+    writeManifest(tmp, `# Secrets Manifest
 
 **Milestone:** M001
 **Generated:** 2025-06-20T10:00:00Z
@@ -185,10 +182,15 @@ test('secrets gate: no pending keys — getManifestStatus shows pending.length =
 1. In env already
 `);
 
-  const result = await getManifestStatus(tmp, 'M001');
-  assert.notStrictEqual(result, null, 'manifest should exist');
-  assert.deepStrictEqual(result!.pending, [], 'no pending keys — gate would skip');
-  assert.deepStrictEqual(result!.collected, ['ALREADY_COLLECTED']);
-  assert.deepStrictEqual(result!.skipped, ['ALREADY_SKIPPED']);
-  assert.deepStrictEqual(result!.existing, ['GSD_GATE_TEST_ENVKEY']);
+    const result = await getManifestStatus(tmp, 'M001');
+    assert.notStrictEqual(result, null, 'manifest should exist');
+    assert.deepStrictEqual(result!.pending, [], 'no pending keys — gate would skip');
+    assert.deepStrictEqual(result!.collected, ['ALREADY_COLLECTED']);
+    assert.deepStrictEqual(result!.skipped, ['ALREADY_SKIPPED']);
+    assert.deepStrictEqual(result!.existing, ['GSD_GATE_TEST_ENVKEY']);
+  } finally {
+    delete process.env.GSD_GATE_TEST_ENVKEY;
+    if (savedKey !== undefined) process.env.GSD_GATE_TEST_ENVKEY = savedKey;
+    rmSync(tmp, { recursive: true, force: true });
+  }
 });

package/src/resources/extensions/gsd/tests/auto-start-needs-discussion.test.ts

@@ -22,8 +22,6 @@
  *   - The !hasSurvivorBranch block has a needs-discussion handler
  */
 
-import { describe, test, afterEach } from "node:test";
-import assert from "node:assert/strict";
 import { mkdtempSync, mkdirSync, rmSync, writeFileSync, readFileSync } from "node:fs";
 import { join } from "node:path";
 import { tmpdir } from "node:os";
@@ -32,6 +30,9 @@ import { dirname } from "node:path";
 
 import { deriveState } from "../state.ts";
 import { invalidateAllCaches } from "../cache.ts";
+import { createTestContext } from "./test-helpers.ts";
+
+const { assertEq, assertTrue, report } = createTestContext();
 
 // ─── Fixture Helpers ─────────────────────────────────────────────────────────
 
@@ -75,46 +76,52 @@ function readAutoStartSource(): string {
 // Tests
 // ═══════════════════════════════════════════════════════════════════════════════
 
-describe("auto-start-needs-discussion (#1726)", () => {
+async function main(): Promise<void> {
 
-  test("1. CONTEXT-DRAFT.md only → needs-discussion phase", async () => {
+  // ─── 1. deriveState returns needs-discussion for CONTEXT-DRAFT only ────────
+  console.log("\n=== 1. CONTEXT-DRAFT.md only → needs-discussion phase ===");
+  {
     const base = createBase();
     try {
       writeContextDraft(base, "M001", "# Draft\nSeed discussion.");
       invalidateAllCaches();
       const state = await deriveState(base);
-      assert.strictEqual(state.phase, "needs-discussion",
+      assertEq(state.phase, "needs-discussion",
         "milestone with only CONTEXT-DRAFT should be needs-discussion");
-      assert.ok(!!state.activeMilestone,
+      assertTrue(!!state.activeMilestone,
         "activeMilestone should be set for needs-discussion");
-      assert.strictEqual(state.activeMilestone?.id, "M001",
+      assertEq(state.activeMilestone?.id, "M001",
         "activeMilestone.id should be M001");
     } finally {
       cleanup(base);
     }
-  });
+  }
 
-  test("2. Survivor branch check excludes needs-discussion", () => {
+  // ─── 2. Survivor branch filter excludes needs-discussion (#1726 bug 1) ────
+  console.log("\n=== 2. Survivor branch check excludes needs-discussion ===");
+  {
     const source = readAutoStartSource();
 
     // Find the survivor branch check block (Milestone branch recovery comment)
     const survivorBlock = source.match(
       /\/\/ Milestone branch recovery.*?hasSurvivorBranch = nativeBranchExists/s,
     );
-    assert.ok(!!survivorBlock,
+    assertTrue(!!survivorBlock,
       "found survivor branch check block in auto-start.ts");
 
     if (survivorBlock) {
       const block = survivorBlock[0];
       // The condition should only check pre-planning, NOT needs-discussion
-      assert.ok(!block.includes("needs-discussion"),
+      assertTrue(!block.includes("needs-discussion"),
         "survivor branch filter must NOT include needs-discussion phase");
-      assert.ok(block.includes("pre-planning"),
+      assertTrue(block.includes("pre-planning"),
         "survivor branch filter should include pre-planning phase");
     }
-  });
+  }
 
-  test("3. needs-discussion handler exists in bootstrap", () => {
+  // ─── 3. needs-discussion handler exists in !hasSurvivorBranch block (#1726 bug 2)
+  console.log("\n=== 3. needs-discussion handler exists in bootstrap ===");
+  {
     const source = readAutoStartSource();
 
     // After the pre-planning handler, there should be a needs-discussion handler
@@ -122,26 +129,30 @@ describe("auto-start-needs-discussion (#1726)", () => {
     const needsDiscussionHandler = source.match(
       /if\s*\(state\.phase\s*===\s*"needs-discussion"\)\s*\{[^}]*showSmartEntry/s,
     );
-    assert.ok(!!needsDiscussionHandler,
+    assertTrue(!!needsDiscussionHandler,
       "needs-discussion handler calling showSmartEntry must exist in !hasSurvivorBranch block");
-  });
+  }
 
-  test("4. needs-discussion handler has abort path", () => {
+  // ─── 4. needs-discussion handler aborts if discussion doesn't promote draft
+  console.log("\n=== 4. needs-discussion handler has abort path ===");
+  {
     const source = readAutoStartSource();
 
     // The handler should check postState.phase !== "needs-discussion" and abort
     // if discussion didn't promote the draft
-    assert.ok(
+    assertTrue(
       source.includes('postState.phase !== "needs-discussion"'),
       "needs-discussion handler must check if phase advanced after showSmartEntry",
     );
-    assert.ok(
+    assertTrue(
       source.includes("milestone draft was not promoted"),
       "needs-discussion handler must have abort message when draft not promoted",
     );
-  });
+  }
 
-  test("5. Full context + roadmap → not needs-discussion", async () => {
+  // ─── 5. CONTEXT-DRAFT + CONTEXT + ROADMAP → not needs-discussion ──────────
+  console.log("\n=== 5. Full context + roadmap → not needs-discussion ===");
+  {
     const base = createBase();
     try {
       writeContextDraft(base, "M001", "# Draft\nSeed discussion.");
@@ -150,14 +161,16 @@ describe("auto-start-needs-discussion (#1726)", () => {
         "# M001: Test\n\n## Slices\n- [ ] **S01: Test Slice** `risk:low` `depends:[]`\n  > After this: works\n");
       invalidateAllCaches();
       const state = await deriveState(base);
-      assert.ok(state.phase !== "needs-discussion",
+      assertTrue(state.phase !== "needs-discussion",
         "milestone with full context + roadmap should NOT be needs-discussion");
     } finally {
       cleanup(base);
     }
-  });
+  }
 
-  test("6. No infinite loop: needs-discussion always routes to showSmartEntry", () => {
+  // ─── 6. Verify the two bug conditions cannot produce infinite loop ────────
+  console.log("\n=== 6. No infinite loop: needs-discussion always routes to showSmartEntry ===");
+  {
     const source = readAutoStartSource();
 
     // Verify needs-discussion does NOT appear in auto-dispatch trigger conditions
@@ -167,7 +180,7 @@ describe("auto-start-needs-discussion (#1726)", () => {
       /\/\/ Milestone branch recovery.*?let hasSurvivorBranch = false;[\s\S]*?if\s*\([^)]*state\.phase[^)]*\)\s*\{/,
     );
     if (survivorSection) {
-      assert.ok(
+      assertTrue(
         !survivorSection[0].includes("needs-discussion"),
         "survivor branch phase condition must not mention needs-discussion",
       );
@@ -177,17 +190,19 @@ describe("auto-start-needs-discussion (#1726)", () => {
     const notSurvivorBlock = source.match(
       /if\s*\(!hasSurvivorBranch\)\s*\{([\s\S]*?)\/\/ Unreachable safety check/,
     );
-    assert.ok(!!notSurvivorBlock,
+    assertTrue(!!notSurvivorBlock,
       "found !hasSurvivorBranch block in auto-start.ts");
     if (notSurvivorBlock) {
-      assert.ok(
+      assertTrue(
         notSurvivorBlock[1].includes('"needs-discussion"'),
         "!hasSurvivorBranch block must handle needs-discussion phase",
       );
     }
-  });
+  }
 
-  test("7. Survivor branch + needs-discussion routes to showSmartEntry", () => {
+  // ─── 7. Survivor branch + needs-discussion routes to showSmartEntry (#1726) ─
+  console.log("\n=== 7. Survivor branch + needs-discussion routes to showSmartEntry ===");
+  {
     const source = readAutoStartSource();
 
     // When hasSurvivorBranch is true AND phase is needs-discussion, the code
@@ -195,24 +210,31 @@ describe("auto-start-needs-discussion (#1726)", () => {
     const survivorNeedsDiscussion = source.match(
       /if\s*\(hasSurvivorBranch\s*&&\s*state\.phase\s*===\s*"needs-discussion"\)\s*\{[^}]*showSmartEntry/s,
     );
-    assert.ok(!!survivorNeedsDiscussion,
+    assertTrue(!!survivorNeedsDiscussion,
       "hasSurvivorBranch && needs-discussion must route to showSmartEntry");
 
     // Verify the handler checks if the discussion succeeded
     const handlerBlock = source.match(
       /if\s*\(hasSurvivorBranch\s*&&\s*state\.phase\s*===\s*"needs-discussion"\)\s*\{([\s\S]*?)\n    \}/,
     );
-    assert.ok(!!handlerBlock,
+    assertTrue(!!handlerBlock,
       "found survivor + needs-discussion handler block");
     if (handlerBlock) {
-      assert.ok(
+      assertTrue(
         handlerBlock[1].includes('postState.phase !== "needs-discussion"'),
         "handler must check if phase advanced after discussion",
       );
-      assert.ok(
+      assertTrue(
         handlerBlock[1].includes("releaseLockAndReturn"),
         "handler must abort if discussion didn't promote draft",
       );
     }
-  });
+  }
+
+  report();
+}
+
+main().catch((err) => {
+  console.error(err);
+  process.exit(1);
 });