git.iko.soy/crush
on main help sign in
Config fix correcting loose viper string check, default model now set correctly (#147)
88711dbJoshua LaMorey-Salzmann
internal/config/config.go+8−6
compare with main
unifiedsplit
@@ -260,7 +260,7 @@ func setProviderDefaults() {
260 // 7. Azure
260 // 7. Azure
261
261
262 // Anthropic configuration
262 // Anthropic configuration
263− if viper.Get("providers.anthropic.apiKey") != "" {
263+ if key := viper.GetString("providers.anthropic.apiKey"); strings.TrimSpace(key) != "" {
264 viper.SetDefault("agents.coder.model", models.Claude37Sonnet)
264 viper.SetDefault("agents.coder.model", models.Claude37Sonnet)
265 viper.SetDefault("agents.task.model", models.Claude37Sonnet)
265 viper.SetDefault("agents.task.model", models.Claude37Sonnet)
266 viper.SetDefault("agents.title.model", models.Claude37Sonnet)
266 viper.SetDefault("agents.title.model", models.Claude37Sonnet)
@@ -268,7 +268,7 @@ func setProviderDefaults() {
268 }
268 }
269
269
270 // OpenAI configuration
270 // OpenAI configuration
271− if viper.Get("providers.openai.apiKey") != "" {
271+ if key := viper.GetString("providers.openai.apiKey"); strings.TrimSpace(key) != "" {
272 viper.SetDefault("agents.coder.model", models.GPT41)
272 viper.SetDefault("agents.coder.model", models.GPT41)
273 viper.SetDefault("agents.task.model", models.GPT41Mini)
273 viper.SetDefault("agents.task.model", models.GPT41Mini)
274 viper.SetDefault("agents.title.model", models.GPT41Mini)
274 viper.SetDefault("agents.title.model", models.GPT41Mini)
@@ -276,7 +276,7 @@ func setProviderDefaults() {
276 }
276 }
277
277
278 // Google Gemini configuration
278 // Google Gemini configuration
279− if viper.Get("providers.google.gemini.apiKey") != "" {
279+ if key := viper.GetString("providers.gemini.apiKey"); strings.TrimSpace(key) != "" {
280 viper.SetDefault("agents.coder.model", models.Gemini25)
280 viper.SetDefault("agents.coder.model", models.Gemini25)
281 viper.SetDefault("agents.task.model", models.Gemini25Flash)
281 viper.SetDefault("agents.task.model", models.Gemini25Flash)
282 viper.SetDefault("agents.title.model", models.Gemini25Flash)
282 viper.SetDefault("agents.title.model", models.Gemini25Flash)
@@ -284,7 +284,7 @@ func setProviderDefaults() {
284 }
284 }
285
285
286 // Groq configuration
286 // Groq configuration
287− if viper.Get("providers.groq.apiKey") != "" {
287+ if key := viper.GetString("providers.groq.apiKey"); strings.TrimSpace(key) != "" {
288 viper.SetDefault("agents.coder.model", models.QWENQwq)
288 viper.SetDefault("agents.coder.model", models.QWENQwq)
289 viper.SetDefault("agents.task.model", models.QWENQwq)
289 viper.SetDefault("agents.task.model", models.QWENQwq)
290 viper.SetDefault("agents.title.model", models.QWENQwq)
290 viper.SetDefault("agents.title.model", models.QWENQwq)
@@ -292,14 +292,15 @@ func setProviderDefaults() {
292 }
292 }
293
293
294 // OpenRouter configuration
294 // OpenRouter configuration
295− if viper.Get("providers.openrouter.apiKey") != "" {
295+ if key := viper.GetString("providers.openrouter.apiKey"); strings.TrimSpace(key) != "" {
296 viper.SetDefault("agents.coder.model", models.OpenRouterClaude37Sonnet)
296 viper.SetDefault("agents.coder.model", models.OpenRouterClaude37Sonnet)
297 viper.SetDefault("agents.task.model", models.OpenRouterClaude37Sonnet)
297 viper.SetDefault("agents.task.model", models.OpenRouterClaude37Sonnet)
298 viper.SetDefault("agents.title.model", models.OpenRouterClaude35Haiku)
298 viper.SetDefault("agents.title.model", models.OpenRouterClaude35Haiku)
299 return
299 return
300 }
300 }
301
301
302+ // XAI configuration
302− if viper.Get("providers.xai.apiKey") != "" {
303+ if key := viper.GetString("providers.xai.apiKey"); strings.TrimSpace(key) != "" {
303 viper.SetDefault("agents.coder.model", models.XAIGrok3Beta)
304 viper.SetDefault("agents.coder.model", models.XAIGrok3Beta)
304 viper.SetDefault("agents.task.model", models.XAIGrok3Beta)
305 viper.SetDefault("agents.task.model", models.XAIGrok3Beta)
305 viper.SetDefault("agents.title.model", models.XAiGrok3MiniFastBeta)
306 viper.SetDefault("agents.title.model", models.XAiGrok3MiniFastBeta)
@@ -314,6 +315,7 @@ func setProviderDefaults() {
314 return
315 return
315 }
316 }
316
317
318+ // Azure OpenAI configuration
317 if os.Getenv("AZURE_OPENAI_ENDPOINT") != "" {
319 if os.Getenv("AZURE_OPENAI_ENDPOINT") != "" {
318 viper.SetDefault("agents.coder.model", models.AzureGPT41)
320 viper.SetDefault("agents.coder.model", models.AzureGPT41)
319 viper.SetDefault("agents.task.model", models.AzureGPT41Mini)
321 viper.SetDefault("agents.task.model", models.AzureGPT41Mini)