From 24a39fd6a68a3c5e731298d86a919923efde7a92 Mon Sep 17 00:00:00 2001 From: Hannes Rudolph Date: Wed, 22 Oct 2025 16:47:15 -0600 Subject: [PATCH] fix(webview): include settingsArray in AutoApproveDropdown dependencies to satisfy eslint react-hooks/exhaustive-deps --- webview-ui/src/components/chat/AutoApproveDropdown.tsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/webview-ui/src/components/chat/AutoApproveDropdown.tsx b/webview-ui/src/components/chat/AutoApproveDropdown.tsx index c27c4edc3a..280d1b479d 100644 --- a/webview-ui/src/components/chat/AutoApproveDropdown.tsx +++ b/webview-ui/src/components/chat/AutoApproveDropdown.tsx @@ -155,7 +155,7 @@ export const AutoApproveDropdown = ({ disabled = false, triggerClassName = "" }: // Filter out Browser toggle when browser tool is disabled const visibleSettingsArray = React.useMemo( () => settingsArray.filter((s) => browserToolEnabled || s.key !== "alwaysAllowBrowser"), - [browserToolEnabled], + [settingsArray, browserToolEnabled], ) const visibleKeys = React.useMemo(() => visibleSettingsArray.map((s) => s.key), [visibleSettingsArray])