quilt-hp-python 0.2.1__tar.gz → 0.2.2__tar.gz

{quilt_hp_python-0.2.1 → quilt_hp_python-0.2.2}/CHANGELOG.md

@@ -7,6 +7,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.2.2] - 2026-05-11
+
+### Fixed
+- Corrected mapping of outdoor units to indoor units in SystemSnapshot
+- Fixed TUI interaction issues with button handling and bindings
+
 ## [0.2.1] - 2026-05-10
 
 ### Fixed

{quilt_hp_python-0.2.1 → quilt_hp_python-0.2.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: quilt-hp-python
-Version: 0.2.1
+Version: 0.2.2
 Summary: Async Python client for Quilt mini-split HVAC systems
 Project-URL: Repository, https://github.com/eman/quilt-hp-python
 Project-URL: Issues, https://github.com/eman/quilt-hp-python/issues

{quilt_hp_python-0.2.1 → quilt_hp_python-0.2.2}/docs/reference/client.md

@@ -69,7 +69,7 @@ Raised when a requested resource does not exist (gRPC `NOT_FOUND`).
 ### `__version__`
 
 ```python
-__version__: str  # e.g. "0.2.1"
+__version__: str  # e.g. "0.2.2"
 ```
 
 ---

{quilt_hp_python-0.2.1 → quilt_hp_python-0.2.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "quilt-hp-python"
-version = "0.2.1"
+version = "0.2.2"
 description = "Async Python client for Quilt mini-split HVAC systems"
 readme = "README.md"
 license = { file = "LICENSE" }

{quilt_hp_python-0.2.1 → quilt_hp_python-0.2.2}/src/quilt_hp/__init__.py

@@ -9,7 +9,7 @@ from quilt_hp.exceptions import (
     QuiltNotFoundError,
 )
 
-__version__ = "0.2.1"
+__version__ = "0.2.2"
 
 __all__ = [
     "Environment",

{quilt_hp_python-0.2.1 → quilt_hp_python-0.2.2}/src/quilt_hp/cli/tui.py

@@ -606,7 +606,7 @@ class DashboardScreen(Screen):
         self.app._persist()
 
     def action_system(self) -> None:
-        self.app.push_screen(SystemScreen(self._snapshot, self._client))
+        self.app.push_screen(SystemScreen(self._snapshot, self._client, use_f=self.use_f))
 
     def action_select_room(self) -> None:
         lv = self.query_one(ListView)
@@ -633,7 +633,7 @@ class DashboardScreen(Screen):
             (c for c in self._snapshot.controllers if c.space_id == space_id),
             None,
         )
-        odu = self._snapshot.outdoor_units[0] if self._snapshot.outdoor_units else None
+        odu = self._snapshot.odu_for_idu(idu) if idu else None
         qsm = self._snapshot.qsm_for_idu(idu) if idu else None
         self.app.push_screen(
             RoomScreen(
@@ -2239,10 +2239,13 @@ class SystemScreen(Screen):
         self,
         snapshot: SystemSnapshot,
         client: QuiltClient,
+        *,
+        use_f: bool = False,
     ) -> None:
         super().__init__()
         self._snapshot = snapshot
         self._client = client
+        self.use_f = use_f
 
     def compose(self) -> ComposeResult:
         yield Header(show_clock=True)
@@ -2283,7 +2286,6 @@ class SystemScreen(Screen):
         yield Footer()
 
     def on_mount(self) -> None:
-        self.use_f = False
         self._populate()
 
     def _populate(self) -> None:
@@ -2486,6 +2488,11 @@ class SystemScreen(Screen):
         self._populate()
 
     def action_toggle_units(self) -> None:
+        self.use_f = not self.use_f
+        self._populate()
+        self.app._persist()
+
+    def action_toggle_schedule(self) -> None:
         loc = self._snapshot.primary_location
         if loc is None:
             self.notify("No location found", severity="error")
@@ -2645,7 +2652,9 @@ class QuiltApp(App[None]):
         if self._snapshot:
             odu = self._snapshot.apply_outdoor_unit(odu)
         screen = self.screen
-        if isinstance(screen, (DashboardScreen, RoomScreen, SystemScreen)):
+        if isinstance(screen, (DashboardScreen, SystemScreen)) or (
+            isinstance(screen, RoomScreen) and screen._odu and screen._odu.id == odu.id
+        ):
             screen.update_odu(odu)
 
     def _dispatch_ctrl(self, ctrl: Controller) -> None:

{quilt_hp_python-0.2.1 → quilt_hp_python-0.2.2}/src/quilt_hp/models/system.py

@@ -416,6 +416,12 @@ class SystemSnapshot:
         self.controller_remote_sensors.append(crs)
         return crs
 
+    def odu_for_idu(self, idu: IndoorUnit) -> OutdoorUnit | None:
+        """Return the OutdoorUnit connected to the given IDU, or None."""
+        if not idu.outdoor_unit_id:
+            return None
+        return next((u for u in self.outdoor_units if u.id == idu.outdoor_unit_id), None)
+
     def qsm_for_idu(self, idu: IndoorUnit) -> QuiltSmartModule | None:
         """Return the QSM embedded in the given IDU, or None."""
         if not idu.qsm_id:

{quilt_hp_python-0.2.1 → quilt_hp_python-0.2.2}/tests/test_cli_surfaces_extra.py

@@ -15,7 +15,7 @@ runner = CliRunner()
 def test_version_option_outputs_package_version() -> None:
     result = runner.invoke(cli_main.app, ["--version"])
     assert result.exit_code == 0
-    assert result.stdout.strip() == "0.2.1"
+    assert result.stdout.strip() == "0.2.2"
 
 
 class _FakeClient:

{quilt_hp_python-0.2.1 → quilt_hp_python-0.2.2}/tests/test_models_from_proto.py

@@ -1216,3 +1216,106 @@ def test_away_setpoints_reflected_in_controls_when_away() -> None:
     assert s1_away.is_away is True
     assert s1_away.controls.heating_setpoint_c == pytest.approx(away.heating_setpoint_c)
     assert s1_away.controls.cooling_setpoint_c == pytest.approx(away.cooling_setpoint_c)
+
+
+# ─── SystemSnapshot.odu_for_idu ───────────────────────────────────────────────
+
+
+def _make_odu_proto(odu_id: str, space_id: str = "space-1") -> SimpleNamespace:
+    return _ns(
+        header=_make_header(odu_id),
+        relationships=_ns(space_id=space_id, hardware_id="", firmware_update_info_id=""),
+        state=_ns(hvac_state=0),
+        performance_data=_ns(
+            ambient_temperature_c=0.0,
+            compressor_frequency_hz=0.0,
+            energy_measurement_j=0.0,
+        ),
+    )
+
+
+def _make_snap_with_multiple_odus() -> SystemSnapshot:
+    from quilt_hp._proto import quilt_hds_pb2 as hds
+
+    loc = _ns(
+        header=_make_header("loc-1"),
+        attributes=_ns(name="", tz_identifier="UTC"),
+        controls=_ns(schedule_execution=hds.SCHEDULE_EXECUTION_RUNNING),
+    )
+
+    def _idu(idu_id: str, space_id: str, odu_id: str) -> SimpleNamespace:
+        proto = _make_idu_proto(idu_id, space_id)
+        proto.relationships = _ns(
+            space_id=space_id,
+            outdoor_unit_id=odu_id,
+            hardware_id="hw-1",
+            quilt_smart_module_id="",
+            firmware_update_info_id="",
+        )
+        return proto
+
+    return SystemSnapshot.from_proto(
+        _ns(
+            spaces=[
+                _make_space_proto("s1", "Room 1", parent_space_id="root"),
+                _make_space_proto("s2", "Room 2", parent_space_id="root"),
+                _make_space_proto("s3", "Room 3", parent_space_id="root"),
+            ],
+            indoor_units=[
+                _idu("idu-1", "s1", "odu-1"),
+                _idu("idu-2", "s2", "odu-2"),
+                _idu("idu-3", "s3", "odu-1"),  # shares odu-1 with room 1
+            ],
+            outdoor_units=[
+                _make_odu_proto("odu-1"),
+                _make_odu_proto("odu-2"),
+                _make_odu_proto("odu-3"),  # present but not linked to any IDU
+            ],
+            outdoor_unit_hardware=[],
+            controller_hardware=[],
+            controllers=[],
+            quilt_smart_modules=[],
+            comfort_settings=[],
+            schedule_weeks=[],
+            schedule_days=[],
+            remote_sensors=[],
+            controller_remote_sensors=[],
+            software_update_infos=[],
+            locations=[loc],
+        )
+    )
+
+
+def test_odu_for_idu_returns_correct_odu() -> None:
+    snap = _make_snap_with_multiple_odus()
+    idu1 = next(u for u in snap.indoor_units if u.id == "idu-1")
+    idu2 = next(u for u in snap.indoor_units if u.id == "idu-2")
+    idu3 = next(u for u in snap.indoor_units if u.id == "idu-3")
+
+    assert snap.odu_for_idu(idu1) is not None
+    assert snap.odu_for_idu(idu1).id == "odu-1"  # type: ignore[union-attr]
+
+    assert snap.odu_for_idu(idu2) is not None
+    assert snap.odu_for_idu(idu2).id == "odu-2"  # type: ignore[union-attr]
+
+    assert snap.odu_for_idu(idu3) is not None
+    assert snap.odu_for_idu(idu3).id == "odu-1"  # type: ignore[union-attr]
+
+
+def test_odu_for_idu_no_outdoor_unit_id_returns_none() -> None:
+    snap = _make_snap_with_multiple_odus()
+    idu1 = next(u for u in snap.indoor_units if u.id == "idu-1")
+    from dataclasses import replace
+
+    idu_no_odu = replace(idu1, outdoor_unit_id=None)
+    assert snap.odu_for_idu(idu_no_odu) is None
+
+
+def test_odu_for_idu_unlinked_odu_not_returned() -> None:
+    """ODU-3 exists in the snapshot but is not linked to any IDU."""
+    snap = _make_snap_with_multiple_odus()
+    assert len(snap.outdoor_units) == 3
+    assert all(
+        u.id != "odu-3"
+        for u in (snap.odu_for_idu(idu) for idu in snap.indoor_units if idu is not None)
+    )

{quilt_hp_python-0.2.1 → quilt_hp_python-0.2.2}/tests/test_tui_bindings.py

@@ -4,7 +4,7 @@ import pytest
 
 pytest.importorskip("textual")
 
-from quilt_hp.cli.tui import RoomScreen
+from quilt_hp.cli.tui import RoomScreen, SystemScreen
 
 
 def _key_action_map() -> dict[str, set[str]]:
@@ -21,3 +21,18 @@ def test_fan_key_does_not_overlap_with_fence_adjustment() -> None:
     assert "fence_fwd_inc" not in keymap["f"]
     assert "ctrl+up" in keymap
     assert keymap["ctrl+up"] == {"fence_fwd_inc"}
+
+
+def test_system_bindings_map_units_and_schedule_actions() -> None:
+    keymap: dict[str, set[str]] = {}
+    for binding in SystemScreen.BINDINGS:
+        for key in binding.key.split(","):
+            keymap.setdefault(key, set()).add(binding.action)
+
+    assert keymap["u"] == {"toggle_units"}
+    assert keymap["p"] == {"toggle_schedule"}
+
+
+def test_system_screen_accepts_initial_unit_preference() -> None:
+    screen = SystemScreen(snapshot=object(), client=object(), use_f=True)
+    assert screen.use_f is True