fix(openai-compat): preserve legacy thinking passthrough

Author: BenjaminAaron196

config.example.yaml

@@ -248,7 +248,7 @@ nonstream-keepalive-interval: 0
 #     models: # The models supported by the provider.
 #       - name: "moonshotai/kimi-k2:free" # The actual model name.
 #         alias: "kimi-k2"               # The alias used in the API.
-#         thinking:                      # optional: omit to default to levels ["low","medium","high"]
+#         thinking:                      # optional: omit to preserve legacy passthrough (no advertised thinking support)
 #           levels: ["low", "medium", "high"]
 #       # You may repeat the same alias to build an internal model pool.
 #       # The client still sees only one alias in the model list.

internal/config/config.go

@@ -551,7 +551,7 @@ type OpenAICompatibilityModel struct {
 	Alias string `yaml:"alias" json:"alias"`
 
 	// Thinking configures the thinking/reasoning capability for this model.
-	// If nil, the model defaults to level-based reasoning with levels ["low", "medium", "high"].
+	// If nil, the model keeps legacy passthrough behavior and does not advertise managed thinking support.
 	Thinking *registry.ThinkingSupport `yaml:"thinking,omitempty" json:"thinking,omitempty"`
 }
 

sdk/cliproxy/service.go

@@ -2136,17 +2136,15 @@ func (s *Service) registerModelsForAuth(a *coreauth.Auth) {
 							modelID = m.Name
 						}
 						thinking := m.Thinking
-						if thinking == nil {
-							thinking = &registry.ThinkingSupport{Levels: []string{"low", "medium", "high"}}
-						}
+						userDefined := thinking == nil
 						ms = append(ms, &ModelInfo{
 							ID:          modelID,
 							Object:      "model",
 							Created:     time.Now().Unix(),
 							OwnedBy:     compat.Name,
 							Type:        "openai-compatibility",
 							DisplayName: modelID,
-							UserDefined: false,
+							UserDefined: userDefined,
 							Thinking:    thinking,
 						})
 					}

sdk/cliproxy/service_openai_compat_thinking_test.go

@@ -0,0 +1,99 @@
+package cliproxy
+
+import (
+	"testing"
+
+	"github.com/router-for-me/CLIProxyAPI/v6/internal/registry"
+	coreauth "github.com/router-for-me/CLIProxyAPI/v6/sdk/cliproxy/auth"
+	"github.com/router-for-me/CLIProxyAPI/v6/sdk/config"
+)
+
+func TestRegisterModelsForAuth_OpenAICompatibility_DefaultThinkingPreservesLegacyPassthrough(t *testing.T) {
+	modelID := "compat-default-" + t.Name()
+	authID := "auth-" + t.Name()
+	service := &Service{
+		cfg: &config.Config{
+			OpenAICompatibility: []config.OpenAICompatibility{{
+				Name: "compat-default",
+				Models: []config.OpenAICompatibilityModel{{
+					Name:  "upstream-default",
+					Alias: modelID,
+				}},
+			}},
+		},
+	}
+	auth := &coreauth.Auth{
+		ID:       authID,
+		Provider: "openai-compatibility",
+		Status:   coreauth.StatusActive,
+		Attributes: map[string]string{
+			"compat_name": "compat-default",
+		},
+	}
+
+	reg := GlobalModelRegistry()
+	reg.UnregisterClient(auth.ID)
+	t.Cleanup(func() {
+		reg.UnregisterClient(auth.ID)
+	})
+
+	service.registerModelsForAuth(auth)
+
+	got := registry.LookupModelInfo(modelID, "compat-default")
+	if got == nil {
+		t.Fatalf("expected model %q to be registered", modelID)
+	}
+	if !got.UserDefined {
+		t.Fatalf("expected model %q to remain user-defined when thinking is omitted", modelID)
+	}
+	if got.Thinking != nil {
+		t.Fatalf("expected model %q to keep nil thinking support when omitted, got %+v", modelID, got.Thinking)
+	}
+}
+
+func TestRegisterModelsForAuth_OpenAICompatibility_ExplicitThinkingEnablesManagedSupport(t *testing.T) {
+	modelID := "compat-thinking-" + t.Name()
+	authID := "auth-" + t.Name()
+	service := &Service{
+		cfg: &config.Config{
+			OpenAICompatibility: []config.OpenAICompatibility{{
+				Name: "compat-thinking",
+				Models: []config.OpenAICompatibilityModel{{
+					Name:     "upstream-thinking",
+					Alias:    modelID,
+					Thinking: &registry.ThinkingSupport{Levels: []string{"low", "medium", "high"}},
+				}},
+			}},
+		},
+	}
+	auth := &coreauth.Auth{
+		ID:       authID,
+		Provider: "openai-compatibility",
+		Status:   coreauth.StatusActive,
+		Attributes: map[string]string{
+			"compat_name": "compat-thinking",
+		},
+	}
+
+	reg := GlobalModelRegistry()
+	reg.UnregisterClient(auth.ID)
+	t.Cleanup(func() {
+		reg.UnregisterClient(auth.ID)
+	})
+
+	service.registerModelsForAuth(auth)
+
+	got := registry.LookupModelInfo(modelID, "compat-thinking")
+	if got == nil {
+		t.Fatalf("expected model %q to be registered", modelID)
+	}
+	if got.UserDefined {
+		t.Fatalf("expected model %q to use managed thinking when explicitly configured", modelID)
+	}
+	if got.Thinking == nil {
+		t.Fatalf("expected model %q to expose configured thinking support", modelID)
+	}
+	if len(got.Thinking.Levels) != 3 || got.Thinking.Levels[0] != "low" || got.Thinking.Levels[1] != "medium" || got.Thinking.Levels[2] != "high" {
+		t.Fatalf("unexpected thinking levels: %+v", got.Thinking)
+	}
+}