@homebridge-plugins/homebridge-ewelink 12.18.2-beta.2 → 12.18.2-beta.3

package/CHANGELOG.md

@@ -17,6 +17,13 @@ All notable changes to `@homebridge-plugins/homebridge-ewelink` will be document
 - fix: make the per-device logging override settable, and drop the debug option that never was
 - fix: move the two-door garage one door at a time, so the second is not cut short
 - fix: stop the lan discovery printing straight to the console, whatever the log level
+- fix: trigger rf sensor channels one at a time, so the reset delay is actually waited on
+- fix: reject an operation or sensor time so large it would make the reset timer fire immediately
+- fix(zigbee): stop forcing debug logs on zigbee motors, which ignored the logging setting
+- test: record what every device handler builds, so a refactor can be shown to change nothing
+- fix: honour the per-device logging setting on groups and the zigbee water valve
+- fix(zigbee): read the snzb-02ld temperature as degrees, so 18.4c stops showing as 0.18c
+- test: check the device handlers against payloads real devices sent
 
 ## v12.18.1 (2026-08-02)
 

package/config.schema.json

@@ -421,6 +421,7 @@
               "description": "Garage doors: the time to open/close the device. Locks: the time to show as unlocked. This setting is in deciseconds - count the time in seconds and multiply by 10, for example 75 for 7.5 seconds. Must be 20 or more.",
               "placeholder": 100,
               "minimum": 20,
+              "maximum": 2073600,
               "condition": {
                 "functionBody": "return (model.singleDevices && model.singleDevices[arrayIndices] && ((model.singleDevices[arrayIndices].deviceModel === 'gddc5' && model.singleDevices[arrayIndices].showAsEachen !== 'lock') || ['garage', 'lock', 'gate'].includes(model.singleDevices[arrayIndices].showAs)) && !model.singleDevices[arrayIndices].ignoreDevice);"
               }
@@ -431,6 +432,7 @@
               "description": "If your garage door has a different operation time for up and down, use this setting to specify the operation time DOWN (in deciseconds) and the other setting to specify the operation time UP. Otherwise leave blank. Must be 20 or more.",
               "placeholder": 100,
               "minimum": 20,
+              "maximum": 2073600,
               "condition": {
                 "functionBody": "return (model.singleDevices && model.singleDevices[arrayIndices] && ['garage'].includes(model.singleDevices[arrayIndices].showAs) && !model.singleDevices[arrayIndices].ignoreDevice);"
               }
@@ -760,6 +762,7 @@
               "description": "Blinds/garage doors: the time to open/close the device. Locks: the time to show as unlocked. This setting is in deciseconds - count the time in seconds and multiply by 10, for example 75 for 7.5 seconds. Must be 20 or more.",
               "placeholder": 100,
               "minimum": 20,
+              "maximum": 2073600,
               "condition": {
                 "functionBody": "return (model.multiDevices && model.multiDevices[arrayIndices] && ['garage', 'garage_two', 'garage_four', 'blind', 'door', 'window', 'lock', 'gate'].includes(model.multiDevices[arrayIndices].showAs) && !model.multiDevices[arrayIndices].ignoreDevice);"
               }
@@ -770,6 +773,7 @@
               "description": "If your garage door or blind has a different operation time for up and down, use this setting to specify the operation time DOWN (in deciseconds) and the other setting to specify the operation time UP. Otherwise leave blank. Must be 20 or more.",
               "placeholder": 100,
               "minimum": 20,
+              "maximum": 2073600,
               "condition": {
                 "functionBody": "return (model.multiDevices && model.multiDevices[arrayIndices] && ['garage', 'garage_two', 'garage_four', 'blind', 'door', 'window'].includes(model.multiDevices[arrayIndices].showAs) && !model.multiDevices[arrayIndices].ignoreDevice);"
               }
@@ -1383,6 +1387,7 @@
               "description": "An offset in seconds to ignore notifications if the delay between this Zigbee Motion/Leak activating or Zigbee Button press and the plugin receiving the notification is greater than this number. Must be 5 or more.",
               "placeholder": 60,
               "minimum": 5,
