From 7e4a87b7092bf6c4ef166b4d931426b273076dc0 Mon Sep 17 00:00:00 2001 From: Roo Code Date: Mon, 4 Aug 2025 12:15:54 +0000 Subject: [PATCH] fix: enable Save button when changing OpenRouter provider routing - Modified setApiConfigurationField to exclude openRouterSpecificProvider from initial sync logic - This ensures changes to the provider routing setting properly trigger the Save button - Fixes #6655 --- webview-ui/src/components/settings/SettingsView.tsx | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/webview-ui/src/components/settings/SettingsView.tsx b/webview-ui/src/components/settings/SettingsView.tsx index 630b59485d..60e2d07099 100644 --- a/webview-ui/src/components/settings/SettingsView.tsx +++ b/webview-ui/src/components/settings/SettingsView.tsx @@ -229,7 +229,9 @@ const SettingsView = forwardRef(({ onDone, t // Don't treat initial sync from undefined to a defined value as a user change // This prevents the dirty state when the component initializes and auto-syncs the model ID - const isInitialSync = previousValue === undefined && value !== undefined + // Exception: openRouterSpecificProvider should always trigger change detection + const isInitialSync = + previousValue === undefined && value !== undefined && field !== "openRouterSpecificProvider" if (!isInitialSync) { setChangeDetected(true)