@nathapp/nax 0.28.0 → 0.30.0

package/test/unit/verification/smart-runner.test.ts

@@ -1,262 +0,0 @@
-import { afterEach, beforeAll, beforeEach, describe, expect, mock, test } from "bun:test";
-
-// Dynamic imports used intentionally: mock.module() in verify-smart-runner.test.ts (which runs
-// before this file alphabetically) poisons the ESM registry. Static imports capture the mock
-// binding at eval time. Dynamic imports inside beforeAll resolve AFTER the other file's afterAll
-// restores the real module, guaranteeing we test the real implementation.
-let buildSmartTestCommand: (testFiles: string[], baseCommand: string) => string;
-let mapSourceToTests: (sourceFiles: string[], workdir: string) => Promise<string[]>;
-let getChangedSourceFiles: (workdir: string) => Promise<string[]>;
-
-beforeAll(async () => {
-  const mod = await import("../../../src/verification/smart-runner");
-  buildSmartTestCommand = mod.buildSmartTestCommand;
-  mapSourceToTests = mod.mapSourceToTests;
-  getChangedSourceFiles = mod.getChangedSourceFiles;
-});
-
-// ---------------------------------------------------------------------------
-// buildSmartTestCommand
-// ---------------------------------------------------------------------------
-
-describe("buildSmartTestCommand", () => {
-  test("returns original command when testFiles is empty", () => {
-    const result = buildSmartTestCommand([], "bun test test/");
-    expect(result).toBe("bun test test/");
-  });
-
-  test("replaces last path argument with specific test file", () => {
-    const result = buildSmartTestCommand(["test/unit/foo.test.ts"], "bun test test/");
-    expect(result).toBe("bun test test/unit/foo.test.ts");
-  });
-
-  test("joins multiple test files with spaces", () => {
-    const result = buildSmartTestCommand(
-      ["test/unit/foo.test.ts", "test/unit/bar.test.ts"],
-      "bun test test/",
-    );
-    expect(result).toBe("bun test test/unit/foo.test.ts test/unit/bar.test.ts");
-  });
-
-  test("appends test files when command has no path argument", () => {
-    const result = buildSmartTestCommand(["test/unit/foo.test.ts"], "bun test");
-    expect(result).toBe("bun test test/unit/foo.test.ts");
-  });
-
-  test("replaces last path-like token even when flags precede it", () => {
-    const result = buildSmartTestCommand(
-      ["test/unit/foo.test.ts"],
-      "bun test --coverage test/",
-    );
-    expect(result).toBe("bun test --coverage test/unit/foo.test.ts");
-  });
-
-  test("preserves trailing flags after path argument (BUG-043)", () => {
-    const result = buildSmartTestCommand(
-      ["test/unit/foo.test.ts"],
-      "bun test test/ --timeout=60000",
-    );
-    expect(result).toBe("bun test test/unit/foo.test.ts --timeout=60000");
-  });
-
-  test("preserves trailing flags with multiple test files", () => {
-    const result = buildSmartTestCommand(
-      ["test/unit/foo.test.ts", "test/unit/bar.test.ts"],
-      "bun test test/ --timeout=60000 --bail",
-    );
-    expect(result).toBe("bun test test/unit/foo.test.ts test/unit/bar.test.ts --timeout=60000 --bail");
-  });
-});
-
-// ---------------------------------------------------------------------------
-// Helpers to mock Bun.spawn (used internally via the "bun" import alias)
-// ---------------------------------------------------------------------------
-
-function makeProc(stdout: string, exitCode: number) {
-  return {
-    exited: Promise.resolve(exitCode),
-    stdout: new ReadableStream({
-      start(controller) {
-        controller.enqueue(new TextEncoder().encode(stdout));
-        controller.close();
-      },
-    }),
-    stderr: new ReadableStream({
-      start(controller) {
-        controller.close();
-      },
-    }),
-  };
-}
-
-// ---------------------------------------------------------------------------
-// mapSourceToTests
-// ---------------------------------------------------------------------------
-
-describe("mapSourceToTests", () => {
-  let originalFile: typeof Bun.file;
-
-  beforeEach(() => {
-    originalFile = Bun.file;
-  });
-
-  afterEach(() => {
-    // biome-ignore lint/suspicious/noExplicitAny: restoring original
-    (Bun as any).file = originalFile;
-  });
-
-  function mockFileExists(existingPaths: string[]) {
-    // biome-ignore lint/suspicious/noExplicitAny: mocking
-    (Bun as any).file = (path: string) => ({
-      exists: () => Promise.resolve(existingPaths.includes(path)),
-    });
-  }
-
-  test("maps src/foo/bar.ts to test/unit/foo/bar.test.ts", async () => {
-    mockFileExists(["/repo/test/unit/foo/bar.test.ts"]);
-
-    const result = await mapSourceToTests(["src/foo/bar.ts"], "/repo");
-
-    expect(result).toEqual(["/repo/test/unit/foo/bar.test.ts"]);
-  });
-
-  test("also checks test/integration/ path", async () => {
-    mockFileExists(["/repo/test/integration/foo/bar.test.ts"]);
-
-    const result = await mapSourceToTests(["src/foo/bar.ts"], "/repo");
-
-    expect(result).toEqual(["/repo/test/integration/foo/bar.test.ts"]);
-  });
-
-  test("returns both unit and integration when both exist", async () => {
-    mockFileExists([
-      "/repo/test/unit/foo/bar.test.ts",
-      "/repo/test/integration/foo/bar.test.ts",
-    ]);
-
-    const result = await mapSourceToTests(["src/foo/bar.ts"], "/repo");
-
-    expect(result).toEqual([
-      "/repo/test/unit/foo/bar.test.ts",
-      "/repo/test/integration/foo/bar.test.ts",
-    ]);
-  });
-
-  test("only returns files that exist on disk", async () => {
-    // Only unit test exists, integration does not
-    mockFileExists(["/repo/test/unit/utils/helper.test.ts"]);
-
-    const result = await mapSourceToTests(["src/utils/helper.ts"], "/repo");
-
-    expect(result).toEqual(["/repo/test/unit/utils/helper.test.ts"]);
-    expect(result).not.toContain("/repo/test/integration/utils/helper.test.ts");
-  });
-
-  test("returns empty array when no test files match", async () => {
-    mockFileExists([]); // nothing exists
-
-    const result = await mapSourceToTests(["src/foo/bar.ts"], "/repo");
-
-    expect(result).toEqual([]);
-  });
-
-  test("returns empty array for empty sourceFiles input", async () => {
-    mockFileExists(["/repo/test/unit/foo/bar.test.ts"]);
-
-    const result = await mapSourceToTests([], "/repo");
-
-    expect(result).toEqual([]);
-  });
-
-  test("handles multiple source files and aggregates results", async () => {
-    mockFileExists([
-      "/repo/test/unit/foo/bar.test.ts",
-      "/repo/test/unit/baz/qux.test.ts",
-    ]);
-
-    const result = await mapSourceToTests(
-      ["src/foo/bar.ts", "src/baz/qux.ts"],
-      "/repo",
-    );
-
-    expect(result).toEqual([
-      "/repo/test/unit/foo/bar.test.ts",
-      "/repo/test/unit/baz/qux.test.ts",
-    ]);
-  });
-});
-
-describe("getChangedSourceFiles", () => {
-  let originalSpawn: typeof Bun.spawn;
-
-  beforeEach(() => {
-    originalSpawn = Bun.spawn;
-  });
-
-  afterEach(() => {
-    // biome-ignore lint/suspicious/noExplicitAny: restoring original
-    (Bun as any).spawn = originalSpawn;
-  });
-
-  test("returns only .ts files under src/", async () => {
-    const gitOutput = [
-      "src/verification/smart-runner.ts",
-      "src/utils/git.ts",
-      "README.md",
-      "src/index.js",
-      "test/unit/foo.test.ts",
-      "src/config/schema.ts",
-    ].join("\n");
-
-    // biome-ignore lint/suspicious/noExplicitAny: mocking
-    (Bun as any).spawn = mock(() => makeProc(gitOutput, 0));
-
-    const result = await getChangedSourceFiles("/fake/repo");
-
-    expect(result).toEqual([
-      "src/verification/smart-runner.ts",
-      "src/utils/git.ts",
-      "src/config/schema.ts",
-    ]);
-  });
-
-  test("returns empty array when git exits with non-zero code", async () => {
-    // biome-ignore lint/suspicious/noExplicitAny: mocking
-    (Bun as any).spawn = mock(() => makeProc("", 128));
-
-    const result = await getChangedSourceFiles("/fake/repo");
-
-    expect(result).toEqual([]);
-  });
-
-  test("returns empty array when git throws (e.g. not a repo)", async () => {
-    // biome-ignore lint/suspicious/noExplicitAny: mocking
-    (Bun as any).spawn = mock(() => {
-      throw new Error("git not found");
-    });
-
-    const result = await getChangedSourceFiles("/fake/repo");
-
-    expect(result).toEqual([]);
-  });
-
-  test("filters out non-.ts src files", async () => {
-    const gitOutput = ["src/foo.js", "src/bar.tsx", "src/baz.ts"].join("\n");
-
-    // biome-ignore lint/suspicious/noExplicitAny: mocking
-    (Bun as any).spawn = mock(() => makeProc(gitOutput, 0));
-
-    const result = await getChangedSourceFiles("/fake/repo");
-
-    expect(result).toEqual(["src/baz.ts"]);
-  });
-
-  test("returns empty array when no files changed", async () => {
-    // biome-ignore lint/suspicious/noExplicitAny: mocking
-    (Bun as any).spawn = mock(() => makeProc("", 0));
-
-    const result = await getChangedSourceFiles("/fake/repo");
-
-    expect(result).toEqual([]);
-  });
-});

