ci(pre-commit): Apply automatic fixes

Author: pre-commit-ci-lite[bot]

libraries/Matter/examples/MatterEnhancedColorLight/MatterEnhancedColorLight.ino

@@ -115,7 +115,7 @@ void setup() {
     Serial.printf("Light Color Temperature changed to %d\r\n", colorTemperature);
     // get correspondent Hue and Saturation of the color temperature
     HsvColor_t hsvTemperature = espRgbColorToHsvColor(espCTToRgbColor(colorTemperature));
-    // keep previous the brightness and just change the Hue and Saturation 
+    // keep previous the brightness and just change the Hue and Saturation
     currentHSVColor.h = hsvTemperature.h;
     currentHSVColor.s = hsvTemperature.s;
     return true;
@@ -128,7 +128,7 @@ void setup() {
   });
   EnhancedColorLight.onChangeColorHSV([](HsvColor_t hsvColor) {
     Serial.printf("Light HSV Color changed to (%d,%d,%d)\r\n", hsvColor.h, hsvColor.s, hsvColor.v);
-    // keep the current brightness and just change Hue and Saturation 
+    // keep the current brightness and just change Hue and Saturation
     currentHSVColor.h = hsvColor.h;
     currentHSVColor.s = hsvColor.s;
     return true;
@@ -140,8 +140,8 @@ void setup() {
   if (Matter.isDeviceCommissioned()) {
     Serial.println("Matter Node is commissioned and connected to Wi-Fi. Ready for use.");
     Serial.printf(
-      "Initial state: %s | RGB Color: (%d,%d,%d) \r\n", EnhancedColorLight ? "ON" : "OFF",
-      EnhancedColorLight.getColorRGB().r, EnhancedColorLight.getColorRGB().g, EnhancedColorLight.getColorRGB().b
+      "Initial state: %s | RGB Color: (%d,%d,%d) \r\n", EnhancedColorLight ? "ON" : "OFF", EnhancedColorLight.getColorRGB().r,
+      EnhancedColorLight.getColorRGB().g, EnhancedColorLight.getColorRGB().b
     );
     // configure the Light based on initial on-off state and its color
     EnhancedColorLight.updateAccessory();
@@ -171,8 +171,8 @@ void loop() {
       }
     }
     Serial.printf(
-      "Initial state: %s | RGB Color: (%d,%d,%d) \r\n", EnhancedColorLight ? "ON" : "OFF",
-      EnhancedColorLight.getColorRGB().r, EnhancedColorLight.getColorRGB().g, EnhancedColorLight.getColorRGB().b
+      "Initial state: %s | RGB Color: (%d,%d,%d) \r\n", EnhancedColorLight ? "ON" : "OFF", EnhancedColorLight.getColorRGB().r,
+      EnhancedColorLight.getColorRGB().g, EnhancedColorLight.getColorRGB().b
     );
     // configure the Light based on initial on-off state and its color
     EnhancedColorLight.updateAccessory();

libraries/Matter/examples/MatterEnhancedColorLight/ci.json

@@ -5,4 +5,4 @@
       "CONFIG_ESP_MATTER_ENABLE_DATA_MODEL=y"
     ]
 }
-  
+

libraries/Matter/src/MatterEndpoints/MatterColorTemperatureLight.h

@@ -57,7 +57,7 @@ class MatterColorTemperatureLight : public MatterEndPoint {
   void onChangeOnOff(EndPointOnOffCB onChangeCB) {
     _onChangeOnOffCB = onChangeCB;
   }
- 
+
   // User Callback for whenever the Light brightness value [0..255] is changed by the Matter Controller
   using EndPointBrightnessCB = std::function<bool(uint8_t)>;
   void onChangeBrightness(EndPointBrightnessCB onChangeCB) {

libraries/Matter/src/MatterEndpoints/MatterEnhancedColorLight.cpp

@@ -25,63 +25,65 @@ using namespace chip::app::Clusters;
 
 // endpoint for enhanced color light device
 namespace esp_matter {
-  using namespace cluster;
-namespace endpoint { 
+using namespace cluster;
+namespace endpoint {
 namespace enhanced_color_light {
 typedef struct config {
-    cluster::descriptor::config_t descriptor;
-    cluster::identify::config_t identify;
-    cluster::groups::config_t groups;
-    cluster::scenes_management::config_t scenes_management;
-    cluster::on_off::config_t on_off;
-    cluster::level_control::config_t level_control;
-    cluster::color_control::config_t color_control;
+  cluster::descriptor::config_t descriptor;
+  cluster::identify::config_t identify;
+  cluster::groups::config_t groups;
+  cluster::scenes_management::config_t scenes_management;
+  cluster::on_off::config_t on_off;
+  cluster::level_control::config_t level_control;
+  cluster::color_control::config_t color_control;
 } config_t;
 
 uint32_t get_device_type_id() {
-    return ESP_MATTER_EXTENDED_COLOR_LIGHT_DEVICE_TYPE_ID;
+  return ESP_MATTER_EXTENDED_COLOR_LIGHT_DEVICE_TYPE_ID;
 }
 
 uint8_t get_device_type_version() {
-    return ESP_MATTER_EXTENDED_COLOR_LIGHT_DEVICE_TYPE_VERSION;
+  return ESP_MATTER_EXTENDED_COLOR_LIGHT_DEVICE_TYPE_VERSION;
 }
 
 esp_err_t add(endpoint_t *endpoint, config_t *config) {
-    if (!endpoint) {
-        log_e("Endpoint cannot be NULL");
-        return ESP_ERR_INVALID_ARG;
-    }
-    esp_err_t err = add_device_type(endpoint, get_device_type_id(), get_device_type_version());
-    if (err != ESP_OK) {
-        log_e("Failed to add device type id:%" PRIu32 ",err: %d", get_device_type_id(), err);
-        return err;
-    }
+  if (!endpoint) {
+    log_e("Endpoint cannot be NULL");
+    return ESP_ERR_INVALID_ARG;
+  }
+  esp_err_t err = add_device_type(endpoint, get_device_type_id(), get_device_type_version());
+  if (err != ESP_OK) {
+    log_e("Failed to add device type id:%" PRIu32 ",err: %d", get_device_type_id(), err);
+    return err;
+  }
 
-    descriptor::create(endpoint, &(config->descriptor), CLUSTER_FLAG_SERVER);
-    cluster_t *identify_cluster = identify::create(endpoint, &(config->identify), CLUSTER_FLAG_SERVER);
-    identify::command::create_trigger_effect(identify_cluster);
-    groups::create(endpoint, &(config->groups), CLUSTER_FLAG_SERVER);
-    cluster_t *scenes_cluster = scenes_management::create(endpoint, &(config->scenes_management), CLUSTER_FLAG_SERVER);
-    scenes_management::command::create_copy_scene(scenes_cluster);
-    scenes_management::command::create_copy_scene_response(scenes_cluster);
-
-    on_off::create(endpoint, &(config->on_off), CLUSTER_FLAG_SERVER, on_off::feature::lighting::get_id());
-    level_control::create(endpoint, &(config->level_control), CLUSTER_FLAG_SERVER,
-                          level_control::feature::on_off::get_id() | level_control::feature::lighting::get_id());
-    color_control::create(endpoint, &(config->color_control), CLUSTER_FLAG_SERVER,
-                          color_control::feature::hue_saturation::get_id() | color_control::feature::color_temperature::get_id());
-    return ESP_OK;
+  descriptor::create(endpoint, &(config->descriptor), CLUSTER_FLAG_SERVER);
+  cluster_t *identify_cluster = identify::create(endpoint, &(config->identify), CLUSTER_FLAG_SERVER);
+  identify::command::create_trigger_effect(identify_cluster);
+  groups::create(endpoint, &(config->groups), CLUSTER_FLAG_SERVER);
+  cluster_t *scenes_cluster = scenes_management::create(endpoint, &(config->scenes_management), CLUSTER_FLAG_SERVER);
+  scenes_management::command::create_copy_scene(scenes_cluster);
+  scenes_management::command::create_copy_scene_response(scenes_cluster);
+
+  on_off::create(endpoint, &(config->on_off), CLUSTER_FLAG_SERVER, on_off::feature::lighting::get_id());
+  level_control::create(
+    endpoint, &(config->level_control), CLUSTER_FLAG_SERVER, level_control::feature::on_off::get_id() | level_control::feature::lighting::get_id()
+  );
+  color_control::create(
+    endpoint, &(config->color_control), CLUSTER_FLAG_SERVER,
+    color_control::feature::hue_saturation::get_id() | color_control::feature::color_temperature::get_id()
+  );
+  return ESP_OK;
 }
 
 endpoint_t *create(node_t *node, config_t *config, uint8_t flags, void *priv_data) {
-    endpoint_t *endpoint = endpoint::create(node, flags, priv_data);
-    add(endpoint, config);
-    return endpoint;
+  endpoint_t *endpoint = endpoint::create(node, flags, priv_data);
+  add(endpoint, config);
+  return endpoint;
 }
-} /* enhanced_color_light */
-} /* endpoint */
-} /* esp_matter */
-
+}  // namespace enhanced_color_light
+}  // namespace endpoint
+}  // namespace esp_matter
 
 bool MatterEnhancedColorLight::attributeChangeCB(uint16_t endpoint_id, uint32_t cluster_id, uint32_t attribute_id, esp_matter_attr_val_t *val) {
   bool ret = true;
@@ -90,7 +92,10 @@ bool MatterEnhancedColorLight::attributeChangeCB(uint16_t endpoint_id, uint32_t
     return false;
   }
 
-  log_d("Enhanced ColorAttr update callback: endpoint: %u, cluster: %u, attribute: %u, val: %u, type: %u", endpoint_id, cluster_id, attribute_id, val->val.u32, val->type);
+  log_d(
+    "Enhanced ColorAttr update callback: endpoint: %u, cluster: %u, attribute: %u, val: %u, type: %u", endpoint_id, cluster_id, attribute_id, val->val.u32,
+    val->type
+  );
 
   if (endpoint_id == getEndPointId()) {
     switch (cluster_id) {
@@ -101,7 +106,7 @@ bool MatterEnhancedColorLight::attributeChangeCB(uint16_t endpoint_id, uint32_t
             ret &= _onChangeOnOffCB(val->val.b);
           }
           if (_onChangeCB != NULL) {
-            ret &= _onChangeCB(val->val.b, colorHSV,  brightnessLevel, colorTemperatureLevel);
+            ret &= _onChangeCB(val->val.b, colorHSV, brightnessLevel, colorTemperatureLevel);
           }
           if (ret == true) {
             onOffState = val->val.b;
@@ -145,13 +150,13 @@ bool MatterEnhancedColorLight::attributeChangeCB(uint16_t endpoint_id, uint32_t
         if (attribute_id == ColorControl::Attributes::CurrentHue::Id) {
           log_d("Enhanced ColorLight Hue changed to %d", val->val.u8);
           hsvColor.h = val->val.u8;
-        } else { // attribute_id == ColorControl::Attributes::CurrentSaturation::Id)
+        } else {  // attribute_id == ColorControl::Attributes::CurrentSaturation::Id)
           log_d("Enhanced ColorLight Saturation changed to %d", val->val.u8);
           hsvColor.s = val->val.u8;
         }
         if (_onChangeColorCB != NULL) {
           ret &= _onChangeColorCB(hsvColor);
-        }  
+        }
         if (_onChangeCB != NULL) {
           ret &= _onChangeCB(onOffState, hsvColor, brightnessLevel, colorTemperatureLevel);
         }
@@ -338,7 +343,7 @@ bool MatterEnhancedColorLight::setColorHSV(espHsvColor_t _hsvColor) {
     return true;
   }
 
-  colorHSV = { _hsvColor.h, _hsvColor.s, _hsvColor.v };
+  colorHSV = {_hsvColor.h, _hsvColor.s, _hsvColor.v};
 
   endpoint_t *endpoint = endpoint::get(node::get(), endpoint_id);
   cluster_t *cluster = cluster::get(endpoint, ColorControl::Id);

libraries/Matter/src/MatterEndpoints/MatterEnhancedColorLight.h

@@ -28,7 +28,7 @@ class MatterEnhancedColorLight : public MatterEndPoint {
   MatterEnhancedColorLight();
   ~MatterEnhancedColorLight();
   // default initial state is off, brigthness = 25 (10%), HSV(21, 216, 25), color temperature is 454 (Warm White)
-  virtual bool begin(bool initialState = false, espHsvColor_t colorHSV = { 21, 216, 25}, uint8_t newBrightness = 25, uint16_t colorTemperature = 454);
+  virtual bool begin(bool initialState = false, espHsvColor_t colorHSV = {21, 216, 25}, uint8_t newBrightness = 25, uint16_t colorTemperature = 454);
   // this will just stop processing Light Matter events
   void end();
 
@@ -47,7 +47,6 @@ class MatterEnhancedColorLight : public MatterEndPoint {
   bool setColorHSV(espHsvColor_t hsvColor);  // returns true if successful
   espHsvColor_t getColorHSV();               // returns current HSV Color
 
-
   // used to update the state of the light using the current Matter Light internal state
   // It is necessary to set a user callback function using onChange() to handle the physical light state
   void updateAccessory();
@@ -90,10 +89,10 @@ class MatterEnhancedColorLight : public MatterEndPoint {
 
 protected:
   bool started = false;
-  bool onOffState = false;                // default initial state is off, but it can be changed by begin(bool)
-  uint8_t brightnessLevel = 0;            // default initial brightness is 0, but it can be changed by begin(bool, uint8_t)
-  espHsvColor_t colorHSV = { 0 };         // default initial color HSV is black, but it can be changed by begin(bool, uint8_t, espHsvColor_t)
-  uint16_t colorTemperatureLevel = 0;     // default initial color temperature is 0, but it can be changed by begin(bool, uint8_t, espHsvColor_t, uint16_t)
+  bool onOffState = false;             // default initial state is off, but it can be changed by begin(bool)
+  uint8_t brightnessLevel = 0;         // default initial brightness is 0, but it can be changed by begin(bool, uint8_t)
+  espHsvColor_t colorHSV = {0};        // default initial color HSV is black, but it can be changed by begin(bool, uint8_t, espHsvColor_t)
+  uint16_t colorTemperatureLevel = 0;  // default initial color temperature is 0, but it can be changed by begin(bool, uint8_t, espHsvColor_t, uint16_t)
   EndPointOnOffCB _onChangeOnOffCB = NULL;
   EndPointBrightnessCB _onChangeBrightnessCB = NULL;
   EndPointRGBColorCB _onChangeColorCB = NULL;