fix: Handling retired or changed scanners (#360)

- Refactor _refresh_scanners to be more efficient, and to handle the case where a scanner still exists but is no longer a scanner.
- Refactor coordinator.init() to move callbacks out into class methods.
- Tamed down lots of debug logging
- Altered self.last_config_entry_update to use MONOTONIC_TIME(), and changed cooldown to a const of 10 seconds, and included check for parallel save tasks.
- Removed assert that excluded scanners from area-by-mindist

custom_components/bermuda/bermuda_device_scanner.py

@@ -191,11 +191,15 @@ def update_advertisement(self, scandata: BluetoothScannerDevice):
             # I want to know if it does.
             # AJG 2024-01-11: This does happen. Looks like maybe apple devices?
             # Changing from warning to debug to quiet users' logs.
-            _LOGGER.debug(
-                "Device changed TX-POWER! That was unexpected: %s %sdB",
-                self.parent_device_address,
-                scandata.advertisement.tx_power,
-            )
+            # Also happens with esphome set with long beacon interval tx, as it alternates
+            # between sending some generic advert and the iBeacon advert. ie, it's bogus for that
+            # case.
+            # _LOGGER.debug(
+            #     "Device changed TX-POWER! That was unexpected: %s %sdB",
+            #     self.parent_device_address,
+            #     scandata.advertisement.tx_power,
+            # )
+            pass
         self.tx_power = scandata.advertisement.tx_power
 
         # Track each advertisement element as or if they change.

custom_components/bermuda/const.py

@@ -92,6 +92,8 @@
 PRUNE_TIME_DEFAULT = 259200  # Max age of regular device entries (3days)
 PRUNE_TIME_IRK = 3600  # Resolvable Private addresses change often, prune regularly (1h)
 
+SAVEOUT_COOLDOWN = 10  # seconds to delay before re-trying config entry save.
+
 DOCS = {}