fix(webview): include settingsArray in AutoApproveDropdown dependencies to satisfy eslint react-hooks/exhaustive-deps

This commit is contained in:
Hannes Rudolph 2025-10-22 16:47:15 -06:00
parent 4cf10e1fab
commit 24a39fd6a6

View file

@ -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])