refactor: rename 'id' to 'providerId' in serialization and requests to comply the spec

Author: Rizzen

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<String, String>? = 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<LlmProtocol>,
     val required: Boolean,
     val current: ProviderCurrentConfig? = null,

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)