fix(web): keep failed tool items in the collapsed group summaries

An ordinary exit-code failure knocked its whole tool group out of the
"Ran N commands" summary (and the live pill mid-run) into the raw
"+N previous tool calls" clamp: v2's derived tone marked any
status=failed item as "error", which the grouping treats as a non-tool
row. That inverts v1's semantics and #8395's muted-failure rule — the
failed lifecycle status already carries the X marker and the summary's
includes-a-failure hint, so the tone override goes.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Author: juliusmarminge

apps/web/src/components/chat/MessagesTimeline.logic.test.ts

@@ -1251,6 +1251,53 @@ describe("deriveMessagesTimelineRows", () => {
     expect(assistantRow?.showAssistantCopyButton).toBe(false);
   });
 
+  it("summarizes a settled tool group even when a command in it failed", () => {
+    const rows = deriveMessagesTimelineRows({
+      timelineEntries: [
+        {
+          id: "work-entry-ok",
+          kind: "work" as const,
+          createdAt: "2026-01-01T00:00:01Z",
+          entry: {
+            id: "work-ok",
+            createdAt: "2026-01-01T00:00:01Z",
+            label: "Ran command",
+            command: "true",
+            tone: "tool" as const,
+            itemType: "command_execution" as const,
+            toolLifecycleStatus: "completed" as const,
+          },
+        },
+        {
+          id: "work-entry-failed",
+          kind: "work" as const,
+          createdAt: "2026-01-01T00:00:02Z",
+          entry: {
+            id: "work-failed",
+            createdAt: "2026-01-01T00:00:02Z",
+            label: "Ran command",
+            command: "ssh host true",
+            detail: "connection refused",
+            tone: "tool" as const,
+            itemType: "command_execution" as const,
+            toolLifecycleStatus: "failed" as const,
+          },
+        },
+      ],
+      isWorking: false,
+      activeTurnStartedAt: null,
+      turnDiffSummaryByAssistantMessageId: new Map(),
+      revertTurnCountByUserMessageId: new Map(),
+    });
+
+    expect(rows).toHaveLength(1);
+    expect(rows[0]).toMatchObject({
+      kind: "work-toggle",
+      summary: "Ran 2 commands",
+      hasFailure: true,
+    });
+  });
+
   it("collapses contiguous settled tool entries behind a summary toggle", () => {
     const timelineEntries = [
       {

apps/web/src/session-logic.test.ts

@@ -379,6 +379,50 @@ describe("V2 session presentation", () => {
     }
   });
 
+  it("keeps failed tool items tool-toned so groups still summarize", () => {
+    const failedCommand = {
+      id: TurnItemId.make("item-failed-command"),
+      threadId: ThreadId.make("thread-1"),
+      runId: RunId.make("run-1"),
+      nodeId: null,
+      providerThreadId: null,
+      providerTurnId: null,
+      nativeItemRef: null,
+      parentItemId: null,
+      ordinal: 0,
+      status: "failed" as const,
+      title: null,
+      startedAt: null,
+      completedAt: null,
+      updatedAt: DateTime.nowUnsafe(),
+      type: "command_execution" as const,
+      input: "ssh host true",
+      output: "connection refused",
+      exitCode: 255,
+    } satisfies OrchestrationV2TurnItem;
+    const entries = deriveTimelineEntriesFromVisibleTurnItems({
+      visibleTurnItems: [
+        {
+          position: 0,
+          visibility: "local" as const,
+          sourceThreadId: ThreadId.make("thread-1"),
+          sourceItemId: failedCommand.id,
+          item: failedCommand,
+        } as never,
+      ],
+      optimisticMessages: [],
+    });
+    const entry = entries[0];
+    expect(entry?.kind).toBe("work");
+    if (entry?.kind === "work") {
+      // An exit-code failure is still an ordinary tool row: the failed
+      // lifecycle status carries the marker, and an "error" tone here would
+      // knock the whole group out of the "Ran N commands" summary.
+      expect(entry.entry.tone).toBe("tool");
+      expect(entry.entry.toolLifecycleStatus).toBe("failed");
+    }
+  });
+
   it("waits for a dispatched turn item before adding queued input to the timeline", () => {
     const projection = makeThreadProjectionFixture();
     const now = DateTime.makeUnsafe("2026-06-20T00:00:00.000Z");

apps/web/src/session-logic.ts

@@ -468,7 +468,6 @@ function projectedWorkEntryStatus(
 
 function projectedWorkEntryTone(item: OrchestrationV2TurnItem): WorkLogEntry["tone"] {
   if (item.type === "error") return "info";
-  if (item.status === "failed") return "error";
   if (item.type === "reasoning") return "thinking";
   switch (item.type) {
     case "command_execution":