lsd-pi 1.0.0 → 1.1.0

package/src/resources/extensions/gsd/tests/agent-end-retry.test.ts

@@ -1,143 +0,0 @@
-/**
- * agent-end-retry.test.ts — Regression checks for the agent_end model.
- *
- * The per-unit one-shot resolve function lives at module level in auto-loop.ts
- * (_currentResolve). handleAgentEnd is a thin compatibility wrapper around
- * resolveAgentEnd().
- */
-
-import 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 AUTO_TS_PATH = join(__dirname, "..", "auto.ts");
-const AUTO_RESOLVE_TS_PATH = join(__dirname, "..", "auto", "resolve.ts");
-const SESSION_TS_PATH = join(__dirname, "..", "auto", "session.ts");
-
-function getAutoTsSource(): string {
-  return readFileSync(AUTO_TS_PATH, "utf-8");
-}
-
-function getAutoResolveTsSource(): string {
-  return readFileSync(AUTO_RESOLVE_TS_PATH, "utf-8");
-}
-
-function getSessionTsSource(): string {
-  return readFileSync(SESSION_TS_PATH, "utf-8");
-}
-
-test("auto/resolve.ts declares _currentResolve for per-unit one-shot promises", () => {
-  const source = getAutoResolveTsSource();
-  assert.ok(
-    source.includes("_currentResolve"),
-    "auto/resolve.ts must declare _currentResolve for the per-unit resolve function",
-  );
-  assert.ok(
-    source.includes("_sessionSwitchInFlight"),
-    "auto/resolve.ts must declare _sessionSwitchInFlight guard",
-  );
-});
-
-test("AutoSession no longer holds promise state (moved to auto-loop.ts module scope)", () => {
-  const source = getSessionTsSource();
-  // Properties should NOT exist as class fields
-  assert.ok(
-    !source.includes("pendingResolve:"),
-    "AutoSession must not declare pendingResolve (moved to auto-loop.ts)",
-  );
-  assert.ok(
-    !source.includes("pendingAgentEndQueue:"),
-    "AutoSession must not declare pendingAgentEndQueue (removed — events are dropped)",
-  );
-});
-
-test("legacy pendingAgentEndRetry state is gone", () => {
-  const source = getSessionTsSource();
-  assert.ok(
-    !source.includes("pendingAgentEndRetry"),
-    "AutoSession should no longer use legacy pendingAgentEndRetry state",
-  );
-});
-
-test("handleAgentEnd is a thin compatibility wrapper", () => {
-  const source = getAutoTsSource();
-  const fnIdx = source.indexOf("export async function handleAgentEnd");
-  assert.ok(fnIdx > -1, "handleAgentEnd must exist in auto.ts");
-  const fnBlock = source.slice(fnIdx, source.indexOf("\n// ─── ", fnIdx + 100));
-
-  assert.ok(
-    fnBlock.includes("resolveAgentEnd("),
-    "handleAgentEnd must delegate to resolveAgentEnd",
-  );
-  assert.ok(
-    !fnBlock.includes("pendingAgentEndRetry"),
-    "handleAgentEnd must not use legacy retry state",
-  );
-  assert.ok(
-    !fnBlock.includes("dispatchNextUnit"),
-    "handleAgentEnd must not dispatch recursively",
-  );
-});
-
-test("handleAgentEnd early return calls resolveAgentEndCancelled", () => {
-  const source = getAutoTsSource();
-  const fnIdx = source.indexOf("export async function handleAgentEnd");
-  assert.ok(fnIdx > -1, "handleAgentEnd must exist in auto.ts");
-  const fnBlock = source.slice(fnIdx, source.indexOf("\n// ─── ", fnIdx + 100));
-
-  assert.ok(
-    fnBlock.includes("resolveAgentEndCancelled()"),
-    "handleAgentEnd must call resolveAgentEndCancelled on early return to prevent orphaned promises",
-  );
-});
-
-test("pauseAuto calls resolveAgentEndCancelled to unblock the loop", () => {
-  const source = getAutoTsSource();
-  const fnIdx = source.indexOf("export async function pauseAuto");
-  assert.ok(fnIdx > -1, "pauseAuto must exist in auto.ts");
-  // Extract the function body (up to the next export or top-level function)
-  const fnBlock = source.slice(fnIdx, source.indexOf("\n/**\n * Build", fnIdx + 100));
-
-  assert.ok(
-    fnBlock.includes("resolveAgentEndCancelled("),
-    "pauseAuto must call resolveAgentEndCancelled to unblock the auto-loop promise",
-  );
-});
-
-test("auto-timers.ts idle watchdog catch calls resolveAgentEndCancelled", () => {
-  const TIMERS_PATH = join(__dirname, "..", "auto-timers.ts");
-  const source = readFileSync(TIMERS_PATH, "utf-8");
-
-  const idleCatchIdx = source.indexOf("[idle-watchdog] Unhandled error");
-  assert.ok(idleCatchIdx > -1, "idle watchdog catch block must exist");
-  // Check that resolveAgentEndCancelled is called near this catch
-  const catchRegion = source.slice(Math.max(0, idleCatchIdx - 200), idleCatchIdx + 200);
-  assert.ok(
-    catchRegion.includes("resolveAgentEndCancelled("),
-    "idle watchdog catch block must call resolveAgentEndCancelled",
-  );
-});
-
-test("auto-timers.ts hard timeout catch calls resolveAgentEndCancelled", () => {
-  const TIMERS_PATH = join(__dirname, "..", "auto-timers.ts");
-  const source = readFileSync(TIMERS_PATH, "utf-8");
-
-  const hardCatchIdx = source.indexOf("[hard-timeout] Unhandled error");
-  assert.ok(hardCatchIdx > -1, "hard timeout catch block must exist");
-  const catchRegion = source.slice(Math.max(0, hardCatchIdx - 200), hardCatchIdx + 200);
-  assert.ok(
-    catchRegion.includes("resolveAgentEndCancelled("),
-    "hard timeout catch block must call resolveAgentEndCancelled",
-  );
-});
-
-test("resolveAgentEndCancelled is exported from auto/resolve.ts", () => {
-  const source = getAutoResolveTsSource();
-  assert.ok(
-    source.includes("export function resolveAgentEndCancelled"),
-    "auto/resolve.ts must export resolveAgentEndCancelled",
-  );
-});

