@nathapp/nax 0.28.0 → 0.30.0

package/test/unit/ui/tui-stories.test.ts

@@ -1,333 +0,0 @@
-// RE-ARCH: keep
-/**
- * TUI Stories Panel Tests
- *
- * Tests the StoriesPanel component rendering with different story states,
- * cost display, elapsed time formatting, and layout breakpoint logic.
- */
-
-import { describe, expect, test } from "bun:test";
-import { render } from "ink-testing-library";
-import { createElement } from "react";
-import type { UserStory } from "../../../src/prd/types";
-import { StatusBar } from "../../../src/tui/components/StatusBar";
-import { StoriesPanel } from "../../../src/tui/components/StoriesPanel";
-import type { StoryDisplayState } from "../../../src/tui/types";
-
-// Helper to create mock stories
-function createMockStory(id: string, status: StoryDisplayState["status"]): StoryDisplayState {
-  const story: UserStory = {
-    id,
-    title: `Test story ${id}`,
-    description: "Test description",
-    acceptanceCriteria: [],
-    dependencies: [],
-    tags: [],
-    passes: status === "passed",
-    status: status === "passed" ? "passed" : "pending",
-    escalations: [],
-    attempts: 0,
-  };
-
-  return {
-    story,
-    status,
-    routing: {
-      complexity: "simple",
-      modelTier: "fast",
-      testStrategy: "test-after",
-      reasoning: "Test routing",
-    },
-    cost: 0.01,
-  };
-}
-
-describe("StoriesPanel", () => {
-  test("renders pending story with ⬚ icon", () => {
-    const stories = [createMockStory("US-001", "pending")];
-    const { lastFrame } = render(
-      createElement(StoriesPanel, {
-        stories,
-        totalCost: 0,
-        elapsedMs: 0,
-        width: 30,
-      }),
-    );
-
-    expect(lastFrame()).toContain("⬚ US-001");
-  });
-
-  test("renders running story with 🔄 icon", () => {
-    const stories = [createMockStory("US-001", "running")];
-    const { lastFrame } = render(
-      createElement(StoriesPanel, {
-        stories,
-        totalCost: 0,
-        elapsedMs: 0,
-        width: 30,
-      }),
-    );
-
-    expect(lastFrame()).toContain("🔄 US-001");
-  });
-
-  test("renders passed story with ✅ icon", () => {
-    const stories = [createMockStory("US-001", "passed")];
-    const { lastFrame } = render(
-      createElement(StoriesPanel, {
-        stories,
-        totalCost: 0,
-        elapsedMs: 0,
-        width: 30,
-      }),
-    );
-
-    expect(lastFrame()).toContain("✅ US-001");
-  });
-
-  test("renders failed story with ❌ icon", () => {
-    const stories = [createMockStory("US-001", "failed")];
-    const { lastFrame } = render(
-      createElement(StoriesPanel, {
-        stories,
-        totalCost: 0,
-        elapsedMs: 0,
-        width: 30,
-      }),
-    );
-
-    expect(lastFrame()).toContain("❌ US-001");
-  });
-
-  test("renders skipped story with ⏭️ icon", () => {
-    const stories = [createMockStory("US-001", "skipped")];
-    const { lastFrame } = render(
-      createElement(StoriesPanel, {
-        stories,
-        totalCost: 0,
-        elapsedMs: 0,
-        width: 30,
-      }),
-    );
-
-    expect(lastFrame()).toContain("⏭️ US-001");
-  });
-
-  test("renders retrying story with 🔁 icon", () => {
-    const stories = [createMockStory("US-001", "retrying")];
-    const { lastFrame } = render(
-      createElement(StoriesPanel, {
-        stories,
-        totalCost: 0,
-        elapsedMs: 0,
-        width: 30,
-      }),
-    );
-
-    expect(lastFrame()).toContain("🔁 US-001");
-  });
-
-  test("renders paused story with ⏸️ icon", () => {
-    const stories = [createMockStory("US-001", "paused")];
-    const { lastFrame } = render(
-      createElement(StoriesPanel, {
-        stories,
-        totalCost: 0,
-        elapsedMs: 0,
-        width: 30,
-      }),
-    );
-
-    expect(lastFrame()).toContain("⏸️ US-001");
-  });
-
-  test("displays routing info (complexity and model tier)", () => {
-    const stories = [createMockStory("US-001", "pending")];
-    const { lastFrame } = render(
-      createElement(StoriesPanel, {
-        stories,
-        totalCost: 0,
-        elapsedMs: 0,
-        width: 30,
-      }),
-    );
-
-    const output = lastFrame();
-    expect(output).toContain("sim"); // "simple".slice(0, 3) = "sim"
-    expect(output).toContain("fast");
-  });
-
-  test("displays total cost", () => {
-    const stories = [createMockStory("US-001", "passed")];
-    const { lastFrame } = render(
-      createElement(StoriesPanel, {
-        stories,
-        totalCost: 0.4235,
-        elapsedMs: 0,
-        width: 30,
-      }),
-    );
-
-    expect(lastFrame()).toContain("$0.4235");
-  });
-
-  test("displays elapsed time in mm:ss format", () => {
-    const stories = [createMockStory("US-001", "running")];
-    const { lastFrame } = render(
-      createElement(StoriesPanel, {
-        stories,
-        totalCost: 0,
-        elapsedMs: 263000, // 4 minutes 23 seconds
-        width: 30,
-      }),
-    );
-
-    expect(lastFrame()).toContain("4m 23s");
-  });
-
-  test("renders multiple stories", () => {
-    const stories = [
-      createMockStory("US-001", "passed"),
-      createMockStory("US-002", "running"),
-      createMockStory("US-003", "pending"),
-    ];
-
-    const { lastFrame } = render(
-      createElement(StoriesPanel, {
-        stories,
-        totalCost: 0.05,
-        elapsedMs: 120000,
-        width: 30,
-      }),
-    );
-
-    const output = lastFrame();
-    expect(output).toContain("✅ US-001");
-    expect(output).toContain("🔄 US-002");
-    expect(output).toContain("⬚ US-003");
-  });
-});
-
-describe("StatusBar", () => {
-  test("displays 'Idle' when no current story", () => {
-    const { lastFrame } = render(createElement(StatusBar, {}));
-    expect(lastFrame()).toContain("Idle");
-  });
-
-  test("displays current story ID", () => {
-    const story: UserStory = {
-      id: "US-042",
-      title: "Test story",
-      description: "Test",
-      acceptanceCriteria: [],
-      dependencies: [],
-      tags: [],
-      passes: false,
-      status: "pending",
-      escalations: [],
-      attempts: 0,
-    };
-
-    const { lastFrame } = render(
-      createElement(StatusBar, {
-        currentStory: story,
-      }),
-    );
-
-    expect(lastFrame()).toContain("Story US-042");
-  });
-
-  test("displays current stage", () => {
-    const story: UserStory = {
-      id: "US-001",
-      title: "Test",
-      description: "Test",
-      acceptanceCriteria: [],
-      dependencies: [],
-      tags: [],
-      passes: false,
-      status: "pending",
-      escalations: [],
-      attempts: 0,
-    };
-
-    const { lastFrame } = render(
-      createElement(StatusBar, {
-        currentStory: story,
-        currentStage: "execution",
-      }),
-    );
-
-    expect(lastFrame()).toContain("execution");
-  });
-
-  test("displays model tier", () => {
-    const story: UserStory = {
-      id: "US-001",
-      title: "Test",
-      description: "Test",
-      acceptanceCriteria: [],
-      dependencies: [],
-      tags: [],
-      passes: false,
-      status: "pending",
-      escalations: [],
-      attempts: 0,
-    };
-
-    const { lastFrame } = render(
-      createElement(StatusBar, {
-        currentStory: story,
-        modelTier: "balanced",
-      }),
-    );
-
-    expect(lastFrame()).toContain("balanced");
-  });
-
-  test("displays test strategy", () => {
-    const story: UserStory = {
-      id: "US-001",
-      title: "Test",
-      description: "Test",
-      acceptanceCriteria: [],
-      dependencies: [],
-      tags: [],
-      passes: false,
-      status: "pending",
-      escalations: [],
-      attempts: 0,
-    };
-
-    const { lastFrame } = render(
-      createElement(StatusBar, {
-        currentStory: story,
-        testStrategy: "three-session-tdd",
-      }),
-    );
-
-    expect(lastFrame()).toContain("three-session-tdd");
-  });
-});
-
-describe("Layout breakpoints", () => {
-  test("single column mode for width < 80", () => {
-    // This would be tested via useLayout hook, but since we can't mock process.stdout.columns
-    // in Bun tests easily, we verify the logic manually:
-    const width = 70;
-    const mode = width < 80 ? "single" : width < 140 ? "narrow" : "wide";
-    expect(mode).toBe("single");
-  });
-
-  test("narrow mode for width 80-140", () => {
-    const width = 100;
-    const mode = width < 80 ? "single" : width < 140 ? "narrow" : "wide";
-    expect(mode).toBe("narrow");
-  });
-
-  test("wide mode for width > 140", () => {
-    const width = 150;
-    const mode = width < 80 ? "single" : width < 140 ? "narrow" : "wide";
-    expect(mode).toBe("wide");
-  });
-});

