gsd-pi 2.81.0-dev.72a81bdf3 → 2.82.0-dev.725028083

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

@@ -2548,6 +2548,81 @@ test("resolveAgentEndCancelled with errorContext passes it through to resolved p
   assert.equal(resolved.errorContext!.isTransient, true);
 });
 
+test("runUnitPhase pauses transient aborted cancellations instead of hard-stopping", async (t) => {
+  _resetPendingResolve();
+
+  const basePath = mkdtempSync(join(tmpdir(), "gsd-aborted-cancel-"));
+  t.after(() => {
+    rmSync(basePath, { recursive: true, force: true });
+  });
+
+  const ctx = {
+    ...makeMockCtx(),
+    ui: {
+      notify: () => {},
+      setStatus: () => {},
+      setWorkingMessage: () => {},
+    },
+    sessionManager: {
+      getEntries: () => [],
+    },
+    modelRegistry: {
+      getProviderAuthMode: () => undefined,
+      isProviderRequestReady: () => true,
+    },
+  } as any;
+  const pi = {
+    ...makeMockPi(),
+    sendMessage: () => {
+      queueMicrotask(() => resolveAgentEndCancelled({
+        message: "Claude Code process aborted by user",
+        category: "aborted",
+        isTransient: true,
+      }));
+    },
+  } as any;
+  const s = makeLoopSession({
+    basePath,
+    canonicalProjectRoot: basePath,
+    originalBasePath: basePath,
+  });
+  const deps = makeMockDeps();
+  let seq = 0;
+
+  const result = await runUnitPhase(
+    { ctx, pi, s, deps, prefs: undefined, iteration: 1, flowId: "flow-aborted", nextSeq: () => ++seq },
+    {
+      unitType: "execute-task",
+      unitId: "M001/S01/T01",
+      prompt: "do work",
+      finalPrompt: "do work",
+      pauseAfterUatDispatch: false,
+      state: {
+        phase: "executing",
+        activeMilestone: { id: "M001", title: "Milestone" },
+        activeSlice: { id: "S01", title: "Slice" },
+        activeTask: { id: "T01", title: "Task" },
+        registry: [{ id: "M001", title: "Milestone", status: "active" }],
+        recentDecisions: [],
+        blockers: [],
+        nextAction: "",
+        progress: { milestones: { done: 0, total: 1 } },
+        requirements: { active: 0, validated: 0, deferred: 0, outOfScope: 0, blocked: 0, total: 0 },
+      } as any,
+      mid: "M001",
+      midTitle: "Milestone",
+      isRetry: false,
+      previousTier: undefined,
+    },
+    { recentUnits: [{ key: "execute-task/M001/S01/T01" }], stuckRecoveryAttempts: 0, consecutiveFinalizeTimeouts: 0 },
+  );
+
+  assert.equal(result.action, "break");
+  assert.equal((result as any).reason, "unit-aborted-pause");
+  assert.equal(deps.callLog.includes("pauseAuto"), true);
+  assert.equal(deps.callLog.includes("stopAuto"), false);
+});
+
 test("runUnitPhase pauses ghost completions before closeout and finalize side effects", async (t) => {
   _resetPendingResolve();
 

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

@@ -0,0 +1,62 @@
+// GSD-2 — 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
+// the model-router filters them out on providers without imageToolResults
+// (OpenAI completions/responses, Azure, Mistral, Ollama). Conditional-image
+// browser tools (navigation, forms, refs, intent, interaction) must NOT
+// declare producesImages — they only attach error screenshots and filtering
+// them would lose the whole tool surface for OpenAI users.
+
+import test from "node:test";
+import assert from "node:assert/strict";
+
+import type { ExtensionAPI } from "@gsd/pi-coding-agent";
+
+import { registerScreenshotTools } from "../../browser-tools/tools/screenshot.ts";
+import { registerZoomTools } from "../../browser-tools/tools/zoom.ts";
+
+interface CapturedToolDef {
+  name: string;
+  compatibility?: { producesImages?: boolean; schemaFeatures?: string[] };
+}
+
+function makeCapturingPi(): { pi: ExtensionAPI; tools: CapturedToolDef[] } {
+  const tools: CapturedToolDef[] = [];
+  const pi = {
+    registerTool(def: CapturedToolDef): void {
+      tools.push({ name: def.name, compatibility: def.compatibility });
+    },
+  } as unknown as ExtensionAPI;
+  return { pi, tools };
+}
+
+// Browser tool registration functions accept a `deps` object. None of the
+// declarations under test reach for these deps at registration time — they
+// only run inside execute(), which we never invoke. A bare object satisfies
+// the type signature.
+const stubDeps = {} as Parameters<typeof registerScreenshotTools>[1];
+
+test("browser_screenshot declares producesImages: true", () => {
+  const { pi, tools } = makeCapturingPi();
+  registerScreenshotTools(pi, stubDeps);
+  const screenshot = tools.find((t) => t.name === "browser_screenshot");
+  assert.ok(screenshot, "browser_screenshot should be registered");
+  assert.equal(
+    screenshot.compatibility?.producesImages,
+    true,
+    "browser_screenshot must declare producesImages so it is filtered on providers without imageToolResults",
+  );
+});
+
+test("browser_zoom_region declares producesImages: true", () => {
+  const { pi, tools } = makeCapturingPi();
+  registerZoomTools(pi, stubDeps);
+  const zoom = tools.find((t) => t.name === "browser_zoom_region");
+  assert.ok(zoom, "browser_zoom_region should be registered");
+  assert.equal(
+    zoom.compatibility?.producesImages,
+    true,
+    "browser_zoom_region must declare producesImages so it is filtered on providers without imageToolResults",
+  );
+});

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

@@ -0,0 +1,312 @@
+// ADR-013 Phase 6 cutover (Stage 1) — queryDecisionsFromMemories parity test.
+//
+// Verifies that reading active decisions from the `memories` table returns
+// the same Decision[] shape and content as the legacy `queryDecisions` read
+// from the `decisions` table, once Phase 5 dual-write has populated both
+// surfaces. Lock-in regression for the prompt-inline read path which was
+// switched to the memories source in auto-prompts.ts:inlineDecisionsFromDb.
+//
+// Scope of parity: ACTIVE decisions only. Superseded rows are intentionally
+// skipped by the existing backfill, so this test does not assert parity for
+// the supersedes-chain — that gap is acknowledged in
+// queryDecisionsFromMemories' contract and tracked for Stage 2/3.
+
+import test from "node:test";
+import assert from "node:assert/strict";
+import { mkdtempSync, mkdirSync, rmSync } from "node:fs";
+import { join } from "node:path";
+import { tmpdir } from "node:os";
+
+import {
+  closeDatabase,
+  insertDecision,
+  openDatabase,
+} from "../gsd-db.ts";
+import { saveDecisionToDb } from "../db-writer.ts";
+import {
+  queryDecisions,
+  queryDecisionsFromMemories,
+} from "../context-store.ts";
+
+function makeTmpBase(): string {
+  const base = mkdtempSync(join(tmpdir(), "gsd-decisions-memories-"));
+  mkdirSync(join(base, ".gsd"), { recursive: true });
+  openDatabase(join(base, ".gsd", "gsd.db"));
+  return base;
+}
+
+function cleanup(base: string): void {
+  try {
+    closeDatabase();
+  } catch {
+    /* noop */
+  }
+  try {
+    rmSync(base, { recursive: true, force: true });
+  } catch {
+    /* noop */
+  }
+}
+
+async function seedDecision(
+  base: string,
+  fields: {
+    when_context: string;
+    scope: string;
+    decision: string;
+    choice: string;
+    rationale: string;
+    revisable?: string;
+    made_by?: "human" | "agent" | "collaborative";
+  },
+): Promise<string> {
+  // saveDecisionToDb writes ONLY to memories post-Stage-3. For parity tests
+  // comparing the legacy `queryDecisions` against `queryDecisionsFromMemories`,
+  // mirror the same row into the legacy decisions table directly so both
+  // surfaces hold the same data and the parity assertion is well-defined.
+  const result = await saveDecisionToDb(
+    {
+      when_context: fields.when_context,
+      scope: fields.scope,
+      decision: fields.decision,
+      choice: fields.choice,
+      rationale: fields.rationale,
+      revisable: fields.revisable ?? "Yes",
+      made_by: fields.made_by ?? "agent",
+    },
+    base,
+  );
+  insertDecision({
+    id: result.id,
+    when_context: fields.when_context,
+    scope: fields.scope,
+    decision: fields.decision,
+    choice: fields.choice,
+    rationale: fields.rationale,
+    revisable: fields.revisable ?? "Yes",
+    made_by: fields.made_by ?? "agent",
+    superseded_by: null,
+  });
+  return result.id;
+}
+
+test("queryDecisionsFromMemories returns empty when no decisions exist", () => {
+  const base = makeTmpBase();
+  try {
+    assert.deepEqual(queryDecisionsFromMemories(), []);
+    assert.deepEqual(queryDecisionsFromMemories({ milestoneId: "M001" }), []);
+  } finally {
+    cleanup(base);
+  }
+});
+
+test("queryDecisionsFromMemories matches queryDecisions for a single active decision", async () => {
+  const base = makeTmpBase();
+  try {
+    await seedDecision(base, {
+      when_context: "M001 discuss phase",
+      scope: "M001",
+      decision: "Adopt SQLite for persistence",
+      choice: "better-sqlite3",
+      rationale: "Native, synchronous, well-supported",
+    });
+
+    const fromDecisions = queryDecisions();
+    const fromMemories = queryDecisionsFromMemories();
+
+    assert.equal(fromDecisions.length, 1);
+    assert.equal(fromMemories.length, 1);
+
+    // Compare the user-visible Decision fields. seq differs across tables
+    // (memories.seq vs decisions.seq) so it's intentionally excluded.
+    const { seq: _seq1, ...d1 } = fromDecisions[0]!;
+    const { seq: _seq2, ...d2 } = fromMemories[0]!;
+    assert.deepEqual(d1, d2);
+  } finally {
+    cleanup(base);
+  }
+});
+
+test("queryDecisionsFromMemories preserves decision order across multiple writes", async () => {
+  const base = makeTmpBase();
+  try {
+    const id1 = await seedDecision(base, {
+      when_context: "M001 discuss",
+      scope: "M001",
+      decision: "First decision",
+      choice: "A",
+      rationale: "first",
+    });
+    const id2 = await seedDecision(base, {
+      when_context: "M001 plan",
+      scope: "M001",
+      decision: "Second decision",
+      choice: "B",
+      rationale: "second",
+    });
+    const id3 = await seedDecision(base, {
+      when_context: "M002 discuss",
+      scope: "M002",
+      decision: "Third decision",
+      choice: "C",
+      rationale: "third",
+    });
+
+    const fromMemories = queryDecisionsFromMemories();
+    assert.equal(fromMemories.length, 3);
+    assert.deepEqual(
+      fromMemories.map((d) => d.id),
+      [id1, id2, id3],
+    );
+  } finally {
+    cleanup(base);
+  }
+});
+
+test("queryDecisionsFromMemories filters by milestoneId (substring match on when_context)", async () => {
+  const base = makeTmpBase();
+  try {
+    await seedDecision(base, {
+      when_context: "M001 discuss",
+      scope: "M001",
+      decision: "M001 decision",
+      choice: "A",
+      rationale: "x",
+    });
+    await seedDecision(base, {
+      when_context: "M002 plan",
+      scope: "M002",
+      decision: "M002 decision",
+      choice: "B",
+      rationale: "y",
+    });
+    await seedDecision(base, {
+      when_context: "M001 execute",
+      scope: "M001-S01",
+      decision: "M001 follow-up",
+      choice: "C",
+      rationale: "z",
+    });
+    await seedDecision(base, {
+      when_context: "M003 plan",
+      scope: "M003",
+      decision: "Use M001 as precedent",
+      choice: "D",
+      rationale: "Mentions M001 outside when_context",
+    });
+
+    const m001 = queryDecisionsFromMemories({ milestoneId: "M001" });
+    assert.equal(m001.length, 2, "two decisions reference M001 in when_context");
+    assert.ok(m001.every((d) => d.when_context.includes("M001")));
+
+    const m002 = queryDecisionsFromMemories({ milestoneId: "M002" });
+    assert.equal(m002.length, 1);
+    assert.equal(m002[0]?.decision, "M002 decision");
+  } finally {
+    cleanup(base);
+  }
+});
+
+test("queryDecisionsFromMemories filters by scope (exact match, no prefix collisions)", async () => {
+  const base = makeTmpBase();
+  try {
+    await seedDecision(base, {
+      when_context: "M001 discuss",
+      scope: "M001",
+      decision: "Milestone-level",
+      choice: "A",
+      rationale: "x",
+    });
+    await seedDecision(base, {
+      when_context: "M001 plan",
+      scope: "M001-S01",
+      decision: "Slice-level",
+      choice: "B",
+      rationale: "y",
+    });
+    await seedDecision(base, {
+      when_context: "M001 plan",
+      scope: "M001-S02",
+      decision: "Different slice",
+      choice: "C",
+      rationale: "z",
+    });
+
+    // Exact-scope filter must not match prefix-similar values.
+    const milestoneScope = queryDecisionsFromMemories({ scope: "M001" });
+    assert.equal(milestoneScope.length, 1, "scope=M001 must not match M001-S01 / M001-S02");
+    assert.equal(milestoneScope[0]?.scope, "M001");
+
+    const sliceScope = queryDecisionsFromMemories({ scope: "M001-S01" });
+    assert.equal(sliceScope.length, 1);
+    assert.equal(sliceScope[0]?.scope, "M001-S01");
+  } finally {
+    cleanup(base);
+  }
+});
+
+test("queryDecisionsFromMemories matches queryDecisions for combined milestoneId + scope filters", async () => {
+  const base = makeTmpBase();
+  try {
+    await seedDecision(base, {
+      when_context: "M001 discuss",
+      scope: "M001",
+      decision: "A",
+      choice: "1",
+      rationale: "x",
+    });
+    await seedDecision(base, {
+      when_context: "M001 plan",
+      scope: "M001-S01",
+      decision: "B",
+      choice: "2",
+      rationale: "y",
+    });
+    await seedDecision(base, {
+      when_context: "M002 discuss",
+      scope: "M002",
+      decision: "C",
+      choice: "3",
+      rationale: "z",
+    });
+
+    const opts = { milestoneId: "M001", scope: "M001-S01" };
+    const fromDecisions = queryDecisions(opts);
+    const fromMemories = queryDecisionsFromMemories(opts);
+
+    assert.equal(fromDecisions.length, fromMemories.length);
+    assert.equal(fromMemories.length, 1);
+    assert.equal(fromMemories[0]?.id, fromDecisions[0]?.id);
+    assert.equal(fromMemories[0]?.scope, "M001-S01");
+  } finally {
+    cleanup(base);
+  }
+});
+
+test("queryDecisionsFromMemories ignores memories without a sourceDecisionId marker", async () => {
+  const base = makeTmpBase();
+  try {
+    // Insert a user-authored memory (no sourceDecisionId) — must not appear in
+    // the decisions-from-memories projection.
+    const { createMemory } = await import("../memory-store.ts");
+    createMemory({
+      category: "architecture",
+      content: "User-authored architecture note, not derived from a decision",
+      scope: "project",
+    });
+
+    await seedDecision(base, {
+      when_context: "M001 discuss",
+      scope: "M001",
+      decision: "Real decision",
+      choice: "A",
+      rationale: "x",
+    });
+
+    const fromMemories = queryDecisionsFromMemories();
+    assert.equal(fromMemories.length, 1, "user-authored memory must not appear as a decision");
+    assert.equal(fromMemories[0]?.decision, "Real decision");
+  } finally {
+    cleanup(base);
+  }
+});

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

@@ -27,6 +27,7 @@ import {
   saveArtifactToDb,
   extractDeferredSliceRef,
 } from '../db-writer.ts';
+import { getAllDecisionsFromMemories } from '../context-store.ts';
 import type { Decision, Requirement } from '../types.ts';
 
 // ═══════════════════════════════════════════════════════════════════════════
@@ -326,11 +327,14 @@ describe('db-writer', () => {
 
       assert.deepStrictEqual(result.id, 'D001', 'saveDecisionToDb returns D001 as first ID');
 
-      // Verify DB state
-      const dbDecision = getDecisionById('D001');
-      assert.ok(!!dbDecision, 'decision exists in DB after save');
-      assert.deepStrictEqual(dbDecision?.scope, 'arch', 'DB decision has correct scope');
-      assert.deepStrictEqual(dbDecision?.choice, 'Option A', 'DB decision has correct choice');
+      // ADR-013 Stage 3: decisions land in memories, not the legacy table.
+      const memoryDecisions = getAllDecisionsFromMemories();
+      assert.equal(memoryDecisions.length, 1, 'one memory row exists after save');
+      const memDecision = memoryDecisions[0];
+      assert.ok(memDecision, 'memory decision exists after save');
+      assert.equal(memDecision.id, 'D001');
+      assert.equal(memDecision.scope, 'arch', 'memory decision has correct scope');
+      assert.equal(memDecision.choice, 'Option A', 'memory decision has correct choice');
 
       // Verify markdown file was written
       const mdPath = path.join(tmpDir, '.gsd', 'DECISIONS.md');
@@ -394,10 +398,11 @@ describe('db-writer', () => {
         assert.match(id, /^D\d{3}$/, `ID ${id} should match D### pattern`);
       }
 
-      // Verify all 5 exist in DB
+      // ADR-013 Stage 3: verify all 5 exist in the memories table (decisions
+      // table receives no writes from saveDecisionToDb post-cutover).
+      const memoryIds = new Set(getAllDecisionsFromMemories().map((d) => d.id));
       for (const id of ids) {
-        const row = getDecisionById(id);
-        assert.ok(row, `Decision ${id} should exist in DB`);
+        assert.ok(memoryIds.has(id), `Decision ${id} should exist in memories`);
       }
     } finally {
       closeDatabase();