@cueloop/client 0.1.0-alpha.42 → 0.1.0-alpha.44

package/src/components/__snapshots__/stories.test.tsx.snap

@@ -849,17 +849,17 @@ exports[`stories catalog ReviewPanel/Compact renders, snapshots, and carries its
 `;
 
 exports[`stories catalog ReviewPanel/ExpandedNarrow renders, snapshots, and carries its colors: ReviewPanel/ExpandedNarrow 1`] = `
-" The plan takes the width the panel hands back.          ╭─────────────────────╮
-                                                         │ Review  Agent       │
-                                                         ╰─────────────────────╯
-                                                         ╭─ COMMENT · me ──────╮
-                                                         │ Whichidaemonsownss" │
-                                                         │ this?               │
-                                                         ╰─────────────────────╯
-                                                         ╭─ COMMENT · me ──────╮
-                                                         │ moveetheestoree"    │
-                                                         │ behind o…           │
-                                                         ╰─────────────────────╯
+" The plan takes the width the panel hands back.         ╭──────────────────────╮
+                                                        │ Review  Agent        │
+                                                        ╰──────────────────────╯
+                                                        ╭─ COMMENT · me ───────╮
+                                                        │ Whichidaemonsownss"  │
+                                                        │ this?                │
+                                                        ╰──────────────────────╯
+                                                        ╭─ COMMENT · me ───────╮
+                                                        │ moveetheestoree"     │
+                                                        │ behind o…            │
+                                                        ╰──────────────────────╯
                                                                                 
                                                                                 
                                                                                 
@@ -871,23 +871,23 @@ exports[`stories catalog ReviewPanel/ExpandedNarrow renders, snapshots, and carr
                                                                                 
                                                                                 
                                                                                 
-                                                         >         Submit review
+                                                        >          Submit review
                                                                                 
 "
 `;
 
 exports[`stories catalog ReviewPanel/ExpandedWide renders, snapshots, and carries its colors: ReviewPanel/ExpandedWide 1`] = `
-" The plan takes the width the panel      ╭───────────────────────────────────────────────╮
- hands back.                             │ Review  Agent                                 │
-                                         ╰───────────────────────────────────────────────╯
-                                         ╭─ COMMENT · me ────────────────────────────────╮
-                                         │ "persists sessions"                           │
-                                         │ Which daemon owns this?                       │
-                                         ╰───────────────────────────────────────────────╯
-                                         ╭─ COMMENT · me ────────────────────────────────╮
-                                         │ "move the store"                              │
-                                         │ move the store behind o…                      │
-                                         ╰───────────────────────────────────────────────╯
+" The plan takes the width the panel     ╭────────────────────────────────────────────────╮
+ hands back.                            │ Review  Agent                                  │
+                                        ╰────────────────────────────────────────────────╯
+                                        ╭─ COMMENT · me ─────────────────────────────────╮
+                                        │ "persists sessions"                            │
+                                        │ Which daemon owns this?                        │
+                                        ╰────────────────────────────────────────────────╯
+                                        ╭─ COMMENT · me ─────────────────────────────────╮
+                                        │ "move the store"                               │
+                                        │ move the store behind o…                       │
+                                        ╰────────────────────────────────────────────────╯
                                                                                           
                                                                                           
                                                                                           
@@ -899,7 +899,7 @@ exports[`stories catalog ReviewPanel/ExpandedWide renders, snapshots, and carrie
                                                                                           
                                                                                           
                                                                                           
-                                         >                                   Submit review
+                                        >                                    Submit review
                                                                                           
 "
 `;
@@ -933,45 +933,45 @@ exports[`stories catalog ReviewPanel/Hidden renders, snapshots, and carries its
 `;
 
 exports[`stories catalog ReviewRail/AgentTab renders, snapshots, and carries its colors: ReviewRail/AgentTab 1`] = `
-" ╭───────────────────────────────╮      
- │ Review  Agent                 │      
- ╰───────────────────────────────╯      
+"╭────────────────────────────────╮      
+│ Review  Agent                  │      
+╰────────────────────────────────╯      
+╭────────────────────────────────╮      
+│ Claude Code                    │      
+╰────────────────────────────────╯      
+╭────────────────────────────────╮      
+│ Pi                             │      
+╰────────────────────────────────╯      
+╭────────────────────────────────╮      
+│ OpenAI Codex                   │      
+╰────────────────────────────────╯      
+plan context: seed the plan             
+                                        
+                                        
+                                        
+                                        
+                                        
+                                        
+                                        
                                         
-   Ask an agent about this plan         
-   ╭─────────────────────────────╮      
-   │  ▛████▜   claude code       │      
-   │ ▜██████▛  launch in rail ▸  │      
-   │  ▝▝  ▝▝                     │      
-   ╰─────────────────────────────╯      
                                         
-   ╭─────────────────────────────╮      
-   │ ███       pi                │      
-   │ █ █       launch in rail ▸  │      
-   │ ██ █                        │      
-   │ █  █                        │      
-   ╰─────────────────────────────╯      
                                         
-   ╭─────────────────────────────╮      
-   │ ┌───┐     codex             │      
-   │ │>_ │     launch in rail ▸  │      
-   │ └───┘                       │      
-   ╰─────────────────────────────╯      
                                         
 "
 `;
 
 exports[`stories catalog ReviewRail/AnnotationStack renders, snapshots, and carries its colors: ReviewRail/AnnotationStack 1`] = `
-" ╭───────────────────────────────╮      
- │ Review  Agent                 │      
- ╰───────────────────────────────╯      
- ╭─ COMMENT · me ────────────────╮      
- │ "persists sessions"           │      
- │ Which daemon owns this?       │      
- ╰───────────────────────────────╯      
- ╭─ COMMENT · me ────────────────╮      
- │ "move the store"              │      
- │ move the store behind o…      │      
- ╰───────────────────────────────╯      
+"╭────────────────────────────────╮      
+│ Review  Agent                  │      
+╰────────────────────────────────╯      
+╭─ COMMENT · me ─────────────────╮      
+│ "persists sessions"            │      
+│ Which daemon owns this?        │      
+╰────────────────────────────────╯      
+╭─ COMMENT · me ─────────────────╮      
+│ "move the store"               │      
+│ move the store behind o…       │      
+╰────────────────────────────────╯      
                                         
                                         
                                         
@@ -983,26 +983,26 @@ exports[`stories catalog ReviewRail/AnnotationStack renders, snapshots, and carr
                                         
                                         
                                         
- Submit review                          
+Submit review                           
                                         
 "
 `;
 
 exports[`stories catalog ReviewRail/InterleavedStack renders, snapshots, and carries its colors: ReviewRail/InterleavedStack 1`] = `
-" ╭───────────────────────────────╮      
- │ Review  Agent                 │      
- ╰───────────────────────────────╯      
- ╭─ CUT · me ────────────────────╮      
- │ - add recovery                │      
- ╰───────────────────────────────╯      
- ╭─ COMMENT · me ────────────────╮      
- │ "persists sessions"           │      
- │ Which daemon owns this?       │      
- ╰───────────────────────────────╯      
- ╭─ COMMENT · me ────────────────╮      
- │ "move the store"              │      
- │ move the store behind o…      │      
- ╰───────────────────────────────╯      
+"╭────────────────────────────────╮      
+│ Review  Agent                  │      
+╰────────────────────────────────╯      
+╭─ CUT · me ─────────────────────╮      
+│ - add recovery                 │      
+╰────────────────────────────────╯      
+╭─ COMMENT · me ─────────────────╮      
+│ "persists sessions"            │      
+│ Which daemon owns this?        │      
+╰────────────────────────────────╯      
+╭─ COMMENT · me ─────────────────╮      
+│ "move the store"               │      
+│ move the store behind o…       │      
+╰────────────────────────────────╯      
                                         
                                         
                                         
@@ -1011,23 +1011,23 @@ exports[`stories catalog ReviewRail/InterleavedStack renders, snapshots, and car
                                         
                                         
                                         
- Submit review                          
+Submit review                           
                                         
 "
 `;
 
 exports[`stories catalog ReviewRail/MixedOwnershipStack renders, snapshots, and carries its colors: ReviewRail/MixedOwnershipStack 1`] = `
-" ╭───────────────────────────────╮      
- │ Review  Agent                 │      
- ╰───────────────────────────────╯      
- ╭─ COMMENT · me ────────────────╮      
- │ "persists sessions"           │      
- │ Which daemon owns this?       │      
- ╰───────────────────────────────╯      
- ╭─ COMMENT · Priya ─────────────╮      
- │ "move the store"              │      
- │ move it behind one inte…      │      
- ╰───────────────────────────────╯      
+"╭────────────────────────────────╮      
+│ Review  Agent                  │      
+╰────────────────────────────────╯      
+╭─ COMMENT · me ─────────────────╮      
+│ "persists sessions"            │      
+│ Which daemon owns this?        │      
+╰────────────────────────────────╯      
+╭─ COMMENT · Priya ──────────────╮      
+│ "move the store"               │      
+│ move it behind one inte…       │      
+╰────────────────────────────────╯      
                                         
                                         
                                         
@@ -1039,25 +1039,25 @@ exports[`stories catalog ReviewRail/MixedOwnershipStack renders, snapshots, and
                                         
                                         
                                         
- Submit review                          
+Submit review                           
                                         
 "
 `;
 
 exports[`stories catalog ReviewRail/Removals renders, snapshots, and carries its colors: ReviewRail/Removals 1`] = `
-" ╭───────────────────────────────╮      
- │ Review  Agent                 │      
- ╰───────────────────────────────╯      
- ╭─ REJECT · me ─────────────────╮      
- │ -   private items = [];       │      
- │ +   private items = new…      │      
- │  undo                         │      
- ╰───────────────────────────────╯      
- ╭─ REJECT · me ─────────────────╮      
- │ -   return null;              │      
- │ +   return fallback();        │      
- │ +   log(fallback);            │      
- ╰───────────────────────────────╯      
+"╭────────────────────────────────╮      
+│ Review  Agent                  │      
+╰────────────────────────────────╯      
+╭─ REJECT · me ──────────────────╮      
+│ -   private items = [];        │      
+│ +   private items = new…       │      
+│  undo                          │      
+╰────────────────────────────────╯      
+╭─ REJECT · me ──────────────────╮      
+│ -   return null;               │      
+│ +   return fallback();         │      
+│ +   log(fallback);             │      
+╰────────────────────────────────╯      
                                         
                                         
                                         
@@ -1067,23 +1067,23 @@ exports[`stories catalog ReviewRail/Removals renders, snapshots, and carries its
                                         
                                         
                                         
- Submit review                          
+Submit review                           
                                         
 "
 `;
 
 exports[`stories catalog ReviewRail/Resolved renders, snapshots, and carries its colors: ReviewRail/Resolved 1`] = `
-" ╭───────────────────────────────╮      
- │ Review  Agent                 │      
- ╰───────────────────────────────╯      
- ╭─ COMMENT · me · ORPHANED ─────╮      
- │ "persists sessions"           │      
- │ Which daemon owns this?       │      
- ╰───────────────────────────────╯      
- ╭─ COMMENT · me · ORPHANED ─────╮      
- │ "move the store"              │      
- │ move the store behind o…      │      
- ╰───────────────────────────────╯      
+"╭────────────────────────────────╮      
+│ Review  Agent                  │      
+╰────────────────────────────────╯      
+╭─ COMMENT · me · ORPHANED ──────╮      
+│ "persists sessions"            │      
+│ Which daemon owns this?        │      
+╰────────────────────────────────╯      
+╭─ COMMENT · me · ORPHANED ──────╮      
+│ "move the store"               │      
+│ move the store behind o…       │      
+╰────────────────────────────────╯      
                                         
                                         
                                         
@@ -1095,23 +1095,23 @@ exports[`stories catalog ReviewRail/Resolved renders, snapshots, and carries its
                                         
                                         
                                         
- resolved: approve                      
+resolved: approve                       
                                         
 "
 `;
 
 exports[`stories catalog ReviewRail/SubmitConfirmOpen renders, snapshots, and carries its colors: ReviewRail/SubmitConfirmOpen 1`] = `
-" ╭───────────────────────────────╮      
- │ Review  Agent                 │      
- ╰───────────────────────────────╯      
- ╭─ COMMENT · me · ORPHANED ─────╮      
- │ "persists sessions"           │      
- │ Which daemon owns this?       │      
- ╰───────────────────────────────╯      
- ╭─ COMMENT · me · ORPHANED ─────╮      
- │ "move the store"              │      
- │ move the store behind o…      │      
- ╰───────────────────────────────╯      
+"╭────────────────────────────────╮      
+│ Review  Agent                  │      
+╰────────────────────────────────╯      
+╭─ COMMENT · me · ORPHANED ──────╮      
+│ "persists sessions"            │      
+│ Which daemon owns this?        │      
+╰────────────────────────────────╯      
+╭─ COMMENT · me · ORPHANED ──────╮      
+│ "move the store"               │      
+│ move the store behind o…       │      
+╰────────────────────────────────╯      
                                         
                                         
                                         
@@ -1121,15 +1121,15 @@ exports[`stories catalog ReviewRail/SubmitConfirmOpen renders, snapshots, and ca
                                         
                                         
                                         
- ╭─ submit review ──────────────╮       
- │ 2 annotations · 0 blocking   │       
- │                              │       
- │  Comment   Approve [Changes] │       
- │                              │       
- │ summary for the agent (option│       
- │                              │       
- │  Submit    Cancel            │       
- ╰──────────────────────────────╯       
+╭─ submit review ───────────────╮       
+│ 2 annotations · 0 blocking    │       
+│                               │       
+│  Comment   Approve  [Changes] │       
+│                               │       
+│ summary for the agent (optiona│       
+│                               │       
+│  Submit    Cancel             │       
+╰───────────────────────────────╯       
                                         
 "
 `;
@@ -1393,30 +1393,30 @@ exports[`stories catalog WalkWizard/WithAgentNote renders, snapshots, and carrie
 `;
 
 exports[`stories catalog AgentLauncher/Launcher renders, snapshots, and carries its colors: AgentLauncher/Launcher 1`] = `
-"  Ask an agent about this plan          
-                                        
-  ╭────────────────────────────────────╮
-  │  ▛████▜   claude code              │
-  │ ▜██████▛  launch in rail ▸         │
-  │  ▝▝  ▝▝                            │
+"  ╭────────────────────────────────────╮
+  │ Claude Code                        │
   ╰────────────────────────────────────╯
-                                        
   ╭────────────────────────────────────╮
-  │ ███       pi                       │
-  │ █ █       launch in rail ▸         │
-  │ ██ █                               │
-  │ █  █                               │
+  │ Pi                                 │
   ╰────────────────────────────────────╯
-                                        
   ╭────────────────────────────────────╮
-  │ ┌───┐     codex                    │
-  │ │>_ │     launch in rail ▸         │
-  │ └───┘                              │
+  │ OpenAI Codex                       │
   ╰────────────────────────────────────╯
+  plan context: seed the plan           
+                                        
+                                        
+                                        
+                                        
+                                        
+                                        
+                                        
+                                        
+                                        
+                                        
+                                        
+                                        
                                         
-  plan context: seed the plan ▸         
                                         
-  agent/worker-3 · pending · rev 1      
 "
 `;
 

package/src/components/agent-launcher.stories.tsx

@@ -21,6 +21,6 @@ export const Launcher: Story = {
       <AgentLauncher session={fixturePlanSession()} onLaunchHarness={() => {}} theme={DARK} />
     </RailFrame>
   ),
-  expectedColors: [DARK.accent, "#cc785c"],
+  expectedColors: [DARK.accent],
   size: { width: 40, height: 24 },
 };

package/src/components/agent-launcher.test.tsx

@@ -1,5 +1,10 @@
 import { describe, expect, test } from "bun:test";
-import { HARNESS_LAUNCHERS, planHandoffBriefing } from "./agent-launcher";
+import React from "react";
+import { testRender } from "@opentui/react/test-utils";
+import { DARK } from "../theme";
+import { fixturePlanSession } from "./story-fixtures";
+import { waitForText } from "../test-support";
+import { AgentLauncher, HARNESS_LAUNCHERS, planHandoffBriefing } from "./agent-launcher";
 
 describe("planHandoffBriefing", () => {
   test("references the session by id and directs the agent to read then annotate, not rewrite", () => {
@@ -14,8 +19,31 @@ describe("planHandoffBriefing", () => {
 });
 
 describe("HARNESS_LAUNCHERS", () => {
-  test("carries the three branded harnesses with their launch commands", () => {
-    // Assert
-    expect(HARNESS_LAUNCHERS.map((harness) => harness.command)).toEqual(["cc", "pi", "codex"]);
+  test("carries the three harnesses with real binary commands (never the `cc` alias)", () => {
+    // Assert - `cc` resolves to the system C compiler when spawned on a PTY, so
+    // the Claude harness must run `claude`, the real binary, not the shell alias
+    expect(HARNESS_LAUNCHERS.map((harness) => harness.command)).toEqual(["claude", "pi", "codex"]);
+  });
+});
+
+describe("AgentLauncher", () => {
+  test("renders a button for every harness, in order", async () => {
+    // Arrange
+    const setup = await testRender(
+      <box style={{ width: 44, height: 20, flexDirection: "column" }}>
+        <AgentLauncher session={fixturePlanSession()} onLaunchHarness={() => {}} theme={DARK} />
+      </box>,
+      { width: 44, height: 20 },
+    );
+
+    // Act
+    await waitForText(setup, "Claude Code");
+    const frame = setup.captureCharFrame();
+
+    // Assert - all three launcher names present, in launch order
+    const names = HARNESS_LAUNCHERS.map((harness) => harness.name);
+    const positions = names.map((name) => frame.indexOf(name));
+    expect(positions.every((position) => position >= 0)).toBe(true);
+    expect(positions).toEqual(positions.toSorted((left, right) => left - right));
   });
 });

package/src/components/agent-launcher.tsx

@@ -24,38 +24,24 @@ export interface AgentTerminalHandle {
   detach: () => void;
 }
 
-/** One launchable harness: its rail command and the real logo, rendered as colored rows. */
+/** One launchable harness: its display name and the rail command. */
 export interface HarnessLauncher {
   id: string;
   name: string;
   /** The shell command that starts the harness, run in the split. */
   command: string;
-  /** Brand color for the mark. */
-  color: string;
-  /** Multi-row mark rendered from the real logo. */
-  logo: string[];
 }
 
-/** Claude Code's brand coral - not a theme token, the mark keeps its own color. */
-const CLAUDE_CORAL = "#CC785C";
+/** Off-white for the running-terminal header, shared by every harness. */
+const HARNESS_HEADER_COLOR = "#e4e6ec";
 
-/** The bring-your-own harnesses, with their real marks (finalized in the prototype). */
+/** The bring-your-own harnesses, in launch order. The command is the real binary
+ *  name spawned on a PTY (not a shell alias, so `claude` - never `cc`, which is
+ *  the system C compiler). */
 export const HARNESS_LAUNCHERS: HarnessLauncher[] = [
-  {
-    id: "claude",
-    name: "claude code",
-    command: "cc",
-    color: CLAUDE_CORAL,
-    logo: [" ▛████▜", "▜██████▛", " ▝▝  ▝▝"],
-  },
-  { id: "pi", name: "pi", command: "pi", color: "#e4e6ec", logo: ["███ ", "█ █ ", "██ █", "█  █"] },
-  {
-    id: "codex",
-    name: "codex",
-    command: "codex",
-    color: "#e4e6ec",
-    logo: ["┌───┐", "│>_ │", "└───┘"],
-  },
+  { id: "claude", name: "Claude Code", command: "claude" },
+  { id: "pi", name: "Pi", command: "pi" },
+  { id: "codex", name: "OpenAI Codex", command: "codex" },
 ];
 
 /** The briefing typed into a launched harness when the plan-context toggle is on. */
@@ -63,18 +49,6 @@ export function planHandoffBriefing(sessionId: string): string {
   return `Review this cueloop plan: read it with 'cueloop session get ${sessionId}', then comment with 'cueloop session annotate ${sessionId} --author me --quote "<exact span>" --body "<comment>"'. Do not rewrite the plan.`;
 }
 
-function LogoMark({ harness }: { harness: HarnessLauncher }): React.ReactNode {
-  return (
-    <box style={{ width: 9, flexShrink: 0, flexDirection: "column", alignItems: "flex-start" }}>
-      {harness.logo.map((row) => (
-        <text key={row} fg={harness.color}>
-          {row}
-        </text>
-      ))}
-    </box>
-  );
-}
-
 /** Props for the Agent tab body: the session under review plus the launch callbacks. */
 export interface AgentLauncherProps {
   session: ReviewSession;
@@ -110,7 +84,12 @@ export function AgentLauncher({
     if (embeddedTerminalAvailable()) setRunning({ harness, seed });
     else onLaunchHarness(harness.command, seed);
   };
-  const detach = (): void => setRunning(null);
+  const detach = (): void => {
+    // The reconciler's removeChild detaches the pane without destroying it, so
+    // kill the child + free the VT here or they leak. shutdown() is idempotent.
+    paneRef.current?.shutdown();
+    setRunning(null);
+  };
 
   useEffect(() => {
     if (!running) return onAgentTerminal?.(null);
@@ -123,9 +102,9 @@ export function AgentLauncher({
     return (
       <box style={{ flexDirection: "column", flexGrow: 1 }}>
         <box style={{ flexDirection: "row" }} onMouseUp={detach}>
-          <text fg={running.harness.color}>{running.harness.name}</text>
+          <text fg={HARNESS_HEADER_COLOR}>{running.harness.name}</text>
           <box style={{ flexGrow: 1 }} />
-          <text fg={tokens.textDim}>✕ detach (⌃])</text>
+          <text fg={tokens.textDim}>✕ detach</text>
         </box>
         {React.createElement("terminalPane", {
           ref: paneRef,
@@ -141,40 +120,28 @@ export function AgentLauncher({
 
   return (
     <box style={{ flexDirection: "column", flexGrow: 1 }}>
-      <text fg={tokens.text}>Ask an agent about this plan</text>
-      <box style={{ height: 1 }} />
       {HARNESS_LAUNCHERS.map((harness) => (
         <box
           key={harness.id}
           style={{
             flexDirection: "row",
-            height: harness.logo.length + 2,
+            height: 3,
             border: true,
             borderStyle: FRAME_BORDER_STYLE,
             backgroundColor: "transparent",
-            borderColor: tokens.border,
-            marginBottom: 1,
+            borderColor: tokens.text,
             paddingLeft: 1,
             paddingRight: 1,
           }}
           onMouseUp={() => launch(harness)}
         >
-          <LogoMark harness={harness} />
-          <box style={{ flexDirection: "column", flexGrow: 1, paddingLeft: 1 }}>
-            <text fg={tokens.textMuted}>{harness.name}</text>
-            <text fg={tokens.textDim}>launch in rail ▸</text>
-          </box>
+          <text fg={tokens.textMuted}>{harness.name}</text>
         </box>
       ))}
       <box style={{ flexDirection: "row" }} onMouseUp={() => setSeedContext((on) => !on)}>
         <text fg={tokens.textDim}>plan context: </text>
-        <text fg={tokens.accent}>{seedContext ? "seed the plan ▸" : "none ▸"}</text>
+        <text fg={tokens.accent}>{seedContext ? "seed the plan" : "none"}</text>
       </box>
-      <box style={{ flexGrow: 1 }} />
-      <text fg={tokens.textDim}>
-        {session.artifact.meta.agent ?? "unknown"} · {session.status} · rev{" "}
-        {session.revisions.length}
-      </text>
     </box>
   );
 }

package/src/components/primitives/Button.tsx

@@ -5,6 +5,7 @@
  */
 
 import React from "react";
+import type { MouseEvent } from "@opentui/core";
 import type { Theme } from "../../theme";
 import { useComponentTheme } from "../theme-context";
 
@@ -38,7 +39,9 @@ export function Button({
   return (
     <box
       style={{ backgroundColor, marginRight }}
-      onMouseUp={() => {
+      onMouseUp={(event: MouseEvent) => {
+        // Consume the press so it never bubbles to an ancestor's onMouseUp (e.g. a card).
+        event.stopPropagation();
         if (!isDisabled) onPress();
       }}
     >