chore(internal): update formatter (#220)

chore(internal): optimize build and test perf

Author: stainless-app[bot]

buildSrc/build.gradle.kts

@@ -1,6 +1,6 @@
 plugins {
     `kotlin-dsl`
-    kotlin("jvm") version "2.1.0"
+    kotlin("jvm") version "2.1.10"
     id("com.vanniktech.maven.publish") version "0.28.0"
 }
 
@@ -10,7 +10,7 @@ repositories {
 }
 
 dependencies {
-    implementation("com.diffplug.spotless:spotless-plugin-gradle:6.25.0")
-    implementation("org.jetbrains.kotlin:kotlin-gradle-plugin:1.9.23")
+    implementation("com.diffplug.spotless:spotless-plugin-gradle:7.0.2")
+    implementation("org.jetbrains.kotlin:kotlin-gradle-plugin:2.1.10")
     implementation("com.vanniktech:gradle-maven-publish-plugin:0.28.0")
 }

buildSrc/src/main/kotlin/openai.java.gradle.kts

@@ -39,9 +39,13 @@ tasks.named<Jar>("jar") {
     }
 }
 
-tasks.named<Test>("test") {
+tasks.withType<Test>().configureEach {
     useJUnitPlatform()
 
+    // Run tests in parallel to some degree.
+    maxParallelForks = (Runtime.getRuntime().availableProcessors() / 2).coerceAtLeast(1)
+    forkEvery = 100
+
     testLogging {
         exceptionFormat = TestExceptionFormat.FULL
     }

buildSrc/src/main/kotlin/openai.kotlin.gradle.kts

@@ -1,4 +1,5 @@
 import com.diffplug.gradle.spotless.SpotlessExtension
+import org.jetbrains.kotlin.gradle.dsl.JvmTarget
 import org.jetbrains.kotlin.gradle.tasks.KotlinCompile
 
 plugins {
@@ -20,13 +21,19 @@ configure<SpotlessExtension> {
 }
 
 tasks.withType<KotlinCompile>().configureEach {
-    kotlinOptions {
+    compilerOptions {
         freeCompilerArgs = listOf(
           "-Xjvm-default=all",
           "-Xjdk-release=1.8",
           // Suppress deprecation warnings because we may still reference and test deprecated members.
           "-Xsuppress-warning=DEPRECATION"
         )
-        jvmTarget = "1.8"
+        jvmTarget.set(JvmTarget.JVM_1_8)
     }
 }
+
+// Run tests in parallel to some degree.
+tasks.withType<Test>().configureEach {
+    maxParallelForks = (Runtime.getRuntime().availableProcessors() / 2).coerceAtLeast(1)
+    forkEvery = 100
+}

gradle.properties

@@ -1,4 +1,6 @@
+org.gradle.configuration-cache=true
 org.gradle.caching=true
-org.gradle.jvmargs=-Xmx4g
 org.gradle.parallel=true
+org.gradle.daemon=false
+org.gradle.jvmargs=-Xmx4g
 kotlin.daemon.jvmargs=-Xmx4g

openai-java-client-okhttp/src/main/kotlin/com/openai/client/okhttp/OkHttpClient.kt

@@ -31,10 +31,7 @@ class OkHttpClient
 private constructor(private val okHttpClient: okhttp3.OkHttpClient, private val baseUrl: HttpUrl) :
     HttpClient {
 
-    override fun execute(
-        request: HttpRequest,
-        requestOptions: RequestOptions,
-    ): HttpResponse {
+    override fun execute(request: HttpRequest, requestOptions: RequestOptions): HttpResponse {
         val call = newCall(request, requestOptions)
 
         return try {
@@ -120,13 +117,13 @@ private constructor(private val okHttpClient: okhttp3.OkHttpClient, private val
         ) {
             builder.header(
                 "X-Stainless-Read-Timeout",
-                Duration.ofMillis(client.readTimeoutMillis.toLong()).seconds.toString()
+                Duration.ofMillis(client.readTimeoutMillis.toLong()).seconds.toString(),
             )
         }
         if (!headers.names().contains("X-Stainless-Timeout") && client.callTimeoutMillis != 0) {
             builder.header(
                 "X-Stainless-Timeout",
-                Duration.ofMillis(client.callTimeoutMillis.toLong()).seconds.toString()
+                Duration.ofMillis(client.callTimeoutMillis.toLong()).seconds.toString(),
             )
         }
 

openai-java-core/src/main/kotlin/com/openai/azure/HttpRequestBuilderExtensions.kt

@@ -8,7 +8,7 @@ import com.openai.credential.BearerTokenCredential
 @JvmSynthetic
 internal fun HttpRequest.Builder.addPathSegmentsForAzure(
     clientOptions: ClientOptions,
-    deploymentModel: String?
+    deploymentModel: String?,
 ): HttpRequest.Builder = apply {
     if (isAzureEndpoint(clientOptions.baseUrl) && deploymentModel != null) {
         addPathSegments("openai", "deployments", deploymentModel)

openai-java-core/src/main/kotlin/com/openai/client/OpenAIClientAsyncImpl.kt

@@ -27,9 +27,7 @@ import com.openai.services.async.ModerationServiceAsyncImpl
 import com.openai.services.async.UploadServiceAsync
 import com.openai.services.async.UploadServiceAsyncImpl
 
-class OpenAIClientAsyncImpl(
-    private val clientOptions: ClientOptions,
-) : OpenAIClientAsync {
+class OpenAIClientAsyncImpl(private val clientOptions: ClientOptions) : OpenAIClientAsync {
 
     private val clientOptionsWithUserAgent =
         if (clientOptions.headers.names().contains("User-Agent")) clientOptions

openai-java-core/src/main/kotlin/com/openai/client/OpenAIClientImpl.kt

@@ -27,9 +27,7 @@ import com.openai.services.blocking.ModerationServiceImpl
 import com.openai.services.blocking.UploadService
 import com.openai.services.blocking.UploadServiceImpl
 
-class OpenAIClientImpl(
-    private val clientOptions: ClientOptions,
-) : OpenAIClient {
+class OpenAIClientImpl(private val clientOptions: ClientOptions) : OpenAIClient {
 
     private val clientOptionsWithUserAgent =
         if (clientOptions.headers.names().contains("User-Agent")) clientOptions

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

@@ -18,7 +18,7 @@ abstract class BaseDeserializer<T : Any>(type: KClass<T>) :
 
     override fun createContextual(
         context: DeserializationContext,
-        property: BeanProperty?
+        property: BeanProperty?,
     ): JsonDeserializer<T> {
         return this
     }
@@ -32,7 +32,7 @@ abstract class BaseDeserializer<T : Any>(type: KClass<T>) :
     protected fun <T> ObjectCodec.tryDeserialize(
         node: JsonNode,
         type: TypeReference<T>,
-        validate: (T) -> Unit = {}
+        validate: (T) -> Unit = {},
     ): T? {
         return try {
             readValue(treeAsTokens(node), type).apply(validate)
@@ -46,7 +46,7 @@ abstract class BaseDeserializer<T : Any>(type: KClass<T>) :
     protected fun <T> ObjectCodec.tryDeserialize(
         node: JsonNode,
         type: JavaType,
-        validate: (T) -> Unit = {}
+        validate: (T) -> Unit = {},
     ): T? {
         return try {
             readValue<T>(treeAsTokens(node), type).apply(validate)

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

@@ -10,10 +10,7 @@ import java.io.OutputStream
 import org.apache.hc.client5.http.entity.mime.MultipartEntityBuilder
 
 @JvmSynthetic
-internal inline fun <reified T> json(
-    jsonMapper: JsonMapper,
-    value: T,
-): HttpRequestBody {
+internal inline fun <reified T> json(jsonMapper: JsonMapper, value: T): HttpRequestBody {
     return object : HttpRequestBody {
         private var cachedBytes: ByteArray? = null
 
@@ -49,7 +46,7 @@ internal inline fun <reified T> json(
 @JvmSynthetic
 internal fun multipartFormData(
     jsonMapper: JsonMapper,
-    parts: Array<MultipartFormValue<*>?>
+    parts: Array<MultipartFormValue<*>?>,
 ): HttpRequestBody {
     val builder = MultipartEntityBuilder.create()
     parts.forEach { part ->
@@ -66,14 +63,14 @@ internal fun multipartFormData(
                         part.name,
                         buffer.toByteArray(),
                         part.contentType,
-                        part.filename
+                        part.filename,
                     )
                 }
                 is Boolean ->
                     builder.addTextBody(
                         part.name,
                         if (part.value) "true" else "false",
-                        part.contentType
+                        part.contentType,
                     )
                 is Int -> builder.addTextBody(part.name, part.value.toString(), part.contentType)
                 is Long -> builder.addTextBody(part.name, part.value.toString(), part.contentType)