gsd-pi 2.58.0-dev.778d6ac → 2.58.0-dev.d63175c

package/src/resources/extensions/gsd/tests/guided-flow-session-isolation.test.ts

@@ -1,97 +0,0 @@
-/**
- * Regression test for #2985 Bugs 3 & 4:
- *   Bug 3 — module-level pendingAutoStart singleton clobbers concurrent sessions.
- *   Bug 4 — getDiscussionMilestoneId() returns wrong project's milestone under concurrency.
- *
- * pendingAutoStart must be keyed by basePath so concurrent discuss sessions
- * in different projects are independent.  getDiscussionMilestoneId() must accept
- * a basePath parameter to perform a keyed lookup.
- */
-
-import { describe, test, beforeEach } from "node:test";
-import assert from "node:assert/strict";
-
-import {
-  getDiscussionMilestoneId,
-  setPendingAutoStart,
-  clearPendingAutoStart,
-} from "../guided-flow.ts";
-
-// ─── Tests ─────────────────────────────────────────────────────────────────
-
-describe("#2985 Bug 3 — concurrent discuss sessions must be independent", () => {
-  beforeEach(() => {
-    clearPendingAutoStart();
-  });
-
-  test("second session does not clobber first session's pending auto-start", () => {
-    // Simulate two concurrent discuss sessions for different projects
-    const projectA = "/projects/alpha";
-    const projectB = "/projects/beta";
-
-    setPendingAutoStart(projectA, {
-      basePath: projectA,
-      milestoneId: "M001-aaa111",
-    });
-
-    setPendingAutoStart(projectB, {
-      basePath: projectB,
-      milestoneId: "M002-bbb222",
-    });
-
-    // Both sessions should be retrievable
-    const milestoneA = getDiscussionMilestoneId(projectA);
-    const milestoneB = getDiscussionMilestoneId(projectB);
-
-    assert.equal(milestoneA, "M001-aaa111", "projectA's milestone should be preserved");
-    assert.equal(milestoneB, "M002-bbb222", "projectB's milestone should be preserved");
-  });
-
-  test("clearing one session does not affect the other", () => {
-    const projectA = "/projects/alpha";
-    const projectB = "/projects/beta";
-
-    setPendingAutoStart(projectA, { basePath: projectA, milestoneId: "M001-aaa111" });
-    setPendingAutoStart(projectB, { basePath: projectB, milestoneId: "M002-bbb222" });
-
-    // Clear only projectA
-    clearPendingAutoStart(projectA);
-
-    assert.equal(getDiscussionMilestoneId(projectA), null, "projectA should be cleared");
-    assert.equal(getDiscussionMilestoneId(projectB), "M002-bbb222", "projectB should survive");
-  });
-});
-
-describe("#2985 Bug 4 — getDiscussionMilestoneId must be keyed by basePath", () => {
-  beforeEach(() => {
-    clearPendingAutoStart();
-  });
-
-  test("getDiscussionMilestoneId(basePath) returns correct milestone for each project", () => {
-    setPendingAutoStart("/proj/a", { basePath: "/proj/a", milestoneId: "M001" });
-    setPendingAutoStart("/proj/b", { basePath: "/proj/b", milestoneId: "M002" });
-
-    assert.equal(getDiscussionMilestoneId("/proj/a"), "M001");
-    assert.equal(getDiscussionMilestoneId("/proj/b"), "M002");
-    assert.equal(getDiscussionMilestoneId("/proj/unknown"), null);
-  });
-
-  test("getDiscussionMilestoneId() without basePath returns null when multiple sessions exist", () => {
-    setPendingAutoStart("/proj/a", { basePath: "/proj/a", milestoneId: "M001" });
-    setPendingAutoStart("/proj/b", { basePath: "/proj/b", milestoneId: "M002" });
-
-    // Without a key, the function should not blindly return the first entry
-    const result = getDiscussionMilestoneId();
-    // When there's ambiguity (multiple sessions), it should return null
-    // to force callers to be explicit
-    assert.equal(result, null, "should not return arbitrary milestone when multiple sessions exist");
-  });
-
-  test("getDiscussionMilestoneId() without basePath returns the milestone when only one session", () => {
-    setPendingAutoStart("/proj/a", { basePath: "/proj/a", milestoneId: "M001" });
-
-    // With only one session, backward compat — return it
-    const result = getDiscussionMilestoneId();
-    assert.equal(result, "M001", "should return the only active milestone for backward compat");
-  });
-});

