whale-code 6.5.10 → 6.6.0

package/dist/cli/services/__tests__/file-history.test.js

@@ -1,136 +0,0 @@
-import { describe, it, expect, vi, beforeEach } from "vitest";
-import { vol } from "memfs";
-
-// ============================================================================
-// Mocks
-// ============================================================================
-
-vi.mock("fs", async () => {
-  const memfs = await import("memfs");
-  return {
-    ...memfs.fs,
-    default: memfs.fs
-  };
-});
-vi.mock("os", () => ({
-  homedir: () => "/home/testuser"
-}));
-import { setFileHistorySessionId, backupFile, listBackups } from "../file-history.js";
-
-// ============================================================================
-// Setup
-// ============================================================================
-
-const HISTORY_BASE = "/home/testuser/.whale/file-history";
-beforeEach(() => {
-  vol.reset();
-  // Reset session id by setting to a known value
-  setFileHistorySessionId("test-session-1");
-});
-
-// ============================================================================
-// setFileHistorySessionId
-// ============================================================================
-
-describe("setFileHistorySessionId", () => {
-  it("sets session ID allowing backups to proceed", () => {
-    vol.fromJSON({
-      "/tmp/file.txt": "content"
-    });
-    setFileHistorySessionId("my-session");
-    backupFile("/tmp/file.txt");
-    const backups = listBackups();
-    expect(backups.length).toBeGreaterThan(0);
-  });
-});
-
-// ============================================================================
-// backupFile
-// ============================================================================
-
-describe("backupFile", () => {
-  it("creates a backup of an existing file", () => {
-    vol.fromJSON({
-      "/tmp/file.txt": "original content"
-    });
-    backupFile("/tmp/file.txt");
-    const backups = listBackups();
-    expect(backups.length).toBe(1);
-    expect(backups[0].name).toContain("file.txt");
-  });
-  it("is a no-op for files that do not exist", () => {
-    backupFile("/tmp/nonexistent.txt");
-    const backups = listBackups();
-    expect(backups.length).toBe(0);
-  });
-  it("creates session directory if needed", () => {
-    vol.fromJSON({
-      "/tmp/file.txt": "data"
-    });
-    backupFile("/tmp/file.txt");
-    expect(vol.existsSync(`${HISTORY_BASE}/test-session-1`)).toBe(true);
-  });
-  it("creates multiple backups for multiple calls", () => {
-    vol.fromJSON({
-      "/tmp/a.txt": "aaa",
-      "/tmp/b.txt": "bbb"
-    });
-    backupFile("/tmp/a.txt");
-    backupFile("/tmp/b.txt");
-    const backups = listBackups();
-    expect(backups.length).toBe(2);
-  });
-  it("performs FIFO cleanup when exceeding 100 backups", () => {
-    // Create 101 files to trigger cleanup
-    const files = {};
-    files["/tmp/source.txt"] = "src";
-    vol.fromJSON(files);
-
-    // Pre-populate 100 existing backups
-    const sessionDir = `${HISTORY_BASE}/test-session-1`;
-    vol.mkdirSync(sessionDir, {
-      recursive: true
-    });
-    for (let i = 0; i < 100; i++) {
-      vol.writeFileSync(`${sessionDir}/${String(i).padStart(5, "0")}-old.txt`, `backup-${i}`);
-    }
-    backupFile("/tmp/source.txt");
-    const backups = listBackups();
-    // After cleanup, should be <= 100
-    expect(backups.length).toBeLessThanOrEqual(100);
-  });
-});
-
-// ============================================================================
-// listBackups
-// ============================================================================
-
-describe("listBackups", () => {
-  it("returns empty array when no backups exist", () => {
-    expect(listBackups()).toEqual([]);
-  });
-  it("returns backups in reverse order (newest first)", () => {
-    const sessionDir = `${HISTORY_BASE}/test-session-1`;
-    vol.mkdirSync(sessionDir, {
-      recursive: true
-    });
-    vol.writeFileSync(`${sessionDir}/100-a.txt`, "a");
-    vol.writeFileSync(`${sessionDir}/200-b.txt`, "b");
-    vol.writeFileSync(`${sessionDir}/300-c.txt`, "c");
-    const backups = listBackups();
-    expect(backups.length).toBe(3);
-    expect(backups[0].name).toBe("300-c.txt");
-    expect(backups[2].name).toBe("100-a.txt");
-  });
-  it("each backup has name and path properties", () => {
-    const sessionDir = `${HISTORY_BASE}/test-session-1`;
-    vol.mkdirSync(sessionDir, {
-      recursive: true
-    });
-    vol.writeFileSync(`${sessionDir}/123-test.ts`, "code");
-    const backups = listBackups();
-    expect(backups[0]).toHaveProperty("name", "123-test.ts");
-    expect(backups[0]).toHaveProperty("path");
-    expect(backups[0].path).toContain("123-test.ts");
-  });
-});

