gsd-pi 2.80.0-dev.710c06e97 → 2.80.0-dev.7582bf37a

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

@@ -1,345 +1,138 @@
-/**
- * Triage dispatch ordering contract tests.
- *
- * These tests verify structural invariants of the triage integration
- * by inspecting the actual source code of auto-post-unit.ts, auto.ts,
- * and post-unit-hooks.ts. Full behavioral testing requires the
- * @gsd/pi-coding-agent runtime.
- */
+// GSD-2 — Triage and quick-task dispatch behavior tests.
 
 import test from "node:test";
 import assert from "node:assert/strict";
-import { readFileSync } from "node:fs";
-import { join, dirname } from "node:path";
-import { fileURLToPath } from "node:url";
-
-const __dirname = dirname(fileURLToPath(import.meta.url));
-const hooksPath = join(__dirname, "..", "post-unit-hooks.ts");
-const registryPath = join(__dirname, "..", "rule-registry.ts");
-const autoPromptsPath = join(__dirname, "..", "auto-prompts.ts");
-
-// After decomposition, triage/dispatch logic lives in auto-post-unit.ts
-const postUnitSrc = readFileSync(join(__dirname, "..", "auto-post-unit.ts"), "utf-8");
-// auto.ts retains top-level orchestration and imports
-const autoSrc = [
-  readFileSync(join(__dirname, "..", "auto.ts"), "utf-8"),
-  postUnitSrc,
-  readFileSync(join(__dirname, "..", "auto-start.ts"), "utf-8"),
-].join("\n");
-// Hook exclusion logic lives in the rule-registry (facade delegates there)
-const hooksSrc = [
-  readFileSync(hooksPath, "utf-8"),
-  readFileSync(registryPath, "utf-8"),
-].join("\n");
-const autoPromptsSrc = (() => { try { return readFileSync(autoPromptsPath, "utf-8"); } catch { return autoSrc; } })();
-
-// ─── Hook exclusion ──────────────────────────────────────────────────────────
-
-test("dispatch: triage-captures excluded from post-unit hook triggering", () => {
-  assert.ok(
-    hooksSrc.includes('"triage-captures"'),
-    "post-unit-hooks.ts should reference triage-captures",
-  );
-  assert.ok(
-    hooksSrc.includes('completedUnitType === "triage-captures"'),
-    "should check for triage-captures in the hook exclusion guard",
-  );
-});
-
-// ─── Triage check placement ──────────────────────────────────────────────────
-
-test("dispatch: triage check appears after hook section and before stepMode check", () => {
-  const triageCheckIndex = postUnitSrc.indexOf("// ── Triage check");
-  const quickTaskIndex = postUnitSrc.indexOf("// ── Quick-task dispatch");
-  const stepModeIndex = postUnitSrc.indexOf("if (s.stepMode)");
-
-  assert.ok(triageCheckIndex > 0, "triage check block should exist");
-  assert.ok(quickTaskIndex > 0, "quick-task dispatch block should exist");
-  assert.ok(stepModeIndex > 0, "step mode check should exist");
-
-  assert.ok(
-    triageCheckIndex < quickTaskIndex,
-    "triage check should come before quick-task dispatch",
-  );
-  assert.ok(
-    quickTaskIndex < stepModeIndex,
-    "quick-task dispatch should come before stepMode check",
-  );
-});
-
-// ─── Guard conditions ────────────────────────────────────────────────────────
-
-test("dispatch: triage check guards against step mode", () => {
-  const triageBlock = postUnitSrc.slice(
-    postUnitSrc.indexOf("// ── Triage check"),
-    postUnitSrc.indexOf("// ── Quick-task dispatch"),
-  );
-  assert.ok(
-    triageBlock.includes("!s.stepMode"),
-    "triage block should guard against step mode",
-  );
-});
-
-test("dispatch: triage check guards against hook unit types", () => {
-  const triageBlock = postUnitSrc.slice(
-    postUnitSrc.indexOf("// ── Triage check"),
-    postUnitSrc.indexOf("// ── Quick-task dispatch"),
-  );
-  assert.ok(
-    triageBlock.includes('!s.currentUnit.type.startsWith("hook/")'),
-    "triage block should not fire for hook units",
-  );
-});
-
-test("dispatch: triage check guards against triage-on-triage", () => {
-  const triageBlock = postUnitSrc.slice(
-    postUnitSrc.indexOf("// ── Triage check"),
-    postUnitSrc.indexOf("// ── Quick-task dispatch"),
-  );
-  assert.ok(
-    triageBlock.includes('s.currentUnit.type !== "triage-captures"'),
-    "triage block should not fire for triage units",
-  );
-});
-
-test("dispatch: triage check guards against quick-task triggering triage", () => {
-  const triageBlock = postUnitSrc.slice(
-    postUnitSrc.indexOf("// ── Triage check"),
-    postUnitSrc.indexOf("// ── Quick-task dispatch"),
-  );
-  assert.ok(
-    triageBlock.includes('s.currentUnit.type !== "quick-task"'),
-    "triage block should not fire for quick-task units",
-  );
-});
-
-test("dispatch: triage dispatch keeps the loop in continue mode", () => {
-  const triageBlock = postUnitSrc.slice(
-    postUnitSrc.indexOf("// ── Triage check"),
-    postUnitSrc.indexOf("// ── Quick-task dispatch"),
-  );
-  assert.ok(
-    triageBlock.includes('return "continue"') || triageBlock.includes("return enqueueSidecar("),
-    "triage dispatch should return 'continue' after enqueuing sidecar work",
-  );
-});
-
-test("dispatch: triage imports hasPendingCaptures and loadPendingCaptures", () => {
-  assert.ok(
-    autoSrc.includes("hasPendingCaptures") && autoSrc.includes("loadPendingCaptures"),
-    "should import capture functions",
-  );
-  assert.ok(
-    autoSrc.includes('from "./captures.js"'),
-    "should import from captures module",
-  );
-});
-
-// ─── Prompt integration ──────────────────────────────────────────────────────
-
-test("dispatch: replan prompt builder loads capture context", () => {
-  const src = autoPromptsSrc;
-  assert.ok(
-    src.includes("loadReplanCaptures"),
-    "buildReplanSlicePrompt should load replan captures",
-  );
-  assert.ok(
-    src.includes("captureContext"),
-    "buildReplanSlicePrompt should pass captureContext to template",
-  );
-});
-
-test("dispatch: reassess prompt builder loads deferred captures", () => {
-  const src = autoPromptsSrc;
-  assert.ok(
-    src.includes("loadDeferredCaptures"),
-    "buildReassessRoadmapPrompt should load deferred captures",
-  );
-  assert.ok(
-    src.includes("deferredCaptures"),
-    "buildReassessRoadmapPrompt should pass deferredCaptures to template",
-  );
-});
-
-// ─── Prompt templates ────────────────────────────────────────────────────────
-
-test("dispatch: replan prompt template includes captureContext variable", () => {
-  const promptPath = join(__dirname, "..", "prompts", "replan-slice.md");
-  const prompt = readFileSync(promptPath, "utf-8");
-  assert.ok(
-    prompt.includes("{{captureContext}}"),
-    "replan-slice.md should include {{captureContext}}",
-  );
-});
-
-test("dispatch: reassess prompt template includes deferredCaptures variable", () => {
-  const promptPath = join(__dirname, "..", "prompts", "reassess-roadmap.md");
-  const prompt = readFileSync(promptPath, "utf-8");
-  assert.ok(
-    prompt.includes("{{deferredCaptures}}"),
-    "reassess-roadmap.md should include {{deferredCaptures}}",
-  );
-});
-
-test("dispatch: triage prompt template exists and has classification criteria", () => {
-  const promptPath = join(__dirname, "..", "prompts", "triage-captures.md");
-  const prompt = readFileSync(promptPath, "utf-8");
-  assert.ok(prompt.includes("quick-task"), "should have quick-task classification");
-  assert.ok(prompt.includes("inject"), "should have inject classification");
-  assert.ok(prompt.includes("defer"), "should have defer classification");
-  assert.ok(prompt.includes("replan"), "should have replan classification");
-  assert.ok(prompt.includes("note"), "should have note classification");
-  assert.ok(prompt.includes("{{pendingCaptures}}"), "should have pending captures variable");
-});
-
-// ─── Dashboard integration ───────────────────────────────────────────────────
-
-test("dashboard: AutoDashboardData includes pendingCaptureCount field", () => {
-  assert.ok(
-    autoSrc.includes("pendingCaptureCount"),
-    "auto.ts should have pendingCaptureCount in AutoDashboardData",
-  );
-});
-
-test("dashboard: getAutoDashboardData computes pendingCaptureCount", () => {
-  assert.ok(
-    autoSrc.includes("pendingCaptureCount = countPendingCaptures") ||
-    autoSrc.includes("pendingCaptureCount = countPendingCaptures(basePath)"),
-    "getAutoDashboardData should compute pendingCaptureCount from countPendingCaptures (single-read)",
-  );
-});
-
-test("dashboard: overlay renders pending captures badge", () => {
-  const overlayPath = join(__dirname, "..", "dashboard-overlay.ts");
-  const overlaySrc = readFileSync(overlayPath, "utf-8");
-  assert.ok(
-    overlaySrc.includes("pendingCaptureCount"),
-    "dashboard-overlay.ts should reference pendingCaptureCount",
-  );
-  assert.ok(
-    overlaySrc.includes("pending capture"),
-    "dashboard-overlay.ts should show pending captures text",
-  );
-});
-
-test("dashboard: overlay labels triage-captures and quick-task unit types", () => {
-  const overlayPath = join(__dirname, "..", "dashboard-overlay.ts");
-  const overlaySrc = readFileSync(overlayPath, "utf-8");
-  assert.ok(
-    overlaySrc.includes('"triage-captures"'),
-    "unitLabel should handle triage-captures",
-  );
-  assert.ok(
-    overlaySrc.includes('"quick-task"'),
-    "unitLabel should handle quick-task",
-  );
-});
-
-// ─── Post-triage resolution execution ─────────────────────────────────────────
-
-test("dispatch: post-triage resolution executor fires after triage-captures unit", () => {
-  const postTriageBlock = postUnitSrc.slice(
-    postUnitSrc.indexOf("Post-triage: execute actionable resolutions"),
-  );
-  assert.ok(
-    postTriageBlock.includes('s.currentUnit.type === "triage-captures"'),
-    "should check for triage-captures unit completion",
-  );
-  assert.ok(
-    postTriageBlock.includes("executeTriageResolutions"),
-    "should call executeTriageResolutions",
-  );
-});
-
-test("dispatch: post-triage executor handles inject results", () => {
-  const postTriageBlock = postUnitSrc.slice(
-    postUnitSrc.indexOf("Post-triage: execute actionable resolutions"),
-  );
-  assert.ok(
-    postTriageBlock.includes("triageResult.injected"),
-    "should check injected count",
-  );
-});
-
-test("dispatch: post-triage executor handles replan results", () => {
-  const postTriageBlock = postUnitSrc.slice(
-    postUnitSrc.indexOf("Post-triage: execute actionable resolutions"),
-  );
-  assert.ok(
-    postTriageBlock.includes("triageResult.replanned"),
-    "should check replanned count",
-  );
-});
-
-test("dispatch: post-triage executor queues quick-tasks", () => {
-  const postTriageBlock = postUnitSrc.slice(
-    postUnitSrc.indexOf("Post-triage: execute actionable resolutions"),
-  );
-  assert.ok(
-    postTriageBlock.includes("s.pendingQuickTasks"),
-    "should push quick-tasks to s.pendingQuickTasks queue",
-  );
-});
-
-// ─── Quick-task dispatch ──────────────────────────────────────────────────────
-
-test("dispatch: quick-task dispatch block exists after triage check", () => {
-  const quickTaskBlock = postUnitSrc.indexOf("// ── Quick-task dispatch");
-  const triageBlock = postUnitSrc.indexOf("// ── Triage check");
-
-  assert.ok(quickTaskBlock > 0, "quick-task dispatch block should exist");
-  assert.ok(
-    quickTaskBlock > triageBlock,
-    "quick-task dispatch should come after triage check",
-  );
-});
-
-test("dispatch: quick-task dispatch uses buildQuickTaskPrompt", () => {
-  const quickTaskSection = postUnitSrc.slice(
-    postUnitSrc.indexOf("// ── Quick-task dispatch"),
-  );
-  assert.ok(
-    quickTaskSection.includes("buildQuickTaskPrompt"),
-    "should call buildQuickTaskPrompt for quick-task dispatch",
-  );
-});
-
-test("dispatch: quick-task dispatch marks capture as executed", () => {
-  const quickTaskSection = postUnitSrc.slice(
-    postUnitSrc.indexOf("// ── Quick-task dispatch"),
-  );
-  assert.ok(
-    quickTaskSection.includes("markCaptureExecuted"),
-    "should mark capture as executed after dispatch",
-  );
-});
-
-test("dispatch: quick-task dispatch keeps the loop in continue mode", () => {
-  const quickTaskSection = postUnitSrc.slice(
-    postUnitSrc.indexOf("// ── Quick-task dispatch"),
-    postUnitSrc.indexOf("if (s.stepMode)"),
-  );
-  assert.ok(
-    quickTaskSection.includes('return "continue"') || quickTaskSection.includes("return enqueueSidecar("),
-    "quick-task dispatch should return 'continue' after enqueuing sidecar work",
-  );
-});
-
-// ─── Post-unit hook exclusion for quick-task ──────────────────────────────────
-
-test("dispatch: quick-task excluded from post-unit hook triggering", () => {
-  assert.ok(
-    hooksSrc.includes('"quick-task"'),
-    "post-unit-hooks.ts should reference quick-task",
-  );
-});
-
-// ─── pendingQuickTasks queue lifecycle ────────────────────────────────────────
-
-test("dispatch: pendingQuickTasks queue is reset on auto-mode start/stop", () => {
-  const resetMatches = autoSrc.match(/s\.pendingQuickTasks = \[\]/g);
-  assert.ok(
-    resetMatches && resetMatches.length >= 2,
-    "s.pendingQuickTasks should be reset in start and stop paths",
-  );
+import { mkdtempSync, mkdirSync, rmSync, writeFileSync } from "node:fs";
+import { join } from "node:path";
+import { tmpdir } from "node:os";
+
+import {
+  appendCapture,
+  countPendingCaptures,
+  hasPendingCaptures,
+  loadPendingCaptures,
+  markCaptureExecuted,
+  markCaptureResolved,
+} from "../captures.ts";
+import { checkPostUnitHooks } from "../post-unit-hooks.ts";
+import {
+  _shouldDispatchQuickTaskForTest,
+  _shouldDispatchTriageForTest,
+} from "../auto-post-unit.ts";
+import {
+  buildQuickTaskPrompt,
+  loadDeferredCaptures,
+  loadReplanCaptures,
+} from "../triage-resolution.ts";
+
+function makeProject(): string {
+  const base = mkdtempSync(join(tmpdir(), "gsd-triage-dispatch-"));
+  mkdirSync(join(base, ".gsd", "milestones", "M001", "slices", "S01"), { recursive: true });
+  writeFileSync(
+    join(base, ".gsd", "milestones", "M001", "M001-ROADMAP.md"),
+    [
+      "# M001",
+      "",
+      "## Slices",
+      "- [ ] **S01: Slice** `risk:low` `depends:[]`",
+    ].join("\n"),
+  );
+  writeFileSync(
+    join(base, ".gsd", "milestones", "M001", "slices", "S01", "S01-PLAN.md"),
+    "# S01 Plan\n\n## Tasks\n- [ ] **T01: Task** `est:10m`\n",
+  );
+  return base;
+}
+
+test("post-unit hooks exclude triage and quick-task units", () => {
+  assert.equal(checkPostUnitHooks("triage-captures", "M001/S01/triage", "/tmp/project"), null);
+  assert.equal(checkPostUnitHooks("quick-task", "M001/CAP-1", "/tmp/project"), null);
+});
+
+test("triage dispatch guard excludes step mode, hook units, triage units, and quick tasks", () => {
+  const normal = { stepMode: false, currentUnit: { type: "execute-task", id: "M001/S01/T01", startedAt: 1 } };
+  assert.equal(_shouldDispatchTriageForTest(normal as any), true);
+  assert.equal(_shouldDispatchTriageForTest({ ...normal, stepMode: true } as any), false);
+  assert.equal(
+    _shouldDispatchTriageForTest({ stepMode: false, currentUnit: { type: "hook/review", id: "M001/S01/T01", startedAt: 1 } } as any),
+    false,
+  );
+  assert.equal(
+    _shouldDispatchTriageForTest({ stepMode: false, currentUnit: { type: "triage-captures", id: "M001/S01/triage", startedAt: 1 } } as any),
+    false,
+  );
+  assert.equal(
+    _shouldDispatchTriageForTest({ stepMode: false, currentUnit: { type: "quick-task", id: "M001/CAP-1", startedAt: 1 } } as any),
+    false,
+  );
+});
+
+test("quick-task dispatch guard requires queued captures and avoids quick-task recursion", () => {
+  const capture = {
+    id: "CAP-test",
+    text: "Fix typo",
+    timestamp: new Date().toISOString(),
+    status: "resolved" as const,
+    classification: "quick-task" as const,
+  };
+  assert.equal(
+    _shouldDispatchQuickTaskForTest({
+      stepMode: false,
+      currentUnit: { type: "execute-task", id: "M001/S01/T01", startedAt: 1 },
+      pendingQuickTasks: [capture],
+    } as any),
+    true,
+  );
+  assert.equal(
+    _shouldDispatchQuickTaskForTest({
+      stepMode: false,
+      currentUnit: { type: "quick-task", id: "M001/CAP-test", startedAt: 1 },
+      pendingQuickTasks: [capture],
+    } as any),
+    false,
+  );
+  assert.equal(
+    _shouldDispatchQuickTaskForTest({
+      stepMode: false,
+      currentUnit: { type: "execute-task", id: "M001/S01/T01", startedAt: 1 },
+      pendingQuickTasks: [],
+    } as any),
+    false,
+  );
+});
+
+test("capture lifecycle exposes pending, replan, deferred, and executed states", () => {
+  const base = makeProject();
+  try {
+    const pendingId = appendCapture(base, "Need a quick follow-up.");
+    const replanId = appendCapture(base, "Plan needs a new task.");
+    const deferId = appendCapture(base, "Create a future milestone.");
+
+    assert.equal(hasPendingCaptures(base), true);
+    assert.equal(countPendingCaptures(base), 3);
+    assert.deepEqual(loadPendingCaptures(base).map((entry) => entry.id).sort(), [deferId, pendingId, replanId].sort());
+
+    markCaptureResolved(base, replanId, "replan", "replan slice", "Need plan update", "M001");
+    markCaptureResolved(base, deferId, "defer", "defer milestone", "Out of current scope", "M001");
+    markCaptureExecuted(base, replanId);
+
+    assert.equal(loadReplanCaptures(base).some((entry) => entry.id === replanId), true);
+    assert.equal(loadDeferredCaptures(base).some((entry) => entry.id === deferId), true);
+  } finally {
+    rmSync(base, { recursive: true, force: true });
+  }
+});
+
+test("quick-task prompt carries capture identity and completion instruction", () => {
+  const prompt = buildQuickTaskPrompt({
+    id: "CAP-quick",
+    text: "Fix the CLI typo",
+    timestamp: new Date().toISOString(),
+    status: "resolved",
+    classification: "quick-task",
+  });
+
+  assert.match(prompt, /CAP-quick/);
+  assert.match(prompt, /Fix the CLI typo/);
+  assert.match(prompt, /Quick task complete/);
 });

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

@@ -21,12 +21,18 @@ import type {
   ComputedArtifactRegistry,
   UnitContextManifest,
 } from "../unit-context-manifest.ts";
-import { UNIT_MANIFESTS } from "../unit-context-manifest.ts";
-import { buildReassessRoadmapPrompt } from "../auto-prompts.ts";
+import { KNOWN_UNIT_TYPES, UNIT_MANIFESTS } from "../unit-context-manifest.ts";
+import {
+  buildExecuteTaskPrompt,
+  buildGateEvaluatePrompt,
+  buildReassessRoadmapPrompt,
+  buildWorkflowPreferencesPrompt,
+} from "../auto-prompts.ts";
 import { invalidateAllCaches } from "../cache.ts";
 import {
   openDatabase,
   closeDatabase,
+  insertGateRow,
   insertMilestone,
   upsertMilestonePlanning,
   insertSlice,
@@ -121,7 +127,7 @@ test("Context Mode composer: standalone output starts with heading and includes
   assert.ok(out.startsWith("## Context Mode"));
   assert.match(out, /execution lane/i);
   assert.match(out, /`gsd_exec`/);
-  assert.match(out, /noisy commands/);
+  assert.match(out, /builds, tests, and diagnostics/);
   assert.match(out, /`gsd_exec_search`/);
   assert.match(out, /before reruns/);
   assert.match(out, /`gsd_resume`/);
@@ -136,7 +142,45 @@ test("Context Mode composer: nested output is compact single sentence", () => {
   assert.match(out, /`gsd_exec`/);
   assert.match(out, /`gsd_exec_search`/);
   assert.match(out, /`gsd_resume`/);
-  assert.ok(out.length < 180, `nested guidance should stay compact, got ${out.length} chars`);
+  assert.ok(out.length < 240, `nested guidance should stay compact, got ${out.length} chars`);
+});
+
+const laneLabelByMode: Record<string, string> = {
+  interview: "interview",
+  research: "research",
+  planning: "planning",
+  execution: "execution",
+  verification: "verification",
+  orchestration: "orchestration",
+  docs: "documentation",
+};
+
+test("Context Mode composer: every known eligible unit renders its configured lane and required tools", () => {
+  for (const unitType of KNOWN_UNIT_TYPES) {
+    const manifest = UNIT_MANIFESTS[unitType];
+    assert.ok(manifest, `missing manifest for ${unitType}`);
+    const out = composeContextModeInstructions(unitType, { enabled: true, renderMode: "standalone" });
+    if (manifest.contextMode === "none") {
+      assert.strictEqual(out, "", `${unitType} should not render Context Mode`);
+      continue;
+    }
+    assert.ok(out.startsWith("## Context Mode"), `${unitType} should render standalone Context Mode heading`);
+    assert.match(out, new RegExp(`Lane: \\*\\*${laneLabelByMode[manifest.contextMode]} lane\\*\\*\\.`, "i"));
+    assert.match(out, /`gsd_exec`/, `${unitType} should mention gsd_exec`);
+    assert.match(out, /`gsd_exec_search`/, `${unitType} should mention gsd_exec_search`);
+    assert.match(out, /`gsd_resume`/, `${unitType} should mention gsd_resume`);
+  }
+});
+
+test("Context Mode composer: workflow-preferences and research-decision render no Context Mode block", () => {
+  assert.strictEqual(
+    composeContextModeInstructions("workflow-preferences", { enabled: true, renderMode: "standalone" }),
+    "",
+  );
+  assert.strictEqual(
+    composeContextModeInstructions("research-decision", { enabled: true, renderMode: "standalone" }),
+    "",
+  );
 });
 
 // ─── Integration: migrated buildReassessRoadmapPrompt ─────────────────────
@@ -220,6 +264,94 @@ test("#4782 phase 2: buildReassessRoadmapPrompt emits composer-shaped context wi
   assert.ok(!prompt.includes("Slice Context (from discussion)"));
 });
 
+test("Context Mode resume injection: eligible prompts include one bounded snapshot block above inlined context", async (t) => {
+  const base = makeFixtureBase();
+  t.after(() => cleanup(base));
+  invalidateAllCaches();
+
+  seed(base, "M001");
+  writeArtifacts(base);
+  writeFileSync(
+    join(base, ".gsd", "last-snapshot.md"),
+    "# GSD context snapshot\n\nResume evidence.\n",
+    "utf-8",
+  );
+
+  const prompt = await buildReassessRoadmapPrompt("M001", "Test", "S01", base);
+
+  assert.equal(prompt.match(/## Context Snapshot/g)?.length, 1);
+  assert.match(prompt, /Source: `\.gsd\/last-snapshot\.md`/);
+  assert.match(prompt, /Resume evidence/);
+  assert.ok(prompt.indexOf("## Context Mode") < prompt.indexOf("## Context Snapshot"));
+  assert.ok(prompt.indexOf("## Context Snapshot") < prompt.indexOf("## Inlined Context"));
+});
+
+test("Context Mode resume injection: missing snapshot does not add an empty block", async (t) => {
+  const base = makeFixtureBase();
+  t.after(() => cleanup(base));
+  invalidateAllCaches();
+
+  seed(base, "M001");
+  writeArtifacts(base);
+
+  const prompt = await buildReassessRoadmapPrompt("M001", "Test", "S01", base);
+
+  assert.match(prompt, /## Context Mode/);
+  assert.doesNotMatch(prompt, /## Context Snapshot/);
+});
+
+test("Context Mode resume injection: disabled mode suppresses guidance and snapshot reads", async (t) => {
+  const base = makeFixtureBase();
+  t.after(() => cleanup(base));
+  invalidateAllCaches();
+
+  seed(base, "M001");
+  writeArtifacts(base);
+  writeFileSync(join(base, ".gsd", "PREFERENCES.md"), "---\ncontext_mode:\n  enabled: false\n---\n", "utf-8");
+  writeFileSync(join(base, ".gsd", "last-snapshot.md"), "# GSD context snapshot\n\nDo not inject.\n", "utf-8");
+
+  const prompt = await buildReassessRoadmapPrompt("M001", "Test", "S01", base);
+
+  assert.doesNotMatch(prompt, /## Context Mode/);
+  assert.doesNotMatch(prompt, /## Context Snapshot/);
+  assert.doesNotMatch(prompt, /Do not inject/);
+});
+
+test("Context Mode resume injection: none-mode units do not inject snapshots", async () => {
+  const base = makeFixtureBase();
+  try {
+    writeFileSync(join(base, ".gsd", "last-snapshot.md"), "# GSD context snapshot\n\nNo lane.\n", "utf-8");
+    const prompt = await buildWorkflowPreferencesPrompt(base);
+    assert.doesNotMatch(prompt, /## Context Mode/);
+    assert.doesNotMatch(prompt, /## Context Snapshot/);
+    assert.doesNotMatch(prompt, /No lane/);
+  } finally {
+    cleanup(base);
+  }
+});
+
+test("Context Mode prompt suppression: disabled inlined, phase-anchor, and nested prompts omit Context Mode", async (t) => {
+  const base = makeFixtureBase();
+  t.after(() => cleanup(base));
+  invalidateAllCaches();
+
+  seed(base, "M001");
+  writeArtifacts(base);
+  insertGateRow({ milestoneId: "M001", sliceId: "S01", gateId: "Q3", scope: "slice" });
+  writeFileSync(join(base, ".gsd", "PREFERENCES.md"), "---\ncontext_mode:\n  enabled: false\n---\n", "utf-8");
+  writeFileSync(join(base, ".gsd", "last-snapshot.md"), "# GSD context snapshot\n\nDo not inject.\n", "utf-8");
+
+  const inlinedPrompt = await buildReassessRoadmapPrompt("M001", "Test", "S01", base);
+  assert.doesNotMatch(inlinedPrompt, /## Context Mode|Context Mode \(|## Context Snapshot/);
+
+  const phaseAnchorPrompt = await buildExecuteTaskPrompt("M001", "S01", "First", "T01", "Task", base);
+  assert.doesNotMatch(phaseAnchorPrompt, /## Context Mode|Context Mode \(|## Context Snapshot/);
+
+  const nestedPrompt = await buildGateEvaluatePrompt("M001", "Test", "S01", "First", base);
+  assert.match(nestedPrompt, /Use this as the prompt for a `subagent` call/);
+  assert.doesNotMatch(nestedPrompt, /## Context Mode|Context Mode \(|## Context Snapshot/);
+});
+
 // ─── v2 surface (#4924) ───────────────────────────────────────────────────
 
 const fakeBase: BaseResolverContext = {