Merge pull request #393 from athombv/master

Merge master into production (#minor)
athombv · May 7, 2024 · c43a3d9 · c43a3d9
2 parents 4ec598f + 1eabe59
commit c43a3d9
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/assets/app/schema.json b/assets/app/schema.json
@@ -1132,7 +1132,8 @@
         "enum": [
           "nfc",
           "speaker",
-          "ledring"
+          "ledring",
+          "matter"
         ]
       }
     },

diff --git a/lib/App/index.js b/lib/App/index.js
@@ -401,6 +401,17 @@ class App {
           throw new Error(`drivers.${driver.id} Matter drivers require 'connectivity' to include 'matter'.`);
         }
       }
+
+      const allDriversMatter = appJson.drivers.every(driver => driver.connectivity && driver.connectivity.includes('matter'));
+      const hasPlatformRequiredFeatureMatter = appJson.platformLocalRequiredFeatures && appJson.platformLocalRequiredFeatures.includes('matter');
+
+      if (allDriversMatter && !hasPlatformRequiredFeatureMatter) {
+        console.warn('Warning: all drivers have connectivity: matter, but matter is not set as a platform required feature.');
+      }
+
+      if (!allDriversMatter && hasPlatformRequiredFeatureMatter) {
+        console.warn('Warning: matter is set as a platform required feature, but not all drivers have connectivity: matter.');
+      }
     }
 
     // validate `appJson.capabilities`