fix(chat): rely on computeDiffStats for new file unified diffs; remove special-case counting and add test (addresses roomote review)\n\n- Remove special-case line counting for newFileCreated in [diffStatsForInline](webview-ui/src/components/chat/ChatRow.tsx:472)\n- Always use [computeDiffStats()](webview-ui/src/components/chat/ChatRow.tsx:118)\n- Add test to assert +N/-0 for new files in [ChatRow.diff-actions.spec.tsx](webview-ui/src/components/chat/__tests__/ChatRow.diff-actions.spec.tsx)

Author: hannesrudolph

webview-ui/src/components/chat/ChatRow.tsx

@@ -470,15 +470,8 @@ export const ChatRowContent = ({
 	}, [tool])
 
 	const diffStatsForInline = useMemo(() => {
-		if (tool?.tool === "newFileCreated") {
-			// For new files, count all lines as additions
-			const content = diffTextForStats
-			if (!content) return null
-			const lines = content.split("\n").length
-			return { added: lines, removed: 0 }
-		}
 		return computeDiffStats(diffTextForStats)
-	}, [diffTextForStats, tool])
+	}, [diffTextForStats])
 
 	// Clean diff content for display (remove CDATA markers and convert to unified diff)
 	const cleanDiffContent = useMemo(() => {

webview-ui/src/components/chat/__tests__/ChatRow.diff-actions.spec.tsx

@@ -90,4 +90,25 @@ describe("ChatRow - inline diff stats and actions", () => {
 		expect(screen.getByText("+1")).toBeInTheDocument()
 		expect(screen.getByText("-2")).toBeInTheDocument()
 	})
+
+	it("counts only added lines for newFileCreated (ignores diff headers)", () => {
+		const content = "a\nb\nc"
+		const message: any = {
+			type: "ask",
+			ask: "tool",
+			ts: Date.now(),
+			partial: false,
+			text: JSON.stringify({
+				tool: "newFileCreated",
+				path: "src/new-file.ts",
+				content,
+			}),
+		}
+
+		renderChatRow(message)
+
+		// Should only count the three content lines as additions
+		expect(screen.getByText("+3")).toBeInTheDocument()
+		expect(screen.getByText("-0")).toBeInTheDocument()
+	})
 })