gsd-pi 2.33.1 → 2.34.0-dev.bbb5216

package/dist/resources/extensions/gsd/tests/native-has-changes-cache.test.ts

@@ -1,61 +0,0 @@
-/**
- * Unit tests for the nativeHasChanges() fallback cache (10s TTL).
- *
- * Verifies:
- *   1. Cached result is returned within the TTL window
- *   2. Cache invalidates after TTL expires
- *   3. Cache invalidates when basePath changes
- */
-
-import test from 'node:test';
-import assert from 'node:assert/strict';
-import { nativeHasChanges, _resetHasChangesCache } from '../native-git-bridge.ts';
-
-// We can't easily mock gitExec or Date.now inside the module, so we test
-// the observable caching behaviour by calling the real function against
-// the current repo (which is a valid git checkout).
-
-const REPO_ROOT = process.cwd();
-
-test('nativeHasChanges: returns a boolean for the current repo', () => {
-  _resetHasChangesCache();
-  const result = nativeHasChanges(REPO_ROOT);
-  assert.strictEqual(typeof result, 'boolean', 'should return a boolean');
-});
-
-test('nativeHasChanges: second call within TTL returns same result (cache hit)', () => {
-  _resetHasChangesCache();
-  const first = nativeHasChanges(REPO_ROOT);
-  const second = nativeHasChanges(REPO_ROOT);
-  assert.strictEqual(first, second, 'cached result should match first call');
-});
-
-test('nativeHasChanges: different basePath invalidates cache', () => {
-  _resetHasChangesCache();
-
-  // Prime cache with REPO_ROOT
-  const first = nativeHasChanges(REPO_ROOT);
-
-  // Call with a different path — should NOT return the stale cached value
-  // (it will compute fresh). We just verify it doesn't throw and returns boolean.
-  const other = nativeHasChanges('/tmp');
-  assert.strictEqual(typeof other, 'boolean', 'should return boolean for different path');
-
-  // After switching path, calling with REPO_ROOT again should recompute
-  const third = nativeHasChanges(REPO_ROOT);
-  assert.strictEqual(typeof third, 'boolean', 'should return boolean after path switch');
-});
-
-test('nativeHasChanges: cache expires after TTL', () => {
-  _resetHasChangesCache();
-
-  // Prime the cache
-  nativeHasChanges(REPO_ROOT);
-
-  // Manually expire the cache by resetting it (simulates TTL expiry)
-  _resetHasChangesCache();
-
-  // This call should recompute (not use stale data)
-  const result = nativeHasChanges(REPO_ROOT);
-  assert.strictEqual(typeof result, 'boolean', 'should recompute after cache reset');
-});

package/dist/resources/extensions/gsd/tests/next-milestone-id.test.ts

@@ -1,23 +0,0 @@
-import test from "node:test";
-import assert from "node:assert/strict";
-import { nextMilestoneId, maxMilestoneNum } from "../guided-flow.ts";
-
-test("nextMilestoneId: empty array returns M001", () => {
-  assert.equal(maxMilestoneNum([]), 0);
-  assert.equal(nextMilestoneId([]), "M001");
-});
-
-test("nextMilestoneId: sequential IDs return next in sequence", () => {
-  assert.equal(nextMilestoneId(["M001", "M002", "M003"]), "M004");
-  assert.equal(maxMilestoneNum(["M001", "M002", "M003"]), 3);
-});
-
-test("nextMilestoneId: gaps use max, not fill", () => {
-  assert.equal(nextMilestoneId(["M001", "M003"]), "M004");
-  assert.equal(maxMilestoneNum(["M001", "M003"]), 3);
-});
-
-test("nextMilestoneId: non-numeric directory names ignored", () => {
-  assert.equal(nextMilestoneId(["M001", "notes", ".DS_Store", "M003"]), "M004");
-  assert.equal(maxMilestoneNum(["M001", "notes", ".DS_Store", "M003"]), 3);
-});