package/test/unit/verification/strategies/acceptance.test.ts

@@ -1,33 +0,0 @@
-// RE-ARCH: keep
-import { describe, expect, test } from "bun:test";
-import { AcceptanceStrategy } from "../../../../src/verification/strategies/acceptance";
-import type { VerifyContext } from "../../../../src/verification/orchestrator-types";
-
-function makeCtx(overrides: Partial<VerifyContext> = {}): VerifyContext {
-  return {
-    workdir: "/tmp/test-repo",
-    testCommand: "bun test",
-    timeoutSeconds: 60,
-    storyId: "US-001",
-    ...overrides,
-  };
-}
-
-describe("AcceptanceStrategy", () => {
-  test("name is acceptance", () => {
-    expect(new AcceptanceStrategy().name).toBe("acceptance");
-  });
-
-  test("returns SKIPPED when no acceptanceTestPath", async () => {
-    const result = await new AcceptanceStrategy().execute(makeCtx());
-    expect(result.status).toBe("SKIPPED");
-    expect(result.success).toBe(true);
-  });
-
-  test("returns SKIPPED when test file does not exist", async () => {
-    const result = await new AcceptanceStrategy().execute(
-      makeCtx({ acceptanceTestPath: "/nonexistent/path/acceptance.test.ts" }),
-    );
-    expect(result.status).toBe("SKIPPED");
-  });
-});

