fix(web): hide empty tool_call/tool_call_update status rows (#4621)

buildBlocks() in AssistantMessage filters a fixed list of internal status
labels before they reach StatusPill, but tool_call / tool_call_update are
not in it, so they fall through and render as raw, expandable status pills
with no tool name, input, output, or detail — looking like "a tool ran but
produced no output".

These are transient ACP markers: on the live SSE path the daemon already
normalizes them to `running` (TRANSIENT_ACP_STATUS_LABELS in
providers/daemon.ts), which buildBlocks() skips. The persisted-events path
does not normalize, so the bare labels survive into the UI. Add both labels
to the existing skip list, with a comment pointing at the daemon source of
truth.

Add a red-spec test asserting the two empty status rows render no pill, plus
a guard that status rows carrying a real detail still render.

Author: thatditsyboy

apps/web/src/components/AssistantMessage.tsx

@@ -4004,7 +4004,15 @@ function buildBlocks(events: AgentEvent[]): Block[] {
         ev.label === "running" ||
         ev.label === "requesting" ||
         ev.label === "thinking" ||
-        ev.label === "empty_response"
+        ev.label === "empty_response" ||
+        // Transient ACP tool-call markers (#4618). On the live SSE path the
+        // daemon normalizes these to `running` (TRANSIENT_ACP_STATUS_LABELS in
+        // providers/daemon.ts), which is already skipped above; the persisted-
+        // events path does not normalize, so they arrive here as bare labels
+        // with no tool name/input/output/detail and would otherwise render as
+        // empty, expandable "tool ran but produced no output" status pills.
+        ev.label === "tool_call" ||
+        ev.label === "tool_call_update"
       )
         continue;
       const last = out[out.length - 1];

apps/web/tests/components/assistant-message-tool-status.test.tsx

@@ -451,6 +451,44 @@ describe('AssistantMessage tool status', () => {
     expect(screen.queryByTestId('task-activity-terminal')).toBeNull();
   });
 
+  it('hides empty tool_call / tool_call_update status rows (no displayable detail) (#4618)', () => {
+    const { container } = render(
+      <AssistantMessage
+        projectKind="prototype"
+        conversationId="conv-1"
+        message={messageWithEvents([
+          { kind: 'status', label: 'tool_call' },
+          { kind: 'status', label: 'tool_call_update' },
+        ])}
+        streaming={false}
+        projectId="project-1"
+      />,
+    );
+
+    // These persisted ACP markers carry no tool name/input/output, so they must
+    // not surface as empty, expandable status pills.
+    expect(container.querySelector('[data-status="tool_call"]')).toBeNull();
+    expect(container.querySelector('[data-status="tool_call_update"]')).toBeNull();
+    expect(container.querySelector('.status-pill')).toBeNull();
+  });
+
+  it('still renders status rows that carry a displayable detail', () => {
+    const { container } = render(
+      <AssistantMessage
+        projectKind="prototype"
+        conversationId="conv-1"
+        message={messageWithEvents([
+          { kind: 'status', label: 'model', detail: 'claude-opus-4-7-high' },
+        ])}
+        streaming={false}
+        projectId="project-1"
+      />,
+    );
+
+    expect(container.querySelector('[data-status="model"]')).not.toBeNull();
+    expect(container.querySelector('.status-detail')?.textContent).toContain('claude-opus-4-7-high');
+  });
+
   it('renders URLs in JSON-like status details without trailing structural characters', () => {
     const { container } = render(
       <AssistantMessage