From d985ff9b90cb8476301b2b611023a4332b47a2f0 Mon Sep 17 00:00:00 2001 From: Paul Buetow Date: Mon, 16 Mar 2026 04:06:39 +0200 Subject: Centralize GPT-5 temperature override into llmutils.ResolveTemperature Eliminates identical temperature resolution logic duplicated in hexaiaction, hexaicli, and lsp packages. Co-Authored-By: Claude Opus 4.6 --- internal/hexaiaction/prompts.go | 18 +++--------------- 1 file changed, 3 insertions(+), 15 deletions(-) (limited to 'internal/hexaiaction') diff --git a/internal/hexaiaction/prompts.go b/internal/hexaiaction/prompts.go index 2424819..61775d4 100644 --- a/internal/hexaiaction/prompts.go +++ b/internal/hexaiaction/prompts.go @@ -42,22 +42,10 @@ func canonicalProvider(name string) string { return llmutils.CanonicalProvider(name) } +// selectActionTemperature resolves the effective temperature for a code action, +// delegating GPT-5 override logic to llmutils.ResolveTemperature. func selectActionTemperature(cfg actionConfig, provider string, entry appconfig.SurfaceConfig, model string) (float64, bool) { - core := cfg.CoreSection() - if entry.Temperature != nil { - return *entry.Temperature, true - } - if core.CodingTemperature != nil { - temp := *core.CodingTemperature - if provider == "openai" && strings.HasPrefix(strings.ToLower(model), "gpt-5") && temp == 0.2 { - temp = 1.0 - } - return temp, true - } - if provider == "openai" && strings.HasPrefix(strings.ToLower(model), "gpt-5") { - return 1.0, true - } - return 0, false + return llmutils.ResolveTemperature(provider, model, entry.Temperature, cfg.CoreSection().CodingTemperature) } func runRewrite(ctx context.Context, cfg actionConfig, client chatDoer, instruction, selection string) (string, error) { -- cgit v1.2.3