package/test/unit/verification/strategies/regression.test.ts

@@ -1,87 +0,0 @@
-// RE-ARCH: keep
-import { describe, expect, test } from "bun:test";
-import { RegressionStrategy, _regressionStrategyDeps } from "../../../../src/verification/strategies/regression";
-import type { VerifyContext } from "../../../../src/verification/orchestrator-types";
-import { DEFAULT_CONFIG } from "../../../../src/config";
-
-function makeCtx(overrides: Partial<VerifyContext> = {}): VerifyContext {
-  return {
-    workdir: "/tmp/test-repo",
-    testCommand: "bun test",
-    timeoutSeconds: 120,
-    storyId: "US-001",
-    acceptOnTimeout: true,
-    config: DEFAULT_CONFIG,
-    ...overrides,
-  };
-}
-
-describe("RegressionStrategy", () => {
-  test("name is regression", () => {
-    expect(new RegressionStrategy().name).toBe("regression");
-  });
-
-  test("returns SKIPPED when gate disabled", async () => {
-    const config = {
-      ...DEFAULT_CONFIG,
-      execution: {
-        ...DEFAULT_CONFIG.execution,
-        regressionGate: { enabled: false, timeoutSeconds: 120, mode: "inline" as const },
-      },
-    };
-    const result = await new RegressionStrategy().execute(makeCtx({ config }));
-    expect(result.status).toBe("SKIPPED");
-    expect(result.success).toBe(true);
-  });
-
-  test("returns PASS when tests pass", async () => {
-    const saved = { ...(_regressionStrategyDeps as any) };
-    _regressionStrategyDeps.runVerification = async () => ({
-      success: true,
-      status: "SUCCESS" as const,
-      countsTowardEscalation: false,
-      output: "10 pass",
-    });
-
-    const result = await new RegressionStrategy().execute(makeCtx());
-
-    Object.assign(_regressionStrategyDeps, saved);
-
-    expect(result.success).toBe(true);
-    expect(result.status).toBe("PASS");
-    expect(result.strategy).toBe("regression");
-  });
-
-  test("accepts TIMEOUT as pass when acceptOnTimeout=true", async () => {
-    const saved = { ...(_regressionStrategyDeps as any) };
-    _regressionStrategyDeps.runVerification = async () => ({
-      success: false,
-      status: "TIMEOUT" as const,
-      countsTowardEscalation: false,
-    });
-
-    const result = await new RegressionStrategy().execute(makeCtx({ acceptOnTimeout: true }));
-
-    Object.assign(_regressionStrategyDeps, saved);
-
-    expect(result.success).toBe(true);
-    expect(result.status).toBe("PASS");
-  });
-
-  test("returns TEST_FAILURE when tests fail", async () => {
-    const saved = { ...(_regressionStrategyDeps as any) };
-    _regressionStrategyDeps.runVerification = async () => ({
-      success: false,
-      status: "TEST_FAILURE" as const,
-      countsTowardEscalation: true,
-      output: "(fail) x > y\n2 fail",
-    });
-
-    const result = await new RegressionStrategy().execute(makeCtx({ acceptOnTimeout: false }));
-
-    Object.assign(_regressionStrategyDeps, saved);
-
-    expect(result.success).toBe(false);
-    expect(result.status).toBe("TEST_FAILURE");
-  });
-});

