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

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

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Tests auto-mode artifact verification and recovery behavior.
 import test, { afterEach } from "node:test";
 import assert from "node:assert/strict";
@@ -7,9 +7,10 @@ import { join } from "node:path";
 import { tmpdir } from "node:os";
 import { randomUUID } from "node:crypto";
 
-import { verifyExpectedArtifact, hasImplementationArtifacts, resolveExpectedArtifactPath, diagnoseExpectedArtifact, diagnoseWorktreeIntegrityFailure, buildLoopRemediationSteps, writeBlockerPlaceholder, refreshRecoveryDbForArtifact } from "../auto-recovery.ts";
+import { verifyExpectedArtifact, hasImplementationArtifacts, resolveExpectedArtifactPath, diagnoseExpectedArtifact, diagnoseWorktreeIntegrityFailure, buildLoopRemediationSteps, writeBlockerPlaceholder, refreshRecoveryDbForArtifact, writeReactiveExecuteBlocker } from "../auto-recovery.ts";
 import { resolveMilestoneFile } from "../paths.ts";
-import { openDatabase, closeDatabase, insertMilestone, insertSlice, insertGateRow, insertTask, insertAssessment, getMilestone, getMilestoneCommitAttributionShas } from "../gsd-db.ts";
+import { openDatabase, closeDatabase, insertMilestone, insertSlice, insertGateRow, insertTask, insertAssessment, getMilestone, getMilestoneCommitAttributionShas, getTask } from "../gsd-db.ts";
+import { readEvents } from "../workflow-events.ts";
 import { clearParseCache } from "../files.ts";
 import { parseRoadmap } from "../parsers-legacy.ts";
 import { invalidateAllCaches } from "../cache.ts";
@@ -893,57 +894,6 @@ test("verifyExpectedArtifact execute-task requires checked checkbox or DB status
   }
 });
 
-// ─── #793: invalidateAllCaches unblocks skip-loop ─────────────────────────
-// When the skip-loop breaker fires, it must call invalidateAllCaches() (not
-// just invalidateStateCache()) to clear path/parse caches that deriveState
-// depends on. Without this, even after cache invalidation, deriveState reads
-// stale directory listings and returns the same unit, looping forever.
-test("#793: invalidateAllCaches clears all caches so deriveState sees fresh disk state", async () => {
-  const base = makeTmpBase();
-  try {
-    const mid = "M001";
-    const sid = "S01";
-    const planDir = join(base, ".gsd", "milestones", mid, "slices", sid);
-    const tasksDir = join(planDir, "tasks");
-    mkdirSync(tasksDir, { recursive: true });
-    mkdirSync(join(base, ".gsd", "milestones", mid), { recursive: true });
-
-    writeFileSync(
-      join(base, ".gsd", "milestones", mid, `${mid}-ROADMAP.md`),
-      `# M001: Test Milestone\n\n**Vision:** test.\n\n## Slices\n\n- [ ] **${sid}: Slice One** \`risk:low\` \`depends:[]\`\n  > After this: done.\n`,
-    );
-    const planUnchecked = `# ${sid}: Slice One\n\n**Goal:** test.\n\n## Tasks\n\n- [ ] **T01: Task One** \`est:10m\`\n- [ ] **T02: Task Two** \`est:10m\`\n`;
-    writeFileSync(join(planDir, `${sid}-PLAN.md`), planUnchecked);
-    writeFileSync(join(tasksDir, "T01-PLAN.md"), "# T01: Task One\n\n**Goal:** t\n\n## Steps\n- step\n\n## Verification\n- v\n");
-    writeFileSync(join(tasksDir, "T02-PLAN.md"), "# T02: Task Two\n\n**Goal:** t\n\n## Steps\n- step\n\n## Verification\n- v\n");
-
-    // Warm all caches
-    const state1 = await deriveState(base);
-    assert.equal(state1.activeTask?.id, "T01", "initial: T01 is active");
-
-    // Simulate task completion on disk (what the LLM does)
-    const planChecked = `# ${sid}: Slice One\n\n**Goal:** test.\n\n## Tasks\n\n- [x] **T01: Task One** \`est:10m\`\n- [ ] **T02: Task Two** \`est:10m\`\n`;
-    writeFileSync(join(planDir, `${sid}-PLAN.md`), planChecked);
-    writeFileSync(join(tasksDir, "T01-SUMMARY.md"), "---\nid: T01\n---\n# Summary\n");
-
-    // invalidateStateCache alone: _stateCache cleared but path/parse caches warm
-    invalidateStateCache();
-
-    // invalidateAllCaches: all caches cleared — deriveState must re-read disk
-    invalidateAllCaches();
-    const state2 = await deriveState(base);
-
-    // After full invalidation, T01 should be complete and T02 should be next
-    assert.notEqual(state2.activeTask?.id, "T01", "#793: T01 not re-dispatched after full invalidation");
-
-    // Verify the caches are truly cleared by calling clearParseCache and clearPathCache
-    // do not throw (they should be no-ops after invalidateAllCaches already cleared them)
-    clearParseCache(); // no-op, but should not throw
-    assert.ok(true, "clearParseCache after invalidateAllCaches is safe");
-  } finally {
-    cleanup(base);
-  }
-});
 
 // ─── hasImplementationArtifacts (#1703) ───────────────────────────────────
 
