fix: move postMessage to the onChange handle to avoid rerender postMessage

Author: monotykamary

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

@@ -75,20 +75,6 @@ const ChatTextArea = forwardRef<HTMLTextAreaElement, ChatTextAreaProps>(
 			}
 		}, [diffEnabled, experimentalDiffStrategy])
 
-		// Update extension settings when diffStrategy changes
-		useEffect(() => {
-			if (diffStrategy === "whole") {
-				vscode.postMessage({ type: "diffEnabled", bool: false })
-				vscode.postMessage({ type: "experimentalDiffStrategy", bool: false })
-			} else if (diffStrategy === "search-replace") {
-				vscode.postMessage({ type: "diffEnabled", bool: true })
-				vscode.postMessage({ type: "experimentalDiffStrategy", bool: false })
-			} else if (diffStrategy === "unified-diff") {
-				vscode.postMessage({ type: "diffEnabled", bool: true })
-				vscode.postMessage({ type: "experimentalDiffStrategy", bool: true })
-			}
-		}, [diffStrategy])
-
 		// Close dropdown when clicking outside
 		useEffect(() => {
 			const handleClickOutside = (event: MouseEvent) => {
@@ -853,12 +839,18 @@ const ChatTextArea = forwardRef<HTMLTextAreaElement, ChatTextAreaProps>(
 									if (value === "whole") {
 										setDiffEnabled(false)
 										setExperimentalDiffStrategy(false)
+										vscode.postMessage({ type: "diffEnabled", bool: false })
+										vscode.postMessage({ type: "experimentalDiffStrategy", bool: false })
 									} else if (value === "search-replace") {
 										setDiffEnabled(true)
 										setExperimentalDiffStrategy(false)
+										vscode.postMessage({ type: "diffEnabled", bool: true })
+										vscode.postMessage({ type: "experimentalDiffStrategy", bool: false })
 									} else if (value === "unified-diff") {
 										setDiffEnabled(true)
 										setExperimentalDiffStrategy(true)
+										vscode.postMessage({ type: "diffEnabled", bool: true })
+										vscode.postMessage({ type: "experimentalDiffStrategy", bool: true })
 									}
 								}}
 								style={{