package/test/unit/verification/strategies/scoped.test.ts

@@ -1,100 +0,0 @@
-// RE-ARCH: keep
-import { describe, expect, test } from "bun:test";
-import { ScopedStrategy, _scopedDeps } from "../../../../src/verification/strategies/scoped";
-import type { VerifyContext } from "../../../../src/verification/orchestrator-types";
-
-function makeCtx(overrides: Partial<VerifyContext> = {}): VerifyContext {
-  return {
-    workdir: "/tmp/test-repo",
-    testCommand: "bun test",
-    timeoutSeconds: 60,
-    storyId: "US-001",
-    storyGitRef: "abc123",
-    regressionMode: "deferred",
-    acceptOnTimeout: true,
-    ...overrides,
-  };
-}
-
-describe("ScopedStrategy", () => {
-  test("name is scoped", () => {
-    expect(new ScopedStrategy().name).toBe("scoped");
-  });
-
-  test("returns SKIPPED when deferred mode and no mapped tests", async () => {
-    const saved = { ..._scopedDeps };
-    _scopedDeps.getChangedSourceFiles = async () => [];
-    _scopedDeps.mapSourceToTests = async () => [];
-    _scopedDeps.importGrepFallback = async () => [];
-
-    const result = await new ScopedStrategy().execute(makeCtx({ regressionMode: "deferred" }));
-
-    Object.assign(_scopedDeps, saved);
-
-    expect(result.status).toBe("SKIPPED");
-    expect(result.success).toBe(true);
-    expect(result.countsTowardEscalation).toBe(false);
-  });
-
-  test("runs full suite when inline mode and no mapped tests", async () => {
-    const saved = { ..._scopedDeps };
-    _scopedDeps.getChangedSourceFiles = async () => [];
-    _scopedDeps.mapSourceToTests = async () => [];
-    _scopedDeps.importGrepFallback = async () => [];
-    _scopedDeps.regression = async () => ({
-      success: true,
-      status: "SUCCESS" as const,
-      countsTowardEscalation: false,
-      output: "1 pass",
-    });
-
-    const result = await new ScopedStrategy().execute(makeCtx({ regressionMode: "inline" }));
-
-    Object.assign(_scopedDeps, saved);
-
-    expect(result.success).toBe(true);
-    expect(result.status).toBe("PASS");
-  });
-
-  test("returns PASS when tests pass", async () => {
-    const saved = { ..._scopedDeps };
-    _scopedDeps.getChangedSourceFiles = async () => ["src/foo.ts"];
-    _scopedDeps.mapSourceToTests = async () => ["test/unit/foo.test.ts"];
-    _scopedDeps.buildSmartTestCommand = (_files: string[], cmd: string) => `${cmd} test/unit/foo.test.ts`;
-    _scopedDeps.regression = async () => ({
-      success: true,
-      status: "SUCCESS" as const,
-      countsTowardEscalation: false,
-      output: "5 pass\n0 fail",
-    });
-
-    const result = await new ScopedStrategy().execute(makeCtx());
-
-    Object.assign(_scopedDeps, saved);
-
-    expect(result.success).toBe(true);
-    expect(result.status).toBe("PASS");
-    expect(result.strategy).toBe("scoped");
-  });
-
-  test("returns TEST_FAILURE when tests fail", async () => {
-    const saved = { ..._scopedDeps };
-    _scopedDeps.getChangedSourceFiles = async () => ["src/foo.ts"];
-    _scopedDeps.mapSourceToTests = async () => ["test/unit/foo.test.ts"];
-    _scopedDeps.buildSmartTestCommand = (_files: string[], cmd: string) => cmd;
-    _scopedDeps.regression = async () => ({
-      success: false,
-      status: "TEST_FAILURE" as const,
-      countsTowardEscalation: true,
-      output: "(fail) foo > bar\n1 fail",
-    });
-
-    const result = await new ScopedStrategy().execute(makeCtx());
-
-    Object.assign(_scopedDeps, saved);
-
-    expect(result.success).toBe(false);
-    expect(result.status).toBe("TEST_FAILURE");
-    expect(result.countsTowardEscalation).toBe(true);
-  });
-});