package/dist/cli/services/__tests__/git-context-cache-reset.test.js

@@ -1,223 +0,0 @@
-import { describe, it, expect, vi, beforeEach, afterEach } from "vitest";
-
-// ============================================================================
-// Mocks — must be declared before imports
-// ============================================================================
-
-const {
-  mockExecAsync
-} = vi.hoisted(() => ({
-  mockExecAsync: vi.fn()
-}));
-vi.mock("child_process", () => ({
-  exec: vi.fn()
-}));
-vi.mock("util", () => ({
-  promisify: () => mockExecAsync
-}));
-import { gatherGitContext, refreshGitContext, resetGitContext } from "../git-context.js";
-
-// ============================================================================
-// Helpers
-// ============================================================================
-
-/** Simulate successful exec returning stdout */
-function mockExecSuccess(stdout) {
-  return Promise.resolve({
-    stdout,
-    stderr: ""
-  });
-}
-
-/** Simulate exec failure */
-function mockExecFailure(message = "Command failed") {
-  return Promise.reject(new Error(message));
-}
-function setupGitMocks(overrides = {}) {
-  const {
-    isRepo = true,
-    branch = "main",
-    status = "",
-    log = "abc1234 Initial commit"
-  } = overrides;
-  mockExecAsync.mockImplementation(cmd => {
-    if (cmd.includes("rev-parse --is-inside-work-tree")) {
-      return isRepo ? mockExecSuccess("true\n") : mockExecFailure("not a git repo");
-    }
-    if (cmd.includes("branch --show-current")) {
-      return branch !== null ? mockExecSuccess(`${branch}\n`) : mockExecFailure("no branch");
-    }
-    if (cmd.includes("git status --short")) {
-      return status !== null ? mockExecSuccess(status) : mockExecFailure("status failed");
-    }
-    if (cmd.includes("git log --oneline")) {
-      return log !== null ? mockExecSuccess(`${log}\n`) : mockExecFailure("log failed");
-    }
-    return mockExecFailure(`Unknown command: ${cmd}`);
-  });
-}
-
-// ============================================================================
-// Setup / Teardown
-// ============================================================================
-
-const originalCwd = process.cwd;
-beforeEach(() => {
-  mockExecAsync.mockReset();
-  resetGitContext();
-  process.cwd = () => "/test/project";
-});
-afterEach(() => {
-  process.cwd = originalCwd;
-});
-
-// ============================================================================
-// Cache behavior
-// ============================================================================
-
-describe("cache behavior", () => {
-  it("returns cached result on second call with same cwd", async () => {
-    setupGitMocks({
-      branch: "main",
-      status: "",
-      log: null
-    });
-    const first = await gatherGitContext();
-    const second = await gatherGitContext();
-    expect(first).toBe(second);
-    // rev-parse should only be called once (first call)
-    const revParseCalls = mockExecAsync.mock.calls.filter(c => typeof c[0] === "string" && c[0].includes("rev-parse"));
-    expect(revParseCalls).toHaveLength(1);
-  });
-  it("re-gathers when cwd changes", async () => {
-    setupGitMocks({
-      branch: "main",
-      status: "",
-      log: null
-    });
-    process.cwd = () => "/project-a";
-    await gatherGitContext();
-
-    // Change cwd
-    process.cwd = () => "/project-b";
-    setupGitMocks({
-      branch: "develop",
-      status: "",
-      log: null
-    });
-    const result = await gatherGitContext();
-    expect(result).toContain("Branch: develop");
-  });
-  it("caches empty string for non-repo directories", async () => {
-    setupGitMocks({
-      isRepo: false
-    });
-    const first = await gatherGitContext();
-    const second = await gatherGitContext();
-    expect(first).toBe("");
-    expect(second).toBe("");
-    // Only one rev-parse call
-    const revParseCalls = mockExecAsync.mock.calls.filter(c => typeof c[0] === "string" && c[0].includes("rev-parse"));
-    expect(revParseCalls).toHaveLength(1);
-  });
-});
-
-// ============================================================================
-// refreshGitContext
-// ============================================================================
-
-describe("refreshGitContext", () => {
-  it("clears the cache so next call re-gathers", async () => {
-    setupGitMocks({
-      branch: "main",
-      status: "",
-      log: null
-    });
-    await gatherGitContext();
-    refreshGitContext();
-
-    // After refresh, should re-execute git commands
-    setupGitMocks({
-      branch: "develop",
-      status: "",
-      log: null
-    });
-    const result = await gatherGitContext();
-    expect(result).toContain("Branch: develop");
-  });
-  it("does not throw when called without prior gathering", () => {
-    expect(() => refreshGitContext()).not.toThrow();
-  });
-});
-
-// ============================================================================
-// resetGitContext
-// ============================================================================
-
-describe("resetGitContext", () => {
-  it("clears the cache so next call re-gathers", async () => {
-    setupGitMocks({
-      branch: "main",
-      status: "",
-      log: null
-    });
-    await gatherGitContext();
-    resetGitContext();
-    setupGitMocks({
-      branch: "feature/x",
-      status: "",
-      log: null
-    });
-    const result = await gatherGitContext();
-    expect(result).toContain("Branch: feature/x");
-  });
-  it("does not throw when called without prior gathering", () => {
-    expect(() => resetGitContext()).not.toThrow();
-  });
-  it("resets both cached context and cwd tracking", async () => {
-    process.cwd = () => "/project-a";
-    setupGitMocks({
-      branch: "main",
-      status: "",
-      log: null
-    });
-    await gatherGitContext();
-    resetGitContext();
-
-    // Same cwd but cache is cleared, so should re-gather
-    setupGitMocks({
-      branch: "hotfix",
-      status: "",
-      log: null
-    });
-    const result = await gatherGitContext();
-    expect(result).toContain("Branch: hotfix");
-  });
-});
-
-// ============================================================================
-// Error scenarios
-// ============================================================================
-
-describe("error scenarios", () => {
-  it("handles rev-parse timeout gracefully", async () => {
-    mockExecAsync.mockImplementation(cmd => {
-      if (cmd.includes("rev-parse")) {
-        return Promise.reject(new Error("Command timed out"));
-      }
-      return mockExecSuccess("");
-    });
-    const result = await gatherGitContext();
-    expect(result).toBe("");
-  });
-  it("handles all commands timing out after successful rev-parse", async () => {
-    mockExecAsync.mockImplementation(cmd => {
-      if (cmd.includes("rev-parse")) {
-        return mockExecSuccess("true\n");
-      }
-      return mockExecFailure("Command timed out");
-    });
-    const result = await gatherGitContext();
-    expect(result).toBe("");
-  });
-});

