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

{python_duco_connectivity-0.12.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.12.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
 

{python_duco_connectivity-0.12.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
 

{python_duco_connectivity-0.12.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.12.0"
+version = "0.13.0"
 description = "Async HTTP client for the local Duco Connectivity API"
 readme = "README.md"
 license = "MIT"

{python_duco_connectivity-0.12.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.12.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,
@@ -2067,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),
         )
 

{python_duco_connectivity-0.12.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.12.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.12.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
 

{python_duco_connectivity-0.12.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: