@homebridge-plugins/homebridge-govee 11.32.2-beta.3 → 11.32.2-beta.5

package/CHANGELOG.md

@@ -9,7 +9,15 @@ All notable changes to `@homebridge-plugins/homebridge-govee` will be documented
 - fix: implement the H7107's oscillation toggle, which threw a not-implemented error when tapped (#1334)
 - fix: also send the H7107's oscillation command as the multiSync frame the fan itself uses, since the first shape only turned it off sometimes (#1334)
 - fix: send the H7107's oscillation-off command without the swing-range bytes, matching the zero-tailed frame the fan echoes back (#1334)
-- feat: add support for the H8121 and H8122 ice makers, with ice size derived from the govee app (pending owner confirmation)
+- feat: add support for the H8121 and H8122 ice makers, with small, medium and large ice sizes (pending owner confirmation)
+- fix: send the H6102 the colour command its firmware expects, since the previous one was accepted but ignored (#1332)
+- fix: read the H5179 sensor's temperature from its own bytes, since it was picking up part of the humidity and reporting slightly wrong
+- feat: log govee sensors whose bluetooth broadcast is not recognised, so unsupported models can be identified
+- feat: report unrecognised device data in one line with the model, firmware and connection, so it can be sent in without follow-up questions
+- feat: work out for itself whether a heater reports the room temperature, and add the H7137 and H713E
+- feat: add support for 85 more light models, covering everything currently sold that was missing
+- feat: add 24 more light models, three thermo-hygrometers and a leak detector
+- feat: add the H7103 and H7112 fans and the H5085 plug, with unknown controls saying so instead of guessing
 
 ## v11.32.1 (2026-08-02)
 

package/config.schema.json

@@ -1260,7 +1260,7 @@
             "tempReporting": {
               "type": "boolean",
               "title": "Temperature Reporting",
-              "description": "Enable this if your H7130 device reports ambient temperature.",
+              "description": "Only needed to override the plugin. It works out on its own whether this heater reports the room temperature, and picks the right tile.",
               "condition": {
                 "functionBody": "return (model.heaterDevices && model.heaterDevices[arrayIndices] && model.heaterDevices[arrayIndices].deviceId && model.heaterDevices[arrayIndices].deviceId.length === 23);"
               }

package/lib/connection/ble.js

@@ -7,6 +7,7 @@ import { BLE_STAGE_TIMEOUTS, UUID } from '../utils/ble-protocol.js'
 import { decodeAny } from '../utils/decode.js'
 import { base64ToHex, generateCodeFromHexValues, hexToTwoItems, sleep } from '../utils/functions.js'
 import platformLang from '../utils/lang-en.js'
+import { isFirstReport } from '../utils/report-unknown.js'
 import { isValidPeripheral } from '../utils/validation.js'
 
 const HYPHEN_REGEX = /-/g
@@ -120,8 +121,32 @@ export default class BLEConnection {
     try {
       const { uuid, address, rssi, advertisement } = peripheral
 
-      // Skip if not a valid Govee sensor
+      // Skip if not a valid Govee sensor. Sensors whose broadcast the plugin
+      // cannot decode yet are logged rather than dropped in silence, since the
+      // hex is the one thing needed to add a model, and there is no other way
+      // for an owner to get hold of it.
       if (!isValidPeripheral(peripheral)) {
+        const unknownName = advertisement?.localName
+        if (unknownName && unknownName.toLowerCase().startsWith('govee')) {
+          const raw = advertisement.manufacturerData
+            ? advertisement.manufacturerData.toString('hex')
+            : 'no manufacturer data'
+          const line = `[BLE] unrecognised govee sensor - please include this line if you report it: ${JSON.stringify({
+            name: unknownName,
+            address,
+            source: 'BLE',
+            kind: 'broadcast',
+            raw,
+          })}`
+
+          // Say it once so it is actually noticed, then stay quiet about this
+          // sensor for the rest of the session - it rebroadcasts constantly.
+          if (isFirstReport(`ble|${address}|${raw}`)) {
+            this.log.warn(line)
+          } else {
+            this.log.debug(line)
+          }
+        }
         return
       }
 

package/lib/device/dehumidifier-H7150.js

@@ -5,6 +5,7 @@ import {
   parseError,
 } from '../utils/functions.js'
 import platformLang from '../utils/lang-en.js'
+import { logUnknownData } from '../utils/report-unknown.js'
 
 /*
   H7150
@@ -206,7 +207,12 @@ export default class {
           break
         }
         default:
-          this.accessory.logDebugWarn(`${platformLang.newScene}: [${command}] [${hexString}]`)
+          logUnknownData(this.accessory, {
+            kind: 'scene',
+            source: params.source,
+            raw: command,
+            hex: hexString,
+          })
           break
       }
     })

package/lib/device/dehumidifier-H7151.js

@@ -5,6 +5,7 @@ import {
   parseError,
 } from '../utils/functions.js'
 import platformLang from '../utils/lang-en.js'
+import { logUnknownData } from '../utils/report-unknown.js'
 
 export default class {
   constructor(platform, accessory) {
@@ -181,7 +182,12 @@ export default class {
           break
         }
         default:
-          this.accessory.logDebugWarn(`${platformLang.newScene}: [${command}] [${hexString}]`)
+          logUnknownData(this.accessory, {
+            kind: 'scene',
+            source: params.source,
+            raw: command,
+            hex: hexString,
+          })
           break
       }
     })

package/lib/device/dehumidifier-H7152.js

@@ -5,6 +5,7 @@ import {
   parseError,
 } from '../utils/functions.js'
 import platformLang from '../utils/lang-en.js'
+import { logUnknownData } from '../utils/report-unknown.js'
 
 export default class {
   constructor(platform, accessory) {
@@ -172,7 +173,12 @@ export default class {
           break
         }
         default:
-          this.accessory.logDebugWarn(`${platformLang.newScene}: [${command}] [${hexString}]`)
+          logUnknownData(this.accessory, {
+            kind: 'scene',
+            source: params.source,
+            raw: command,
+            hex: hexString,
+          })
           break
       }
     })

package/lib/device/diffuser-H7161.js

@@ -5,6 +5,7 @@ import {
   parseError,
 } from '../utils/functions.js'
 import platformLang from '../utils/lang-en.js'
+import { logUnknownData } from '../utils/report-unknown.js'
 
 export default class {
   constructor(platform, accessory) {
@@ -115,7 +116,12 @@ export default class {
 
       switch (deviceFunction) {
         default:
-          this.accessory.logDebugWarn(`${platformLang.newScene}: [${command}] [${hexString}]`)
+          logUnknownData(this.accessory, {
+            kind: 'scene',
+            source: params.source,
+            raw: command,
+            hex: hexString,
+          })
           break
       }
     })

package/lib/device/diffuser-H7162.js

@@ -5,6 +5,7 @@ import {
   parseError,
 } from '../utils/functions.js'
 import platformLang from '../utils/lang-en.js'
+import { logUnknownData } from '../utils/report-unknown.js'
 
 export default class {
   constructor(platform, accessory) {
@@ -115,7 +116,12 @@ export default class {
 
       switch (deviceFunction) {
         default:
-          this.accessory.logDebugWarn(`${platformLang.newScene}: [${command}] [${hexString}]`)
+          logUnknownData(this.accessory, {
+            kind: 'scene',
+            source: params.source,
+            raw: command,
+            hex: hexString,
+          })
           break
       }
     })

package/lib/device/fan-H1310.js

@@ -7,6 +7,7 @@ import {
   parseError,
 } from '../utils/functions.js'
 import platformLang from '../utils/lang-en.js'
+import { logUnknownData } from '../utils/report-unknown.js'
 
 export default class {
   constructor(platform, accessory) {
@@ -276,7 +277,12 @@ export default class {
         //   break
         // }
         default:
-          this.accessory.logDebugWarn(`${platformLang.newScene}: [${command}] [${hexString}]`)
+          logUnknownData(this.accessory, {
+            kind: 'scene',
+            source: params.source,
+            raw: command,
+            hex: hexString,
+          })
           break
       }
     })

package/lib/device/fan-H1370.js

@@ -10,6 +10,7 @@ import {
   sleep,
 } from '../utils/functions.js'
 import platformLang from '../utils/lang-en.js'
+import { logUnknownData } from '../utils/report-unknown.js'
 
 /*
   Codes below were decoded from a real device's AWS status reports in
@@ -365,7 +366,12 @@ export default class {
           break
         }
         default:
-          this.accessory.logDebugWarn(`${platformLang.newScene}: [${command}] [${hexString}]`)
+          logUnknownData(this.accessory, {
+            kind: 'scene',
+            source: params.source,
+            raw: command,
+            hex: hexString,
+          })
           break
       }
     })

package/lib/device/fan-H7100.js

@@ -7,6 +7,7 @@ import {
   parseError,
 } from '../utils/functions.js'
 import platformLang from '../utils/lang-en.js'
+import { logUnknownData } from '../utils/report-unknown.js'
 
 export default class {
   constructor(platform, accessory) {
@@ -276,7 +277,12 @@ export default class {
           break
         }
         default:
-          this.accessory.logDebugWarn(`${platformLang.newScene}: [${command}] [${hexString}]`)
+          logUnknownData(this.accessory, {
+            kind: 'scene',
+            source: params.source,
+            raw: command,
+            hex: hexString,
+          })
           break
       }
     })

package/lib/device/fan-H7101.js

@@ -7,6 +7,7 @@ import {
   parseError,
 } from '../utils/functions.js'
 import platformLang from '../utils/lang-en.js'
+import { logUnknownData } from '../utils/report-unknown.js'
 
 /*
   H7101
@@ -332,7 +333,12 @@ export default class {
           break
         }
         default:
-          this.accessory.logDebugWarn(`${platformLang.newScene}: [${command}] [${hexString}]`)
+          logUnknownData(this.accessory, {
+            kind: 'scene',
+            source: params.source,
+            raw: command,
+            hex: hexString,
+          })
           break
       }
     })

package/lib/device/fan-H7102.js

@@ -7,6 +7,7 @@ import {
   parseError,
 } from '../utils/functions.js'
 import platformLang from '../utils/lang-en.js'
+import { logUnknownData } from '../utils/report-unknown.js'
 
 export default class {
   constructor(platform, accessory) {
@@ -276,7 +277,12 @@ export default class {
           break
         }
         default:
-          this.accessory.logDebugWarn(`${platformLang.newScene}: [${command}] [${hexString}]`)
+          logUnknownData(this.accessory, {
+            kind: 'scene',
+            source: params.source,
+            raw: command,
+            hex: hexString,
+          })
           break
       }
     })

package/lib/device/fan-H7105.js

@@ -12,6 +12,7 @@ import {
   sleep,
 } from '../utils/functions.js'
 import platformLang from '../utils/lang-en.js'
+import { logUnknownData } from '../utils/report-unknown.js'
 
 export default class {
   constructor(platform, accessory) {
@@ -514,7 +515,12 @@ export default class {
           break
         }
         default:
-          this.accessory.logDebugWarn(`${platformLang.newScene}: [${command}] [${hexString}]`)
+          logUnknownData(this.accessory, {
+            kind: 'scene',
+            source: params.source,
+            raw: command,
+            hex: hexString,
+          })
           break
       }
     })

package/lib/device/fan-H7106.js

@@ -7,6 +7,7 @@ import {
   parseError,
 } from '../utils/functions.js'
 import platformLang from '../utils/lang-en.js'
+import { logUnknownData } from '../utils/report-unknown.js'
 
 export default class {
   constructor(platform, accessory) {
@@ -276,7 +277,12 @@ export default class {
           break
         }
         default:
-          this.accessory.logDebugWarn(`${platformLang.newScene}: [${command}] [${hexString}]`)
+          logUnknownData(this.accessory, {
+            kind: 'scene',
+            source: params.source,
+            raw: command,
+            hex: hexString,
+          })
           break
       }
     })

package/lib/device/fan-H7107.js

@@ -12,6 +12,7 @@ import {
   sleep,
 } from '../utils/functions.js'
 import platformLang from '../utils/lang-en.js'
+import { logUnknownData } from '../utils/report-unknown.js'
 
 export default class {
   constructor(platform, accessory) {
@@ -527,7 +528,12 @@ export default class {
           break
         }
         default:
-          this.accessory.logDebugWarn(`${platformLang.newScene}: [${command}] [${hexString}]`)
+          logUnknownData(this.accessory, {
+            kind: 'scene',
+            source: params.source,
+            raw: command,
+            hex: hexString,
+          })
           break
       }
     })

package/lib/device/fan-H7111.js

@@ -7,6 +7,7 @@ import {
   parseError,
 } from '../utils/functions.js'
 import platformLang from '../utils/lang-en.js'
+import { logUnknownData } from '../utils/report-unknown.js'
 
 /*
 
@@ -337,7 +338,12 @@ export default class {
           break
         }
         default:
-          this.accessory.logDebugWarn(`${platformLang.newScene}: [${command}] [${hexString}]`)
+          logUnknownData(this.accessory, {
+            kind: 'scene',
+            source: params.source,
+            raw: command,
+            hex: hexString,
+          })
           break
       }
     })

package/lib/device/fan-basic.js

@@ -0,0 +1,143 @@
+import { base64ToHex, getTwoItemPosition, hexToTwoItems, parseError } from '../utils/functions.js'
+import platformLang from '../utils/lang-en.js'
+import { logUnknownData, reportUnsupportedControl } from '../utils/report-unknown.js'
+
+/**
+ * A fan whose power command is known but whose speeds are not.
+ *
+ * Govee's power command is the same across every appliance the plugin handles,
+ * so on and off can be offered with confidence for a model nobody has tested.
+ * The speeds are the part that differs per model, and getting the number of
+ * them wrong is worse than not offering them - so the slider reports what it
+ * needs and fails the request rather than quietly sending a guess.
+ *
+ * Once an owner sends in what their fan reports, the model gets its own handler
+ * with real speeds and stops using this one.
+ */
+export default class {
+  constructor(platform, accessory) {
+    // Set up variables from the platform
+    this.hapChar = platform.api.hap.Characteristic
+    this.hapErr = platform.api.hap.HapStatusError
+    this.hapServ = platform.api.hap.Service
+    this.platform = platform
+
+    // Set up variables from the accessory
+    this.accessory = accessory
+
+    // Add the fan service if it doesn't already exist
+    this.service = this.accessory.getService(this.hapServ.Fanv2)
+      || this.accessory.addService(this.hapServ.Fanv2)
+
+    // Add the set handler to the fan active characteristic
+    this.service
+      .getCharacteristic(this.hapChar.Active)
+      .onSet(async value => this.internalStateUpdate(value))
+    this.cacheState = this.service.getCharacteristic(this.hapChar.Active).value ? 'on' : 'off'
+
+    // Offer the speed slider so the fan can report what it wants, but make it
+    // clear it does nothing yet rather than sending a guessed command
+    this.service
+      .getCharacteristic(this.hapChar.RotationSpeed)
+      .setProps({
+        maxValue: 100,
+        minStep: 1,
+        minValue: 0,
+        unit: 'unitless',
+      })
+      .onSet(async value => this.internalSpeedUpdate(value))
+    this.cacheSpeed = this.service.getCharacteristic(this.hapChar.RotationSpeed).value
+
+    // Output the customised options to the log
+    const opts = JSON.stringify({})
+    platform.log('[%s] %s %s.', accessory.displayName, platformLang.devInitOpts, opts)
+    this.accessory.log('on and off are supported for this model, the speeds are not known yet')
+  }
+
+  async internalStateUpdate(value) {
+    try {
+      const newValue = value ? 'on' : 'off'
+
+      // Don't continue if the new value is the same as before
+      if (this.cacheState === newValue) {
+        return
+      }
+
+      // Send the request to the platform sender function. This power command is
+      // the same for every govee appliance, so it is safe for an untested model
+      await this.platform.sendDeviceUpdate(this.accessory, {
+        cmd: 'ptReal',
+        value: value ? 'MwEBAAAAAAAAAAAAAAAAAAAAADM=' : 'MwEAAAAAAAAAAAAAAAAAAAAAADI=',
+        openApi: this.accessory.context.openApiCapabilities?.powerSwitch
+          ? { instance: 'powerSwitch', capabilityType: 'devices.capabilities.on_off', value: value ? 1 : 0 }
+          : undefined,
+      })
+
+      // Cache the new state and log if appropriate
+      this.cacheState = newValue
+      this.accessory.log(`${platformLang.curState} [${newValue}]`)
+    } catch (err) {
+      // Catch any errors during the process
+      this.accessory.logWarn(`${platformLang.devNotUpdated} ${parseError(err)}`)
+
+      // Throw a 'no response' error and set a timeout to revert this after 2 seconds
+      setTimeout(() => {
+        this.service.updateCharacteristic(this.hapChar.Active, this.cacheState === 'on' ? 1 : 0)
+      }, 2000)
+      throw new this.hapErr(-70402)
+    }
+  }
+
+  async internalSpeedUpdate(value) {
+    // 0 is the slider following the fan being turned off, not a speed request
+    if (value === 0) {
+      return
+    }
+
+    reportUnsupportedControl(this.accessory, { control: 'fan speed', value })
+
+    // Put the slider back where it was, and fail the request so HomeKit shows
+    // it as not responding - that is what sends the owner to the log
+    setTimeout(() => {
+      this.service.updateCharacteristic(this.hapChar.RotationSpeed, this.cacheSpeed)
+    }, 2000)
+    throw new this.hapErr(-70402)
+  }
+
+  externalUpdate(params) {
+    // Power is the one thing that can be read back with confidence
+    if (params.state && params.state !== this.cacheState) {
+      this.cacheState = params.state
+      this.service.updateCharacteristic(this.hapChar.Active, this.cacheState === 'on' ? 1 : 0)
+      this.accessory.log(`${platformLang.curState} [${this.cacheState}]`)
+    }
+
+    // Everything else is reported so the model can be finished off
+    ;(params.commands || []).forEach((command) => {
+      const hexString = base64ToHex(command)
+      const hexParts = hexToTwoItems(hexString)
+
+      if (getTwoItemPosition(hexParts, 1) !== 'aa') {
+        return
+      }
+
+      // The power report is understood, anything else is worth sending in
+      if (getTwoItemPosition(hexParts, 2) === '01') {
+        const newState = getTwoItemPosition(hexParts, 3) === '01' ? 'on' : 'off'
+        if (this.cacheState !== newState) {
+          this.cacheState = newState
+          this.service.updateCharacteristic(this.hapChar.Active, this.cacheState === 'on' ? 1 : 0)
+          this.accessory.log(`${platformLang.curState} [${this.cacheState}]`)
+        }
+        return
+      }
+
+      logUnknownData(this.accessory, {
+        kind: 'scene',
+        source: params.source,
+        raw: command,
+        hex: hexString,
+      })
+    })
+  }
+}

package/lib/device/heater1a.js

@@ -1,3 +1,4 @@
+import { getHeaterFeatures } from '../utils/appliance-capabilities.js'
 import {
   base64ToHex,
   farToCen,
@@ -8,6 +9,7 @@ import {
   parseError,
 } from '../utils/functions.js'
 import platformLang from '../utils/lang-en.js'
+import { logUnknownData } from '../utils/report-unknown.js'
 
 /*
   H7130 (without temperature reporting)
@@ -90,17 +92,29 @@ export default class {
       .onSet(async value => this.internalSpeedUpdate(value))
     this.cacheSpeed = this.service.getCharacteristic(this.hapChar.RotationSpeed).value
 
+    // Only offer the controls this model actually has, so a heater without a
+    // turning head does not get an oscillation switch that does nothing
+    this.features = getHeaterFeatures(this.accessory.context.gvModel)
+
     // Add the set handler to the heater swing mode characteristic (for oscillation)
-    this.service
-      .getCharacteristic(this.hapChar.SwingMode)
-      .onSet(async value => this.internalSwingUpdate(value))
-    this.cacheSwing = this.service.getCharacteristic(this.hapChar.SwingMode).value === 1 ? 'on' : 'off'
+    if (this.features.swing) {
+      this.service
+        .getCharacteristic(this.hapChar.SwingMode)
+        .onSet(async value => this.internalSwingUpdate(value))
+      this.cacheSwing = this.service.getCharacteristic(this.hapChar.SwingMode).value === 1 ? 'on' : 'off'
+    } else if (this.service.testCharacteristic(this.hapChar.SwingMode)) {
+      this.service.removeCharacteristic(this.service.getCharacteristic(this.hapChar.SwingMode))
+    }
 
-    // Add the set handler to the heater lock characteristic (for oscillation)
-    this.service
-      .getCharacteristic(this.hapChar.LockPhysicalControls)
-      .onSet(async value => this.internalLockUpdate(value))
-    this.cacheLock = this.service.getCharacteristic(this.hapChar.LockPhysicalControls).value === 1 ? 'on' : 'off'
+    // Add the set handler to the heater lock characteristic
+    if (this.features.lock) {
+      this.service
+        .getCharacteristic(this.hapChar.LockPhysicalControls)
+        .onSet(async value => this.internalLockUpdate(value))
+      this.cacheLock = this.service.getCharacteristic(this.hapChar.LockPhysicalControls).value === 1 ? 'on' : 'off'
+    } else if (this.service.testCharacteristic(this.hapChar.LockPhysicalControls)) {
+      this.service.removeCharacteristic(this.service.getCharacteristic(this.hapChar.LockPhysicalControls))
+    }
 
     // Output the customised options to the log
     const opts = JSON.stringify({
@@ -275,8 +289,15 @@ export default class {
     if (hasProperty(params, 'temperature')) {
       const newTemp = nearestHalf(farToCen(params.temperature / 100))
       if (newTemp <= 100) {
-        // Device must be one that DOES support ambient temperature
-        this.accessory.logWarn('you should enable `tempReporting` in the config for this device')
+        // A real reading, so this heater does report the room temperature and
+        // belongs on the other tile. Remember that for next time rather than
+        // asking the owner to work it out and tick a box themselves. The tile
+        // is not swapped mid-run, since that would rebuild the accessory
+        // underneath HomeKit while it is in use.
+        if (!this.accessory.context.supportsAmbientTemp) {
+          this.accessory.context.supportsAmbientTemp = true
+          this.accessory.logWarn('this heater reports the room temperature, so it will move to a temperature-capable tile the next time homebridge restarts')
+        }
       }
     }
 
@@ -330,6 +351,9 @@ export default class {
         case '1800':
         case '1801': {
           // Swing Mode
+          if (!this.features.swing) {
+            break
+          }
           const newSwing = getTwoItemPosition(hexParts, 3) === '01' ? 'on' : 'off'
           if (this.cacheSwing !== newSwing) {
             this.cacheSwing = newSwing
@@ -341,6 +365,9 @@ export default class {
         case '1000':
         case '1001': {
           // Child Lock
+          if (!this.features.lock) {
+            break
+          }
           const newLock = getTwoItemPosition(hexParts, 3) === '01' ? 'on' : 'off'
           if (this.cacheLock !== newLock) {
             this.cacheLock = newLock
@@ -388,7 +415,12 @@ export default class {
           break
         }
         default:
-          this.accessory.logDebugWarn(`${platformLang.newScene}: [${command}] [${hexString}]`)
+          logUnknownData(this.accessory, {
+            kind: 'scene',
+            source: params.source,
+            raw: command,
+            hex: hexString,
+          })
           break
       }
     })

package/lib/device/heater1b.js

@@ -8,6 +8,7 @@ import {
   parseError,
 } from '../utils/functions.js'
 import platformLang from '../utils/lang-en.js'
+import { logUnknownData } from '../utils/report-unknown.js'
 
 /*
   H7130 (with temperature reporting)
@@ -498,8 +499,13 @@ export default class {
       const newTemp = nearestHalf(farToCen(params.temperature / 100))
       if (newTemp !== this.cacheTemp) {
         if (newTemp > 100) {
-          // Device must be one that does not support ambient temperature
-          this.accessory.logWarn('you should disable `tempReporting` in the config for this device')
+          // Not a real reading, so this heater does not report the room
+          // temperature and belongs on the simpler tile. Remember that for next
+          // time rather than asking the owner to work it out themselves.
+          if (this.accessory.context.supportsAmbientTemp !== false) {
+            this.accessory.context.supportsAmbientTemp = false
+            this.accessory.logWarn('this heater does not report the room temperature, so it will move to a simpler tile the next time homebridge restarts')
+          }
         } else {
           this.cacheTemp = newTemp
           this.service.updateCharacteristic(this.hapChar.CurrentTemperature, this.cacheTemp)
@@ -664,7 +670,12 @@ export default class {
         case '1601': // DND on?
           break
         default:
-          this.accessory.logDebugWarn(`${platformLang.newScene}: [${command}] [${hexString}]`)
+          logUnknownData(this.accessory, {
+            kind: 'scene',
+            source: params.source,
+            raw: command,
+            hex: hexString,
+          })
           break
       }
     })