package/test/unit/unit-isolation.test.ts

@@ -1,135 +0,0 @@
-// RE-ARCH: keep
-/**
- * Tests for src/tdd/isolation.ts
- *
- * Covers: isTestFile, isSourceFile, matchesAllowedPath (via verify functions)
- */
-
-import { describe, expect, it } from "bun:test";
-import { isSourceFile, isTestFile } from "../../src/tdd/isolation";
-
-// ─────────────────────────────────────────────────────────────────────────────
-// isTestFile
-// ─────────────────────────────────────────────────────────────────────────────
-
-describe("isTestFile", () => {
-  it("detects test/ directory files", () => {
-    expect(isTestFile("test/unit/foo.ts")).toBe(true);
-    expect(isTestFile("test/integration/bar.test.ts")).toBe(true);
-    expect(isTestFile("test/fixtures/data.json")).toBe(true);
-  });
-
-  it("detects tests/ directory files", () => {
-    expect(isTestFile("tests/unit/foo.ts")).toBe(true);
-    expect(isTestFile("tests/integration/bar.test.ts")).toBe(true);
-  });
-
-  it("detects __tests__/ directory files", () => {
-    expect(isTestFile("__tests__/unit/foo.ts")).toBe(true);
-    expect(isTestFile("src/__tests__/foo.test.ts")).toBe(true);
-  });
-
-  it("detects .spec extension", () => {
-    expect(isTestFile("src/foo.spec.ts")).toBe(true);
-    expect(isTestFile("src/utils/bar.spec.js")).toBe(true);
-    expect(isTestFile("lib/baz.spec.tsx")).toBe(true);
-  });
-
-  it("detects .test extension", () => {
-    expect(isTestFile("src/foo.test.ts")).toBe(true);
-    expect(isTestFile("src/utils/bar.test.js")).toBe(true);
-    expect(isTestFile("lib/baz.test.tsx")).toBe(true);
-  });
-
-  it("detects .e2e-spec extension (NestJS convention)", () => {
-    expect(isTestFile("src/app.e2e-spec.ts")).toBe(true);
-    expect(isTestFile("test/e2e/auth.e2e-spec.ts")).toBe(true);
-  });
-
-  it("returns false for source files", () => {
-    expect(isTestFile("src/index.ts")).toBe(false);
-    expect(isTestFile("lib/utils.ts")).toBe(false);
-    expect(isTestFile("packages/core/foo.ts")).toBe(false);
-  });
-
-  it("returns false for config/doc files", () => {
-    expect(isTestFile("package.json")).toBe(false);
-    expect(isTestFile("README.md")).toBe(false);
-    expect(isTestFile("tsconfig.json")).toBe(false);
-  });
-});
-
-// ─────────────────────────────────────────────────────────────────────────────
-// isSourceFile
-// ─────────────────────────────────────────────────────────────────────────────
-
-describe("isSourceFile", () => {
-  it("detects src/ directory files", () => {
-    expect(isSourceFile("src/index.ts")).toBe(true);
-    expect(isSourceFile("src/utils/foo.ts")).toBe(true);
-    expect(isSourceFile("src/components/Bar.tsx")).toBe(true);
-  });
-
-  it("detects lib/ directory files", () => {
-    expect(isSourceFile("lib/index.ts")).toBe(true);
-    expect(isSourceFile("lib/utils/foo.ts")).toBe(true);
-  });
-
-  it("detects packages/ directory files (monorepo)", () => {
-    expect(isSourceFile("packages/core/index.ts")).toBe(true);
-    expect(isSourceFile("packages/utils/foo.ts")).toBe(true);
-  });
-
-  it("returns false for test files", () => {
-    expect(isSourceFile("test/unit/foo.test.ts")).toBe(false);
-    expect(isSourceFile("tests/integration/bar.test.ts")).toBe(false);
-    expect(isSourceFile("__tests__/baz.ts")).toBe(false);
-  });
-
-  it("returns false for config/doc files", () => {
-    expect(isSourceFile("package.json")).toBe(false);
-    expect(isSourceFile("README.md")).toBe(false);
-    expect(isSourceFile("tsconfig.json")).toBe(false);
-  });
-
-  it("returns false for root files", () => {
-    expect(isSourceFile("index.ts")).toBe(false);
-    expect(isSourceFile("utils.ts")).toBe(false);
-  });
-});
-
-// ─────────────────────────────────────────────────────────────────────────────
-// Combined behavior tests
-// ─────────────────────────────────────────────────────────────────────────────
-
-describe("combined isTestFile + isSourceFile", () => {
-  it("test files in src/ are detected as both test and source", () => {
-    const file = "src/foo.test.ts";
-    expect(isTestFile(file)).toBe(true);
-    expect(isSourceFile(file)).toBe(true);
-  });
-
-  it("spec files in src/ are detected as both test and source", () => {
-    const file = "src/foo.spec.ts";
-    expect(isTestFile(file)).toBe(true);
-    expect(isSourceFile(file)).toBe(true);
-  });
-
-  it("regular src/ files are source but not test", () => {
-    const file = "src/foo.ts";
-    expect(isTestFile(file)).toBe(false);
-    expect(isSourceFile(file)).toBe(true);
-  });
-
-  it("test/ directory files are test but not source (default)", () => {
-    const file = "test/foo.ts";
-    expect(isTestFile(file)).toBe(true);
-    expect(isSourceFile(file)).toBe(false);
-  });
-
-  it("config files are neither test nor source", () => {
-    const file = "package.json";
-    expect(isTestFile(file)).toBe(false);
-    expect(isSourceFile(file)).toBe(false);
-  });
-});

