@homebridge-plugins/homebridge-meross 10.19.1-beta.0 → 10.19.1-beta.2

package/CHANGELOG.md

@@ -16,6 +16,11 @@ All notable changes to `@homebridge-plugins/homebridge-meross` will be documente
 - fix: stop the polling timers on shutdown for a local-only setup, not just a cloud one
 - fix: stop the garage's quick polling when the door is operated again mid-close
 - fix: apply the smoke sensor's battery reading, instead of failing on every update
+- fix: reject a garage opening time so large it would make the reset timer fire immediately
+- test: record what every device handler builds, so a refactor can be shown to change nothing
+- fix(ms600): stop writing every incoming message to the warning log
+- fix(ms600): accept a light level of zero, so a dark room is reported as dark
+- test: check the device handlers against payloads real devices sent
 
 ## v10.19.0 (2026-08-01)
 

package/config.schema.json

@@ -1486,6 +1486,7 @@
             "type": "integer",
             "placeholder": "20",
             "minimum": 1,
+            "maximum": 2073600,
             "condition": {
               "functionBody": "return (model.garageDevices && model.garageDevices[arrayIndices] && model.garageDevices[arrayIndices].serialNumber && model.garageDevices[arrayIndices].serialNumber.length === 32 && !model.garageDevices[arrayIndices].ignoreDevice);"
             }

package/lib/device/sensor-presence.js

@@ -165,7 +165,11 @@ export default class {
   receiveUpdate(params) {
     try {
       // Log the received data
-      this.accessory.logWarn(`${platformLang.incMQTT}: ${JSON.stringify(params)}`)
+      // At debug, like every other handler. This sensor pushes a message every
+      // few seconds and the whole message is logged, including the header's
+      // `sign` - a value derived from the owner's device key - so at warn level
+      // it filled the log with traffic nobody asked to see.
+      this.accessory.logDebug(`${platformLang.incMQTT}: ${JSON.stringify(params)}`)
       if (params.payload?.latest?.[0]?.data) {
         this.applyUpdate(params.payload.latest[0].data)
       }
@@ -175,7 +179,10 @@ export default class {
   }
 
   applyUpdate(data) {
-    if (data.light?.[0]?.value) {
+    // Compared against undefined rather than checked for truth: zero lux is a
+    // real reading from a dark room, and skipping it left the sensor showing
+    // whatever the light level was when the lights went out.
+    if (data.light?.[0]?.value !== undefined) {
       // Check against the cache and update HomeKit and the cache if needed
       const newLux = data.light[0].value
       if (this.cacheLux !== newLux) {

package/lib/platform.js

@@ -90,6 +90,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)
     }
@@ -162,6 +165,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
                     }

package/lib/utils/constants.js

@@ -50,6 +50,14 @@ export default {
     refreshRate: 5,
   },
 
+  // A Node timer holds its delay in a signed 32-bit integer. Over 2147483647 ms
+  // it does not throw - it quietly becomes 1 ms. refreshRate and cloudRefreshRate
+  // are already capped at 600 further down, but the garage opening time was not,
+  // so a large value made the door's reset timer fire immediately and repeatedly.
+  maxValues: {
+    garageDoorOpeningTime: 2073600,
+  },
+
   allowed: {
     singleDevices: [
       'name',

package/lib/utils/lang-en.js

@@ -13,6 +13,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',
   cfgMoveEntry: 'configuration entry should be moved to appropriate section in the UI',
   cfgRmv: 'is unused and can be removed',

package/package.json

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