package/src/resources/extensions/gsd/tests/hook-key-parsing.test.ts

@@ -1,107 +0,0 @@
-import { describe, it } 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 gsdDir = join(__dirname, "..");
-
-/**
- * Regression tests for #2826: hook/* completed-unit keys were parsed
- * incorrectly by forensics + doctor, causing false-positive missing-artifact
- * errors for all hook units.
- *
- * The root cause: `key.indexOf("/")` splits "hook/telegram-progress/M007/S01"
- * into unitType="hook" + unitId="telegram-progress/M007/S01" instead of
- * unitType="hook/telegram-progress" + unitId="M007/S01".
- */
-
-describe("splitCompletedKey (#2826)", () => {
-  it("is exported from forensics.ts", () => {
-    const source = readFileSync(join(gsdDir, "forensics.ts"), "utf-8");
-    assert.ok(
-      source.includes("export function splitCompletedKey"),
-      "forensics.ts must export splitCompletedKey helper",
-    );
-  });
-
-  it("splits simple unit types correctly", async () => {
-    const { splitCompletedKey } = await import("../forensics.ts");
-    const result = splitCompletedKey("execute-task/M007/S01/T01");
-    assert.deepStrictEqual(result, {
-      unitType: "execute-task",
-      unitId: "M007/S01/T01",
-    });
-  });
-
-  it("splits hook unit types preserving the compound hook/<hookName> prefix", async () => {
-    const { splitCompletedKey } = await import("../forensics.ts");
-    const result = splitCompletedKey("hook/telegram-progress/M007/S01");
-    assert.deepStrictEqual(result, {
-      unitType: "hook/telegram-progress",
-      unitId: "M007/S01",
-    });
-  });
-
-  it("splits hook unit types with task-level unitId", async () => {
-    const { splitCompletedKey } = await import("../forensics.ts");
-    const result = splitCompletedKey("hook/telegram-progress/M007/S02/T01");
-    assert.deepStrictEqual(result, {
-      unitType: "hook/telegram-progress",
-      unitId: "M007/S02/T01",
-    });
-  });
-
-  it("returns null for malformed keys without a slash", async () => {
-    const { splitCompletedKey } = await import("../forensics.ts");
-    assert.strictEqual(splitCompletedKey("noslash"), null);
-  });
-
-  it("returns null for malformed hook keys with only 'hook/' and no more segments", async () => {
-    const { splitCompletedKey } = await import("../forensics.ts");
-    // "hook/someName" has no unitId segment after the hook name
-    assert.strictEqual(splitCompletedKey("hook/someName"), null);
-  });
-});
-
-describe("forensics detectMissingArtifacts uses splitCompletedKey (#2826)", () => {
-  it("does not use indexOf for key splitting", () => {
-    const source = readFileSync(join(gsdDir, "forensics.ts"), "utf-8");
-    // Extract only the detectMissingArtifacts function body
-    const fnStart = source.indexOf("function detectMissingArtifacts");
-    assert.ok(fnStart !== -1, "detectMissingArtifacts must exist");
-    const fnBody = source.slice(fnStart, source.indexOf("\n}\n", fnStart) + 3);
-
-    assert.ok(
-      !fnBody.includes('key.indexOf("/")'),
-      "detectMissingArtifacts must not use key.indexOf('/') — use splitCompletedKey instead",
-    );
-    assert.ok(
-      fnBody.includes("splitCompletedKey"),
-      "detectMissingArtifacts must use splitCompletedKey helper",
-    );
-  });
-});
-
-describe("doctor-runtime-checks uses splitCompletedKey (#2826)", () => {
-  it("does not use indexOf for key splitting in orphaned-key check", () => {
-    const source = readFileSync(
-      join(gsdDir, "doctor-runtime-checks.ts"),
-      "utf-8",
-    );
-    // Find the orphaned completed-units section
-    const sectionStart = source.indexOf("Orphaned completed-units");
-    assert.ok(sectionStart !== -1, "orphaned completed-units section must exist");
-    const sectionBody = source.slice(sectionStart, source.indexOf("} catch", sectionStart));
-
-    assert.ok(
-      !sectionBody.includes('key.indexOf("/")'),
-      "doctor orphaned-key check must not use key.indexOf('/') — use splitCompletedKey instead",
-    );
-    assert.ok(
-      sectionBody.includes("splitCompletedKey"),
-      "doctor orphaned-key check must use splitCompletedKey helper",
-    );
-  });
-});

