homebridge-tuya-without-developer-account 1.0.1 → 1.0.2

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 1.0.2
+
+- Fixed startup abort when Homebridge UI saves an empty or incomplete `deviceOverrides` row. Invalid override rows without `id` are now skipped with a warning instead of stopping QR cloud startup.
+- Duplicate device override IDs are now ignored safely, keeping the first valid entry.
+- Invalid or duplicate schema override entries are now skipped with warnings instead of blocking Homebridge startup.
+
 ## 1.0.1
 
 - Fixes repeated `code=-9999999, msg=sign invalid` errors caused by incomplete token expiry handling and non-persistent token refreshes.

package/README.md

@@ -192,6 +192,11 @@ Use `
 
 ## Troubleshooting
 
+### Plugin starts from cache only and logs `Each device override must include an "id"`
+
+Version 1.0.2 and newer no longer abort startup for empty override rows created by the Homebridge UI. Invalid override entries are skipped with a warning. If you still see old warnings, remove empty rows from the Device Overrides section in the plugin settings and restart Homebridge.
+
+
 ### The QR code does not appear
 
 Make sure you opened the settings for this plugin, not another Tuya plugin. The plugin name should be:

package/dist/platform.js

@@ -68,24 +68,45 @@ class TuyaPlatform {
     if (!this.options.deviceOverrides) {
       return true;
     }
-    const idMap = new Map();
+    if (!Array.isArray(this.options.deviceOverrides)) {
+      this.log.warn('[Tuya QR] Ignoring invalid deviceOverrides value because it is not an array.');
+      this.options.deviceOverrides = [];
+      return true;
+    }
+
+    const validOverrides = [];
+    const seenIds = new Set();
+    let skippedMissingId = 0;
+    let skippedDuplicateId = 0;
+
     for (const item of this.options.deviceOverrides) {
-      if (!item || !item.id) {
-        this.log.error('[Tuya QR] Each device override must include an "id".');
-        return false;
+      if (!item || typeof item !== 'object') {
+        skippedMissingId++;
+        continue;
       }
-      if (idMap.has(item.id)) {
-        idMap.get(item.id).push(item);
-      } else {
-        idMap.set(item.id, [item]);
+      const id = String(item.id || '').trim();
+      if (!id) {
+        skippedMissingId++;
+        continue;
       }
-    }
-    for (const items of idMap.values()) {
-      if (items.length > 1) {
-        this.log.error('[Tuya QR] "deviceOverrides" conflict, "id" must be unique: %o.', items);
-        return false;
+      if (seenIds.has(id)) {
+        skippedDuplicateId++;
+        this.log.warn('[Tuya QR] Ignoring duplicate device override for id "%s". Keeping the first one.', id);
+        continue;
       }
+      item.id = id;
+      seenIds.add(id);
+      validOverrides.push(item);
+    }
+
+    if (skippedMissingId > 0) {
+      this.log.warn('[Tuya QR] Ignored %d invalid device override(s) without an "id". QR cloud startup will continue.', skippedMissingId);
+    }
+    if (skippedDuplicateId > 0) {
+      this.log.warn('[Tuya QR] Ignored %d duplicate device override(s). QR cloud startup will continue.', skippedDuplicateId);
     }
+
+    this.options.deviceOverrides = validOverrides;
     return true;
   }
 
@@ -97,20 +118,43 @@ class TuyaPlatform {
       if (!deviceOverride.schema) {
         continue;
       }
-      const idMap = new Map();
+      if (!Array.isArray(deviceOverride.schema)) {
+        this.log.warn('[Tuya QR] Ignoring invalid schema override for device id "%s" because schema is not an array.', deviceOverride.id);
+        deviceOverride.schema = undefined;
+        continue;
+      }
+      const validSchema = [];
+      const seenCodes = new Set();
+      let skippedMissingCode = 0;
+      let skippedDuplicateCode = 0;
+
       for (const item of deviceOverride.schema) {
-        if (idMap.has(item.code)) {
-          idMap.get(item.code).push(item);
-        } else {
-          idMap.set(item.code, [item]);
+        if (!item || typeof item !== 'object') {
+          skippedMissingCode++;
+          continue;
         }
-      }
-      for (const items of idMap.values()) {
-        if (items.length > 1) {
-          this.log.error('[Tuya QR] "schema" conflict, "code" must be unique: %o.', items);
-          return false;
+        const code = String(item.code || '').trim();
+        if (!code) {
+          skippedMissingCode++;
+          continue;
+        }
+        if (seenCodes.has(code)) {
+          skippedDuplicateCode++;
+          this.log.warn('[Tuya QR] Ignoring duplicate schema override code "%s" for device id "%s". Keeping the first one.', code, deviceOverride.id);
+          continue;
         }
+        item.code = code;
+        seenCodes.add(code);
+        validSchema.push(item);
+      }
+
+      if (skippedMissingCode > 0) {
+        this.log.warn('[Tuya QR] Ignored %d invalid schema override(s) without a "code" for device id "%s".', skippedMissingCode, deviceOverride.id);
+      }
+      if (skippedDuplicateCode > 0) {
+        this.log.warn('[Tuya QR] Ignored %d duplicate schema override(s) for device id "%s".', skippedDuplicateCode, deviceOverride.id);
       }
+      deviceOverride.schema = validSchema;
     }
     return true;
   }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "homebridge-tuya-without-developer-account",
   "displayName": "Tuya without developer account for Homebridge",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "description": "Homebridge plugin for Tuya and Smart Life devices using QR cloud authentication without a Tuya IoT developer account.",
   "license": "MIT",
   "author": "Kosztyk",