@nathapp/nax 0.28.0 → 0.29.0

package/test/unit/test-output-parser.test.ts

@@ -1,377 +0,0 @@
-import { describe, expect, test } from "bun:test";
-import { type TestFailure, formatFailureSummary, parseBunTestOutput } from "../../src/execution/test-output-parser";
-
-describe("parseBunTestOutput", () => {
-  test("parses passing output (0 failures)", () => {
-    const output = `
-bun test v1.0.0
-
-test/example.test.ts:
-✓ test 1 [0.5ms]
-✓ test 2 [0.3ms]
-✓ test 3 [0.7ms]
-
-3 tests passed [1.5ms]
-    `.trim();
-
-    const result = parseBunTestOutput(output);
-
-    expect(result.passed).toBe(3);
-    expect(result.failed).toBe(0);
-    expect(result.failures).toHaveLength(0);
-  });
-
-  test("parses mixed pass/fail output", () => {
-    const output = `
-bun test v1.0.0
-
-test/example.test.ts:
-✓ passing test [0.5ms]
-✗ failing test [1.2ms]
-
-(fail) failing test [1.2ms]
-Error: Expected 1 to equal 2
-  at /path/to/file.ts:10:15
-  at Object.test (/path/to/file.ts:8:3)
-
-1 passed, 1 failed [1.7ms]
-    `.trim();
-
-    const result = parseBunTestOutput(output);
-
-    expect(result.passed).toBe(1);
-    expect(result.failed).toBe(1);
-    expect(result.failures).toHaveLength(1);
-    expect(result.failures[0].file).toBe("test/example.test.ts");
-    expect(result.failures[0].testName).toBe("failing test");
-    expect(result.failures[0].error).toBe("Error: Expected 1 to equal 2");
-    expect(result.failures[0].stackTrace).toHaveLength(2);
-  });
-
-  test("extracts test names from nested describe blocks", () => {
-    const output = `
-bun test v1.0.0
-
-test/nested.test.ts:
-✓ outer test [0.2ms]
-✗ inner test [0.8ms]
-
-(fail) describe block > nested block > inner test [0.8ms]
-Error: Assertion failed
-  at /path/to/nested.ts:20:10
-
-1 passed, 1 failed [1.0ms]
-    `.trim();
-
-    const result = parseBunTestOutput(output);
-
-    expect(result.failures).toHaveLength(1);
-    expect(result.failures[0].testName).toBe("describe block > nested block > inner test");
-    expect(result.failures[0].file).toBe("test/nested.test.ts");
-  });
-
-  test("truncates stack trace to 5 lines", () => {
-    const output = `
-bun test v1.0.0
-
-test/stack.test.ts:
-✗ test with long stack [2.0ms]
-
-(fail) test with long stack [2.0ms]
-Error: Stack overflow
-  at line1 (/path/to/file.ts:1:1)
-  at line2 (/path/to/file.ts:2:2)
-  at line3 (/path/to/file.ts:3:3)
-  at line4 (/path/to/file.ts:4:4)
-  at line5 (/path/to/file.ts:5:5)
-  at line6 (/path/to/file.ts:6:6)
-  at line7 (/path/to/file.ts:7:7)
-  at line8 (/path/to/file.ts:8:8)
-
-0 passed, 1 failed [2.0ms]
-    `.trim();
-
-    const result = parseBunTestOutput(output);
-
-    expect(result.failures).toHaveLength(1);
-    expect(result.failures[0].stackTrace).toHaveLength(5);
-    expect(result.failures[0].stackTrace[0]).toBe("at line1 (/path/to/file.ts:1:1)");
-    expect(result.failures[0].stackTrace[4]).toBe("at line5 (/path/to/file.ts:5:5)");
-  });
-
-  test("handles empty/malformed input", () => {
-    const emptyResult = parseBunTestOutput("");
-    expect(emptyResult.passed).toBe(0);
-    expect(emptyResult.failed).toBe(0);
-    expect(emptyResult.failures).toHaveLength(0);
-
-    const malformedResult = parseBunTestOutput("random text\nno test output");
-    expect(malformedResult.passed).toBe(0);
-    expect(malformedResult.failed).toBe(0);
-    expect(malformedResult.failures).toHaveLength(0);
-  });
-
-  test("handles multiple test files", () => {
-    const output = `
-bun test v1.0.0
-
-test/file1.test.ts:
-✓ test 1 [0.5ms]
-✗ test 2 [1.2ms]
-
-(fail) test 2 [1.2ms]
-Error: File 1 error
-  at /path/to/file1.ts:10:15
-
-test/file2.test.ts:
-✓ test 3 [0.3ms]
-✗ test 4 [0.8ms]
-
-(fail) test 4 [0.8ms]
-Error: File 2 error
-  at /path/to/file2.ts:20:25
-
-2 passed, 2 failed [2.8ms]
-    `.trim();
-
-    const result = parseBunTestOutput(output);
-
-    expect(result.passed).toBe(2);
-    expect(result.failed).toBe(2);
-    expect(result.failures).toHaveLength(2);
-    expect(result.failures[0].file).toBe("test/file1.test.ts");
-    expect(result.failures[0].error).toBe("Error: File 1 error");
-    expect(result.failures[1].file).toBe("test/file2.test.ts");
-    expect(result.failures[1].error).toBe("Error: File 2 error");
-  });
-
-  test("handles test files with .js extension", () => {
-    const output = `
-bun test v1.0.0
-
-test/example.test.js:
-✗ failing test [1.0ms]
-
-(fail) failing test [1.0ms]
-Error: JS test error
-  at /path/to/file.js:5:10
-
-0 passed, 1 failed [1.0ms]
-    `.trim();
-
-    const result = parseBunTestOutput(output);
-
-    expect(result.failures).toHaveLength(1);
-    expect(result.failures[0].file).toBe("test/example.test.js");
-  });
-
-  test("handles failures without file context", () => {
-    const output = `
-bun test v1.0.0
-
-✗ orphan test [1.0ms]
-
-(fail) orphan test [1.0ms]
-Error: No file context
-  at /path/to/unknown.ts:1:1
-
-0 passed, 1 failed [1.0ms]
-    `.trim();
-
-    const result = parseBunTestOutput(output);
-
-    expect(result.failures).toHaveLength(1);
-    expect(result.failures[0].file).toBe("unknown");
-  });
-
-  test("handles failures with no error message", () => {
-    const output = `
-bun test v1.0.0
-
-test/minimal.test.ts:
-✗ minimal fail [0.5ms]
-
-(fail) minimal fail [0.5ms]
-
-0 passed, 1 failed [0.5ms]
-    `.trim();
-
-    const result = parseBunTestOutput(output);
-
-    expect(result.failures).toHaveLength(1);
-    expect(result.failures[0].error).toBe("Unknown error");
-    expect(result.failures[0].stackTrace).toHaveLength(0);
-  });
-
-  test("handles alternative check marks (✔ and ✘)", () => {
-    const output = `
-bun test v1.0.0
-
-test/marks.test.ts:
-✔ pass with heavy check [0.2ms]
-✘ fail with heavy X [0.5ms]
-
-(fail) fail with heavy X [0.5ms]
-Error: Alternative marks error
-
-1 passed, 1 failed [0.7ms]
-    `.trim();
-
-    const result = parseBunTestOutput(output);
-
-    expect(result.passed).toBe(1);
-    expect(result.failed).toBe(1);
-    expect(result.failures).toHaveLength(1);
-  });
-});
-
-describe("formatFailureSummary", () => {
-  test("returns 'No test failures' for empty array", () => {
-    const result = formatFailureSummary([]);
-    expect(result).toBe("No test failures");
-  });
-
-  test("formats single failure", () => {
-    const failures: TestFailure[] = [
-      {
-        file: "test/example.test.ts",
-        testName: "failing test",
-        error: "Expected 1 to equal 2",
-        stackTrace: ["at /path/to/file.ts:10:15"],
-      },
-    ];
-
-    const result = formatFailureSummary(failures);
-
-    expect(result).toContain("1. test/example.test.ts > failing test");
-    expect(result).toContain("Error: Expected 1 to equal 2");
-    expect(result).toContain("at /path/to/file.ts:10:15");
-  });
-
-  test("formats multiple failures", () => {
-    const failures: TestFailure[] = [
-      {
-        file: "test/file1.test.ts",
-        testName: "test 1",
-        error: "Error 1",
-        stackTrace: ["at /path/file1.ts:5:10"],
-      },
-      {
-        file: "test/file2.test.ts",
-        testName: "test 2",
-        error: "Error 2",
-        stackTrace: ["at /path/file2.ts:15:20"],
-      },
-    ];
-
-    const result = formatFailureSummary(failures);
-
-    expect(result).toContain("1. test/file1.test.ts > test 1");
-    expect(result).toContain("Error: Error 1");
-    expect(result).toContain("2. test/file2.test.ts > test 2");
-    expect(result).toContain("Error: Error 2");
-  });
-
-  test("respects maxChars limit", () => {
-    const failures: TestFailure[] = Array.from({ length: 10 }, (_, i) => ({
-      file: `test/file${i}.test.ts`,
-      testName: `test ${i}`,
-      error: `Error message ${i}`,
-      stackTrace: [`at /path/file${i}.ts:${i}:${i}`],
-    }));
-
-    const result = formatFailureSummary(failures, 300);
-
-    expect(result.length).toBeLessThanOrEqual(350); // Some buffer for truncation message
-    expect(result).toContain("... and");
-    expect(result).toContain("more failure(s) (truncated)");
-  });
-
-  test("handles failure without stack trace", () => {
-    const failures: TestFailure[] = [
-      {
-        file: "test/nostack.test.ts",
-        testName: "no stack",
-        error: "Error without stack",
-        stackTrace: [],
-      },
-    ];
-
-    const result = formatFailureSummary(failures);
-
-    expect(result).toContain("1. test/nostack.test.ts > no stack");
-    expect(result).toContain("Error: Error without stack");
-    expect(result).not.toContain("at ");
-  });
-
-  test("handles nested test names", () => {
-    const failures: TestFailure[] = [
-      {
-        file: "test/nested.test.ts",
-        testName: "outer > middle > inner",
-        error: "Nested test error",
-        stackTrace: ["at /path/nested.ts:30:5"],
-      },
-    ];
-
-    const result = formatFailureSummary(failures);
-
-    expect(result).toContain("1. test/nested.test.ts > outer > middle > inner");
-    expect(result).toContain("Error: Nested test error");
-  });
-
-  test("includes only first stack trace line", () => {
-    const failures: TestFailure[] = [
-      {
-        file: "test/multi.test.ts",
-        testName: "multi stack",
-        error: "Error with multiple stack lines",
-        stackTrace: ["at /path/file.ts:10:5", "at /path/file.ts:20:10", "at /path/file.ts:30:15"],
-      },
-    ];
-
-    const result = formatFailureSummary(failures);
-
-    expect(result).toContain("at /path/file.ts:10:5");
-    expect(result).not.toContain("at /path/file.ts:20:10");
-    expect(result).not.toContain("at /path/file.ts:30:15");
-  });
-
-  test("separates failures with blank lines", () => {
-    const failures: TestFailure[] = [
-      {
-        file: "test/a.test.ts",
-        testName: "test a",
-        error: "Error A",
-        stackTrace: ["at a.ts:1:1"],
-      },
-      {
-        file: "test/b.test.ts",
-        testName: "test b",
-        error: "Error B",
-        stackTrace: ["at b.ts:2:2"],
-      },
-    ];
-
-    const result = formatFailureSummary(failures);
-
-    // Check that there's proper separation between failures
-    const lines = result.split("\n");
-    const blankLines = lines.filter((line) => line.trim() === "");
-    expect(blankLines.length).toBeGreaterThan(0);
-  });
-
-  test("uses default maxChars of 2000", () => {
-    const failures: TestFailure[] = Array.from({ length: 50 }, (_, i) => ({
-      file: `test/file${i}.test.ts`,
-      testName: `test ${i}`,
-      error: `Error message ${i}`,
-      stackTrace: [`at /path/file${i}.ts:${i}:${i}`],
-    }));
-
-    const result = formatFailureSummary(failures); // No maxChars argument
-
-    expect(result.length).toBeLessThanOrEqual(2100); // Some buffer
-    expect(result).toContain("(truncated)");
-  });
-});

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

