From a30a8b9ea387ba1d11bf57722b546cfbf265b8ba Mon Sep 17 00:00:00 2001 From: Carlos Alexandro Becker Date: Tue, 20 Jan 2026 14:26:05 -0300 Subject: [PATCH] fix: lint Signed-off-by: Carlos Alexandro Becker --- cmd/copilot/main.go | 12 ++++++++---- internal/providers/providers_test.go | 8 ++++---- 2 files changed, 12 insertions(+), 8 deletions(-) diff --git a/cmd/copilot/main.go b/cmd/copilot/main.go index 8954c307fff681b111bd3d465132ac889ebd8afd..07caa84572d479a4e2905c6548d6ef70b72d48d9 100644 --- a/cmd/copilot/main.go +++ b/cmd/copilot/main.go @@ -171,6 +171,10 @@ func modelToCatwalk(m Model) catwalk.Model { } } +const defaultReasoningEffort = "medium" + +var defaultReasoningLevels = []string{"low", "medium", "high"} + func detectReasoningCapabilities(m Model) (canReason bool, levels []string, defaultLevel string) { // Claude models with reasoning support if m.ID == "claude-3.7-sonnet" || @@ -183,17 +187,17 @@ func detectReasoningCapabilities(m Model) (canReason bool, levels []string, defa // Gemini models with reasoning support if strings.HasPrefix(m.ID, "gemini-2.5-") || strings.HasPrefix(m.ID, "gemini-3-") { - return true, []string{"low", "medium", "high"}, "medium" + return true, defaultReasoningLevels, defaultReasoningEffort } // GPT-5 series with reasoning levels if strings.HasPrefix(m.ID, "gpt-5") && !strings.Contains(m.ID, "chat") { - return true, []string{"low", "medium", "high"}, "medium" + return true, defaultReasoningLevels, defaultReasoningEffort } // OpenAI o-series with reasoning levels if strings.HasPrefix(m.ID, "o3-") || strings.HasPrefix(m.ID, "o4-") { - return true, []string{"low", "medium", "high"}, "medium" + return true, defaultReasoningLevels, defaultReasoningEffort } // DeepSeek R1 models @@ -205,7 +209,7 @@ func detectReasoningCapabilities(m Model) (canReason bool, levels []string, defa if m.ID == "grok-3-mini" || m.ID == "grok-3-mini-beta" || strings.HasPrefix(m.ID, "grok-4") || m.ID == "grok-code-fast-1" { - return true, []string{"low", "medium", "high"}, "medium" + return true, defaultReasoningLevels, defaultReasoningEffort } return false, nil, "" diff --git a/internal/providers/providers_test.go b/internal/providers/providers_test.go index ffaa47744ed1a6453734042d5c954fa5e15912e3..53808b725e692e164bb5dab362a8c97836a85d61 100644 --- a/internal/providers/providers_test.go +++ b/internal/providers/providers_test.go @@ -8,14 +8,14 @@ import ( func TestValidDefaultModels(t *testing.T) { for _, p := range GetAll() { t.Run(p.Name, func(t *testing.T) { - var modelIds []string + var modelIDs []string for _, m := range p.Models { - modelIds = append(modelIds, m.ID) + modelIDs = append(modelIDs, m.ID) } - if !slices.Contains(modelIds, p.DefaultLargeModelID) { + if !slices.Contains(modelIDs, p.DefaultLargeModelID) { t.Errorf("Default large model %q not found in provider %q", p.DefaultLargeModelID, p.Name) } - if !slices.Contains(modelIds, p.DefaultSmallModelID) { + if !slices.Contains(modelIDs, p.DefaultSmallModelID) { t.Errorf("Default small model %q not found in provider %q", p.DefaultSmallModelID, p.Name) } })