gsd-pi 2.78.1-dev.84a383f51 → 2.78.1

package/src/resources/extensions/gsd/tests/native-git-infra-errors.test.ts

@@ -1,50 +0,0 @@
-import test from "node:test";
-import assert from "node:assert/strict";
-import { chmodSync, mkdirSync, mkdtempSync, rmSync, writeFileSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-
-import { git } from "./test-utils.ts";
-import { GSD_GIT_ERROR } from "../errors.js";
-
-test("nativeAddAllWithExclusions preserves infrastructure failures from git add", async () => {
-  const base = mkdtempSync(join(tmpdir(), "gsd-native-git-infra-"));
-  const repo = join(base, "repo");
-  const bin = join(base, "bin");
-  mkdirSync(repo);
-  mkdirSync(bin);
-
-  const fakeGit = join(bin, "git");
-  writeFileSync(
-    fakeGit,
-    "#!/bin/sh\n" +
-      "echo 'fatal: ENFILE: file table overflow' >&2\n" +
-      "exit 1\n",
-    "utf-8",
-  );
-  chmodSync(fakeGit, 0o755);
-
-  const originalPath = process.env.PATH ?? "";
-  try {
-    git(repo, "init");
-    git(repo, "config", "user.email", "test@example.com");
-    git(repo, "config", "user.name", "Test User");
-    writeFileSync(join(repo, "README.md"), "# Test\n", "utf-8");
-
-    process.env.PATH = `${bin}:${originalPath}`;
-    const { nativeAddAllWithExclusions } = await import("../native-git-bridge.ts");
-
-    assert.throws(
-      () => nativeAddAllWithExclusions(repo, [".gsd/activity/"]),
-      (err) => {
-        const shaped = err as { code?: string; stderr?: string; message?: string };
-        assert.notEqual(shaped.code, GSD_GIT_ERROR);
-        assert.match(`${shaped.stderr ?? ""}${shaped.message ?? ""}`, /ENFILE/);
-        return true;
-      },
-    );
-  } finally {
-    process.env.PATH = originalPath;
-    rmSync(base, { recursive: true, force: true });
-  }
-});

package/src/resources/extensions/gsd/tests/register-hooks-compaction-checkpoint.test.ts

@@ -1,93 +0,0 @@
-import test from "node:test";
-import assert from "node:assert/strict";
-import { existsSync, mkdirSync, mkdtempSync, readFileSync, rmSync, writeFileSync } from "node:fs";
-import { join } from "node:path";
-import { tmpdir } from "node:os";
-
-import { registerHooks } from "../bootstrap/register-hooks.ts";
-import { parseContinue } from "../files.ts";
-import { closeDatabase } from "../gsd-db.ts";
-import { deriveState, invalidateStateCache } from "../state.ts";
-
-function createPlanningFixtureBase(): string {
-  const base = mkdtempSync(join(tmpdir(), "gsd-compact-checkpoint-"));
-  const milestoneDir = join(base, ".gsd", "milestones", "M001");
-  const sliceDir = join(milestoneDir, "slices", "S01");
-  mkdirSync(sliceDir, { recursive: true });
-
-  writeFileSync(
-    join(milestoneDir, "M001-ROADMAP.md"),
-    `# M001: Test Milestone
-
-**Vision:** Validate compaction checkpointing.
-
-## Slices
-
-- [ ] **S01: Test Slice** \`risk:low\` \`depends:[]\`
-  > After this: Slice is done.
-`,
-  );
-
-  writeFileSync(
-    join(sliceDir, "S01-PLAN.md"),
-    `# S01: Test Slice
-
-**Goal:** Validate planning checkpoint.
-**Demo:** Checkpoint exists after compaction.
-
-## Tasks
-`,
-  );
-
-  return base;
-}
-
-test("register-hooks writes CONTINUE checkpoint during planning phase without active task (#4258)", async (t) => {
-  const base = createPlanningFixtureBase();
-  const originalCwd = process.cwd();
-  process.chdir(base);
-  invalidateStateCache();
-  closeDatabase();
-
-  t.after(() => {
-    invalidateStateCache();
-    closeDatabase();
-    process.chdir(originalCwd);
-    rmSync(base, { recursive: true, force: true });
-  });
-
-  const state = await deriveState(base);
-  assert.equal(state.phase, "planning", "fixture should derive planning phase");
-  assert.equal(state.activeMilestone?.id, "M001", "fixture should have active milestone");
-  assert.equal(state.activeSlice?.id, "S01", "fixture should have active slice");
-  assert.equal(state.activeTask, null, "fixture should have no active task");
-
-  const handlers = new Map<string, Array<(event: any, ctx?: any) => Promise<any> | any>>();
-  const pi = {
-    on(event: string, handler: (event: any, ctx?: any) => Promise<any> | any) {
-      const existing = handlers.get(event) ?? [];
-      existing.push(handler);
-      handlers.set(event, existing);
-    },
-  } as any;
-
-  registerHooks(pi, []);
-
-  const compactHandlers = handlers.get("session_before_compact");
-  assert.ok(compactHandlers && compactHandlers.length > 0, "session_before_compact handler should be registered");
-
-  for (const handler of compactHandlers ?? []) {
-    await handler({});
-  }
-
-  const continuePath = join(base, ".gsd", "milestones", "M001", "slices", "S01", "S01-CONTINUE.md");
-  assert.ok(existsSync(continuePath), "compaction should create slice CONTINUE checkpoint");
-
-  const parsed = parseContinue(readFileSync(continuePath, "utf-8"));
-  assert.equal(parsed.frontmatter.milestone, "M001");
-  assert.equal(parsed.frontmatter.slice, "S01");
-  assert.equal(parsed.frontmatter.task, "none", "planning checkpoint should use non-task placeholder");
-  assert.equal(parsed.frontmatter.status, "compacted");
-  assert.match(parsed.completedWork, /planning phase/i, "completed-work should capture non-executing phase context");
-  assert.match(parsed.nextAction, /slice S01/i, "next action should route resume to the active slice");
-});

package/src/resources/extensions/gsd/tests/system-context-message-routing.test.ts

@@ -1,101 +0,0 @@
-// GSD bootstrap + system-context-message-routing.test — regression coverage
-// for #5019. `memoryBlock` is FTS-queried against the user prompt and changes
-// per call; embedding it in the cached system prefix invalidates Anthropic
-// prompt-cache hits on every request. The fix routes memory through the
-// existing context-message channel (volatile user-message suffix) and combines
-// it with any active guided-execute or forensics injection.
-
-import { describe, test } from "node:test";
-import assert from "node:assert/strict";
-
-import { buildContextMessage } from "../bootstrap/system-context.ts";
-
-describe("buildContextMessage (#5019 — memory routing)", () => {
-  test("returns null when nothing to inject", () => {
-    const result = buildContextMessage({
-      memoryBlock: "",
-      injection: null,
-      forensicsInjection: null,
-    });
-    assert.equal(result, null);
-  });
-
-  test("whitespace-only memoryBlock counts as empty", () => {
-    const result = buildContextMessage({
-      memoryBlock: "   \n\n   ",
-      injection: null,
-      forensicsInjection: null,
-    });
-    assert.equal(result, null);
-  });
-
-  test("memory-only path emits gsd-memory message with trimmed content", () => {
-    const result = buildContextMessage({
-      memoryBlock: "\n\n[MEMORY]\nrule one\nrule two\n\n",
-      injection: null,
-      forensicsInjection: null,
-    });
-    assert.ok(result, "expected a context message");
-    assert.equal(result.customType, "gsd-memory");
-    assert.equal(result.content, "[MEMORY]\nrule one\nrule two");
-    assert.equal(result.display, false);
-  });
-
-  test("guided-execute injection alone emits gsd-guided-context", () => {
-    const result = buildContextMessage({
-      memoryBlock: "",
-      injection: "[GUIDED]\nexecute T01",
-      forensicsInjection: null,
-    });
-    assert.ok(result);
-    assert.equal(result.customType, "gsd-guided-context");
-    assert.equal(result.content, "[GUIDED]\nexecute T01");
-  });
-
-  test("forensics injection alone emits gsd-forensics", () => {
-    const result = buildContextMessage({
-      memoryBlock: "",
-      injection: null,
-      forensicsInjection: "[FORENSICS]\ninvestigation context",
-    });
-    assert.ok(result);
-    assert.equal(result.customType, "gsd-forensics");
-    assert.equal(result.content, "[FORENSICS]\ninvestigation context");
-  });
-
-  test("memory + guided injection: memory prepended, customType is gsd-guided-context", () => {
-    const result = buildContextMessage({
-      memoryBlock: "[MEMORY]\nrule one",
-      injection: "[GUIDED]\nexecute T01",
-      forensicsInjection: null,
-    });
-    assert.ok(result);
-    assert.equal(result.customType, "gsd-guided-context");
-    assert.equal(result.content, "[MEMORY]\nrule one\n\n[GUIDED]\nexecute T01");
-  });
-
-  test("memory + forensics: memory prepended, customType is gsd-forensics", () => {
-    const result = buildContextMessage({
-      memoryBlock: "[MEMORY]\nrule one",
-      injection: null,
-      forensicsInjection: "[FORENSICS]\ninvestigation context",
-    });
-    assert.ok(result);
-    assert.equal(result.customType, "gsd-forensics");
-    assert.equal(result.content, "[MEMORY]\nrule one\n\n[FORENSICS]\ninvestigation context");
-  });
-
-  test("guided takes precedence over forensics when both are somehow present", () => {
-    // The caller in buildBeforeAgentStartResult already gates forensics on
-    // `!injection`, but the helper's documented priority is guided > forensics.
-    // Test the contract directly so a future refactor can't silently flip it.
-    const result = buildContextMessage({
-      memoryBlock: "",
-      injection: "[GUIDED]",
-      forensicsInjection: "[FORENSICS]",
-    });
-    assert.ok(result);
-    assert.equal(result.customType, "gsd-guided-context");
-    assert.equal(result.content, "[GUIDED]");
-  });
-});

package/src/resources/extensions/gsd/worktree-session-state.ts

@@ -1,35 +0,0 @@
-// GSD worktree session state
-let originalCwd: string | null = null;
-
-export function getWorktreeOriginalCwd(): string | null {
-  return originalCwd;
-}
-
-export function setWorktreeOriginalCwd(cwd: string): void {
-  originalCwd = cwd;
-}
-
-export function clearWorktreeOriginalCwd(): void {
-  originalCwd = null;
-}
-
-export function ensureWorktreeOriginalCwdFromPath(cwd: string = process.cwd()): string | null {
-  if (originalCwd) return originalCwd;
-  const marker = `${/\\/.test(cwd) ? "\\" : "/"}.gsd${/\\/.test(cwd) ? "\\" : "/"}worktrees${/\\/.test(cwd) ? "\\" : "/"}`;
-  const markerIdx = cwd.indexOf(marker);
-  if (markerIdx !== -1) {
-    originalCwd = cwd.slice(0, markerIdx);
-  }
-  return originalCwd;
-}
-
-export function getActiveWorktreeName(): string | null {
-  if (!originalCwd) return null;
-  const cwd = process.cwd();
-  const wtDir = `${originalCwd.replace(/[\\/]+$/, "")}/.gsd/worktrees`.replaceAll("\\", "/");
-  const normalizedCwd = cwd.replaceAll("\\", "/");
-  if (!normalizedCwd.startsWith(`${wtDir}/`)) return null;
-  const rel = normalizedCwd.slice(wtDir.length + 1);
-  const name = rel.split("/")[0];
-  return name || null;
-}

package/src/resources/extensions/mcp-client/tests/global-config.test.ts

@@ -1,91 +0,0 @@
-/**
- * Regression test for #4757 — readConfigs() must also read the global
- * ~/.gsd/mcp.json (resolved as $GSD_HOME/mcp.json when GSD_HOME is set).
- *
- * Behaviour test against the exported getServerConfig — no source grep.
- * The fixture is anchored via $GSD_HOME so the test never touches the
- * developer's real ~/.gsd directory.
- */
-
-import test, { before, after } from "node:test";
-import assert from "node:assert/strict";
-import { writeFileSync, mkdtempSync, realpathSync, rmSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-
-import { getServerConfig } from "../index.js";
-
-let cwdDir: string;
-let gsdHomeDir: string;
-let originalCwd: string;
-let originalGsdHome: string | undefined;
-
-before(() => {
-	originalCwd = process.cwd();
-	originalGsdHome = process.env.GSD_HOME;
-
-	// realpathSync resolves any symlink in tmpdir() — on macOS /var → /private/var
-	// so process.cwd() after chdir matches what mkdtempSync returned.
-	cwdDir = realpathSync(mkdtempSync(join(tmpdir(), "mcp-cwd-")));
-	gsdHomeDir = realpathSync(mkdtempSync(join(tmpdir(), "mcp-gsdhome-")));
-
-	// Project-local fixture (also defines `shared-server` for the precedence test)
-	writeFileSync(
-		join(cwdDir, ".mcp.json"),
-		JSON.stringify({
-			mcpServers: {
-				"project-server": { command: "echo", args: ["proj"] },
-				"shared-server": { command: "echo", args: ["from-project"] },
-			},
-		}),
-		"utf-8",
-	);
-
-	// Global fixture rooted at $GSD_HOME (also defines `shared-server` to test
-	// that project-local takes precedence on name collision)
-	writeFileSync(
-		join(gsdHomeDir, "mcp.json"),
-		JSON.stringify({
-			mcpServers: {
-				"global-server": { command: "echo", args: ["glob"] },
-				"shared-server": { command: "echo", args: ["from-global"] },
-			},
-		}),
-		"utf-8",
-	);
-
-	process.chdir(cwdDir);
-	process.env.GSD_HOME = gsdHomeDir;
-});
-
-after(() => {
-	process.chdir(originalCwd);
-	if (originalGsdHome === undefined) delete process.env.GSD_HOME;
-	else process.env.GSD_HOME = originalGsdHome;
-	try { rmSync(cwdDir, { recursive: true, force: true }); } catch { /* best-effort */ }
-	try { rmSync(gsdHomeDir, { recursive: true, force: true }); } catch { /* best-effort */ }
-});
-
-test("#4757: getServerConfig resolves servers declared in $GSD_HOME/mcp.json", () => {
-	const cfg = getServerConfig("global-server");
-	assert.ok(cfg, "server defined in $GSD_HOME/mcp.json must resolve");
-	assert.equal(cfg?.name, "global-server");
-	assert.equal(cfg?.sourcePath, join(gsdHomeDir, "mcp.json"));
-});
-
-test("#4757: project-local servers still resolve when global config exists", () => {
-	const cfg = getServerConfig("project-server");
-	assert.ok(cfg, "project-local server must continue to resolve");
-	assert.equal(cfg?.sourcePath, join(cwdDir, ".mcp.json"));
-});
-
-test("#4757: project-local config wins on server-name collision", () => {
-	const cfg = getServerConfig("shared-server");
-	assert.ok(cfg, "shared server must resolve");
-	assert.equal(
-		cfg?.sourcePath,
-		join(cwdDir, ".mcp.json"),
-		"project-local config must take precedence over $GSD_HOME on name collision",
-	);
-	assert.deepEqual(cfg?.args, ["from-project"]);
-});

package/src/resources/skills/create-gsd-extension/templates/templates.test.ts

@@ -1,58 +0,0 @@
-// GSD-2 — Extension template import path validation
-// Copyright (c) 2026 Jeremy McSpadden <jeremy@fluxlabs.net>
-
-import { describe, it } from "node:test";
-import assert from "node:assert/strict";
-import { readFileSync, readdirSync, statSync } from "node:fs";
-import { join, dirname, relative } from "node:path";
-import { fileURLToPath } from "node:url";
-
-const __dirname = dirname(fileURLToPath(import.meta.url));
-const skillRoot = join(__dirname, "..");
-
-function walk(dir: string): string[] {
-	const out: string[] = [];
-	for (const entry of readdirSync(dir)) {
-		const full = join(dir, entry);
-		if (statSync(full).isDirectory()) {
-			out.push(...walk(full));
-		} else {
-			out.push(full);
-		}
-	}
-	return out;
-}
-
-describe("extension templates use @gsd/* imports", () => {
-	const templates = ["extension-skeleton.ts", "stateful-tool-skeleton.ts"];
-
-	for (const template of templates) {
-		it(`${template} uses @gsd/pi-coding-agent (not @mariozechner)`, () => {
-			const content = readFileSync(join(__dirname, template), "utf-8");
-			assert.ok(content.includes("@gsd/pi-coding-agent"), `Expected @gsd/pi-coding-agent import in ${template}`);
-			assert.ok(!content.includes("@mariozechner/"), `Found stale @mariozechner/ import in ${template}`);
-		});
-	}
-
-	it("extension-skeleton.ts uses @gsd/pi-ai for StringEnum", () => {
-		const content = readFileSync(join(__dirname, "extension-skeleton.ts"), "utf-8");
-		assert.ok(content.includes("@gsd/pi-ai"), "Expected @gsd/pi-ai import");
-	});
-
-	it("stateful-tool-skeleton.ts uses @gsd/pi-tui", () => {
-		const content = readFileSync(join(__dirname, "stateful-tool-skeleton.ts"), "utf-8");
-		assert.ok(content.includes("@gsd/pi-tui"), "Expected @gsd/pi-tui import");
-	});
-
-	it("no @mariozechner/ references anywhere in create-gsd-extension/", () => {
-		const offenders: string[] = [];
-		for (const file of walk(skillRoot)) {
-			if (file.endsWith("templates.test.ts")) continue;
-			const content = readFileSync(file, "utf-8");
-			if (content.includes("@mariozechner/")) {
-				offenders.push(relative(skillRoot, file));
-			}
-		}
-		assert.deepEqual(offenders, [], `Stale @mariozechner/ references found in: ${offenders.join(", ")}`);
-	});
-});