feat(cli): ensure category is preserved during update

Author: wmontwe

cli/weblate-cli/src/main/kotlin/net/thunderbird/cli/weblate/WeblateCli.kt

@@ -9,6 +9,7 @@ import com.github.ajalt.clikt.parameters.options.required
 import java.io.File
 import net.thunderbird.cli.weblate.api.Component
 import net.thunderbird.cli.weblate.api.ComponentConfig
+import net.thunderbird.cli.weblate.api.ComponentPatch
 import net.thunderbird.cli.weblate.api.WeblateClient
 
 @Suppress("TooGenericExceptionCaught")
@@ -68,7 +69,14 @@ class WeblateCli : CliktCommand(
             diffs.forEach { println("     $it") }
             if (!dryRun) {
                 try {
-                    val result = client.patchComponent(token, component.info.url, goldenConfig)
+                    val result = client.patchComponent(
+                        token,
+                        component.info.url,
+                        ComponentPatch(
+                            category = component.info.category,
+                            config = goldenConfig,
+                        ),
+                    )
                     if (result) {
                         println("    ✅ Updated component config successfully")
                     } else {

cli/weblate-cli/src/main/kotlin/net/thunderbird/cli/weblate/api/ComponentInfo.kt

@@ -9,11 +9,13 @@ import kotlinx.serialization.Serializable
  * @property name The name of the component.
  * @property url The URL of the component in Weblate.
  * @property slug The slug identifier of the component (suitable for matching/include lists)
+ * @property category The category of the component
  */
 @Serializable
 data class ComponentInfo(
     val id: Int,
     val name: String,
     val slug: String,
     val url: String,
+    val category: String?,
 )

cli/weblate-cli/src/main/kotlin/net/thunderbird/cli/weblate/api/ComponentPatch.kt

@@ -0,0 +1,46 @@
+package net.thunderbird.cli.weblate.api
+
+import kotlinx.serialization.KSerializer
+import kotlinx.serialization.Serializable
+import kotlinx.serialization.descriptors.SerialDescriptor
+import kotlinx.serialization.descriptors.buildClassSerialDescriptor
+import kotlinx.serialization.descriptors.element
+import kotlinx.serialization.encoding.Decoder
+import kotlinx.serialization.encoding.Encoder
+import kotlinx.serialization.encoding.encodeStructure
+
+/**
+ * Weblate Component Patch
+ *
+ * We need the category to prevent the API from resetting it to undefined when we update the config.
+ *
+ * @property category The category of the component
+ * @property config The configuration of the component to be updated
+ */
+@Serializable(with = ComponentPatch.ComponentPatchSerializer::class)
+data class ComponentPatch(
+    val category: String?,
+    val config: ComponentConfig,
+) {
+    companion object ComponentPatchSerializer : KSerializer<ComponentPatch> {
+        override val descriptor: SerialDescriptor = buildClassSerialDescriptor("ComponentPatch") {
+            element<String>("category")
+            element("config", ComponentConfig.serializer().descriptor)
+        }
+
+        override fun deserialize(decoder: Decoder): ComponentPatch {
+            error("Deserialization is not supported for ComponentPatch")
+        }
+
+        override fun serialize(encoder: Encoder, value: ComponentPatch) {
+            encoder.encodeStructure(descriptor) {
+                if (value.category != null) {
+                    encodeStringElement(descriptor, 0, value.category)
+                    encodeSerializableElement(descriptor, 1, ComponentConfig.serializer(), value.config)
+                } else {
+                    encodeSerializableElement(descriptor, 0, ComponentConfig.serializer(), value.config)
+                }
+            }
+        }
+    }
+}

cli/weblate-cli/src/main/kotlin/net/thunderbird/cli/weblate/api/WeblateClient.kt

@@ -41,7 +41,7 @@ class WeblateClient(
         return components
     }
 
-    fun patchComponent(token: String, url: String, patch: ComponentConfig): Boolean {
+    fun patchComponent(token: String, url: String, patch: ComponentPatch): Boolean {
         var success = false
 
         runBlocking {