@homebridge-plugins/homebridge-govee 11.33.0 → 11.33.2-beta.0

package/CHANGELOG.md

@@ -2,6 +2,21 @@
 
 All notable changes to `@homebridge-plugins/homebridge-govee` will be documented in this file.
 
+## v11.33.2 (Pending Release)
+
+### Changed
+
+- fix: implement the H7105 oscillation toggle, which threw a not-implemented error (#1339)
+
+## 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/fan-H7105.js

@@ -187,33 +187,42 @@ export default class extends GoveeDevice {
 
   async internalSwingUpdate(value) {
     try {
-      // Don't continue if the new value is the same as before
-      if (this.cacheSwing === value) {
+      const newValue = value ? 'on' : 'off'
+      // Don't continue if the new value is the same as before. The comparison
+      // has to be against the mapped string - cacheSwing holds 'on'/'off' and
+      // value arrives as a boolean, so comparing them directly never matched
+      if (this.cacheSwing === newValue) {
         return
       }
 
-      throw new Error('Swing mode update not implemented yet')
-
-      // const newValue = value ? 'on' : 'off'
-      // The existing cacheSwingCode might be something like aa1d0101960384000000000000000000000000a6
-      // We need to change the third hex value to 00 for off or 01 for on
-      // const hexValues = [
-      //   0x3A,
-      //   0x1D,
-      //   value ? 0x01 : 0x00,
-      //   ...this.cacheSwingCode.slice(6, 14).match(/.{1,2}/g).map(byte => Number.parseInt(byte, 16)),
-      // ]
-      //
-      // await this.platform.sendDeviceUpdate(this.accessory, {
-      //   cmd: 'multiSync',
-      //   value: generateCodeFromHexValues(hexValues),
-      // })
+      // Same oscillation shape as the H7107, which is the only other fan that
+      // reports a swing RANGE rather than a plain on/off. Turning ON carries
+      // over the range bytes from the last report seen; turning OFF must not,
+      // because a parameterised off command was answered with a zero-tailed
+      // frame while the fan kept oscillating (#1334, #1339)
+      const tailBytes = value && this.cacheSwingCode
+        ? this.cacheSwingCode.slice(6, 14).match(/.{1,2}/g).map(byte => Number.parseInt(byte, 16))
+        : []
+      await this.platform.sendDeviceUpdate(this.accessory, {
+        cmd: 'ptReal',
+        value: generateCodeFromHexValues([0x33, 0x1D, value ? 0x01 : 0x00, ...tailBytes]),
+        openApi: this.accessory.context.openApiCapabilities?.oscillationToggle
+          ? { instance: 'oscillationToggle', capabilityType: 'devices.capabilities.toggle', value: value ? 1 : 0 }
+          : undefined,
+      })
+
+      // The fan's own oscillation state travels as multiSync traffic, so send
+      // that shape too. Whichever the model ignores is harmless
+      await this.platform.sendDeviceUpdate(this.accessory, {
+        cmd: 'multiSync',
+        value: generateCodeFromHexValues([0x3A, 0x1D, value ? 0x01 : 0x00, ...tailBytes]),
+      })
 
       // Cache the new state and log if appropriate
-      // if (this.cacheSwing !== newValue) {
-      //   this.cacheSwing = newValue
-      //   this.accessory.log(`${platformLang.curSwing} [${newValue}]`)
-      // }
+      if (this.cacheSwing !== newValue) {
+        this.cacheSwing = newValue
+        this.accessory.log(`${platformLang.curSwing} [${newValue}]`)
+      }
     } catch (err) {
       this.failUpdate(err, () => {
         this.service.updateCharacteristic(this.hapChar.SwingMode, this.cacheSwing === 'on' ? 1 : 0)

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.33.0",
+  "version": "11.33.2-beta.0",
   "description": "Homebridge plugin to integrate Govee devices into HomeKit.",
   "author": {
     "name": "bwp91",