python-duco-connectivity 0.11.0__tar.gz → 0.13.0__tar.gz

{python_duco_connectivity-0.11.0/src/python_duco_connectivity.egg-info → python_duco_connectivity-0.13.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-duco-connectivity
-Version: 0.11.0
+Version: 0.13.0
 Summary: Async HTTP client for the local Duco Connectivity API
 Author: Ronald van der Meer
 License-Expression: MIT
@@ -80,9 +80,11 @@ bypass-target helper returns a typed target model for successful parameter-
 specific reads and raises `DucoError` if the requested target field is missing
 from an otherwise valid `/config` response.
 
-Diagnostic subsystem reads now keep raw component and status strings from
-`Diag.SubSystems`, so future subsystem names or status values remain available
-to downstream consumers without parse fallbacks or product-specific filtering.
+Diagnostic subsystem reads expose known status values as normalized `DiagStatus`
+members (`ok`, `disabled`, or `error`). Each `DiagComponent` also keeps the exact
+API value in `raw_status`; an unrecognized future value produces `status=None`
+without discarding the raw value. Subsystem names remain unfiltered so future
+components are available to downstream consumers.
 
 ## Getting started
 
@@ -184,6 +186,7 @@ For local function probes without activating the environment first:
 .venv/bin/python -m duco_connectivity --host 192.168.1.10 call async_get_board_info
 .venv/bin/duco-probe --host 192.168.1.10 call async_get_board_info
 .venv/bin/duco-probe --host 192.168.1.10 call async_get_ventilation_temperature_info
+.venv/bin/duco-probe --host 192.168.1.10 call async_get_bypass_supply_temperature_targets
 .venv/bin/duco-probe --host 192.168.1.10 call async_get_bypass_supply_temperature_target --kwargs '{"zone_id": 1}'
 ```
 

{python_duco_connectivity-0.11.0 → python_duco_connectivity-0.13.0}/README.md

@@ -48,9 +48,11 @@ bypass-target helper returns a typed target model for successful parameter-
 specific reads and raises `DucoError` if the requested target field is missing
 from an otherwise valid `/config` response.
 
-Diagnostic subsystem reads now keep raw component and status strings from
-`Diag.SubSystems`, so future subsystem names or status values remain available
-to downstream consumers without parse fallbacks or product-specific filtering.
+Diagnostic subsystem reads expose known status values as normalized `DiagStatus`
+members (`ok`, `disabled`, or `error`). Each `DiagComponent` also keeps the exact
+API value in `raw_status`; an unrecognized future value produces `status=None`
+without discarding the raw value. Subsystem names remain unfiltered so future
+components are available to downstream consumers.
 
 ## Getting started
 
@@ -152,6 +154,7 @@ For local function probes without activating the environment first:
 .venv/bin/python -m duco_connectivity --host 192.168.1.10 call async_get_board_info
 .venv/bin/duco-probe --host 192.168.1.10 call async_get_board_info
 .venv/bin/duco-probe --host 192.168.1.10 call async_get_ventilation_temperature_info
+.venv/bin/duco-probe --host 192.168.1.10 call async_get_bypass_supply_temperature_targets
 .venv/bin/duco-probe --host 192.168.1.10 call async_get_bypass_supply_temperature_target --kwargs '{"zone_id": 1}'
 ```
 

{python_duco_connectivity-0.11.0 → python_duco_connectivity-0.13.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "python-duco-connectivity"
-version = "0.11.0"
+version = "0.13.0"
 description = "Async HTTP client for the local Duco Connectivity API"
 readme = "README.md"
 license = "MIT"

{python_duco_connectivity-0.11.0 → python_duco_connectivity-0.13.0}/src/duco_connectivity/__init__.py

@@ -54,6 +54,7 @@ from .models import (
     DeviceGroupConfigSubmoduleSelector,
     DiagComponent,
     DiagInfo,
+    DiagStatus,
     DucoSerialNumber,
     DucoVersion,
     HostName,
@@ -167,6 +168,7 @@ __all__ = [
     "DeviceGroupConfigSubmoduleSelector",
     "DiagComponent",
     "DiagInfo",
+    "DiagStatus",
     "DucoClient",
     "DucoConnectionError",
     "DucoError",

{python_duco_connectivity-0.11.0 → python_duco_connectivity-0.13.0}/src/duco_connectivity/client.py

@@ -58,6 +58,7 @@ from .models import (
     DeviceGroupConfigSubmoduleSelector,
     DiagComponent,
     DiagInfo,
+    DiagStatus,
     InfoGeneralSubmoduleSelector,
     InfoGroup,
     InfoGroupStruct,
@@ -400,6 +401,19 @@ class DucoClient:
 
         return cls._build_bypass_supply_temperature_target(zone_id, raw_value)
 
+    @classmethod
+    def _extract_bypass_supply_temperature_targets(
+        cls,
+        config: Config,
+    ) -> dict[int, BypassSupplyTemperatureTarget]:
+        """Read all bypass supply targets from a typed config response."""
+        return {
+            zone_id: target
+            for zone_id in range(1, 9)
+            if (target := cls._extract_bypass_supply_temperature_target(config, zone_id))
+            is not None
+        }
+
     @staticmethod
     def _read_scalar_value(payload: dict[str, Any], key: str) -> Any:
         raw_value = payload[key]
@@ -2054,7 +2068,8 @@ class DucoClient:
 
         return DiagComponent(
             component=component,
-            status=status,
+            status=DiagStatus.from_api_value(status),
+            raw_status=status,
             raw_payload=cls._preserve_raw_payload(payload),
         )
 
@@ -2318,6 +2333,25 @@ class DucoClient:
             raise DucoError(msg)
         return target
 
+    async def async_get_bypass_supply_temperature_targets(
+        self,
+    ) -> dict[int, BypassSupplyTemperatureTarget]:
+        """Return all bypass supply targets exposed through `/config` in Celsius."""
+        try:
+            config = await self.async_get_config(
+                module=ConfigModuleSelector.HEAT_RECOVERY,
+                submodule=ConfigHeatRecoverySubmoduleSelector.BYPASS,
+            )
+        except DucoResponseError as err:
+            if _is_unsupported_optional_endpoint_error(err, "/config"):
+                raise DucoUnsupportedCapabilityError(
+                    err.status,
+                    err.path,
+                    err.body,
+                ) from err
+            raise
+        return self._extract_bypass_supply_temperature_targets(config)
+
     async def async_set_bypass_supply_temperature_target(
         self,
         zone_id: int,

{python_duco_connectivity-0.11.0 → python_duco_connectivity-0.13.0}/src/duco_connectivity/models.py

@@ -587,12 +587,20 @@ def _coerce_node_motor_position(value: NodeMotorPosition | int) -> NodeMotorPosi
 
 
 class DiagStatus(StrEnum):
-    """Health states returned by the diagnostics API, plus a client-side fallback."""
+    """Known normalized health states returned by the diagnostics API."""
 
-    OK = "Ok"
-    DISABLE = "Disable"
-    ERROR = "Error"
-    UNKNOWN = "UNKNOWN"
+    OK = "ok"
+    DISABLED = "disabled"
+    ERROR = "error"
+
+    @classmethod
+    def from_api_value(cls, value: str) -> Self | None:
+        """Return the known normalized status for a raw API value."""
+        return {
+            "Ok": cls.OK,
+            "Disable": cls.DISABLED,
+            "Error": cls.ERROR,
+        }.get(value)
 
 
 class ActionResultStatus(StrEnum):
@@ -1572,7 +1580,8 @@ class DiagComponent:
     """Health state for a diagnostic subsystem."""
 
     component: str
-    status: str
+    status: DiagStatus | None
+    raw_status: str
     raw_payload: dict[str, Any] = field(default_factory=dict, repr=False, compare=False)
 
 

{python_duco_connectivity-0.11.0 → python_duco_connectivity-0.13.0/src/python_duco_connectivity.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-duco-connectivity
-Version: 0.11.0
+Version: 0.13.0
 Summary: Async HTTP client for the local Duco Connectivity API
 Author: Ronald van der Meer
 License-Expression: MIT
@@ -80,9 +80,11 @@ bypass-target helper returns a typed target model for successful parameter-
 specific reads and raises `DucoError` if the requested target field is missing
 from an otherwise valid `/config` response.
 
-Diagnostic subsystem reads now keep raw component and status strings from
-`Diag.SubSystems`, so future subsystem names or status values remain available
-to downstream consumers without parse fallbacks or product-specific filtering.
+Diagnostic subsystem reads expose known status values as normalized `DiagStatus`
+members (`ok`, `disabled`, or `error`). Each `DiagComponent` also keeps the exact
+API value in `raw_status`; an unrecognized future value produces `status=None`
+without discarding the raw value. Subsystem names remain unfiltered so future
+components are available to downstream consumers.
 
 ## Getting started
 
@@ -184,6 +186,7 @@ For local function probes without activating the environment first:
 .venv/bin/python -m duco_connectivity --host 192.168.1.10 call async_get_board_info
 .venv/bin/duco-probe --host 192.168.1.10 call async_get_board_info
 .venv/bin/duco-probe --host 192.168.1.10 call async_get_ventilation_temperature_info
+.venv/bin/duco-probe --host 192.168.1.10 call async_get_bypass_supply_temperature_targets
 .venv/bin/duco-probe --host 192.168.1.10 call async_get_bypass_supply_temperature_target --kwargs '{"zone_id": 1}'
 ```
 

{python_duco_connectivity-0.11.0 → python_duco_connectivity-0.13.0}/tests/test_client.py

@@ -42,6 +42,7 @@ from duco_connectivity import (
     DeviceGroupConfigSubmoduleSelector,
     DiagComponent,
     DiagInfo,
+    DiagStatus,
     DucoClient,
     DucoConnectionError,
     DucoError,
@@ -2066,7 +2067,8 @@ async def test_get_diagnostics(diag_data: dict[str, object]) -> None:
 
     assert len(diags) == 3
     assert diags[0].component == "Ventilation"
-    assert diags[0].status == "Ok"
+    assert diags[0].status is DiagStatus.OK
+    assert diags[0].raw_status == "Ok"
     assert diags[0].raw_payload is diag_data["Diag"]["SubSystems"][0]
 
 
@@ -2085,13 +2087,19 @@ async def test_get_diagnostics_info_exposes_typed_subsystems(
     assert_type(diag_info, DiagInfo)
     assert_type(diag_info.diagnostic_subsystems, tuple[DiagComponent, ...])
     assert_type(diag_info.diagnostic_subsystems[0].component, str)
-    assert_type(diag_info.diagnostic_subsystems[0].status, str)
+    assert_type(diag_info.diagnostic_subsystems[0].status, DiagStatus | None)
+    assert_type(diag_info.diagnostic_subsystems[0].raw_status, str)
     assert tuple(item.component for item in diag_info.diagnostic_subsystems) == (
         "Ventilation",
         "VentCool",
         "SunCtrl",
     )
     assert tuple(item.status for item in diag_info.diagnostic_subsystems) == (
+        DiagStatus.OK,
+        DiagStatus.OK,
+        DiagStatus.OK,
+    )
+    assert tuple(item.raw_status for item in diag_info.diagnostic_subsystems) == (
         "Ok",
         "Ok",
         "Ok",
@@ -2115,7 +2123,8 @@ async def test_get_diagnostics_preserves_unknown_status() -> None:
             diags = await client.async_get_diagnostics()
 
     assert len(diags) == 1
-    assert diags[0].status == "FutureState"
+    assert diags[0].status is None
+    assert diags[0].raw_status == "FutureState"
 
 
 async def test_get_diagnostics_preserves_unknown_component_name() -> None:
@@ -2182,9 +2191,11 @@ async def test_get_diagnostics_info_skips_partial_entries() -> None:
         with patch.object(session, "request", _request(mock_response)):
             diag_info = await client.async_get_diagnostics_info()
 
-    assert tuple((item.component, item.status) for item in diag_info.diagnostic_subsystems) == (
-        ("Ventilation", "Ok"),
-        ("SunCtrl", "FutureState"),
+    assert tuple(
+        (item.component, item.status, item.raw_status) for item in diag_info.diagnostic_subsystems
+    ) == (
+        ("Ventilation", DiagStatus.OK, "Ok"),
+        ("SunCtrl", None, "FutureState"),
     )
 
 
@@ -2213,6 +2224,12 @@ async def test_get_diagnostics_info_does_not_filter_product_specific_subsystems(
         "SunCtrl",
         "FutureSubsystem",
     )
+    assert tuple((item.status, item.raw_status) for item in diag_info.diagnostic_subsystems) == (
+        (DiagStatus.ERROR, "Error"),
+        (DiagStatus.OK, "Ok"),
+        (DiagStatus.OK, "Ok"),
+        (DiagStatus.DISABLED, "Disable"),
+    )
 
 
 async def test_get_nodes_parses_full_payload(nodes_data: dict[str, object]) -> None:
@@ -3794,6 +3811,81 @@ async def test_get_bypass_supply_temperature_target_returns_converted_values(
     }
 
 
+async def test_get_bypass_supply_temperature_targets_returns_available_targets(
+    config_data: dict[str, object],
+) -> None:
+    """Bulk bypass target reads should return all available targets in Celsius."""
+    mock_response = _response(json_payload=config_data)
+
+    async with aiohttp.ClientSession() as session:
+        client = DucoClient(session=session, host="192.0.2.94")
+        request_mock = _request(mock_response)
+        with patch.object(session, "request", request_mock):
+            targets = await client.async_get_bypass_supply_temperature_targets()
+
+    assert targets == {
+        1: BypassSupplyTemperatureTarget(
+            zone_id=1,
+            value=18.0,
+            minimum=12.0,
+            increment=0.5,
+            maximum=22.0,
+        ),
+        2: BypassSupplyTemperatureTarget(
+            zone_id=2,
+            value=18.5,
+            minimum=12.0,
+            increment=0.5,
+            maximum=22.0,
+        ),
+    }
+    assert request_mock.call_args.kwargs["params"] == {
+        "module": "HeatRecovery",
+        "submodule": "Bypass",
+    }
+
+
+async def test_get_bypass_supply_temperature_targets_returns_empty_when_absent() -> None:
+    """Bulk bypass target reads should omit targets absent from the response."""
+    mock_response = _response(json_payload={"HeatRecovery": {"Bypass": {}}})
+
+    async with aiohttp.ClientSession() as session:
+        client = DucoClient(session=session, host="192.0.2.94")
+        with patch.object(session, "request", _request(mock_response)):
+            targets = await client.async_get_bypass_supply_temperature_targets()
+
+    assert targets == {}
+
+
+async def test_get_bypass_supply_temperature_targets_raises_when_unsupported() -> None:
+    """Unsupported bulk bypass target reads should raise a typed capability error."""
+    mock_response = _response(
+        status=400,
+        text_payload='{"Code":3,"Result":"FAILED"}',
+    )
+
+    async with aiohttp.ClientSession() as session:
+        client = DucoClient(session=session, host="192.0.2.94")
+        with (
+            patch.object(session, "request", _request(mock_response)),
+            pytest.raises(DucoUnsupportedCapabilityError),
+        ):
+            await client.async_get_bypass_supply_temperature_targets()
+
+
+async def test_get_bypass_supply_temperature_targets_reraises_other_client_errors() -> None:
+    """Unexpected bulk bypass target failures should remain visible to callers."""
+    mock_response = _response(status=400, text_payload='{"Code":4,"Result":"FAILED"}')
+
+    async with aiohttp.ClientSession() as session:
+        client = DucoClient(session=session, host="192.0.2.94")
+        with (
+            patch.object(session, "request", _request(mock_response)),
+            pytest.raises(DucoResponseError),
+        ):
+            await client.async_get_bypass_supply_temperature_targets()
+
+
 async def test_get_bypass_supply_temperature_target_raises_when_not_reported() -> None:
     """Missing bypass targets should fail for parameter-specific helper reads."""
     mock_response = _response(json_payload={"HeatRecovery": {"Bypass": {}}})