Add 'Ask agent to fix' for CI failures (#8)

* Add "Ask agent to fix" for CI failures, fix sendPrompt truncation

Replace chunked tmux send-keys with load-buffer/paste-buffer to avoid
flag interpretation errors (e.g. chunks starting with "-" parsed as
tmux flags). The preamble is typed via send-keys so it shows inline,
while logs are pasted via buffer (collapsed as [pasted text]).

Also adds --log with --log-failed fallback for the CI logs endpoint.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* Restore "done" status icon in AgentStatusIcon

The {:else if status === "done"} branch was accidentally removed during
the formatter refactor, leaving no icon rendered for completed agents.
Add back the green checkmark SVG between the "waiting" and "error" branches.

Co-authored-by: centdix <centdix@users.noreply.github.com>

* Use unique tmux buffer name per sendPrompt invocation

Hardcoded "wm-prompt" buffer could be overwritten by a concurrent
sendPrompt call before paste-buffer runs. Use a timestamp+random suffix
(e.g. wm-prompt-1234567890-abc12) to avoid the race.

Co-authored-by: centdix <centdix@users.noreply.github.com>

* Cap CI logs sent to agent at 30000 chars

Large CI outputs (verbose test suites, etc.) could overwhelm the tmux
paste buffer or the agent context. Truncate from the start, keeping the
last 30000 chars (most relevant tail), with a [... truncated] marker.

Co-authored-by: centdix <centdix@users.noreply.github.com>

* Always use --log-failed for CI log fetching

--log returns logs for all jobs (including passing ones), producing far
more data than needed. Use --log-failed exclusively to stay focused on
failing job output.

Co-authored-by: centdix <centdix@users.noreply.github.com>

* Nits: blank line between functions, defensive colon split

- Add missing blank line between handleFix and handleCopy
- Use indexOf(":") in findWorktreeSession so session/window names
  containing colons don't silently mis-parse

Co-authored-by: centdix <centdix@users.noreply.github.com>

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com>
Co-authored-by: centdix <centdix@users.noreply.github.com>

Author: 3 people

backend/src/server.ts

@@ -285,10 +285,10 @@ async function handleApi(req: Request, url: URL): Promise<Response> {
     // POST /api/worktrees/:name/send
     if (parts[0] === "worktrees" && parts.length === 3 && parts[2] === "send" && method === "POST") {
       const name = decodeURIComponent(parts[1]);
-      const body = await req.json() as { text?: string };
+      const body = await req.json() as { text?: string; preamble?: string };
       if (!body.text) return errorResponse("Missing 'text' field", 400);
       console.log(`[worktree:send] name=${name} text="${body.text.slice(0, 80)}"`);
-      const result = sendPrompt(name, body.text);
+      const result = sendPrompt(name, body.text, 0, body.preamble);
       if (!result.ok) return errorResponse(result.error, 404);
       return jsonResponse({ ok: true });
     }
@@ -310,12 +310,14 @@ async function handleApi(req: Request, url: URL): Promise<Response> {
         stdout: "pipe",
         stderr: "pipe",
       });
-      const logs = new TextDecoder().decode(result.stdout);
-      if (result.exitCode !== 0) {
-        const stderr = new TextDecoder().decode(result.stderr).trim();
-        return errorResponse(`Failed to fetch logs: ${stderr}`, 502);
+
+      if (result.exitCode === 0) {
+        const logs = new TextDecoder().decode(result.stdout);
+        return jsonResponse({ logs });
       }
-      return jsonResponse({ logs });
+
+      const stderr = new TextDecoder().decode(result.stderr).trim();
+      return errorResponse(`Failed to fetch logs: ${stderr || "unknown error"}`, 502);
     }
 
     // GET /api/worktrees/:name/status

backend/src/workmux.ts

@@ -325,19 +325,83 @@ export async function removeWorktree(name: string): Promise<string> {
   return result;
 }
 
-export function sendPrompt(branch: string, text: string, pane = 0): { ok: true } | { ok: false; error: string } {
-  const session = `wm-${branch}`;
-  const check = Bun.spawnSync(["tmux", "has-session", "-t", session], { stderr: "pipe" });
-  if (check.exitCode !== 0) {
-    return { ok: false, error: `tmux session "${session}" not found` };
+export function sendPrompt(
+  branch: string,
+  text: string,
+  pane = 0,
+  preamble?: string,
+): { ok: true } | { ok: false; error: string } {
+  const windowName = `wm-${branch}`;
+  const session = findWorktreeSession(windowName);
+  if (!session) {
+    return { ok: false, error: `tmux window "${windowName}" not found` };
   }
-  const result = Bun.spawnSync(["tmux", "send-keys", "-t", `${session}.${pane}`, text, "Enter"]);
-  if (result.exitCode !== 0) {
-    return { ok: false, error: `send-keys failed (exit ${result.exitCode})` };
+  const target = `${session}:${windowName}.${pane}`;
+
+  // Type the preamble as regular keystrokes so it shows inline in the agent,
+  // then paste the bulk payload via a tmux buffer (appears as [pasted text]).
+  if (preamble) {
+    const pre = Bun.spawnSync(["tmux", "send-keys", "-t", target, "-l", "--", preamble], { stderr: "pipe" });
+    if (pre.exitCode !== 0) {
+      const stderr = new TextDecoder().decode(pre.stderr).trim();
+      return { ok: false, error: `send-keys preamble failed (exit ${pre.exitCode})${stderr ? `: ${stderr}` : ""}` };
+    }
   }
+
+  const cleaned = text.replace(/\0/g, "");
+
+  // Use a unique buffer name per invocation to avoid races when concurrent
+  // sendPrompt calls overlap (e.g. two worktrees sending at the same time).
+  const bufName = `wm-prompt-${Date.now()}-${Math.random().toString(36).slice(2, 7)}`;
+
+  // Load text into a named tmux buffer via stdin — avoids all send-keys
+  // escaping/chunking issues and handles any text size in a single operation.
+  const load = Bun.spawnSync(["tmux", "load-buffer", "-b", bufName, "-"], {
+    stdin: new TextEncoder().encode(cleaned),
+    stderr: "pipe",
+  });
+  if (load.exitCode !== 0) {
+    const stderr = new TextDecoder().decode(load.stderr).trim();
+    return { ok: false, error: `load-buffer failed (exit ${load.exitCode})${stderr ? `: ${stderr}` : ""}` };
+  }
+
+  // Paste buffer into target pane; -d deletes the buffer after pasting.
+  const paste = Bun.spawnSync(["tmux", "paste-buffer", "-b", bufName, "-t", target, "-d"], {
+    stderr: "pipe",
+  });
+  if (paste.exitCode !== 0) {
+    const stderr = new TextDecoder().decode(paste.stderr).trim();
+    return { ok: false, error: `paste-buffer failed (exit ${paste.exitCode})${stderr ? `: ${stderr}` : ""}` };
+  }
+
+  // Submit
+  const enter = Bun.spawnSync(["tmux", "send-keys", "-t", target, "Enter"], { stderr: "pipe" });
+  if (enter.exitCode !== 0) {
+    const stderr = new TextDecoder().decode(enter.stderr).trim();
+    return { ok: false, error: `send-keys Enter failed (exit ${enter.exitCode})${stderr ? `: ${stderr}` : ""}` };
+  }
+
   return { ok: true };
 }
 
+function findWorktreeSession(windowName: string): string | null {
+  const result = Bun.spawnSync(
+    ["tmux", "list-windows", "-a", "-F", "#{session_name}:#{window_name}"],
+    { stdout: "pipe", stderr: "pipe" }
+  );
+  if (result.exitCode !== 0) return null;
+  const output = new TextDecoder().decode(result.stdout).trim();
+  if (!output) return null;
+  for (const line of output.split("\n")) {
+    const colonIdx = line.indexOf(":");
+    if (colonIdx === -1) continue;
+    const session = line.slice(0, colonIdx);
+    const name = line.slice(colonIdx + 1);
+    if (name === windowName) return session;
+  }
+  return null;
+}
+
 export async function openWorktree(name: string): Promise<string> {
   return runChecked(["workmux", "open", name]);
 }

frontend/src/App.svelte

@@ -302,5 +302,9 @@
 {/if}
 
 {#if ciDetailsPr}
-  <CiDetailsDialog pr={ciDetailsPr} onclose={() => (ciDetailsPr = null)} />
+  <CiDetailsDialog
+    pr={ciDetailsPr}
+    branch={selectedWorktree?.branch ?? ""}
+    onclose={() => (ciDetailsPr = null)}
+  />
 {/if}

frontend/src/lib/AgentStatusIcon.svelte

@@ -1,5 +1,9 @@
 <script lang="ts">
-  let { status, size = 10, pill = false }: { status: string; size?: number; pill?: boolean } = $props();
+  let {
+    status,
+    size = 10,
+    pill = false,
+  }: { status: string; size?: number; pill?: boolean } = $props();
 
   function pillClass(s: string): string {
     if (s === "working") return "bg-success/15 text-success";
@@ -12,18 +16,68 @@
 
 {#snippet icon()}
   {#if status === "working"}
-    <span class="spinner text-success" style="width:{size}px;height:{size}px;border-width:1.5px;"></span>
+    <span
+      class="spinner text-success"
+      style="width:{size}px;height:{size}px;border-width:1.5px;"
+    ></span>
   {:else if status === "waiting"}
-    <svg class="text-warning" xmlns="http://www.w3.org/2000/svg" width={size} height={size} viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><path d="M21 15a2 2 0 0 1-2 2H7l-4 4V5a2 2 0 0 1 2-2h14a2 2 0 0 1 2 2z"/><path d="M12 7v2"/><path d="M12 13h.01"/></svg>
+    <svg
+      class="text-warning"
+      xmlns="http://www.w3.org/2000/svg"
+      width={size}
+      height={size}
+      viewBox="0 0 24 24"
+      fill="none"
+      stroke="currentColor"
+      stroke-width="2"
+      stroke-linecap="round"
+      stroke-linejoin="round"
+      ><path
+        d="M21 15a2 2 0 0 1-2 2H7l-4 4V5a2 2 0 0 1 2-2h14a2 2 0 0 1 2 2z"
+      /><path d="M12 7v2" /><path d="M12 13h.01" /></svg
+    >
   {:else if status === "done"}
-    <svg class="text-success" xmlns="http://www.w3.org/2000/svg" width={size} height={size} viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="3" stroke-linecap="round" stroke-linejoin="round"><polyline points="20 6 9 17 4 12"/></svg>
+    <svg
+      class="text-success"
+      xmlns="http://www.w3.org/2000/svg"
+      width={size}
+      height={size}
+      viewBox="0 0 24 24"
+      fill="none"
+      stroke="currentColor"
+      stroke-width="3"
+      stroke-linecap="round"
+      stroke-linejoin="round"
+      ><polyline points="20 6 9 17 4 12" /></svg
+    >
   {:else if status === "error"}
-    <svg class="text-danger" xmlns="http://www.w3.org/2000/svg" width={size} height={size} viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="3" stroke-linecap="round" stroke-linejoin="round"><line x1="18" y1="6" x2="6" y2="18"/><line x1="6" y1="6" x2="18" y2="18"/></svg>
+    <svg
+      class="text-danger"
+      xmlns="http://www.w3.org/2000/svg"
+      width={size}
+      height={size}
+      viewBox="0 0 24 24"
+      fill="none"
+      stroke="currentColor"
+      stroke-width="3"
+      stroke-linecap="round"
+      stroke-linejoin="round"
+      ><line x1="18" y1="6" x2="6" y2="18" /><line
+        x1="6"
+        y1="6"
+        x2="18"
+        y2="18"
+      /></svg
+    >
   {/if}
 {/snippet}
 
 {#if pill}
-  <span class="text-xs px-2 py-0.5 rounded-xl flex items-center gap-1 {pillClass(status)}">
+  <span
+    class="text-xs px-2 py-0.5 rounded-xl flex items-center gap-1 {pillClass(
+      status,
+    )}"
+  >
     {@render icon()}
     {status || "idle"}
   </span>