gsd-pi 2.80.0-dev.3065f7bfa → 2.80.0-dev.488219eb6

package/src/resources/extensions/gsd/tests/unstructured-continue-context-injection.test.ts

@@ -1,100 +1,11 @@
-// Project/App: GSD-2
-// File Purpose: Regression test for unstructured continue task-context injection.
-// Copyright (c) 2026 Jeremy McSpadden <jeremy@fluxlabs.net>
-
-/**
- * Bug #3615: When a user types "continue" (or any bare text) to resume
- * an in-progress session, buildGuidedExecuteContextInjection() only
- * matched two hardcoded regex patterns (auto-dispatch and guided-resume).
- * The function returned null for any other input, so no task context was
- * injected — causing the agent to rebuild everything from scratch and
- * burn ~86k tokens.
- *
- * This test verifies:
- *   1. Structural: the fallback exists with phase + intent guards
- *   2. Behavioral: RESUME_INTENT_PATTERNS matches expected prompts and
- *      rejects non-resume prompts (control, help, diagnostic, etc.)
- */
+// GSD-2 — Regression test for #3615: unstructured "continue" must inject task context
 
 import { describe, 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 systemContextSource = readFileSync(
-  join(__dirname, "..", "bootstrap", "system-context.ts"),
-  "utf-8",
-);
-
-// ── Structural tests ────────────────────────────────────────────────
-
-describe("#3615 — structural: fallback exists with correct guards", () => {
-  const fnStart = systemContextSource.indexOf("async function buildGuidedExecuteContextInjection(");
-  assert.ok(fnStart >= 0, "should find buildGuidedExecuteContextInjection");
-  const fnEnd = systemContextSource.indexOf("\nasync function ", fnStart + 1);
-  const fnBody = fnEnd >= 0
-    ? systemContextSource.slice(fnStart, fnEnd)
-    : systemContextSource.slice(fnStart);
-
-  test("has a deriveState fallback after the two regex branches", () => {
-    const deriveStateCalls = fnBody.match(/deriveState\(basePath\)/g);
-    assert.ok(
-      deriveStateCalls && deriveStateCalls.length >= 2,
-      `expected >=2 deriveState(basePath) calls, got ${deriveStateCalls?.length ?? 0}`,
-    );
-  });
-
-  test("fallback is phase-gated to executing only", () => {
-    const afterFallback = fnBody.indexOf("// Fallback:");
-    assert.ok(afterFallback >= 0, "should have a fallback comment");
-    const fallbackSection = fnBody.slice(afterFallback);
-    assert.ok(
-      fallbackSection.includes('state.phase === "executing"'),
-      'fallback must be gated on state.phase === "executing"',
-    );
-  });
-
-  test("fallback is intent-gated via isLowEntropyResumePrompt", () => {
-    const afterFallback = fnBody.indexOf("// Fallback:");
-    const fallbackSection = fnBody.slice(afterFallback);
-    assert.ok(
-      fallbackSection.includes("isLowEntropyResumePrompt(prompt)"),
-      "fallback must check isLowEntropyResumePrompt before deriveState",
-    );
-  });
-
-  test("fallback calls buildTaskExecutionContextInjection with derived state", () => {
-    const afterFallback = fnBody.indexOf("// Fallback:");
-    const fallbackSection = fnBody.slice(afterFallback);
-    assert.ok(
-      fallbackSection.includes("buildTaskExecutionContextInjection") &&
-      fallbackSection.includes("state.activeMilestone.id") &&
-      fallbackSection.includes("state.activeSlice.id") &&
-      fallbackSection.includes("state.activeTask.id"),
-      "fallback must call buildTaskExecutionContextInjection with state-derived IDs",
-    );
-  });
-
-  // Removed: source-grep count of `return null;` occurrences. The behaviour
-  // we care about ("function returns null only when no auto-dispatch /
-  // guided-resume / fallback path matches") is exercised by the behavioural
-  // tests below — counting literal `return null;` tokens encodes statement
-  // shape, not contract. Refs #4851.
-});
-
-// ── Behavioral tests: RESUME_INTENT_PATTERNS ────────────────────────
-
-describe("#3615 — behavioral: RESUME_INTENT_PATTERNS matches resume prompts", () => {
-  // Extract the regex from source so the test stays in sync
-  const patternMatch = systemContextSource.match(/const RESUME_INTENT_PATTERNS\s*=\s*\/(.+)\/;/);
-  assert.ok(patternMatch, "should find RESUME_INTENT_PATTERNS definition");
-  const pattern = new RegExp(patternMatch[1]);
 
-  // Helper: normalize prompt the same way the production code does
-  const normalize = (s: string) => s.trim().toLowerCase().replace(/[.!?,]+$/g, "");
+import { isLowEntropyResumePrompt } from "../bootstrap/system-context.ts";
 
+describe("#3615 — RESUME_INTENT_PATTERNS behavior", () => {
   const shouldMatch = [
     "continue",
     "Continue",
@@ -119,7 +30,7 @@ describe("#3615 — behavioral: RESUME_INTENT_PATTERNS matches resume prompts",
     "do it",
     "let's go",
     "pick up where you left off",
-    "  continue  ",  // whitespace padded
+    "  continue  ",
   ];
 
   const shouldNotMatch = [
@@ -144,19 +55,13 @@ describe("#3615 — behavioral: RESUME_INTENT_PATTERNS matches resume prompts",
 
   for (const prompt of shouldMatch) {
     test(`matches resume prompt: "${prompt}"`, () => {
-      assert.ok(
-        pattern.test(normalize(prompt)),
-        `expected RESUME_INTENT_PATTERNS to match "${prompt}" (normalized: "${normalize(prompt)}")`,
-      );
+      assert.equal(isLowEntropyResumePrompt(prompt), true);
     });
   }
 
   for (const prompt of shouldNotMatch) {
     test(`rejects non-resume prompt: "${prompt}"`, () => {
-      assert.ok(
-        !pattern.test(normalize(prompt)),
-        `expected RESUME_INTENT_PATTERNS to NOT match "${prompt}" (normalized: "${normalize(prompt)}")`,
-      );
+      assert.equal(isLowEntropyResumePrompt(prompt), false);
     });
   }
 });

package/src/resources/extensions/gsd/tests/uok-plan-v2-wiring.test.ts

@@ -1,9 +1,8 @@
 import test from "node:test";
 import assert from "node:assert/strict";
 import { mkdtempSync, mkdirSync, readFileSync, rmSync, writeFileSync } from "node:fs";
-import { join, dirname } from "node:path";
+import { join } from "node:path";
 import { tmpdir } from "node:os";
-import { fileURLToPath } from "node:url";
 
 import {
   closeDatabase,
@@ -19,9 +18,13 @@ import {
   isEmptyPlanV2GraphResult,
   isMissingFinalizedContextResult,
 } from "../uok/plan-v2.ts";
+import {
+  _needsPlanV2GateForTest,
+  _runPlanV2GateForTest,
+} from "../guided-flow.ts";
+import { shouldRunPlanV2Gate } from "../auto/phases.ts";
+import { resolveUokFlags } from "../uok/flags.ts";
 
-const __dirname = dirname(fileURLToPath(import.meta.url));
-const gsdDir = join(__dirname, "..");
 const MILESTONE_ID = "M001";
 const SLICE_ID = "S01";
 const TASK_ID = "T01";
@@ -94,48 +97,52 @@ test.afterEach(() => {
 });
 
 test("guided flow keeps plan-v2 fail-closed handling for non-recoverable failures", () => {
-  const source = readFileSync(join(gsdDir, "guided-flow.ts"), "utf-8");
-  assert.ok(
-    source.includes("needsPlanV2Gate") &&
-    source.includes("ensurePlanV2Graph") &&
-    source.includes("Plan gate failed-closed"),
-    "guided flow should fail-closed when plan-v2 graph compilation fails",
+  const basePath = createBasePath();
+  writeMilestoneFile(basePath, "CONTEXT", "Finalized context.");
+  insertMilestone({ id: MILESTONE_ID, title: "Milestone", status: "active" });
+  insertSlice({
+    id: SLICE_ID,
+    milestoneId: MILESTONE_ID,
+    title: "Slice",
+    status: "pending",
+    sequence: 1,
+  });
+
+  const notifications: string[] = [];
+  const decision = _runPlanV2GateForTest(
+    { ui: { notify: (message: string) => notifications.push(message) } } as any,
+    basePath,
+    buildState("executing"),
   );
+
+  assert.equal(decision, "block");
+  assert.match(notifications[0] ?? "", /Plan gate failed-closed/);
 });
 
 test("guided flow routes recoverable missing finalized context to discuss-milestone", () => {
-  const source = readFileSync(join(gsdDir, "guided-flow.ts"), "utf-8");
-  assert.ok(
-    source.includes('PlanV2GateDecision = "pass" | "recover-missing-context" | "block"') &&
-    source.includes("isMissingFinalizedContextResult(compiled)") &&
-    source.includes('planV2GateDecision === "recover-missing-context"') &&
-    source.includes("buildDiscussMilestonePrompt"),
-    "guided flow should redispatch missing finalized context to discuss-milestone instead of fail-closing",
+  const basePath = createBasePath();
+  seedGraphRows();
+  writeMilestoneFile(basePath, "CONTEXT-DRAFT", "Draft context only.");
+
+  const decision = _runPlanV2GateForTest(
+    { ui: { notify: () => undefined } } as any,
+    basePath,
+    buildState("executing"),
   );
+
+  assert.equal(decision, "recover-missing-context");
 });
 
-test("guided flow checks pending deep setup before plan-v2 gate", () => {
-  const source = readFileSync(join(gsdDir, "guided-flow.ts"), "utf-8");
-  const showSmartEntryIdx = source.indexOf("export async function showSmartEntry");
-  assert.notEqual(showSmartEntryIdx, -1);
-  const deepIdx = source.indexOf("shouldRunDeepProjectSetup(state, prefs, basePath)", showSmartEntryIdx);
-  const planIdx = source.indexOf("runPlanV2Gate(ctx, basePath, state)", showSmartEntryIdx);
-  assert.ok(
-    deepIdx > -1 && planIdx > -1 && deepIdx < planIdx,
-    "foreground deep setup must run before plan-v2 can fail-close guided /gsd",
-  );
-  assert.ok(
-    source.includes("loadEffectiveGSDPreferences(basePath)?.preferences"),
-    "guided plan-v2 gate must load preferences from the target project root",
-  );
+test("guided and auto plan-v2 phase gates agree on execution phases", () => {
+  assert.equal(_needsPlanV2GateForTest(buildState("planning")), false);
+  assert.equal(_needsPlanV2GateForTest(buildState("executing")), true);
+  assert.equal(shouldRunPlanV2Gate("planning"), false);
+  assert.equal(shouldRunPlanV2Gate("executing"), true);
 });
 
 test("auto pre-dispatch uses resolved plan-v2 defaults", () => {
-  const source = readFileSync(join(gsdDir, "auto", "phases.ts"), "utf-8");
-  assert.ok(
-    source.includes("uokFlags.planV2 && shouldRunPlanV2Gate(state.phase)"),
-    "auto-mode should honor resolveUokFlags defaults, not only explicit uok.plan_v2.enabled",
-  );
+  assert.equal(resolveUokFlags(undefined).planV2, true);
+  assert.equal(resolveUokFlags({ uok: { plan_v2: { enabled: false } } } as any).planV2, false);
 });
 
 test("plan-v2 gate fails closed for execution phase when finalized context is missing", () => {