fix: Enforce bounds on color temp in line with profile

Author: dljsjr

drivers/SmartThings/philips-hue/src/handlers.lua

@@ -148,9 +148,8 @@ local function do_color_temp_action(driver, device, args)
     return
   end
 
-  local clamped_kelvin = st_utils.clamp_value(
-    kelvin, HueApi.MIN_TEMP_KELVIN, HueApi.MAX_TEMP_KELVIN
-  )
+  local min = device:get_field(Fields.MIN_KELVIN) or HueApi.MIN_TEMP_KELVIN_WHITE_AMBIANCE
+  local clamped_kelvin = st_utils.clamp_value(kelvin, min, HueApi.MAX_TEMP_KELVIN)
   local mirek = math.floor(handlers.kelvin_to_mirek(clamped_kelvin))
 
   local resp, err = hue_api:set_light_color_temp(light_id, mirek)

drivers/SmartThings/philips-hue/src/hue/api.lua

@@ -41,7 +41,8 @@ PhilipsHueApi.__gc = function(self)
 end
 
 PhilipsHueApi.MIN_CLIP_V2_SWVERSION = 1948086000
-PhilipsHueApi.MIN_TEMP_KELVIN = 2000
+PhilipsHueApi.MIN_TEMP_KELVIN_COLOR_AMBIANCE = 2000
+PhilipsHueApi.MIN_TEMP_KELVIN_WHITE_AMBIANCE = 2200
 PhilipsHueApi.MAX_TEMP_KELVIN = 6500
 PhilipsHueApi.APPLICATION_KEY_HEADER = APPLICATION_KEY_HEADER
 

drivers/SmartThings/philips-hue/src/hue/fields.lua

@@ -23,6 +23,7 @@ local Fields = {
   HUE_DEVICE_ID = "hue_device_id",
   IPV4 = "ipv4",
   MIN_DIMMING = "mindim",
+  MIN_KELVIN = "mintemp",
   MODEL_ID = "modelid",
   PARENT_DEVICE_ID = "parent_device_id_local",
   RESOURCE_ID = "rid"

drivers/SmartThings/philips-hue/src/init.lua

@@ -78,9 +78,11 @@ local function emit_light_status_events(light_device, light)
       if light.color_temperature.mirek_valid then
         mirek = light.color_temperature.mirek
       end
-      light_device:emit_event(
-        capabilities.colorTemperature.colorTemperature(math.floor(handlers.mirek_to_kelvin(mirek)))
+      local min = light_device:get_field(Fields.MIN_KELVIN) or HueApi.MIN_TEMP_KELVIN_WHITE_AMBIANCE
+      local kelvin = math.floor(
+        st_utils.clamp_value(handlers.mirek_to_kelvin(mirek), min, HueApi.MAX_TEMP_KELVIN)
       )
+      light_device:emit_event(capabilities.colorTemperature.colorTemperature(kelvin))
     end
 
     if light.color then
@@ -501,6 +503,14 @@ end
 ---@param driver HueDriver
 ---@param device HueDevice
 local function init_light(driver, device)
+  local caps = device.profile.components.main.capabilities
+  if caps.colorTemperature then
+    if caps.colorControl then
+      device:set_field(Fields.MIN_KELVIN, HueApi.MIN_TEMP_KELVIN_COLOR_AMBIANCE, { persist = true })
+    else
+      device:set_field(Fields.MIN_KELVIN, HueApi.MIN_TEMP_KELVIN_WHITE_AMBIANCE, { persist = true})
+    end
+  end
   local device_light_resource_id = device:get_field(Fields.RESOURCE_ID) or device.device_network_id
   local hue_device_id = device:get_field(Fields.HUE_DEVICE_ID)
   if not driver.light_id_to_device[device_light_resource_id] then