@opengsd/gsd-pi 1.0.1 → 1.0.2

package/src/resources/extensions/gsd/tests/workspace-index.test.ts

@@ -1,38 +0,0 @@
-import test from "node:test";
-import assert from "node:assert/strict";
-import { mkdtempSync, mkdirSync, rmSync, writeFileSync } from "node:fs";
-import { join } from "node:path";
-import { tmpdir } from "node:os";
-import { getSuggestedNextCommands, indexWorkspace, listDoctorScopeSuggestions } from "../workspace-index.ts";
-
-test("workspace index: indexes active milestone/slice/task and suggests commands", async () => {
-  const base = mkdtempSync(join(tmpdir(), "gsd-workspace-index-test-"));
-  const gsd = join(base, ".gsd");
-  const mDir = join(gsd, "milestones", "M001");
-  const sDir = join(mDir, "slices", "S01");
-  mkdirSync(join(sDir, "tasks"), { recursive: true });
-
-  writeFileSync(join(mDir, "M001-ROADMAP.md"), `# M001: Demo Milestone\n\n## Slices\n- [ ] **S01: Demo Slice** \`risk:low\` \`depends:[]\`\n  > After this: demo works\n`);
-  writeFileSync(join(sDir, "S01-PLAN.md"), `# S01: Demo Slice\n\n**Goal:** Demo\n**Demo:** Demo\n\n## Must-Haves\n- done\n\n## Tasks\n- [ ] **T01: Implement thing** \`est:10m\`\n  Task is in progress.\n`);
-  writeFileSync(join(sDir, "tasks", "T01-PLAN.md"), `# T01: Implement thing\n\n## Steps\n- do it\n`);
-
-  try {
-    const index = await indexWorkspace(base);
-    assert.equal(index.active.milestoneId, "M001");
-    assert.equal(index.active.sliceId, "S01");
-    assert.equal(index.active.taskId, "T01");
-    assert.ok(index.scopes.some(s => s.scope === "M001/S01"));
-    assert.ok(index.scopes.some(s => s.scope === "M001/S01/T01"));
-
-    const suggestions = await listDoctorScopeSuggestions(base);
-    assert.equal(suggestions[0].value, "M001/S01");
-    assert.ok(suggestions.some(item => item.value === "M001/S01/T01"));
-
-    const commands = await getSuggestedNextCommands(base);
-    assert.ok(commands.includes("/gsd auto"));
-    assert.ok(commands.includes("/gsd doctor M001/S01"));
-    assert.ok(commands.includes("/gsd status"));
-  } finally {
-    rmSync(base, { recursive: true, force: true });
-  }
-});

package/src/resources/extensions/gsd/tests/worktree-integration.test.ts

