@opengsd/gsd-pi 1.3.0 → 1.4.0-dev.c18009cd

package/src/resources/extensions/gsd/tests/knowledge-cold-start.test.ts

@@ -0,0 +1,386 @@
+import test from "node:test";
+import assert from "node:assert/strict";
+import {
+  existsSync,
+  mkdirSync,
+  mkdtempSync,
+  readFileSync,
+  realpathSync,
+  rmSync,
+  writeFileSync,
+} from "node:fs";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
+import { execFileSync } from "node:child_process";
+
+import type { ExtensionCommandContext, ExtensionContext } from "@gsd/pi-coding-agent";
+
+import { buildBeforeAgentStartResult, _flushDeferredContextMaintenanceForTest } from "../bootstrap/system-context.ts";
+import { handleKnowledge } from "../commands-handlers.ts";
+import { withCommandCwd } from "../commands/context.ts";
+import {
+  _getAdapter,
+  closeDatabase,
+  isDbAvailable,
+  openDatabase,
+} from "../gsd-db.ts";
+import { createMemory } from "../memory-store.ts";
+import { invalidateStateCache } from "../state.ts";
+
+test("#830 startup opens the project DB before projecting KNOWLEDGE.md", async (t) => {
+  const base = realpathSync(mkdtempSync(join(tmpdir(), "gsd-knowledge-startup-")));
+  const originalCwd = process.cwd();
+  const originalGsdHome = process.env.GSD_HOME;
+  const gsdDir = join(base, ".gsd");
+  const knowledgePath = join(gsdDir, "KNOWLEDGE.md");
+  mkdirSync(gsdDir, { recursive: true });
+  execFileSync("git", ["init", "-q"], { cwd: base, stdio: "ignore" });
+  process.chdir(base);
+  process.env.GSD_HOME = join(base, ".gsd-home");
+
+  t.after(async () => {
+    await _flushDeferredContextMaintenanceForTest(base);
+    if (isDbAvailable()) closeDatabase();
+    invalidateStateCache();
+    process.chdir(originalCwd);
+    if (originalGsdHome === undefined) delete process.env.GSD_HOME;
+    else process.env.GSD_HOME = originalGsdHome;
+    rmSync(base, { recursive: true, force: true });
+  });
+
+  assert.equal(openDatabase(join(gsdDir, "gsd.db")), true);
+  const memoryId = createMemory({
+    category: "pattern",
+    content: "Cold-start projections open the canonical DB first",
+    scope: "project",
+    confidence: 0.9,
+    structuredFields: {
+      sourceKnowledgeId: "P001",
+      sourceKnowledgeTable: "patterns",
+      pattern: "Cold-start projections open the canonical DB first",
+      where: "bootstrap",
+      notes: "regression coverage",
+    },
+  });
+  assert.ok(memoryId);
+  closeDatabase();
+  assert.equal(isDbAvailable(), false);
+  assert.equal(existsSync(knowledgePath), false);
+
+  const ctx = {
+    projectRoot: base,
+    ui: { notify: () => undefined },
+  } as unknown as ExtensionContext;
+  await buildBeforeAgentStartResult(
+    { prompt: "Inspect project knowledge", systemPrompt: "base system prompt" },
+    ctx,
+  );
+
+  assert.equal(isDbAvailable(), true, "startup should open the project DB");
+  assert.equal(existsSync(knowledgePath), false, "startup should defer KNOWLEDGE.md projection off the prompt path");
+  await _flushDeferredContextMaintenanceForTest(base);
+  assert.equal(existsSync(knowledgePath), true, "deferred maintenance should write KNOWLEDGE.md");
+  assert.match(
+    readFileSync(knowledgePath, "utf-8"),
+    /\| P001 \| Cold-start projections open the canonical DB first \| bootstrap \| regression coverage \|/,
+  );
+});
+
+test("#896 startup maintenance skips repeated sentinel work in one session", async (t) => {
+  const base = realpathSync(mkdtempSync(join(tmpdir(), "gsd-startup-maintenance-once-")));
+  const originalCwd = process.cwd();
+  const originalGsdHome = process.env.GSD_HOME;
+  const gsdDir = join(base, ".gsd");
+  const knowledgePath = join(gsdDir, "KNOWLEDGE.md");
+  mkdirSync(gsdDir, { recursive: true });
+  execFileSync("git", ["init", "-q"], { cwd: base, stdio: "ignore" });
+  process.chdir(base);
+  process.env.GSD_HOME = join(base, ".gsd-home");
+
+  t.after(async () => {
+    await _flushDeferredContextMaintenanceForTest(base);
+    if (isDbAvailable()) closeDatabase();
+    invalidateStateCache();
+    process.chdir(originalCwd);
+    if (originalGsdHome === undefined) delete process.env.GSD_HOME;
+    else process.env.GSD_HOME = originalGsdHome;
+    rmSync(base, { recursive: true, force: true });
+  });
+
+  writeFileSync(
+    knowledgePath,
+    [
+      "# Project Knowledge",
+      "",
+      "## Rules",
+      "",
+      "| # | Scope | Rule | Why | Added |",
+      "|---|-------|------|-----|-------|",
+      "",
+      "## Patterns",
+      "",
+      "| # | Pattern | Where | Notes |",
+      "|---|---------|-------|-------|",
+      "| P001 | Run startup maintenance once | bootstrap | first pass |",
+      "",
+      "## Lessons Learned",
+      "",
+      "| # | What Happened | Root Cause | Fix | Scope |",
+      "|---|--------------|------------|-----|-------|",
+      "",
+    ].join("\n"),
+    "utf-8",
+  );
+
+  assert.equal(openDatabase(join(gsdDir, "gsd.db")), true);
+  closeDatabase();
+  assert.equal(isDbAvailable(), false);
+
+  let sessionId = "session-one";
+  const ctx = {
+    projectRoot: base,
+    sessionManager: { getSessionId: () => sessionId },
+    ui: { notify: () => undefined },
+  } as unknown as ExtensionContext;
+
+  await buildBeforeAgentStartResult(
+    { prompt: "Inspect project knowledge", systemPrompt: "base system prompt" },
+    ctx,
+  );
+  await _flushDeferredContextMaintenanceForTest(base);
+
+  const adapter = _getAdapter();
+  assert.ok(adapter);
+  const firstPass = adapter
+    .prepare("SELECT COUNT(*) AS count FROM memories WHERE structured_fields LIKE '%\"sourceKnowledgeId\":\"P001\"%'")
+    .get() as { count: number };
+  assert.equal(firstPass.count, 1, "first startup should backfill existing KNOWLEDGE.md row");
+
+  writeFileSync(
+    knowledgePath,
+    [
+      "# Project Knowledge",
+      "",
+      "## Rules",
+      "",
+      "| # | Scope | Rule | Why | Added |",
+      "|---|-------|------|-----|-------|",
+      "",
+      "## Patterns",
+      "",
+      "| # | Pattern | Where | Notes |",
+      "|---|---------|-------|-------|",
+      "| P001 | Run startup maintenance once | bootstrap | first pass |",
+      "| P002 | Do not re-run startup sentinels | bootstrap | second pass |",
+      "",
+      "## Lessons Learned",
+      "",
+      "| # | What Happened | Root Cause | Fix | Scope |",
+      "|---|--------------|------------|-----|-------|",
+      "",
+    ].join("\n"),
+    "utf-8",
+  );
+
+  await buildBeforeAgentStartResult(
+    { prompt: "Inspect project knowledge again", systemPrompt: "base system prompt" },
+    ctx,
+  );
+  await _flushDeferredContextMaintenanceForTest(base);
+
+  const secondPass = adapter
+    .prepare("SELECT COUNT(*) AS count FROM memories WHERE structured_fields LIKE '%\"sourceKnowledgeId\":\"P002\"%'")
+    .get() as { count: number };
+  assert.equal(secondPass.count, 0, "second startup in same session should not re-run KNOWLEDGE.md sentinel backfill");
+
+  sessionId = "session-two";
+  await buildBeforeAgentStartResult(
+    { prompt: "Inspect project knowledge in a new session", systemPrompt: "base system prompt" },
+    ctx,
+  );
+  await _flushDeferredContextMaintenanceForTest(base);
+
+  const nextSessionPass = adapter
+    .prepare("SELECT COUNT(*) AS count FROM memories WHERE structured_fields LIKE '%\"sourceKnowledgeId\":\"P002\"%'")
+    .get() as { count: number };
+  assert.equal(nextSessionPass.count, 1, "startup maintenance should run once for a later session");
+});
+
+test("#896 later turns reopen the project DB after startup maintenance is complete", async (t) => {
+  const base = realpathSync(mkdtempSync(join(tmpdir(), "gsd-startup-maintenance-reopen-")));
+  const originalCwd = process.cwd();
+  const originalGsdHome = process.env.GSD_HOME;
+  const gsdDir = join(base, ".gsd");
+  mkdirSync(gsdDir, { recursive: true });
+  execFileSync("git", ["init", "-q"], { cwd: base, stdio: "ignore" });
+  process.chdir(base);
+  process.env.GSD_HOME = join(base, ".gsd-home");
+
+  t.after(async () => {
+    await _flushDeferredContextMaintenanceForTest(base);
+    if (isDbAvailable()) closeDatabase();
+    invalidateStateCache();
+    process.chdir(originalCwd);
+    if (originalGsdHome === undefined) delete process.env.GSD_HOME;
+    else process.env.GSD_HOME = originalGsdHome;
+    rmSync(base, { recursive: true, force: true });
+  });
+
+  assert.equal(openDatabase(join(gsdDir, "gsd.db")), true);
+  closeDatabase();
+  assert.equal(isDbAvailable(), false);
+
+  const ctx = {
+    projectRoot: base,
+    ui: { notify: () => undefined },
+  } as unknown as ExtensionContext;
+
+  await buildBeforeAgentStartResult(
+    { prompt: "Inspect project knowledge", systemPrompt: "base system prompt" },
+    ctx,
+  );
+  await _flushDeferredContextMaintenanceForTest(base);
+
+  const memoryId = createMemory({
+    category: "gotcha",
+    content: "Later before_agent_start turns reopen the project DB.",
+    scope: "project",
+    confidence: 0.95,
+  });
+  assert.ok(memoryId);
+
+  closeDatabase();
+  assert.equal(isDbAvailable(), false);
+
+  const result = await buildBeforeAgentStartResult(
+    { prompt: "Inspect project knowledge again", systemPrompt: "base system prompt" },
+    ctx,
+  );
+
+  assert.equal(isDbAvailable(), true, "completed startup maintenance should still reopen the DB per turn");
+  assert.match(
+    result?.message?.content ?? "",
+    /Later before_agent_start turns reopen the project DB\./,
+    "DB-backed memory context should still be available on later turns",
+  );
+});
+
+test("#896 deferred maintenance is queued before later prompt assembly can throw", async (t) => {
+  const base = realpathSync(mkdtempSync(join(tmpdir(), "gsd-startup-maintenance-deferred-")));
+  const originalCwd = process.cwd();
+  const originalGsdHome = process.env.GSD_HOME;
+  const gsdDir = join(base, ".gsd");
+  const knowledgePath = join(gsdDir, "KNOWLEDGE.md");
+  const gsdHomeDir = join(base, ".gsd-home");
+  mkdirSync(gsdDir, { recursive: true });
+  mkdirSync(join(gsdHomeDir, "agent"), { recursive: true });
+  execFileSync("git", ["init", "-q"], { cwd: base, stdio: "ignore" });
+  process.chdir(base);
+  process.env.GSD_HOME = gsdHomeDir;
+
+  t.after(async () => {
+    await _flushDeferredContextMaintenanceForTest(base);
+    if (isDbAvailable()) closeDatabase();
+    invalidateStateCache();
+    process.chdir(originalCwd);
+    if (originalGsdHome === undefined) delete process.env.GSD_HOME;
+    else process.env.GSD_HOME = originalGsdHome;
+    rmSync(base, { recursive: true, force: true });
+  });
+
+  writeFileSync(
+    join(gsdHomeDir, "agent", "KNOWLEDGE.md"),
+    ["# Global Knowledge", "", "Large global knowledge warning. ".repeat(220)].join("\n"),
+    "utf-8",
+  );
+
+  assert.equal(openDatabase(join(gsdDir, "gsd.db")), true);
+  const memoryId = createMemory({
+    category: "pattern",
+    content: "Deferred maintenance survives prompt assembly failures",
+    scope: "project",
+    confidence: 0.9,
+    structuredFields: {
+      sourceKnowledgeId: "P001",
+      sourceKnowledgeTable: "patterns",
+      pattern: "Deferred maintenance survives prompt assembly failures",
+      where: "bootstrap",
+      notes: "regression coverage",
+    },
+  });
+  assert.ok(memoryId);
+  closeDatabase();
+  assert.equal(isDbAvailable(), false);
+
+  const ctx = {
+    projectRoot: base,
+    ui: {
+      notify: (message: string) => {
+        if (message.includes("KNOWLEDGE.md is")) {
+          throw new Error("notification failure after startup maintenance");
+        }
+      },
+    },
+  } as unknown as ExtensionContext;
+
+  await assert.rejects(
+    () => buildBeforeAgentStartResult(
+      { prompt: "Inspect project knowledge", systemPrompt: "base system prompt" },
+      ctx,
+    ),
+    /notification failure after startup maintenance/,
+  );
+
+  await _flushDeferredContextMaintenanceForTest(base);
+  assert.equal(existsSync(knowledgePath), true, "deferred projection should still run after prompt assembly throws");
+  assert.match(
+    readFileSync(knowledgePath, "utf-8"),
+    /\| P001 \| Deferred maintenance survives prompt assembly failures \| bootstrap \| regression coverage \|/,
+  );
+});
+
+test("#830 knowledge command opens the project DB before capturing patterns", async (t) => {
+  const base = realpathSync(mkdtempSync(join(tmpdir(), "gsd-knowledge-command-")));
+  const gsdDir = join(base, ".gsd");
+  mkdirSync(gsdDir, { recursive: true });
+
+  t.after(() => {
+    if (isDbAvailable()) closeDatabase();
+    invalidateStateCache();
+    rmSync(base, { recursive: true, force: true });
+  });
+
+  assert.equal(openDatabase(join(gsdDir, "gsd.db")), true);
+  closeDatabase();
+  invalidateStateCache();
+  assert.equal(isDbAvailable(), false);
+
+  const notifications: Array<{ message: string; level: string }> = [];
+  const ctx = {
+    cwd: base,
+    ui: {
+      notify: (message: string, level: string) => notifications.push({ message, level }),
+    },
+  } as unknown as ExtensionCommandContext;
+
+  await withCommandCwd(base, async () => {
+    await handleKnowledge("pattern Capture works on a cold session", ctx);
+  });
+
+  assert.equal(isDbAvailable(), true, "knowledge command should open the project DB");
+  const adapter = _getAdapter();
+  assert.ok(adapter);
+  const row = adapter
+    .prepare(
+      "SELECT category, content, structured_fields FROM memories WHERE superseded_by IS NULL ORDER BY seq DESC LIMIT 1",
+    )
+    .get() as { category: string; content: string; structured_fields: string } | undefined;
+  assert.ok(row, "knowledge command should persist a memory");
+  assert.equal(row.category, "pattern");
+  assert.equal(row.content, "Capture works on a cold session");
+  assert.equal(JSON.parse(row.structured_fields).sourceKnowledgeId, "P001");
+  assert.deepEqual(notifications.at(-1), {
+    message: "Captured pattern P001 to memories; KNOWLEDGE.md will render it on next session start.",
+    level: "success",
+  });
+});