+              "maximum": 2073600,
               "condition": {
                 "functionBody": "return (model.sensorDevices && model.sensorDevices[arrayIndices] && ['snzb01', 'snzb03', 'leak'].includes(model.sensorDevices[arrayIndices].deviceModel) && !model.sensorDevices[arrayIndices].ignoreDevice);"
               }
@@ -1522,6 +1527,7 @@
               "description": "The time to open/close the device. This setting is in deciseconds - count the time in seconds and multiply by 10, for example 75 for 7.5 seconds. Must be 20 or more.",
               "placeholder": 100,
               "minimum": 20,
+              "maximum": 2073600,
               "condition": {
                 "functionBody": "return (model.bridgeSensors && model.bridgeSensors[arrayIndices] && model.bridgeSensors[arrayIndices].fullDeviceId && model.bridgeSensors[arrayIndices].fullDeviceId.length >= 13 && model.bridgeSensors[arrayIndices].deviceType === 'curtain' && ['blind', 'door', 'window'].includes(model.bridgeSensors[arrayIndices].curtainType));"
               }
@@ -1532,6 +1538,7 @@
               "description": "If your device has a different operation time for up and down, use this setting to specify the operation time DOWN (in deciseconds) and the other setting to specify the operation time UP. Otherwise leave blank. Must be 20 or more.",
               "placeholder": 100,
               "minimum": 20,
+              "maximum": 2073600,
               "condition": {
                 "functionBody": "return (model.bridgeSensors && model.bridgeSensors[arrayIndices] && model.bridgeSensors[arrayIndices].fullDeviceId && model.bridgeSensors[arrayIndices].fullDeviceId.length >= 13 && model.bridgeSensors[arrayIndices].deviceType === 'curtain' && ['blind', 'door', 'window'].includes(model.bridgeSensors[arrayIndices].curtainType));"
               }
@@ -1588,6 +1595,7 @@
               "description": "The number of seconds which the sensor tile in the Home app will light up for if this sensor is activated. Must be 1 or more.",
               "placeholder": 60,
               "minimum": 1,
+              "maximum": 2073600,
               "condition": {
                 "functionBody": "return (model.bridgeSensors && model.bridgeSensors[arrayIndices] && model.bridgeSensors[arrayIndices].type && model.bridgeSensors[arrayIndices].deviceType === 'sensor' && ['motion', 'smoke', 'water', 'co', 'co2', 'occupancy', 'contact'].includes(model.bridgeSensors[arrayIndices].type));"
               }
@@ -1598,6 +1606,7 @@
               "description": "An offset in seconds to ignore notifications if the delay between this sensor activating and the plugin receiving the notification is greater than this number. Must be 5 or more.",
               "placeholder": 60,
               "minimum": 5,
+              "maximum": 2073600,
               "condition": {
                 "functionBody": "return (model.bridgeSensors && model.bridgeSensors[arrayIndices] && model.bridgeSensors[arrayIndices].type && model.bridgeSensors[arrayIndices].deviceType === 'sensor' && ['motion', 'smoke', 'water', 'co', 'co2', 'occupancy', 'contact'].includes(model.bridgeSensors[arrayIndices].type));"
               }

package/lib/device/group.js

