From ee2976cca06ba15924e6860c6bf379ef8d980439 Mon Sep 17 00:00:00 2001
From: hkfires <10558748+hkfires@users.noreply.github.com>
Date: Thu, 15 Jan 2026 13:06:26 +0800
Subject: [PATCH] refactor(thinking): improve logging for user-defined models

---
 internal/thinking/apply.go | 26 +++++++++++---------------
 1 file changed, 11 insertions(+), 15 deletions(-)

diff --git a/internal/thinking/apply.go b/internal/thinking/apply.go
index ce210dfb..a2919cea 100644
--- a/internal/thinking/apply.go
+++ b/internal/thinking/apply.go
@@ -247,31 +247,27 @@ func applyUserDefinedModel(body []byte, modelInfo *registry.ModelInfo, provider
 
 	if !hasThinkingConfig(config) {
 		log.WithFields(log.Fields{
-			"model":        modelID,
-			"provider":     provider,
-			"user_defined": true,
-			"passthrough":  true,
-		}).Debug("thinking: user-defined model, no config, passthrough")
+			"model":    modelID,
+			"provider": provider,
+		}).Debug("thinking: user-defined model, passthrough (no config)")
 		return body, nil
 	}
 
 	applier := GetProviderApplier(provider)
 	if applier == nil {
 		log.WithFields(log.Fields{
-			"model":        modelID,
-			"provider":     provider,
-			"user_defined": true,
-			"passthrough":  true,
-		}).Debug("thinking: user-defined model, unknown provider, passthrough")
+			"model":    modelID,
+			"provider": provider,
+		}).Debug("thinking: user-defined model, passthrough (unknown provider)")
 		return body, nil
 	}
 
 	log.WithFields(log.Fields{
-		"model":        modelID,
-		"provider":     provider,
-		"user_defined": true,
-		"passthrough":  false,
-		"config":       config,
+		"provider": provider,
+		"model":    modelID,
+		"mode":     config.Mode,
+		"budget":   config.Budget,
+		"level":    config.Level,
 	}).Debug("thinking: applying config for user-defined model (skip validation)")
 
 	return applier.Apply(body, config, modelInfo)