package/src/resources/extensions/gsd/tests/atomic-write.test.ts

@@ -1,144 +0,0 @@
-import test from "node:test";
-import assert from "node:assert/strict";
-
-import {
-  atomicWriteAsyncWithOps,
-  atomicWriteSyncWithOps,
-  type AtomicWriteAsyncOps,
-  type AtomicWriteSyncOps,
-} from "../atomic-write.ts";
-
-function makeError(code: string, message = code): NodeJS.ErrnoException {
-  const err = new Error(message) as NodeJS.ErrnoException;
-  err.code = code;
-  return err;
-}
-
-function createAsyncHarness(plan: Array<Error | null>) {
-  const files = new Map<string, string>();
-  const renameCalls: Array<{ from: string; to: string }> = [];
-  const unlinkCalls: string[] = [];
-  const sleepCalls: number[] = [];
-  let tempCounter = 0;
-
-  const ops: AtomicWriteAsyncOps = {
-    mkdir: async () => {},
-    writeFile: async (path, content) => {
-      files.set(path, String(content));
-    },
-    rename: async (from, to) => {
-      renameCalls.push({ from, to });
-      const outcome = plan.shift() ?? null;
-      if (outcome) throw outcome;
-      const content = files.get(from);
-      if (content === undefined) throw makeError("ENOENT", "temp missing");
-      files.set(to, content);
-      files.delete(from);
-    },
-    unlink: async (path) => {
-      unlinkCalls.push(path);
-      files.delete(path);
-    },
-    sleep: async (ms) => {
-      sleepCalls.push(ms);
-    },
-    createTempPath: (filePath) => `${filePath}.tmp.test-${++tempCounter}`,
-  };
-
-  return { ops, files, renameCalls, unlinkCalls, sleepCalls };
-}
-
-function createSyncHarness(plan: Array<Error | null>) {
-  const files = new Map<string, string>();
-  const renameCalls: Array<{ from: string; to: string }> = [];
-  const unlinkCalls: string[] = [];
-  const sleepCalls: number[] = [];
-  let tempCounter = 0;
-
-  const ops: AtomicWriteSyncOps = {
-    mkdir: () => {},
-    writeFile: (path, content) => {
-      files.set(path, String(content));
-    },
-    rename: (from, to) => {
-      renameCalls.push({ from, to });
-      const outcome = plan.shift() ?? null;
-      if (outcome) throw outcome;
-      const content = files.get(from);
-      if (content === undefined) throw makeError("ENOENT", "temp missing");
-      files.set(to, content);
-      files.delete(from);
-    },
-    unlink: (path) => {
-      unlinkCalls.push(path);
-      files.delete(path);
-    },
-    sleep: (ms) => {
-      sleepCalls.push(ms);
-    },
-    createTempPath: (filePath) => `${filePath}.tmp.test-${++tempCounter}`,
-  };
-
-  return { ops, files, renameCalls, unlinkCalls, sleepCalls };
-}
-
-test("atomicWriteAsync retries transient rename failures and preserves atomicity", async () => {
-  const harness = createAsyncHarness([makeError("EBUSY"), makeError("EPERM"), null]);
-  harness.files.set("C:/tmp/output.txt", "old-content");
-
-  await atomicWriteAsyncWithOps("C:/tmp/output.txt", "new-content", "utf-8", harness.ops);
-
-  assert.equal(harness.renameCalls.length, 3);
-  assert.equal(harness.files.get("C:/tmp/output.txt"), "new-content");
-  assert.equal(harness.unlinkCalls.length, 0);
-  assert.equal(harness.sleepCalls.length, 2);
-});
-
-test("atomicWriteAsync cleans up temp file and reports attempts after repeated transient failures", async () => {
-  const harness = createAsyncHarness([
-    makeError("EACCES"),
-    makeError("EBUSY"),
-    makeError("EPERM"),
-    makeError("EACCES"),
-    makeError("EBUSY"),
-  ]);
-  harness.files.set("C:/tmp/output.txt", "old-content");
-
-  await assert.rejects(
-    atomicWriteAsyncWithOps("C:/tmp/output.txt", "new-content", "utf-8", harness.ops),
-    (error: unknown) => {
-      assert.match(String(error), /C:\\\/tmp\/output\.txt|C:\/tmp\/output\.txt/);
-      assert.match(String(error), /attempt/i);
-      assert.match(String(error), /EBUSY|EPERM|EACCES/);
-      return true;
-    },
-  );
-
-  assert.equal(harness.renameCalls.length, 5);
-  assert.equal(harness.files.get("C:/tmp/output.txt"), "old-content");
-  assert.equal(harness.unlinkCalls.length, 1);
-});
-
-test("atomicWriteAsync does not retry non-transient rename failures", async () => {
-  const harness = createAsyncHarness([makeError("ENOENT")]);
-  harness.files.set("C:/tmp/output.txt", "old-content");
-
-  await assert.rejects(() => atomicWriteAsyncWithOps("C:/tmp/output.txt", "new-content", "utf-8", harness.ops));
-
-  assert.equal(harness.renameCalls.length, 1);
-  assert.equal(harness.sleepCalls.length, 0);
-  assert.equal(harness.unlinkCalls.length, 1);
-  assert.equal(harness.files.get("C:/tmp/output.txt"), "old-content");
-});
-
-test("atomicWriteSync retries transient rename failures and succeeds", () => {
-  const harness = createSyncHarness([makeError("EACCES"), makeError("EBUSY"), null]);
-  harness.files.set("C:/tmp/output.txt", "old-content");
-
-  atomicWriteSyncWithOps("C:/tmp/output.txt", "new-content", "utf-8", harness.ops);
-
-  assert.equal(harness.renameCalls.length, 3);
-  assert.equal(harness.sleepCalls.length, 2);
-  assert.equal(harness.unlinkCalls.length, 0);
-  assert.equal(harness.files.get("C:/tmp/output.txt"), "new-content");
-});

