chore: fixes around identity reveal; update chat title for balance screen if known

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

Author: bmc08gt

api/src/main/java/com/getcode/mapper/ConversationMapper.kt

@@ -2,6 +2,8 @@ package com.getcode.mapper
 
 import com.getcode.model.Conversation
 import com.getcode.model.chat.Chat
+import com.getcode.model.chat.self
+import com.getcode.network.TipController
 import com.getcode.network.localized
 import com.getcode.network.repository.base58
 import com.getcode.util.resources.ResourceHelper
@@ -12,12 +14,13 @@ class ConversationMapper @Inject constructor(
 ) : Mapper<Chat, Conversation> {
     override fun map(from: Chat): Conversation {
 
+        val self = from.self?.identity
         val identity = from.members.filterNot { it.isSelf }.firstNotNullOfOrNull { it.identity }
 
         return Conversation(
             idBase58 = from.id.base58,
             title = from.title.localized(resources),
-            hasRevealedIdentity = identity != null,
+            hasRevealedIdentity = self != null,
             lastActivity = null, // TODO: ?
             user = identity?.username,
             userImage = null,

api/src/main/java/com/getcode/network/ConversationController.kt

@@ -15,16 +15,16 @@ import com.getcode.model.ConversationWithLastPointers
 import com.getcode.model.ID
 import com.getcode.model.MessageStatus
 import com.getcode.model.chat.ChatType
+import com.getcode.model.chat.MessageContent
 import com.getcode.model.chat.OutgoingMessageContent
 import com.getcode.model.chat.Platform
+import com.getcode.model.chat.isConversation
 import com.getcode.model.chat.selfId
-import com.getcode.model.uuid
 import com.getcode.network.client.ChatMessageStreamReference
 import com.getcode.network.exchange.Exchange
 import com.getcode.network.repository.base58
 import com.getcode.network.service.ChatServiceV2
 import com.getcode.utils.ErrorUtils
-import com.getcode.utils.timestamp
 import kotlinx.coroutines.CoroutineScope
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.flow.Flow
@@ -137,6 +137,21 @@ class ConversationStreamController @Inject constructor(
                     messageWithContentMapper.map(chat.id to it)
                 }
 
+                val identityRevealed = messages
+                    .flatMap { it.contents }
+                    .filterIsInstance<MessageContent.IdentityRevealed>()
+                    .firstOrNull()
+                    .takeIf { chat.isConversation }
+
+                if (identityRevealed != null && conversation.user == null) {
+                    scope.launch(Dispatchers.IO) {
+                        db.conversationDao()
+                            .upsertConversations(
+                                conversation.copy(user = identityRevealed.identity.username)
+                            )
+                    }
+                }
+
                 println("chat messages: ${messages.count()}, pointers=${pointers.count()}")
 
                 scope.launch(Dispatchers.IO) {

api/src/main/java/com/getcode/network/HistoryController.kt

@@ -246,6 +246,21 @@ class HistoryController @Inject constructor(
     private suspend fun fetchChatsWithoutMessages(): List<Chat> {
         val owner = owner() ?: return emptyList()
         val result = client.fetchChats(owner)
+            .map { chats ->
+                chats.map { chat ->
+                    // map revealed identity as title if known
+                    if (chat.isConversation) {
+                        val conversation = conversationMapper.map(chat)
+                        if (conversation.user != null) {
+                            chat.copy(title = Title.Localized(conversation.user))
+                        } else {
+                            chat
+                        }
+                    } else {
+                        chat
+                    }
+                }
+            }
             .onSuccess { result ->
                 result.filter { it.isConversation }
                     .let { chats ->
@@ -257,11 +272,8 @@ class HistoryController @Inject constructor(
                         chats
                     }
                     .onEach {
-                        if (db.conversationDao().findConversation(it.id) == null) {
-                            trace("adding conversation for chat ${it.id.description}")
-                            val conversation = conversationMapper.map(it)
-                            db.conversationDao().upsertConversations(conversation)
-                        }
+                        val conversation = conversationMapper.map(it)
+                        db.conversationDao().upsertConversations(conversation)
                     }
             }
         return result.getOrNull().orEmpty()