@cueloop/client 0.1.0-alpha.10 → 0.1.0-alpha.101

package/src/App.rendering.test.tsx

@@ -1,108 +0,0 @@
-/** Plan rendering (#67): code blocks verbatim + contained, spacing never collapses. */
-
-import { afterEach, beforeEach, describe, expect, test } from "bun:test";
-import { mkdtempSync, rmSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import React from "react";
-import { testRender } from "@opentui/react/test-utils";
-import { DaemonServer } from "@cueloop/daemon";
-import type { ReviewSession } from "@cueloop/schema";
-import { App } from "./App";
-
-const PLAN = `# Render Plan
-
-## Sensor
-
-- wire the duck over MQTT
-
-\`\`\`tsx
-export function Gate({ full }: { full: number }) {
-  const blocked = full < 0.6;
-  return <text>{blocked ? "blocked" : "clear"}</text>;
-}
-\`\`\`
-
-## Gate
-
-Deploys block below the threshold.
-`;
-
-let home: string;
-let server: DaemonServer;
-let session: ReviewSession;
-
-beforeEach(() => {
-  home = mkdtempSync(join(tmpdir(), "cueloop-render-"));
-  server = new DaemonServer({ home, idleExitMs: 0 });
-  server.start();
-  session = server.core.sessionCreate({
-    workspace: { repoRoot: "/repo", branch: "main" },
-    artifact: { type: "plan", content: PLAN, meta: { title: "Render Plan" } },
-  });
-});
-afterEach(() => {
-  server.stop();
-  rmSync(home, { recursive: true, force: true });
-});
-
-async function frame(): Promise<string> {
-  const setup = await testRender(<App home={home} sessionId={session.id} />, { width: 120, height: 36 });
-  const deadline = Date.now() + 10_000;
-  while (Date.now() < deadline && !setup.captureCharFrame().includes("Render Plan")) {
-    await Bun.sleep(25);
-    await setup.renderOnce();
-  }
-  // give the async highlighter a beat, then take the final frame
-  await Bun.sleep(150);
-  await setup.renderOnce();
-  return setup.captureCharFrame();
-}
-
-describe("code blocks", () => {
-  test("lines render verbatim: indentation preserved, no word-wrap", async () => {
-    const f = await frame();
-    expect(f).toContain("  const blocked = full < 0.6;");
-    expect(f).toContain('  return <text>{blocked ? "blocked" : "clear"}</text>;');
-  });
-
-  test("the block carries its language tag", async () => {
-    const f = await frame();
-    expect(f).toContain("tsx");
-  });
-});
-
-describe("block spacing", () => {
-  test("a code block never glues to the list item above it", async () => {
-    const lines = (await frame()).split("\n").map((l) => l.trimEnd());
-    const li = lines.findIndex((l) => l.includes("wire the duck over MQTT"));
-    const codeHeader = lines.findIndex((l) => l.trim().startsWith("tsx"));
-    expect(li).toBeGreaterThan(-1);
-    expect(codeHeader).toBeGreaterThan(li);
-    expect(lines.slice(li + 1, codeHeader).some((l) => l === "")).toBe(true);
-  });
-
-  test("a heading never sits directly on the previous block", async () => {
-    const lines = (await frame()).split("\n").map((l) => l.trimEnd());
-    const gate = lines.findIndex((l) => l.trim() === "Gate");
-    expect(gate).toBeGreaterThan(0);
-    expect(lines[gate - 1]).toBe("");
-  });
-
-  test("consecutive list items stay tight", async () => {
-    const listPlan = "# T\n\n- one\n- two\n";
-    const s2 = server.core.sessionCreate({
-      workspace: { repoRoot: "/repo", branch: "main" },
-      artifact: { type: "plan", content: listPlan, meta: {} },
-    });
-    const setup = await testRender(<App home={home} sessionId={s2.id} />, { width: 120, height: 30 });
-    const deadline = Date.now() + 10_000;
-    while (Date.now() < deadline && !setup.captureCharFrame().includes("one")) {
-      await Bun.sleep(25);
-      await setup.renderOnce();
-    }
-    const lines = setup.captureCharFrame().split("\n").map((l) => l.trimEnd());
-    const one = lines.findIndex((l) => l.includes("- one"));
-    expect(lines[one + 1]).toContain("- two");
-  });
-});

package/src/App.submit-confirm.test.tsx

@@ -1,183 +0,0 @@
-/**
- * The rail submit confirm (tier 2): pressing submit expands the rail's
- * Submit button into a bordered confirm card - counts, verdict selector,
- * summary input, word-buttons. Char-frame assertions over the real App
- * and an in-process daemon, like App.test.tsx.
- */
-
-import { afterEach, beforeEach, describe, expect, test } from "bun:test";
-import { mkdtempSync, rmSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import React from "react";
-import { testRender } from "@opentui/react/test-utils";
-import { DaemonServer } from "@cueloop/daemon";
-import { makeAnchor, parseBlocks, type ReviewSession } from "@cueloop/schema";
-import { App } from "./App";
-
-const PLAN = `# Migration Plan
-
-## Context
-
-The daemon persists sessions to disk atomically.
-
-## Steps
-
-- move the store
-- add recovery
-`;
-
-let home: string;
-let server: DaemonServer;
-let session: ReviewSession;
-
-beforeEach(() => {
-  home = mkdtempSync(join(tmpdir(), "cueloop-confirm-"));
-  server = new DaemonServer({ home, idleExitMs: 0 });
-  server.start();
-  session = server.core.sessionCreate({
-    workspace: { repoRoot: "/repo", branch: "main" },
-    artifact: { type: "plan", content: PLAN, meta: { title: "Migration Plan", planPath: "plan.md" } },
-  });
-});
-afterEach(() => {
-  server.stop();
-  rmSync(home, { recursive: true, force: true });
-});
-
-async function renderApp(options: { readOnly?: boolean } = {}) {
-  const setup = await testRender(<App home={home} sessionId={session.id} readOnly={options.readOnly ?? false} />, {
-    width: 120,
-    height: 32,
-  });
-  for (let i = 0; i < 40 && !setup.captureCharFrame().includes("cueloop"); i++) {
-    await Bun.sleep(25);
-    await setup.renderOnce();
-  }
-  await setup.renderOnce();
-  return setup;
-}
-
-type Setup = Awaited<ReturnType<typeof renderApp>>;
-
-async function press(setup: Setup, k: string): Promise<void> {
-  if (k === "enter") setup.mockInput.pressKey("RETURN");
-  else if (k === "escape") {
-    setup.mockInput.pressKey("ESCAPE");
-    // a bare ESC byte is an ambiguous sequence prefix; the parser settles late
-    await Bun.sleep(120);
-  } else if (k === "left") setup.mockInput.pressKey("ARROW_LEFT");
-  else if (k === "right") setup.mockInput.pressKey("ARROW_RIGHT");
-  else await setup.mockInput.typeText(k);
-  await Bun.sleep(15);
-  await setup.renderOnce();
-}
-
-/** Seed annotations directly through the daemon core (all on one block). */
-function seedAnnotations(count: number): void {
-  const blocks = parseBlocks(PLAN);
-  for (let index = 1; index <= count; index++) {
-    server.core.sessionAnnotate(session.id, {
-      id: `a_confirm_${index}`,
-      kind: "comment",
-      anchor: makeAnchor(blocks, 2, 0, 10),
-      body: `note ${String(index).padStart(2, "0")}`,
-    });
-  }
-}
-
-describe("rail submit confirm", () => {
-  test("submit expands the Submit button into the confirm card", async () => {
-    const setup = await renderApp();
-    expect(setup.captureCharFrame()).toContain("Submit review (0)");
-    await press(setup, "enter");
-    const frame = setup.captureCharFrame();
-    // the card: title, honest counts, verdict selector, word-buttons
-    expect(frame).toContain("submit review");
-    expect(frame).toContain("0 annotations · 0 blocking");
-    expect(frame).toContain("[Approve]"); // nothing pending: approve default
-    expect(frame).toContain(" Submit ");
-    expect(frame).toContain(" Cancel ");
-    // key hints live in the status line, not on the buttons
-    expect(frame).toContain("verdict ←/→ · ⏎ submit · esc cancel");
-    // the bottom bar stayed a one-line hint: no detached verdict bar
-    expect(frame).not.toContain("Submit review (0) on ⏎");
-  });
-
-  test("left/right cycles the verdict selector in the card", async () => {
-    const setup = await renderApp();
-    await press(setup, "enter");
-    expect(setup.captureCharFrame()).toContain("[Approve]");
-    await press(setup, "right");
-    expect(setup.captureCharFrame()).toContain("[Changes]");
-    await press(setup, "right");
-    expect(setup.captureCharFrame()).toContain("[Comment]");
-    await press(setup, "left");
-    expect(setup.captureCharFrame()).toContain("[Changes]");
-  });
-
-  test("esc cancels the card and restores the plain Submit button", async () => {
-    const setup = await renderApp();
-    await press(setup, "enter");
-    expect(setup.captureCharFrame()).toContain("[Approve]");
-    await press(setup, "escape");
-    const frame = setup.captureCharFrame();
-    expect(frame).not.toContain("[Approve]");
-    expect(frame).not.toContain("0 annotations");
-    expect(frame).toContain("Submit review (0)");
-  });
-
-  test("enter in the card resolves the session through the controller", async () => {
-    seedAnnotations(1);
-    const setup = await renderApp();
-    await press(setup, "enter");
-    const frame = setup.captureCharFrame();
-    expect(frame).toContain("1 annotations · 0 blocking");
-    expect(frame).toContain("[Changes]"); // pending items: request changes default
-    await setup.mockInput.typeText("Tighten the steps.");
-    await press(setup, "enter");
-    await Bun.sleep(120);
-    await setup.renderOnce();
-    const stored = server.core.sessionGet(session.id);
-    expect(stored.status).toBe("resolved");
-    expect(stored.verdict!.kind).toBe("request_changes");
-    // the completion flow after submit is unchanged
-    expect(setup.captureCharFrame()).toContain("✎ feedback sent");
-  });
-
-  test("the annotation stack stays scrollable while the card is open", async () => {
-    seedAnnotations(12);
-    const setup = await renderApp();
-    // walk annotation focus to the last card: the rail scrolls it into view
-    for (let index = 0; index < 12; index++) await press(setup, "n");
-    const scrolled = setup.captureCharFrame();
-    expect(scrolled).toContain("note 12");
-    expect(scrolled).not.toContain("note 01");
-    await press(setup, "enter");
-    const opened = setup.captureCharFrame();
-    // the card is pinned outside the scrollbox; the stack above stays
-    // scrolled away from the top when the card takes its space
-    expect(opened).toContain("12 annotations · 0 blocking");
-    expect(opened).not.toContain("note 01");
-    // the stack still scrolls with the card open: wheel over the rail
-    // brings the last card back into view while the confirm card stays put
-    for (let turn = 0; turn < 12; turn++) await setup.mockMouse.scroll(100, 10, "down");
-    await setup.renderOnce();
-    const scrolledWithCard = setup.captureCharFrame();
-    expect(scrolledWithCard).toContain("note 12");
-    expect(scrolledWithCard).toContain("12 annotations · 0 blocking");
-    expect(scrolledWithCard).not.toContain("note 01");
-  });
-
-  test("read-only observers never see the confirm card", async () => {
-    seedAnnotations(1);
-    const setup = await renderApp({ readOnly: true });
-    await press(setup, "enter");
-    const frame = setup.captureCharFrame();
-    expect(frame).toContain("observer - read-only");
-    expect(frame).not.toContain("1 annotations");
-    expect(frame).not.toContain("[Changes]");
-    expect(setup.captureCharFrame()).not.toContain("verdict ←/→");
-    expect(server.core.sessionGet(session.id).status).toBe("pending");
-  });
-});

package/src/App.test.tsx

@@ -1,218 +0,0 @@
-/**
- * Virtual-terminal component tests (tier 2): the real App over a real
- * in-process daemon in a temp home. Char-frame assertions + mock keys -
- * the whole review loop drivable without a terminal.
- */
-
-import { afterEach, beforeEach, describe, expect, test } from "bun:test";
-import { mkdtempSync, rmSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import React from "react";
-import { testRender } from "@opentui/react/test-utils";
-import { DaemonServer } from "@cueloop/daemon";
-import type { ReviewSession } from "@cueloop/schema";
-import { App } from "./App";
-
-const PLAN = `# Migration Plan
-
-## Context
-
-The daemon persists sessions to disk atomically.
-
-## Steps
-
-- move the store
-- add recovery
-`;
-
-let home: string;
-let server: DaemonServer;
-let session: ReviewSession;
-
-beforeEach(() => {
-  home = mkdtempSync(join(tmpdir(), "cueloop-app-"));
-  server = new DaemonServer({ home, idleExitMs: 0 });
-  server.start();
-  session = server.core.sessionCreate({
-    workspace: { repoRoot: "/repo", branch: "main" },
-    artifact: { type: "plan", content: PLAN, meta: { title: "Migration Plan", planPath: "plan.md" } },
-  });
-});
-afterEach(() => {
-  server.stop();
-  rmSync(home, { recursive: true, force: true });
-});
-
-async function renderApp(sessionId?: string) {
-  const setup = await testRender(<App home={home} sessionId={sessionId ?? session.id} />, {
-    width: 120,
-    height: 32,
-  });
-  // let the async daemon connect + first fetch land
-  for (let i = 0; i < 40 && !setup.captureCharFrame().includes("cueloop"); i++) {
-    await Bun.sleep(25);
-    await setup.renderOnce();
-  }
-  await setup.renderOnce();
-  return setup;
-}
-
-type Setup = Awaited<ReturnType<typeof renderApp>>;
-
-/** Drive one key press: letters type as text; named keys use KeyCodes ids. */
-async function press(setup: Setup, k: string): Promise<void> {
-  if (k === "enter") setup.mockInput.pressKey("RETURN");
-  else if (k === "escape") setup.mockInput.pressKey("ESCAPE");
-  else if (k === "backspace") setup.mockInput.pressKey("BACKSPACE");
-  else if (k === "left") setup.mockInput.pressKey("ARROW_LEFT");
-  else if (k === "right") setup.mockInput.pressKey("ARROW_RIGHT");
-  else await type(setup, k);
-  await Bun.sleep(15);
-  await setup.renderOnce();
-}
-
-async function type(setup: Setup, text: string): Promise<void> {
-  await setup.mockInput.typeText(text);
-  await Bun.sleep(15);
-  await setup.renderOnce();
-}
-
-describe("plan rendering", () => {
-  test("renders the plan with headings, list markers, and the rail", async () => {
-    const setup = await renderApp();
-    const frame = setup.captureCharFrame();
-    expect(frame).toContain("Migration Plan");
-    expect(frame).toContain("Context");
-    expect(frame).toContain("persists sessions to disk atomically");
-    expect(frame).toContain("- move the store");
-    expect(frame).toContain("Review (0)");
-    expect(frame).toContain("Submit review (0)");
-  });
-});
-
-describe("keyboard grammar", () => {
-  test("j/k moves the cursor glyph between blocks", async () => {
-    const setup = await renderApp();
-    await press(setup, "j");
-    await press(setup, "j");
-    await setup.renderOnce();
-    const lines = setup.captureCharFrame().split("\n");
-    const cursorLine = lines.find((l) => l.includes("▎"))!;
-    expect(cursorLine).toContain("persists sessions");
-  });
-
-  test("comment flow: c types a body, ⏎ saves to the daemon", async () => {
-    const setup = await renderApp();
-    await press(setup, "j");
-    await press(setup, "j");
-    await press(setup, "c");
-    await setup.renderOnce();
-    expect(setup.captureCharFrame()).toContain('comment on "The daemon');
-    await type(setup, "Define atomically.");
-    await press(setup, "enter");
-    await Bun.sleep(80);
-    await setup.renderOnce();
-    const stored = server.core.sessionGet(session.id);
-    expect(stored.annotations.length).toBe(1);
-    expect(stored.annotations[0]!.body).toBe("Define atomically.");
-    expect(setup.captureCharFrame()).toContain("COMMENT");
-  });
-
-  test("span mode: v + l selects words, c anchors the exact span", async () => {
-    const setup = await renderApp();
-    await press(setup, "j");
-    await press(setup, "j");
-    await press(setup, "v");
-    await press(setup, "l"); // "The daemon"
-    await press(setup, "c");
-    await type(setup, "Which daemon?");
-    await press(setup, "enter");
-    await Bun.sleep(80);
-    const stored = server.core.sessionGet(session.id);
-    expect(stored.annotations[0]!.anchor.quote).toBe("The daemon");
-  });
-
-  test("x cuts a block into the working copy; x restores it", async () => {
-    const setup = await renderApp();
-    // move to "- move the store" (h1, h2, p, h2 = 4 steps in)
-    for (let i = 0; i < 4; i++) await press(setup, "j");
-    await press(setup, "x");
-    await Bun.sleep(80);
-    await setup.renderOnce();
-    expect(server.core.sessionGet(session.id).workingCopy).not.toContain("move the store");
-    expect(setup.captureCharFrame()).toContain("[cut]");
-    await press(setup, "x");
-    await Bun.sleep(80);
-    await setup.renderOnce();
-    expect(server.core.sessionGet(session.id).workingCopy).toBeUndefined();
-  });
-
-  test("e runs $EDITOR on the working copy and tracks the diff", async () => {
-    const script = join(home, "fake-editor.sh");
-    await Bun.write(script, `#!/bin/sh\nsed -i '' 's/atomically/very atomically/' "$1"\n`);
-    Bun.spawnSync(["chmod", "+x", script]);
-    process.env.CUELOOP_EDITOR = script;
-    try {
-      const setup = await renderApp();
-      await press(setup, "e");
-      await Bun.sleep(150);
-      await setup.renderOnce();
-      expect(server.core.sessionGet(session.id).workingCopy).toContain("very atomically");
-      expect(setup.captureCharFrame()).toContain("[edited]");
-    } finally {
-      delete process.env.CUELOOP_EDITOR;
-    }
-  });
-});
-
-describe("submit", () => {
-  test("⏎ opens the rail confirm card; verdict + summary resolve the session", async () => {
-    const setup = await renderApp();
-    await press(setup, "j");
-    await press(setup, "j");
-    await press(setup, "c");
-    await type(setup, "Needs a phase list.");
-    await press(setup, "enter");
-    await Bun.sleep(80);
-    await press(setup, "enter"); // open submit
-    await setup.renderOnce();
-    expect(setup.captureCharFrame()).toContain("[Changes]");
-    await type(setup, "Expand the steps.");
-    await press(setup, "enter");
-    await Bun.sleep(120);
-    await setup.renderOnce();
-    const stored = server.core.sessionGet(session.id);
-    expect(stored.status).toBe("resolved");
-    expect(stored.verdict!.kind).toBe("request_changes");
-    expect(stored.verdict!.feedback).toContain("Needs a phase list.");
-    // submit hands the reviewer back to the agent via the completion overlay
-    expect(setup.captureCharFrame()).toContain("✎ feedback sent");
-  });
-
-  test("approve via ←/→ verdict cycling", async () => {
-    const setup = await renderApp();
-    await press(setup, "enter");
-    await setup.renderOnce();
-    expect(setup.captureCharFrame()).toContain("[Approve]"); // no pending items → approve default
-    await press(setup, "enter");
-    await Bun.sleep(120);
-    expect(server.core.sessionGet(session.id).verdict!.kind).toBe("approve");
-  });
-});
-
-describe("inbox", () => {
-  test("inbox mode renders and opens a session", async () => {
-    const setup = await testRender(<App home={home} />, { width: 120, height: 32 });
-    for (let i = 0; i < 40 && !setup.captureCharFrame().includes("inbox"); i++) {
-      await Bun.sleep(25);
-      await setup.renderOnce();
-    }
-    expect(setup.captureCharFrame()).toContain("inbox (1 pending)");
-    expect(setup.captureCharFrame()).toContain("Migration Plan");
-    await press(setup, "enter");
-    await Bun.sleep(80);
-    await setup.renderOnce();
-    expect(setup.captureCharFrame()).toContain("Submit review (0)");
-  });
-});