Skip to content

Commit 9074fe1

Browse files
fix(client): make some classes and constructors non-public (#169)
chore(internal): remove some unnecessary `constructor` keywords
1 parent b744bac commit 9074fe1

File tree

233 files changed

+613
-630
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

233 files changed

+613
-630
lines changed

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -171,7 +171,7 @@ private constructor(private val okHttpClient: okhttp3.OkHttpClient, private val
171171
@JvmStatic fun builder() = Builder()
172172
}
173173

174-
class Builder {
174+
class Builder internal constructor() {
175175

176176
private var baseUrl: HttpUrl? = null
177177
// The default timeout is 10 minutes.

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ class OpenAIOkHttpClient private constructor() {
2424
@JvmStatic fun fromEnv(): OpenAIClient = builder().fromEnv().build()
2525
}
2626

27-
class Builder {
27+
class Builder internal constructor() {
2828

2929
private var clientOptions: ClientOptions.Builder = ClientOptions.builder()
3030
private var baseUrl: String = ClientOptions.PRODUCTION_URL

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ class OpenAIOkHttpClientAsync private constructor() {
2424
@JvmStatic fun fromEnv(): OpenAIClientAsync = builder().fromEnv().build()
2525
}
2626

27-
class Builder {
27+
class Builder internal constructor() {
2828

2929
private var clientOptions: ClientOptions.Builder = ClientOptions.builder()
3030
private var baseUrl: String = ClientOptions.PRODUCTION_URL

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

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,7 @@ import com.openai.services.async.ModerationServiceAsyncImpl
2727
import com.openai.services.async.UploadServiceAsync
2828
import com.openai.services.async.UploadServiceAsyncImpl
2929

30-
class OpenAIClientAsyncImpl
31-
constructor(
30+
class OpenAIClientAsyncImpl(
3231
private val clientOptions: ClientOptions,
3332
) : OpenAIClientAsync {
3433

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

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,7 @@ import com.openai.services.blocking.ModerationServiceImpl
2727
import com.openai.services.blocking.UploadService
2828
import com.openai.services.blocking.UploadServiceImpl
2929

30-
class OpenAIClientImpl
31-
constructor(
30+
class OpenAIClientImpl(
3231
private val clientOptions: ClientOptions,
3332
) : OpenAIClient {
3433

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ private constructor(
4848
@JvmStatic fun fromEnv(): ClientOptions = builder().fromEnv().build()
4949
}
5050

51-
class Builder {
51+
class Builder internal constructor() {
5252

5353
private var httpClient: HttpClient? = null
5454
private var jsonMapper: JsonMapper = jsonMapper()

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

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,8 @@ private constructor(
2323
@JvmStatic fun builder() = Builder()
2424
}
2525

26-
class Builder {
26+
class Builder internal constructor() {
27+
2728
private var responseValidation: Boolean? = null
2829
private var timeout: Duration? = null
2930

openai-java-core/src/main/kotlin/com/openai/core/http/Headers.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ private constructor(
2222
@JvmStatic fun builder() = Builder()
2323
}
2424

25-
class Builder {
25+
class Builder internal constructor() {
2626

2727
private val map: MutableMap<String, MutableList<String>> =
2828
TreeMap(String.CASE_INSENSITIVE_ORDER)

openai-java-core/src/main/kotlin/com/openai/core/http/HttpRequest.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ private constructor(
2222
@JvmStatic fun builder() = Builder()
2323
}
2424

25-
class Builder {
25+
class Builder internal constructor() {
2626

2727
private var method: HttpMethod? = null
2828
private var url: String? = null

openai-java-core/src/main/kotlin/com/openai/core/http/QueryParams.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ private constructor(
2121
@JvmStatic fun builder() = Builder()
2222
}
2323

24-
class Builder {
24+
class Builder internal constructor() {
2525

2626
private val map: MutableMap<String, MutableList<String>> = mutableMapOf()
2727
private var size: Int = 0

0 commit comments

Comments
 (0)