diff --git a/internal/config/load.go b/internal/config/load.go index 16cb531ddfa9d452ed55dd82914c6d77f7650f0d..ad2b75b75df8c6f8d7a5cd2e62df1a831157b9e1 100644 --- a/internal/config/load.go +++ b/internal/config/load.go @@ -126,11 +126,6 @@ func (c *Config) configureProviders(env env.Env, resolver VariableResolver, know config, configExists := c.Providers.Get(string(p.ID)) // if the user configured a known provider we need to allow it to override a couple of parameters if configExists { - if config.Disable { - slog.Debug("Skipping provider due to disable flag", "provider", p.ID) - c.Providers.Del(string(p.ID)) - continue - } if config.BaseURL != "" { p.APIEndpoint = config.BaseURL } diff --git a/internal/config/load_test.go b/internal/config/load_test.go index 90276c96ad113f453ed699c8deeb30b4f5fef9d5..756f849db426e226c197879740f0dc47d3048dd9 100644 --- a/internal/config/load_test.go +++ b/internal/config/load_test.go @@ -543,10 +543,10 @@ func TestConfig_configureProvidersWithDisabledProvider(t *testing.T) { err := cfg.configureProviders(env, resolver, knownProviders) require.NoError(t, err) - // Provider should be removed from config when disabled - require.Equal(t, cfg.Providers.Len(), 0) - _, exists := cfg.Providers.Get("openai") - require.False(t, exists) + require.Equal(t, cfg.Providers.Len(), 1) + prov, exists := cfg.Providers.Get("openai") + require.True(t, exists) + require.True(t, prov.Disable) } func TestConfig_configureProvidersCustomProviderValidation(t *testing.T) {