Fix Kotlin 2.3.0 warnings to maintain allWarningsAsErrors

[Copilot]

Kotlin 2.3.0 introduced stricter compiler warnings. This PR fixes all warnings to maintain allWarningsAsErrors enabled.

Changes

• Fixed all Kotlin 2.3.0 warnings across 22 files while keeping allWarningsAsErrors.set(true) enabled
• Removed redundant else branches in exhaustive when expressions (2 occurrences)
• Removed unnecessary safe calls (?.) on non-null receivers (11 occurrences)
• Removed Elvis operators (?:) that always return the left operand (13 occurrences)
• Removed unnecessary type casts (2 occurrences)
• Fixed serializer visibility to match data class visibility (1 occurrence)

Warnings Fixed

The warnings introduced in Kotlin 2.3.0 that were addressed:

• Redundant else in exhaustive when expressions
• Unnecessary safe calls (?.) on non-null receivers
• Elvis operators (?:) that always return the left operand
• Unnecessary type casts
• Visibility mismatch between custom serializer and data class

These code quality improvements align with Kotlin 2.3.0's stricter type checking and null-safety analysis.

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[Copilot]

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• central.sonatype.com
  • Triggering command: /usr/lib/jvm/temurin-17-jdk-amd64/bin/java /usr/lib/jvm/temurin-17-jdk-amd64/bin/java --add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.lang.invoke=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED --add-exports=jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-opens=java.base/java.nio.charset=ALL-UNNAMED --add-opens=java.base/java.net=ALL-UNNAMED --add-opens=java.base/java.util.concurrent=ALL-UNNAMED --add-opens=java.base/java.util.concurrent.atomic=ALL-UNNAMED --add-opens=java.xml/javax.xml.namespace=ALL-UNNAMED -Xmx8g -Dfile.encoding=UTF-8 -Duser.country -Duser.language=en -Duser.variant -cp (dns block)
• jitpack.io
  • Triggering command: /usr/lib/jvm/temurin-17-jdk-amd64/bin/java /usr/lib/jvm/temurin-17-jdk-amd64/bin/java --add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.lang.invoke=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED --add-exports=jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-opens=java.base/java.nio.charset=ALL-UNNAMED --add-opens=java.base/java.net=ALL-UNNAMED --add-opens=java.base/java.util.concurrent=ALL-UNNAMED --add-opens=java.base/java.util.concurrent.atomic=ALL-UNNAMED --add-opens=java.xml/javax.xml.namespace=ALL-UNNAMED -Xmx8g -Dfile.encoding=UTF-8 -Duser.country -Duser.language=en -Duser.variant -cp (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

[Tlaster]

@copilot please keep warning as error.

[Copilot]

@copilot please keep warning as error.

Fixed all Kotlin 2.3.0 warnings while keeping allWarningsAsErrors enabled. Changes include removing redundant else branches, unnecessary safe calls, and Elvis operators. See commit 1e1d4d0.