diff --git a/acp-model/src/commonMain/kotlin/com/agentclientprotocol/model/Requests.kt b/acp-model/src/commonMain/kotlin/com/agentclientprotocol/model/Requests.kt index 9df2608..28e3238 100644 --- a/acp-model/src/commonMain/kotlin/com/agentclientprotocol/model/Requests.kt +++ b/acp-model/src/commonMain/kotlin/com/agentclientprotocol/model/Requests.kt @@ -522,7 +522,7 @@ public data class ListProvidersRequest( @UnstableApi @Serializable public data class SetProvidersRequest( - val id: String, + @SerialName("providerId") val id: String, val apiType: LlmProtocol, val baseUrl: String, val headers: Map? = null, @@ -539,7 +539,7 @@ public data class SetProvidersRequest( @UnstableApi @Serializable public data class DisableProvidersRequest( - val id: String, + @SerialName("providerId") val id: String, override val _meta: JsonElement? = null ) : AcpRequest @@ -663,7 +663,7 @@ public data class ProviderCurrentConfig( @UnstableApi @Serializable public data class ProviderInfo( - val id: String, + @SerialName("providerId") val id: String, val supported: List, val required: Boolean, val current: ProviderCurrentConfig? = null, diff --git a/acp-model/src/commonTest/kotlin/com/agentclientprotocol/model/ProvidersSerializationTest.kt b/acp-model/src/commonTest/kotlin/com/agentclientprotocol/model/ProvidersSerializationTest.kt index 173edbb..f88468b 100644 --- a/acp-model/src/commonTest/kotlin/com/agentclientprotocol/model/ProvidersSerializationTest.kt +++ b/acp-model/src/commonTest/kotlin/com/agentclientprotocol/model/ProvidersSerializationTest.kt @@ -57,6 +57,8 @@ class ProvidersSerializationTest { val encoded = ACPJson.encodeToString(ProviderInfo.serializer(), info) val decoded = ACPJson.decodeFromString(ProviderInfo.serializer(), encoded) + assertTrue(encoded.contains("\"providerId\"")) + assertFalse(encoded.contains("\"id\"")) assertEquals(info, decoded) } @@ -80,7 +82,7 @@ class ProvidersSerializationTest { fun `incoming JSON with current null is accepted and mapped to disabled`() { val payload = """ { - "id": "main", + "providerId": "main", "supported": ["openai"], "required": false, "current": null @@ -105,6 +107,7 @@ class ProvidersSerializationTest { val withoutHeadersJson = ACPJson.encodeToString(SetProvidersRequest.serializer(), withoutHeaders) val withoutHeadersDecoded = ACPJson.decodeFromString(SetProvidersRequest.serializer(), withoutHeadersJson) + assertTrue(withoutHeadersJson.contains("\"providerId\"")) assertFalse(withoutHeadersJson.contains("\"headers\"")) assertNull(withoutHeadersDecoded.headers)