package/src/resources/extensions/gsd/tests/integration/doctor-false-positives.test.ts

@@ -1,243 +0,0 @@
-import { describe, test } from 'node:test';
-import assert from 'node:assert/strict';
-import { mkdtempSync, mkdirSync, rmSync, writeFileSync, existsSync, readdirSync } from "node:fs";
-import { join } from "node:path";
-import { tmpdir } from "node:os";
-
-import { runGSDDoctor } from "../../doctor.js";
-import { parsePlan } from "../../parsers-legacy.js";
-
-// ── Helpers ─────────────────────────────────────────────────────────────────
-
-function makeBase(): { base: string; gsd: string; mDir: string } {
-  const base = mkdtempSync(join(tmpdir(), "gsd-doctor-fp-"));
-  const gsd = join(base, ".gsd");
-  const mDir = join(gsd, "milestones", "M001");
-  mkdirSync(join(mDir, "slices"), { recursive: true });
-  return { base, gsd, mDir };
-}
-
-function writeRoadmap(mDir: string, content: string): void {
-  writeFileSync(join(mDir, "M001-ROADMAP.md"), content);
-}
-
-function writeSlice(mDir: string, sliceId: string, planContent: string): string {
-  const sDir = join(mDir, "slices", sliceId);
-  const tDir = join(sDir, "tasks");
-  mkdirSync(tDir, { recursive: true });
-  writeFileSync(join(sDir, `${sliceId}-PLAN.md`), planContent);
-  return sDir;
-}
-
-describe('doctor false-positives (#3105)', async () => {
-
-  // ═══════════════════════════════════════════════════════════════════════════
-  // Bug 1: Orphaned worktree directory recreated by appendDoctorHistory
-  // ═══════════════════════════════════════════════════════════════════════════
-
-  test('Bug 1: orphaned worktree check ignores dirs containing only .gsd/doctor-history.jsonl', async () => {
-    // Simulate: a worktree dir that only contains .gsd/doctor-history.jsonl
-    // (created by appendDoctorHistory writing to the worktree-scoped path).
-    // The orphan check should NOT warn about this directory.
-    const { base, gsd } = makeBase();
-    writeRoadmap(join(gsd, "milestones", "M001"), `# M001: Test\n\n## Slices\n- [ ] **S01: Slice** \`risk:low\` \`depends:[]\`\n  > After this: done\n`);
-    writeSlice(join(gsd, "milestones", "M001"), "S01", "# S01: Slice\n\n**Goal:** G\n**Demo:** D\n\n## Tasks\n- [ ] **T01: Task** `est:10m`\n  Pending.\n");
-
-    // Create a worktree directory that only has .gsd/doctor-history.jsonl
-    const wtDir = join(gsd, "worktrees", "M042");
-    const wtGsdDir = join(wtDir, ".gsd");
-    mkdirSync(wtGsdDir, { recursive: true });
-    writeFileSync(join(wtGsdDir, "doctor-history.jsonl"), '{"ts":"2026-01-01","ok":true}\n');
-
-    const result = await runGSDDoctor(base, { fix: false });
-
-    // Should NOT produce worktree_directory_orphaned for a dir that only has doctor history
-    const orphanIssues = result.issues.filter(
-      i => i.code === "worktree_directory_orphaned" && i.unitId === "M042"
-    );
-    assert.equal(orphanIssues.length, 0,
-      "should not warn about worktree dir that only contains .gsd/doctor-history.jsonl");
-
-    rmSync(base, { recursive: true, force: true });
-  });
-
-  // ═══════════════════════════════════════════════════════════════════════════
-  // Bug 2: blocker_discovered + all tasks done = unfixable deadlock
-  // ═══════════════════════════════════════════════════════════════════════════
-
-  test('Bug 2: blocker_discovered with all tasks done should not warn (implicitly resolved)', async () => {
-    // Scenario: blocker was discovered and resolved within the same task.
-    // blocker_discovered: true, no REPLAN, but all tasks are done.
-    // Neither blocker_discovered_no_replan nor stale_replan_file should fire.
-    const { base, mDir } = makeBase();
-    writeRoadmap(mDir, `# M001: Blocker Test\n\n## Slices\n- [ ] **S01: Slice** \`risk:low\` \`depends:[]\`\n  > After this: done\n`);
-    const sDir = writeSlice(mDir, "S01",
-      "# S01: Slice\n\n**Goal:** G\n**Demo:** D\n\n## Tasks\n- [x] **T01: Task** `est:10m`\n  Done.\n");
-    writeFileSync(join(sDir, "tasks", "T01-SUMMARY.md"), `---
-id: T01
-parent: S01
-milestone: M001
-provides: []
-requires: []
-affects: []
-key_files: []
-key_decisions: []
-patterns_established: []
-observability_surfaces: []
-drill_down_paths: []
-duration: 10m
-verification_result: passed
-completed_at: 2026-01-01T00:00:00Z
-blocker_discovered: true
----
-
-# T01: Task
-
-**Done**
-
-## What Happened
-Found a blocker, resolved it in-task.
-
-## Diagnostics
-- log
-`);
-
-    const result = await runGSDDoctor(base, { fix: false });
-
-    // Should NOT produce blocker_discovered_no_replan when all tasks are done
-    const blockerIssues = result.issues.filter(i => i.code === "blocker_discovered_no_replan");
-    assert.equal(blockerIssues.length, 0,
-      "should not warn about blocker_discovered when all tasks are done (blocker was implicitly resolved)");
-
-    // Also should NOT produce stale_replan_file (no REPLAN exists, so this shouldn't fire anyway)
-    const staleReplanIssues = result.issues.filter(i => i.code === "stale_replan_file");
-    assert.equal(staleReplanIssues.length, 0,
-      "should not produce stale_replan_file when no REPLAN exists");
-
-    rmSync(base, { recursive: true, force: true });
-  });
-
-  test('Bug 2: blocker_discovered with incomplete tasks should still warn', async () => {
-    // Sanity check: when there IS an incomplete task and blocker_discovered, warn as before.
-    const { base, mDir } = makeBase();
-    writeRoadmap(mDir, `# M001: Blocker Warn Test\n\n## Slices\n- [ ] **S01: Slice** \`risk:low\` \`depends:[]\`\n  > After this: done\n`);
-    const sDir = writeSlice(mDir, "S01",
-      "# S01: Slice\n\n**Goal:** G\n**Demo:** D\n\n## Tasks\n- [x] **T01: Task A** `est:10m`\n  Done.\n- [ ] **T02: Task B** `est:10m`\n  Pending.\n");
-    writeFileSync(join(sDir, "tasks", "T01-SUMMARY.md"), `---
-id: T01
-parent: S01
-milestone: M001
-provides: []
-requires: []
-affects: []
-key_files: []
-key_decisions: []
-patterns_established: []
-observability_surfaces: []
-drill_down_paths: []
-duration: 10m
-verification_result: passed
-completed_at: 2026-01-01T00:00:00Z
-blocker_discovered: true
----
-
-# T01: Task A
-
-**Done**
-
-## What Happened
-Found blocker, but T02 is still pending.
-
-## Diagnostics
-- log
-`);
-
-    const result = await runGSDDoctor(base, { fix: false });
-
-    const blockerIssues = result.issues.filter(i => i.code === "blocker_discovered_no_replan");
-    assert.ok(blockerIssues.length > 0,
-      "should still warn about blocker_discovered when some tasks are not done");
-
-    rmSync(base, { recursive: true, force: true });
-  });
-
-  // ═══════════════════════════════════════════════════════════════════════════
-  // Bug 3: Multi-task plan — T02+ outside ## Tasks section
-  // ═══════════════════════════════════════════════════════════════════════════
-
-  test('Bug 3: parsePlan finds all tasks even when interleaved with detail sections', () => {
-    // Multi-task plan where T02 checkbox appears after T01's ## Steps heading,
-    // which ends the ## Tasks section for extractSection().
-    const planContent = `# S01: Demo Slice
-
-**Goal:** Build the demo
-**Demo:** Run it
-
-## Must-Haves
-- Feature A
-
-## Tasks
-- [x] **T01: First task** \`est:30m\`
-  Implement the first thing.
-## Steps
-1. Step one
-2. Step two
-## Must-Haves
-- Requirement A
-- [x] **T02: Second task** \`est:1h\`
-  Implement the second thing.
-## Steps
-1. Step one
-2. Step two
-`;
-
-    const plan = parsePlan(planContent);
-    const taskIds = plan.tasks.map(t => t.id);
-
-    assert.ok(taskIds.includes("T01"), "should find T01");
-    assert.ok(taskIds.includes("T02"), "should find T02 even when after T01 detail headings");
-    assert.equal(plan.tasks.length, 2, "should find exactly 2 tasks");
-  });
-
-  test('Bug 3: task_file_not_in_plan should not fire for T02 in multi-task plan', async () => {
-    const { base, mDir } = makeBase();
-    writeRoadmap(mDir, `# M001: Multi-Task Test\n\n## Slices\n- [ ] **S01: Slice** \`risk:low\` \`depends:[]\`\n  > After this: done\n`);
-
-    // Plan with interleaved headings (the problematic format)
-    const sDir = writeSlice(mDir, "S01", `# S01: Demo Slice
-
-**Goal:** Build the demo
-**Demo:** Run it
-
-## Must-Haves
-- Feature A
-
-## Tasks
-- [x] **T01: First task** \`est:30m\`
-  Implement the first thing.
-## Steps
-1. Step one
-## Must-Haves
-- Req A
-- [x] **T02: Second task** \`est:1h\`
-  Implement the second thing.
-## Steps
-1. Step one
-`);
-
-    // Both tasks have summaries on disk
-    writeFileSync(join(sDir, "tasks", "T01-SUMMARY.md"), "---\nstatus: done\ncompleted_at: 2026-01-01T00:00:00Z\n---\n# T01\nDone.\n");
-    writeFileSync(join(sDir, "tasks", "T02-SUMMARY.md"), "---\nstatus: done\ncompleted_at: 2026-01-01T00:00:00Z\n---\n# T02\nDone.\n");
-
-    const result = await runGSDDoctor(base, { fix: false });
-
-    // T02 should NOT be flagged as "not in plan"
-    const notInPlan = result.issues.filter(
-      i => i.code === "task_file_not_in_plan" && i.message.includes("T02")
-    );
-    assert.equal(notInPlan.length, 0,
-      "should not report T02 as 'not in plan' when it exists in the interleaved plan format");
-
-    rmSync(base, { recursive: true, force: true });
-  });
-});