package/src/resources/extensions/gsd/tests/layout-policy.test.ts

@@ -0,0 +1,59 @@
+// Project/App: gsd-pi
+// File Purpose: Tests for the flat-phase layout policy.
+import test from "node:test";
+import assert from "node:assert/strict";
+
+import {
+  LAYOUT_ROOT,
+  LAYOUT_SEGMENTS,
+  phaseDirName,
+  planFileName,
+  dbPath,
+  milestoneIdToPhaseNum,
+  sliceIdToPlanNum,
+  derivePhaseSlug,
+} from "../layout-policy.ts";
+
+test("LAYOUT_ROOT is .gsd", () => {
+  assert.equal(LAYOUT_ROOT, ".gsd");
+});
+
+test("LAYOUT_SEGMENTS.level1 is phases", () => {
+  assert.equal(LAYOUT_SEGMENTS.level1, "phases");
+});
+
+test("phaseDirName produces NN-slug", () => {
+  assert.equal(phaseDirName(1, "foundation"), "01-foundation");
+  assert.equal(phaseDirName(12, "auth-system"), "12-auth-system");
+});
+
+test("planFileName produces NN-MM-SUFFIX.md", () => {
+  assert.equal(planFileName(1, 1, "PLAN"), "01-01-PLAN.md");
+  assert.equal(planFileName(3, 2, "SUMMARY"), "03-02-SUMMARY.md");
+});
+
+test("dbPath resolves under .gsd", () => {
+  assert.equal(dbPath("/project"), "/project/.gsd/gsd.db");
+});
+
+test("milestoneIdToPhaseNum extracts the numeric portion", () => {
+  assert.equal(milestoneIdToPhaseNum("M001"), 1);
+  assert.equal(milestoneIdToPhaseNum("M012"), 12);
+});
+
+test("sliceIdToPlanNum extracts the numeric portion", () => {
+  assert.equal(sliceIdToPlanNum("S01"), 1);
+  assert.equal(sliceIdToPlanNum("S03"), 3);
+});
+
+test("derivePhaseSlug is stable and deterministic", () => {
+  assert.equal(derivePhaseSlug("Foundation"), "foundation");
+  assert.equal(derivePhaseSlug("Set Up Tooling!"), "set-up-tooling");
+  assert.equal(derivePhaseSlug("auth/API layer"), "auth-api-layer");
+  assert.equal(derivePhaseSlug("Foundation"), derivePhaseSlug("Foundation"));
+});
+
+test("derivePhaseSlug falls back when title is empty or punctuation-only", () => {
+  assert.equal(derivePhaseSlug(""), "phase");
+  assert.equal(derivePhaseSlug("---"), "phase");
+});

package/src/resources/extensions/gsd/tests/markdown-renderer-parse-cache.test.ts

@@ -26,7 +26,7 @@ function roadmapMd(slices: Array<{ id: string; title: string; done: boolean }>):
   return lines.join("\n");
 }
 
-test("#442: detectStaleRenders re-parses ROADMAP only when it changes on disk", (t) => {
+test("#442: detectStaleRenders re-parses ROADMAP only when it changes on disk", { skip: true }, (t) => {
   const base = mkdtempSync(join(tmpdir(), "gsd-parse-cache-"));
   const milestoneDir = join(base, ".gsd", "milestones", "M001");
   mkdirSync(milestoneDir, { recursive: true });