clean up

Author: playcations

pnpm-lock.yaml

@@ -45,7 +45,6 @@
 		"hast-util-to-jsx-runtime": "^2.3.6",
 		"i18next": "^25.0.0",
 		"i18next-http-backend": "^3.0.2",
-		"katex": "^0.16.11",
 		"knuth-shuffle-seeded": "^1.0.6",
 		"lru-cache": "^11.1.0",
 		"lucide-react": "^0.518.0",
@@ -67,7 +66,6 @@
 		"remove-markdown": "^0.6.0",
 		"shell-quote": "^1.8.2",
 		"shiki": "^3.2.1",
-		"source-map": "^0.7.4",
 		"styled-components": "^6.1.13",
 		"tailwind-merge": "^3.0.0",
 		"tailwindcss": "^4.0.0",
@@ -85,15 +83,13 @@
 		"@testing-library/react": "^16.2.0",
 		"@testing-library/user-event": "^14.6.1",
 		"@types/jest": "^29.0.0",
-		"@types/katex": "^0.16.7",
 		"@types/node": "20.x",
 		"@types/react": "^18.3.23",
 		"@types/react-dom": "^18.3.5",
 		"@types/shell-quote": "^1.7.5",
 		"@types/vscode-webview": "^1.57.5",
 		"@vitejs/plugin-react": "^4.3.4",
 		"@vitest/ui": "^3.2.3",
-		"identity-obj-proxy": "^3.0.0",
 		"jsdom": "^26.0.0",
 		"typescript": "5.8.3",
 		"vite": "6.3.5",

webview-ui/package.json

@@ -61,6 +61,16 @@ const FilesChangedOverview: React.FC<FilesChangedOverviewProps> = ({
 
 	// Simple double-click prevention
 	const [isProcessing, setIsProcessing] = React.useState(false)
+	const timeoutRef = React.useRef<NodeJS.Timeout | null>(null)
+
+	// Cleanup timeout on unmount
+	React.useEffect(() => {
+		return () => {
+			if (timeoutRef.current) {
+				clearTimeout(timeoutRef.current)
+			}
+		}
+	}, [])
 
 	const handleWithDebounce = React.useCallback(
 		async (operation: () => void) => {
@@ -73,7 +83,10 @@ const FilesChangedOverview: React.FC<FilesChangedOverviewProps> = ({
 				// Debug logging removed for production
 			}
 			// Brief delay to prevent double-clicks
-			setTimeout(() => setIsProcessing(false), 300)
+			if (timeoutRef.current) {
+				clearTimeout(timeoutRef.current)
+			}
+			timeoutRef.current = setTimeout(() => setIsProcessing(false), 300)
 		},
 		[isProcessing],
 	)
@@ -136,6 +149,7 @@ const FilesChangedOverview: React.FC<FilesChangedOverviewProps> = ({
 	return (
 		<div
 			className="files-changed-overview"
+			data-testid="files-changed-overview"
 			style={{
 				border: "1px solid var(--vscode-panel-border)",
 				borderRadius: "4px",
@@ -180,7 +194,7 @@ const FilesChangedOverview: React.FC<FilesChangedOverviewProps> = ({
 							transition: "transform 0.2s ease",
 						}}
 					/>
-					<h3 style={{ margin: 0, fontSize: "14px", fontWeight: "bold" }}>
+					<h3 style={{ margin: 0, fontSize: "14px", fontWeight: "bold" }} data-testid="files-changed-header">
 						{t("file-changes:summary.count_with_changes", {
 							count: files.length,
 							changes: totalChanges,
@@ -197,6 +211,7 @@ const FilesChangedOverview: React.FC<FilesChangedOverviewProps> = ({
 						onClick={() => handleWithDebounce(onRejectAll)}
 						disabled={isProcessing}
 						tabIndex={0}
+						data-testid="reject-all-button"
 						style={{
 							backgroundColor: "var(--vscode-button-secondaryBackground)",
 							color: "var(--vscode-button-secondaryForeground)",
@@ -214,6 +229,7 @@ const FilesChangedOverview: React.FC<FilesChangedOverviewProps> = ({
 						onClick={() => handleWithDebounce(onAcceptAll)}
 						disabled={isProcessing}
 						tabIndex={0}
+						data-testid="accept-all-button"
 						style={{
 							backgroundColor: "var(--vscode-button-background)",
 							color: "var(--vscode-button-foreground)",

webview-ui/src/components/file-changes/FilesChangedOverview.tsx

@@ -9,7 +9,6 @@ import { ExtensionStateContext } from "@src/context/ExtensionStateContext"
 import { vscode } from "@src/utils/vscode"
 import { FileChangeType } from "@roo-code/types"
 import FilesChangedOverview from "../../FilesChangedOverview"
-import FilesChangedOverviewWrapper from "../../FilesChangedOverviewWrapper"
 
 // Mock vscode API
 vi.mock("@src/utils/vscode", () => ({
@@ -200,14 +199,21 @@ describe("FilesChangedOverview Webview Message Flow Integration", () => {
 				currentFileChangeset: { baseCheckpoint: "abc123", files: [] },
 			}
 
-			const { container } = render(
+			render(
 				<ExtensionStateContext.Provider value={emptyState as any}>
-					<FilesChangedOverviewWrapper />
+					<FilesChangedOverview
+						changeset={{ baseCheckpoint: "abc123", files: [] }}
+						onViewDiff={(uri) => vscode.postMessage({ type: "viewDiff", uri })}
+						onAcceptFile={(uri) => vscode.postMessage({ type: "acceptFileChange", uri })}
+						onRejectFile={(uri) => vscode.postMessage({ type: "rejectFileChange", uri })}
+						onAcceptAll={() => vscode.postMessage({ type: "acceptAllFileChanges" })}
+						onRejectAll={() => vscode.postMessage({ type: "rejectAllFileChanges" })}
+					/>
 				</ExtensionStateContext.Provider>,
 			)
 
-			// Wrapper returns null for empty states, so container should be empty
-			expect(container.firstChild).toBeNull()
+			// Component should show empty state header
+			expect(screen.getByTestId("files-changed-header")).toBeInTheDocument()
 		})
 	})