python-izone 1.3.6__tar.gz → 1.3.7__tar.gz

{python_izone-1.3.6 → python_izone-1.3.7}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-izone
-Version: 1.3.6
+Version: 1.3.7
 Summary: A python interface to the iZone airconditioner controller
 Project-URL: Homepage, https://github.com/Swamp-Ig/pizone
 Author-email: Penny Wood <pypl@ninjateaparty.com>

{python_izone-1.3.6 → python_izone-1.3.7}/docs/release-notes/v1.3.6.md

@@ -4,86 +4,60 @@ Released 2026-07-19.
 
 **Compare range:** https://github.com/Swamp-Ig/pizone/compare/v1.3.5...v1.3.6
 
-This is a **pre-release** for integrators (especially Home Assistant Phase 2
-work). It is not aimed at end users as a standalone upgrade. Prefer waiting for
-a future stable line unless you are developing against the new discovery API.
+This is a **pre-release** for integrators adopting the newer discovery API. It is not aimed at end users as a standalone upgrade. Prefer waiting for a future stable line unless you are developing against that API.
 
 ## Since v1.3.5
 
 - Discovery scan wait (`SCAN_TIMEOUT`) reduced from 20s to 5s
-- `close()` waits until the UDP transport has finished tearing down before
-  returning, so a following `create_discovery()` can rebind `:7005` immediately
-  (fixes delete → HomeKit rediscovery `EADDRINUSE` races)
-- Failed `create_discovery` / `start_discovery` no longer leaves a sticky
-  process singleton that blocked retries until process restart
-- `PLACEHOLDER_DEVICE_UID` is no longer exported from the public package API
-  (`UnpairedBridgeError` remains the supported signal)
+- `close()` waits until the UDP transport has finished tearing down before returning, so a following `create_discovery()` can rebind `:7005` immediately (fixes delete → HomeKit rediscovery `EADDRINUSE` races)
+- Failed `create_discovery` / `start_discovery` no longer leaves a sticky process singleton that blocked retries until process restart
+- `PLACEHOLDER_DEVICE_UID` is no longer exported from the public package API (`UnpairedBridgeError` remains the supported signal)
 
 Everything from v1.3.5 below still applies.
 
 ## Summary (since v1.3.4)
 
