@opengsd/gsd-pi 1.0.1 → 1.0.2-dev.0ee79e5

package/src/resources/extensions/gsd/tests/originalbase-path-comparison.test.ts

@@ -1,4 +1,4 @@
-// GSD-2 — Regression tests for originalBase path comparison correctness (M5 fix)
+// gsd-pi — Regression tests for originalBase path comparison correctness (M5 fix)
 //
 // After commit ade55a7f5, getAutoWorktreeOriginalBase() returns ws.projectRoot
 // which is realpath-normalized. Callers that compare it with string === against

package/src/resources/extensions/gsd/tests/orphan-merge-bootstrap.test.ts

@@ -1,4 +1,4 @@
-// GSD-2 + src/resources/extensions/gsd/tests/orphan-merge-bootstrap.test.ts
+// gsd-pi + src/resources/extensions/gsd/tests/orphan-merge-bootstrap.test.ts
 // Regression: bootstrap must actively merge orphan completed-but-unmerged
 // milestones, not just seed `s.currentMilestoneId` (the seed approach was
 // silently overwritten at auto-start.ts:948 — caught in audit of PR #5549).

package/src/resources/extensions/gsd/tests/orphan-stash-audit.test.ts

@@ -1,4 +1,4 @@
-// GSD-2 + src/resources/extensions/gsd/tests/orphan-stash-audit.test.ts
+// gsd-pi + src/resources/extensions/gsd/tests/orphan-stash-audit.test.ts
 // Regression: orphaned gsd-preflight-stash entries from completed milestones
 // must be auto-applied at startup so the user's pre-merge work returns.
 

package/src/resources/extensions/gsd/tests/orphaned-worktree-audit.test.ts

@@ -1,4 +1,4 @@
-// GSD2 — Tests for auditOrphanedMilestoneBranches bootstrap audit
+// gsd-pi — Tests for auditOrphanedMilestoneBranches bootstrap audit
 import { describe, test, beforeEach, afterEach } from "node:test";
 import assert from "node:assert/strict";
 import { mkdtempSync, mkdirSync, writeFileSync, rmSync, existsSync, realpathSync } from "node:fs";

package/src/resources/extensions/gsd/tests/parallel-milestone-isolation.test.ts

@@ -1,4 +1,4 @@
-// gsd-2 + Parallel-worker isolation regression (Phase B coordination)
+// gsd-pi + Parallel-worker isolation regression (Phase B coordination)
 //
 // Two simulated workers attempt to claim leases on the same project. The
 // lease infrastructure must guarantee:
@@ -41,7 +41,7 @@ test("two workers contesting the same milestone: only one wins the lease", (t) =
   insertMilestone({ id: "M001", title: "Contested", status: "active" });
 
   const w1 = registerAutoWorker({ projectRootRealpath: base });
-  const w2 = registerAutoWorker({ projectRootRealpath: base });
+  const w2 = registerAutoWorker({ projectRootRealpath: join(base, "other-project") });
 
   const r1 = claimMilestoneLease(w1, "M001");
   const r2 = claimMilestoneLease(w2, "M001");

package/src/resources/extensions/gsd/tests/parallel-monitor-overlay.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Regression tests for parallel monitor overlay rendering and input handling.
 
 import { describe, it } from "node:test";

package/src/resources/extensions/gsd/tests/parallel-orchestrator-fast-forward.test.ts

@@ -1,4 +1,4 @@
-// GSD-2 + src/resources/extensions/gsd/tests/parallel-orchestrator-fast-forward.test.ts
+// gsd-pi + src/resources/extensions/gsd/tests/parallel-orchestrator-fast-forward.test.ts
 // Regression: parallel-orchestrator's `_createMilestoneWorktree` must
 // fast-forward a reused milestone branch onto integration before creating
 // the worktree, matching the behavior added to the auto-mode path in

package/src/resources/extensions/gsd/tests/park-milestone.test.ts

@@ -5,7 +5,7 @@ import { join } from 'node:path';
 import { tmpdir } from 'node:os';
 import { execSync } from 'node:child_process';
 
-import { deriveState, invalidateStateCache, getActiveMilestoneId } from '../state.ts';
+import { invalidateStateCache, getActiveMilestoneId } from '../state.ts';
 import { clearPathCache } from '../paths.ts';
 import { parkMilestone, unparkMilestone, discardMilestone, isParked, getParkedReason } from '../milestone-actions.ts';
 import {
@@ -20,10 +20,6 @@ import {
 } from "../gsd-db.ts";
 import { createWorktree } from "../worktree-manager.ts";
 
-// This suite exercises the explicit legacy markdown derivation path.
-process.env.GSD_ALLOW_MARKDOWN_DERIVE_FALLBACK = '1';
-
-
 // ─── Fixture Helpers ───────────────────────────────────────────────────────
 
 function createFixtureBase(): string {
@@ -174,54 +170,6 @@ test('unparkMilestone fails if not parked', () => {
     }
 });
 
-  // ─── Test 5: deriveState returns 'parked' status ──────────────────────
-test('deriveState returns parked status', async () => {
-    const base = createFixtureBase();
-    try {
-      createMilestone(base, 'M001', { withRoadmap: true });
-      clearCaches();
-
-      parkMilestone(base, 'M001', 'Test reason');
-
-      const state = await deriveState(base);
-      const entry = state.registry.find(e => e.id === 'M001');
-      assert.ok(!!entry, 'M001 in registry');
-      assert.deepStrictEqual(entry?.status, 'parked', 'status is parked');
-    } finally {
-      cleanup(base);
-    }
-});
-
-  // ─── Test 6: deriveState skips parked milestone for active ─────────────
-test('deriveState skips parked milestone', async () => {
-    const base = createFixtureBase();
-    try {
-      createMilestone(base, 'M001', { withRoadmap: true });
-      createMilestone(base, 'M002', { withRoadmap: true });
-      clearCaches();
-
-      // Before park: M001 is active
-      const stateBefore = await deriveState(base);
-      assert.deepStrictEqual(stateBefore.activeMilestone?.id, 'M001', 'before park: M001 is active');
-
-      parkMilestone(base, 'M001', 'Testing');
-
-      // After park: M002 becomes active
-      const stateAfter = await deriveState(base);
-      assert.deepStrictEqual(stateAfter.activeMilestone?.id, 'M002', 'after park: M002 is active');
-
-      // M001 still in registry as parked
-      const m001 = stateAfter.registry.find(e => e.id === 'M001');
-      assert.deepStrictEqual(m001?.status, 'parked', 'M001 has parked status');
-
-      // M002 is active
-      const m002 = stateAfter.registry.find(e => e.id === 'M002');
-      assert.deepStrictEqual(m002?.status, 'active', 'M002 has active status');
-    } finally {
-      cleanup(base);
-    }
-});
-
   // ─── Test 7: getActiveMilestoneId skips parked ────────────────────────
 test('getActiveMilestoneId skips parked', async () => {
     const base = createFixtureBase();
@@ -239,51 +187,6 @@ test('getActiveMilestoneId skips parked', async () => {
     }
 });
 
-  // ─── Test 8: Parked milestone does NOT satisfy depends_on ─────────────
-test('Parked milestone does not satisfy depends_on', async () => {
-    const base = createFixtureBase();
-    try {
-      createMilestone(base, 'M001', { withRoadmap: true });
-      createMilestone(base, 'M002', { withRoadmap: true, dependsOn: ['M001'] });
-      clearCaches();
-
-      parkMilestone(base, 'M001', 'Testing');
-
-      const state = await deriveState(base);
-      // M001 is parked, M002 depends on M001 → M002 should be pending, not active
-      const m002 = state.registry.find(e => e.id === 'M002');
-      assert.deepStrictEqual(m002?.status, 'pending', 'M002 stays pending when M001 is parked');
-
-      // No active milestone (both are blocked/parked)
-      assert.deepStrictEqual(state.activeMilestone, null, 'no active milestone');
-    } finally {
-      cleanup(base);
-    }
-});
-
-  // ─── Test 9: Park then unpark restores correct status ─────────────────
-test('Park then unpark restores status', async () => {
-    const base = createFixtureBase();
-    try {
-      createMilestone(base, 'M001', { withRoadmap: true });
-      createMilestone(base, 'M002', { withRoadmap: true });
-      clearCaches();
-
-      // Park M001
-      parkMilestone(base, 'M001', 'Testing');
-      const stateParked = await deriveState(base);
-      assert.deepStrictEqual(stateParked.activeMilestone?.id, 'M002', 'while parked: M002 is active');
-
-      // Unpark M001 — M001 should become active again (it's first in queue)
-      unparkMilestone(base, 'M001');
-      const stateUnparked = await deriveState(base);
-      assert.deepStrictEqual(stateUnparked.activeMilestone?.id, 'M001', 'after unpark: M001 is active again');
-      assert.deepStrictEqual(stateUnparked.registry.find(e => e.id === 'M001')?.status, 'active', 'M001 is active status');
-    } finally {
-      cleanup(base);
-    }
-});
-
   // ─── Test 10: discardMilestone removes directory ──────────────────────
 test('discardMilestone removes directory', async () => {
     const base = createFixtureBase();
@@ -297,9 +200,6 @@ test('discardMilestone removes directory', async () => {
       const success = discardMilestone(base, 'M001');
       assert.ok(success, 'discardMilestone returns true');
       assert.ok(!existsSync(mDir), 'milestone dir removed after discard');
-
-      const state = await deriveState(base);
-      assert.ok(!state.registry.some(e => e.id === 'M001'), 'M001 not in registry after discard');
     } finally {
       cleanup(base);
     }
@@ -360,61 +260,4 @@ test('discardMilestone removes DB rows, worktree, and milestone branch', () => {
     }
 });
 
-  // ─── Test 12: All milestones parked → no active milestone ─────────────
-test('All milestones parked → no active', async () => {
-    const base = createFixtureBase();
-    try {
-      createMilestone(base, 'M001', { withRoadmap: true });
-      clearCaches();
-
-      parkMilestone(base, 'M001', 'Testing');
-
-      const state = await deriveState(base);
-      assert.deepStrictEqual(state.activeMilestone, null, 'no active milestone when all parked');
-      assert.deepStrictEqual(state.phase, 'pre-planning', 'phase is pre-planning');
-      assert.ok(state.registry.length === 1, 'registry still has 1 entry');
-      assert.deepStrictEqual(state.registry[0]?.status, 'parked', 'entry is parked');
-    } finally {
-      cleanup(base);
-    }
-});
-
-  // ─── Test 13: Parked milestone without roadmap ────────────────────────
-test('Park milestone without roadmap', async () => {
-    const base = createFixtureBase();
-    try {
-      createMilestone(base, 'M001'); // No roadmap
-      createMilestone(base, 'M002', { withRoadmap: true });
-      clearCaches();
-
-      parkMilestone(base, 'M001', 'Not ready yet');
-
-      const state = await deriveState(base);
-      assert.deepStrictEqual(state.activeMilestone?.id, 'M002', 'M002 is active when M001 (no roadmap) is parked');
-      assert.deepStrictEqual(state.registry.find(e => e.id === 'M001')?.status, 'parked', 'M001 is parked');
-    } finally {
-      cleanup(base);
-    }
-});
-
-  // ─── Test 14: Progress counts with parked milestone ───────────────────
-test('Progress counts with parked', async () => {
-    const base = createFixtureBase();
-    try {
-      createMilestone(base, 'M001', { withRoadmap: true, withSummary: true }); // complete
-      createMilestone(base, 'M002', { withRoadmap: true }); // will park
-      createMilestone(base, 'M003', { withRoadmap: true }); // will be active
-      clearCaches();
-
-      parkMilestone(base, 'M002', 'Parked');
-
-      const state = await deriveState(base);
-      assert.deepStrictEqual(state.progress?.milestones.done, 1, '1 complete milestone');
-      assert.deepStrictEqual(state.progress?.milestones.total, 3, '3 total milestones (including parked)');
-      assert.deepStrictEqual(state.activeMilestone?.id, 'M003', 'M003 is active');
-    } finally {
-      cleanup(base);
-    }
-});
-
 });

package/src/resources/extensions/gsd/tests/path-cache-decoupled.test.ts

@@ -1,4 +1,4 @@
-// GSD-2 — Tests verifying gsdRootCache is decoupled from per-turn clearPathCache()
+// gsd-pi — Tests verifying gsdRootCache is decoupled from per-turn clearPathCache()
 
 import { describe, test, beforeEach, afterEach } from 'node:test';
 import assert from 'node:assert/strict';

package/src/resources/extensions/gsd/tests/path-normalization-unified.test.ts

@@ -1,4 +1,4 @@
-// GSD-2 + Unified path normalization tests: normalizeRealPath and tryRealpath parity
+// gsd-pi + Unified path normalization tests: normalizeRealPath and tryRealpath parity
 
 import { describe, test, before, beforeEach, afterEach } from "node:test";
 import assert from "node:assert/strict";

package/src/resources/extensions/gsd/tests/paths-cache.test.ts

@@ -1,4 +1,4 @@
-// GSD2 — paths cache normalization and clearPathCache() invalidation tests
+// gsd-pi — paths cache normalization and clearPathCache() invalidation tests
 
 import { describe, test, beforeEach, afterEach } from 'node:test';
 import assert from 'node:assert/strict';

package/src/resources/extensions/gsd/tests/paused-session-via-db.test.ts

@@ -1,4 +1,4 @@
-// gsd-2 + Paused-session via runtime_kv (Phase C pt 2 — paused-session.json migration)
+// gsd-pi + Paused-session via runtime_kv (Phase C pt 2 — paused-session.json migration)
 //
 // runtime/paused-session.json is gone. The metadata that the old file
 // stored now lives in runtime_kv (global scope, key PAUSED_SESSION_KV_KEY).

package/src/resources/extensions/gsd/tests/pending-autostart-scope.test.ts

@@ -1,4 +1,4 @@
-// GSD-2 + Tests for MilestoneScope pinning in pendingAutoStartMap (C1 regression guard)
+// gsd-pi + Tests for MilestoneScope pinning in pendingAutoStartMap (C1 regression guard)
 
 import { describe, test, beforeEach, afterEach } from "node:test";
 import assert from "node:assert/strict";

package/src/resources/extensions/gsd/tests/pipeline-variant-dispatch.test.ts

@@ -1,4 +1,4 @@
-// GSD-2 — #4781 phase 2: dispatch-rule gates read pipeline variant from DB.
+// gsd-pi — #4781 phase 2: dispatch-rule gates read pipeline variant from DB.
 // Behavior tests (not source-grep) — construct a real tmpdir DB, insert a
 // milestone whose planning fields classify to the target variant, exercise
 // DISPATCH_RULES.match(), assert the gate result.

package/src/resources/extensions/gsd/tests/plan-milestone-rendering.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Verifies the milestone planning prompt renders compact required guidance.
 
 import test from "node:test";
@@ -35,6 +35,7 @@ test("plan-milestone prompt renders compact DB-backed planning guidance", async
 
   assert.match(prompt, /Explore First, Then Decompose/);
   assert.match(prompt, /Call `gsd_plan_milestone`/);
+  assert.match(prompt, /NEVER call `gsd_plan_milestone` with only `milestoneId` and `sliceId`/);
   assert.match(prompt, /call `gsd_decision_save`/);
   assert.match(prompt, /Every relevant Active requirement must end as mapped/);
   assert.match(prompt, /Risk-first means proof-first/);

package/src/resources/extensions/gsd/tests/plan-milestone-schema-recovery.test.ts

@@ -0,0 +1,59 @@
+/**
+ * Regression tests for gsd_plan_milestone schema confusion recovery.
+ * Models call gsd_plan_milestone with { milestoneId, sliceId } only (gsd_plan_slice shape).
+ */
+
+import { describe, test } from "node:test";
+import assert from "node:assert/strict";
+import {
+  buildPlanMilestoneRecoverySteering,
+  enrichPlanMilestoneValidationError,
+  isPlanMilestoneSliceIdConfusion,
+  isPlanMilestoneToolName,
+} from "../bootstrap/plan-milestone-schema-recovery.js";
+
+describe("plan-milestone schema recovery", () => {
+  test("isPlanMilestoneToolName recognizes canonical and alias names", () => {
+    assert.equal(isPlanMilestoneToolName("gsd_plan_milestone"), true);
+    assert.equal(isPlanMilestoneToolName("gsd_milestone_plan"), true);
+    assert.equal(isPlanMilestoneToolName("gsd_plan_slice"), false);
+  });
+
+  test("isPlanMilestoneSliceIdConfusion detects slice-only payload", () => {
+    assert.equal(
+      isPlanMilestoneSliceIdConfusion({ milestoneId: "M001", sliceId: "S01" }),
+      true,
+    );
+    assert.equal(
+      isPlanMilestoneSliceIdConfusion({
+        milestoneId: "M001",
+        title: "Title",
+        vision: "Vision",
+        slices: [{ sliceId: "S01" }],
+      }),
+      false,
+    );
+    assert.equal(
+      isPlanMilestoneSliceIdConfusion({ milestoneId: "M001", sliceId: "S01", title: "T" }),
+      false,
+    );
+  });
+
+  test("enrichPlanMilestoneValidationError documents required fields and minimal JSON", () => {
+    const enriched = enrichPlanMilestoneValidationError(
+      'Validation failed for tool "gsd_plan_milestone":\n  - title: must have required property',
+      { milestoneId: "M001", sliceId: "S01" },
+    );
+    assert.match(enriched, /gsd_plan_slice/);
+    assert.match(enriched, /title, vision, slices/);
+    assert.match(enriched, /"milestoneId": "M001"/);
+    assert.match(enriched, /"slices":/);
+  });
+
+  test("buildPlanMilestoneRecoverySteering references milestone and full payload", () => {
+    const steering = buildPlanMilestoneRecoverySteering("M002");
+    assert.match(steering, /M002/);
+    assert.match(steering, /title, vision/);
+    assert.match(steering, /gsd_plan_slice/);
+  });
+});

package/src/resources/extensions/gsd/tests/plan-milestone.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Regression tests for DB-backed milestone planning.
 
 import test from 'node:test';

package/src/resources/extensions/gsd/tests/plan-slice-prompt.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Verifies GSD planning prompt placeholder rendering and DB-backed tool guidance.
 
 import test from "node:test";

package/src/resources/extensions/gsd/tests/planning-depth-setter.test.ts

@@ -1,4 +1,4 @@
-// GSD-2 — Deep planning mode setPlanningDepth helper.
+// gsd-pi — Deep planning mode setPlanningDepth helper.
 // Verifies the helper correctly creates and updates .gsd/PREFERENCES.md while
 // preserving existing frontmatter keys and body content.
 

package/src/resources/extensions/gsd/tests/post-exec-retry-bypass.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Tests for post-execution retry bypass and verification gate failure handling.
 /**
  * post-exec-retry-bypass.test.ts — Tests for post-execution blocking failure retry bypass.

package/src/resources/extensions/gsd/tests/pr-evidence-equivalence.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Golden-fixture equivalence tests pinning PR-body output for buildPrEvidence and formatSwarmLanePRBody.
 
 import test from "node:test";

package/src/resources/extensions/gsd/tests/pr-evidence-hardening.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Hardening tests for buildPrEvidence — HTML-comment stripping, fake commit-trailer removal, and per-item length capping.
 
 import test from "node:test";

package/src/resources/extensions/gsd/tests/pr-evidence.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Tests for the shared GSD pull request evidence generator.
 
 import test from "node:test";

package/src/resources/extensions/gsd/tests/pre-execution-checks.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Unit tests for pre-execution validation checks.
 
 /**
@@ -1884,7 +1884,7 @@ describe("checkTaskOrdering false positive for pre-execution refs (#4071)", () =
   });
 
   test("pending-first then completed-later: completed replaces pending in fileCreators (#4572)", () => {
-    // Regression for CodeRabbit Major finding on PR #4572:
+    // Regression for PR #4572:
     // fileCreators only stored the FIRST task for a given path. If a PENDING task at
     // array index 1 was registered first, and a COMPLETED task at array index 2 also
     // declared the same output path, the completed entry was discarded. Line ~529 then

package/src/resources/extensions/gsd/tests/pre-execution-pause-wiring.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Integration tests for pre-execution check retry and pause wiring.
 /**
  * pre-execution-pause-wiring.test.ts — Integration tests for pre-execution check → retry/pause wiring.

package/src/resources/extensions/gsd/tests/prefs-wizard-coverage.test.ts

@@ -1,4 +1,4 @@
-// GSD-2 + prefs-wizard-coverage.test.ts - Behavioral coverage for preferences wizard persistence.
+// gsd-pi + prefs-wizard-coverage.test.ts - Behavioral coverage for preferences wizard persistence.
 
 import test from "node:test";
 import assert from "node:assert/strict";

package/src/resources/extensions/gsd/tests/process-task-path.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Tests for canonical process recommendations by task size.
 
 import test from "node:test";

package/src/resources/extensions/gsd/tests/prompt-contracts.test.ts

@@ -283,6 +283,16 @@ test("plan-milestone prompt references DB-backed planning tool and explicitly fo
   const prompt = readPrompt("plan-milestone");
   assert.match(prompt, /gsd_plan_milestone/);
   assert.match(prompt, /Do \*\*not\*\* write `?\{\{outputPath\}\}`?, `?ROADMAP\.md`?, or other planning artifacts manually/i);
+  assert.match(prompt, /NEVER call `gsd_plan_milestone` with only `milestoneId` and `sliceId`/);
+  assert.match(prompt, /gsd_plan_slice/);
+});
+
+test("discuss prompts forbid gsd_plan_milestone slice-only tool args", () => {
+  for (const name of ["discuss", "discuss-headless"] as const) {
+    const prompt = readPrompt(name);
+    assert.match(prompt, /NEVER call `gsd_plan_milestone` with only `milestoneId` and `sliceId`/);
+    assert.match(prompt, /title`, `vision`, `slices\[\]/);
+  }
 });
 
 test("plan-slice prompt no longer frames direct PLAN writes as the source of truth", () => {

package/src/resources/extensions/gsd/tests/prompt-duplication-cuts.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Verifies low-risk auto-prompt duplication cuts render through prompt builders.
 
 import test from "node:test";

package/src/resources/extensions/gsd/tests/provider-switch-observer.test.ts

@@ -1,4 +1,4 @@
-// GSD-2 — ADR-005 Phase 3b: ProviderSwitchObserver Tests
+// gsd-pi — ADR-005 Phase 3b: ProviderSwitchObserver Tests
 import test from "node:test";
 import assert from "node:assert/strict";
 import { mkdtempSync, readFileSync, rmSync, existsSync } from "node:fs";

package/src/resources/extensions/gsd/tests/quality-gates.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Validates planning and task template quality-gate content.
 
 import { readFileSync } from "node:fs";

package/src/resources/extensions/gsd/tests/queue-prompt-rendering.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Verifies the queue prompt renders compact discussion and write-gate guidance.
 
 import test from "node:test";

package/src/resources/extensions/gsd/tests/reactive-executor.test.ts

@@ -16,6 +16,7 @@ import {
 import { validatePreferences } from "../preferences-validation.ts";
 import type { ReactiveExecutionState } from "../types.ts";
 import { parseUnitId } from "../unit-id.ts";
+import { resolveDispatch } from "../auto-dispatch.ts";
 
 // ─── Preference Validation ────────────────────────────────────────────────
 
@@ -176,6 +177,68 @@ test("reactive dispatch requires enabled config and multiple ready tasks", async
   }
 });
 
+test("reactive dispatch falls through when slice has REACTIVE-BLOCKER", async () => {
+  const repo = mkdtempSync(join(tmpdir(), "gsd-reactive-blocker-dispatch-"));
+  try {
+    const sliceDir = join(repo, ".gsd", "milestones", "M001", "slices", "S01");
+    const tasksDir = join(sliceDir, "tasks");
+    mkdirSync(tasksDir, { recursive: true });
+    writeFileSync(
+      join(sliceDir, "S01-PLAN.md"),
+      [
+        "# S01: Test Slice",
+        "",
+        "## Tasks",
+        "",
+        "- [ ] **T01: First**",
+        "- [ ] **T02: Second**",
+        "- [ ] **T03: Third**",
+        "",
+      ].join("\n"),
+    );
+    for (const tid of ["T01", "T02", "T03"]) {
+      writeFileSync(
+        join(tasksDir, `${tid}-PLAN.md`),
+        [
+          `# ${tid}`,
+          "",
+          "## Inputs",
+          "",
+          `- \`src/${tid}.input\``,
+          "",
+          "## Expected Output",
+          "",
+          `- \`src/${tid}.output\``,
+        ].join("\n"),
+      );
+    }
+    writeFileSync(join(sliceDir, "S01-REACTIVE-BLOCKER.md"), "# BLOCKER\n");
+
+    const action = await resolveDispatch({
+      basePath: repo,
+      mid: "M001",
+      midTitle: "Milestone",
+      state: {
+        phase: "executing",
+        activeMilestone: { id: "M001", title: "Milestone", status: "active" },
+        activeSlice: { id: "S01", title: "Test Slice" },
+        activeTask: { id: "T01", title: "First" },
+        registry: [],
+        blockers: [],
+      } as any,
+      prefs: { reactive_execution: { enabled: true, max_parallel: 3 } } as any,
+    });
+
+    assert.notEqual(
+      action.action === "dispatch" ? action.unitType : null,
+      "reactive-execute",
+      "reactive blocker should prevent another reactive batch dispatch",
+    );
+  } finally {
+    rmSync(repo, { recursive: true, force: true });
+  }
+});
+
 test("reactive dispatch falls back when graph is ambiguous (task without IO)", async () => {
   const repo = mkdtempSync(join(tmpdir(), "gsd-reactive-ambiguous-"));
   try {

package/src/resources/extensions/gsd/tests/ready-phrase-no-files-4573.test.ts

@@ -1,5 +1,5 @@
 /**
- * GSD-2 / guided-flow — regression tests for #4573
+ * gsd-pi / guided-flow — regression tests for #4573
  *
  * Covers two recovery paths:
  *   - maybeHandleReadyPhraseWithoutFiles: nudge when LLM emits

package/src/resources/extensions/gsd/tests/reassess-default-optin.test.ts

@@ -1,4 +1,4 @@
-// GSD-2 — ADR-003 §4 behavior contract: reassess-roadmap is opt-in.
+// gsd-pi — ADR-003 §4 behavior contract: reassess-roadmap is opt-in.
 // Companion to (eventually replacing) the source-grep assertions in
 // token-profile.test.ts. This file verifies the dispatch rule's guard
 // behavior directly rather than inspecting source text.

package/src/resources/extensions/gsd/tests/register-extension-guard.test.ts

@@ -2,7 +2,7 @@ import test from "node:test";
 import assert from "node:assert/strict";
 import { join } from "node:path";
 import { tmpdir } from "node:os";
-import { existsSync, readdirSync, rmSync } from "node:fs";
+import { existsSync, rmSync } from "node:fs";
 import { randomUUID } from "node:crypto";
 
 import { handleRecoverableExtensionProcessError } from "../bootstrap/register-extension.ts";
@@ -94,7 +94,7 @@ test("handleRecoverableExtensionProcessError leaves unrelated errors unhandled",
   assert.equal(handled, false);
 });
 
-test("handleRecoverableExtensionProcessError swallows EPIPE, warns, and writes crash log when stdout is alive", () => {
+test("handleRecoverableExtensionProcessError swallows EPIPE without writing a crash log", () => {
   const tmpHome = join(tmpdir(), `gsd-epipe-test-${randomUUID()}`);
   const origHome = process.env.GSD_HOME;
   process.env.GSD_HOME = tmpHome;
@@ -116,8 +116,7 @@ test("handleRecoverableExtensionProcessError swallows EPIPE, warns, and writes c
     assert.equal(handled, true);
     assert.match(stderr, /swallowed EPIPE/);
     const crashDir = join(tmpHome, "crash");
-    assert.equal(existsSync(crashDir), true);
-    assert.equal(readdirSync(crashDir).some((f) => f.endsWith(".log")), true);
+    assert.equal(existsSync(crashDir), false);
   } finally {
     process.stderr.write = originalWrite;
     process.env.GSD_HOME = origHome;

package/src/resources/extensions/gsd/tests/repository-registry.test.ts

@@ -1,4 +1,4 @@
-// GSD-2 + Repository registry seam tests.
+// gsd-pi + Repository registry seam tests.
 
 import test from "node:test";
 import assert from "node:assert/strict";