@@ -1,216 +0,0 @@
-/**
- * Worktree Integration Tests
- *
- * Tests the full lifecycle of GSD operations inside a worktree:
- * - Branch namespacing (gsd/<wt>/<M>/<S> instead of gsd/<M>/<S>)
- * - getMainBranch returns worktree/<name> inside a worktree
- * - Parallel worktrees don't conflict on branch names
- * - State derivation works correctly inside worktrees
- */
-
-import { mkdtempSync, mkdirSync, rmSync, writeFileSync, readFileSync, existsSync } from "node:fs";
-import { join } from "node:path";
-import { tmpdir } from "node:os";
-import { execSync } from "node:child_process";
-
-import {
-  createWorktree,
-  listWorktrees,
-  removeWorktree,
-} from "../worktree-manager.ts";
-
-import {
-  detectWorktreeName,
-  getCurrentBranch,
-  getMainBranch,
-  getSliceBranchName,
-  autoCommitCurrentBranch,
-  SLICE_BRANCH_RE,
-  _resetServiceCache,
-} from "../worktree.ts";
-
-import { deriveState } from "../state.ts";
-import { _clearGsdRootCache } from "../paths.ts";
-import { describe, test } from 'node:test';
-import assert from 'node:assert/strict';
-
-function run(command: string, cwd: string): string {
-  return execSync(command, { cwd, stdio: ["ignore", "pipe", "pipe"], encoding: "utf-8" }).trim();
-}
-
-// ─── Test repo setup ──────────────────────────────────────────────────────────
-
-const base = mkdtempSync(join(tmpdir(), "gsd-wt-integration-"));
-run("git init -b main", base);
-run("git config user.name 'Pi Test'", base);
-run("git config user.email 'pi@example.com'", base);
-
-// Create a project with one milestone and two slices
-mkdirSync(join(base, ".gsd", "milestones", "M001", "slices", "S01", "tasks"), { recursive: true });
-mkdirSync(join(base, ".gsd", "milestones", "M001", "slices", "S02", "tasks"), { recursive: true });
-writeFileSync(join(base, "README.md"), "# Test Project\n", "utf-8");
-writeFileSync(
-  join(base, ".gsd", "milestones", "M001", "M001-ROADMAP.md"),
-  [
-    "# M001: Demo",
-    "",
-    "## Slices",
-    "- [ ] **S01: First** `risk:low` `depends:[]`",
-    "  > After this: part one works",
-    "- [ ] **S02: Second** `risk:low` `depends:[]`",
-    "  > After this: part two works",
-  ].join("\n") + "\n",
-  "utf-8",
-);
-writeFileSync(
-  join(base, ".gsd", "milestones", "M001", "slices", "S01", "S01-PLAN.md"),
-  "# S01: First\n\n**Goal:** Demo\n**Demo:** Demo\n\n## Must-Haves\n- done\n\n## Tasks\n- [ ] **T01: Implement** `est:10m`\n  do it\n",
-  "utf-8",
-);
-writeFileSync(
-  join(base, ".gsd", "milestones", "M001", "slices", "S02", "S02-PLAN.md"),
-  "# S02: Second\n\n**Goal:** Demo\n**Demo:** Demo\n\n## Must-Haves\n- done\n\n## Tasks\n- [ ] **T01: Implement** `est:10m`\n  do it\n",
-  "utf-8",
-);
-run("git add .", base);
-run('git commit -m "chore: init"', base);
-
-describe('worktree-integration', async () => {
-  // Isolate from user's global preferences (which may have git.main_branch set).
-  // Reset caches so getService() creates a fresh instance with empty preferences.
-  const originalHome = process.env.HOME;
-  const fakeHome = mkdtempSync(join(tmpdir(), "gsd-fake-home-"));
-  process.env.HOME = fakeHome;
-  _clearGsdRootCache();
-  _resetServiceCache();
-
-  // ── Verify main tree baseline ──────────────────────────────────────────────
-
-  console.log("\n=== Main tree baseline ===");
-  assert.deepStrictEqual(getMainBranch(base), "main", "main tree getMainBranch returns main");
-  assert.deepStrictEqual(detectWorktreeName(base), null, "main tree not detected as worktree");
-
-  // ── Create worktree and verify detection ───────────────────────────────────
-
-  console.log("\n=== Create worktree ===");
-  const wt = createWorktree(base, "alpha");
-  assert.ok(existsSync(wt.path), "worktree created on disk");
-  assert.deepStrictEqual(wt.branch, "worktree/alpha", "worktree branch name");
-
-  console.log("\n=== Worktree detection ===");
-  assert.deepStrictEqual(detectWorktreeName(wt.path), "alpha", "detectWorktreeName inside worktree");
-  assert.deepStrictEqual(getMainBranch(wt.path), "worktree/alpha", "getMainBranch returns worktree branch inside worktree");
-
-  // ── Verify current branch inside worktree ──────────────────────────────────
-
-  console.log("\n=== Worktree initial branch ===");
-  assert.deepStrictEqual(getCurrentBranch(wt.path), "worktree/alpha", "worktree starts on its own branch");
-
-  // ── Verify branch name helper ──────────────────────────────────────────────
-
-  console.log("\n=== getSliceBranchName with worktree ===");
-  assert.deepStrictEqual(getSliceBranchName("M001", "S01", "alpha"), "gsd/alpha/M001/S01", "explicit worktree param");
-  assert.deepStrictEqual(getSliceBranchName("M001", "S01"), "gsd/M001/S01", "no worktree param = plain branch");
-
-  // ── Slice branch creation and detection inside worktree ────────────────────
-
-  console.log("\n=== Slice branch in worktree ===");
-  const sliceBranch = getSliceBranchName("M001", "S01", "alpha");
-  run(`git checkout -b ${sliceBranch}`, wt.path);
-  assert.deepStrictEqual(getCurrentBranch(wt.path), "gsd/alpha/M001/S01", "worktree-namespaced slice branch");
-  assert.ok(SLICE_BRANCH_RE.test(getCurrentBranch(wt.path)), "slice branch regex matches namespaced branch");
-
-  // ── Do work on slice branch, then merge to worktree branch ─────────────────
-
-  console.log("\n=== Work and merge slice in worktree ===");
-  writeFileSync(join(wt.path, "feature.txt"), "new feature\n", "utf-8");
-  run("git add .", wt.path);
-  run('git commit -m "feat: add feature"', wt.path);
-
-  // Checkout worktree base branch and merge slice branch
-  run("git checkout worktree/alpha", wt.path);
-  assert.deepStrictEqual(getCurrentBranch(wt.path), "worktree/alpha", "back on worktree branch");
-
-  run(`git merge --no-ff ${sliceBranch} -m "feat(M001/S01): First"`, wt.path);
-  run(`git branch -d ${sliceBranch}`, wt.path);
-  assert.deepStrictEqual(getCurrentBranch(wt.path), "worktree/alpha", "still on worktree branch after merge");
-  assert.ok(readFileSync(join(wt.path, "feature.txt"), "utf-8").includes("new feature"), "merge brought feature to worktree branch");
-
-  // Verify slice branch is gone
-  const branches = run("git branch", base);
-  assert.ok(!branches.includes("gsd/alpha/M001/S01"), "slice branch cleaned up");
-
-  // ── Second slice in same worktree ──────────────────────────────────────────
-
-  console.log("\n=== Second slice in worktree ===");
-  const sliceBranch2 = getSliceBranchName("M001", "S02", "alpha");
-  run(`git checkout -b ${sliceBranch2}`, wt.path);
-  assert.deepStrictEqual(getCurrentBranch(wt.path), "gsd/alpha/M001/S02", "on S02 namespaced branch");
-
-  writeFileSync(join(wt.path, "feature2.txt"), "second feature\n", "utf-8");
-  run("git add .", wt.path);
-  run('git commit -m "feat: add feature 2"', wt.path);
-
-  run("git checkout worktree/alpha", wt.path);
-  run(`git merge --no-ff ${sliceBranch2} -m "feat(M001/S02): Second"`, wt.path);
-  run(`git branch -d ${sliceBranch2}`, wt.path);
-  assert.deepStrictEqual(getCurrentBranch(wt.path), "worktree/alpha", "back on worktree branch");
-
-  // ── Parallel worktrees don't conflict ──────────────────────────────────────
-
-  console.log("\n=== Parallel worktrees ===");
-  const wt2 = createWorktree(base, "beta");
-  assert.deepStrictEqual(getMainBranch(wt2.path), "worktree/beta", "second worktree has its own base branch");
-
-  // Both worktrees can create S01 branches without conflict
-  const betaBranch = getSliceBranchName("M001", "S01", "beta");
-  run(`git checkout -b ${betaBranch}`, wt2.path);
-  assert.deepStrictEqual(getCurrentBranch(wt2.path), "gsd/beta/M001/S01", "beta has its own namespaced branch");
-
-  // Alpha worktree can re-create S01 too (it was already merged+deleted earlier)
-  const alphaReBranch = getSliceBranchName("M001", "S01", "alpha");
-  run(`git checkout -b ${alphaReBranch}`, wt.path);
-  assert.deepStrictEqual(getCurrentBranch(wt.path), "gsd/alpha/M001/S01", "alpha re-created S01");
-
-  // Both exist simultaneously
-  const allBranches = run("git branch", base);
-  assert.ok(allBranches.includes("gsd/alpha/M001/S01"), "alpha S01 branch exists");
-  assert.ok(allBranches.includes("gsd/beta/M001/S01"), "beta S01 branch exists");
-
-  // ── State derivation in worktree ───────────────────────────────────────────
-
-  console.log("\n=== State derivation in worktree ===");
-  // Switch alpha back to its base so deriveState sees milestone files
-  run("git checkout worktree/alpha", wt.path);
-  const state = await deriveState(wt.path);
-  assert.ok(state.activeMilestone !== null, "worktree has active milestone");
-  assert.deepStrictEqual(state.activeMilestone?.id, "M001", "correct milestone");
-
-  // ── autoCommitCurrentBranch in worktree ────────────────────────────────────
-
-  console.log("\n=== autoCommitCurrentBranch in worktree ===");
-  // Re-checkout the beta slice branch
-  run(`git checkout ${betaBranch}`, wt2.path);
-  writeFileSync(join(wt2.path, "dirty.txt"), "uncommitted\n", "utf-8");
-  const commitMsg = autoCommitCurrentBranch(wt2.path, "execute-task", "M001/S01/T01");
-  assert.ok(commitMsg !== null, "auto-commit works in worktree");
-  assert.deepStrictEqual(run("git status --short", wt2.path), "", "worktree clean after auto-commit");
-
-  // ── Cleanup ────────────────────────────────────────────────────────────────
-
-  console.log("\n=== Cleanup ===");
-  // Switch worktrees back to their base branches before removal
-  run("git checkout worktree/alpha", wt.path);
-  run("git checkout worktree/beta", wt2.path);
-  removeWorktree(base, "alpha", { deleteBranch: true });
-  removeWorktree(base, "beta", { deleteBranch: true });
-  assert.deepStrictEqual(listWorktrees(base).length, 0, "all worktrees removed");
-
-  rmSync(base, { recursive: true, force: true });
-
-  // Restore HOME and reset caches
-  process.env.HOME = originalHome;
-  _clearGsdRootCache();
-  _resetServiceCache();
-  rmSync(fakeHome, { recursive: true, force: true });
-});

