From 8480a84b3c4e2c2788c367ceb5ec14f4a580d8fe Mon Sep 17 00:00:00 2001
From: roboomp <omp@can.ac>
Date: Thu, 9 Jul 2026 17:04:32 +0000
Subject: [PATCH] fix(collab-web): preserve active transcript args

- Render deduped active tool cards with the execution args carried by ActiveTool.
- Covered differing raw assistant args and effective active args in the transcript regression test.

Fixes #4948
---
 .../src/components/transcript/Transcript.tsx          |  3 ++-
 packages/collab-web/test/transcript.test.tsx          | 11 ++++++++---
 2 files changed, 10 insertions(+), 4 deletions(-)

diff --git a/packages/collab-web/src/components/transcript/Transcript.tsx b/packages/collab-web/src/components/transcript/Transcript.tsx
index 30e105564..947883ee1 100644
--- a/packages/collab-web/src/components/transcript/Transcript.tsx
+++ b/packages/collab-web/src/components/transcript/Transcript.tsx
@@ -109,13 +109,14 @@ function AssistantBody({
 			case "toolCall": {
 				const act = active.get(block.id);
 				const result = results.get(block.id);
+				const args = act?.args ?? block.arguments;
 				return (
 					<ToolCard
 						key={block.id}
 						toolCallId={block.id}
 						name={block.name}
 						intent={block.intent ?? act?.intent}
-						args={block.arguments}
+						args={args}
 						result={result}
 						host={host}
 						running={!result && (act !== undefined || pending)}
diff --git a/packages/collab-web/test/transcript.test.tsx b/packages/collab-web/test/transcript.test.tsx
index f3b004eac..1d3d60d99 100644
--- a/packages/collab-web/test/transcript.test.tsx
+++ b/packages/collab-web/test/transcript.test.tsx
@@ -8,6 +8,9 @@ import type { ActiveTool } from "../src/lib/client";
 const TOOL_CALL_ID = "call-running-tool";
 const TOOL_NAME = "probe_tool";
 
+const RAW_ASSISTANT_TARGET = "stale-raw-assistant-target";
+const ACTIVE_TOOL_TARGET = "effective-active-tool-target";
+
 function assistantUsage(): AssistantMessage["usage"] {
 	return { input: 1, output: 2, cacheRead: 0, cacheWrite: 0, totalTokens: 3, cost: { total: 0 } };
 }
@@ -26,7 +29,7 @@ function committedAssistantToolCall(): SessionEntry {
 					type: "toolCall",
 					id: TOOL_CALL_ID,
 					name: TOOL_NAME,
-					arguments: { target: "fixture-input" },
+					arguments: { target: RAW_ASSISTANT_TARGET },
 					intent: "Inspect fixture input",
 				},
 			],
@@ -42,7 +45,7 @@ function activeTool(): ActiveTool {
 	return {
 		toolCallId: TOOL_CALL_ID,
 		toolName: TOOL_NAME,
-		args: { target: "fixture-input" },
+		args: { target: ACTIVE_TOOL_TARGET },
 		intent: "Inspect fixture input",
 		startedAt: 1,
 	};
@@ -88,7 +91,7 @@ function countOccurrences(text: string, needle: string): number {
 }
 
 describe("Transcript live tool rendering", () => {
-	it("renders one running card for a committed tool call without the working shimmer", () => {
+	it("renders one running card for a committed tool call using active args without the working shimmer", () => {
 		const html = renderTranscript({
 			entries: [committedAssistantToolCall()],
 			activeTools: new Map([[TOOL_CALL_ID, activeTool()]]),
@@ -99,6 +102,8 @@ describe("Transcript live tool rendering", () => {
 		expect(countElements(html, '[aria-label="running"]')).toBe(1);
 		expect(countOccurrences(html, TOOL_NAME)).toBe(1);
 		expect(html).not.toContain("thinking…");
+		expect(html).toContain(ACTIVE_TOOL_TARGET);
+		expect(html).not.toContain(RAW_ASSISTANT_TARGET);
 	});
 
 	it("keeps the working shimmer when no tool is active", () => {