Add required TODOs and optimize the skeleton code

Author: arun-silabs

examples/oven-app/oven-app-common/include/CookSurfaceEndpoint.h

@@ -30,7 +30,7 @@ namespace CookSurface {
 class CookSurfaceEndpoint
 {
 public:
-    CookSurfaceEndpoint(EndpointId endpointId) {}
+    CookSurfaceEndpoint(EndpointId endpointId) : mEndpointId(endpointId) {}
 
     /**
      * @brief Initialize the CookSurface endpoint.
@@ -41,6 +41,10 @@ class CookSurfaceEndpoint
      * @brief Handle the "off" command for the CookSurface.
      */
     void HandleOffCommand();
+
+private:
+    bool currentOnOffState = false;
+    EndpointId mEndpointId = kInvalidEndpointId;
 };
 
 } // namespace CookSurface

examples/oven-app/oven-app-common/include/CookTopEndpoint.h

@@ -29,7 +29,7 @@ namespace CookTop {
 class CookTopEndpoint
 {
 public:
-    CookTopEndpoint(EndpointId endpointId) {}
+    CookTopEndpoint(EndpointId endpointId) : mEndpointId(endpointId) {}
 
     /**
      * @brief Initialize the cooktop endpoint.
@@ -42,6 +42,10 @@ class CookTopEndpoint
      * @brief Handle the "off" command for the cooktop.
      */
     void HandleOffCommand();
+
+private:
+    bool currentOnOffState = false;
+    EndpointId mEndpointId = kInvalidEndpointId;
 };
 
 } // namespace CookTop

examples/oven-app/oven-app-common/include/OvenEndpoint.h

@@ -21,24 +21,87 @@
 #include <app-common/zap-generated/cluster-objects.h>
 #include <lib/core/CHIPError.h>
 #include <lib/core/DataModelTypes.h>
+#include <app/clusters/mode-base-server/mode-base-server.h>
+#include <app/clusters/temperature-control-server/supported-temperature-levels-manager.h>
+#include <app/data-model/List.h>
+#include <lib/support/Span.h>
 
 namespace chip {
 namespace app {
 namespace Clusters {
-namespace Oven {
+namespace TemperatureControlledCabinet {
+class OvenModeDelegate : public ModeBase::Delegate
+{
+public:
+    // Oven mode constants - made static and public for potential reuse
+    static constexpr uint8_t kModeBake            = 0;
+    static constexpr uint8_t kModeConvection      = 1;
+    static constexpr uint8_t kModeGrill           = 2;
+    static constexpr uint8_t kModeRoast           = 3;
+    static constexpr uint8_t kModeClean           = 4;
+    static constexpr uint8_t kModeConvectionBake  = 5;
+    static constexpr uint8_t kModeConvectionRoast = 6;
+    static constexpr uint8_t kModeWarming         = 7;
+    static constexpr uint8_t kModeProofing        = 8;
+    static constexpr uint8_t kModeCount           = 9;
+
+    OvenModeDelegate(EndpointId endpointId) : mEndpointId(endpointId) {}
+
+    // ModeBase::Delegate interface
+    CHIP_ERROR Init() override;
+    void HandleChangeToMode(uint8_t mode, ModeBase::Commands::ChangeToModeResponse::Type & response) override;
+    CHIP_ERROR GetModeLabelByIndex(uint8_t modeIndex, MutableCharSpan & label) override;
+    CHIP_ERROR GetModeValueByIndex(uint8_t modeIndex, uint8_t & value) override;
+    CHIP_ERROR GetModeTagsByIndex(uint8_t modeIndex, DataModel::List<detail::Structs::ModeTagStruct::Type> & tags) override;
+
+private:
+    EndpointId mEndpointId;
+
+    // Static arrays moved to implementation file to reduce header size
+    static const detail::Structs::ModeTagStruct::Type sModeTagsBake[];
+    static const detail::Structs::ModeTagStruct::Type sModeTagsConvection[];
+    static const detail::Structs::ModeTagStruct::Type sModeTagsGrill[];
+    static const detail::Structs::ModeTagStruct::Type sModeTagsRoast[];
+    static const detail::Structs::ModeTagStruct::Type sModeTagsClean[];
+    static const detail::Structs::ModeTagStruct::Type sModeTagsConvectionBake[];
+    static const detail::Structs::ModeTagStruct::Type sModeTagsConvectionRoast[];
+    static const detail::Structs::ModeTagStruct::Type sModeTagsWarming[];
+    static const detail::Structs::ModeTagStruct::Type sModeTagsProofing[];
+
+    static const detail::Structs::ModeOptionStruct::Type skModeOptions[];
+};
+
+class TemperatureControlledCabinetEndpoint
+{
+public:
+    TemperatureControlledCabinetEndpoint(EndpointId endpointId) :
+        mEndpointId(endpointId), mOvenModeDelegate(endpointId), mOvenModeInstance(&mOvenModeDelegate, mEndpointId, OvenMode::Id, 0)
+    {}
+
+    /**
+     * Initialize the temperature controlled cabinet endpoint.
+     */
+    CHIP_ERROR Init();
+
+private:
+    EndpointId mEndpointId = kInvalidEndpointId;
+    OvenModeDelegate mOvenModeDelegate;
+    ModeBase::Instance mOvenModeInstance;
+};
 
+} // namespace TemperatureControlledCabinet
+
+namespace Oven {
 /**
  * @brief Base oven endpoint placeholder.
  */
 class OvenEndpoint
 {
 public:
-    OvenEndpoint(EndpointId endpointId) {}
+    OvenEndpoint() {}
 
     /**
      * @brief Initialize the oven endpoint.
-     *
-     * @return CHIP_ERROR indicating success or failure of the initialization.
      */
     CHIP_ERROR Init();
 };

examples/oven-app/oven-app-common/include/OvenModeDelegate.h

@@ -24,8 +24,6 @@ using namespace chip::app::Clusters::CookSurface;
 
 CHIP_ERROR CookSurfaceEndpoint::Init()
 {
-    ChipLogProgress(AppServer, "CookSurfaceEndpoint::Init()");
-    // TODO: Add Endpoint initialization logic
     return CHIP_NO_ERROR;
 }
 

examples/oven-app/oven-app-common/include/TemperatureControlledCabinetEndpoint.h

@@ -24,7 +24,6 @@ using namespace chip::app::Clusters::CookTop;
 
 CHIP_ERROR CookTopEndpoint::Init()
 {
-    // TODO: Add CookTop Endpoint Initialization logic
     return CHIP_NO_ERROR;
 }