@@ -955,6 +905,7 @@ function makeGitBase(): string {
   execFileSync("git", ["init", "--initial-branch=main"], { cwd: base, stdio: "ignore" });
   execFileSync("git", ["config", "user.email", "test@test.com"], { cwd: base, stdio: "ignore" });
   execFileSync("git", ["config", "user.name", "Test"], { cwd: base, stdio: "ignore" });
+  execFileSync("git", ["config", "gc.auto", "0"], { cwd: base, stdio: "ignore" });
   // Create initial commit so HEAD exists
   writeFileSync(join(base, ".gitkeep"), "");
   execFileSync("git", ["add", "."], { cwd: base, stdio: "ignore" });
@@ -1287,6 +1238,7 @@ test("hasImplementationArtifacts returns unknown for empty integration self-diff
 test("hasImplementationArtifacts uses milestone path history instead of rolling depth (#4699)", () => {
   const base = makeGitBase();
   try {
+    execFileSync("git", ["checkout", "-b", "milestone/M001"], { cwd: base, stdio: "ignore" });
     mkdirSync(join(base, ".gsd", "milestones", "M001", "slices", "S01", "tasks"), { recursive: true });
     mkdirSync(join(base, "src"), { recursive: true });
     writeFileSync(join(base, "src", "feature.ts"), "export function feature() {}");
@@ -1295,14 +1247,14 @@ test("hasImplementationArtifacts uses milestone path history instead of rolling
     execFileSync("git", ["commit", "-m", "feat: old milestone implementation\n\nGSD-Task: S01/T01"], { cwd: base, stdio: "ignore" });
 
     mkdirSync(join(base, "docs"), { recursive: true });
-    for (let i = 0; i < 205; i++) {
+    for (let i = 0; i < 25; i++) {
       writeFileSync(join(base, "docs", `note-${i}.md`), `# Note ${i}\n`);
       execFileSync("git", ["add", "."], { cwd: base, stdio: "ignore" });
       execFileSync("git", ["commit", "-m", `docs: filler ${i}`], { cwd: base, stdio: "ignore" });
     }
 
     const result = hasImplementationArtifacts(base, "M001");
-    assert.equal(result, "present", "milestone evidence should not age out after 200 unrelated commits");
+    assert.equal(result, "present", "milestone evidence should not age out beyond the old rolling-depth fallback");
   } finally {
     cleanup(base);
   }
@@ -1712,6 +1664,69 @@ test("#4068: verifyExpectedArtifact parallel-research treats PARALLEL-BLOCKER as
   }
 });
 
+test("#57: verifyExpectedArtifact reactive-execute treats REACTIVE-BLOCKER as terminal completion", () => {
+  const base = makeTmpBase();
+  try {
+    const blockerPath = resolveExpectedArtifactPath("reactive-execute", "M001/S01/reactive+T01,T02", base);
+    assert.ok(blockerPath, "reactive blocker path resolves");
+    writeFileSync(blockerPath!, "# BLOCKER\n", "utf-8");
+
+    assert.equal(
+      verifyExpectedArtifact("reactive-execute", "M001/S01/reactive+T01,T02", base),
+      true,
+      "REACTIVE-BLOCKER must satisfy verification even when task summaries are missing",
+    );
+  } finally {
+    cleanup(base);
+  }
+});
+
+test("#57: writeReactiveExecuteBlocker reconciles batch task statuses and appends events", () => {
+  const base = makeTmpBase();
+  try {
+    openDatabase(join(base, ".gsd", "gsd.db"));
+    insertMilestone({ id: "M001", title: "Milestone", status: "active" });
+    insertSlice({ id: "S01", milestoneId: "M001", title: "Slice", status: "pending" });
+    insertTask({ id: "T01", milestoneId: "M001", sliceId: "S01", title: "One", status: "pending" });
+    insertTask({ id: "T02", milestoneId: "M001", sliceId: "S01", title: "Two", status: "pending" });
+    insertTask({ id: "T03", milestoneId: "M001", sliceId: "S01", title: "Three", status: "complete" });
+    writeFileSync(
+      join(base, ".gsd", "milestones", "M001", "slices", "S01", "tasks", "T01-SUMMARY.md"),
+      "# T01 Summary\n",
+      "utf-8",
+    );
+    writeFileSync(
+      join(base, ".gsd", "milestones", "M001", "slices", "S01", "tasks", "T03-SUMMARY.md"),
+      "# T03 Summary\n",
+      "utf-8",
+    );
+
+    const recovery = writeReactiveExecuteBlocker(
+      "M001/S01/reactive+T01,T02,T03",
+      base,
+      "verification retries exhausted",
+    );
+
+    assert.ok(recovery, "recovery should run with DB available");
+    assert.deepEqual(recovery!.completedTaskIds, ["T01"]);
+    assert.deepEqual(recovery!.skippedTaskIds, ["T02"]);
+    assert.equal(getTask("M001", "S01", "T01")?.status, "complete");
+    assert.equal(getTask("M001", "S01", "T02")?.status, "skipped");
+    assert.equal(getTask("M001", "S01", "T03")?.status, "complete");
+    assert.ok(existsSync(recovery!.blockerPath), "reactive blocker should be written");
+    const blocker = readFileSync(recovery!.blockerPath, "utf-8");
+    assert.match(blocker, /Summary present\*\*: T01, T03/);
+    assert.match(blocker, /Summary missing\*\*: T02/);
+
+    const events = readEvents(join(base, ".gsd", "event-log.jsonl"));
+    assert.ok(events.some((e) => e.cmd === "complete-task" && e.params.taskId === "T01" && e.trigger_reason === "reactive-execute-blocker-recovery"));
+    assert.ok(events.some((e) => e.cmd === "skip-task" && e.params.taskId === "T02" && e.trigger_reason === "reactive-execute-blocker-recovery"));
+  } finally {
+    closeDatabase();
+    cleanup(base);
+  }
+});
+
 test("#4414: verifyExpectedArtifact parallel-research succeeds when all research-ready slices have RESEARCH", () => {
   const base = makeTmpBase();
   try {

package/src/resources/extensions/gsd/tests/auto-runtime-state.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Auto runtime state snapshot regression tests.
 
 import test from "node:test";

package/src/resources/extensions/gsd/tests/auto-session-scope.test.ts

@@ -1,4 +1,4 @@
-// GSD-2 + Tests for MilestoneScope threading through AutoSession state (C2)
+// gsd-pi + Tests for MilestoneScope threading through AutoSession state (C2)
 //
 // Strategy: construct AutoSession directly + call createWorkspace/scopeMilestone
 // to mirror the rebuildScope() helper in auto.ts — avoids importing the full

package/src/resources/extensions/gsd/tests/auto-start-clean-runtime-db-gated.test.ts

@@ -1,4 +1,4 @@
-// GSD-2 auto-start regression test: cleanStaleRuntimeUnits is DB-gated (#4663)
+// gsd-pi auto-start regression test: cleanStaleRuntimeUnits is DB-gated (#4663)
 
 import { describe, test } from "node:test";
 import assert from "node:assert/strict";

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

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Bootstrap behavior tests for completed milestone orphan merges.
 import test from "node:test";
 import assert from "node:assert/strict";

package/src/resources/extensions/gsd/tests/auto-start-time-persistence.test.ts

@@ -1,4 +1,4 @@
-// GSD2 — Verify autoStartTime is carried through dashboard state
+// gsd-pi — Verify autoStartTime is carried through dashboard state
 
 import test from "node:test";
 import assert from "node:assert/strict";

package/src/resources/extensions/gsd/tests/auto-start-validation-block.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Regression tests for auto-start validation block handling.
 
 import test from "node:test";

package/src/resources/extensions/gsd/tests/auto-status-message.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Visual contract tests for compact GSD status notification cards.
 
 import test from "node:test";

package/src/resources/extensions/gsd/tests/auto-stop-notification.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Regression tests for auto-mode stop notification formatting.
 
 import test from "node:test";

package/src/resources/extensions/gsd/tests/auto-unit-closeout.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Regression tests for auto-unit closeout activity classification.
 
 import test from "node:test";

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

@@ -1,4 +1,4 @@
-// gsd-2 + Auto-mode worker registry tests (Phase B coordination)
+// gsd-pi + Auto-mode worker registry tests (Phase B coordination)
 
 import test from "node:test";
 import assert from "node:assert/strict";

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

@@ -1,4 +1,4 @@
-// GSD-2 + Unit tests for the workspace registry that replaced the originalBase singleton
+// gsd-pi + Unit tests for the workspace registry that replaced the originalBase singleton
 
 import { describe, test, beforeEach } from "node:test";
 import assert from "node:assert/strict";

package/src/resources/extensions/gsd/tests/auto-worktree-repair.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Regression tests for safe auto-mode milestone worktree repair.
 
 import test from "node:test";

package/src/resources/extensions/gsd/tests/auto-wrapup-inflight-guard.test.ts

@@ -1,4 +1,4 @@
-// GSD-2 — Regression tests for #3512: gsd-auto-wrapup mid-turn interruption
+// gsd-pi — Regression tests for #3512: gsd-auto-wrapup mid-turn interruption
 // Copyright (c) 2026 Jeremy McSpadden <jeremy@fluxlabs.net>
 
 import { describe, test } from "node:test";

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

@@ -1,4 +1,4 @@
-// GSD-2 + /gsd brief command behavior tests
+// gsd-pi + /gsd brief command behavior tests
 
 import test from "node:test";
 import assert from "node:assert/strict";

package/src/resources/extensions/gsd/tests/browser-tools-compatibility-declarations.test.ts

@@ -1,4 +1,4 @@
-// GSD-2 — ADR-005 Phase 2: Verify browser tool compatibility declarations.
+// gsd-pi — ADR-005 Phase 2: Verify browser tool compatibility declarations.
 //
 // Locks in the declarations that always-image-producing browser tools
 // (browser_screenshot, browser_zoom_region) carry `producesImages: true` so

package/src/resources/extensions/gsd/tests/bundled-skill-triggers.test.ts

@@ -1,4 +1,4 @@
-// GSD-2 — BUNDLED_SKILL_TRIGGERS regression test
+// gsd-pi — BUNDLED_SKILL_TRIGGERS regression test
 //
 // Guards the skill-trigger table in system-context.ts against accidental
 // regression. Every entry must have a non-empty trigger + skill, and the

package/src/resources/extensions/gsd/tests/check-auto-start-pending-gate.test.ts

@@ -1,4 +1,4 @@
-// GSD-2 + Regression tests for checkAutoStartAfterDiscuss Gate 1a (R2)
+// gsd-pi + Regression tests for checkAutoStartAfterDiscuss Gate 1a (R2)
 //
 // When a depth-verification gate is still pending (the LLM emitted the
 // confirmation question via ask_user_questions or plain chat but the user has

package/src/resources/extensions/gsd/tests/check-auto-start-ready-guard.test.ts

@@ -1,4 +1,4 @@
-// GSD-2 + Regression tests for checkAutoStartAfterDiscuss "ready" notify guard (R3b)
+// gsd-pi + Regression tests for checkAutoStartAfterDiscuss "ready" notify guard (R3b)
 //
 // Belt-and-suspenders: even when CONTEXT.md and STATE.md exist on disk, the
 // "Milestone X ready." success notify must not fire when the milestone DB row

package/src/resources/extensions/gsd/tests/claude-skill-dirs.test.ts

@@ -2,7 +2,8 @@
  * Tests for Claude Code skill directory support in getSkillSearchDirs().
  *
  * Verifies that ~/.claude/skills/ and .claude/skills/ are included in
- * the skill search path alongside ~/.agents/skills/ and .agents/skills/.
+ * the skill search path alongside GSD bundled skills, ~/.agents/skills/,
+ * and .agents/skills/.
  */
 
 import { describe, test } from "node:test";
@@ -10,10 +11,18 @@ import assert from "node:assert/strict";
 import { join } from "node:path";
 import { homedir } from "node:os";
 import { getSkillSearchDirs } from "../preferences-skills.ts";
+import { gsdHome } from "../gsd-home.ts";
 
 describe("getSkillSearchDirs — Claude Code directory support", () => {
   const cwd = "/tmp/test-project";
 
+  test("includes ~/.gsd/agent/skills/ as bundled user-skill", () => {
+    const dirs = getSkillSearchDirs(cwd);
+    const gsd = dirs.find((d) => d.dir === join(gsdHome(), "agent", "skills"));
+    assert.ok(gsd, "should include ~/.gsd/agent/skills/");
+    assert.equal(gsd!.method, "user-skill");
+  });
+
   test("includes ~/.agents/skills/ as user-skill", () => {
     const dirs = getSkillSearchDirs(cwd);
     const agents = dirs.find((d) => d.dir === join(homedir(), ".agents", "skills"));
@@ -42,10 +51,12 @@ describe("getSkillSearchDirs — Claude Code directory support", () => {
     assert.equal(projectClaude!.method, "project-skill");
   });
 
-  test("~/.agents/skills/ appears before ~/.claude/skills/ (priority order)", () => {
+  test("~/.gsd/agent/skills/ appears before ecosystem and Claude dirs (priority order)", () => {
     const dirs = getSkillSearchDirs(cwd);
+    const gsdIdx = dirs.findIndex((d) => d.dir === join(gsdHome(), "agent", "skills"));
     const agentsIdx = dirs.findIndex((d) => d.dir === join(homedir(), ".agents", "skills"));
     const claudeIdx = dirs.findIndex((d) => d.dir === join(homedir(), ".claude", "skills"));
+    assert.ok(gsdIdx < agentsIdx, "~/.gsd/agent/skills/ should have higher priority than ~/.agents/skills/");
     assert.ok(agentsIdx < claudeIdx, "~/.agents/skills/ should have higher priority than ~/.claude/skills/");
   });
 });

package/src/resources/extensions/gsd/tests/closeout-git-deferral.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Tests closeout git action deferral policy for auto-mode units.
 
 import test from "node:test";

package/src/resources/extensions/gsd/tests/closeout-recovery.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Closeout git recovery regression tests.
 
 import test from "node:test";

package/src/resources/extensions/gsd/tests/cmux.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Unit tests for cmux integration, layout, and CLI isolation.
 import test, { describe, beforeEach, afterEach } from "node:test";
 import assert from "node:assert/strict";

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

@@ -1,4 +1,4 @@
-// gsd-2 + Command queue tests (Phase B coordination — IPC inbox + broadcast NULL semantics)
+// gsd-pi + Command queue tests (Phase B coordination — IPC inbox + broadcast NULL semantics)
 
 import test from "node:test";
 import assert from "node:assert/strict";

package/src/resources/extensions/gsd/tests/commands-dispatcher-unmerged-milestone.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Dispatcher regression tests for completed-but-unmerged milestone branches.
 
 import test from "node:test";

package/src/resources/extensions/gsd/tests/commands-dispatcher-validation-block.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Dispatcher regression tests for validation-blocked milestones.
 
 import test from "node:test";

package/src/resources/extensions/gsd/tests/commands-extensions-version-compare.test.ts

@@ -1,4 +1,4 @@
-// GSD2 — Regression test for Issue #4946 isVersionGreater (commands-extensions.ts)
+// gsd-pi — Regression test for Issue #4946 isVersionGreater (commands-extensions.ts)
 //
 // Covers the inline npm-version comparator that replaced the `semver` import in
 // commands-extensions.ts. The original import broke `tsc -p tsconfig.json` whenever

package/src/resources/extensions/gsd/tests/commands-extract-learnings.test.ts

@@ -1,4 +1,4 @@
-// GSD2 commands-extract-learnings tests
+// gsd-pi commands-extract-learnings tests
 import { describe, it, beforeEach, afterEach } from "node:test";
 import assert from "node:assert/strict";
 import { mkdirSync, writeFileSync, rmSync, readFileSync } from "node:fs";

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

@@ -1,4 +1,4 @@
-// GSD-2 — #4780: slice-summary excerpts replace full inlining in
+// gsd-pi — #4780: slice-summary excerpts replace full inlining in
 // buildCompleteMilestonePrompt. Verify (a) the excerpt helper emits
 // frontmatter fields + section heads + on-demand path, (b) the closer
 // prompt lists all slice SUMMARY paths under "On-demand Slice Summaries",
@@ -196,13 +196,13 @@ test("#4780 excerpt: missing file reports not-found fallback", async (t) => {
   assert.match(out, /not found — file does not exist yet/);
 });
 
-test("#4780 excerpt: section bodies are capped (coderabbit review)", async (t) => {
+test("#4780 excerpt: section bodies are capped", async (t) => {
   const base = createBase();
   t.after(() => cleanup(base));
   invalidateAllCaches();
 
   // Long Follow-ups section (~4.8KB) would balloon the excerpt without
-  // the cap — regression coverage for the coderabbit finding on #4908.
+  // the cap — regression coverage for #4908.
   const longFollowUps = "A verbose follow-up bullet that keeps restating the same point. ".repeat(60);
   const content = [
     "---",

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

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Verifies the complete milestone prompt renders required completion and verification guardrails.
 
 import test from "node:test";

package/src/resources/extensions/gsd/tests/complete-slice-composer.test.ts

@@ -1,4 +1,4 @@
-// GSD-2 — #4782 phase 3 batch 3: complete-slice migrated through composer.
+// gsd-pi — #4782 phase 3 batch 3: complete-slice migrated through composer.
 
 import test from "node:test";
 import assert from "node:assert/strict";

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

@@ -11,6 +11,7 @@ import {
   insertSlice,
   insertTask,
   getSlice,
+  getMilestone,
   updateSliceStatus,
   getSliceTasks,
   setSliceSummaryMd,
@@ -269,6 +270,32 @@ console.log('\n=== complete-slice: handler happy path ===');
   cleanup(dbPath);
 }
 
+// ═══════════════════════════════════════════════════════════════════════════
+// complete-slice: final slice promotes planned milestone before validation
+// ═══════════════════════════════════════════════════════════════════════════
+
+console.log('\n=== complete-slice: final slice promotes planned milestone ===');
+{
+  const dbPath = tempDbPath();
+  openDatabase(dbPath);
+
+  const { basePath } = createTempProject();
+
+  insertMilestone({ id: 'M001', title: 'Test Milestone', status: 'planned' });
+  insertSlice({ id: 'S01', milestoneId: 'M001', title: 'Test Slice', risk: 'medium', sequence: 1 });
+  insertTask({ id: 'T01', sliceId: 'S01', milestoneId: 'M001', status: 'complete', title: 'Task 1' });
+
+  const result = await handleCompleteSlice(makeValidSliceParams(), basePath);
+
+  assertTrue(!('error' in result), 'final slice completion should succeed');
+  const milestone = getMilestone('M001');
+  assertTrue(milestone !== null, 'milestone should exist after completion');
+  assertEq(milestone!.status, 'active', 'final slice should transition a planned milestone to active');
+
+  cleanupDir(basePath);
+  cleanup(dbPath);
+}
+
 // ═══════════════════════════════════════════════════════════════════════════
 // complete-slice: Handler rejects incomplete tasks
 // ═══════════════════════════════════════════════════════════════════════════

package/src/resources/extensions/gsd/tests/component-loader.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Verifies component loading across modern and legacy formats.
 
 import { describe, it, beforeEach, afterEach } from 'node:test';

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

@@ -1,4 +1,4 @@
-// GSD-2 + context-store.test.ts — Regression coverage for DB-backed context query helpers.
+// gsd-pi + context-store.test.ts — Regression coverage for DB-backed context query helpers.
 
 import { describe, test, afterEach } from "node:test";
 import assert from "node:assert/strict";

package/src/resources/extensions/gsd/tests/crash-handler-secondary.test.ts

@@ -49,6 +49,28 @@ describe('register-extension crash handler secondary fixes (#3348)', () => {
     }
   });
 
+  test('writeCrashLog appends repeated crashes from one process to a single file', async () => {
+    const tmpHome = join(tmpdir(), `gsd-crash-test-${randomUUID()}`);
+    const origHome = process.env.GSD_HOME;
+    process.env.GSD_HOME = tmpHome;
+    try {
+      const { writeCrashLog } = await import('../bootstrap/crash-log.ts');
+      writeCrashLog(new Error('first crash'), 'uncaughtException');
+      writeCrashLog(new Error('second crash'), 'unhandledRejection');
+
+      const crashDir = join(tmpHome, 'crash');
+      const logs = readdirSync(crashDir).filter((f) => f.endsWith('.log'));
+      assert.equal(logs.length, 1, 'repeated writes in one process should share one crash log');
+
+      const content = readFileSync(join(crashDir, logs[0]), 'utf-8');
+      assert.ok(content.includes('first crash'), 'log should contain first error message');
+      assert.ok(content.includes('second crash'), 'log should contain second error message');
+    } finally {
+      process.env.GSD_HOME = origHome;
+      rmSync(tmpHome, { recursive: true, force: true });
+    }
+  });
+
   test('_gsdRejectionGuard is registered for unhandledRejection', () => {
     installEpipeGuard();
     const listener = process.listeners("unhandledRejection").find((candidate) =>

package/src/resources/extensions/gsd/tests/crash-recovery-via-db.test.ts

@@ -1,4 +1,4 @@
-// gsd-2 + Crash recovery via DB (Phase C pt 2 — auto.lock migration)
+// gsd-pi + Crash recovery via DB (Phase C pt 2 — auto.lock migration)
 //
 // auto.lock file IO is gone. readCrashLock now synthesizes a LockData
 // from the workers + unit_dispatches + runtime_kv tables. These tests

package/src/resources/extensions/gsd/tests/custom-verify-retry-store.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Unit tests for custom workflow verification retry persistence.
 
 import assert from "node:assert/strict";

package/src/resources/extensions/gsd/tests/db-access-guardrails.test.ts

@@ -1,4 +1,4 @@
-// GSD2 — Regression tests: DB anti-pattern guardrails in prompt templates
+// gsd-pi — Regression tests: DB anti-pattern guardrails in prompt templates
 
 import test from "node:test";
 import assert from "node:assert/strict";

package/src/resources/extensions/gsd/tests/db-adapter.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Unit tests for the normalized SQLite adapter wrapper.
 
 import assert from "node:assert/strict";

package/src/resources/extensions/gsd/tests/db-authority-regression.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Regression tests for DB authority over markdown projections.
 
 import test from "node:test";

package/src/resources/extensions/gsd/tests/db-base-schema.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Tests for base GSD database schema DDL helper.
 
 import { describe, test } from "node:test";

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

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Tests for workspace database connection cache lifecycle helpers.
 
 import { describe, test } from "node:test";

package/src/resources/extensions/gsd/tests/db-coordination-schema.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Tests for auto-mode coordination schema helper.
 
 import { describe, test } from "node:test";

package/src/resources/extensions/gsd/tests/db-decision-requirement-rows.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Tests for decision and requirement database row mappers.
 
 import { describe, test } from "node:test";

package/src/resources/extensions/gsd/tests/db-gate-rows.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Tests for quality gate database row mapper.
 
 import { describe, test } from "node:test";

package/src/resources/extensions/gsd/tests/db-lightweight-query-rows.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Tests for lightweight DB query row mappers.
 
 import { describe, test } from "node:test";

package/src/resources/extensions/gsd/tests/db-memory-fts-schema.test.ts

@@ -1,4 +1,4 @@
-// Project/App: GSD-2
+// Project/App: gsd-pi
 // File Purpose: Tests for memory FTS5 schema helpers.
 
 import { describe, test } from "node:test";