Connect device only once

takahirom · Jan 23, 2025 · 6bb3276 · 6bb3276
1 parent c23024b
commit 6bb3276
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/arbigent-core/src/main/java/io/github/takahirom/arbigent/ArbigentAgent.kt b/arbigent-core/src/main/java/io/github/takahirom/arbigent/ArbigentAgent.kt
@@ -717,7 +717,7 @@ private fun step(
   return StepResult.Continue
 }
 
-public fun detectStuckScreen(lastScreenshot: String?, newScreenshot: File): Boolean {
+private fun detectStuckScreen(lastScreenshot: String?, newScreenshot: File): Boolean {
   if (lastScreenshot == null) {
     return false
   }

diff --git a/arbigent-ui/src/main/kotlin/io/github/takahirom/arbigent/ui/ArbigentAppStateHolder.kt b/arbigent-ui/src/main/kotlin/io/github/takahirom/arbigent/ui/ArbigentAppStateHolder.kt
@@ -300,7 +300,10 @@ class ArbigentAppStateHolder(
       if (devicesStateHolder.selectedDevice.value == null) {
         devicesStateHolder.onSelectedDeviceChanged(devicesStateHolder.devices.value.firstOrNull())
       }
-      val device = deviceFactory(devicesStateHolder.selectedDevice.value!!)
+      val selectedDevice = devicesStateHolder.selectedDevice.value!!
+      val device = deviceCache.getOrPut(selectedDevice) {
+        deviceFactory(devicesStateHolder.selectedDevice.value!!)
+      }
       deviceConnectionState.value = DeviceConnectionState.Connected(device)
     }
   }