fix(fc/conversation): handle potential null pointer in sequenced map

Signed-off-by: Brandon McAnsh <git@bmcreations.dev>

Author: bmc08gt

flipchatApp/src/main/kotlin/xyz/flipchat/app/features/chat/conversation/ConversationViewModel.kt

@@ -113,7 +113,7 @@ class ConversationViewModel @Inject constructor(
         val imageUri: String?,
         val lastSeen: Instant?,
         val members: Int?,
-        val pointers: Map<UUID, MessageStatus>,
+        val pointers: Map<UUID, MessageStatus?>,
         val pointerRefs: Map<Long, MessageStatus>,
         val showTypingIndicator: Boolean,
         val isSelfTyping: Boolean,
@@ -963,7 +963,9 @@ class ConversationViewModel @Inject constructor(
                             ?: findLastReadMessage(event.conversationWithPointers.pointers),
                         pointerRefs = event.conversationWithPointers.pointers
                             .asSequence()
-                            .mapNotNull { (key, value) -> key.timestamp?.let { it to value } }
+                            .mapNotNull { (key, value) ->
+                                key.timestamp?.let { it to (value ?: MessageStatus.Unknown) }
+                            }
                             .toMap(),
                         members = members.count(),
                         hostId = host?.id,
@@ -1045,7 +1047,7 @@ class ConversationViewModel @Inject constructor(
 }
 
 private fun findLastReadMessage(
-    statusMap: Map<UUID, MessageStatus>,
+    statusMap: Map<UUID, MessageStatus?>,
 ): UUID? {
     return statusMap
         .filter { it.value == MessageStatus.Read }

services/flipchat/chat/src/main/kotlin/xyz/flipchat/services/domain/model/chat/Conversation.kt

@@ -60,7 +60,7 @@ data class ConversationWithMembersAndLastPointers(
     )
     val pointersCrossRef: List<ConversationPointerCrossRef>,
 ) {
-    val pointers: Map<UUID, MessageStatus>
+    val pointers: Map<UUID, MessageStatus?>
         get() {
             return pointersCrossRef
                 .associateBy { it.status }