package/src/resources/extensions/gsd/tests/worktree-path-injection.test.ts

@@ -1,242 +0,0 @@
-import test, { after } from "node:test";
-import assert from "node:assert/strict";
-import { mkdirSync, mkdtempSync, realpathSync, rmSync, writeFileSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-
-const ownsGsdHome = process.env.GSD_HOME_TEST_OVERRIDE === undefined;
-const previousGsdHome = process.env.GSD_HOME;
-const synthesizedGsdHome = join(tmpdir(), `gsd-test-home-${process.pid}-${Date.now()}`);
-process.env.GSD_HOME = process.env.GSD_HOME_TEST_OVERRIDE
-  ?? synthesizedGsdHome;
-
-after(() => {
-  if (ownsGsdHome) {
-    rmSync(synthesizedGsdHome, { recursive: true, force: true });
-  }
-  if (previousGsdHome === undefined) {
-    delete process.env.GSD_HOME;
-  } else {
-    process.env.GSD_HOME = previousGsdHome;
-  }
-});
-
-const { dispatchDirectPhase } = await import("../auto-direct-dispatch.ts");
-const {
-  buildDiscussMilestonePrompt,
-  buildParallelResearchSlicesPrompt,
-  buildRewriteDocsPrompt,
-} = await import("../auto-prompts.ts");
-const { invalidateStateCache } = await import("../state.ts");
-const { resolveAgentEnd, _resetPendingResolve } = await import("../auto/resolve.ts");
-const { runUnit } = await import("../auto/run-unit.ts");
-
-function writeMilestone(base: string, mid = "M001", title = "Worktree Path Injection"): void {
-  const milestoneDir = join(base, ".gsd", "milestones", mid);
-  mkdirSync(milestoneDir, { recursive: true });
-  writeFileSync(
-    join(milestoneDir, `${mid}-CONTEXT.md`),
-    `# ${mid}: ${title}\n\nContext.\n`,
-    "utf-8",
-  );
-  writeFileSync(
-    join(milestoneDir, `${mid}-ROADMAP.md`),
-    [
-      `# ${mid}: ${title}`,
-      "",
-      "## Slices",
-      "",
-      "- [ ] **S01: First slice** `risk:low` `depends:[]`",
-      "",
-    ].join("\n"),
-    "utf-8",
-  );
-}
-
-function makeLiveMilestoneWorktree(base: string, mid = "M001"): string {
-  const worktreeRoot = join(base, ".gsd", "worktrees", mid);
-  mkdirSync(worktreeRoot, { recursive: true });
-  writeFileSync(
-    join(worktreeRoot, ".git"),
-    `gitdir: ${join(base, ".git", "worktrees", mid)}\n`,
-    "utf-8",
-  );
-  writeMilestone(worktreeRoot, mid);
-  return worktreeRoot;
-}
-
-async function waitFor(condition: () => boolean, label: string): Promise<void> {
-  const rawTimeout = process.env.READABLE_WAIT_TIMEOUT_MS;
-  const parsedTimeout = rawTimeout === undefined ? NaN : Number.parseInt(rawTimeout, 10);
-  const timeoutMs = Number.isFinite(parsedTimeout) && parsedTimeout > 0 ? parsedTimeout : 1000;
-  const deadline = Date.now() + timeoutMs;
-
-  while (Date.now() < deadline) {
-    if (condition()) return;
-    await new Promise((resolve) => setTimeout(resolve, 5));
-  }
-  if (condition()) return;
-  assert.fail(`Timed out waiting for ${label} after ${timeoutMs}ms`);
-}
-
-test("runUnit passes basePath as workspaceRoot without changing process cwd", async (t) => {
-  _resetPendingResolve();
-
-  const originalCwd = process.cwd();
-  const base = realpathSync(mkdtempSync(join(tmpdir(), "gsd-rununit-base-")));
-  const drifted = realpathSync(mkdtempSync(join(tmpdir(), "gsd-rununit-drift-")));
-  t.after(() => {
-    process.chdir(originalCwd);
-    rmSync(base, { recursive: true, force: true });
-    rmSync(drifted, { recursive: true, force: true });
-  });
-
-  process.chdir(drifted);
-
-  let newSessionWorkspaceRoot: string | undefined;
-  let cwdAtNewSession: string | undefined;
-  const session = {
-    active: true,
-    basePath: base,
-    verbose: false,
-    cmdCtx: {
-      newSession: (options?: { workspaceRoot?: string }) => {
-        newSessionWorkspaceRoot = options?.workspaceRoot;
-        cwdAtNewSession = process.cwd();
-        return Promise.resolve({ cancelled: false });
-      },
-    },
-  } as any;
-  const pi = {
-    calls: [] as unknown[],
-    sendMessage(...args: unknown[]) {
-      this.calls.push(args);
-    },
-  } as any;
-  const ctx = { ui: { notify: () => {} }, model: { id: "test-model" } } as any;
-
-  const resultPromise = runUnit(ctx, pi, session, "task", "T01", "prompt");
-  await waitFor(() => pi.calls.length === 1, "runUnit dispatch");
-  resolveAgentEnd({ messages: [{ role: "assistant" }] });
-
-  const result = await resultPromise;
-  assert.equal(result.status, "completed");
-  assert.equal(newSessionWorkspaceRoot, base);
-  assert.equal(cwdAtNewSession, drifted);
-  assert.equal(process.cwd(), drifted);
-});
-
-test("runUnit does not chdir or cancel when basePath is not a live directory", async (t) => {
-  _resetPendingResolve();
-
-  const originalCwd = process.cwd();
-  const base = realpathSync(mkdtempSync(join(tmpdir(), "gsd-rununit-missing-base-")));
-  const drifted = realpathSync(mkdtempSync(join(tmpdir(), "gsd-rununit-missing-drift-")));
-  rmSync(base, { recursive: true, force: true });
-  t.after(() => {
-    process.chdir(originalCwd);
-    rmSync(drifted, { recursive: true, force: true });
-  });
-
-  process.chdir(drifted);
-
-  let newSessionWorkspaceRoot: string | undefined;
-  const session = {
-    active: true,
-    basePath: base,
-    verbose: false,
-    cmdCtx: {
-      newSession: (options?: { workspaceRoot?: string }) => {
-        newSessionWorkspaceRoot = options?.workspaceRoot;
-        return Promise.resolve({ cancelled: false });
-      },
-    },
-  } as any;
-  const pi = {
-    calls: [] as unknown[],
-    sendMessage(...args: unknown[]) {
-      this.calls.push(args);
-    },
-  } as any;
-  const ctx = { ui: { notify: () => {} }, model: { id: "test-model" } } as any;
-
-  const resultPromise = runUnit(ctx, pi, session, "task", "T01", "prompt");
-  await waitFor(() => pi.calls.length === 1, "runUnit dispatch");
-  resolveAgentEnd({ messages: [{ role: "assistant" }] });
-
-  const result = await resultPromise;
-  assert.equal(result.status, "completed");
-  assert.equal(newSessionWorkspaceRoot, base);
-  assert.equal(process.cwd(), drifted);
-});
-
-test("direct dispatch redirects to the canonical milestone worktree before newSession", async (t) => {
-  invalidateStateCache();
-
-  const originalCwd = process.cwd();
-  const base = realpathSync(mkdtempSync(join(tmpdir(), "gsd-direct-base-")));
-  const drifted = realpathSync(mkdtempSync(join(tmpdir(), "gsd-direct-drift-")));
-  writeMilestone(base);
-  const worktreeRoot = makeLiveMilestoneWorktree(base);
-
-  t.after(() => {
-    process.chdir(originalCwd);
-    rmSync(base, { recursive: true, force: true });
-    rmSync(drifted, { recursive: true, force: true });
-    invalidateStateCache();
-  });
-
-  process.chdir(drifted);
-
-  let newSessionWorkspaceRoot: string | undefined;
-  let sentPrompt: string | undefined;
-  const ctx = {
-    ui: { notify: () => {} },
-    newSession: async (options?: { workspaceRoot?: string }) => {
-      newSessionWorkspaceRoot = options?.workspaceRoot;
-      return { cancelled: false };
-    },
-  } as any;
-  const pi = {
-    sendMessage(message: { content: string }) {
-      sentPrompt = message.content;
-    },
-  } as any;
-
-  await dispatchDirectPhase(ctx, pi, "research-milestone", base);
-
-  assert.equal(newSessionWorkspaceRoot, worktreeRoot);
-  assert.equal(process.cwd(), drifted);
-  assert.ok(sentPrompt?.includes(worktreeRoot), "prompt should name the canonical worktree root");
-});
-
-test("worktree-aware prompt builders include the explicit working directory", async (t) => {
-  const base = realpathSync(mkdtempSync(join(tmpdir(), "gsd-prompt-base-")));
-  writeMilestone(base);
-  t.after(() => rmSync(base, { recursive: true, force: true }));
-
-  const prompts = await Promise.all([
-    buildDiscussMilestonePrompt("M001", "Worktree Path Injection", base),
-    buildParallelResearchSlicesPrompt(
-      "M001",
-      "Worktree Path Injection",
-      [{ id: "S01", title: "First slice" }],
-      base,
-    ),
-    buildRewriteDocsPrompt(
-      "M001",
-      "Worktree Path Injection",
-      null,
-      base,
-      [{ change: "Refresh docs", timestamp: "2026-04-27T00:00:00.000Z", appliedAt: "test" }] as any,
-    ),
-  ]);
-
-  assert.ok(prompts[0].includes("## Context Mode"), "discuss-milestone should include standalone Context Mode guidance");
-  assert.ok(prompts[0].includes("interview lane"), "discuss-milestone should render the interview lane");
-
-  for (const prompt of prompts) {
-    assert.match(prompt, /working directory/i);
-    assert.ok(prompt.includes(base), "prompt should include the provided working directory");
-  }
-});