Add 'Ask agent to fix' for CI failures

[centdix]

Summary

• Adds an "Ask agent to fix" button in the CI details dialog that sends failing CI logs to the agent
• Fixes sendPrompt truncation by replacing chunked tmux send-keys -l with load-buffer/paste-buffer (chunks starting with - were parsed as tmux flags)
• Preamble is typed via send-keys (visible inline), logs are pasted via buffer (collapsed as [pasted text])
• CI logs endpoint now tries --log first with --log-failed fallback

Test plan

• Open a worktree with a failing CI check
• Click "View logs" then "Ask agent to fix"
• Confirm no send-keys errors in backend terminal
• Confirm the preamble shows inline and logs appear as [pasted text] in the agent

🤖 Generated with Claude Code

[claude]

Claude finished @centdix's task in 1m 36s —— View job

---

Code Review: Add "Ask agent to fix" for CI failures

• Read repository setup instructions (CLAUDE.md)
• Analyze PR diff and changed files
• Review backend changes (server.ts, workmux.ts)
• Review frontend changes (App.svelte, AgentStatusIcon.svelte, CiDetailsDialog.svelte, api.ts)
• Post review findings

---

Overall this is a well-structured PR. The load-buffer/paste-buffer approach is a solid fix for the send-keys truncation issue, and the preamble/payload split is a nice UX touch. A few issues worth addressing:

Bug

Dropped "done" icon in AgentStatusIcon.svelte — The formatter refactor accidentally removed the {:else if status === "done"} branch that rendered a green checkmark SVG. The pillClass function still handles "done" (returns success colors), but no icon will render when status === "done". This looks like an unintentional deletion during the formatting cleanup.

Fix this →

Potential Issues

Race condition on shared tmux buffer name (workmux.ts:355) — The buffer name wm-prompt is hardcoded. If two concurrent sendPrompt calls happen (e.g., two different worktrees), the second load-buffer could overwrite the first's buffer before paste-buffer runs. Consider using a unique buffer name per invocation (e.g., wm-prompt-${branch} or a random suffix).

No log size limit before sending to agent (CiDetailsDialog.svelte:78) — normalizeLogsForPrompt strips ANSI codes but doesn't truncate. CI logs can be very large (especially verbose test output). Sending megabytes of text to a tmux buffer could overwhelm the agent or hit paste limits. Consider truncating to the last N lines or capping at a max character count with an indicator like [... truncated].

--log fetches all job logs, not just failures (server.ts:309) — Switching to --log as the primary option returns logs for all jobs in a workflow run, not just failing ones. For workflows with many passing jobs, this could return significantly more data than needed. Consider keeping --log-failed as the primary and using --log as the fallback, or adding the --failed filter.

Nits

• CiDetailsDialog.svelte:90-91 — Missing blank line between handleFix and handleCopy functions (minor style inconsistency).
• findWorktreeSession (workmux.ts:391) — line.split(":") assumes neither the session name nor window name contain colons. This is fine in practice since wm-* names won't have colons, but a targeted split (e.g., line.split(":", 2) or indexOf) would be more defensive.

What looks good

• The load-buffer/paste-buffer approach is the correct fix — avoids all the send-keys escaping issues with large payloads.
• Good defensive measures: null byte stripping, -- separator for preamble send-keys, individual error handling per tmux operation.
• The ANSI stripping regex handles both CSI and Fe escape sequences well.
• Clean separation of preamble (visible inline) vs. logs (pasted as buffer) is good UX.
• findWorktreeSession properly decouples session lookup from the hardcoded wm-* convention.

[centdix]

@claude fix the issues, one commit per fix
for max log size use const = 30000 char
use --log-failed always, not --log

[claude]

Claude finished @centdix's task in 2m 37s —— View job

---

Fixed review issues — 5 commits pushed

• Read CLAUDE.md and relevant files
• Fix 1: Restore dropped "done" icon in AgentStatusIcon.svelte
• Fix 2: Fix race condition with hardcoded tmux buffer name in workmux.ts
• Fix 3: Add 30000 char log size limit in CiDetailsDialog.svelte
• Fix 4: Use --log-failed always in server.ts
• Fix 5: Nits (blank line, defensive split)

Summary of changes (one commit each):

Commit	Change
0b37a18	Restore "done" checkmark SVG in AgentStatusIcon — was missing after formatter refactor
a6981da	Unique tmux buffer name per sendPrompt call (wm-prompt-<timestamp>-<random>) to prevent concurrent overwrites
2cb7faf	Cap CI logs at MAX_LOG_CHARS = 30000, keeping the last 30k chars with [... truncated] prefix
422ac70	Use --log-failed exclusively (remove --log primary + fallback logic)
57a332e	Blank line between handleFix/handleCopy; use indexOf(":") in findWorktreeSession for correctness