@homebridge-plugins/homebridge-wemo 7.7.1-beta.0 → 7.7.1-beta.2

package/CHANGELOG.md

@@ -12,6 +12,10 @@ All notable changes to `@homebridge-plugins/homebridge-wemo` will be documented
 - fix: send a colour change when only the saturation moves, and mend the al cleanup
 - fix: explain a port clash on the notification listener instead of crashing the bridge
 - fix: stop gating a few log lines on a debug setting that was never readable
+- fix: keep an accessory when a device answers without its service list
+- fix: cancel a garage door's settle timer when homebridge shuts down
+- fix: wait for the manually configured devices before reporting which ones failed to connect
+- fix: reject an interval so large it would make the plugin poll every millisecond
 
 ## v7.7.0 (2026-08-02)
 

package/config.schema.json

@@ -42,7 +42,8 @@
         "title": "Discovery Interval",
         "description": "An interval (in seconds) in which the plugin will broadcast on your network to initally find devices and repair connection errors. Must be 15 or more.",
         "placeholder": 30,
-        "minimum": 15
+        "minimum": 15,
+        "maximum": 2073600
       },
       "wemoClient": {
         "title": "Discovery Settings",
@@ -87,14 +88,16 @@
         "title": "HTTP Polling Interval",
         "description": "An interval (in seconds) in which the plugin will request status updates for devices that don't support UPnP or have it manually disabled. Must be 15 or more.",
         "placeholder": 30,
-        "minimum": 15
+        "minimum": 15,
+        "maximum": 2073600
       },
       "upnpInterval": {
         "type": "integer",
         "title": "UPnP Subscription Interval",
         "description": "An interval (in seconds) in which the plugin will resubscribe to UPnP notifications. Must be 60 or more.",
         "placeholder": 300,
-        "minimum": 60
+        "minimum": 60,
+        "maximum": 2073600
       },
       "disableUPNP": {
         "type": "boolean",
@@ -162,6 +165,7 @@
               "description": "The time in seconds for how long it takes the garage door to open/close. Must be 1 or more.",
               "placeholder": 20,
               "minimum": 1,
+              "maximum": 2073600,
               "condition": {
                 "functionBody": "return (model.makerTypes && model.makerTypes[arrayIndices] && model.makerTypes[arrayIndices].makerType === 'garageDoor' && !model.makerTypes[arrayIndices].ignoreDevice);"
               }
@@ -252,6 +256,7 @@
               "description": "The plugin will not log device wattage if the time difference (in seconds) from the previous value is less than this number. Must be 1 or more.",
               "placeholder": 1,
               "minimum": 1,
+              "maximum": 2073600,
               "condition": {
                 "functionBody": "return (model.wemoInsights && model.wemoInsights[arrayIndices] && model.wemoInsights[arrayIndices].serialNumber && model.wemoInsights[arrayIndices].serialNumber.length > 0 && !model.wemoInsights[arrayIndices].ignoreDevice);"
               }
@@ -492,6 +497,7 @@
               "description": "A timer (in seconds) for how long after motion is not detected that the state is changed.",
               "placeholder": 60,
               "minimum": 0,
+              "maximum": 2073600,
               "condition": {
                 "functionBody": "return (model.wemoMotions && model.wemoMotions[arrayIndices] && model.wemoMotions[arrayIndices].serialNumber && model.wemoMotions[arrayIndices].serialNumber.length > 0 && !model.wemoMotions[arrayIndices].ignoreDevice);"
               }

package/lib/platform.js

@@ -100,6 +100,9 @@ export default class {
     const logIncrease = (k, min) => {
       this.log.warn('%s [%s] %s %s.', platformLang.cfgItem, k, platformLang.cfgLow, min)
     }
+    const logDecrease = (k, max) => {
+      this.log.warn('%s [%s] %s %s.', platformLang.cfgItem, k, platformLang.cfgHigh, max)
+    }
     const logQuotes = (k) => {
       this.log.warn('%s [%s] %s.', platformLang.cfgItem, k, platformLang.cfgQts)
     }
@@ -133,6 +136,9 @@ export default class {
           } else if (intVal < platformConsts.minValues[key]) {
             logIncrease(key, platformConsts.minValues[key])
             this.config[key] = platformConsts.minValues[key]
+          } else if (platformConsts.maxValues[key] !== undefined && intVal > platformConsts.maxValues[key]) {
+            logDecrease(key, platformConsts.maxValues[key])
+            this.config[key] = platformConsts.maxValues[key]
           } else {
             this.config[key] = intVal
           }
@@ -187,6 +193,9 @@ export default class {
                     } else if (intVal < platformConsts.minValues[k]) {
                       logIncrease(`${key}.${id}.${k}`, platformConsts.minValues[k])
                       this.deviceConf[id][k] = platformConsts.minValues[k]
+                    } else if (platformConsts.maxValues[k] !== undefined && intVal > platformConsts.maxValues[k]) {
+                      logDecrease(`${key}.${id}.${k}`, platformConsts.maxValues[k])
+                      this.deviceConf[id][k] = platformConsts.maxValues[k]
                     } else {
                       this.deviceConf[id][k] = intVal
                     }
@@ -531,6 +540,13 @@ export default class {
         if (accessory.control?.pollingInterval) {
           clearInterval(accessory.control.pollingInterval)
         }
+        // A garage door part-way through opening or closing has a timer waiting to
+        // write its settled state. That was left running, so it fired against a
+        // torn-down accessory and held the process open until it did.
+        if (accessory.control?.movingTimer) {
+          clearTimeout(accessory.control.movingTimer)
+          accessory.control.movingTimer = false
+        }
         if (accessory.client) {
           accessory.client.stopSubscriptions()
         }
@@ -648,7 +664,15 @@ export default class {
     // *************************** \\
     // Manually Configured Devices \\
     // *************************** \\
-    this.manualDevices.forEach(async (device) => {
+    // Awaited, unlike the `forEach(async ...)` this used to be, which threw the
+    // promises away. The "still awaiting" warning further down then ran before
+    // these probes had finished, so a manually configured device that was about
+    // to connect could be reported as one that had not.
+    // Still concurrent rather than a for..of loop, because each entry is an
+    // independent port scan and running them one after another would drag out
+    // discovery. Each callback keeps its own try/catch, so one failure cannot
+    // reject the whole batch.
+    await Promise.all(this.manualDevices.map(async (device) => {
       try {
         // Check to see if the entry is a full address or an IP
         let deviceIP
@@ -703,7 +727,7 @@ export default class {
           this.log.warn('[%s] %s: %s.', device, platformLang.connError, eText)
         }
       }
-    })
+    }))
 
     // ************************ \\
     // Erroneous Device Logging \\
@@ -842,10 +866,15 @@ export default class {
           }
         })
       } else {
-        // Device has no services so is useless to Homebridge
-        if (devicesInHB.has(uuid)) {
-          this.removeAccessory(devicesInHB.get(uuid))
-        }
+        // Device has no services so cannot be set up on this run.
+        //
+        // The cached accessory used to be unregistered here. A setup document
+        // that arrives truncated, or from a device part-way through a reboot,
+        // parses fine but has no serviceList - and one of those was enough to
+        // take the accessory out of HomeKit along with its room, its scenes and
+        // any automations using it. A later restart brings it back as a brand new
+        // accessory, so none of that comes back with it. Leave it alone: it shows
+        // as No Response until the device answers properly again.
         throw new Error(platformLang.noServices)
       }
 

package/lib/utils/constants.js

@@ -44,6 +44,20 @@ export default {
     wattDiff: 1,
   },
 
+  // A Node timer holds its delay in a signed 32-bit integer. Over 2147483647 ms
+  // it does not throw - it quietly becomes 1 ms, so any of these set past that
+  // point would fire a thousand times a second rather than slowly. 2073600
+  // seconds is 24 days, comfortably inside the limit once multiplied up.
+  maxValues: {
+    discoveryInterval: 2073600,
+    doorOpenTimer: 2073600,
+    makerTimer: 2073600,
+    noMotionTimer: 2073600,
+    pollingInterval: 2073600,
+    timeDiff: 2073600,
+    upnpInterval: 2073600,
+  },
+
   minValues: {
     adaptiveLightingShift: -1,
     discoveryInterval: 15,

package/lib/utils/lang-en.js

@@ -12,6 +12,7 @@ export default {
   cfgIgn: 'is not configured correctly so ignoring',
   cfgIgnItem: 'has an invalid entry which will be ignored',
   cfgItem: 'Config entry',
+  cfgHigh: 'is set too high so reducing to',
   cfgLow: 'is set too low so increasing to',
   cfgRmv: 'is unused and can be removed',
   cfgQts: 'should not have quotes around its entry',

package/package.json

@@ -2,7 +2,7 @@
   "name": "@homebridge-plugins/homebridge-wemo",
   "alias": "Wemo",
   "type": "module",
-  "version": "7.7.1-beta.0",
+  "version": "7.7.1-beta.2",
   "description": "Homebridge plugin to integrate Wemo devices into HomeKit.",
   "author": {
     "name": "bwp91",