From d1b3fc7a68ae87128d3f219b8f2422729a39780a Mon Sep 17 00:00:00 2001
From: Matt Toohey <contact@matttoohey.com>
Date: Fri, 7 Aug 2026 11:31:09 +1000
Subject: [PATCH 1/2] fix(sessions): drop the redundant In progress footer row
 from tool call details

The expanded tool call card ended its output with an "In progress" status
row, but the header's status dot already shows a clock icon for running
calls, so the row repeated information without adding any. Extend the
existing success-row suppression in OutputSections.svelte to the running
tone as well; failed, cancelled, and pending rows still render.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Signed-off-by: Matt Toohey <contact@matttoohey.com>
---
 .../features/sessions/tool-calls/OutputSections.svelte    | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/apps/staged/src/lib/features/sessions/tool-calls/OutputSections.svelte b/apps/staged/src/lib/features/sessions/tool-calls/OutputSections.svelte
index ef3b76e1f..77fab33b4 100644
--- a/apps/staged/src/lib/features/sessions/tool-calls/OutputSections.svelte
+++ b/apps/staged/src/lib/features/sessions/tool-calls/OutputSections.svelte
@@ -131,10 +131,10 @@
   <div class="tool-empty-row">{viewModel.output.emptyLabel}</div>
 {/if}
 
-<!-- A success row only echoes the green check already in the card header, so
-     the footer status shows only when it adds something (failed, cancelled,
-     still running). -->
-{#if includeStatus && viewModel.statusTone !== 'success'}
+<!-- Success and in-progress rows only echo the check and clock icons already
+     in the card header, so the footer status shows only when it adds something
+     (failed, cancelled, pending). -->
+{#if includeStatus && viewModel.statusTone !== 'success' && viewModel.statusTone !== 'running'}
   <div
     class="tool-code-status"
     class:status-danger={viewModel.statusTone === 'danger'}

From d76ed855159511ef1bdc9cc48b4304db0f017e40 Mon Sep 17 00:00:00 2001
From: Matt Toohey <contact@matttoohey.com>
Date: Fri, 7 Aug 2026 11:41:36 +1000
Subject: [PATCH 2/2] refactor(sessions): allow-list the tones that render a
 footer status row

Review feedback on d1b3fc7a: the footer-row condition had grown into a
two-tone exclusion list, expressing "which tones add information" as its
negation. Invert it to an explicit allow-list (danger, cancelled, muted)
so the intent reads directly and any future tone must consciously opt in
to rendering the row instead of defaulting to shown.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Signed-off-by: Matt Toohey <contact@matttoohey.com>
---
 .../src/lib/features/sessions/tool-calls/OutputSections.svelte  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/apps/staged/src/lib/features/sessions/tool-calls/OutputSections.svelte b/apps/staged/src/lib/features/sessions/tool-calls/OutputSections.svelte
index 77fab33b4..571064e2a 100644
--- a/apps/staged/src/lib/features/sessions/tool-calls/OutputSections.svelte
+++ b/apps/staged/src/lib/features/sessions/tool-calls/OutputSections.svelte
@@ -134,7 +134,7 @@
 <!-- Success and in-progress rows only echo the check and clock icons already
      in the card header, so the footer status shows only when it adds something
      (failed, cancelled, pending). -->
-{#if includeStatus && viewModel.statusTone !== 'success' && viewModel.statusTone !== 'running'}
+{#if includeStatus && ['danger', 'cancelled', 'muted'].includes(viewModel.statusTone)}
   <div
     class="tool-code-status"
     class:status-danger={viewModel.statusTone === 'danger'}