From 07e285f1bb28c832a78de123392be80fcd0c79b2 Mon Sep 17 00:00:00 2001 From: Paul Buetow Date: Mon, 16 Mar 2026 04:08:41 +0200 Subject: Address review: add zero-temp test, remove unused surface parameter Co-Authored-By: Claude Opus 4.6 --- internal/lsp/handlers_completion.go | 2 +- internal/lsp/handlers_utils.go | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) (limited to 'internal/lsp') diff --git a/internal/lsp/handlers_completion.go b/internal/lsp/handlers_completion.go index 7c293b0..b5fd03f 100644 --- a/internal/lsp/handlers_completion.go +++ b/internal/lsp/handlers_completion.go @@ -436,7 +436,7 @@ func (s *Server) tryProviderNativeCompletion(ctx context.Context, plan completio sentBytes := len(prompt) + len(after) modelUsed := spec.effectiveModel(client.DefaultModel()) tempVal := 0.0 - if val, ok := chooseSurfaceTemperature(surfaceCompletion, cfg, spec.entry, provider, modelUsed); ok { + if val, ok := chooseSurfaceTemperature(cfg, spec.entry, provider, modelUsed); ok { tempVal = val } suggestions, err := cc.CodeCompletion(ctx2, prompt, after, 1, "", tempVal) diff --git a/internal/lsp/handlers_utils.go b/internal/lsp/handlers_utils.go index d1a9ec3..292ff12 100644 --- a/internal/lsp/handlers_utils.go +++ b/internal/lsp/handlers_utils.go @@ -68,7 +68,7 @@ func (s *Server) buildRequestSpecs(surface surfaceKind) []requestSpec { if entry.Model != "" { opts = append(opts, llm.WithModel(entry.Model)) } - if temp, ok := chooseSurfaceTemperature(surface, cfg, entry, provider, fallbackModel); ok { + if temp, ok := chooseSurfaceTemperature(cfg, entry, provider, fallbackModel); ok { opts = append(opts, llm.WithTemperature(temp)) } specs = append(specs, requestSpec{ @@ -117,7 +117,7 @@ func surfaceConfigsFor(cfg appconfig.App, surface surfaceKind) []appconfig.Surfa // chooseSurfaceTemperature resolves the effective temperature for a surface // request, delegating GPT-5 override logic to llmutils.ResolveTemperature. -func chooseSurfaceTemperature(surface surfaceKind, cfg appconfig.App, entry appconfig.SurfaceConfig, provider string, fallbackModel string) (float64, bool) { +func chooseSurfaceTemperature(cfg appconfig.App, entry appconfig.SurfaceConfig, provider string, fallbackModel string) (float64, bool) { effectiveModel := strings.TrimSpace(entry.Model) if effectiveModel == "" { effectiveModel = strings.TrimSpace(fallbackModel) -- cgit v1.2.3