Skip to content

Commit 7d3908d

Browse files
committed
refactor(dsl)!: mark constructors in DSL builders as internal to restrict visibility
1 parent 7ad5cad commit 7d3908d

6 files changed

Lines changed: 10 additions & 9 deletions

File tree

kotlin-sdk-core/api/kotlin-sdk-core.api

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3675,6 +3675,7 @@ public final class io/modelcontextprotocol/kotlin/sdk/types/RequestMeta$Companio
36753675
}
36763676

36773677
public final class io/modelcontextprotocol/kotlin/sdk/types/RequestMetaBuilder {
3678+
public fun <init> ()V
36783679
public final fun progressToken (I)V
36793680
public final fun progressToken (J)V
36803681
public final fun progressToken (Ljava/lang/String;)V

kotlin-sdk-core/src/commonMain/kotlin/io/modelcontextprotocol/kotlin/sdk/types/content.dsl.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ import kotlinx.serialization.json.buildJsonObject
2121
* @see AudioContentBuilder
2222
*/
2323
@McpDsl
24-
public abstract class MediaContentBuilder {
24+
public abstract class MediaContentBuilder @PublishedApi internal constructor() {
2525
protected var annotations: Annotations? = null
2626
protected var meta: JsonObject? = null
2727

kotlin-sdk-core/src/commonMain/kotlin/io/modelcontextprotocol/kotlin/sdk/types/prompts.dsl.kt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -155,7 +155,7 @@ public class ListPromptsRequestBuilder @PublishedApi internal constructor() : Pa
155155
* Creates a [GetPromptResult] using a type-safe DSL builder.
156156
*
157157
* ## Required
158-
* - [messages][GetPromptResultBuilder.messages] - List of prompt messages (at least one)
158+
* - [messages][GetPromptResultBuilder.messagesList] - List of prompt messages (at least one)
159159
*
160160
* ## Optional
161161
* - [description][GetPromptResultBuilder.description] - Description of the prompt
@@ -260,7 +260,7 @@ public class GetPromptResultBuilder @PublishedApi internal constructor() : Resul
260260
* Creates a [ListPromptsResult] using a type-safe DSL builder.
261261
*
262262
* ## Required
263-
* - [prompts][ListPromptsResultBuilder.prompts] - List of available prompts (at least one)
263+
* - [prompts][ListPromptsResultBuilder.promptsList] - List of available prompts (at least one)
264264
*
265265
* ## Optional
266266
* - [nextCursor][ListPromptsResultBuilder.nextCursor] - Pagination cursor for next page

kotlin-sdk-core/src/commonMain/kotlin/io/modelcontextprotocol/kotlin/sdk/types/request.dsl.kt

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ import kotlinx.serialization.json.buildJsonObject
2121
* @see PaginatedRequestBuilder
2222
*/
2323
@McpDsl
24-
public abstract class RequestBuilder {
24+
public abstract class RequestBuilder @PublishedApi internal constructor() {
2525
protected var meta: RequestMeta? = null
2626

2727
/**
@@ -73,7 +73,7 @@ public abstract class RequestBuilder {
7373
* @see RequestBuilder.meta
7474
*/
7575
@McpDsl
76-
public class RequestMetaBuilder internal constructor() {
76+
public class RequestMetaBuilder @PublishedApi internal constructor() {
7777
private val content: MutableMap<String, JsonElement> = linkedMapOf()
7878

7979
/**
@@ -196,7 +196,7 @@ public class RequestMetaBuilder internal constructor() {
196196
* @see RequestBuilder
197197
*/
198198
@McpDsl
199-
public abstract class PaginatedRequestBuilder : RequestBuilder() {
199+
public abstract class PaginatedRequestBuilder @PublishedApi internal constructor() : RequestBuilder() {
200200
/**
201201
* Optional pagination cursor for fetching the next page of results.
202202
*

kotlin-sdk-core/src/commonMain/kotlin/io/modelcontextprotocol/kotlin/sdk/types/result.dsl.kt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ import kotlinx.serialization.json.buildJsonObject
1616
* @see ServerResult
1717
*/
1818
@McpDsl
19-
public abstract class ResultBuilder {
19+
public abstract class ResultBuilder @PublishedApi internal constructor() {
2020
protected var meta: JsonObject? = null
2121

2222
/**
@@ -78,7 +78,7 @@ public abstract class ResultBuilder {
7878
* @see PaginatedResult
7979
*/
8080
@McpDsl
81-
public abstract class PaginatedResultBuilder : ResultBuilder() {
81+
public abstract class PaginatedResultBuilder @PublishedApi internal constructor() : ResultBuilder() {
8282
/**
8383
* Optional pagination cursor for fetching the next page of results.
8484
*

kotlin-sdk-core/src/commonMain/kotlin/io/modelcontextprotocol/kotlin/sdk/types/tools.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ public data class Tool(
8484
public data class ToolSchema(
8585
val properties: JsonObject? = null,
8686
val required: List<String>? = null,
87-
@SerialName("\$defs")
87+
@SerialName($$"$defs")
8888
val defs: JsonObject? = null,
8989
) {
9090
@OptIn(ExperimentalSerializationApi::class)

0 commit comments

Comments
 (0)