chore(internal): add and tweak check functions (#117)

chore(internal): tweak client options nullability handling

Author: stainless-app[bot]

openai-java-core/src/main/kotlin/com/openai/core/Check.kt

@@ -4,4 +4,27 @@ package com.openai.core
 
 @JvmSynthetic
 internal fun <T : Any> checkRequired(name: String, value: T?): T =
-    checkNotNull(value) { "`$name` is required but was not set" }
+    checkNotNull(value) { "`$name` is required, but was not set" }
+
+@JvmSynthetic
+internal fun checkLength(name: String, value: String, length: Int): String =
+    value.also {
+        check(it.length == length) { "`$name` must have length $length, but was ${it.length}" }
+    }
+
+@JvmSynthetic
+internal fun checkMinLength(name: String, value: String, minLength: Int): String =
+    value.also {
+        check(it.length >= minLength) {
+            if (minLength == 1) "`$name` must be non-empty, but was empty"
+            else "`$name` must have at least length $minLength, but was ${it.length}"
+        }
+    }
+
+@JvmSynthetic
+internal fun checkMaxLength(name: String, value: String, maxLength: Int): String =
+    value.also {
+        check(it.length <= maxLength) {
+            "`$name` must have at most length $maxLength, but was ${it.length}"
+        }
+    }

openai-java-core/src/main/kotlin/com/openai/core/ClientOptions.kt

@@ -230,8 +230,8 @@ private constructor(
         }
 
         fun build(): ClientOptions {
-            checkRequired("httpClient", httpClient)
-            checkRequired("credential", credential)
+            val httpClient = checkRequired("httpClient", httpClient)
+            val credential = checkRequired("credential", credential)
 
             val headers = Headers.builder()
             val queryParams = QueryParams.builder()
@@ -245,12 +245,12 @@ private constructor(
             organization?.let { headers.put("OpenAI-Organization", it) }
             project?.let { headers.put("OpenAI-Project", it) }
 
-            when (val currentCredential = credential) {
+            when (credential) {
                 is AzureApiKeyCredential -> {
-                    headers.put("api-key", currentCredential.apiKey())
+                    headers.put("api-key", credential.apiKey())
                 }
                 is BearerTokenCredential -> {
-                    headers.put("Authorization", "Bearer ${currentCredential.token()}")
+                    headers.put("Authorization", "Bearer ${credential.token()}")
                 }
                 else -> {
                     throw IllegalArgumentException("Invalid credential type")
@@ -268,10 +268,10 @@ private constructor(
             queryParams.replaceAll(this.queryParams.build())
 
             return ClientOptions(
-                httpClient!!,
+                httpClient,
                 PhantomReachableClosingHttpClient(
                     RetryingHttpClient.builder()
-                        .httpClient(httpClient!!)
+                        .httpClient(httpClient)
                         .clock(clock)
                         .maxRetries(maxRetries)
                         .build()
@@ -298,7 +298,7 @@ private constructor(
                 queryParams.build(),
                 responseValidation,
                 maxRetries,
-                credential!!,
+                credential,
                 organization,
                 project,
             )

openai-java-core/src/main/kotlin/com/openai/core/Values.kt

@@ -117,6 +117,8 @@ sealed class JsonField<out T : Any> {
             is JsonValue -> this
         }
 
+    @JvmSynthetic fun accept(consume: (T) -> Unit) = asKnown().ifPresent(consume)
+
     fun <R> accept(visitor: Visitor<T, R>): R =
         when (this) {
             is KnownValue -> visitor.visitKnown(value)