package/test/unit/worktree-manager.test.ts

@@ -1,159 +0,0 @@
-// RE-ARCH: keep
-/**
- * Tests for src/worktree/manager.ts
- *
- * Covers: WorktreeManager create, remove, list, parseWorktreeList
- */
-
-import { describe, expect, it } from "bun:test";
-import { WorktreeManager } from "../../src/worktree/manager";
-
-// ─────────────────────────────────────────────────────────────────────────────
-// Test fixtures
-// ─────────────────────────────────────────────────────────────────────────────
-
-const mockWorktreeListOutput = `worktree /path/to/project
-HEAD abc123def456
-branch refs/heads/master
-
-worktree /path/to/project/.nax-wt/US-001
-HEAD 123456abcdef
-branch refs/heads/nax/US-001
-
-worktree /path/to/project/.nax-wt/US-002
-HEAD 789012abcdef
-branch refs/heads/nax/US-002
-
-`;
-
-// ─────────────────────────────────────────────────────────────────────────────
-// WorktreeManager.parseWorktreeList
-// ─────────────────────────────────────────────────────────────────────────────
-
-describe("WorktreeManager.parseWorktreeList", () => {
-  it("parses git worktree list --porcelain output", () => {
-    const manager = new WorktreeManager();
-    // @ts-expect-error - accessing private method for testing
-    const worktrees = manager.parseWorktreeList(mockWorktreeListOutput);
-
-    expect(worktrees.length).toBe(3);
-
-    expect(worktrees[0].path).toBe("/path/to/project");
-    expect(worktrees[0].branch).toBe("master");
-
-    expect(worktrees[1].path).toBe("/path/to/project/.nax-wt/US-001");
-    expect(worktrees[1].branch).toBe("nax/US-001");
-
-    expect(worktrees[2].path).toBe("/path/to/project/.nax-wt/US-002");
-    expect(worktrees[2].branch).toBe("nax/US-002");
-  });
-
-  it("handles empty output", () => {
-    const manager = new WorktreeManager();
-    // @ts-expect-error - accessing private method for testing
-    const worktrees = manager.parseWorktreeList("");
-
-    expect(worktrees.length).toBe(0);
-  });
-
-  it("handles single worktree", () => {
-    const singleOutput = `worktree /path/to/project
-HEAD abc123def456
-branch refs/heads/master
-
-`;
-
-    const manager = new WorktreeManager();
-    // @ts-expect-error - accessing private method for testing
-    const worktrees = manager.parseWorktreeList(singleOutput);
-
-    expect(worktrees.length).toBe(1);
-    expect(worktrees[0].path).toBe("/path/to/project");
-    expect(worktrees[0].branch).toBe("master");
-  });
-
-  it("handles output without trailing newline", () => {
-    const noTrailingNewline = `worktree /path/to/project
-HEAD abc123def456
-branch refs/heads/master`;
-
-    const manager = new WorktreeManager();
-    // @ts-expect-error - accessing private method for testing
-    const worktrees = manager.parseWorktreeList(noTrailingNewline);
-
-    expect(worktrees.length).toBe(1);
-    expect(worktrees[0].path).toBe("/path/to/project");
-    expect(worktrees[0].branch).toBe("master");
-  });
-
-  it("strips refs/heads/ prefix from branches", () => {
-    const output = `worktree /path/to/project
-branch refs/heads/feature/my-feature
-
-`;
-
-    const manager = new WorktreeManager();
-    // @ts-expect-error - accessing private method for testing
-    const worktrees = manager.parseWorktreeList(output);
-
-    expect(worktrees[0].branch).toBe("feature/my-feature");
-  });
-
-  it("handles worktrees with detached HEAD", () => {
-    const output = `worktree /path/to/project
-HEAD abc123def456
-
-worktree /path/to/project/.nax-wt/US-001
-HEAD 123456abcdef
-branch refs/heads/nax/US-001
-
-`;
-
-    const manager = new WorktreeManager();
-    // @ts-expect-error - accessing private method for testing
-    const worktrees = manager.parseWorktreeList(output);
-
-    // First worktree has no branch, should be filtered out
-    expect(worktrees.length).toBe(1);
-    expect(worktrees[0].branch).toBe("nax/US-001");
-  });
-
-  it("filters incomplete entries missing path", () => {
-    const output = `branch refs/heads/master
-
-worktree /path/to/project
-branch refs/heads/feature
-
-`;
-
-    const manager = new WorktreeManager();
-    // @ts-expect-error - accessing private method for testing
-    const worktrees = manager.parseWorktreeList(output);
-
-    expect(worktrees.length).toBe(1);
-    expect(worktrees[0].path).toBe("/path/to/project");
-    expect(worktrees[0].branch).toBe("feature");
-  });
-
-  it("handles multiple empty lines between entries", () => {
-    const output = `worktree /path/to/project
-branch refs/heads/master
-
-
-worktree /path/to/project/.nax-wt/US-001
-branch refs/heads/nax/US-001
-
-`;
-
-    const manager = new WorktreeManager();
-    // @ts-expect-error - accessing private method for testing
-    const worktrees = manager.parseWorktreeList(output);
-
-    expect(worktrees.length).toBe(2);
-    expect(worktrees[0].branch).toBe("master");
-    expect(worktrees[1].branch).toBe("nax/US-001");
-  });
-});
-
-// Note: Error handling tests for WorktreeManager require git integration
-// and are better suited for integration tests rather than unit tests

package/tsconfig.json

@@ -1,27 +0,0 @@
-{
-  "compilerOptions": {
-    "target": "ESNext",
-    "module": "ESNext",
-    "moduleResolution": "bundler",
-    "lib": ["ESNext"],
-    "types": ["bun-types"],
-    "strict": true,
-    "esModuleInterop": true,
-    "skipLibCheck": true,
-    "forceConsistentCasingInFileNames": true,
-    "resolveJsonModule": true,
-    "declaration": true,
-    "declarationMap": true,
-    "sourceMap": true,
-    "outDir": "./dist",
-    "rootDir": ".",
-    "baseUrl": ".",
-    "paths": {
-      "@/*": ["./src/*"]
-    },
-    "jsx": "react-jsx",
-    "jsxImportSource": "react"
-  },
-  "include": ["src/**/*.ts", "src/**/*.tsx", "bin/**/*.ts"],
-  "exclude": ["node_modules", "dist"]
-}