Fix crash issue

Author: ismartcoding

app/build.gradle.kts

@@ -41,9 +41,9 @@ android {
                 else -> 0
             }
 
-        val vCode = 412
+        val vCode = 415
         versionCode = vCode - singleAbiNum
-        versionName = "2.1.10"
+        versionName = "2.1.11"
 
         ndk {
             //noinspection ChromeOsAbiSupport

app/src/main/java/com/ismartcoding/plain/ui/MainActivity.kt

@@ -131,7 +131,7 @@ class MainActivity : AppCompatActivity() {
                     val service = Intent(this, ScreenMirrorService::class.java)
                     service.putExtra("code", result.resultCode)
                     service.putExtra("data", result.data)
-                    ContextCompat.startForegroundService(this, service)
+                    startService(service)
                 } else {
                     sendEvent(WebSocketEvent(EventType.SCREEN_MIRRORING, image))
                 }

app/src/main/java/com/ismartcoding/plain/ui/models/ChatListViewModel.kt

@@ -18,6 +18,7 @@ import com.ismartcoding.plain.web.ChatApiManager
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.Job
 import kotlinx.coroutines.launch
+import kotlinx.coroutines.withContext
 import kotlinx.datetime.Clock
 import kotlinx.datetime.Instant
 import kotlin.time.Duration.Companion.seconds
@@ -96,19 +97,35 @@ class ChatListViewModel : ViewModel() {
                     }
                 }
 
-                latestChatCache.clear()
-                latestChatCache.putAll(chatCache)
-
-                // Sort paired peers by latest chat time (most recent first), then by name
-                pairedPeers.clear()
-                pairedPeers.addAll(allPeers.filter { it.status == "paired" }.toList()
-                    .sortedWith(compareByDescending<DPeer> { getLatestChat(it.id)?.createdAt ?: Instant.DISTANT_PAST }.thenBy { Pinyin.toPinyin(it.name) }))
-                unpairedPeers.clear()
-                unpairedPeers.addAll(allPeers.filter { it.status == "unpaired" }.toList().sortedBy { Pinyin.toPinyin(it.name) })
+                // Prepare new lists off the main thread
+                val newPairedPeers = allPeers
+                    .filter { it.status == "paired" }
+                    .sortedWith(
+                        compareByDescending<DPeer> { peer ->
+                            chatCache[peer.id]?.createdAt ?: Instant.DISTANT_PAST
+                        }.thenBy { Pinyin.toPinyin(it.name) }
+                    )
+                val newUnpairedPeers = allPeers
+                    .filter { it.status == "unpaired" }
+                    .sortedBy { Pinyin.toPinyin(it.name) }
+
+                // Apply state updates on the main thread to avoid snapshot violations
+                withContext(Dispatchers.Main) {
+                    latestChatCache.clear()
+                    latestChatCache.putAll(chatCache)
+
+                    pairedPeers.clear()
+                    pairedPeers.addAll(newPairedPeers)
+
+                    unpairedPeers.clear()
+                    unpairedPeers.addAll(newUnpairedPeers)
+                }
             } catch (e: Exception) {
-                pairedPeers.clear()
-                unpairedPeers.clear()
-                latestChatCache.clear()
+                withContext(Dispatchers.Main) {
+                    pairedPeers.clear()
+                    unpairedPeers.clear()
+                    latestChatCache.clear()
+                }
             }
         }
     }
@@ -142,9 +159,12 @@ class ChatListViewModel : ViewModel() {
     }
 
     fun updatePeerLastActive(peerId: String) {
-        val currentMap = onlineMap.value.toMutableMap()
-        currentMap[peerId] = Clock.System.now()
-        onlineMap.value = currentMap
+        // Ensure state mutation happens on the main thread
+        viewModelScope.launch(Dispatchers.Main) {
+            val currentMap = onlineMap.value.toMutableMap()
+            currentMap[peerId] = Clock.System.now()
+            onlineMap.value = currentMap
+        }
     }
 
     fun isPeerOnline(peerId: String): Boolean {

app/src/main/java/com/ismartcoding/plain/ui/page/chat/components/ForwardTargetDialog.kt

@@ -56,7 +56,7 @@ fun ForwardTargetDialog(
                 }
 
                 if (pairedPeers.isNotEmpty()) {
-                    items(pairedPeers) { peer ->
+                    items(pairedPeers, key = { it.id }) { peer ->
                         PDialogListItem(
                             modifier = Modifier.clickable {
                                 onTargetSelected(ForwardTarget.Peer(peer))

app/src/main/java/com/ismartcoding/plain/ui/page/root/contents/TabContentChat.kt

@@ -142,8 +142,10 @@ fun TabContentChat(
     // Device discovery timer with dynamic interval
     LaunchedEffect(Unit) {
         while (isActive) {
+            // Snapshot the current peers list on the main thread to avoid reading state on IO
+            val peersSnapshot = pairedPeers.toList()
             scope.launch(Dispatchers.IO) {
-                pairedPeers.forEach { peer ->
+                peersSnapshot.forEach { peer ->
                     val key = ChatApiManager.peerKeyCache[peer.id]
                     if (key != null) {
                         try {
@@ -243,7 +245,8 @@ fun TabContentChat(
                 }
 
                 items(
-                    items = pairedPeers,
+                    items = pairedPeers.toList(),
+                    key = { it.id }
                 ) { peer ->
                     PeerListItem(
                         title = peer.name,
@@ -271,7 +274,8 @@ fun TabContentChat(
                 }
 
                 items(
-                    items = unpairedPeers,
+                    items = unpairedPeers.toList(),
+                    key = { it.id }
                 ) { peer ->
                     PeerListItem(
                         title = peer.name,