desktop: prevent deadlock (#2785)

* desktop: prevent deadlock

* debug info

apps/multiplatform/common/src/commonMain/kotlin/chat/simplex/common/model/ChatModel.kt

@@ -56,7 +56,7 @@ object ChatModel {
   val chatItems = mutableStateListOf<ChatItem>()
   val groupMembers = mutableStateListOf<GroupMember>()
 
-  val terminalItems = mutableStateListOf<TerminalItem>()
+  val terminalItems = mutableStateOf(emptyList<TerminalItem>())
   val userAddress = mutableStateOf<UserContactLinkRec?>(null)
   // Allows to temporary save servers that are being edited on multiple screens
   val userSMPServersUnsaved = mutableStateOf<(List<ServerCfg>)?>(null)
@@ -484,10 +484,11 @@ object ChatModel {
     networkStatuses[contact.activeConn.agentConnId] ?: NetworkStatus.Unknown()
 
   fun addTerminalItem(item: TerminalItem) {
-    if (terminalItems.size >= 500) {
+    if (terminalItems.value.size >= 500) {
-      terminalItems.removeAt(0)
+      terminalItems.value = terminalItems.value.takeLast(499) + item
+    } else {
+      terminalItems.value += item
     }
-    terminalItems.add(item)
   }
 }
 

apps/multiplatform/common/src/commonMain/kotlin/chat/simplex/common/views/TerminalView.kt

@@ -34,7 +34,7 @@ fun TerminalView(chatModel: ChatModel, close: () -> Unit) {
     close()
   })
   TerminalLayout(
-    remember { chatModel.terminalItems },
+    chatModel.terminalItems,
     composeState,
     sendCommand = { sendCommand(chatModel, composeState) },
     close
@@ -62,7 +62,7 @@ private fun sendCommand(chatModel: ChatModel, composeState: MutableState<Compose
 
 @Composable
 fun TerminalLayout(
-  terminalItems: List<TerminalItem>,
+  terminalItems: MutableState<List<TerminalItem>>,
   composeState: MutableState<ComposeState>,
   sendCommand: () -> Unit,
   close: () -> Unit
@@ -115,12 +115,12 @@ fun TerminalLayout(
 private var lazyListState = 0 to 0
 
 @Composable
-fun TerminalLog(terminalItems: List<TerminalItem>) {
+fun TerminalLog(terminalItems: MutableState<List<TerminalItem>>) {
   val listState = rememberLazyListState(lazyListState.first, lazyListState.second)
   DisposableEffect(Unit) {
     onDispose { lazyListState = listState.firstVisibleItemIndex to listState.firstVisibleItemScrollOffset }
   }
-  val reversedTerminalItems by remember { derivedStateOf { terminalItems.reversed().toList() } }
+  val reversedTerminalItems by remember { derivedStateOf { terminalItems.value.reversed().toList() } }
   val clipboard = LocalClipboardManager.current
   LazyColumn(state = listState, reverseLayout = true) {
     items(reversedTerminalItems) { item ->
@@ -152,7 +152,7 @@ fun TerminalLog(terminalItems: List<TerminalItem>) {
 fun PreviewTerminalLayout() {
   SimpleXTheme {
     TerminalLayout(
-      terminalItems = TerminalItem.sampleData,
+      terminalItems = remember { mutableStateOf(TerminalItem.sampleData) },
       composeState = remember { mutableStateOf(ComposeState(useLinkPreviews = false)) },
       sendCommand = {},
       close = {}

apps/multiplatform/desktop/build.gradle.kts

@@ -30,8 +30,16 @@ kotlin {
 compose {
   desktop {
     application {
+      // For debugging via VisualVM
+      /*jvmArgs += listOf(
+        "-Dcom.sun.management.jmxremote.port=8080",
+        "-Dcom.sun.management.jmxremote.ssl=false",
+        "-Dcom.sun.management.jmxremote.authenticate=false"
+      )*/
       mainClass = "chat.simplex.desktop.MainKt"
       nativeDistributions {
+        // For debugging via VisualVM
+        //modules("jdk.zipfs", "jdk.management.agent")
         modules("jdk.zipfs")
         //includeAllModules = true
         outputBaseDir.set(project.file("../release"))