fixes to make Bleak (BLE) work better

Author: geeksville

meshtastic/__main__.py

@@ -1122,11 +1122,11 @@ def addConnectionArgs(parser: argparse.ArgumentParser) -> argparse.ArgumentParse
         action="store_true",
     )
 
-    group.add_argument(
+    outer.add_argument(
         "--ble-dest",
         help="The BLE device address or name to connect to",
         default=None,
-    )
+    )    
 
     return parser
 

meshtastic/ble_interface.py

@@ -51,9 +51,9 @@ def __init__(self, address: Optional[str], noProto: bool = False, debugOut = Non
             self.client = self.connect(address)
             self.state.BLE = True
             logging.debug("BLE connected")
-        # except BLEInterface.BLEError as e:
-        finally:
+        except BLEInterface.BLEError as e:
             self.close()
+            raise e
 
         logging.debug("Mesh init starting")
         MeshInterface.__init__(self, debugOut = debugOut, noProto = noProto, noNodes = noNodes)
@@ -112,17 +112,13 @@ def _sanitize_address(address): # pylint: disable=E0213
             .replace(":", "") \
             .lower()
 
-    def connect(self, address):
+    def connect(self, address: Optional[str] = None):
         "Connect to a device by address"
+
+        # Bleak docs recommend always doing a scan before connecting (even if we know addr)
         device = self.find_device(address)
         client = BLEClient(device.address)
         client.connect()
-        try:
-            client.pair()
-        except NotImplementedError:
-            # Some bluetooth backends do not require explicit pairing.
-            # See Bleak docs for details on this.
-            pass
         return client
 
 
@@ -212,7 +208,7 @@ def close(self): # pylint: disable=C0116
 
     def __enter__(self):
         return self
-
+    
     def __exit__(self, _type, _value, _traceback):
         self.close()