@homebridge-plugins/homebridge-wemo 7.7.1-beta.1 → 7.7.1

package/CHANGELOG.md

@@ -2,7 +2,7 @@
 
 All notable changes to `@homebridge-plugins/homebridge-wemo` will be documented in this file.
 
-## v7.7.1 (Pending Release)
+## v7.7.1 (2026-08-10)
 
 ### Changed
 
@@ -14,6 +14,9 @@ All notable changes to `@homebridge-plugins/homebridge-wemo` will be documented
 - 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
+- chore(deps): dependency updates
 
 ## 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
                     }
@@ -655,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
@@ -710,7 +727,7 @@ export default class {
           this.log.warn('[%s] %s: %s.', device, platformLang.connError, eText)
         }
       }
-    })
+    }))
 
     // ************************ \\
     // Erroneous Device Logging \\

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.1",
+  "version": "7.7.1",
   "description": "Homebridge plugin to integrate Wemo devices into HomeKit.",
   "author": {
     "name": "bwp91",
@@ -61,14 +61,14 @@
   },
   "dependencies": {
     "@homebridge/plugin-ui-utils": "^2.2.5",
-    "axios": "^1.18.1",
+    "axios": "^1.19.0",
     "node-ssdp": "^4.0.1",
     "p-queue": "^9.3.3",
     "xml2js": "^0.6.2",
     "xmlbuilder": "^15.1.1"
   },
   "devDependencies": {
-    "@antfu/eslint-config": "^9.2.0",
+    "@antfu/eslint-config": "^9.3.0",
     "@vitest/coverage-v8": "^4.1.10",
     "vitest": "^4.1.10"
   }