@golba98/codexa 1.0.2 → 1.0.3

package/src/ui/TurnGroup.test.tsx

@@ -1,365 +0,0 @@
-import assert from "node:assert/strict";
-import test from "node:test";
-import React from "react";
-import { PassThrough } from "node:stream";
-import { render } from "ink";
-import type { AssistantEvent, RunEvent, UIState, UserPromptEvent } from "../session/types.js";
-import { TEST_RUNTIME } from "../test/runtimeTestUtils.js";
-import { ThemeProvider } from "./theme.js";
-import { TurnGroup, resolveTurnRunPhase } from "./TurnGroup.js";
-
-class TestInput extends PassThrough {
-  readonly isTTY = true;
-
-  setRawMode(): this {
-    return this;
-  }
-
-  override resume(): this {
-    return this;
-  }
-
-  override pause(): this {
-    return this;
-  }
-
-  ref(): this {
-    return this;
-  }
-
-  unref(): this {
-    return this;
-  }
-}
-
-class TestOutput extends PassThrough {
-  readonly isTTY = true;
-  columns = 120;
-  rows = 40;
-}
-
-function stripAnsi(value: string): string {
-  return value.replace(/\u001B\[[0-?]*[ -/]*[@-~]/g, "");
-}
-
-function sleep(ms = 50): Promise<void> {
-  return new Promise((resolve) => setTimeout(resolve, ms));
-}
-
-function makeRunningRun(turnId: number): RunEvent {
-  return {
-    id: 2,
-    type: "run",
-    createdAt: 2,
-    startedAt: 2,
-    durationMs: null,
-    backendId: "codex-subprocess",
-    backendLabel: "Codexa",
-    runtime: TEST_RUNTIME,
-    prompt: "Do work",
-    progressEntries: [],
-    status: "running",
-    summary: "Running",
-    truncatedOutput: false,
-    toolActivities: [],
-    activity: [],
-    touchedFileCount: 0,
-    errorMessage: null,
-    turnId,
-  };
-}
-
-function makeUser(turnId: number): UserPromptEvent {
-  return {
-    id: 1,
-    type: "user",
-    createdAt: 1,
-    prompt: "Do work",
-    turnId,
-  };
-}
-
-function makeAssistant(turnId: number, content: string): AssistantEvent {
-  return {
-    id: 3,
-    type: "assistant",
-    createdAt: 3,
-    content,
-    contentChunks: [],
-    turnId,
-  };
-}
-
-function renderTurnGroup(node: React.ReactElement) {
-  const stdin = new TestInput();
-  const stdout = new TestOutput();
-  let output = "";
-  stdout.on("data", (chunk) => {
-    output += chunk.toString();
-  });
-  const instance = render(node, {
-    stdin: stdin as unknown as NodeJS.ReadStream,
-    stdout: stdout as unknown as NodeJS.WriteStream,
-    stderr: stdout as unknown as NodeJS.WriteStream,
-    debug: true,
-    exitOnCtrlC: false,
-  });
-
-  return {
-    instance,
-    readOutput: () => stripAnsi(output),
-    resetOutput: () => {
-      output = "";
-    },
-  };
-}
-
-test("does not render duplicate transcript working placeholder before streaming appears", async () => {
-  const turnId = 10;
-  const user = makeUser(turnId);
-  const run = makeRunningRun(turnId);
-  const assistant = makeAssistant(turnId, "Streaming line");
-
-  const harness = renderTurnGroup(
-    <ThemeProvider theme="purple">
-      <TurnGroup
-        cols={120}
-        turnIndex={1}
-        user={user}
-        run={run}
-        assistant={null}
-        opacity="active"
-        question={null}
-        runPhase="thinking"
-        streamPreviewRows={8}
-        streamMode="assistant-first"
-      />
-    </ThemeProvider>,
-  );
-
-  await sleep();
-  let frame = harness.readOutput();
-  assert.doesNotMatch(frame, /Codex is working/i);
-
-  harness.resetOutput();
-  harness.instance.rerender(
-    <ThemeProvider theme="purple">
-      <TurnGroup
-        cols={120}
-        turnIndex={1}
-        user={user}
-        run={run}
-        assistant={assistant}
-        opacity="active"
-        question={null}
-        runPhase="streaming"
-        streamPreviewRows={8}
-        streamMode="assistant-first"
-      />
-    </ThemeProvider>,
-  );
-
-  await sleep();
-  frame = harness.readOutput();
-  assert.match(frame, /Codexa/);
-  assert.match(frame, /Streaming line/i);
-
-  harness.instance.unmount();
-});
-
-test("renders progressive content updates during streaming", async () => {
-  const turnId = 12;
-  const user = makeUser(turnId);
-  const run = makeRunningRun(turnId);
-
-  const harness = renderTurnGroup(
-    <ThemeProvider theme="purple">
-      <TurnGroup
-        cols={120}
-        turnIndex={1}
-        user={user}
-        run={run}
-        assistant={makeAssistant(turnId, "First chunk")}
-        opacity="active"
-        question={null}
-        runPhase="streaming"
-        streamPreviewRows={8}
-        streamMode="assistant-first"
-      />
-    </ThemeProvider>,
-  );
-
-  await sleep();
-  let frame = harness.readOutput();
-  assert.match(frame, /first chunk/i);
-
-  // Update with more content — should render incrementally
-  harness.resetOutput();
-  harness.instance.rerender(
-    <ThemeProvider theme="purple">
-      <TurnGroup
-        cols={120}
-        turnIndex={1}
-        user={user}
-        run={run}
-        assistant={makeAssistant(turnId, "First chunk\nSecond chunk")}
-        opacity="active"
-        question={null}
-        runPhase="streaming"
-        streamPreviewRows={8}
-        streamMode="assistant-first"
-      />
-    </ThemeProvider>,
-  );
-
-  await sleep();
-  frame = harness.readOutput();
-  assert.match(frame, /second chunk/i);
-
-  harness.instance.unmount();
-});
-
-test("finalization with same content does not cause visual flash", async () => {
-  const turnId = 13;
-  const user = makeUser(turnId);
-  const run = makeRunningRun(turnId);
-  const content = "The response content stays the same";
-  const assistant = makeAssistant(turnId, content);
-
-  const harness = renderTurnGroup(
-    <ThemeProvider theme="purple">
-      <TurnGroup
-        cols={120}
-        turnIndex={1}
-        user={user}
-        run={run}
-        assistant={assistant}
-        opacity="active"
-        question={null}
-        runPhase="streaming"
-        streamPreviewRows={8}
-        streamMode="assistant-first"
-      />
-    </ThemeProvider>,
-  );
-
-  await sleep();
-  let streamingFrame = harness.readOutput();
-  assert.match(streamingFrame, /response content stays/i);
-
-  // Transition to final with same content
-  harness.resetOutput();
-  harness.instance.rerender(
-    <ThemeProvider theme="purple">
-      <TurnGroup
-        cols={120}
-        turnIndex={1}
-        user={user}
-        run={{ ...run, status: "completed", durationMs: 800 }}
-        assistant={assistant}
-        opacity="active"
-        question={null}
-        runPhase="final"
-        streamPreviewRows={8}
-        streamMode="assistant-first"
-      />
-    </ThemeProvider>,
-  );
-
-  await sleep();
-  let finalFrame = harness.readOutput();
-  // Content should still be present — no flash/disappearance
-  assert.match(finalFrame, /response content stays/i);
-  assert.match(finalFrame, /Codexa/);
-
-  harness.instance.unmount();
-});
-
-test("assistant output 'Codexa' label is indented by transcriptContentIndent", async () => {
-  const turnId = 100;
-  const user = makeUser(turnId);
-  const run = makeRunningRun(turnId);
-  const assistant = makeAssistant(turnId, "Indented response");
-
-  const harness = renderTurnGroup(
-    <ThemeProvider theme="purple">
-      <TurnGroup
-        cols={120}
-        turnIndex={1}
-        user={user}
-        run={run}
-        assistant={assistant}
-        opacity="active"
-        question={null}
-        runPhase="streaming"
-        streamPreviewRows={8}
-        streamMode="assistant-first"
-      />
-    </ThemeProvider>,
-  );
-
-  await sleep();
-  const frame = harness.readOutput();
-  const lines = frame.split("\n");
-  const codexaLine = lines.find((l) => l.includes("Codexa"));
-  assert.ok(codexaLine, "expected Codexa label");
-  // Check that the line starts with exactly 4 spaces (transcriptContentIndent = 4)
-  assert.match(codexaLine, /^    Codexa/);
-
-  harness.instance.unmount();
-});
-
-
-test("snaps cleanly from streaming cursor view to completion view", async () => {
-  const turnId = 11;
-  const user = makeUser(turnId);
-  const running = makeRunningRun(turnId);
-  const assistant = makeAssistant(turnId, "Final response text");
-
-  const harness = renderTurnGroup(
-    <ThemeProvider theme="purple">
-      <TurnGroup
-        cols={120}
-        turnIndex={2}
-        user={user}
-        run={running}
-        assistant={assistant}
-        opacity="active"
-        question={null}
-        runPhase="streaming"
-        streamPreviewRows={6}
-        streamMode="assistant-first"
-      />
-    </ThemeProvider>,
-  );
-
-  await sleep();
-  let frame = harness.readOutput();
-  assert.match(frame, /final response text/i);
-  assert.match(frame, /▌/);
-
-  harness.resetOutput();
-  harness.instance.rerender(
-    <ThemeProvider theme="purple">
-      <TurnGroup
-        cols={120}
-        turnIndex={2}
-        user={user}
-        run={{ ...running, status: "completed", durationMs: 1200 }}
-        assistant={assistant}
-        opacity="active"
-        question={null}
-        runPhase="final"
-        streamPreviewRows={6}
-        streamMode="assistant-first"
-      />
-    </ThemeProvider>,
-  );
-
-  await sleep();
-  frame = harness.readOutput();
-  assert.match(frame, /final response text/i);
-  assert.doesNotMatch(frame, /▌/);
-
-  harness.instance.unmount();
-});

package/src/ui/busyStatusAnimation.test.ts

@@ -1,30 +0,0 @@
-import assert from "node:assert/strict";
-import test from "node:test";
-import {
-  BUSY_STATUS_FRAME_MS,
-  BUSY_STATUS_FRAMES,
-  getBusyStatusFrame,
-  isAnimatedBusyState,
-} from "./busyStatusAnimation.js";
-
-test("busy status frames advance in a fixed-width dot slot", () => {
-  assert.deepEqual(
-    [0, 1, 2, 3, 4].map(getBusyStatusFrame),
-    [" .  ", " .. ", " ...", " .  ", " .. "],
-  );
-  assert.ok(BUSY_STATUS_FRAMES.every((frame) => frame.length === BUSY_STATUS_FRAMES[0]!.length));
-});
-
-test("busy status animation uses a low redraw cadence", () => {
-  assert.equal(BUSY_STATUS_FRAME_MS, 900);
-});
-
-test("busy animation runs only for active work states", () => {
-  assert.equal(isAnimatedBusyState("THINKING"), true);
-  assert.equal(isAnimatedBusyState("RESPONDING"), true);
-  assert.equal(isAnimatedBusyState("SHELL_RUNNING"), true);
-  assert.equal(isAnimatedBusyState("IDLE"), false);
-  assert.equal(isAnimatedBusyState("ANSWER_VISIBLE"), false);
-  assert.equal(isAnimatedBusyState("ERROR"), false);
-  assert.equal(isAnimatedBusyState("AWAITING_USER_ACTION"), false);
-});

package/src/ui/commandNormalize.test.ts

@@ -1,142 +0,0 @@
-import assert from "node:assert/strict";
-import test from "node:test";
-import { normalizeCommand, getFriendlyActionLabel } from "./commandNormalize.js";
-
-// ─── normalizeCommand ─────────────────────────────────────────────────────────
-
-test("normalizeCommand: strips full PowerShell path with single-quoted command", () => {
-  const raw = `"C:\\Program Files\\PowerShell\\7\\pwsh.exe" -Command 'Get-ChildItem -Force | Select-Object Name,Mode,Length'`;
-  assert.equal(normalizeCommand(raw), "Get-ChildItem -Force | Select-Object Name,Mode,Length");
-});
-
-test("normalizeCommand: strips full PowerShell path with double-quoted command", () => {
-  const raw = `"C:\\Windows\\System32\\WindowsPowerShell\\v1.0\\powershell.exe" -Command "Get-Content package.json"`;
-  assert.equal(normalizeCommand(raw), "Get-Content package.json");
-});
-
-test("normalizeCommand: strips bare pwsh.exe -Command", () => {
-  assert.equal(normalizeCommand(`pwsh.exe -Command 'git status'`), "git status");
-  assert.equal(normalizeCommand(`pwsh -Command "npm install"`), "npm install");
-});
-
-test("normalizeCommand: strips bare powershell.exe -Command", () => {
-  assert.equal(normalizeCommand(`powershell.exe -Command 'dir'`), "dir");
-  assert.equal(normalizeCommand(`powershell -Command "Get-Content readme.md"`), "Get-Content readme.md");
-});
-
-test("normalizeCommand: strips cmd.exe /c wrapper", () => {
-  assert.equal(normalizeCommand(`cmd.exe /c "dir /b"`), "dir /b");
-  assert.equal(normalizeCommand(`cmd /C "npm test"`), "npm test");
-});
-
-test("normalizeCommand: strips bash -lc wrapper", () => {
-  assert.equal(normalizeCommand(`bash -lc 'bun install'`), "bun install");
-  assert.equal(normalizeCommand(`bash -lc "git diff"`), "git diff");
-});
-
-test("normalizeCommand: returns command unchanged when no wrapper is detected", () => {
-  assert.equal(normalizeCommand("git status"), "git status");
-  assert.equal(normalizeCommand("bun test"), "bun test");
-  assert.equal(normalizeCommand("python -m pytest"), "python -m pytest");
-  assert.equal(normalizeCommand("tsc --noEmit"), "tsc --noEmit");
-});
-
-test("normalizeCommand: collapses local file paths inside commands", () => {
-  assert.equal(
-    normalizeCommand(`pwsh.exe -Command 'Get-Content C:\\Users\\Example\\Project\\README.md'`),
-    "Get-Content README.md",
-  );
-  assert.equal(
-    normalizeCommand(`Get-Content C:/Users/Example/Project/src/App.tsx`),
-    "Get-Content src/App.tsx",
-  );
-});
-
-test("normalizeCommand: is case-insensitive for -Command flag", () => {
-  assert.equal(normalizeCommand(`pwsh.exe -command 'ls'`), "ls");
-  assert.equal(normalizeCommand(`pwsh.exe -COMMAND 'ls'`), "ls");
-});
-
-// ─── getFriendlyActionLabel ───────────────────────────────────────────────────
-
-test("getFriendlyActionLabel: Get-ChildItem → List files", () => {
-  assert.equal(getFriendlyActionLabel("Get-ChildItem -Force | Select-Object Name,Mode,Length"), "List files");
-  assert.equal(getFriendlyActionLabel("Get-ChildItem"), "List files");
-  assert.equal(getFriendlyActionLabel("dir /b"), "List files");
-  assert.equal(getFriendlyActionLabel("ls -la"), "List files");
-  assert.equal(getFriendlyActionLabel("rg --files"), "List files");
-});
-
-test("getFriendlyActionLabel: Get-Content / cat → Read file", () => {
-  assert.equal(getFriendlyActionLabel("Get-Content package.json"), "Read file");
-  assert.equal(getFriendlyActionLabel("cat src/index.ts"), "Read file");
-});
-
-test("getFriendlyActionLabel: git status → Check git status", () => {
-  assert.equal(getFriendlyActionLabel("git status"), "Check git status");
-  assert.equal(getFriendlyActionLabel("git status --short"), "Check git status");
-});
-
-test("getFriendlyActionLabel: git diff → Inspect changes", () => {
-  assert.equal(getFriendlyActionLabel("git diff"), "Inspect changes");
-  assert.equal(getFriendlyActionLabel("git diff HEAD~1"), "Inspect changes");
-});
-
-test("getFriendlyActionLabel: install commands → Install dependencies", () => {
-  assert.equal(getFriendlyActionLabel("bun install"), "Install dependencies");
-  assert.equal(getFriendlyActionLabel("npm install"), "Install dependencies");
-  assert.equal(getFriendlyActionLabel("npm install --save-dev"), "Install dependencies");
-});
-
-test("getFriendlyActionLabel: test commands → Run tests", () => {
-  assert.equal(getFriendlyActionLabel("bun test"), "Run tests");
-  assert.equal(getFriendlyActionLabel("npm test"), "Run tests");
-});
-
-test("getFriendlyActionLabel: typecheck commands → Run typecheck", () => {
-  assert.equal(getFriendlyActionLabel("tsc --noEmit"), "Run typecheck");
-  assert.equal(getFriendlyActionLabel("bun run typecheck"), "Run typecheck");
-});
-
-test("getFriendlyActionLabel: returns null for unrecognized commands", () => {
-  assert.equal(getFriendlyActionLabel("python -m pytest"), null);
-  assert.equal(getFriendlyActionLabel("echo hello"), null);
-  assert.equal(getFriendlyActionLabel("node scripts/build.js"), null);
-});
-
-test("normalizeCommand: decodes JSON-escaped quotes in PowerShell commands", () => {
-  const raw = `pwsh.exe -Command 'Get-ChildItem | Where-Object { $_.Name -match \\"requirements\\" }'`;
-  assert.equal(
-    normalizeCommand(raw),
-    'Get-ChildItem | Where-Object { $_.Name -match "requirements" }',
-  );
-});
-
-test("normalizeCommand: decodes escaped quotes in bare commands", () => {
-  assert.equal(normalizeCommand(`grep -r \\"TODO\\" ./src`), 'grep -r "TODO" ./src');
-});
-
-test("normalizeCommand: decodes escaped quotes in cmd.exe wrapper", () => {
-  assert.equal(normalizeCommand(`cmd.exe /c "grep \\"error\\" log.txt"`), 'grep "error" log.txt');
-});
-
-test("normalizeCommand: passes through MCP tool call format unchanged", () => {
-  assert.equal(normalizeCommand("filesystem:read_file"), "filesystem:read_file");
-  assert.equal(normalizeCommand("github:create_issue"), "github:create_issue");
-});
-
-test("normalizeCommand: passes through web search format unchanged", () => {
-  assert.equal(
-    normalizeCommand("web search: TypeScript union types"),
-    "web search: TypeScript union types",
-  );
-});
-
-test("getFriendlyActionLabel: returns null for MCP tool call format", () => {
-  assert.equal(getFriendlyActionLabel("filesystem:read_file"), null);
-  assert.equal(getFriendlyActionLabel("github:list_issues"), null);
-});
-
-test("getFriendlyActionLabel: returns null for web search commands", () => {
-  assert.equal(getFriendlyActionLabel("web search: TypeScript generics tutorial"), null);
-});

package/src/ui/diffRenderer.test.ts

@@ -1,102 +0,0 @@
-import assert from "node:assert/strict";
-import test from "node:test";
-import { isUnifiedDiff, maybeRenderDiff, renderUnifiedDiff } from "./diffRenderer.js";
-
-const GIT_DIFF = [
-  "diff --git a/src/example.ts b/src/example.ts",
-  "index 1111111..2222222 100644",
-  "--- a/src/example.ts",
-  "+++ b/src/example.ts",
-  "@@ -1,3 +1,4 @@",
-  " const name = \"Codexa\";",
-  "-console.log(\"old\");",
-  "+console.log(\"new\");",
-  "+console.log(\"added\");",
-  " export default name;",
-].join("\n");
-
-test("detects a git unified diff", () => {
-  assert.equal(isUnifiedDiff(GIT_DIFF), true);
-});
-
-test("detects a simple file-header and hunk diff", () => {
-  const diff = [
-    "--- old.txt",
-    "+++ new.txt",
-    "@@ -1 +1 @@",
-    "-old",
-    "+new",
-  ].join("\n");
-
-  assert.equal(isUnifiedDiff(diff), true);
-});
-
-test("classifies file, hunk, added, removed, context, and meta lines", () => {
-  const rendered = renderUnifiedDiff(GIT_DIFF);
-  assert.deepEqual(rendered.map((line) => line.type), [
-    "file",
-    "meta",
-    "file",
-    "file",
-    "hunk",
-    "context",
-    "remove",
-    "add",
-    "add",
-    "context",
-  ]);
-});
-
-test("handles malformed diff-like text without throwing", () => {
-  assert.doesNotThrow(() => renderUnifiedDiff("diff --git a/a b/a\nnot enough here"));
-  assert.equal(maybeRenderDiff("diff --git a/a b/a\nnot enough here"), null);
-});
-
-test("does not classify normal text as a diff", () => {
-  const text = [
-    "Here are options:",
-    "- use --help",
-    "+ consider reading docs",
-  ].join("\n");
-
-  assert.equal(isUnifiedDiff(text), false);
-  assert.equal(maybeRenderDiff(text), null);
-});
-
-test("keeps Windows paths readable in file headers", () => {
-  const diff = [
-    "--- a\\src\\example.ts",
-    "+++ b\\src\\example.ts",
-    "@@ -1 +1 @@",
-    "-old",
-    "+new",
-  ].join("\n");
-
-  const rendered = renderUnifiedDiff(diff);
-  assert.equal(rendered[0]?.text, "--- a\\src\\example.ts");
-  assert.equal(rendered[1]?.text, "+++ b\\src\\example.ts");
-});
-
-test("strips ANSI and unsafe controls before classification", () => {
-  const diff = [
-    "\u001b[31m--- a/file.ts\u001b[0m",
-    "\u001b[32m+++ b/file.ts\u001b[0m",
-    "@@ -1 +1 @@",
-    "\u001b[31m-old\u001b[0m",
-    "\u001b[32m+new\u001b[0m\u0007",
-  ].join("\n");
-
-  const rendered = renderUnifiedDiff(diff);
-  assert.deepEqual(rendered.map((line) => line.text), [
-    "--- a/file.ts",
-    "+++ b/file.ts",
-    "@@ -1 +1 @@",
-    "-old",
-    "+new",
-  ]);
-});
-
-test("forced rendering accepts explicit diff fences with change lines", () => {
-  assert.equal(isUnifiedDiff("-old\n+new", { force: true }), true);
-  assert.equal(isUnifiedDiff("-old\n+new"), false);
-});