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

package/src/App.diff.test.tsx

@@ -1,99 +0,0 @@
-/** Diff-review flow in the virtual terminal (slice 3). */
-
-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 PATCH = `diff --git a/src/store.ts b/src/store.ts
-index 111..222 100644
---- a/src/store.ts
-+++ b/src/store.ts
-@@ -1,4 +1,4 @@
- export class Store {
--  private items = [];
-+  private items = new Map();
- }
-`;
-
-let home: string;
-let server: DaemonServer;
-let session: ReviewSession;
-
-beforeEach(() => {
-  home = mkdtempSync(join(tmpdir(), "cueloop-diff-"));
-  server = new DaemonServer({ home, idleExitMs: 0 });
-  server.start();
-  session = server.core.sessionCreate({
-    workspace: { repoRoot: "/repo", branch: "main" },
-    artifact: { type: "diff", content: PATCH, meta: { title: "working tree" } },
-  });
-});
-afterEach(() => {
-  server.stop();
-  rmSync(home, { recursive: true, force: true });
-});
-
-async function renderApp() {
-  const setup = await testRender(<App home={home} sessionId={session.id} />, { width: 120, height: 30 });
-  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 await setup.mockInput.typeText(k);
-  await Bun.sleep(15);
-  await setup.renderOnce();
-}
-
-describe("diff review", () => {
-  test("renders file header, hunks, and signed lines", async () => {
-    const setup = await renderApp();
-    const frame = setup.captureCharFrame();
-    expect(frame).toContain("■ src/store.ts");
-    expect(frame).toContain("@@ -1,4 +1,4 @@");
-    expect(frame).toContain("-  private items = [];");
-    expect(frame).toContain("+  private items = new Map();");
-  });
-
-  test("line comment: quote-anchored, lands in feedback with the code line", async () => {
-    const setup = await renderApp();
-    // rows: file(0), hunk(1), ctx(2), del(3), add(4)
-    for (let i = 0; i < 4; i++) await press(setup, "j");
-    await press(setup, "c");
-    expect(setup.captureCharFrame()).toContain("COMMENT ON");
-    await setup.mockInput.typeText("Map needs an eviction story.");
-    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]!.anchor.quote).toContain("new Map()");
-    // inline annotation card rendered under the line
-    expect(setup.captureCharFrame()).toContain("◆ Map needs an eviction story.");
-
-    await press(setup, "enter"); // submit
-    await press(setup, "enter"); // confirm request_changes
-    await Bun.sleep(100);
-    const resolved = server.core.sessionGet(session.id);
-    expect(resolved.verdict!.feedback).toContain("new Map()");
-    expect(resolved.verdict!.feedback).toContain("Map needs an eviction story.");
-  });
-
-  test("plan verbs are guarded in diff sessions", async () => {
-    const setup = await renderApp();
-    await press(setup, "x");
-    expect(setup.captureCharFrame()).toContain("plan-only verb");
-  });
-});

package/src/App.export.test.tsx

