fix: lint

Carlos Alexandro Becker created 3 weeks ago

Signed-off-by: Carlos Alexandro Becker <[e-mail]>

Change summary

cmd/copilot/main.go                  | 12 ++++++++----
internal/providers/providers_test.go |  8 ++++----
2 files changed, 12 insertions(+), 8 deletions(-)

Detailed changes

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, ""

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)
 			}
 		})