feat(openai): temperature is no longer a required field for OpenAI client configuration

Closes #360

Author: Blarc

CHANGELOG.md

@@ -2,6 +2,10 @@
 
 ## [Unreleased]
 
+### Changed
+
+- Temperature is no longer a required field for OpenAI client configuration (#360).
+
 ## [2.13.0] - 2025-06-17
 
 ### Added

src/main/kotlin/com/github/blarc/ai/commits/intellij/plugin/AICommitsExtensions.kt

@@ -36,6 +36,19 @@ fun ValidationInfoBuilder.temperatureValid(value: String): ValidationInfo? {
     return error(message("validation.temperature"))
 }
 
+fun ValidationInfoBuilder.temperatureValidNullable(value: String): ValidationInfo? {
+    if (value.isNotBlank()) {
+        value.toFloatOrNull().let {
+            return if (it != null && it in 0.0..2.0) {
+                null
+            } else {
+                error(message("validation.temperature"))
+            }
+        }
+    }
+    return null
+}
+
 fun ValidationInfoBuilder.unique(value: String, existingValues: Set<String>): ValidationInfo? =
     if (existingValues.contains(value)) error(message("validation.unique")) else null
 

src/main/kotlin/com/github/blarc/ai/commits/intellij/plugin/settings/clients/LLMClientPanel.kt

@@ -3,9 +3,11 @@ package com.github.blarc.ai.commits.intellij.plugin.settings.clients
 import com.github.blarc.ai.commits.intellij.plugin.*
 import com.github.blarc.ai.commits.intellij.plugin.AICommitsBundle.message
 import com.intellij.openapi.ui.ComboBox
+import com.intellij.openapi.ui.ValidationInfo
 import com.intellij.ui.components.JBLabel
 import com.intellij.ui.components.JBTextField
 import com.intellij.ui.dsl.builder.*
+import com.intellij.ui.layout.ValidationInfoBuilder
 import kotlin.reflect.KMutableProperty0
 
 
@@ -94,15 +96,15 @@ abstract class LLMClientPanel(
         }
     }
 
-    open fun Panel.temperatureRow() {
+    open fun Panel.temperatureRow(validation: ValidationInfoBuilder.(String) -> ValidationInfo? = ValidationInfoBuilder::temperatureValid) {
         row {
             label(message("settings.llmClient.temperature"))
                 .widthGroup("label")
 
             cell(temperatureTextField)
                 .bindText(clientConfiguration::temperature)
                 .align(Align.FILL)
-                .validationOnInput { temperatureValid(it.text) }
+                .validationOnInput { validation(it.text) }
                 .resizableColumn()
 
             contextHelp(message("settings.llmClient.temperature.comment"))

src/main/kotlin/com/github/blarc/ai/commits/intellij/plugin/settings/clients/openAi/OpenAiClientPanel.kt

@@ -3,9 +3,11 @@ package com.github.blarc.ai.commits.intellij.plugin.settings.clients.openAi
 import com.github.blarc.ai.commits.intellij.plugin.AICommitsBundle.message
 import com.github.blarc.ai.commits.intellij.plugin.emptyText
 import com.github.blarc.ai.commits.intellij.plugin.settings.clients.LLMClientPanel
+import com.github.blarc.ai.commits.intellij.plugin.temperatureValidNullable
 import com.intellij.ui.components.JBPasswordField
 import com.intellij.ui.components.JBTextField
 import com.intellij.ui.dsl.builder.*
+import com.intellij.ui.layout.ValidationInfoBuilder
 
 class OpenAiClientPanel(private val clientConfiguration: OpenAiClientConfiguration) : LLMClientPanel(clientConfiguration) {
     private val tokenPasswordField = JBPasswordField()
@@ -19,7 +21,7 @@ class OpenAiClientPanel(private val clientConfiguration: OpenAiClientConfigurati
         tokenRow()
         modelIdRow()
         organizationIdRow()
-        temperatureRow()
+        temperatureRow(ValidationInfoBuilder::temperatureValidNullable)
         topPDoubleRow(topPTextField, clientConfiguration::topP.toNullableProperty())
         verifyRow()
 

src/main/kotlin/com/github/blarc/ai/commits/intellij/plugin/settings/clients/openAi/OpenAiClientService.kt

@@ -31,11 +31,14 @@ class OpenAiClientService(private val cs: CoroutineScope) : LLMClientService<Ope
         val builder = OpenAiChatModel.builder()
             .apiKey(token ?: "")
             .modelName(client.modelId)
-            .temperature(client.temperature.toDouble())
             .timeout(Duration.ofSeconds(client.timeout.toLong()))
             .topP(client.topP)
             .baseUrl(client.host)
 
+        client.temperature.takeIf { it.isNotBlank() }?.let {
+            builder.temperature(it.toDouble())
+        }
+
         client.organizationId?.takeIf { it.isNotBlank() }?.let {
             builder.organizationId(it)
         }
@@ -48,11 +51,14 @@ class OpenAiClientService(private val cs: CoroutineScope) : LLMClientService<Ope
         val builder = OpenAiStreamingChatModel.builder()
             .apiKey(token ?: "")
             .modelName(client.modelId)
-            .temperature(client.temperature.toDouble())
             .timeout(Duration.ofSeconds(client.timeout.toLong()))
             .topP(client.topP)
             .baseUrl(client.host)
 
+        client.temperature.takeIf { it.isNotBlank() }?.let {
+            builder.temperature(it.toDouble())
+        }
+
         client.organizationId?.takeIf { it.isNotBlank() }?.let {
             builder.organizationId(it)
         }