feat(matter): api simplification with begin

Author: SuGlider

libraries/Matter/examples/MatterCommissionTest/MatterCommissionTest.ino

@@ -39,13 +39,11 @@ void setup() {
   Serial.println(WiFi.localIP());
   delay(500);
 
-  // Initialize Matter Node
-  Matter.begin();
-  // Initialize Matter EndPoint
+  // Initialize at least one Matter EndPoint
   OnOffLight.begin();
-  // Matter start - Last step, after all EndPoints are initialized
-
-  Matter.start();
+  
+  // Matter begining - Last step, after all EndPoints are initialized
+  Matter.begin();
 }
 
 void loop() {

libraries/Matter/examples/MatterComposedLights/MatterComposedLights.ino

@@ -57,9 +57,6 @@ void setup() {
   Serial.println(WiFi.localIP());
   delay(500);
 
-  // Initialize Matter Node
-  Matter.begin();
-
   // Initialize all 3 Matter EndPoints
   OnOffLight1.begin();
   OnOffLight2.begin();
@@ -68,8 +65,8 @@ void setup() {
   OnOffLight2.onChangeOnOff(setLightOnOff2);
   OnOffLight3.onChangeOnOff(setLightOnOff3);
 
-  // Matter start - Last step, after all EndPoints are initialized
-  Matter.start();
+  // Matter begining - Last step, after all EndPoints are initialized
+  Matter.begin();
 }
 
 void loop() {

libraries/Matter/examples/MatterOnOffLight/MatterOnOffLight.ino

@@ -59,15 +59,12 @@ void setup() {
   Serial.println(WiFi.localIP());
   delay(500);
 
-  // Initialize Matter Node
-  Matter.begin();
-
   // Initialize Matter EndPoint
   OnOffLight.begin(true);
   OnOffLight.onChangeOnOff(setLightOnOff);
 
-  // Matter start - Last step, after all EndPoints are initialized
-  Matter.start();
+  // Matter begining - Last step, after all EndPoints are initialized
+  Matter.begin();
 }
 
 uint32_t lastMillis = millis();

libraries/Matter/src/Matter.cpp

@@ -91,9 +91,8 @@ static void app_event_cb(const ChipDeviceEvent *event, intptr_t arg) {
   }
 }
 
-void ArduinoMatter::begin() {
+void ArduinoMatter::_init() {
   if (_matter_has_started) {
-    log_w("Matter has already started.");
     return;
   }
 
@@ -102,15 +101,15 @@ void ArduinoMatter::begin() {
   deviceNode = node::create(&node_config, app_attribute_update_cb, app_identification_cb);
   if (deviceNode == nullptr) {
     log_e("Failed to create Matter node");
-    abort();
+    return;
   }
 
   _matter_has_started = true;
 }
 
-void ArduinoMatter::start() {
+void ArduinoMatter::begin() {
   if (!_matter_has_started) {
-    log_w("No Matter Node has been created. Execute Matter.begin() first.");
+    log_w("No Matter endpoint has been created. Please create an endpoint first.");
     return;
   }
 

libraries/Matter/src/Matter.h

@@ -9,7 +9,6 @@ using namespace esp_matter;
 
 class ArduinoMatter {
   public:
-    static void begin();
     static inline String getManualPairingCode()  {
       // return the pairing code for manual pairing
       return String("34970112332");
@@ -18,7 +17,7 @@ class ArduinoMatter {
       // return the URL for the QR code for onboarding
       return String("https://project-chip.github.io/connectedhomeip/qrcode.html?data=MT:Y.K9042C00KA0648G00");
     }
-    static void start();
+    static void begin();
     static bool isDeviceCommissioned();
 #if CHIP_DEVICE_CONFIG_ENABLE_WIFI_STATION
     static bool isWiFiConnected();
@@ -28,6 +27,11 @@ class ArduinoMatter {
 #endif
     static bool isDeviceConnected();
     static void decommission();
+    
+    // list of Matter EndPoints Friend Classes
+    friend class MatterOnOffLight;
+  protected:
+    static void _init();
 };
 
 extern ArduinoMatter Matter;

libraries/Matter/src/MatterOnOffLight.cpp

@@ -40,7 +40,7 @@ MatterOnOffLight::~MatterOnOffLight() {
 }
 
 bool MatterOnOffLight::begin(bool initialState) {
-
+  ArduinoMatter::_init();
   on_off_light::config_t light_config;
   light_config.on_off.on_off = initialState;
   state = initialState;
@@ -50,7 +50,7 @@ bool MatterOnOffLight::begin(bool initialState) {
   endpoint_t *endpoint = on_off_light::create(node::get(), &light_config, ENDPOINT_FLAG_NONE, (void *) this);
   if (endpoint == nullptr) {
     log_e("Failed to create on-off light endpoint");
-    abort();
+    return false;
   }
 
   setEndPointId(endpoint::get_id(endpoint));