@@ -15,9 +15,28 @@ export default class {
     this.name = accessory.displayName
     this.accessory = accessory
 
+    // Set up custom variables for this device type
+    const deviceConf = platform.deviceConf[accessory.context.eweDeviceId] || {}
+
     // Set the correct logging variables for this accessory
-    this.enableLogging = !platform.config.disableDeviceLogging
-    this.enableDebugLogging = false
+    switch (deviceConf.overrideLogging) {
+      case 'standard':
+        this.enableLogging = true
+        this.enableDebugLogging = false
+        break
+      case 'debug':
+        this.enableLogging = true
+        this.enableDebugLogging = true
+        break
+      case 'disable':
+        this.enableLogging = false
+        this.enableDebugLogging = false
+        break
+      default:
+        this.enableLogging = !platform.config.disableDeviceLogging
+        this.enableDebugLogging = false
+        break
+    }
 
     if (platformConsts.devices.switchSCM.includes(this.accessory.context.groupUIID)) {
       this.isSCM = true

package/lib/device/rf-bridge.js

@@ -116,202 +116,204 @@ export default class {
         || params.updateSource === 'LAN'
       ) {
         // RF Sensor
-        Object.keys(params)
-          .filter(name => rfTrigRegex.test(name))
-          .forEach(async (chan) => {
-            try {
-              this.devicesInHB.forEach((acc) => {
-                if (
-                  acc.context.eweDeviceId === this.accessory.context.eweDeviceId
-                  && hasProperty(acc.context, 'buttons')
-                  && hasProperty(acc.context.buttons, chan.split('g')[1].toString())
-                ) {
-                  subAccessory = acc
-                  channel = chan.split('g')[1].toString()
-                }
-              })
-              if (subAccessory) {
-                // Avoid duplicate triggers from LAN and WS
-                if (params[chan] === subAccessory.context.cacheLastAct) {
-                  return
+        // One channel at a time. This was a `forEach(async ...)`, so every
+        // triggered channel ran at once and the promise each returned was thrown
+        // away - the `sleep` calls below held nothing back, and a rejection
+        // escaped the try in externalUpdate rather than being caught by it.
+        for (const chan of Object.keys(params).filter(name => rfTrigRegex.test(name))) {
+          try {
+            this.devicesInHB.forEach((acc) => {
+              if (
+                acc.context.eweDeviceId === this.accessory.context.eweDeviceId
+                && hasProperty(acc.context, 'buttons')
+                && hasProperty(acc.context.buttons, chan.split('g')[1].toString())
+              ) {
+                subAccessory = acc
+                channel = chan.split('g')[1].toString()
+              }
+            })
+            if (subAccessory) {
+              // Avoid duplicate triggers from LAN and WS
+              if (params[chan] === subAccessory.context.cacheLastAct) {
+                continue
+              }
+              const deviceConf = this.platform.rfSubdevices[subAccessory.context.hbDeviceId] || {}
+              let enableLogging
+              switch (deviceConf.overrideLogging) {
+                case 'standard':
+                case 'debug':
+                  enableLogging = true
+                  break
+                case 'disable':
+                  enableLogging = false
+                  break
+                default:
+                  enableLogging = !this.disableDeviceLogging
+                  break
+              }
+              switch (subAccessory.context.subType) {
+                case 'blind':
+                case 'door':
+                case 'window':
+                  // Simulations don't support external updates
+                  continue
+                case 'button':
+                case 'curtain': {
+                  const service = subAccessory.context.buttons[channel]
+                  subAccessory.getService(service).updateCharacteristic(this.hapChar.On, true)
+                  this.log(
+                    '[%s] %s [%s].',
+                    subAccessory.displayName,
+                    this.lang.curState,
+                    this.lang.buttonTrig,
+                  )
+                  await sleep(3000)
+                  subAccessory.getService(service).updateCharacteristic(this.hapChar.On, false)
+                  continue
                 }
-                const deviceConf = this.platform.rfSubdevices[subAccessory.context.hbDeviceId] || {}
-                let enableLogging
-                switch (deviceConf.overrideLogging) {
-                  case 'standard':
-                  case 'debug':
-                    enableLogging = true
+                default:
+                  break
+              }
+              const sensorTimeLength = deviceConf.sensorTimeLength || platformConsts.defaultValues.sensorTimeLength
+              const sensorTimeDifference = deviceConf.sensorTimeDifference || platformConsts.defaultValues.sensorTimeDifference
+              subAccessory.context.cacheLastAct = params[chan]
+              const timeOfMotion = new Date(params[chan])
+              const diff = (timeNow.getTime() - timeOfMotion.getTime()) / 1000
+              let serv
+              let char
+              if (diff < sensorTimeDifference) {
+                switch (deviceConf.type) {
+                  case 'water':
+                    serv = this.hapServ.LeakSensor
+                    char = this.hapChar.LeakDetected
                     break
-                  case 'disable':
-                    enableLogging = false
+                  case 'fire':
+                  case 'smoke':
+                    serv = this.hapServ.SmokeSensor
+                    char = this.hapChar.SmokeDetected
                     break
-                  default:
-                    enableLogging = !this.disableDeviceLogging
+                  case 'co':
+                    serv = this.hapServ.CarbonMonoxideSensor
+                    char = this.hapChar.CarbonMonoxideDetected
+                    break
+                  case 'co2':
+                    serv = this.hapServ.CarbonDioxideSensor
+                    char = this.hapChar.CarbonDioxideDetected
+                    break
+                  case 'contact': {
+                    serv = this.hapServ.ContactSensor
+                    char = this.hapChar.ContactSensorState
+                    const initialTime = subAccessory.eveService.getInitialTime()
+                    subAccessory
+                      .getService(serv)
+                      .updateCharacteristic(
+                        this.eveChar.LastActivation,
+                        Math.round(new Date().valueOf() / 1000) - initialTime,
+                      )
                     break
-                }
-                switch (subAccessory.context.subType) {
-                  case 'blind':
-                  case 'door':
-                  case 'window':
-                    // Simulations don't support external updates
-                    return
-                  case 'button':
-                  case 'curtain': {
-                    const service = subAccessory.context.buttons[channel]
-                    subAccessory.getService(service).updateCharacteristic(this.hapChar.On, true)
-                    this.log(
-                      '[%s] %s [%s].',
-                      subAccessory.displayName,
-                      this.lang.curState,
-                      this.lang.buttonTrig,
-                    )
-                    await sleep(3000)
-                    subAccessory.getService(service).updateCharacteristic(this.hapChar.On, false)
-                    return
                   }
-                  default:
+                  case 'occupancy':
+                    serv = this.hapServ.OccupancySensor
+                    char = this.hapChar.OccupancyDetected
+                    break
+                  case 'p_button':
+                    serv = this.hapServ.StatelessProgrammableSwitch
+                    char = this.hapChar.ProgrammableSwitchEvent
+                    break
+                  case 'doorbell':
+                    serv = this.hapServ.Doorbell
+                    char = this.hapChar.ProgrammableSwitchEvent
+                    break
+                  default: {
+                    serv = this.hapServ.MotionSensor
+                    char = this.hapChar.MotionDetected
+                    const initialTime = subAccessory.eveService.getInitialTime()
+                    subAccessory
+                      .getService(serv)
+                      .updateCharacteristic(
+                        this.eveChar.LastActivation,
+                        Math.round(new Date().valueOf() / 1000) - initialTime,
+                      )
                     break
-                }
-                const sensorTimeLength = deviceConf.sensorTimeLength || platformConsts.defaultValues.sensorTimeLength
-                const sensorTimeDifference = deviceConf.sensorTimeDifference || platformConsts.defaultValues.sensorTimeDifference
-                subAccessory.context.cacheLastAct = params[chan]
-                const timeOfMotion = new Date(params[chan])
-                const diff = (timeNow.getTime() - timeOfMotion.getTime()) / 1000
-                let serv
-                let char
-                if (diff < sensorTimeDifference) {
-                  switch (deviceConf.type) {
-                    case 'water':
-                      serv = this.hapServ.LeakSensor
-                      char = this.hapChar.LeakDetected
-                      break
-                    case 'fire':
-                    case 'smoke':
-                      serv = this.hapServ.SmokeSensor
-                      char = this.hapChar.SmokeDetected
-                      break
-                    case 'co':
-                      serv = this.hapServ.CarbonMonoxideSensor
-                      char = this.hapChar.CarbonMonoxideDetected
-                      break
-                    case 'co2':
-                      serv = this.hapServ.CarbonDioxideSensor
-                      char = this.hapChar.CarbonDioxideDetected
-                      break
-                    case 'contact': {
-                      serv = this.hapServ.ContactSensor
-                      char = this.hapChar.ContactSensorState
-                      const initialTime = subAccessory.eveService.getInitialTime()
-                      subAccessory
-                        .getService(serv)
-                        .updateCharacteristic(
-                          this.eveChar.LastActivation,
-                          Math.round(new Date().valueOf() / 1000) - initialTime,
-                        )
-                      break
-                    }
-                    case 'occupancy':
-                      serv = this.hapServ.OccupancySensor
-                      char = this.hapChar.OccupancyDetected
-                      break
-                    case 'p_button':
-                      serv = this.hapServ.StatelessProgrammableSwitch
-                      char = this.hapChar.ProgrammableSwitchEvent
-                      break
-                    case 'doorbell':
-                      serv = this.hapServ.Doorbell
-                      char = this.hapChar.ProgrammableSwitchEvent
-                      break
-                    default: {
-                      serv = this.hapServ.MotionSensor
-                      char = this.hapChar.MotionDetected
-                      const initialTime = subAccessory.eveService.getInitialTime()
-                      subAccessory
-                        .getService(serv)
-                        .updateCharacteristic(
-                          this.eveChar.LastActivation,
-                          Math.round(new Date().valueOf() / 1000) - initialTime,
-                        )
-                      break
-                    }
                   }
+                }
 
-                  // Call any webhook defined in the config
-                  if (deviceConf.sensorWebHook) {
-                    try {
-                      await axios.get(deviceConf.sensorWebHook, {
-                        timeout: 5000,
-                      })
-                      if (enableLogging) {
-                        this.log(
-                          '[%s] %s [GET %s].',
-                          subAccessory.displayName,
-                          'http webhook request successful',
-                          deviceConf.sensorWebHook,
-                        )
-                      }
-                    } catch (e) {
-                      if (enableLogging) {
-                        this.log(
-                          '[%s] %s %s.',
-                          subAccessory.displayName,
-                          'http webhook request failed as',
-                          e.message,
-                        )
-                      }
+                // Call any webhook defined in the config
+                if (deviceConf.sensorWebHook) {
+                  try {
+                    await axios.get(deviceConf.sensorWebHook, {
+                      timeout: 5000,
+                    })
+                    if (enableLogging) {
+                      this.log(
+                        '[%s] %s [GET %s].',
+                        subAccessory.displayName,
+                        'http webhook request successful',
+                        deviceConf.sensorWebHook,
+                      )
                     }
-                  }
-
-                  // Programmable button and doorbell activate differently
-                  if (char === this.hapChar.ProgrammableSwitchEvent) {
-                    subAccessory.getService(serv).updateCharacteristic(char, 0)
-                    if (params.updateSource && enableLogging) {
+                  } catch (e) {
+                    if (enableLogging) {
                       this.log(
-                        '[%s] %s [%s].',
+                        '[%s] %s %s.',
                         subAccessory.displayName,
-                        this.lang.curState,
-                        this.lang.buttonSingle,
+                        'http webhook request failed as',
+                        e.message,
                       )
                     }
-                    return
-                  }
-                  subAccessory.getService(serv).updateCharacteristic(char, 1)
-                  subAccessory.eveService.addEntry({ status: 1 })
-                  if (params.updateSource && enableLogging) {
-                    this.log(
-                      '[%s] %s [%s].',
-                      subAccessory.displayName,
-                      this.lang.curState,
-                      this.lang.rfTrigYes,
-                    )
                   }
+                }
 
-                  const updateKey = Math.random()
-                    .toString(36)
-                    .substring(2, 10)
-                  subAccessory.context.updateKey = updateKey
-                  await sleep(sensorTimeLength * 1000)
-                  if (updateKey !== subAccessory.context.updateKey) {
-                    return
-                  }
+                // Programmable button and doorbell activate differently
+                if (char === this.hapChar.ProgrammableSwitchEvent) {
                   subAccessory.getService(serv).updateCharacteristic(char, 0)
-                  subAccessory.eveService.addEntry({ status: 0 })
                   if (params.updateSource && enableLogging) {
                     this.log(
                       '[%s] %s [%s].',
                       subAccessory.displayName,
                       this.lang.curState,
-                      this.lang.rfTrigNo,
+                      this.lang.buttonSingle,
                     )
                   }
+                  continue
+                }
+                subAccessory.getService(serv).updateCharacteristic(char, 1)
+                subAccessory.eveService.addEntry({ status: 1 })
+                if (params.updateSource && enableLogging) {
+                  this.log(
+                    '[%s] %s [%s].',
+                    subAccessory.displayName,
+                    this.lang.curState,
+                    this.lang.rfTrigYes,
+                  )
+                }
+
+                const updateKey = Math.random()
+                  .toString(36)
+                  .substring(2, 10)
+                subAccessory.context.updateKey = updateKey
+                await sleep(sensorTimeLength * 1000)
+                if (updateKey !== subAccessory.context.updateKey) {
+                  continue
+                }
+                subAccessory.getService(serv).updateCharacteristic(char, 0)
+                subAccessory.eveService.addEntry({ status: 0 })
+                if (params.updateSource && enableLogging) {
+                  this.log(
+                    '[%s] %s [%s].',
+                    subAccessory.displayName,
+                    this.lang.curState,
+                    this.lang.rfTrigNo,
+                  )
                 }
-              } else {
-                throw new Error(`${this.lang.rfNotFound}[${chan}]`)
               }
-            } catch (err) {
-              this.log.warn('[%s] %s %s.', this.name, this.lang.devNotRf, parseError(err))
+            } else {
+              throw new Error(`${this.lang.rfNotFound}[${chan}]`)
             }
-          })
+          } catch (err) {
+            this.log.warn('[%s] %s %s.', this.name, this.lang.devNotRf, parseError(err))
+          }
+        }
       }
     } catch (err) {
       this.platform.deviceUpdateError(this.accessory, err, false)

package/lib/device/zigbee/motor.js

@@ -42,10 +42,6 @@ export default class {
         break
     }
 
-    // Temp override for debugging
-    this.enableLogging = true
-    this.enableDebugLogging = true
-
     // Add the battery service if it doesn't already exist
     this.battService = this.accessory.getService(this.hapServ.Battery)
       || this.accessory.addService(this.hapServ.Battery)

package/lib/device/zigbee/sensor-ambient.js

@@ -98,11 +98,19 @@ export default class {
         }
       }
       if (hasProperty(params, 'temperature')) {
-        const rawTemp = Number.parseInt(params.temperature, 10)
+        // Two conventions arrive here. Most of these sensors send hundredths of
+        // a degree as a whole number - 2620 for 26.20C. The SNZB-02LD and 02WD
+        // send the degrees themselves, with a decimal point ("18.4"), and
+        // reading that as hundredths showed 18.4C in HomeKit as 0.18C (#701).
+        // The decimal point is the only thing that tells them apart.
+        const sendsDegrees = String(params.temperature).includes('.')
+        const rawTemp = sendsDegrees
+          ? Number.parseFloat(params.temperature)
+          : Number.parseInt(params.temperature, 10)
         if (Number.isNaN(rawTemp) || rawTemp < -27000 || rawTemp > 27000) {
           return
         }
-        let newTemp = rawTemp / 100
+        let newTemp = sendsDegrees ? rawTemp : rawTemp / 100
         if (this.tempOffsetFactor) {
           newTemp *= this.tempOffset
         } else {
@@ -121,11 +129,15 @@ export default class {
         }
       }
       if (hasProperty(params, 'humidity')) {
-        const rawHumi = Number.parseInt(params.humidity, 10)
+        // Same two conventions as the temperature above, handled the same way
+        const sendsPercent = String(params.humidity).includes('.')
+        const rawHumi = sendsPercent
+          ? Number.parseFloat(params.humidity)
+          : Number.parseInt(params.humidity, 10)
         if (rawHumi === 0) {
           return
         }
-        let newHumi = rawHumi / 100
+        let newHumi = sendsPercent ? rawHumi : rawHumi / 100
         if (this.humiOffsetFactor) {
           newHumi *= this.humiOffset
         } else {

package/lib/device/zigbee/zigbee-water-valve.js

@@ -12,7 +12,27 @@ export default class {
     this.accessory = accessory
 
     // Set up custom variables for this device type
-    this.enableLogging = !platform.config.disableDeviceLogging
+    const deviceConf = platform.deviceConf[accessory.context.eweDeviceId] || {}
+
+    // Set the correct logging variables for this accessory
+    switch (deviceConf.overrideLogging) {
+      case 'standard':
+        this.enableLogging = true
+        this.enableDebugLogging = false
+        break
+      case 'debug':
+        this.enableLogging = true
+        this.enableDebugLogging = true
+        break
+      case 'disable':
+        this.enableLogging = false
+        this.enableDebugLogging = false
+        break
+      default:
+        this.enableLogging = !platform.config.disableDeviceLogging
+        this.enableDebugLogging = false
+        break
+    }
 
     // The newer SWV-ZNE irrigation valve (uiid 7047) reports and accepts its
     // state under 'switch_00' rather than 'switch' (#778)

package/lib/platform.js

@@ -113,6 +113,9 @@ export default class {
     const logIncrease = (k, min) => {
       this.log.warn('%s [%s] %s %s.', this.lang.cfgItem, k, this.lang.cfgLow, min)
     }
+    const logDecrease = (k, max) => {
+      this.log.warn('%s [%s] %s %s.', this.lang.cfgItem, k, this.lang.cfgHigh, max)
+    }
     const logQuotes = (k) => {
       this.log.warn('%s [%s] %s.', this.lang.cfgItem, k, this.lang.cfgQts)
     }
@@ -200,6 +203,9 @@ export default class {
                     } else if (intVal < platformConsts.minValues[k]) {
                       logIncrease(`${key}.${id}.${k}`, platformConsts.minValues[k])
                       this.rfSubdevices[id][k] = platformConsts.minValues[k]
+                    } else if (platformConsts.maxValues[k] !== undefined && intVal > platformConsts.maxValues[k]) {
+                      logDecrease(`${key}.${id}.${k}`, platformConsts.maxValues[k])
+                      this.rfSubdevices[id][k] = platformConsts.maxValues[k]
                     } else {
                       this.rfSubdevices[id][k] = intVal
                     }
@@ -282,6 +288,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

@@ -61,6 +61,18 @@ export default {
     targetTempThreshold: 0,
   },
 
+  // 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 a device set past that point
+  // would run its reset timer immediately and repeatedly. operationTime is
+  // multiplied by 100 and sensorTimeLength by 1000, so both are capped well
+  // inside the limit.
+  maxValues: {
+    operationTime: 2073600,
+    operationTimeDown: 2073600,
+    sensorTimeDifference: 2073600,
+    sensorTimeLength: 2073600,
+  },
+
   appId: 'Uw83EKZFxdif7XFXEsrpduz5YyjP7nTl',
 
   appSecret: 'mXLOjea0woSMvK9gw7Fjsy7YlFO4iSu6',

package/lib/utils/lang-en.js

@@ -14,6 +14,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

@@ -3,7 +3,7 @@
   "displayName": "Homebridge eWeLink",
   "alias": "eWeLink",
   "type": "module",
-  "version": "12.18.2-beta.2",
+  "version": "12.18.2-beta.3",
   "description": "Homebridge plugin to integrate eWeLink devices into HomeKit.",
   "author": {
     "name": "bwp91",