on main help sign in
Config fix correcting loose viper string check, default model now set correctly (#147)
commit88711db7961c9a6c13ad7f70f669ee895ea55af9 authorJoshua LaMorey-Salzmann <97018563+jdlms@users.noreply.github.com> dateMon 5 May 2025, 09:40 (1 year ago) parent9fec8df
1 file changed+8−6
internal/config/config.go+8−6
internal/config/config.go+8−6
@@ -260,7 +260,7 @@ func setProviderDefaults() {
260260 // 7. Azure
261261
262262 // Anthropic configuration
263− if viper.Get("providers.anthropic.apiKey") != "" {
263+ if key := viper.GetString("providers.anthropic.apiKey"); strings.TrimSpace(key) != "" {
264264 viper.SetDefault("agents.coder.model", models.Claude37Sonnet)
265265 viper.SetDefault("agents.task.model", models.Claude37Sonnet)
266266 viper.SetDefault("agents.title.model", models.Claude37Sonnet)
@@ -268,7 +268,7 @@ func setProviderDefaults() {
268268 }
269269
270270 // OpenAI configuration
271− if viper.Get("providers.openai.apiKey") != "" {
271+ if key := viper.GetString("providers.openai.apiKey"); strings.TrimSpace(key) != "" {
272272 viper.SetDefault("agents.coder.model", models.GPT41)
273273 viper.SetDefault("agents.task.model", models.GPT41Mini)
274274 viper.SetDefault("agents.title.model", models.GPT41Mini)
@@ -276,7 +276,7 @@ func setProviderDefaults() {
276276 }
277277
278278 // Google Gemini configuration
279− if viper.Get("providers.google.gemini.apiKey") != "" {
279+ if key := viper.GetString("providers.gemini.apiKey"); strings.TrimSpace(key) != "" {
280280 viper.SetDefault("agents.coder.model", models.Gemini25)
281281 viper.SetDefault("agents.task.model", models.Gemini25Flash)
282282 viper.SetDefault("agents.title.model", models.Gemini25Flash)
@@ -284,7 +284,7 @@ func setProviderDefaults() {
284284 }
285285
286286 // Groq configuration
287− if viper.Get("providers.groq.apiKey") != "" {
287+ if key := viper.GetString("providers.groq.apiKey"); strings.TrimSpace(key) != "" {
288288 viper.SetDefault("agents.coder.model", models.QWENQwq)
289289 viper.SetDefault("agents.task.model", models.QWENQwq)
290290 viper.SetDefault("agents.title.model", models.QWENQwq)
@@ -292,14 +292,15 @@ func setProviderDefaults() {
292292 }
293293
294294 // OpenRouter configuration
295− if viper.Get("providers.openrouter.apiKey") != "" {
295+ if key := viper.GetString("providers.openrouter.apiKey"); strings.TrimSpace(key) != "" {
296296 viper.SetDefault("agents.coder.model", models.OpenRouterClaude37Sonnet)
297297 viper.SetDefault("agents.task.model", models.OpenRouterClaude37Sonnet)
298298 viper.SetDefault("agents.title.model", models.OpenRouterClaude35Haiku)
299299 return
300300 }
301301
302− if viper.Get("providers.xai.apiKey") != "" {
302+ // XAI configuration
303+ if key := viper.GetString("providers.xai.apiKey"); strings.TrimSpace(key) != "" {
303304 viper.SetDefault("agents.coder.model", models.XAIGrok3Beta)
304305 viper.SetDefault("agents.task.model", models.XAIGrok3Beta)
305306 viper.SetDefault("agents.title.model", models.XAiGrok3MiniFastBeta)
@@ -314,6 +315,7 @@ func setProviderDefaults() {
314315 return
315316 }
316317
318+ // Azure OpenAI configuration
317319 if os.Getenv("AZURE_OPENAI_ENDPOINT") != "" {
318320 viper.SetDefault("agents.coder.model", models.AzureGPT41)
319321 viper.SetDefault("agents.task.model", models.AzureGPT41Mini)