@homebridge-plugins/homebridge-govee 11.32.2 → 11.33.1

package/CHANGELOG.md

@@ -2,7 +2,16 @@
 
 All notable changes to `@homebridge-plugins/homebridge-govee` will be documented in this file.
 
-## v11.32.2 (2026-08-03)
+## v11.33.1 (2026-08-04)
+
+### Changed
+
+- fix: stop the kettle reporting its startup status as unrecognised scenes (#1337)
+- fix: stop reporting a bluetooth broadcast that carries no data to read (#1338)
+- fix: report an unrecognised code once per kind, not once per value it carries (#1338)
+- chore(deps): dependency updates
+
+## v11.33.0 (2026-08-03)
 
 ### Changed
 

package/lib/connection/ble.js

@@ -129,10 +129,12 @@ export default class BLEConnection {
       // model and an owner has no other way 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'
+        // A device sends more than one kind of advertisement, and the ones
+        // carrying only its name have no manufacturer data at all. There is
+        // nothing in those to decode and nothing to be learnt from reporting
+        // them, so a working sensor was warning about itself (#1338)
+        if (unknownName && unknownName.toLowerCase().startsWith('govee') && advertisement.manufacturerData) {
+          const raw = advertisement.manufacturerData.toString('hex')
 
           // A device rebroadcasts constantly, so say this once and then stay
           // quiet about it for the rest of the session. This has to come first:

package/lib/device/kettle.js

@@ -29,6 +29,9 @@ export default class extends GoveeDevice {
     // Set up variables from the accessory
     const deviceConf = platform.deviceConf[accessory.context.gvDeviceId] || {}
 
+    // What each preset was last reported as, so a repeat is not logged twice
+    this.reportedPresets = new Map()
+
     this.codes = {
       greenTea: 'MwUAAgAAAAAAAAAAAAAAAAAAADQ=',
       oolongTea: 'MwUAAwAAAAAAAAAAAAAAAAAAADU=',
@@ -251,6 +254,27 @@ export default class extends GoveeDevice {
 
       const deviceFunction = `${getTwoItemPosition(hexParts, 2)}${getTwoItemPosition(hexParts, 3)}`
 
+      // On startup the kettle lists the temperature each of its presets is set
+      // to, one frame per preset: aa 05 <slot> <degrees F>. That is a different
+      // thing from aa 05 00 <mode> below, which is a mode change - the slot
+      // byte is what tells them apart. Nothing here needs the values, so they
+      // are logged for reference rather than reported as unrecognised (#1337)
+      if (getTwoItemPosition(hexParts, 2) === '05' && getTwoItemPosition(hexParts, 3) !== '00') {
+        const slot = Number.parseInt(getTwoItemPosition(hexParts, 3), 16)
+        const tempInF = Number.parseInt(getTwoItemPosition(hexParts, 4), 16)
+        const line = `preset ${slot} is set to ${tempInF}°F [${Math.round(((tempInF - 32) * 5) / 9)}°C]`
+        // Worth seeing in a normal log, but only once each. The kettle sends
+        // these when it connects, and if a firmware ever repeats them on a
+        // timer this keeps five lines from becoming five every few minutes
+        if (this.reportedPresets.get(slot) === line) {
+          this.accessory.logDebug(line)
+        } else {
+          this.reportedPresets.set(slot, line)
+          this.accessory.log(line)
+        }
+        return
+      }
+
       switch (deviceFunction) {
         case '0500': { // current mode
           // switch to green tea_: aa050002000000000000000000000000000000ad
@@ -287,9 +311,19 @@ export default class extends GoveeDevice {
         case '2301': { // scheduled start on
           break
         }
+        // Also sent on startup, alongside the preset list above. What each one
+        // carries has not been worked out, and nothing here needs them, so they
+        // are known-and-ignored rather than reported every time (#1337)
+        case '1701': // a second form of the on-base report
+        case '1f06':
+        case '1f08':
+        case '1900': {
+          this.accessory.logDebug(`startup status [${deviceFunction}]: ${hexString}`)
+          break
+        }
         default:
           logUnknownData(this.accessory, {
-            kind: 'scene',
+            kind: 'status',
             source: params.source,
             raw: command,
             hex: hexString,

package/lib/utils/report-unknown.js

@@ -14,10 +14,33 @@ import { base64ToHex } from './functions.js'
  */
 
 // Remembers what has already been reported, so a code the device repeats every
-// few seconds is only shouted about once. Keyed by device, kind and payload, so
-// a genuinely different unknown value is still surfaced.
+// few seconds is only shouted about once. Keyed by device, kind and the shape
+// of the frame rather than its whole payload - see frameShape below.
 const alreadyReported = new Set()
 
+/**
+ * What kind of message a frame is, ignoring the values it carries.
+ *
+ * A frame is `<prefix> <command> <sub-command> <data...>`, and the first three
+ * bytes are what the device handlers themselves switch on to decide what a
+ * frame means. Keying on those means one report per kind of message.
+ *
+ * Keying on the whole payload instead looks equivalent but is not: a frame
+ * carrying a reading or a counter differs every time it is sent, so each one
+ * counts as new and gets reported again. One fan sent the same message every
+ * twenty minutes with a changing value, which worked out at around seventy
+ * warnings a day for a single message type, burying the frames worth seeing
+ * (#1338).
+ *
+ * @param {string} [hex] the frame as hex, where there is one
+ * @returns {string|undefined} the first three bytes, or undefined if not a frame
+ */
+function frameShape(hex) {
+  return typeof hex === 'string' && /^[0-9a-f]{6}/i.test(hex)
+    ? hex.slice(0, 6).toLowerCase()
+    : undefined
+}
+
 /**
  * Reset the record of what has been reported. Only needed by tests, and when
  * the platform restarts and wants a fresh session.
@@ -142,10 +165,14 @@ export function logUnknownData(accessory, {
   })
 
   const line = `unrecognised ${kind} - please include this line if you report it: ${JSON.stringify(report)}`
-  const fingerprint = `${context.gvDeviceId || accessory.displayName}|${kind}|${report.raw}`
-
-  // Say it once loudly enough to be noticed, then keep quiet about that exact
-  // payload for the rest of the session.
+  // Only the callers that pass `hex` are handing over a real device frame. The
+  // rest report things like a settings object, and base64ToHex will happily
+  // turn any string into hex that looks like one - two different messages can
+  // then share a first three bytes and wrongly count as the same shape.
+  const fingerprint = `${context.gvDeviceId || accessory.displayName}|${kind}|${frameShape(hex) ?? report.raw}`
+
+  // Say it once loudly enough to be noticed, then keep quiet about that shape
+  // for the rest of the session.
   if (alreadyReported.has(fingerprint)) {
     accessory.logDebug?.(line)
     return

package/package.json

@@ -2,7 +2,7 @@
   "name": "@homebridge-plugins/homebridge-govee",
   "alias": "Govee",
   "type": "module",
-  "version": "11.32.2",
+  "version": "11.33.1",
   "description": "Homebridge plugin to integrate Govee devices into HomeKit.",
   "author": {
     "name": "bwp91",