fix: add emitFinalTokenUsageUpdate mock to test and include usageUpdatedAt in toolColumns deps

hannesrudolph · hannesrudolph · commit 06e7809c03d6 · 2025-12-08T13:17:40.000-07:00
- Add emitFinalTokenUsageUpdate mock to clineC and clineB in nested-delegation-resume test
- Add usageUpdatedAt to toolColumns useMemo dependency array for proper recomputation when streaming tool usage updates
diff --git a/apps/web-evals/src/app/runs/[id]/run.tsx b/apps/web-evals/src/app/runs/[id]/run.tsx
@@ -389,7 +389,8 @@ export function Run({ run }: { run: Run }) {
 		return Array.from(toolTotals.entries())
 			.sort((a, b) => b[1] - a[1])
 			.map(([name]): ToolName => name)
-	}, [tasks, toolUsage])
+		// eslint-disable-next-line react-hooks/exhaustive-deps
+	}, [tasks, toolUsage, usageUpdatedAt])
 
 	// Compute aggregate stats
 	const stats = useMemo(() => {
diff --git a/src/__tests__/nested-delegation-resume.spec.ts b/src/__tests__/nested-delegation-resume.spec.ts
@@ -180,6 +180,7 @@ describe("Nested delegation resume (A → B → C)", () => {
 			clineMessages: [],
 			userMessageContent: [],
 			consecutiveMistakeCount: 0,
+			emitFinalTokenUsageUpdate: vi.fn(),
 		} as unknown as Task
 
 		const blockC = {
@@ -223,6 +224,7 @@ describe("Nested delegation resume (A → B → C)", () => {
 			clineMessages: [],
 			userMessageContent: [],
 			consecutiveMistakeCount: 0,
+			emitFinalTokenUsageUpdate: vi.fn(),
 		} as unknown as Task
 
 		const blockB = {