package/test/unit/utils/git.test.ts

@@ -1,50 +0,0 @@
-/**
- * Unit tests for git utility functions (TC-003)
- *
- * Covers: detectMergeConflict helper
- */
-
-import { describe, expect, test } from "bun:test";
-import { detectMergeConflict } from "../../../src/utils/git";
-
-describe("detectMergeConflict", () => {
-  test("returns true when output contains uppercase CONFLICT", () => {
-    expect(detectMergeConflict("CONFLICT (content): Merge conflict in src/foo.ts")).toBe(true);
-  });
-
-  test("returns true when output contains lowercase conflict", () => {
-    expect(detectMergeConflict("Auto-merging failed due to conflict in file")).toBe(true);
-  });
-
-  test("returns true for typical git merge CONFLICT output", () => {
-    const output = [
-      "Auto-merging src/index.ts",
-      "CONFLICT (content): Merge conflict in src/index.ts",
-      "Automatic merge failed; fix conflicts and then commit the result.",
-    ].join("\n");
-    expect(detectMergeConflict(output)).toBe(true);
-  });
-
-  test("returns true for git rebase CONFLICT output", () => {
-    const output = "CONFLICT (modify/delete): src/bar.ts deleted in HEAD";
-    expect(detectMergeConflict(output)).toBe(true);
-  });
-
-  test("returns false when output has no conflict markers", () => {
-    expect(detectMergeConflict("All changes committed successfully.")).toBe(false);
-  });
-
-  test("returns false for empty string", () => {
-    expect(detectMergeConflict("")).toBe(false);
-  });
-
-  test("returns false for unrelated git output", () => {
-    const output = "3 files changed, 10 insertions(+), 2 deletions(-)";
-    expect(detectMergeConflict(output)).toBe(false);
-  });
-
-  test("returns true when CONFLICT appears in stderr portion of combined output", () => {
-    const combined = "stdout: commit abc123\nstderr: CONFLICT detected in merge";
-    expect(detectMergeConflict(combined)).toBe(true);
-  });
-});