package/src/resources/extensions/gsd/tests/auto-budget-alerts.test.ts

@@ -1,50 +0,0 @@
-import test from "node:test";
-import assert from "node:assert/strict";
-
-import {
-  getBudgetAlertLevel,
-  getBudgetEnforcementAction,
-  getNewBudgetAlertLevel,
-} from "../auto.js";
-
-test("getBudgetAlertLevel returns the expected threshold bucket", () => {
-  assert.equal(getBudgetAlertLevel(0.10), 0);
-  assert.equal(getBudgetAlertLevel(0.74), 0);
-  assert.equal(getBudgetAlertLevel(0.75), 75);
-  assert.equal(getBudgetAlertLevel(0.79), 75);
-  assert.equal(getBudgetAlertLevel(0.80), 80);
-  assert.equal(getBudgetAlertLevel(0.85), 80);
-  assert.equal(getBudgetAlertLevel(0.89), 80);
-  assert.equal(getBudgetAlertLevel(0.90), 90);
-  assert.equal(getBudgetAlertLevel(1.00), 100);
-});
-
-test("getNewBudgetAlertLevel only emits once per threshold", () => {
-  assert.equal(getNewBudgetAlertLevel(0, 0.74), null);
-  assert.equal(getNewBudgetAlertLevel(0, 0.75), 75);
-  assert.equal(getNewBudgetAlertLevel(75, 0.79), null);
-  assert.equal(getNewBudgetAlertLevel(75, 0.80), 80);
-  assert.equal(getNewBudgetAlertLevel(80, 0.85), null);
-  assert.equal(getNewBudgetAlertLevel(80, 0.90), 90);
-  assert.equal(getNewBudgetAlertLevel(90, 0.95), null);
-  assert.equal(getNewBudgetAlertLevel(90, 1.0), 100);
-  assert.equal(getNewBudgetAlertLevel(100, 1.2), null);
-});
-
-test("80% alert fires exactly once between 75% and 90%", () => {
-  // Transition from 75 → 80 emits 80
-  assert.equal(getNewBudgetAlertLevel(75, 0.80), 80);
-  // Already at 80 — no re-emission
-  assert.equal(getNewBudgetAlertLevel(80, 0.82), null);
-  assert.equal(getNewBudgetAlertLevel(80, 0.89), null);
-  // Transition from 80 → 90 emits 90
-  assert.equal(getNewBudgetAlertLevel(80, 0.90), 90);
-});
-
-test("getBudgetEnforcementAction maps the configured ceiling behavior", () => {
-  assert.equal(getBudgetEnforcementAction("warn", 0.80), "none");
-  assert.equal(getBudgetEnforcementAction("warn", 0.99), "none");
-  assert.equal(getBudgetEnforcementAction("warn", 1.0), "warn");
-  assert.equal(getBudgetEnforcementAction("pause", 1.0), "pause");
-  assert.equal(getBudgetEnforcementAction("halt", 1.0), "halt");
-});