@@ -1,335 +0,0 @@
-// RE-ARCH: keep
-/**
- * TUI Controls Tests
- *
- * Tests keyboard shortcuts, overlays, focus mode switching,
- * and queue command writing.
- */
-
-import { afterEach, beforeEach, describe, expect, test } from "bun:test";
-import { unlink } from "node:fs/promises";
-import { render } from "ink-testing-library";
-import { createElement } from "react";
-import type { UserStory } from "../../../src/prd/types";
-import { CostOverlay } from "../../../src/tui/components/CostOverlay";
-import { HelpOverlay } from "../../../src/tui/components/HelpOverlay";
-import type { KeyboardAction } from "../../../src/tui/hooks/useKeyboard";
-import { PanelFocus } from "../../../src/tui/types";
-import type { StoryDisplayState } from "../../../src/tui/types";
-import { writeQueueCommand } from "../../../src/utils/queue-writer";
-
-// Helper to create mock stories
-function createMockStory(id: string, status: StoryDisplayState["status"], cost = 0.01): 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,
-  };
-}
-
-describe("HelpOverlay", () => {
-  test("does not render when visible=false", () => {
-    const { lastFrame } = render(createElement(HelpOverlay, { visible: false }));
-    expect(lastFrame()).toBe("");
-  });
-
-  test("renders keybindings when visible=true", () => {
-    const { lastFrame } = render(createElement(HelpOverlay, { visible: true }));
-    const output = lastFrame();
-
-    expect(output).toContain("Keyboard Shortcuts");
-    expect(output).toContain("p"); // Pause
-    expect(output).toContain("a"); // Abort
-    expect(output).toContain("s"); // Skip
-    expect(output).toContain("Tab"); // Toggle focus
-    expect(output).toContain("q"); // Quit
-    expect(output).toContain("?"); // Show help
-    expect(output).toContain("c"); // Show cost
-    expect(output).toContain("r"); // Retry
-    expect(output).toContain("Esc"); // Close overlay
-    expect(output).toContain("Ctrl+]"); // Escape agent panel
-  });
-
-  test("displays Stories panel keybindings", () => {
-    const { lastFrame } = render(createElement(HelpOverlay, { visible: true }));
-    const output = lastFrame();
-
-    expect(output).toContain("Stories Panel");
-    expect(output).toContain("Pause after current story");
-    expect(output).toContain("Abort run");
-    expect(output).toContain("Skip current story");
-  });
-
-  test("displays Agent panel keybindings", () => {
-    const { lastFrame } = render(createElement(HelpOverlay, { visible: true }));
-    const output = lastFrame();
-
-    expect(output).toContain("Agent Panel");
-    expect(output).toContain("Escape back to Stories panel");
-    expect(output).toContain("Forwarded to agent PTY");
-  });
-});
-
-describe("CostOverlay", () => {
-  test("does not render when visible=false", () => {
-    const { lastFrame } = render(createElement(CostOverlay, { visible: false }));
-    expect(lastFrame()).toBe("");
-  });
-
-  test("renders cost breakdown when visible=true", () => {
-    const stories = [
-      createMockStory("US-001", "passed", 0.023),
-      createMockStory("US-002", "running", 0.015),
-      createMockStory("US-003", "pending", 0),
-    ];
-
-    const { lastFrame } = render(
-      createElement(CostOverlay, {
-        visible: true,
-        stories,
-        totalCost: 0.038,
-      }),
-    );
-
-    const output = lastFrame();
-    expect(output).toContain("Cost Breakdown");
-    expect(output).toContain("Story ID");
-    expect(output).toContain("Status");
-    expect(output).toContain("Cost");
-  });
-
-  test("displays executed stories with costs", () => {
-    const stories = [createMockStory("US-001", "passed", 0.023), createMockStory("US-002", "failed", 0.015)];
-
-    const { lastFrame } = render(
-      createElement(CostOverlay, {
-        visible: true,
-        stories,
-        totalCost: 0.038,
-      }),
-    );
-
-    const output = lastFrame();
-    expect(output).toContain("US-001");
-    expect(output).toContain("passed");
-    expect(output).toContain("$0.0230");
-    expect(output).toContain("US-002");
-    expect(output).toContain("failed");
-    expect(output).toContain("$0.0150");
-  });
-
-  test("displays total cost", () => {
-    const stories = [createMockStory("US-001", "passed", 0.023)];
-
-    const { lastFrame } = render(
-      createElement(CostOverlay, {
-        visible: true,
-        stories,
-        totalCost: 0.123456,
-      }),
-    );
-
-    const output = lastFrame();
-    expect(output).toContain("Total Cost:");
-    expect(output).toContain("$0.1235");
-  });
-
-  test("shows message when no stories executed", () => {
-    const stories = [createMockStory("US-001", "pending", 0)];
-
-    const { lastFrame } = render(
-      createElement(CostOverlay, {
-        visible: true,
-        stories,
-        totalCost: 0,
-      }),
-    );
-
-    const output = lastFrame();
-    expect(output).toContain("No stories executed yet");
-  });
-});
-
-describe("Keyboard action types", () => {
-  test("PAUSE action has correct type", () => {
-    const action: KeyboardAction = { type: "PAUSE" };
-    expect(action.type).toBe("PAUSE");
-  });
-
-  test("ABORT action has correct type", () => {
-    const action: KeyboardAction = { type: "ABORT" };
-    expect(action.type).toBe("ABORT");
-  });
-
-  test("SKIP action has story ID", () => {
-    const action: KeyboardAction = { type: "SKIP", storyId: "US-042" };
-    expect(action.type).toBe("SKIP");
-    expect(action.storyId).toBe("US-042");
-  });
-
-  test("TOGGLE_FOCUS action has correct type", () => {
-    const action: KeyboardAction = { type: "TOGGLE_FOCUS" };
-    expect(action.type).toBe("TOGGLE_FOCUS");
-  });
-
-  test("ESCAPE_AGENT action has correct type", () => {
-    const action: KeyboardAction = { type: "ESCAPE_AGENT" };
-    expect(action.type).toBe("ESCAPE_AGENT");
-  });
-
-  test("QUIT action has correct type", () => {
-    const action: KeyboardAction = { type: "QUIT" };
-    expect(action.type).toBe("QUIT");
-  });
-
-  test("SHOW_HELP action has correct type", () => {
-    const action: KeyboardAction = { type: "SHOW_HELP" };
-    expect(action.type).toBe("SHOW_HELP");
-  });
-
-  test("SHOW_COST action has correct type", () => {
-    const action: KeyboardAction = { type: "SHOW_COST" };
-    expect(action.type).toBe("SHOW_COST");
-  });
-
-  test("RETRY action has correct type", () => {
-    const action: KeyboardAction = { type: "RETRY" };
-    expect(action.type).toBe("RETRY");
-  });
-
-  test("CLOSE_OVERLAY action has correct type", () => {
-    const action: KeyboardAction = { type: "CLOSE_OVERLAY" };
-    expect(action.type).toBe("CLOSE_OVERLAY");
-  });
-});
-
-describe("Focus mode", () => {
-  test("PanelFocus enum has Stories value", () => {
-    expect(PanelFocus.Stories).toBe("stories");
-  });
-
-  test("PanelFocus enum has Agent value", () => {
-    expect(PanelFocus.Agent).toBe("agent");
-  });
-
-  test("focus mode toggles between Stories and Agent", () => {
-    let focus: PanelFocus = PanelFocus.Stories;
-    focus = focus === PanelFocus.Stories ? PanelFocus.Agent : PanelFocus.Stories;
-    expect(focus).toBe(PanelFocus.Agent);
-
-    focus = focus === PanelFocus.Stories ? PanelFocus.Agent : PanelFocus.Stories;
-    expect(focus).toBe(PanelFocus.Stories);
-  });
-});
-
-describe("Queue command writer", () => {
-  const tempQueueFile = "/tmp/nax-test-queue.txt";
-
-  beforeEach(async () => {
-    // Clean up any existing test file
-    try {
-      await unlink(tempQueueFile);
-    } catch {
-      // Ignore if doesn't exist
-    }
-  });
-
-  afterEach(async () => {
-    // Clean up after tests
-    try {
-      await unlink(tempQueueFile);
-    } catch {
-      // Ignore if doesn't exist
-    }
-  });
-
-  test("writes PAUSE command to queue file", async () => {
-    await writeQueueCommand(tempQueueFile, { type: "PAUSE" });
-
-    const content = await Bun.file(tempQueueFile).text();
-    expect(content.trim()).toBe("PAUSE");
-  });
-
-  test("writes ABORT command to queue file", async () => {
-    await writeQueueCommand(tempQueueFile, { type: "ABORT" });
-
-    const content = await Bun.file(tempQueueFile).text();
-    expect(content.trim()).toBe("ABORT");
-  });
-
-  test("writes SKIP command with story ID to queue file", async () => {
-    await writeQueueCommand(tempQueueFile, { type: "SKIP", storyId: "US-042" });
-
-    const content = await Bun.file(tempQueueFile).text();
-    expect(content.trim()).toBe("SKIP US-042");
-  });
-
-  test("appends multiple commands to queue file", async () => {
-    await writeQueueCommand(tempQueueFile, { type: "SKIP", storyId: "US-001" });
-    await writeQueueCommand(tempQueueFile, { type: "PAUSE" });
-    await writeQueueCommand(tempQueueFile, { type: "ABORT" });
-
-    const content = await Bun.file(tempQueueFile).text();
-    const lines = content.trim().split("\n");
-
-    expect(lines).toEqual(["SKIP US-001", "PAUSE", "ABORT"]);
-  });
-
-  test("creates queue file if it doesn't exist", async () => {
-    // Ensure file doesn't exist
-    try {
-      await unlink(tempQueueFile);
-    } catch {
-      // Ignore
-    }
-
-    await writeQueueCommand(tempQueueFile, { type: "PAUSE" });
-
-    const file = Bun.file(tempQueueFile);
-    expect(await file.exists()).toBe(true);
-
-    const content = await file.text();
-    expect(content.trim()).toBe("PAUSE");
-  });
-});
-
-describe("Ctrl+] escape sequence", () => {
-  test("Ctrl+] should escape from agent panel", () => {
-    // This would be tested in integration with the useKeyboard hook
-    // For now, we verify the concept:
-    const input = "]";
-    const keyCtrl = true;
-
-    if (keyCtrl && input === "]") {
-      const action: KeyboardAction = { type: "ESCAPE_AGENT" };
-      expect(action.type).toBe("ESCAPE_AGENT");
-    }
-  });
-
-  test("regular ] without Ctrl should not escape", () => {
-    const input = "]";
-    const keyCtrl = false;
-
-    // Should not trigger escape
-    expect(keyCtrl && input === "]").toBe(false);
-  });
-});