package/test/unit/utils/path-security.test.ts

@@ -1,47 +0,0 @@
-import { describe, expect, test } from "bun:test";
-import { validateModulePath } from "../../../src/utils/path-security";
-import { resolve } from "node:path";
-
-describe("path-security utility", () => {
-  const projectRoot = "/home/project";
-  const globalRoot = "/home/global";
-  const roots = [projectRoot, globalRoot];
-
-  test("allows relative path within project root", () => {
-    // Relative paths are resolved relative to the first allowed root by our validator
-    const result = validateModulePath("./plugins/my-plugin.ts", roots);
-    expect(result.valid).toBe(true);
-    expect(result.absolutePath).toBe(resolve(projectRoot, "plugins/my-plugin.ts"));
-  });
-
-  test("allows absolute path within global root", () => {
-    const result = validateModulePath("/home/global/plugins/my-plugin.ts", roots);
-    expect(result.valid).toBe(true);
-    expect(result.absolutePath).toBe("/home/global/plugins/my-plugin.ts");
-  });
-
-  test("blocks traversal out of root (../)", () => {
-    // resolve handles the ../ then our startsWith check fails
-    const result = validateModulePath("../../etc/passwd", roots);
-    expect(result.valid).toBe(false);
-    expect(result.error).toContain("outside allowed roots");
-  });
-
-  test("blocks absolute path outside roots", () => {
-    const result = validateModulePath("/usr/bin/node", roots);
-    expect(result.valid).toBe(false);
-    expect(result.error).toContain("outside allowed roots");
-  });
-
-  test("handles root itself", () => {
-    const result = validateModulePath("/home/project", roots);
-    expect(result.valid).toBe(true);
-    expect(result.absolutePath).toBe("/home/project");
-  });
-
-  test("blocks empty path", () => {
-    const result = validateModulePath("", roots);
-    expect(result.valid).toBe(false);
-    expect(result.error).toContain("empty");
-  });
-});