@cueloop/client 0.1.0-alpha.102 → 0.1.0-alpha.103

package/src/thread/components/AgentThreadPane.test.tsx

@@ -43,6 +43,33 @@ const empty: ThreadAgentState = {
   comments: [],
 };
 
+test("a failed harness turn displays its error below the conversation", async () => {
+  const failure = "OAuth auth derivation failed: missing provider module";
+  const { client } = createTestAgentClient({ ...empty, phase: { kind: "failed", error: failure } });
+  const setup = await testRender(
+    <AgentThreadPane
+      thread={thread}
+      client={client}
+      focused
+      theme={DARK}
+      onActiveChange={noop}
+      onOpenFile={noop}
+    >
+      {artifactView(thread)}
+    </AgentThreadPane>,
+    { width: 100, height: 24 },
+  );
+
+  try {
+    await waitForText(setup, "Original artifact");
+    await settle(setup);
+    expect(setup.captureCharFrame()).toContain(failure);
+    expect(setup.captureCharFrame()).not.toContain("Thinking");
+  } finally {
+    setup.renderer.destroy();
+  }
+});
+
 function createTestAgentClient(initial: ThreadAgentState) {
   let state = initial;
   const prompts: AgentPromptRequest[] = [];
@@ -56,6 +83,7 @@ function createTestAgentClient(initial: ThreadAgentState) {
     agentGet: async () => state,
     agentPrompt: async (params) => {
       prompts.push(params);
+
       if (params.text)
         state = {
           ...state,
@@ -287,6 +315,7 @@ test("three bottom prompts survive delayed acceptance and remain separate submis
 
   client.agentPrompt = async (params) => {
     prompts.push(params.text);
+
     if (prompts.length === 1)
       return new Promise((resolve) => {
         release = resolve;
@@ -868,6 +897,7 @@ test("an arriving reply preserves a bottom draft and submits it as the next prom
 
   client.agentPrompt = async (params) => {
     prompts.push(params);
+
     if (prompts.length === 1)
       return new Promise<ThreadAgentState>((resolve) => {
         finish = resolve;
@@ -1038,6 +1068,7 @@ test("a failed mutation flush restores the prompt alongside a newer visible draf
       onOpenFile={noop}
       flushMutations={() => {
         if (flushStarted) return Promise.resolve();
+
         flushStarted = true;
 
         return failedFlush;

package/src/thread/components/AgentThreadPane.tsx

@@ -80,6 +80,7 @@ export function AgentThreadPane(props: AgentThreadPaneProps): React.ReactNode {
 
   useEffect(() => {
     if (!busy) return;
+
     const timer = setInterval(() => setPulse((value) => !value), 600);
 
     return () => clearInterval(timer);
@@ -119,7 +120,9 @@ export function AgentThreadPane(props: AgentThreadPaneProps): React.ReactNode {
 
   const invoke = async (commentId?: string): Promise<void> => {
     if (thread.status !== "pending") return;
+
     if (!commentId && !draft.current.trim()) return;
+
     invocations.current.push({
       text: draft.current,
       commentId: commentId
@@ -128,7 +131,9 @@ export function AgentThreadPane(props: AgentThreadPaneProps): React.ReactNode {
       writes: pendingWrites.current.splice(0),
     });
     draft.current = "";
+
     if (invoking.current) return;
+
     invoking.current = true;
 
     const rejected: string[] = [];
@@ -144,6 +149,7 @@ export function AgentThreadPane(props: AgentThreadPaneProps): React.ReactNode {
             rejected.push(input.text);
             continue;
           }
+
           await props.flushMutations?.();
           const accepted = await agent.act((client) =>
             client.agentPrompt({
@@ -219,6 +225,7 @@ export function AgentThreadPane(props: AgentThreadPaneProps): React.ReactNode {
     );
 
     if (!submission) return undefined;
+
     if (submission.status === "failed" && props.client?.canControlAgent !== false)
       return {
         label: "Retry",
@@ -293,6 +300,7 @@ export function AgentThreadPane(props: AgentThreadPaneProps): React.ReactNode {
 
           return;
         }
+
         if (source?.kind === "mirror") {
           const mirrorId = projection.marks.get(span.start.blockIndex)?.[0]?.annotationId;
 
@@ -300,15 +308,18 @@ export function AgentThreadPane(props: AgentThreadPaneProps): React.ReactNode {
 
           return;
         }
+
         if (source?.kind === "artifact" && end?.kind === "artifact") {
           return child?.onAnnotate(span, body);
         }
+
         if (
           source?.kind !== "message" ||
           end?.kind !== "message" ||
           source.messageId !== end.messageId
         )
           return;
+
         const message = state.messages.find((entry) => entry.id === source.messageId)!;
         const localState = { ...state, messages: [message] };
         const comment = commentOnAgentSpan(
@@ -347,6 +358,7 @@ export function AgentThreadPane(props: AgentThreadPaneProps): React.ReactNode {
             {state.phase.kind === "offline" ? (
               <text fg={theme.textMuted}>Owner offline</text>
             ) : null}
+            {state.phase.kind === "failed" ? <text fg={theme.red}>{state.phase.error}</text> : null}
             {busy ? <text fg={pulse ? theme.textDim : theme.textMuted}>Thinking…</text> : null}
             {state.phase.kind === "permission" && props.client?.canControlAgent !== false ? (
               <>

package/src/thread/components/ThreadTree.tsx

@@ -116,6 +116,7 @@ function ThreadRow(props: ThreadRowProps): React.ReactNode {
 
   useEffect(() => {
     if (!menuOpen) return;
+
     // the full-screen box closes the menu on an outside click; the menu drops from under the kebab
     setOverlay(
       "thread-menu",

package/src/thread/components/thread-tree.test.ts

@@ -6,6 +6,7 @@ function session(id: string, title: string, rootCommit?: string, remote?: string
   const workspace: WorkspaceKey = { repoRoot: `/home/dev/${id}-checkout`, branch: "main" };
 
   if (rootCommit !== undefined) workspace.rootCommit = rootCommit;
+
   if (remote !== undefined) workspace.remote = remote;
 
   return {

package/src/thread/thread-controller.curate.test.ts

@@ -99,6 +99,7 @@ function fakeClient(initial: Thread, sink: WorkingCopySink): ThreadClient {
 
       sink.workingCopy = content;
       session = { ...session, workingCopy: content };
+
       if (content === undefined) delete session.workingCopy;
 
       return session;
@@ -111,6 +112,7 @@ function fakeClient(initial: Thread, sink: WorkingCopySink): ThreadClient {
 
       sink.workingCopy = content;
       session = { ...session, workingCopy: content };
+
       if (rejections.length) session.curation = rejections;
       else delete session.curation;