feat(thinking): rename isBudgetBasedProvider to isBudgetCapableProvider and update logic for provider checks
This commit is contained in:
@@ -293,7 +293,7 @@ func normalizeUserDefinedConfig(config ThinkingConfig, fromFormat, toFormat stri
|
|||||||
if config.Mode != ModeLevel {
|
if config.Mode != ModeLevel {
|
||||||
return config
|
return config
|
||||||
}
|
}
|
||||||
if !isBudgetBasedProvider(toFormat) || !isLevelBasedProvider(fromFormat) {
|
if !isBudgetCapableProvider(toFormat) {
|
||||||
return config
|
return config
|
||||||
}
|
}
|
||||||
budget, ok := ConvertLevelToBudget(string(config.Level))
|
budget, ok := ConvertLevelToBudget(string(config.Level))
|
||||||
|
|||||||
@@ -335,7 +335,9 @@ func normalizeLevels(levels []string) []string {
|
|||||||
return out
|
return out
|
||||||
}
|
}
|
||||||
|
|
||||||
func isBudgetBasedProvider(provider string) bool {
|
// isBudgetCapableProvider returns true if the provider supports budget-based thinking.
|
||||||
|
// These providers may also support level-based thinking (hybrid models).
|
||||||
|
func isBudgetCapableProvider(provider string) bool {
|
||||||
switch provider {
|
switch provider {
|
||||||
case "gemini", "gemini-cli", "antigravity", "claude":
|
case "gemini", "gemini-cli", "antigravity", "claude":
|
||||||
return true
|
return true
|
||||||
@@ -344,15 +346,6 @@ func isBudgetBasedProvider(provider string) bool {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
func isLevelBasedProvider(provider string) bool {
|
|
||||||
switch provider {
|
|
||||||
case "openai", "openai-response", "codex":
|
|
||||||
return true
|
|
||||||
default:
|
|
||||||
return false
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
func isGeminiFamily(provider string) bool {
|
func isGeminiFamily(provider string) bool {
|
||||||
switch provider {
|
switch provider {
|
||||||
case "gemini", "gemini-cli", "antigravity":
|
case "gemini", "gemini-cli", "antigravity":
|
||||||
|
|||||||
Reference in New Issue
Block a user