@@ -1,88 +0,0 @@
-/**
- * Tier-2 wiring test for the notes-vault export: with [integrations.obsidian]
- * configured to export on resolve, submitting a review writes the note into
- * the vault and surfaces the path in the status line.
- */
-
-import { afterEach, beforeEach, describe, expect, test } from "bun:test";
-import { existsSync, mkdirSync, mkdtempSync, readFileSync, readdirSync, rmSync, writeFileSync } 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
-
-Move the store atomically.
-`;
-
-let home: string;
-let vault: string;
-let server: DaemonServer;
-let session: ReviewSession;
-let savedConfigEnv: string | undefined;
-
-beforeEach(() => {
-  home = mkdtempSync(join(tmpdir(), "cueloop-app-export-"));
-  vault = join(home, "vault");
-  mkdirSync(vault);
-  const configPath = join(home, "config.toml");
-  writeFileSync(configPath, `[integrations.obsidian]\nvault = ${JSON.stringify(vault)}\nexportOn = "resolve"\n`);
-  savedConfigEnv = process.env.CUELOOP_CONFIG;
-  process.env.CUELOOP_CONFIG = configPath;
-  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(() => {
-  if (savedConfigEnv === undefined) delete process.env.CUELOOP_CONFIG;
-  else process.env.CUELOOP_CONFIG = savedConfigEnv;
-  server.stop();
-  rmSync(home, { recursive: true, force: true });
-});
-
-describe("obsidian export on resolve", () => {
-  test("submitting a review writes the plan into the vault and shows the path", async () => {
-    const setup = await testRender(<App home={home} sessionId={session.id} />, { width: 120, height: 32 });
-    for (let i = 0; i < 40 && !setup.captureCharFrame().includes("cueloop"); i++) {
-      await Bun.sleep(25);
-      await setup.renderOnce();
-    }
-    // open the submit overlay, keep the default verdict (approve), submit
-    setup.mockInput.pressKey("RETURN");
-    await Bun.sleep(15);
-    await setup.renderOnce();
-    expect(setup.captureCharFrame()).toContain("verdict");
-    setup.mockInput.pressKey("RETURN");
-    // wall-clock deadlines: the export is an async round-trip after resolve,
-    // and an iteration count times out on a loaded runner
-    const dir = join(vault, "cueloop");
-    const fileDeadline = Date.now() + 30_000;
-    while (Date.now() < fileDeadline && !existsSync(dir)) {
-      await Bun.sleep(25);
-      await setup.renderOnce();
-    }
-    // the status line lands on a later render than the file write
-    const statusDeadline = Date.now() + 30_000;
-    while (Date.now() < statusDeadline && !setup.captureCharFrame().includes("exported to")) {
-      await Bun.sleep(25);
-      await setup.renderOnce();
-    }
-
-    const notes = readdirSync(dir);
-    expect(notes.length).toBe(1);
-    expect(notes[0]).toMatch(/^\d{4}-\d{2}-\d{2} - Migration Plan\.md$/);
-    const written = readFileSync(join(dir, notes[0]!), "utf8");
-    expect(written).toContain("source: cueloop");
-    expect(written).toContain(`session: ${session.id}`);
-    expect(written).toContain("verdict: approve");
-    expect(written).toContain("Move the store atomically.");
-    expect(setup.captureCharFrame()).toContain("exported to");
-  });
-});

package/src/App.plan-review.test.tsx

@@ -1,259 +0,0 @@
-/**
- * The plan review surface v2 (#71): native selection feeds the annotation
- * quote, the inline compose box keeps its anchor painted, the rail edits what
- * the document selects, and edit-exit reconciliation orphans annotations
- * whose passage was removed. Char-frame + styled-span assertions over the
- * real App and a real in-process daemon.
- */
-
-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";
-import { DARK as T } from "./theme";
-
-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-plan-review-"));
-  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", agent: "agent/worker-3" } },
-  });
-});
-afterEach(() => {
-  server.stop();
-  rmSync(home, { recursive: true, force: true });
-});
-
-async function renderApp() {
-  const setup = await testRender(<App home={home} sessionId={session.id} />, {
-    width: 120,
-    height: 32,
-  });
-  for (let attempt = 0; attempt < 40 && !setup.captureCharFrame().includes("cueloop"); attempt++) {
-    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 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();
-}
-
-/**
- * Background colors (hex) of every styled span containing the needle - the
- * document highlight and the rail's quote excerpt can both match.
- */
-function backgroundsOf(setup: Setup, needle: string): string[] {
-  const backgrounds: string[] = [];
-  for (const line of setup.captureSpans().lines) {
-    for (const span of line.spans) {
-      if (!span.text.includes(needle)) continue;
-      const [red, green, blue] = span.bg.toInts();
-      backgrounds.push("#" + [red, green, blue].map((part) => part.toString(16).padStart(2, "0")).join(""));
-    }
-  }
-  return backgrounds;
-}
-
-/** Move the cursor to "The daemon persists sessions to disk atomically." */
-async function toContextParagraph(setup: Setup): Promise<void> {
-  await press(setup, "j");
-  await press(setup, "j");
-}
-
-describe("selection feeds the annotation quote", () => {
-  test("v anchors, l extends, c composes with the char-precise quote", async () => {
-    const setup = await renderApp();
-    await toContextParagraph(setup);
-    await press(setup, "v");
-    await press(setup, "l"); // "The daemon"
-    await press(setup, "c");
-    expect(setup.captureCharFrame()).toContain('comment on "The daemon"');
-    await type(setup, "Which daemon?");
-    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]!.anchor.quote).toBe("The daemon");
-  });
-
-  test("a mouse drag over the text becomes the compose quote", async () => {
-    const setup = await renderApp();
-    const lines = setup.captureCharFrame().split("\n");
-    const row = lines.findIndex((line) => line.includes("persists sessions"));
-    const startColumn = lines[row]!.indexOf("persists");
-    expect(row).toBeGreaterThan(-1);
-    // the drag's end cell is exclusive: release one past the last character
-    await setup.mockMouse.drag(startColumn, row, startColumn + "persists sessions".length, row);
-    await setup.renderOnce();
-    await press(setup, "c");
-    expect(setup.captureCharFrame()).toContain('comment on "persists sessions"');
-    await type(setup, "sessions plural?");
-    await press(setup, "enter");
-    await Bun.sleep(80);
-    const stored = server.core.sessionGet(session.id);
-    expect(stored.annotations[0]!.anchor.quote).toBe("persists sessions");
-  });
-});
-
-describe("inline compose keeps the anchor painted", () => {
-  test("the selection stays painted while composing; cancel un-paints; save converts to the comment highlight", async () => {
-    const setup = await renderApp();
-    await toContextParagraph(setup);
-    await press(setup, "v");
-    await press(setup, "l");
-    await press(setup, "c");
-    // compose open: the anchor is painted selection-style
-    expect(setup.captureCharFrame()).toContain("Save ⏎");
-    expect(setup.captureCharFrame()).toContain("Cancel esc");
-    expect(backgroundsOf(setup, "The daemon")).toContain(T.accent);
-    // cancel un-paints
-    await press(setup, "escape");
-    expect(setup.captureCharFrame()).not.toContain('comment on "');
-    expect(backgroundsOf(setup, "The daemon")).not.toContain(T.accent);
-    // save converts the paint to the kind-colored annotation highlight
-    await press(setup, "c");
-    await type(setup, "Which daemon?");
-    await press(setup, "enter");
-    await Bun.sleep(80);
-    await setup.renderOnce();
-    // the whole cursor block is the anchor here; check the highlight landed
-    const stored = server.core.sessionGet(session.id);
-    expect(stored.annotations.length).toBe(1);
-    expect(backgroundsOf(setup, stored.annotations[0]!.anchor.quote.slice(0, 20))).toContain(T.markCommentBg);
-  });
-});
-
-describe("the document selects, the rail edits", () => {
-  test("e edits the selected card in place; enter saves through the controller", async () => {
-    const setup = await renderApp();
-    await toContextParagraph(setup);
-    await press(setup, "c");
-    await type(setup, "Needs a citation.");
-    await press(setup, "enter");
-    await Bun.sleep(80);
-    await setup.renderOnce();
-    expect(setup.captureCharFrame()).toContain("COMMENT · pending");
-    // the saved card is selected; e turns its body into an input in place
-    await press(setup, "e");
-    await type(setup, " And a link.");
-    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("Needs a citation. And a link.");
-  });
-
-  test("x deletes the selected card and un-paints the document highlight", async () => {
-    const setup = await renderApp();
-    await toContextParagraph(setup);
-    await press(setup, "c");
-    await type(setup, "Delete me.");
-    await press(setup, "enter");
-    await Bun.sleep(80);
-    await setup.renderOnce();
-    expect(server.core.sessionGet(session.id).annotations.length).toBe(1);
-    expect(backgroundsOf(setup, "persists sessions")).toContain(T.markCommentBg);
-    await press(setup, "x");
-    await Bun.sleep(80);
-    await setup.renderOnce();
-    expect(server.core.sessionGet(session.id).annotations.length).toBe(0);
-    expect(setup.captureCharFrame()).not.toContain("COMMENT · pending");
-    expect(backgroundsOf(setup, "persists sessions")).not.toContain(T.markCommentBg);
-  });
-});
-
-describe("sheet header", () => {
-  test("shows the submitting agent, the revision, and the Edit word-button", async () => {
-    const setup = await renderApp();
-    const frame = setup.captureCharFrame();
-    expect(frame).toContain("submitted by agent/worker-3 · revision 1");
-    expect(frame).toContain("Edit");
-  });
-
-  test("the Agent tab shows the submitter, status, and revision", async () => {
-    const setup = await renderApp();
-    const lines = setup.captureCharFrame().split("\n");
-    const tabRow = lines.findIndex((line) => line.includes("Review (0)"));
-    const tabColumn = lines[tabRow]!.indexOf("Agent");
-    await setup.mockMouse.click(tabColumn + 1, tabRow);
-    await setup.renderOnce();
-    const frame = setup.captureCharFrame();
-    expect(frame).toContain("agent/worker-3");
-    expect(frame).toContain("status: pending");
-    expect(frame).toContain("revision 1");
-  });
-});
-
-describe("edit-exit reconciliation", () => {
-  test("an edit that removes an anchored passage orphans the annotation and shows the banner", async () => {
-    const script = join(home, "fake-editor.sh");
-    await Bun.write(script, `#!/bin/sh\nsed -i '' '/^The daemon persists/d' "$1"\n`);
-    Bun.spawnSync(["chmod", "+x", script]);
-    process.env.CUELOOP_EDITOR = script;
-    try {
-      const setup = await renderApp();
-      await toContextParagraph(setup);
-      await press(setup, "c");
-      await type(setup, "Anchor me to the doomed passage.");
-      await press(setup, "enter");
-      await Bun.sleep(80);
-      await setup.renderOnce();
-      expect(setup.captureCharFrame()).toContain("· pending");
-      // deselect the card so e reaches the editor hand-off, then edit
-      await press(setup, "escape");
-      await press(setup, "e");
-      await Bun.sleep(150);
-      await setup.renderOnce();
-      const frame = setup.captureCharFrame();
-      expect(frame).toContain("1 annotation no longer match - the passage was removed.");
-      expect(frame).toContain("· ORPHANED");
-      // the annotation is NOT deleted: the feedback serializer handles orphans
-      expect(server.core.sessionGet(session.id).annotations.length).toBe(1);
-    } finally {
-      delete process.env.CUELOOP_EDITOR;
-    }
-  });
-});

package/src/App.readonly.test.tsx

@@ -1,148 +0,0 @@
-/**
- * Observer mode (tier 2): the App with readOnly renders and navigates like the
- * controller's, but every mutating verb answers "observer - read-only" and
- * leaves daemon state untouched.
- */
-
-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-observer-"));
-  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 renderObserver() {
-  const setup = await testRender(<App home={home} sessionId={session.id} readOnly />, {
-    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 renderObserver>>;
-
-async function press(setup: Setup, k: string): Promise<void> {
-  if (k === "enter") setup.mockInput.pressKey("RETURN");
-  else await setup.mockInput.typeText(k);
-  await Bun.sleep(15);
-  await setup.renderOnce();
-}
-
-/** Session state that any mutating verb would change. */
-function snapshot() {
-  const s = server.core.sessionGet(session.id);
-  return { annotations: s.annotations.length, workingCopy: s.workingCopy, status: s.status };
-}
-
-describe("observer rendering", () => {
-  test("shows the observer badge and read-only hint bar", async () => {
-    const setup = await renderObserver();
-    const frame = setup.captureCharFrame();
-    expect(frame).toContain("· observer");
-    expect(frame).toContain("observer - read-only · j/k move");
-    expect(frame).not.toContain("⏎ submit");
-  });
-});
-
-describe("observer verbs are blocked", () => {
-  for (const [key, verb] of [
-    ["c", "comment"],
-    ["s", "suggest"],
-    ["x", "cut"],
-    ["e", "edit"],
-    ["enter", "submit"],
-  ] as const) {
-    test(`${verb} (${key}) answers observer - read-only and mutates nothing`, async () => {
-      const setup = await renderObserver();
-      const before = snapshot();
-      await press(setup, "j");
-      await press(setup, "j");
-      await press(setup, key);
-      await Bun.sleep(60);
-      await setup.renderOnce();
-      const frame = setup.captureCharFrame();
-      expect(frame).toContain("observer - read-only");
-      // no overlay opened: compose/submit bars never appear
-      expect(frame).not.toContain("COMMENT ON");
-      expect(frame).not.toContain("SUGGEST REPLACEMENT");
-      expect(frame).not.toContain("verdict ←/→");
-      expect(snapshot()).toEqual(before);
-    });
-  }
-
-  test("span mode c/s is blocked too", async () => {
-    const setup = await renderObserver();
-    await press(setup, "j");
-    await press(setup, "j");
-    await press(setup, "v");
-    await press(setup, "c");
-    await setup.renderOnce();
-    const frame = setup.captureCharFrame();
-    expect(frame).toContain("observer - read-only");
-    expect(frame).not.toContain("COMMENT ON");
-    expect(snapshot().annotations).toBe(0);
-  });
-});
-
-describe("observer navigation still works", () => {
-  test("j/k moves the cursor between blocks", async () => {
-    const setup = await renderObserver();
-    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("n focuses annotations made by the controller", async () => {
-    server.core.sessionAnnotate(session.id, {
-      id: "a_obs1",
-      kind: "comment",
-      anchor: makeAnchor(parseBlocks(PLAN), 2, 0, 10),
-      body: "From the controller.",
-    });
-    const setup = await renderObserver();
-    await press(setup, "n");
-    await setup.renderOnce();
-    expect(setup.captureCharFrame()).toContain("▸ COMMENT");
-  });
-});