-- HTTP connections are closed after each GET, which greatly reduces stalls on
-  some iZone bridges ([home-assistant/core#176536](https://github.com/home-assistant/core/issues/176536))
-- A new discovery and refresh API is available for integrators (the older API
-  still works for now)
+- HTTP connections are closed after each GET, which greatly reduces stalls on some iZone bridges ([home-assistant/core#176536](https://github.com/home-assistant/core/issues/176536))
+- A new discovery and refresh API is available for integrators (the older API still works for now)
 - iPower monitoring is off unless you opt in with `ENABLE_POWER = True`
-- Bridges that are not paired to an air conditioner (UID `000000000`) are
-  ignored or rejected instead of showing up as broken controllers
+- Bridges that are not paired to an air conditioner (UID `000000000`) are ignored or rejected instead of showing up as broken controllers
 - Faster discovery scans and reliable UDP close → recreate (this release)
 
 ## Bridge connection reliability
 
-Some ASH bridges misbehave when HTTP connections are reused. This release
-line:
+Some ASH bridges misbehave when HTTP connections are reused. This release line:
 
 - Sends `Connection: close` on every HTTP GET (polling and discovery probes)
-- Closes TCP connections after each request when the library creates its own
-  aiohttp session
+- Closes TCP connections after each request when the library creates its own aiohttp session
 
-This reduces how often the bridge stops responding. It does not fix every
-hub-side stall; replacing faulty hardware may still be needed.
+This reduces how often the bridge stops responding. It does not fix every hub-side stall; replacing faulty hardware may still be needed.
 
 ## New discovery and refresh API (optional for now)
 
-Integrators can adopt a clearer model without background poll loops inside the
-library:
+Integrators can adopt a clearer model without background poll loops inside the library:
 
-- `create_discovery()` with callbacks when a controller is found or its IP
-  changes
-- `discover_by_host`, `discover_by_uid`, `discover_all` to find bridges on the
-  network
+- `create_discovery()` with callbacks when a controller is found or its IP changes
+- `discover_by_host`, `discover_by_uid`, `discover_all` to find bridges on the network
 - `create_controller(uid, host)` to connect to a known bridge
-- `refresh_all`, `refresh_system`, `refresh_zones`, and related methods so the
-  caller controls when to poll
+- `refresh_all`, `refresh_system`, `refresh_zones`, and related methods so the caller controls when to poll
 
-`close()` on the discovery service now waits for UDP teardown so the next
-`create_discovery()` can bind the discovery port again without racing the OS
-socket release.
+`close()` on the discovery service now waits for UDP teardown so the next `create_discovery()` can bind the discovery port again without racing the OS socket release.
 
-The previous API (`discovery()`, `fetch_controller`, `Listener`, and so on)
-still works.
+The previous API (`discovery()`, `fetch_controller`, `Listener`, and so on) still works.
 
 ## iPower opt-in
 
-`ENABLE_POWER` now defaults to `False`. Power monitor HTTP traffic only runs
-after you set `pizone.ENABLE_POWER = True` before creating or refreshing a
-controller. This avoids extra requests on bridges that advertise iPower but are
-not set up for it. The default may change if more field data shows power
-monitoring is reliable enough to enable by default.
+`ENABLE_POWER` now defaults to `False`. Power monitor HTTP traffic only runs after you set `pizone.ENABLE_POWER = True` before creating or refreshing a controller. This avoids extra requests on bridges that advertise iPower but are not set up for it. The default may change if more field data shows power monitoring is reliable enough to enable by default.
 
 ## Unpaired bridges
 
-Wireless ASH bridges with no AC paired report device UID `000000000`. The
-library now:
+Wireless ASH bridges with no AC paired report device UID `000000000`. The library now:
 
 - Ignores them during passive UDP discovery
 - Raises `UnpairedBridgeError` if you try to connect to that UID on purpose
 - Refuses to cache them after an IP lookup
 
-Paired bridges that temporarily report fault-shaped settings (AC off, reboot,
-and so on) are unchanged: they keep their real UID and show as disconnected
-until the AC data recovers.
+Paired bridges that temporarily report fault-shaped settings (AC off, reboot, and so on) are unchanged: they keep their real UID and show as disconnected until the AC data recovers.
 
 ## Other changes
 
@@ -93,6 +67,4 @@ until the AC data recovers.
 
 ## Home Assistant
 
-Pin `python-izone==1.3.6` when adopting the Phase 2 discovery / coordinator
-runtime. This pre-release is intended for that integration work, not as a
-general end-user library upgrade on the legacy path alone.
+Pin `python-izone==1.3.6` when moving the iZone integration onto the newer discovery and coordinator path. This pre-release is intended for that integration work, not as a general end-user library upgrade on the legacy path alone.

python_izone-1.3.7/docs/release-notes/v1.3.7.md

@@ -0,0 +1,7 @@
+# Release notes — v1.3.7
+
+**Compare range:** https://github.com/Swamp-Ig/pizone/compare/v1.3.6...v1.3.7
+
+## Since v1.3.6
+
+- Add `ControllerAlreadyClaimedError` for duplicate controller claim/discovery attempts (replaces bare `RuntimeError` with message `already created` on controller paths; discovery-service singleton errors unchanged)

{python_izone-1.3.6 → python_izone-1.3.7}/pizone/__init__.py

@@ -31,7 +31,12 @@ from typing import Any
 from . import power as power_mod
 from .controller import Controller
 from .discovery import DiscoveryService, Listener, create_discovery, discovery
-from .exceptions import ControllerCommandError, ResponseDecodeError, UnpairedBridgeError
+from .exceptions import (
+    ControllerAlreadyClaimedError,
+    ControllerCommandError,
+    ResponseDecodeError,
+    UnpairedBridgeError,
+)
 from .power import BatteryLevel, Power, PowerChannel, PowerDevice, PowerGroup
 from .types import ControllerEndpoint
 from .zone import Zone
@@ -61,6 +66,7 @@ __all__ = [
     "ENABLE_POWER",
     "BatteryLevel",
     "Controller",
+    "ControllerAlreadyClaimedError",
     "ControllerCommandError",
     "ControllerEndpoint",
     "DiscoveryService",

{python_izone-1.3.6 → python_izone-1.3.7}/pizone/discovery.py

@@ -24,7 +24,7 @@ import ifaddr
 
 from .const import PLACEHOLDER_DEVICE_UID
 from .controller import Controller
-from .exceptions import raise_if_placeholder_uid
+from .exceptions import ControllerAlreadyClaimedError, raise_if_placeholder_uid
 from .types import ControllerEndpoint
 from .zone import Zone
 
@@ -405,7 +405,9 @@ class DiscoveryService:
     def _ensure_endpoint_available(self, endpoint: ControllerEndpoint) -> None:
         """Raise if *endpoint* is already claimed."""
         if endpoint.uid in self._claimed_endpoints:
-            raise RuntimeError(f"Controller {endpoint.uid} already created")
+            raise ControllerAlreadyClaimedError(
+                f"Controller {endpoint.uid} already created"
+            )
 
     # disposition: 1.4
     def _claim_endpoint(self, endpoint: ControllerEndpoint) -> None:
@@ -451,7 +453,9 @@ class DiscoveryService:
         """
         claimed = self._endpoint_by_host(self._claimed_endpoints, host)
         if claimed is not None:
-            raise RuntimeError(f"Controller {claimed.uid} already created")
+            raise ControllerAlreadyClaimedError(
+                f"Controller {claimed.uid} already created"
+            )
         known = self._endpoint_by_host(self._known_endpoints, host)
         if known is not None:
             return known
@@ -476,7 +480,7 @@ class DiscoveryService:
         """
         raise_if_placeholder_uid(uid)
         if uid in self._claimed_endpoints:
-            raise RuntimeError(f"Controller {uid} already created")
+            raise ControllerAlreadyClaimedError(f"Controller {uid} already created")
         endpoint = self._known_endpoints.get(uid)
         if endpoint is not None:
             return endpoint
@@ -569,7 +573,7 @@ class DiscoveryService:
 
         Raises:
             UnpairedBridgeError: If *uid* is the unpaired placeholder (no I/O).
-            RuntimeError: If a controller with *uid* already exists.
+            ControllerAlreadyClaimedError: If a controller with *uid* already exists.
             ConnectionError: If the controller cannot be reached after
                 address fallback.
             ControllerCommandError: If the device rejects the protocol request.
@@ -579,7 +583,7 @@ class DiscoveryService:
         """
         raise_if_placeholder_uid(uid)
         if uid in self._claimed_endpoints:
-            raise RuntimeError(f"Controller {uid} already created")
+            raise ControllerAlreadyClaimedError(f"Controller {uid} already created")
         pending_address_changed: ControllerEndpoint | None = None
 
         try:

{python_izone-1.3.6 → python_izone-1.3.7}/pizone/exceptions.py

@@ -28,6 +28,14 @@ class UnpairedBridgeError(ValueError):
     """
 
 
+class ControllerAlreadyClaimedError(Exception):
+    """A controller UID or host is already claimed on this discovery service.
+
+    Raised when ``discover_by_host``, ``discover_by_uid``, or ``create_controller``
+    would duplicate an active controller registration.
+    """
+
+
 def raise_if_placeholder_uid(uid: str) -> None:
     """Raise :exc:`UnpairedBridgeError` if *uid* is the unpaired placeholder."""
     if uid == PLACEHOLDER_DEVICE_UID:

{python_izone-1.3.6 → python_izone-1.3.7}/pizone/version.py

@@ -18,7 +18,7 @@ version_tuple: tuple[int | str, ...]
 commit_id: str | None
 __commit_id__: str | None
 
-__version__ = version = '1.3.6'
-__version_tuple__ = version_tuple = (1, 3, 6)
+__version__ = version = '1.3.7'
+__version_tuple__ = version_tuple = (1, 3, 7)
 
 __commit_id__ = commit_id = None

{python_izone-1.3.6 → python_izone-1.3.7}/tests/test_discovery_api.py

@@ -17,6 +17,7 @@ import pytest
 
 from pizone import (
     Controller,
+    ControllerAlreadyClaimedError,
     ControllerCommandError,
     ControllerEndpoint,
     UnpairedBridgeError,
@@ -282,7 +283,7 @@ async def test_discover_by_host_raises_if_controller_exists() -> None:
         ClientSession,
         FakeHttpSession(get_error=OSError("should not probe")),
     )
-    with pytest.raises(RuntimeError, match="already created"):
+    with pytest.raises(ControllerAlreadyClaimedError, match="already created"):
         await service.discover_by_host("10.0.0.90")
     await service.close()
 
@@ -324,7 +325,7 @@ async def test_discover_by_uid_raises_if_controller_exists() -> None:
     scan = AsyncMock()
     with (
         patch.object(service, "scan", scan),
-        pytest.raises(RuntimeError, match="already created"),
+        pytest.raises(ControllerAlreadyClaimedError, match="already created"),
     ):
         await service.discover_by_uid("000025841")
     scan.assert_not_awaited()
@@ -656,7 +657,7 @@ async def test_create_controller_raises_if_uid_exists() -> None:
         FakeHttpSession(get_response=_system_settings_response("000025841")),
     )
     await service.create_controller("000025841", "10.0.0.90")
-    with pytest.raises(RuntimeError, match="already created"):
+    with pytest.raises(ControllerAlreadyClaimedError, match="already created"):
         await service.create_controller("000025841", "10.0.0.90")
     await service.close()