From be908cd0d5d2673349a20e01fb296f8445e6f109 Mon Sep 17 00:00:00 2001 From: Bryan Helmkamp Date: Thu, 7 May 2026 22:39:21 -0700 Subject: [PATCH] style(web): drop redundant when/tool labels in tool group accordion The accordion row already shows elapsed/duration and the group header already names the tool, so the When and Tool fields inside the expanded event detail are redundant. Keep them for the standalone single-event panel. Co-Authored-By: Claude Opus 4.7 (1M context) --- apps/fabro-web/app/routes/run-stages.tsx | 30 +++++++++++++++++------- 1 file changed, 21 insertions(+), 9 deletions(-) diff --git a/apps/fabro-web/app/routes/run-stages.tsx b/apps/fabro-web/app/routes/run-stages.tsx index d3983350b..39ec1a7c8 100644 --- a/apps/fabro-web/app/routes/run-stages.tsx +++ b/apps/fabro-web/app/routes/run-stages.tsx @@ -697,7 +697,15 @@ function JsonBlock({ value }: { value: string }) { ); } -function EventDetails({ turn, runStart }: { turn: TurnType; runStart: string | undefined }) { +function EventDetails({ + turn, + runStart, + hideMeta = false, +}: { + turn: TurnType; + runStart: string | undefined; + hideMeta?: boolean; +}) { const elapsed = formatElapsed(turn.ts, runStart); const absolute = (() => { const ms = Date.parse(turn.ts); @@ -707,9 +715,11 @@ function EventDetails({ turn, runStart }: { turn: TurnType; runStart: string | u return (
- - {elapsed ? `${elapsed} · ${absolute}` : absolute} - + {!hideMeta && ( + + {elapsed ? `${elapsed} · ${absolute}` : absolute} + + )} {(turn.kind === "system" || turn.kind === "assistant") && ( @@ -719,10 +729,12 @@ function EventDetails({ turn, runStart }: { turn: TurnType; runStart: string | u {turn.kind === "tool" && ( <> - - {humanizeToolName(turn.toolName)}{" "} - ({turn.toolName}) - + {!hideMeta && ( + + {humanizeToolName(turn.toolName)}{" "} + ({turn.toolName}) + + )} @@ -1058,7 +1070,7 @@ function ToolGroupDetails({ /> {expandedIndex === i && (
- +
)}