package/dist/resources/extensions/gsd/tests/none-mode-gates.test.ts

@@ -1,114 +0,0 @@
-/**
- * none-mode-gates.test.ts — Tests for isolation-mode gate functions.
- *
- * Verifies that shouldUseWorktreeIsolation(), getIsolationMode(), and
- * getActiveAutoWorktreeContext() behave correctly across all three
- * isolation modes (none, branch, worktree) and at baseline (no prefs).
- *
- * Uses the writeRunnerPreferences pattern from doctor-git.test.ts:
- * PROJECT_PREFERENCES_PATH is a module-level constant frozen at import
- * time, so process.chdir() won't redirect preference loading. We write
- * prefs to the runner's cwd .gsd/preferences.md and clean up in finally.
- */
-
-import { mkdirSync, writeFileSync, rmSync, existsSync } from "node:fs";
-import { join } from "node:path";
-import { homedir } from "node:os";
-
-import { shouldUseWorktreeIsolation } from "../auto.ts";
-import { getIsolationMode } from "../preferences.ts";
-import { getActiveAutoWorktreeContext } from "../auto-worktree.ts";
-import { invalidateAllCaches } from "../cache.ts";
-import { createTestContext } from "./test-helpers.ts";
-
-const { assertEq, assertTrue, report } = createTestContext();
-
-// --- Preferences helpers (same pattern as doctor-git.test.ts K001) ---
-
-const RUNNER_PREFS_PATH = join(process.cwd(), ".gsd", "preferences.md");
-
-function writeRunnerPreferences(isolation: "none" | "worktree" | "branch"): void {
-  mkdirSync(join(process.cwd(), ".gsd"), { recursive: true });
-  writeFileSync(RUNNER_PREFS_PATH, `---\ngit:\n  isolation: "${isolation}"\n---\n`);
-}
-
-function removeRunnerPreferences(): void {
-  try { rmSync(RUNNER_PREFS_PATH); } catch { /* ignore if already gone */ }
-}
-
-// --- Tests ---
-
-// Test 1: shouldUseWorktreeIsolation returns false for none
-console.log("Test 1: shouldUseWorktreeIsolation returns false for none");
-try {
-  writeRunnerPreferences("none");
-  invalidateAllCaches();
-  assertEq(shouldUseWorktreeIsolation(), false, "shouldUseWorktreeIsolation() with none prefs");
-} finally {
-  removeRunnerPreferences();
-  invalidateAllCaches();
-}
-
-// Test 2: shouldUseWorktreeIsolation returns false for branch
-console.log("Test 2: shouldUseWorktreeIsolation returns false for branch");
-try {
-  writeRunnerPreferences("branch");
-  invalidateAllCaches();
-  assertEq(shouldUseWorktreeIsolation(), false, "shouldUseWorktreeIsolation() with branch prefs");
-} finally {
-  removeRunnerPreferences();
-  invalidateAllCaches();
-}
-
-// Test 3: shouldUseWorktreeIsolation returns true for worktree
-console.log("Test 3: shouldUseWorktreeIsolation returns true for worktree");
-try {
-  writeRunnerPreferences("worktree");
-  invalidateAllCaches();
-  assertEq(shouldUseWorktreeIsolation(), true, "shouldUseWorktreeIsolation() with worktree prefs");
-} finally {
-  removeRunnerPreferences();
-  invalidateAllCaches();
-}
-
-// Test 4: shouldUseWorktreeIsolation returns true for no prefs (default)
-// Skip if global prefs exist — they override the default and this test
-// cannot control ~/.gsd/preferences.md.
-const globalPrefsExist = existsSync(join(homedir(), ".gsd", "preferences.md"))
-  || existsSync(join(homedir(), ".gsd", "PREFERENCES.md"));
-if (!globalPrefsExist) {
-  console.log("Test 4: shouldUseWorktreeIsolation returns true for no prefs (default)");
-  try {
-    removeRunnerPreferences(); // ensure no prefs file
-    invalidateAllCaches();
-    assertEq(shouldUseWorktreeIsolation(), true, "shouldUseWorktreeIsolation() with no prefs (default worktree)");
-  } finally {
-    invalidateAllCaches();
-  }
-} else {
-  console.log("Test 4: SKIPPED — global prefs file exists, cannot test bare default");
-}
-
-// Test 5: getIsolationMode returns "none" with none prefs
-console.log("Test 5: getIsolationMode returns 'none' with none prefs");
-try {
-  writeRunnerPreferences("none");
-  invalidateAllCaches();
-  assertEq(getIsolationMode(), "none", "getIsolationMode() with none prefs");
-} finally {
-  removeRunnerPreferences();
-  invalidateAllCaches();
-}
-
-// Test 6: getActiveAutoWorktreeContext returns null at baseline
-console.log("Test 6: getActiveAutoWorktreeContext returns null at baseline");
-assertEq(getActiveAutoWorktreeContext(), null, "getActiveAutoWorktreeContext() returns null without enterAutoWorktree()");
-
-// Test 7: System prompt worktree block absent without active worktree
-console.log("Test 7: System prompt worktree block absent without active worktree");
-{
-  const ctx = getActiveAutoWorktreeContext();
-  assertTrue(ctx === null, "getActiveAutoWorktreeContext() null confirms system prompt worktree block will not be injected");
-}
-
-report();

package/dist/resources/extensions/gsd/tests/notifications.test.ts

@@ -1,67 +0,0 @@
-import test from "node:test";
-import assert from "node:assert/strict";
-
-import {
-  buildDesktopNotificationCommand,
-  shouldSendDesktopNotification,
-} from "../notifications.js";
-import type { NotificationPreferences } from "../types.js";
-
-test("shouldSendDesktopNotification honors granular preferences", () => {
-  const prefs: NotificationPreferences = {
-    enabled: true,
-    on_complete: false,
-    on_error: true,
-    on_budget: false,
-    on_milestone: true,
-    on_attention: false,
-  };
-
-  assert.equal(shouldSendDesktopNotification("complete", prefs), false);
-  assert.equal(shouldSendDesktopNotification("error", prefs), true);
-  assert.equal(shouldSendDesktopNotification("budget", prefs), false);
-  assert.equal(shouldSendDesktopNotification("milestone", prefs), true);
-  assert.equal(shouldSendDesktopNotification("attention", prefs), false);
-});
-
-test("shouldSendDesktopNotification disables all categories when notifications are disabled", () => {
-  const prefs: NotificationPreferences = { enabled: false, on_error: true, on_milestone: true };
-
-  assert.equal(shouldSendDesktopNotification("error", prefs), false);
-  assert.equal(shouldSendDesktopNotification("milestone", prefs), false);
-});
-
-test("buildDesktopNotificationCommand uses argument arrays for macOS notifications", () => {
-  const command = buildDesktopNotificationCommand(
-    "darwin",
-    `Bob's "Milestone"`,
-    `Budget!\nPath: C:\\temp`,
-    "error",
-  );
-
-  assert.ok(command);
-  assert.equal(command.file, "osascript");
-  assert.deepEqual(command.args.slice(0, 1), ["-e"]);
-  assert.match(command.args[1], /Bob's \\"Milestone\\"/);
-  assert.match(command.args[1], /Budget! Path: C:\\\\temp/);
-  assert.doesNotMatch(command.args[1], /\n/);
-});
-
-test("buildDesktopNotificationCommand preserves literal shell characters on linux", () => {
-  const command = buildDesktopNotificationCommand(
-    "linux",
-    `Bob's $PATH !`,
-    "line 1\nline 2",
-    "warning",
-  );
-
-  assert.ok(command);
-  assert.deepEqual(command, {
-    file: "notify-send",
-    args: ["-u", "normal", `Bob's $PATH !`, "line 1 line 2"],
-  });
-});
-
-test("buildDesktopNotificationCommand skips unsupported platforms", () => {
-  assert.equal(buildDesktopNotificationCommand("win32", "Title", "Message"), null);
-});

package/dist/resources/extensions/gsd/tests/overrides.test.ts

@@ -1,131 +0,0 @@
-// GSD Extension - Override Tests
-// Tests for parseOverrides, appendOverride, loadActiveOverrides, formatOverridesSection, resolveAllOverrides
-
-import { mkdtempSync, mkdirSync, readFileSync, writeFileSync, rmSync } from "node:fs";
-import { join } from "node:path";
-import { tmpdir } from "node:os";
-import { createTestContext } from './test-helpers.ts';
-import { parseOverrides, appendOverride, loadActiveOverrides, formatOverridesSection, resolveAllOverrides } from '../files.ts';
-import type { Override } from '../files.ts';
-
-const { assertEq, assertTrue, assertMatch, assertNoMatch, report } = createTestContext();
-
-const tempDirs: string[] = [];
-
-function makeTempDir(prefix: string): string {
-  const dir = mkdtempSync(join(tmpdir(), `gsd-overrides-test-${prefix}-`));
-  mkdirSync(join(dir, ".gsd"), { recursive: true });
-  tempDirs.push(dir);
-  return dir;
-}
-
-function cleanup(): void {
-  for (const dir of tempDirs) {
-    try { rmSync(dir, { recursive: true, force: true }); } catch { /* ignore */ }
-  }
-  tempDirs.length = 0;
-}
-
-console.log('\n=== parseOverrides: empty content ===');
-{ const result = parseOverrides(""); assertEq(result.length, 0, "empty content returns no overrides"); }
-
-console.log('\n=== parseOverrides: single active override ===');
-{
-  const content = `# GSD Overrides\n\nUser-issued overrides that supersede plan document content.\n\n---\n\n## Override: 2026-03-14T10:00:00.000Z\n\n**Change:** Use Postgres instead of SQLite\n**Scope:** active\n**Applied-at:** M001/S02/T03\n\n---\n`;
-  const result = parseOverrides(content);
-  assertEq(result.length, 1, "parses one override");
-  assertEq(result[0].timestamp, "2026-03-14T10:00:00.000Z", "correct timestamp");
-  assertEq(result[0].change, "Use Postgres instead of SQLite", "correct change");
-  assertEq(result[0].scope, "active", "correct scope");
-  assertEq(result[0].appliedAt, "M001/S02/T03", "correct appliedAt");
-}
-
-console.log('\n=== parseOverrides: multiple overrides, mixed scopes ===');
-{
-  const content = `# GSD Overrides\n\n---\n\n## Override: 2026-03-14T10:00:00.000Z\n\n**Change:** Use Postgres instead of SQLite\n**Scope:** resolved\n**Applied-at:** M001/S02/T03\n\n---\n\n## Override: 2026-03-14T11:00:00.000Z\n\n**Change:** Use JWT instead of session cookies\n**Scope:** active\n**Applied-at:** M001/S03/T01\n\n---\n`;
-  const result = parseOverrides(content);
-  assertEq(result.length, 2, "parses two overrides");
-  assertEq(result[0].scope, "resolved", "first is resolved");
-  assertEq(result[1].scope, "active", "second is active");
-  assertEq(result[1].change, "Use JWT instead of session cookies", "second change text");
-}
-
-console.log('\n=== appendOverride: creates new file ===');
-{
-  const tmp = makeTempDir("append-new");
-  await appendOverride(tmp, "Use Postgres", "M001/S01/T01");
-  const content = readFileSync(join(tmp, ".gsd", "OVERRIDES.md"), "utf-8");
-  assertTrue(content.includes("# GSD Overrides"), "has header");
-  assertTrue(content.includes("**Change:** Use Postgres"), "has change");
-  assertTrue(content.includes("**Scope:** active"), "has active scope");
-  assertTrue(content.includes("**Applied-at:** M001/S01/T01"), "has appliedAt");
-}
-
-console.log('\n=== appendOverride: appends to existing file ===');
-{
-  const tmp = makeTempDir("append-existing");
-  await appendOverride(tmp, "First override", "M001/S01/T01");
-  await appendOverride(tmp, "Second override", "M001/S02/T02");
-  const content = readFileSync(join(tmp, ".gsd", "OVERRIDES.md"), "utf-8");
-  assertTrue(content.includes("**Change:** First override"), "has first override");
-  assertTrue(content.includes("**Change:** Second override"), "has second override");
-  const parsed = parseOverrides(content);
-  assertEq(parsed.length, 2, "two overrides in file");
-}
-
-console.log('\n=== loadActiveOverrides: no file ===');
-{
-  const tmp = makeTempDir("load-no-file");
-  const result = await loadActiveOverrides(tmp);
-  assertEq(result.length, 0, "returns empty when no file");
-}
-
-console.log('\n=== loadActiveOverrides: filters to active only ===');
-{
-  const tmp = makeTempDir("load-filter");
-  const content = `# GSD Overrides\n\n---\n\n## Override: 2026-03-14T10:00:00.000Z\n\n**Change:** Resolved change\n**Scope:** resolved\n**Applied-at:** M001/S01/T01\n\n---\n\n## Override: 2026-03-14T11:00:00.000Z\n\n**Change:** Active change\n**Scope:** active\n**Applied-at:** M001/S02/T01\n\n---\n`;
-  writeFileSync(join(tmp, ".gsd", "OVERRIDES.md"), content, "utf-8");
-  const result = await loadActiveOverrides(tmp);
-  assertEq(result.length, 1, "only one active override");
-  assertEq(result[0].change, "Active change", "correct active change");
-}
-
-console.log('\n=== formatOverridesSection: empty array ===');
-{ const result = formatOverridesSection([]); assertEq(result, "", "empty overrides returns empty string"); }
-
-console.log('\n=== formatOverridesSection: formats section ===');
-{
-  const overrides: Override[] = [
-    { timestamp: "2026-03-14T10:00:00.000Z", change: "Use Postgres", scope: "active", appliedAt: "M001/S01/T01" },
-  ];
-  const result = formatOverridesSection(overrides);
-  assertTrue(result.includes("## Active Overrides (supersede plan content)"), "has header");
-  assertTrue(result.includes("**Use Postgres**"), "has change text");
-  assertTrue(result.includes("supersede any conflicting content"), "has instruction");
-}
-
-console.log('\n=== resolveAllOverrides: marks all as resolved ===');
-{
-  const tmp = makeTempDir("resolve-all");
-  await appendOverride(tmp, "First", "M001/S01/T01");
-  await appendOverride(tmp, "Second", "M001/S02/T01");
-  let active = await loadActiveOverrides(tmp);
-  assertEq(active.length, 2, "two active before resolve");
-  await resolveAllOverrides(tmp);
-  active = await loadActiveOverrides(tmp);
-  assertEq(active.length, 0, "no active after resolve");
-  const content = readFileSync(join(tmp, ".gsd", "OVERRIDES.md"), "utf-8");
-  const allOverrides = parseOverrides(content);
-  assertEq(allOverrides.length, 2, "still two overrides total");
-  assertTrue(allOverrides.every(o => o.scope === "resolved"), "all resolved");
-}
-
-console.log('\n=== resolveAllOverrides: no file — no error ===');
-{
-  const tmp = makeTempDir("resolve-no-file");
-  await resolveAllOverrides(tmp);
-  assertTrue(true, "resolveAllOverrides with no file does not throw");
-}
-
-cleanup();
-report();