fix gson npe when deserializing older preset versions

Although, I am fairly certain that R8 is causing this issue
by mistakenly removing a critical member of PresetV* classes.
After extensive debugging, I couldn't figure out what exactly
it removes, but it seemingly concerns parameterized list type
and player state class. Either adding `@Keep` annotation or
replacing List types with Array resolves the issue.

Author: ashutoshgngwr

app/src/main/java/com/github/ashutoshgngwr/noice/models/Preset.kt

@@ -56,10 +56,10 @@ fun Preset.toRoomDto(gson: Gson): PresetDto {
 data class PresetV2(
   val id: String,
   val name: String,
-  val playerStates: List<PlayerState>,
-) : Serializable {
+  val playerStates: Array<PlayerState>,
+) {
 
-  constructor(name: String, playerStates: List<PlayerState>)
+  constructor(name: String, playerStates: Array<PlayerState>)
     : this(UUID.randomUUID().toString(), name, playerStates)
 
   /**
@@ -70,13 +70,27 @@ data class PresetV2(
     return Preset(id = id, name = name, soundStates = soundsV3)
   }
 
-  companion object {
-    val GSON_TYPE_PLAYER_STATES: Type = TypeToken
-      .getParameterized(List::class.java, PlayerState::class.java)
-      .type
+  override fun equals(other: Any?): Boolean {
+    if (this === other) return true
+    if (javaClass != other?.javaClass) return false
+
+    other as PresetV2
+
+    if (id != other.id) return false
+    if (name != other.name) return false
+    if (!playerStates.contentEquals(other.playerStates)) return false
+
+    return true
+  }
+
+  override fun hashCode(): Int {
+    var result = id.hashCode()
+    result = 31 * result + name.hashCode()
+    result = 31 * result + playerStates.contentHashCode()
+    return result
   }
 
-  data class PlayerState(val soundId: String, val volume: Int) : Serializable
+  data class PlayerState(val soundId: String, val volume: Int)
 }
 
 /**
@@ -95,7 +109,7 @@ data class PresetV2(
 data class PresetV1(
   val id: String,
   val name: String,
-  val playerStates: List<PlayerState>,
+  val playerStates: Array<PlayerState>,
 ) {
 
   /**
@@ -113,7 +127,27 @@ data class PresetV1(
     }
 
     playerStatesV2.sortBy { it.soundId }
-    return PresetV2(id = id, name = name, playerStates = playerStatesV2)
+    return PresetV2(id = id, name = name, playerStates = playerStatesV2.toTypedArray())
+  }
+
+  override fun equals(other: Any?): Boolean {
+    if (this === other) return true
+    if (javaClass != other?.javaClass) return false
+
+    other as PresetV1
+
+    if (id != other.id) return false
+    if (name != other.name) return false
+    if (!playerStates.contentEquals(other.playerStates)) return false
+
+    return true
+  }
+
+  override fun hashCode(): Int {
+    var result = id.hashCode()
+    result = 31 * result + name.hashCode()
+    result = 31 * result + playerStates.contentHashCode()
+    return result
   }
 
   companion object {
@@ -176,7 +210,7 @@ data class PresetV1(
  */
 data class PresetV0(
   @SerializedName("a") val name: String,
-  @SerializedName("b") val playerStates: List<PlayerState>,
+  @SerializedName("b") val playerStates: Array<PlayerState>,
 ) {
 
   /**
@@ -194,10 +228,29 @@ data class PresetV0(
             volume = (it.volume * 25).roundToInt(),
             timePeriod = it.timePeriod + 30,
           )
-        },
+        }
+        .toTypedArray(),
     )
   }
 
+  override fun equals(other: Any?): Boolean {
+    if (this === other) return true
+    if (javaClass != other?.javaClass) return false
+
+    other as PresetV0
+
+    if (name != other.name) return false
+    if (!playerStates.contentEquals(other.playerStates)) return false
+
+    return true
+  }
+
+  override fun hashCode(): Int {
+    var result = name.hashCode()
+    result = 31 * result + playerStates.contentHashCode()
+    return result
+  }
+
   data class PlayerState(
     @SerializedName("a") val soundKey: String,
     @SerializedName("b") val volume: Double,

app/src/main/java/com/github/ashutoshgngwr/noice/repository/PresetRepository.kt

@@ -242,7 +242,7 @@ class PresetRepository @Inject constructor(
       URI_VERSION_0 -> {
         val playerStatesJson = uri.getQueryParameter(URI_PARAM_PLAYER_STATES) ?: return null
         return try {
-          PresetV2(name, gson.fromJson(playerStatesJson, PresetV2.GSON_TYPE_PLAYER_STATES))
+          PresetV2(name, gson.fromJson(playerStatesJson, Array<PresetV2.PlayerState>::class.java))
             .toPresetV3()
         } catch (e: JsonSyntaxException) {
           null