package/dist/cli/services/__tests__/git-context.test.js

@@ -1,241 +0,0 @@
-import { describe, it, expect, vi, beforeEach, afterEach } from "vitest";
-
-// ============================================================================
-// Mocks — must be declared before imports
-// ============================================================================
-
-const {
-  mockExecAsync
-} = vi.hoisted(() => ({
-  mockExecAsync: vi.fn()
-}));
-vi.mock("child_process", () => ({
-  exec: vi.fn()
-}));
-vi.mock("util", () => ({
-  promisify: () => mockExecAsync
-}));
-import { gatherGitContext, resetGitContext } from "../git-context.js";
-
-// ============================================================================
-// Helpers
-// ============================================================================
-
-/** Simulate successful exec returning stdout */
-function mockExecSuccess(stdout) {
-  return Promise.resolve({
-    stdout,
-    stderr: ""
-  });
-}
-
-/** Simulate exec failure */
-function mockExecFailure(message = "Command failed") {
-  return Promise.reject(new Error(message));
-}
-
-/**
- * Set up mockExecAsync to respond to different git commands.
- * Commands matched by substring in the first argument.
- */
-function setupGitMocks(overrides = {}) {
-  const {
-    isRepo = true,
-    branch = "main",
-    status = "",
-    log = "abc1234 Initial commit"
-  } = overrides;
-  mockExecAsync.mockImplementation(cmd => {
-    if (cmd.includes("rev-parse --is-inside-work-tree")) {
-      return isRepo ? mockExecSuccess("true\n") : mockExecFailure("not a git repo");
-    }
-    if (cmd.includes("branch --show-current")) {
-      return branch !== null ? mockExecSuccess(`${branch}\n`) : mockExecFailure("no branch");
-    }
-    if (cmd.includes("git status --short")) {
-      return status !== null ? mockExecSuccess(status) : mockExecFailure("status failed");
-    }
-    if (cmd.includes("git log --oneline")) {
-      return log !== null ? mockExecSuccess(`${log}\n`) : mockExecFailure("log failed");
-    }
-    return mockExecFailure(`Unknown command: ${cmd}`);
-  });
-}
-
-// ============================================================================
-// Setup / Teardown
-// ============================================================================
-
-const originalCwd = process.cwd;
-beforeEach(() => {
-  mockExecAsync.mockReset();
-  // Reset the module-level cache before each test
-  resetGitContext();
-  // Default cwd
-  process.cwd = () => "/test/project";
-});
-afterEach(() => {
-  process.cwd = originalCwd;
-});
-
-// ============================================================================
-// gatherGitContext
-// ============================================================================
-
-describe("gatherGitContext", () => {
-  it("returns empty string when not inside a git repo", async () => {
-    setupGitMocks({
-      isRepo: false
-    });
-    const result = await gatherGitContext();
-    expect(result).toBe("");
-  });
-  it("includes branch name when available", async () => {
-    setupGitMocks({
-      branch: "feature/login",
-      status: "",
-      log: null
-    });
-    const result = await gatherGitContext();
-    expect(result).toContain("Branch: feature/login");
-  });
-  it("includes clean status when no changes", async () => {
-    setupGitMocks({
-      status: "",
-      log: null
-    });
-    const result = await gatherGitContext();
-    expect(result).toContain("Status: clean");
-  });
-  it("shows changed file count for single changed file", async () => {
-    setupGitMocks({
-      status: " M src/index.ts\n",
-      log: null
-    });
-    const result = await gatherGitContext();
-    expect(result).toContain("Status: 1 changed file");
-    // No trailing 's' for singular
-    expect(result).not.toContain("Status: 1 changed files");
-  });
-  it("shows pluralized changed file count for multiple files", async () => {
-    const statusLines = " M src/a.ts\n M src/b.ts\n M src/c.ts";
-    setupGitMocks({
-      status: statusLines,
-      log: null
-    });
-    const result = await gatherGitContext();
-    expect(result).toContain("Status: 3 changed files");
-  });
-  it("includes the status output lines", async () => {
-    const statusLines = " M src/index.ts\n?? src/new-file.ts";
-    setupGitMocks({
-      status: statusLines,
-      log: null
-    });
-    const result = await gatherGitContext();
-    expect(result).toContain("M src/index.ts");
-    expect(result).toContain("?? src/new-file.ts");
-  });
-  it("includes recent commits", async () => {
-    const log = "abc1234 Fix bug\ndef5678 Add feature";
-    setupGitMocks({
-      status: "",
-      log
-    });
-    const result = await gatherGitContext();
-    expect(result).toContain("Recent commits:");
-    expect(result).toContain("abc1234 Fix bug");
-    expect(result).toContain("def5678 Add feature");
-  });
-  it("combines branch, status, and log into a single context string", async () => {
-    const log = "abc1234 Initial commit";
-    setupGitMocks({
-      branch: "main",
-      status: " M README.md\n",
-      log
-    });
-    const result = await gatherGitContext();
-    expect(result).toContain("Branch: main");
-    expect(result).toContain("Status: 1 changed file");
-    expect(result).toContain("Recent commits:");
-    expect(result).toContain("abc1234 Initial commit");
-  });
-  it("skips branch section when branch command fails", async () => {
-    setupGitMocks({
-      branch: null,
-      status: "",
-      log: null
-    });
-    const result = await gatherGitContext();
-    expect(result).not.toContain("Branch:");
-    expect(result).toContain("Status: clean");
-  });
-  it("skips status section when status command fails", async () => {
-    setupGitMocks({
-      branch: "main",
-      status: null,
-      log: null
-    });
-    const result = await gatherGitContext();
-    expect(result).toContain("Branch: main");
-    expect(result).not.toContain("Status:");
-  });
-  it("skips log section when log command fails", async () => {
-    setupGitMocks({
-      branch: "main",
-      status: "",
-      log: null
-    });
-    const result = await gatherGitContext();
-    expect(result).toContain("Branch: main");
-    expect(result).not.toContain("Recent commits:");
-  });
-  it("returns empty string when inside a repo but all subcommands fail", async () => {
-    setupGitMocks({
-      branch: null,
-      status: null,
-      log: null
-    });
-    const result = await gatherGitContext();
-    expect(result).toBe("");
-  });
-  it("passes cwd to exec calls", async () => {
-    process.cwd = () => "/custom/path";
-    setupGitMocks({
-      isRepo: true
-    });
-    await gatherGitContext();
-    // Verify the first exec call used the correct cwd
-    expect(mockExecAsync).toHaveBeenCalledWith(expect.stringContaining("rev-parse"), expect.objectContaining({
-      cwd: "/custom/path"
-    }));
-  });
-  it("passes timeout of 3000ms to exec calls", async () => {
-    setupGitMocks({
-      isRepo: true
-    });
-    await gatherGitContext();
-    expect(mockExecAsync).toHaveBeenCalledWith(expect.any(String), expect.objectContaining({
-      timeout: 3000
-    }));
-  });
-  it("handles empty branch name gracefully", async () => {
-    setupGitMocks({
-      branch: "",
-      status: "",
-      log: null
-    });
-    const result = await gatherGitContext();
-    expect(result).not.toContain("Branch:");
-    expect(result).toContain("Status: clean");
-  });
-  it("handles empty log output", async () => {
-    setupGitMocks({
-      branch: "main",
-      status: "",
-      log: ""
-    });
-    const result = await gatherGitContext();
-    expect(result).not.toContain("Recent commits:");
-  });
-});