gsd-pi 2.72.0-dev.3118184 → 2.72.0-dev.4f3264a

package/src/resources/extensions/gsd/tests/memory-pressure-stuck-state.test.ts

@@ -1,54 +0,0 @@
-/**
- * Regression tests for memory pressure monitoring (#3331) and
- * stuck detection persistence (#3704) in auto/loop.ts.
- */
-
-import { describe, test } 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 loopSource = readFileSync(join(__dirname, "..", "auto", "loop.ts"), "utf-8");
-
-describe("memory pressure monitoring (#3331)", () => {
-  test("checkMemoryPressure function exists", () => {
-    assert.match(loopSource, /function checkMemoryPressure/);
-  });
-
-  test("MEMORY_PRESSURE_THRESHOLD constant is defined", () => {
-    assert.match(loopSource, /MEMORY_PRESSURE_THRESHOLD\s*=\s*0\.\d+/);
-  });
-
-  test("memory check runs every MEMORY_CHECK_INTERVAL iterations", () => {
-    assert.match(loopSource, /iteration\s*%\s*MEMORY_CHECK_INTERVAL\s*===\s*0/);
-  });
-
-  test("memory pressure triggers graceful stopAuto", () => {
-    assert.match(loopSource, /mem\.pressured/);
-    assert.match(loopSource, /Stopping gracefully to prevent OOM/);
-  });
-});
-
-describe("stuck detection persistence (#3704)", () => {
-  test("loadStuckState function exists", () => {
-    assert.match(loopSource, /function loadStuckState/);
-  });
-
-  test("saveStuckState function exists", () => {
-    assert.match(loopSource, /function saveStuckState/);
-  });
-
-  test("loopState initialized from persisted state", () => {
-    assert.match(loopSource, /loadStuckState\(s\.basePath\)/);
-  });
-
-  test("stuck state saved after each iteration", () => {
-    assert.match(loopSource, /saveStuckState\(s\.basePath,\s*loopState\)/);
-  });
-
-  test("stuck state file path uses runtime directory", () => {
-    assert.match(loopSource, /stuck-state\.json/);
-  });
-});

package/src/resources/extensions/gsd/tests/post-unit-state-rebuild.test.ts

@@ -1,34 +0,0 @@
-/**
- * Regression test for #3869: normal post-unit flow should rebuild STATE.md
- * before syncing worktree state back to the project root.
- */
-
-import test from "node:test";
-import assert from "node:assert/strict";
-import { readFileSync } from "node:fs";
-import { join } from "node:path";
-
-const source = readFileSync(join(import.meta.dirname, "..", "auto-post-unit.ts"), "utf-8");
-
-test("auto-post-unit imports rebuildState", () => {
-  assert.ok(
-    source.includes('import { rebuildState } from "./doctor.js";'),
-    "auto-post-unit.ts should import rebuildState from doctor.ts",
-  );
-});
-
-test("postUnitPreVerification rebuilds STATE.md before worktree sync", () => {
-  const fnStart = source.indexOf("export async function postUnitPreVerification");
-  assert.ok(fnStart > 0, "postUnitPreVerification should exist");
-
-  const section = source.slice(fnStart, fnStart + 8000);
-  const rebuildIdx = section.indexOf('await runSafely("postUnit", "state-rebuild"');
-  const syncIdx = section.indexOf('await runSafely("postUnit", "worktree-sync"');
-
-  assert.ok(rebuildIdx > 0, "postUnitPreVerification should rebuild STATE.md after unit completion");
-  assert.ok(syncIdx > 0, "postUnitPreVerification should sync worktree state back to the project root");
-  assert.ok(
-    rebuildIdx < syncIdx,
-    "STATE.md rebuild should happen before worktree sync so synced state is fresh",
-  );
-});

package/src/resources/extensions/gsd/tests/preferences-formatting.test.ts