package/src/resources/extensions/gsd/tests/auto-dashboard.test.ts

@@ -1,211 +0,0 @@
-import test from "node:test";
-import assert from "node:assert/strict";
-import { readFileSync } from "node:fs";
-import { join } from "node:path";
-
-import {
-  unitVerb,
-  unitPhaseLabel,
-  describeNextUnit,
-  formatAutoElapsed,
-  formatWidgetTokens,
-  estimateTimeRemaining,
-  extractUatSliceId,
-} from "../auto-dashboard.ts";
-
-const autoSource = readFileSync(join(process.cwd(), "src", "resources", "extensions", "gsd", "auto.ts"), "utf-8");
-const dashboardSource = readFileSync(join(process.cwd(), "src", "resources", "extensions", "gsd", "auto-dashboard.ts"), "utf-8");
-
-// ─── unitVerb ─────────────────────────────────────────────────────────────
-
-test("unitVerb maps known unit types to verbs", () => {
-  assert.equal(unitVerb("research-milestone"), "researching");
-  assert.equal(unitVerb("research-slice"), "researching");
-  assert.equal(unitVerb("plan-milestone"), "planning");
-  assert.equal(unitVerb("plan-slice"), "planning");
-  assert.equal(unitVerb("execute-task"), "executing");
-  assert.equal(unitVerb("complete-slice"), "completing");
-  assert.equal(unitVerb("replan-slice"), "replanning");
-  assert.equal(unitVerb("reassess-roadmap"), "reassessing");
-  assert.equal(unitVerb("run-uat"), "running UAT");
-});
-
-test("unitVerb returns raw type for unknown types", () => {
-  assert.equal(unitVerb("custom-thing"), "custom-thing");
-});
-
-test("unitVerb handles hook types", () => {
-  assert.equal(unitVerb("hook/verify-code"), "hook: verify-code");
-  assert.equal(unitVerb("hook/"), "hook: ");
-});
-
-// ─── unitPhaseLabel ───────────────────────────────────────────────────────
-
-test("unitPhaseLabel maps known types to labels", () => {
-  assert.equal(unitPhaseLabel("research-milestone"), "RESEARCH");
-  assert.equal(unitPhaseLabel("research-slice"), "RESEARCH");
-  assert.equal(unitPhaseLabel("plan-milestone"), "PLAN");
-  assert.equal(unitPhaseLabel("plan-slice"), "PLAN");
-  assert.equal(unitPhaseLabel("execute-task"), "EXECUTE");
-  assert.equal(unitPhaseLabel("complete-slice"), "COMPLETE");
-  assert.equal(unitPhaseLabel("replan-slice"), "REPLAN");
-  assert.equal(unitPhaseLabel("reassess-roadmap"), "REASSESS");
-  assert.equal(unitPhaseLabel("run-uat"), "UAT");
-});
-
-test("unitPhaseLabel uppercases unknown types", () => {
-  assert.equal(unitPhaseLabel("custom-thing"), "CUSTOM-THING");
-});
-
-test("unitPhaseLabel returns HOOK for hook types", () => {
-  assert.equal(unitPhaseLabel("hook/verify"), "HOOK");
-});
-
-// ─── describeNextUnit ─────────────────────────────────────────────────────
-
-test("describeNextUnit handles pre-planning phase", () => {
-  const result = describeNextUnit({
-    phase: "pre-planning",
-    activeMilestone: { id: "M001", title: "Test" },
-  } as any);
-  assert.equal(result.label, "Research & plan milestone");
-});
-
-test("describeNextUnit handles executing phase", () => {
-  const result = describeNextUnit({
-    phase: "executing",
-    activeMilestone: { id: "M001", title: "Test" },
-    activeSlice: { id: "S01", title: "Slice" },
-    activeTask: { id: "T01", title: "Task One" },
-  } as any);
-  assert.ok(result.label.includes("T01"));
-  assert.ok(result.label.includes("Task One"));
-});
-
-test("describeNextUnit handles summarizing phase", () => {
-  const result = describeNextUnit({
-    phase: "summarizing",
-    activeMilestone: { id: "M001", title: "Test" },
-    activeSlice: { id: "S01", title: "First Slice" },
-  } as any);
-  assert.ok(result.label.includes("S01"));
-});
-
-test("describeNextUnit handles needs-discussion phase", () => {
-  const result = describeNextUnit({
-    phase: "needs-discussion",
-    activeMilestone: { id: "M001", title: "Test" },
-  } as any);
-  assert.ok(
-    result.label.toLowerCase().includes("discuss") || result.label.toLowerCase().includes("draft"),
-  );
-});
-
-test("describeNextUnit handles completing-milestone phase", () => {
-  const result = describeNextUnit({
-    phase: "completing-milestone",
-    activeMilestone: { id: "M001", title: "Test" },
-  } as any);
-  assert.ok(result.label.toLowerCase().includes("milestone"));
-});
-
-test("describeNextUnit returns fallback for unknown phase", () => {
-  const result = describeNextUnit({
-    phase: "some-future-phase" as any,
-    activeMilestone: { id: "M001", title: "Test" },
-  } as any);
-  assert.equal(result.label, "Continue");
-});
-
-// ─── formatAutoElapsed ────────────────────────────────────────────────────
-
-test("formatAutoElapsed returns empty for zero startTime", () => {
-  assert.equal(formatAutoElapsed(0), "");
-});
-
-test("formatAutoElapsed formats seconds", () => {
-  const result = formatAutoElapsed(Date.now() - 30_000);
-  assert.match(result, /^\d+s$/);
-});
-
-test("formatAutoElapsed formats minutes", () => {
-  const result = formatAutoElapsed(Date.now() - 180_000); // 3 min
-  assert.match(result, /^3m/);
-});
-
-test("formatAutoElapsed formats hours", () => {
-  const result = formatAutoElapsed(Date.now() - 3_700_000); // ~1h
-  assert.match(result, /^1h/);
-});
-
-// ─── formatWidgetTokens ──────────────────────────────────────────────────
-
-test("formatWidgetTokens formats small numbers directly", () => {
-  assert.equal(formatWidgetTokens(0), "0");
-  assert.equal(formatWidgetTokens(500), "500");
-  assert.equal(formatWidgetTokens(999), "999");
-});
-
-test("formatWidgetTokens formats thousands with k", () => {
-  assert.equal(formatWidgetTokens(1000), "1.0k");
-  assert.equal(formatWidgetTokens(5500), "5.5k");
-  assert.equal(formatWidgetTokens(10000), "10k");
-  assert.equal(formatWidgetTokens(99999), "100k");
-});
-
-test("formatWidgetTokens formats millions with M", () => {
-  assert.equal(formatWidgetTokens(1_000_000), "1.0M");
-  assert.equal(formatWidgetTokens(10_000_000), "10M");
-  assert.equal(formatWidgetTokens(25_000_000), "25M");
-});
-
-// ─── estimateTimeRemaining ──────────────────────────────────────────────
-
-test("estimateTimeRemaining returns null when no ledger data", () => {
-  // With no active auto-mode session, ledger is empty
-  const result = estimateTimeRemaining();
-  assert.equal(result, null);
-});
-
-test("estimateTimeRemaining is exported and callable", () => {
-  assert.equal(typeof estimateTimeRemaining, "function");
-});
-
-// ─── getAutoDashboardData elapsed guard ──────────────────────────────────────
-// These tests verify the elapsed time calculation in getAutoDashboardData()
-// doesn't produce absurd values when autoStartTime is 0 (uninitialized).
-// The actual function is in auto.ts and tested structurally here by verifying
-// that formatAutoElapsed properly handles the zero case.
-
-test("formatAutoElapsed returns empty string for negative autoStartTime", () => {
-  // A negative value should be treated as invalid — the guard in
-  // getAutoDashboardData prevents this, but formatAutoElapsed should also
-  // handle it gracefully via its falsy check.
-  assert.equal(formatAutoElapsed(-1), "");
-  assert.equal(formatAutoElapsed(NaN), "");
-});
-
-test("getAutoDashboardData returns RTK savings in the dashboard payload", () => {
-  assert.match(autoSource, /const rtkSavings = sessionId && s\.basePath/);
-  assert.match(autoSource, /rtkSavings,/);
-});
-
-test("auto progress widget renders RTK savings under the footer stats line", () => {
-  assert.match(dashboardSource, /formatRtkSavingsLabel/);
-  assert.match(dashboardSource, /getRtkSessionSavings\(accessors\.getBasePath\(\), sessionId\)/);
-  assert.match(dashboardSource, /lines\.push\(rightAlign\("", theme\.fg\("dim", cachedRtkLabel\), width\)\);/);
-});
-
-// ─── extractUatSliceId ───────────────────────────────────────────────────
-
-test("extractUatSliceId extracts slice ID from M001/S01 format", () => {
-  assert.equal(extractUatSliceId("M001/S01"), "S01");
-  assert.equal(extractUatSliceId("M002/S03"), "S03");
-  assert.equal(extractUatSliceId("M001/S12"), "S12");
-});
-
-test("extractUatSliceId returns null for invalid formats", () => {
-  assert.equal(extractUatSliceId("M001"), null);
-  assert.equal(extractUatSliceId(""), null);
-  assert.equal(extractUatSliceId("M001/T01"), null);
-});