python-mobius 0.1.3__tar.gz → 0.1.4__tar.gz

{python_mobius-0.1.3 → python_mobius-0.1.4}/CHANGELOG.md

@@ -1,5 +1,18 @@
 # Changelog
 
+## 0.1.4
+
+- Added `MobiusDevice.is_connected` property -- lets callers check
+  connection state before deciding whether to read directly or reconnect
+  first, enabling a persistent-connection usage pattern (connect() once,
+  read many times, only reconnect on an actual failure) instead of
+  connecting/disconnecting on every read. `connect()`/`disconnect()` were
+  already separate public methods (not just wrapped by `__aenter__`/
+  `__aexit__`), so this required no other changes -- just a convenience
+  property exposing what was previously only checked internally via
+  `self._client.is_connected`. Added for `ha-mobius`'s move to persistent
+  connections. 1 new test.
+
 ## 0.1.3
 
 - **Added firmware/hardware version reading.** `get_firmware_versions()`

{python_mobius-0.1.3 → python_mobius-0.1.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-mobius
-Version: 0.1.3
+Version: 0.1.4
 Summary: Reverse-engineered Python client for the Mobius BLE protocol (EcoTech Marine VorTech/Radion, AquaIllumination, Neptune Systems, NYOS)
 Project-URL: Homepage, https://code.r3pek.org/r3pek/python-mobius
 Project-URL: Documentation, https://code.r3pek.org/r3pek/python-mobius/src/branch/main/documentation

{python_mobius-0.1.3 → python_mobius-0.1.4}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "python-mobius"
-version = "0.1.3"
+version = "0.1.4"
 description = "Reverse-engineered Python client for the Mobius BLE protocol (EcoTech Marine VorTech/Radion, AquaIllumination, Neptune Systems, NYOS)"
 readme = "README.md"
 license = { text = "GPL-2.0-only" }

{python_mobius-0.1.3 → python_mobius-0.1.4}/src/mobius/__init__.py

@@ -51,7 +51,7 @@ from .discovery import (
     dedupe_by_serial, find_device_by_serial,
 )
 
-__version__ = "0.1.3"
+__version__ = "0.1.4"
 
 __all__ = [
     "__version__",

{python_mobius-0.1.3 → python_mobius-0.1.4}/src/mobius/device.py

@@ -237,6 +237,21 @@ class MobiusDevice:
         if self._client and self._client.is_connected:
             await self._client.disconnect()
 
+    @property
+    def is_connected(self) -> bool:
+        """
+        True if there's currently a live BLE connection. connect()/
+        disconnect() are already separate public methods (not just
+        wrapped by __aenter__/__aexit__), so this device can be used as a
+        persistent, long-lived connection -- connect() once, read many
+        times, disconnect() only when actually done -- rather than
+        reconnecting on every read. Callers doing this should check this
+        property before each read and reconnect (ideally via `serial=`,
+        not a cached address -- see documentation/
+        12-device-identity-and-address-stability.md) if it's False.
+        """
+        return self._client is not None and self._client.is_connected
+
     def _on_rx_data(self, _handle, data: bytearray):
         self._rx_buffer.extend(data)
 

{python_mobius-0.1.3 → python_mobius-0.1.4}/tests/test_connection_cleanup.py

@@ -105,3 +105,17 @@ async def test_pump_telemetry_speed_percent_is_always_nonnegative():
 
     assert telemetry["speed"] == -447
     assert telemetry["speed_percent"] == 44.7
+
+
+@pytest.mark.asyncio
+async def test_is_connected_reflects_client_state():
+    device = MobiusDevice(_fake_ble_device())
+    assert device.is_connected is False  # never connected
+
+    fake_client = MagicMock()
+    fake_client.is_connected = True
+    device._client = fake_client
+    assert device.is_connected is True
+
+    fake_client.is_connected = False
+    assert device.is_connected is False