@@ -1,87 +0,0 @@
-/**
- * Tests for formatSkillRef — pure formatting logic for skill references
- * in the system prompt. Moved from preferences-skills.ts to preferences-types.ts
- * to break the preferences ↔ preferences-skills circular dependency.
- */
-
-import { describe, test } from "node:test";
-import assert from "node:assert/strict";
-
-import { formatSkillRef } from "../preferences-types.ts";
-import type { SkillResolution } from "../preferences-types.ts";
-
-function makeResolutions(entries: [string, Partial<SkillResolution>][]): Map<string, SkillResolution> {
-  const map = new Map<string, SkillResolution>();
-  for (const [key, partial] of entries) {
-    map.set(key, {
-      original: partial.original ?? key,
-      resolvedPath: partial.resolvedPath ?? null,
-      method: partial.method ?? "unresolved",
-    });
-  }
-  return map;
-}
-
-describe("formatSkillRef", () => {
-  test("marks unresolved references with a warning", () => {
-    const resolutions = makeResolutions([
-      ["my-skill", { method: "unresolved" }],
-    ]);
-    const result = formatSkillRef("my-skill", resolutions);
-    assert.match(result, /my-skill/);
-    assert.match(result, /not found/);
-  });
-
-  test("marks unknown references (not in map) with a warning", () => {
-    const resolutions = new Map<string, SkillResolution>();
-    const result = formatSkillRef("unknown-skill", resolutions);
-    assert.match(result, /unknown-skill/);
-    assert.match(result, /not found/);
-  });
-
-  test("returns bare ref for absolute-path resolution", () => {
-    const resolutions = makeResolutions([
-      ["/home/user/skills/SKILL.md", {
-        method: "absolute-path",
-        resolvedPath: "/home/user/skills/SKILL.md",
-      }],
-    ]);
-    const result = formatSkillRef("/home/user/skills/SKILL.md", resolutions);
-    assert.equal(result, "/home/user/skills/SKILL.md");
-  });
-
-  test("returns bare ref for absolute-dir resolution", () => {
-    const resolutions = makeResolutions([
-      ["/home/user/skills/my-skill", {
-        method: "absolute-dir",
-        resolvedPath: "/home/user/skills/my-skill/SKILL.md",
-      }],
-    ]);
-    const result = formatSkillRef("/home/user/skills/my-skill", resolutions);
-    assert.equal(result, "/home/user/skills/my-skill");
-  });
-
-  test("shows resolved path for user-skill resolution", () => {
-    const resolutions = makeResolutions([
-      ["code-review", {
-        method: "user-skill",
-        resolvedPath: "/home/user/.claude/skills/code-review/SKILL.md",
-      }],
-    ]);
-    const result = formatSkillRef("code-review", resolutions);
-    assert.match(result, /code-review/);
-    assert.match(result, /\.claude\/skills\/code-review\/SKILL\.md/);
-  });
-
-  test("shows resolved path for project-skill resolution", () => {
-    const resolutions = makeResolutions([
-      ["lint-fix", {
-        method: "project-skill",
-        resolvedPath: "/repo/.gsd/skills/lint-fix/SKILL.md",
-      }],
-    ]);
-    const result = formatSkillRef("lint-fix", resolutions);
-    assert.match(result, /lint-fix/);
-    assert.match(result, /\.gsd\/skills\/lint-fix\/SKILL\.md/);
-  });
-});

package/src/resources/extensions/gsd/tests/prompt-loader-working-directory.test.ts

@@ -1,19 +0,0 @@
-import test from "node:test";
-import assert from "node:assert/strict";
-
-import { loadPrompt } from "../prompt-loader.ts";
-
-test("loadPrompt normalizes workingDirectory backslashes for bash-friendly prompts (#4048)", () => {
-  const prompt = loadPrompt("research-milestone", {
-    milestoneId: "M001",
-    milestoneTitle: "Windows path fix",
-    workingDirectory: "C:\\Dev\\NB\\TR",
-    inlinedContext: "context",
-    skillActivation: "skill activation",
-    skillDiscoveryMode: "off",
-    skillDiscoveryInstructions: " disabled",
-  });
-
-  assert.match(prompt, /Your working directory is `C:\/Dev\/NB\/TR`/);
-  assert.doesNotMatch(prompt, /C:\\Dev\\NB\\TR/);
-});

package/src/resources/extensions/gsd/tests/register-hooks-depth-verification.test.ts

