bugfix/FAT-588 Improve LLM background device communication

.changeset/cyan-feet-worry.md

@@ -0,0 +1,5 @@
+---
+"@ledgerhq/live-common": patch
+---
+
+Allow runner errors to bubble up and not be silenced

.changeset/giant-walls-march.md

@@ -0,0 +1,5 @@
+---
+"@ledgerhq/react-native-hw-transport-ble": patch
+---
+
+Provide a forceful disconnection mechanism for close request on BleTransport

.changeset/sour-countries-learn.md

@@ -0,0 +1,5 @@
+---
+"live-mobile": patch
+---
+
+Enabled BLE background capabilities again

apps/ledger-live-mobile/ios/ledgerlivemobile/Info.plist

@@ -2,6 +2,11 @@
 <!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
 <plist version="1.0">
 <dict>
+	<key>Braze</key>
+	<dict>
+		<key>Endpoint</key>
+		<string>sdk.fra-02.braze.eu</string>
+	</dict>
 	<key>CFBundleDevelopmentRegion</key>
 	<string>en</string>
 	<key>CFBundleDisplayName</key>
@@ -103,6 +108,7 @@
 	</array>
 	<key>UIBackgroundModes</key>
 	<array>
+		<string>bluetooth-central</string>
 		<string>fetch</string>
 		<string>remote-notification</string>
 	</array>
@@ -118,10 +124,5 @@
 	</array>
 	<key>UIViewControllerBasedStatusBarAppearance</key>
 	<false/>
-	<key>Braze</key>
-	<dict>
-		<key>Endpoint</key>
-		<string>sdk.fra-02.braze.eu</string>
-	</dict>
 </dict>
 </plist>

libs/ledger-live-common/src/apps/runner.ts

@@ -100,13 +100,15 @@ export const runAllWithProgress = (
   return concat(
     ...getActionPlan(state).map((appOp) => runAppOp(state, appOp, exec))
   ).pipe(
-    map(
-      (event) =>
-        <Action>{
-          type: "onRunnerEvent",
-          event,
-        }
-    ),
+    map((event) => {
+      if (event.type === "runError") {
+        throw event.error;
+      }
+      return <Action>{
+        type: "onRunnerEvent",
+        event,
+      };
+    }),
     scan(reducer, state),
     mergeMap((s) => {
       // Nb if you also want to expose the uninstall queue, feel free.

libs/ledgerjs/packages/react-native-hw-transport-ble/src/BleTransport.ts

@@ -40,6 +40,7 @@ import { decoratePromiseErrors, remapError } from "./remapErrors";
 let connectOptions: Record<string, unknown> = {
   requestMTU: 156,
   connectionPriority: 1,
+  forceDisconnectTimeout: 4000,
 };
 const transportsCache = {};
 const bleManager = new BleManager();
@@ -537,9 +538,35 @@ export default class BluetoothTransport extends Transport {
     }
   };
 
-  async close() {
+  async close(): Promise<void> {
+    // Prevent double runs
+    let alreadyDisconnected = false;
+
+    // Actually disconnect from the device onClose
+    const triggerDisconnect = () => {
+      if (!alreadyDisconnected) {
+        BluetoothTransport.disconnect(this.id); // This triggers cache cleanup above.
+        alreadyDisconnected = true;
+      }
+    };
+
+    // For cases where an exchange doesn't resolve and we triggered a `close` we
+    // introduce a timeout to forcefully disconnect in order to unblock subsequent
+    // usages of the `withDevice` logic. It's also a good practice to disconnect
+    // from the device when not actively interacting with it.
     if (this.exchangeBusyPromise) {
-      await this.exchangeBusyPromise;
+      const forceDisconnect = new Promise<void>((resolve) =>
+        setTimeout(() => {
+          triggerDisconnect();
+          resolve();
+        }, connectOptions.forceDisconnectTimeout as number)
+      );
+
+      await Promise.race([this.exchangeBusyPromise, forceDisconnect]);
+      // At this point we've either disconnected forcefully, or completed the exchange
     }
+
+    triggerDisconnect();
+    return;
   }
 }