From c3e3a36b4c8deeb59fb4c00ba110ee9f5b69938c Mon Sep 17 00:00:00 2001
From: Avi Miller <me@dje.li>
Date: Thu, 12 Jun 2025 22:32:04 +1000
Subject: [PATCH] Fix palette handling for LIFX Ceiling SKY effect (#146582)

Signed-off-by: Avi Miller <me@dje.li>
---
 homeassistant/components/lifx/manager.py | 9 ++++++---
 tests/components/lifx/test_light.py      | 4 ++--
 2 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/homeassistant/components/lifx/manager.py b/homeassistant/components/lifx/manager.py
index 887bc3c3527..9fae2628f1d 100644
--- a/homeassistant/components/lifx/manager.py
+++ b/homeassistant/components/lifx/manager.py
@@ -5,7 +5,7 @@ from __future__ import annotations
 import asyncio
 from collections.abc import Callable
 from datetime import timedelta
-from typing import Any
+from typing import TYPE_CHECKING, Any
 
 import aiolifx_effects
 from aiolifx_themes.painter import ThemePainter
@@ -31,9 +31,12 @@ from homeassistant.helpers import config_validation as cv
 from homeassistant.helpers.service import async_extract_referenced_entity_ids
 
 from .const import _ATTR_COLOR_TEMP, ATTR_THEME, DATA_LIFX_MANAGER, DOMAIN
-from .coordinator import LIFXUpdateCoordinator, Light
+from .coordinator import LIFXUpdateCoordinator
 from .util import convert_8_to_16, find_hsbk
 
+if TYPE_CHECKING:
+    from aiolifx.aiolifx import Light
+
 SCAN_INTERVAL = timedelta(seconds=10)
 
 SERVICE_EFFECT_COLORLOOP = "effect_colorloop"
@@ -426,8 +429,8 @@ class LIFXManager:
     ) -> None:
         """Start the firmware-based Sky effect."""
         palette = kwargs.get(ATTR_PALETTE)
+        theme = Theme()
         if palette is not None:
-            theme = Theme()
             for hsbk in palette:
                 theme.add_hsbk(hsbk[0], hsbk[1], hsbk[2], hsbk[3])
 
diff --git a/tests/components/lifx/test_light.py b/tests/components/lifx/test_light.py
index 58843d63f9a..d66908c1b1a 100644
--- a/tests/components/lifx/test_light.py
+++ b/tests/components/lifx/test_light.py
@@ -843,7 +843,7 @@ async def test_sky_effect(hass: HomeAssistant) -> None:
         SERVICE_EFFECT_SKY,
         {
             ATTR_ENTITY_ID: entity_id,
-            ATTR_PALETTE: [],
+            ATTR_PALETTE: None,
             ATTR_SKY_TYPE: "Clouds",
             ATTR_CLOUD_SATURATION_MAX: 180,
             ATTR_CLOUD_SATURATION_MIN: 50,
@@ -854,7 +854,7 @@ async def test_sky_effect(hass: HomeAssistant) -> None:
     bulb.power_level = 65535
     bulb.effect = {
         "effect": "SKY",
-        "palette": [],
+        "palette": None,
         "sky_type": 2,
         "cloud_saturation_min": 50,
         "cloud_saturation_max": 180,