@@ -1,97 +0,0 @@
-import test from "node:test";
-import assert from "node:assert/strict";
-import { mkdirSync, rmSync } from "node:fs";
-import { join } from "node:path";
-import { tmpdir } from "node:os";
-
-import { registerHooks } from "../bootstrap/register-hooks.ts";
-import {
-  getPendingGate,
-  resetWriteGateState,
-  shouldBlockContextArtifactSave,
-} from "../bootstrap/write-gate.ts";
-
-function makeTempDir(prefix: string): string {
-  const dir = join(
-    tmpdir(),
-    `gsd-depth-gate-${prefix}-${Date.now()}-${Math.random().toString(36).slice(2, 8)}`,
-  );
-  mkdirSync(dir, { recursive: true });
-  return dir;
-}
-
-test("register-hooks unlocks milestone depth verification from question id without guided-flow state (#4047)", async (t) => {
-  const dir = makeTempDir("manual");
-  const originalCwd = process.cwd();
-  process.chdir(dir);
-  resetWriteGateState();
-
-  t.after(() => {
-    resetWriteGateState();
-    process.chdir(originalCwd);
-    rmSync(dir, { recursive: true, force: true });
-  });
-
-  const handlers = new Map<string, Array<(event: any, ctx?: any) => Promise<void> | void>>();
-  const pi = {
-    on(event: string, handler: (event: any, ctx?: any) => Promise<void> | void) {
-      const existing = handlers.get(event) ?? [];
-      existing.push(handler);
-      handlers.set(event, existing);
-    },
-  } as any;
-
-  registerHooks(pi);
-
-  const questionId = "depth_verification_M001_confirm";
-  const questions = [
-    {
-      id: questionId,
-      question: "Do you agree?",
-      options: [
-        { label: "Yes, you got it (Recommended)" },
-        { label: "Needs adjustment" },
-      ],
-    },
-  ];
-
-  const toolCallHandlers = handlers.get("tool_call");
-  const toolResultHandlers = handlers.get("tool_result");
-  assert.ok(toolCallHandlers?.length, "tool_call handler should be registered");
-  assert.ok(toolResultHandlers?.length, "tool_result handler should be registered");
-
-  for (const handler of toolCallHandlers ?? []) {
-    await handler({
-      toolName: "ask_user_questions",
-      input: { questions },
-    });
-  }
-
-  assert.equal(getPendingGate(), questionId, "gate should be set even without guided-flow state");
-  assert.equal(
-    shouldBlockContextArtifactSave("CONTEXT", "M001").block,
-    true,
-    "milestone context should still be blocked before confirmation",
-  );
-
-  for (const handler of toolResultHandlers ?? []) {
-    await handler({
-      toolName: "ask_user_questions",
-      input: { questions },
-      details: {
-        response: {
-          answers: {
-            [questionId]: { selected: "Yes, you got it (Recommended)" },
-          },
-        },
-      },
-    });
-  }
-
-  assert.equal(getPendingGate(), null, "confirming the depth question should clear the pending gate");
-  assert.equal(
-    shouldBlockContextArtifactSave("CONTEXT", "M001").block,
-    false,
-    "question-id milestone inference should unlock the matching milestone context write",
-  );
-});

package/src/resources/extensions/gsd/tests/stale-slice-rows.test.ts

@@ -1,41 +0,0 @@
-/**
- * stale-slice-rows.test.ts — #3658
- *
- * Verify that state.ts contains slice-level status reconciliation that
- * updates stale DB rows (status "pending") when disk artifacts (SUMMARY)
- * prove the slice is complete. Without this, the dependency resolver builds
- * doneSliceIds from stale DB rows and downstream slices stay blocked.
- */
-
-import { describe, test } 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 sourceFile = join(__dirname, "..", "state.ts");
-
-describe("stale slice row reconciliation (#3658)", () => {
-  const source = readFileSync(sourceFile, "utf-8");
-
-  test("imports updateSliceStatus from gsd-db", () => {
-    assert.match(source, /import\s*\{[^}]*updateSliceStatus[^}]*\}\s*from/);
-  });
-
-  test("checks isStatusDone before reconciling slice rows", () => {
-    assert.match(source, /isStatusDone\(dbSlice\.status\)/);
-  });
-
-  test("resolves SUMMARY file to detect completed slices on disk", () => {
-    assert.match(source, /resolveSliceFile\(basePath,\s*mid,\s*dbSlice\.id,\s*["']SUMMARY["']\)/);
-  });
-
-  test("calls updateSliceStatus to reconcile stale rows", () => {
-    assert.match(source, /updateSliceStatus\(mid,\s*dbSlice\.id,\s*["']complete["']\)/);
-  });
-
-  test("references issue #3599 in reconciliation comment", () => {
-    assert.match(source, /#3599/);
-  });
-});