From 9dc9f208f39cdaea74991e38c59bb38fb85f73fe Mon Sep 17 00:00:00 2001 From: Abhijay Jain Date: Wed, 17 Dec 2025 01:48:54 +0530 Subject: [PATCH] fix : Custom providers with empty API keys show as configured in desktop (#6105) Signed-off-by: Abhijay007 --- crates/goose-server/src/routes/utils.rs | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/crates/goose-server/src/routes/utils.rs b/crates/goose-server/src/routes/utils.rs index 81bd41d6b4..9d5e19957d 100644 --- a/crates/goose-server/src/routes/utils.rs +++ b/crates/goose-server/src/routes/utils.rs @@ -94,13 +94,11 @@ pub fn inspect_keys( pub fn check_provider_configured(metadata: &ProviderMetadata, provider_type: ProviderType) -> bool { let config = Config::global(); - // TODO(Douwe): if the provider doesn't need an API key, it should be considered configured always if provider_type == ProviderType::Custom || provider_type == ProviderType::Declarative { if let Ok(loaded_provider) = load_provider(metadata.name.as_str()) { return config .get_secret::(&loaded_provider.config.api_key_env) - .map(|s| !s.is_empty()) - .unwrap_or(false); + .is_ok(); } } // Special case: Zero-config providers (no config keys)