package/src/resources/extensions/gsd/tests/integration/gitignore-staging-2570.test.ts

@@ -1,150 +0,0 @@
-/**
- * gitignore-staging-2570.test.ts — Regression tests for #2570.
- *
- * Verifies that:
- * 1. isGsdGitignored() detects when .gsd is covered by .gitignore
- * 2. The rethink prompt uses {{commitInstruction}} instead of hardcoded git add .gsd/
- * 3. rethink.ts passes the correct commitInstruction based on gitignore state
- *
- * Uses real temporary git repos — no mocks.
- */
-
-import test from "node:test";
-import assert from "node:assert/strict";
-import { execFileSync } from "node:child_process";
-import {
-  mkdirSync,
-  mkdtempSync,
-  readFileSync,
-  rmSync,
-  writeFileSync,
-} from "node:fs";
-import { join } from "node:path";
-import { tmpdir } from "node:os";
-
-// Dynamic import — isGsdGitignored is the function under test (may not exist yet during TDD red phase)
-const { isGsdGitignored } = await import("../../gitignore.ts");
-
-// ─── Helpers ─────────────────────────────────────────────────────────
-
-function git(dir: string, ...args: string[]): string {
-  return execFileSync("git", args, { cwd: dir, stdio: "pipe", encoding: "utf-8" }).trim();
-}
-
-function makeTempRepo(): string {
-  const dir = mkdtempSync(join(tmpdir(), "gsd-staging-2570-"));
-  git(dir, "init");
-  git(dir, "config", "user.email", "test@test.com");
-  git(dir, "config", "user.name", "Test");
-  writeFileSync(join(dir, "README.md"), "# init\n");
-  git(dir, "add", "-A");
-  git(dir, "commit", "-m", "init");
-  git(dir, "branch", "-M", "main");
-  return dir;
-}
-
-function cleanup(dir: string): void {
-  try {
-    rmSync(dir, { recursive: true, force: true });
-  } catch {
-    // ignore
-  }
-}
-
-// ─── isGsdGitignored ─────────────────────────────────────────────────
-
-test("isGsdGitignored returns true when .gsd is in .gitignore (#2570)", (t) => {
-  const dir = makeTempRepo();
-  t.after(() => { cleanup(dir); });
-
-  writeFileSync(join(dir, ".gitignore"), ".gsd\n");
-  assert.equal(isGsdGitignored(dir), true);
-});
-
-test("isGsdGitignored returns true when .gsd/ (with slash) is in .gitignore", (t) => {
-  const dir = makeTempRepo();
-  t.after(() => { cleanup(dir); });
-
-  writeFileSync(join(dir, ".gitignore"), ".gsd/\n");
-  // Create .gsd directory so git check-ignore can match the directory-only pattern
-  mkdirSync(join(dir, ".gsd"), { recursive: true });
-  assert.equal(isGsdGitignored(dir), true);
-});
-
-test("isGsdGitignored returns false when .gsd is NOT in .gitignore", (t) => {
-  const dir = makeTempRepo();
-  t.after(() => { cleanup(dir); });
-
-  writeFileSync(join(dir, ".gitignore"), "node_modules/\n");
-  assert.equal(isGsdGitignored(dir), false);
-});
-
-test("isGsdGitignored returns false when no .gitignore exists", (t) => {
-  const dir = makeTempRepo();
-  t.after(() => { cleanup(dir); });
-
-  // No .gitignore — default
-  assert.equal(isGsdGitignored(dir), false);
-});
-
-// ─── rethink.md prompt template ─────────────────────────────────────
-
-test("rethink.md prompt uses {{commitInstruction}} not hardcoded git add .gsd/ (#2570)", () => {
-  const promptPath = join(
-    import.meta.dirname!,
-    "..",
-    "..",
-    "prompts",
-    "rethink.md",
-  );
-  const content = readFileSync(promptPath, "utf-8");
-
-  // Must NOT contain hardcoded `git add .gsd/`
-  assert.ok(
-    !content.includes("git add .gsd/"),
-    `rethink.md must not contain hardcoded "git add .gsd/" — use {{commitInstruction}} instead.\nFound: ${content.match(/.*git add .gsd\/.*/)?.[0]}`,
-  );
-
-  // Must contain the {{commitInstruction}} placeholder
-  assert.ok(
-    content.includes("{{commitInstruction}}"),
-    "rethink.md must use {{commitInstruction}} template variable for commit step",
-  );
-});
-
-// ─── smartStage respects .gitignore for .gsd/ (#2570) ───────────────
-
-test("smartStage does not stage .gsd/ files when .gsd is gitignored (#2570)", async (t) => {
-  // This imports GitServiceImpl to test through the public commit() method
-  // which calls smartStage() internally.
-  const { GitServiceImpl } = await import("../../git-service.ts");
-
-  const dir = makeTempRepo();
-  t.after(() => { cleanup(dir); });
-
-  // Add .gsd to .gitignore
-  writeFileSync(join(dir, ".gitignore"), ".gsd\nnode_modules/\n");
-  git(dir, "add", ".gitignore");
-  git(dir, "commit", "-m", "add gitignore with .gsd");
-
-  // Create .gsd/ milestone artifacts (NOT tracked, NOT symlinked)
-  mkdirSync(join(dir, ".gsd", "milestones", "M001", "slices", "S01"), { recursive: true });
-  writeFileSync(join(dir, ".gsd", "milestones", "M001", "slices", "S01", "S01-PLAN.md"), "# Plan");
-  writeFileSync(join(dir, ".gsd", "DECISIONS.md"), "# Decisions");
-
-  // Create a normal source file
-  writeFileSync(join(dir, "src.ts"), "export const x = 1;");
-
-  // Commit through GitServiceImpl (uses smartStage internally)
-  const svc = new GitServiceImpl(dir);
-  const msg = svc.commit({ message: "test: should not include .gsd files" });
-  assert.ok(msg !== null, "commit should succeed");
-
-  // Check what was committed
-  const committed = git(dir, "show", "--name-only", "HEAD");
-  assert.ok(committed.includes("src.ts"), "source files ARE committed");
-  assert.ok(
-    !committed.includes(".gsd/"),
-    `gitignored .gsd/ files must NOT be staged by smartStage.\nCommitted files: ${committed}`,
-  );
-});