fixup! Also check collisions for JsonNamingStrategy-transformed names during serialization

sandwwraith · sandwwraith · commit 7d45b9af01da · 2025-11-13T18:52:18.000+07:00
diff --git a/formats/json/commonMain/src/kotlinx/serialization/json/internal/JsonNamesMap.kt b/formats/json/commonMain/src/kotlinx/serialization/json/internal/JsonNamesMap.kt
@@ -59,7 +59,7 @@ internal fun SerialDescriptor.serializationNamesIndices(json: Json, strategy: Js
         Array(elementsCount) { i ->
             val baseName = getElementName(i)
             val name = strategy.serialNameForJson(this, i, baseName)
-            if (!trackingSet.add(name)) throw JsonException(
+            if (!trackingSet.add(name)) throw JsonEncodingException(
                 "The transformed name '$name' for property $baseName already exists " +
                     "in ${this@serializationNamesIndices}"
             )

Original file line number	Diff line number	Diff line change
`@@ -59,7 +59,7 @@ internal fun SerialDescriptor.serializationNamesIndices(json: Json, strategy: Js`
`59`	`59`	`Array(elementsCount) { i ->`
`60`	`60`	`val baseName = getElementName(i)`
`61`	`61`	`val name = strategy.serialNameForJson(this, i, baseName)`
`62`		`- if (!trackingSet.add(name)) throw JsonException(`
	`62`	`+ if (!trackingSet.add(name)) throw JsonEncodingException(`
`63`	`63`	`"The transformed name '$name' for property $baseName already exists " +`
`64`	`64`	`"in ${this@serializationNamesIndices}"`
`65`	`65`	`)`