lifx-async 4.7.4__tar.gz → 4.7.5__tar.gz

{lifx_async-4.7.4 → lifx_async-4.7.5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: lifx-async
-Version: 4.7.4
+Version: 4.7.5
 Summary: A modern, type-safe, async Python library for controlling LIFX lights
 Author-email: Avi Miller <me@dje.li>
 Maintainer-email: Avi Miller <me@dje.li>

{lifx_async-4.7.4 → lifx_async-4.7.5}/docs/changelog.md

@@ -2,6 +2,14 @@
 
 <!-- version list -->
 
+## v4.7.5 (2025-12-16)
+
+### Bug Fixes
+
+- **devices**: Override set_color in CeilingLight to track component state
+  ([`0d20563`](https://github.com/Djelibeybi/lifx-async/commit/0d20563c170363229ab17620398283bd85ee7829))
+
+
 ## v4.7.4 (2025-12-16)
 
 ### Performance Improvements

{lifx_async-4.7.4 → lifx_async-4.7.5}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "lifx-async"
-version = "4.7.4"
+version = "4.7.5"
 description = "A modern, type-safe, async Python library for controlling LIFX lights"
 readme = "README.md"
 requires-python = ">=3.11"

{lifx_async-4.7.4 → lifx_async-4.7.5}/src/lifx/devices/ceiling.py

@@ -347,6 +347,22 @@ class CeilingLight(MatrixLight):
 
         return layout.downlight_zones
 
+    @property
+    def downlight_zone_count(self) -> int:
+        """Number of downlight zones.
+
+        Returns:
+            Zone count (63 for standard 8x8, 127 for Capsule 16x8)
+
+        Raises:
+            LifxError: If device version is not available or not a Ceiling product
+        """
+        # downlight_zones is slice(0, N), so stop equals the count
+        stop = self.downlight_zones.stop
+        if stop is None:
+            raise LifxError("Invalid downlight zones configuration")
+        return stop
+
     @property
     def uplight_is_on(self) -> bool:
         """True if uplight component is currently on.
@@ -496,7 +512,7 @@ class CeilingLight(MatrixLight):
                     "Cannot set downlight color with brightness=0. "
                     "Use turn_downlight_off() instead."
                 )
-            downlight_colors = [colors] * len(range(*self.downlight_zones.indices(256)))
+            downlight_colors = [colors] * self.downlight_zone_count
         else:
             if all(c.brightness == 0 for c in colors):
                 raise ValueError(
@@ -504,10 +520,10 @@ class CeilingLight(MatrixLight):
                     "Use turn_downlight_off() instead."
                 )
 
-            expected_count = len(range(*self.downlight_zones.indices(256)))
-            if len(colors) != expected_count:
+            if len(colors) != self.downlight_zone_count:
                 raise ValueError(
-                    f"Expected {expected_count} colors for downlight, got {len(colors)}"
+                    f"Expected {self.downlight_zone_count} colors for downlight, "
+                    f"got {len(colors)}"
                 )
             downlight_colors = colors
 
@@ -684,10 +700,6 @@ class CeilingLight(MatrixLight):
             ValueError: If list length doesn't match downlight zone count
             LifxTimeoutError: Device did not respond
         """
-        # Number of downlight zones equals the uplight zone index
-        # (downlight is zones 0 to uplight_zone-1)
-        downlight_zone_count = self.uplight_zone
-
         # Validate provided colors early
         if colors is not None:
             if isinstance(colors, HSBK):
@@ -696,9 +708,9 @@ class CeilingLight(MatrixLight):
             else:
                 if all(c.brightness == 0 for c in colors):
                     raise ValueError("Cannot turn on downlight with brightness=0")
-                if len(colors) != downlight_zone_count:
+                if len(colors) != self.downlight_zone_count:
                     raise ValueError(
-                        f"Expected {downlight_zone_count} colors for downlight, "
+                        f"Expected {self.downlight_zone_count} colors for downlight, "
                         f"got {len(colors)}"
                     )
 
@@ -711,7 +723,7 @@ class CeilingLight(MatrixLight):
             # Determine target colors (pass pre-fetched colors to avoid extra fetch)
             if colors is not None:
                 if isinstance(colors, HSBK):
-                    target_colors = [colors] * downlight_zone_count
+                    target_colors = [colors] * self.downlight_zone_count
                 else:
                     target_colors = list(colors)
             else:
@@ -750,7 +762,7 @@ class CeilingLight(MatrixLight):
             # Light is already on - determine target colors first, then set
             if colors is not None:
                 if isinstance(colors, HSBK):
-                    target_colors = [colors] * downlight_zone_count
+                    target_colors = [colors] * self.downlight_zone_count
                 else:
                     target_colors = list(colors)
             else:
@@ -824,6 +836,54 @@ class CeilingLight(MatrixLight):
         if turning_off and self._state_file:
             self._save_state_to_file()
 
+    async def set_color(self, color: HSBK, duration: float = 0.0) -> None:
+        """Set light color, updating component state tracking.
+
+        Overrides Light.set_color() to track the color change in the ceiling
+        light's component state. When set_color() is called, all zones (uplight
+        and downlight) are set to the same color. This override ensures that
+        the cached component colors stay in sync so that subsequent component
+        control methods (like turn_uplight_on or turn_downlight_on) use the
+        correct color values.
+
+        Args:
+            color: HSBK color to set for the entire light
+            duration: Transition duration in seconds (default 0.0)
+
+        Raises:
+            LifxDeviceNotFoundError: If device is not connected
+            LifxTimeoutError: If device does not respond
+            LifxUnsupportedCommandError: If device doesn't support this command
+
+        Example:
+            ```python
+            from lifx.color import HSBK
+
+            # Set entire ceiling light to warm white
+            await ceiling.set_color(
+                HSBK(hue=0, saturation=0, brightness=1.0, kelvin=2700)
+            )
+
+            # Later component control will use this color
+            await ceiling.turn_uplight_off()  # Uplight off
+            await ceiling.turn_uplight_on()  # Restores to warm white
+            ```
+        """
+        # Call parent to perform actual color change
+        await super().set_color(color, duration)
+
+        # Update cached component colors - all zones now have the same color
+        self._last_uplight_color = color
+        self._last_downlight_colors = [color] * self.downlight_zone_count
+
+        # Also update stored state for restoration
+        self._stored_uplight_state = color
+        self._stored_downlight_state = [color] * self.downlight_zone_count
+
+        # Persist if enabled
+        if self._state_file:
+            self._save_state_to_file()
+
     async def turn_downlight_off(
         self, colors: HSBK | list[HSBK] | None = None, duration: float = 0.0
     ) -> None:
@@ -845,8 +905,6 @@ class CeilingLight(MatrixLight):
         Note:
             Sets all downlight zone brightness to 0 on device while preserving H, S, K.
         """
-        expected_count = len(range(*self.downlight_zones.indices(256)))
-
         # Validate provided colors early (before fetching)
         stored_colors: list[HSBK] | None = None
         if colors is not None:
@@ -856,16 +914,16 @@ class CeilingLight(MatrixLight):
                         "Provided color cannot have brightness=0. "
                         "Omit the parameter to use current colors."
                     )
-                stored_colors = [colors] * expected_count
+                stored_colors = [colors] * self.downlight_zone_count
             else:
                 if all(c.brightness == 0 for c in colors):
                     raise ValueError(
                         "Provided colors cannot have brightness=0. "
                         "Omit the parameter to use current colors."
                     )
-                if len(colors) != expected_count:
+                if len(colors) != self.downlight_zone_count:
                     raise ValueError(
-                        f"Expected {expected_count} colors for downlight, "
+                        f"Expected {self.downlight_zone_count} colors for downlight, "
                         f"got {len(colors)}"
                     )
                 stored_colors = list(colors)

{lifx_async-4.7.4 → lifx_async-4.7.5}/tests/test_devices/test_ceiling.py

@@ -807,23 +807,49 @@ class TestCeilingLightBackwardCompatibility:
         """Create a Ceiling product 176 instance with mocked connection."""
         ceiling = CeilingLight(serial="d073d5010203", ip="192.168.1.100")
         ceiling.connection = AsyncMock()
+        ceiling._save_state_to_file = MagicMock()
+
+        # Mock version for product detection
+        ceiling._version = MagicMock()
+        ceiling._version.product = 176
         return ceiling
 
     async def test_set_color_affects_both_components(
         self, ceiling_176: CeilingLight
     ) -> None:
-        """Test inherited set_color affects both uplight and downlight."""
+        """Test set_color calls parent and updates component state tracking."""
         color = HSBK(hue=180, saturation=0.8, brightness=1.0, kelvin=5000)
 
-        # Mock the parent set_color method
-        ceiling_176.set_matrix_colors = AsyncMock()
-
         await ceiling_176.set_color(color)
 
-        # Verify set_color was called (from parent class)
-        # This would set all zones including both components
+        # Verify parent set_color was called (via connection.request)
         assert ceiling_176.connection.request.called
 
+        # Verify uplight state was updated
+        assert ceiling_176._last_uplight_color == color
+        assert ceiling_176._stored_uplight_state == color
+
+        # Verify downlight state was updated (63 zones for product 176)
+        assert ceiling_176._last_downlight_colors == [color] * 63
+        assert ceiling_176._stored_downlight_state == [color] * 63
+
+        # Verify state was NOT persisted (no state_file configured)
+        ceiling_176._save_state_to_file.assert_not_called()
+
+    async def test_set_color_persists_state_when_file_configured(
+        self, ceiling_176: CeilingLight
+    ) -> None:
+        """Test set_color persists state when state_file is configured."""
+        color = HSBK(hue=180, saturation=0.8, brightness=1.0, kelvin=5000)
+
+        # Enable state file persistence
+        ceiling_176._state_file = "/tmp/test_state.json"
+
+        await ceiling_176.set_color(color)
+
+        # Verify state was persisted
+        ceiling_176._save_state_to_file.assert_called_once()
+
     async def test_matrixlight_methods_still_work(
         self, ceiling_176: CeilingLight
     ) -> None:

{lifx_async-4.7.4 → lifx_async-4.7.5}/uv.lock

@@ -337,7 +337,7 @@ wheels = [
 
 [[package]]
 name = "lifx-async"
-version = "4.7.4"
+version = "4.7.5"
 source = { editable = "." }
 
 [package.dev-dependencies]