@cueloop/client 0.1.0-alpha.64 → 0.1.0-alpha.66

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cueloop/client",
-  "version": "0.1.0-alpha.64",
+  "version": "0.1.0-alpha.66",
   "description": "The cueloop review UI: projection renderer, keyboard grammar, review rail",
   "homepage": "https://github.com/mmurakaru/cueloop#readme",
   "bugs": {
@@ -26,10 +26,10 @@
     "access": "public"
   },
   "dependencies": {
-    "@cueloop/daemon": "0.1.0-alpha.64",
-    "@cueloop/extension-api": "0.1.0-alpha.64",
-    "@cueloop/integration-obsidian": "0.1.0-alpha.64",
-    "@cueloop/schema": "0.1.0-alpha.64",
+    "@cueloop/daemon": "0.1.0-alpha.66",
+    "@cueloop/extension-api": "0.1.0-alpha.66",
+    "@cueloop/integration-obsidian": "0.1.0-alpha.66",
+    "@cueloop/schema": "0.1.0-alpha.66",
     "@opentui/core": "^0.5.8",
     "@opentui/keymap": "0.5.8",
     "@opentui/react": "^0.5.8",

package/src/App.autoclose.test.tsx

@@ -10,7 +10,7 @@ import { ManualClock } from "@opentui/core/testing";
 import { DaemonServer } from "@cueloop/daemon";
 import type { ReviewSession } from "@cueloop/schema";
 import { App } from "./App";
-import { isolateUserConfig, press, settle, waitForText } from "./test-support";
+import { isolateUserConfig, press, settle, waitForText, pressKey } from "./test-support";
 
 const PLAN = "# Plan\n\nShip the thing.\n";
 
@@ -56,7 +56,7 @@ async function renderApp(onExit?: (code: number) => void) {
 type Setup = Awaited<ReturnType<typeof renderApp>>;
 
 async function submitApprove(setup: Setup): Promise<void> {
-  await press(setup, "enter"); // open submit (approve default: no pending items)
+  await pressKey(setup, "RETURN", { meta: true }); // open submit (approve default: no pending items)
   await press(setup, "enter"); // confirm
   await waitForText(setup, "review approved");
 }

package/src/App.diff.test.tsx

@@ -152,50 +152,9 @@ describe("diff review", () => {
     expect(stored.workingCopy).toBe("");
   });
 
-  test("a rejected change lists in the rail and undo (u) restores it", async () => {
-    // Arrange - a diff session that carries full file contents
-    const withFiles = server.core.sessionCreate({
-      workspace: { repoRoot: "/repo", branch: "main" },
-      artifact: {
-        type: "diff",
-        content: PATCH,
-        meta: { title: "working tree" },
-        files: [
-          {
-            path: "src/store.ts",
-            oldContents: "export class Store {\n  private items = [];\n}\n",
-            newContents: "export class Store {\n  private items = new Map();\n}\n",
-            status: "modified",
-          },
-        ],
-      },
-    });
-    const setup = await testRender(<App home={home} sessionId={withFiles.id} />, {
-      width: 120,
-      height: 30,
-    });
-
-    await waitForText(setup, "cueloop");
-
-    // Act - reject the change under the added line
-    // 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, "x");
-
-    // Assert - the rail shows a removal card titled REJECT · me for the change
-    await waitForText(setup, "REJECT · me");
-    expect(server.core.sessionGet(withFiles.id).workingCopy).toBe("");
-
-    // Act - undo restores it (no selection, so the last item)
-    await press(setup, "u");
-
-    // Assert - the change returns to the curated diff (working copy cleared)
-    await waitForText(setup, "removal restored");
-    expect(server.core.sessionGet(withFiles.id).workingCopy).toBeUndefined();
-  });
-
-  test("a selected removal card shows an undo button that restores it on click", async () => {
-    // Arrange - a diff session that carries full file contents
+  test("a diff opens with the Changes column listing every changed file", async () => {
+    // Arrange - two changed files; config.ts is not in the patch text, so it can
+    // only appear here by way of the Changes column reading artifact.files
     const withFiles = server.core.sessionCreate({
       workspace: { repoRoot: "/repo", branch: "main" },
       artifact: {
@@ -203,12 +162,8 @@ describe("diff review", () => {
         content: PATCH,
         meta: { title: "working tree" },
         files: [
-          {
-            path: "src/store.ts",
-            oldContents: "export class Store {\n  private items = [];\n}\n",
-            newContents: "export class Store {\n  private items = new Map();\n}\n",
-            status: "modified",
-          },
+          { path: "src/store.ts", oldContents: "a", newContents: "b", status: "modified" },
+          { path: "src/config.ts", oldContents: "", newContents: "c", status: "added" },
         ],
       },
     });
@@ -219,28 +174,8 @@ describe("diff review", () => {
 
     await waitForText(setup, "cueloop");
 
-    // Act - reject the change; the removal card lands unselected (no undo button)
-    for (let i = 0; i < 4; i++) await press(setup, "j");
-    await press(setup, "x");
-    await waitForText(setup, "REJECT · me");
-    expect(setup.captureCharFrame()).not.toContain("undo");
-
-    // Act - click the card to select it; the undo button appears
-    const clickText = async (needle: string): Promise<void> => {
-      const lines = setup.captureCharFrame().split("\n");
-      const row = lines.findIndex((line) => line.includes(needle));
-
-      await setup.mockMouse.click(lines[row]!.indexOf(needle) + 1, row);
-    };
-
-    await clickText("REJECT · me");
-    await waitForText(setup, "undo");
-
-    // Act - click the undo button
-    await clickText("undo");
-
-    // Assert - the change returns to the curated diff (working copy cleared)
-    await waitForText(setup, "removal restored");
-    expect(server.core.sessionGet(withFiles.id).workingCopy).toBeUndefined();
+    // Assert - the context default opens the column for a diff, so both files show
+    await waitForText(setup, "config.ts");
+    expect(setup.captureCharFrame()).toContain("store.ts");
   });
 });

package/src/App.export.test.tsx

@@ -17,7 +17,7 @@ import { testRender } from "@opentui/react/test-utils";
 import { DaemonServer } from "@cueloop/daemon";
 import type { ReviewSession } from "@cueloop/schema";
 import { App } from "./App";
-import { isolateUserConfig, press, waitForState, waitForText } from "./test-support";
+import { isolateUserConfig, press, waitForState, waitForText, pressKey } from "./test-support";
 
 const PLAN = `# Migration Plan
 
@@ -69,7 +69,7 @@ describe("obsidian export on resolve", () => {
     await waitForText(setup, "cueloop");
 
     // Act
-    await press(setup, "enter"); // open the submit overlay, keep the default verdict (approve)
+    await pressKey(setup, "RETURN", { meta: true }); // open the submit overlay, keep the default verdict (approve)
 
     // Assert
     await waitForText(setup, "[Approve]");