@homebridge-plugins/homebridge-govee 11.2.0 → 11.2.1-beta.1

package/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 All notable changes to homebridge-govee will be documented in this file.
 
+## v11.2.1 (Unreleased)
+
+### Other Changes
+
+- fix permission in release workflow
+
 ## v11.2.0 (2025-07-12)
 
 ### Notable Changes

package/config.schema.json

@@ -1153,6 +1153,14 @@
               "condition": {
                 "functionBody": "return (model.fanDevices && model.fanDevices[arrayIndices] && model.fanDevices[arrayIndices].deviceId && model.fanDevices[arrayIndices].deviceId.length === 23);"
               }
+            },
+            "hideLight": {
+              "type": "boolean",
+              "title": "Hide Light",
+              "description": "Enable this to not expose the light service in Homebridge/Homekit if your fan has one.",
+              "condition": {
+                "functionBody": "return (model.fanDevices && model.fanDevices[arrayIndices] && model.fanDevices[arrayIndices].deviceId && model.fanDevices[arrayIndices].deviceId.length === 23 && !model.fanDevices[arrayIndices].ignoreDevice);"
+              }
             }
           }
         }
@@ -1604,7 +1612,8 @@
           "items": [
             "fanDevices[].label",
             "fanDevices[].deviceId",
-            "fanDevices[].ignoreDevice"
+            "fanDevices[].ignoreDevice",
+            "fanDevices[].hideLight"
           ]
         }
       ]

package/lib/device/fan-H7105.js

@@ -22,20 +22,24 @@ export default class {
     // Set up variables from the accessory
     this.accessory = accessory
 
+    // Set up custom variables for this device type
+    const deviceConf = platform.deviceConf[accessory.context.gvDeviceId]
+    this.hideLight = deviceConf && deviceConf.hideLight
+
     // Codes etc
     this.speedCodes = {
-      7: 'MwUBAQAAAAAAAAAAAAAAAAAAADY=',
-      14: 'MwUBAgAAAAAAAAAAAAAAAAAAADU=',
-      21: 'MwUBAwAAAAAAAAAAAAAAAAAAADQ=',
-      28: 'MwUBBAAAAAAAAAAAAAAAAAAAADM=',
-      35: 'MwUBBQAAAAAAAAAAAAAAAAAAADI=',
-      42: 'MwUBBgAAAAAAAAAAAAAAAAAAADE=',
-      49: 'MwUBBwAAAAAAAAAAAAAAAAAAADA=',
-      56: 'MwUBCAAAAAAAAAAAAAAAAAAAAD8=',
-      63: 'MwUBCQAAAAAAAAAAAAAAAAAAAD4=',
-      70: 'MwUBCgAAAAAAAAAAAAAAAAAAAD0=', // guessed
-      77: 'MwUBCwAAAAAAAAAAAAAAAAAAADw=', // guessed
-      84: 'MwUBDAAAAAAAAAAAAAAAAAAAADs=', // guessed
+      1: 'MwUBAQAAAAAAAAAAAAAAAAAAADY=',
+      2: 'MwUBAgAAAAAAAAAAAAAAAAAAADU=',
+      3: 'MwUBAwAAAAAAAAAAAAAAAAAAADQ=',
+      4: 'MwUBBAAAAAAAAAAAAAAAAAAAADM=',
+      5: 'MwUBBQAAAAAAAAAAAAAAAAAAADI=',
+      6: 'MwUBBgAAAAAAAAAAAAAAAAAAADE=',
+      7: 'MwUBBwAAAAAAAAAAAAAAAAAAADA=',
+      8: 'MwUBCAAAAAAAAAAAAAAAAAAAAD8=',
+      9: 'MwUBCQAAAAAAAAAAAAAAAAAAAD4=',
+      10: 'MwUBCgAAAAAAAAAAAAAAAAAAAD0=',
+      11: 'MwUBCwAAAAAAAAAAAAAAAAAAADw=',
+      12: 'MwUBDAAAAAAAAAAAAAAAAAAAADs=',
     }
 
     // Remove any old original Fan services
@@ -46,9 +50,6 @@ export default class {
     // Add the fan service for the fan if it doesn't already exist
     this.service = this.accessory.getService(this.hapServ.Fanv2) || this.accessory.addService(this.hapServ.Fanv2)
 
-    // Add the night light service if it doesn't already exist
-    this.lightService = this.accessory.getService(this.hapServ.Lightbulb) || this.accessory.addService(this.hapServ.Lightbulb)
-
     // Add the set handler to the fan on/off characteristic
     this.service
       .getCharacteristic(this.hapChar.Active)
@@ -59,43 +60,56 @@ export default class {
     this.service
       .getCharacteristic(this.hapChar.RotationSpeed)
       .setProps({
-        minStep: 7,
+        maxValue: 12,
+        minStep: 1,
         minValue: 0,
-        validValues: [0, 7, 14, 21, 28, 35, 42, 49, 56, 63, 70, 77, 84, 91],
+        unit: 'unitless', // This is actually from HAP for Bluetooth LE Specification, but fits
       })
       .onSet(async value => this.internalSpeedUpdate(value))
     this.cacheSpeed = this.service.getCharacteristic(this.hapChar.RotationSpeed).value
-    this.cacheMode = this.cacheSpeed >= 91 ? 'auto' : 'manual'
 
     // Add the set handler to the fan swing mode
     this.service
       .getCharacteristic(this.hapChar.SwingMode)
       .onSet(async value => this.internalSwingUpdate(value))
     this.cacheSwing = this.service.getCharacteristic(this.hapChar.SwingMode).value === 1 ? 'on' : 'off'
+    this.cacheSwingCode = ''
 
-    // Add the set handler to the lightbulb on/off characteristic
-    this.lightService.getCharacteristic(this.hapChar.On).onSet(async (value) => {
-      await this.internalLightStateUpdate(value)
-    })
-    this.cacheLightState = this.lightService.getCharacteristic(this.hapChar.On).value ? 'on' : 'off'
+    if (this.hideLight) {
+      if (this.accessory.getService(this.hapServ.Lightbulb)) {
+        // Remove the light service if it exists
+        this.accessory.removeService(this.accessory.getService(this.hapServ.Lightbulb))
+      }
+    } else {
+      // Add the night light service if it doesn't already exist
+      this.lightService = this.accessory.getService(this.hapServ.Lightbulb) || this.accessory.addService(this.hapServ.Lightbulb)
 
-    // Add the set handler to the lightbulb brightness characteristic
-    this.lightService
-      .getCharacteristic(this.hapChar.Brightness)
-      .onSet(async (value) => {
-        await this.internalBrightnessUpdate(value)
+      // Add the set handler to the lightbulb on/off characteristic
+      this.lightService.getCharacteristic(this.hapChar.On).onSet(async (value) => {
+        await this.internalLightStateUpdate(value)
       })
-    this.cacheBright = this.lightService.getCharacteristic(this.hapChar.Brightness).value
-
-    // Add the set handler to the lightbulb hue characteristic
-    this.lightService.getCharacteristic(this.hapChar.Hue).onSet(async (value) => {
-      await this.internalColourUpdate(value)
-    })
-    this.cacheHue = this.lightService.getCharacteristic(this.hapChar.Hue).value
-    this.cacheSat = this.lightService.getCharacteristic(this.hapChar.Saturation).value
+      this.cacheLightState = this.lightService.getCharacteristic(this.hapChar.On).value ? 'on' : 'off'
+
+      // Add the set handler to the lightbulb brightness characteristic
+      this.lightService
+        .getCharacteristic(this.hapChar.Brightness)
+        .onSet(async (value) => {
+          await this.internalBrightnessUpdate(value)
+        })
+      this.cacheBright = this.lightService.getCharacteristic(this.hapChar.Brightness).value
+
+      // Add the set handler to the lightbulb hue characteristic
+      this.lightService.getCharacteristic(this.hapChar.Hue).onSet(async (value) => {
+        await this.internalColourUpdate(value)
+      })
+      this.cacheHue = this.lightService.getCharacteristic(this.hapChar.Hue).value
+      this.cacheSat = this.lightService.getCharacteristic(this.hapChar.Saturation).value
+    }
 
     // Output the customised options to the log
-    const opts = JSON.stringify({})
+    const opts = JSON.stringify({
+      hideLight: this.hideLight,
+    })
     platform.log('[%s] %s %s.', accessory.displayName, platformLang.devInitOpts, opts)
   }
 
@@ -133,74 +147,20 @@ export default class {
 
   async internalSpeedUpdate(value) {
     try {
-      if (value < 3) {
-        return
-      }
-
-      let newValue
-      if (value < 10) {
-        newValue = 7
-      } else if (value < 17) {
-        newValue = 14
-      } else if (value < 24) {
-        newValue = 21
-      } else if (value < 31) {
-        newValue = 28
-      } else if (value < 38) {
-        newValue = 35
-      } else if (value < 45) {
-        newValue = 42
-      } else if (value < 52) {
-        newValue = 49
-      } else if (value < 59) {
-        newValue = 56
-      } else if (value < 66) {
-        newValue = 63
-      } else if (value < 73) {
-        newValue = 70
-      } else if (value < 80) {
-        newValue = 77
-      } else if (value < 87) {
-        newValue = 84
-      } else {
-        newValue = 91
-      }
-
-      let newMode = value === 91 ? 'auto' : 'manual'
-
       // Don't continue if the new value is the same as before
-      if (this.cacheSpeed === newValue) {
+      if (this.cacheSpeed === value || value === 0) {
         return
       }
 
-      // Don't continue if trying to access auto mode but there is no sensor attached
-      let codeToSend
-      if (newMode === 'auto') {
-        if (!this.accessory.context.sensorAttached || !this.cacheAutoCode) {
-          this.accessory.logWarn('auto mode not supported without a linked sensor')
-          codeToSend = this.speedCodes[84]
-          newMode = 'manual'
-          newValue = 84
-        } else {
-          codeToSend = this.cacheAutoCode
-        }
-      } else {
-        codeToSend = this.speedCodes[newValue]
-      }
-
       await this.platform.sendDeviceUpdate(this.accessory, {
         cmd: 'ptReal',
-        value: codeToSend,
+        value: this.speedCodes[value],
       })
 
       // Cache the new state and log if appropriate
-      if (this.cacheMode !== newMode) {
-        this.cacheMode = newMode
-        this.accessory.log(`${platformLang.curMode} [${this.cacheMode}]`)
-      }
-      if (this.cacheSpeed !== newValue) {
-        this.cacheSpeed = newValue
-        this.accessory.log(`${platformLang.curSpeed} [${newValue}%]`)
+      if (this.cacheSpeed !== value) {
+        this.cacheSpeed = value
+        this.accessory.log(`${platformLang.curSpeed} [${value}]`)
       }
     } catch (err) {
       // Catch any errors during the process
@@ -218,13 +178,22 @@ export default class {
     try {
       const newValue = value ? 'on' : 'off'
       // Don't continue if the new value is the same as before
-      if (this.cacheSwing === value) {
+      if (this.cacheSwing === value || !this.cacheSwingCode) {
         return
       }
 
+      // 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: 'ptReal',
-        value: value ? 'Mx8BAQAAAAAAAAAAAAAAAAAAACw=' : 'Mx8BAAAAAAAAAAAAAAAAAAAAAC0=',
+        value: generateCodeFromHexValues(hexValues),
       })
 
       // Cache the new state and log if appropriate
@@ -408,16 +377,11 @@ export default class {
         case '0501': {
           // Fan speed
           const newSpeed = getTwoItemPosition(hexParts, 4)
-          const newSpeedInt = Number.parseInt(newSpeed, 16) * 7
-          const newMode = 'manual'
-          if (this.cacheMode !== newMode) {
-            this.cacheMode = newMode
-            this.accessory.log(`${platformLang.curMode} [${this.cacheMode}]`)
-          }
+          const newSpeedInt = Number.parseInt(newSpeed, 16)
           if (this.cacheSpeed !== newSpeedInt) {
             this.cacheSpeed = newSpeedInt
             this.service.updateCharacteristic(this.hapChar.RotationSpeed, this.cacheSpeed)
-            this.accessory.log(`${platformLang.curSpeed} [${this.cacheSpeed}%]`)
+            this.accessory.log(`${platformLang.curSpeed} [${this.cacheSpeed}]`)
           }
           break
         }
@@ -430,66 +394,66 @@ export default class {
           // Sleep:               5
           // Nature:              6
           // Turbo:               7
-          const newMode = getTwoItemPosition(hexParts, 4) === '03' ? 'auto' : 'manual'
-          if (this.cacheMode !== newMode) {
-            this.cacheMode = newMode
-            this.accessory.log(`${platformLang.curMode} [${this.cacheMode}]`)
-
-            if (this.cacheMode === 'auto' && this.cacheSpeed < 91) {
-              this.cacheSpeed = 91
-              this.service.updateCharacteristic(this.hapChar.RotationSpeed, this.cacheSpeed)
-              this.accessory.log(`${platformLang.curSpeed} [${this.cacheSpeed}%]`)
-            }
-          }
-          break
-        }
-        case '0503': {
-          // Auto mode, we need to keep this code to send it back to the device
-          const code = hexToTwoItems(`33${hexString.substring(2, hexString.length - 2)}`)
-          this.cacheAutoCode = generateCodeFromHexValues(code.map(p => Number.parseInt(p, 16)))
           break
         }
         case '1b01': {
-          const newLightState = getTwoItemPosition(hexParts, 4) === '01' ? 'on' : 'off'
-          if (this.cacheLightState !== newLightState) {
-            this.cacheLightState = newLightState
-            this.lightService.updateCharacteristic(this.hapChar.On, this.cacheLightState === 'on')
-            this.accessory.log(`${platformLang.curLight} [${this.cacheLightState}]`)
-          }
-          const newBrightness = hexToDecimal(getTwoItemPosition(hexParts, 5))
-          if (this.cacheBright !== newBrightness) {
-            this.cacheBright = newBrightness
-            this.lightService.updateCharacteristic(this.hapChar.Brightness, this.cacheBright)
-            this.accessory.log(`${platformLang.curBright} [${this.cacheBright}%]`)
+          // Night light on/off
+          if (!this.hideLight) {
+            const newLightState = getTwoItemPosition(hexParts, 4) === '01' ? 'on' : 'off'
+            if (this.cacheLightState !== newLightState) {
+              this.cacheLightState = newLightState
+              this.lightService.updateCharacteristic(this.hapChar.On, this.cacheLightState === 'on')
+              this.accessory.log(`${platformLang.curLight} [${this.cacheLightState}]`)
+            }
+            const newBrightness = hexToDecimal(getTwoItemPosition(hexParts, 5))
+            if (this.cacheBright !== newBrightness) {
+              this.cacheBright = newBrightness
+              this.lightService.updateCharacteristic(this.hapChar.Brightness, this.cacheBright)
+              this.accessory.log(`${platformLang.curBright} [${this.cacheBright}%]`)
+            }
           }
           break
         }
         case '1b05': {
           // Night light colour
-          const newR = hexToDecimal(getTwoItemPosition(hexParts, 5))
-          const newG = hexToDecimal(getTwoItemPosition(hexParts, 6))
-          const newB = hexToDecimal(getTwoItemPosition(hexParts, 7))
-
-          const hs = rgb2hs(newR, newG, newB)
-
-          // Check for a colour change
-          if (hs[0] !== this.cacheHue) {
-            // Colour is different so update Homebridge with new values
-            this.lightService.updateCharacteristic(this.hapChar.Hue, hs[0])
-            this.lightService.updateCharacteristic(this.hapChar.Saturation, hs[1]);
-            [this.cacheHue] = hs
-
-            // Log the change
-            this.accessory.log(`${platformLang.curColour} [rgb ${newR} ${newG} ${newB}]`)
+          if (!this.hideLight) {
+            const newR = hexToDecimal(getTwoItemPosition(hexParts, 5))
+            const newG = hexToDecimal(getTwoItemPosition(hexParts, 6))
+            const newB = hexToDecimal(getTwoItemPosition(hexParts, 7))
+
+            const hs = rgb2hs(newR, newG, newB)
+
+            // Check for a colour change
+            if (hs[0] !== this.cacheHue) {
+              // Colour is different so update Homebridge with new values
+              this.lightService.updateCharacteristic(this.hapChar.Hue, hs[0])
+              this.lightService.updateCharacteristic(this.hapChar.Saturation, hs[1]);
+              [this.cacheHue] = hs
+
+              // Log the change
+              this.accessory.log(`${platformLang.curColour} [rgb ${newR} ${newG} ${newB}]`)
+            }
+          }
+          break
+        }
+        case '1d00':{
+          // Swing Mode Off
+          const newSwing = 'off'
+          this.cacheSwingCode = hexParts
+          if (this.cacheSwing !== newSwing) {
+            this.cacheSwing = newSwing
+            this.service.updateCharacteristic(this.hapChar.SwingMode, 0)
+            this.accessory.log(`${platformLang.curSwing} [${this.cacheSwing}]`)
           }
           break
         }
-        case '1f01': {
-          // Swing Mode
-          const newSwing = getTwoItemPosition(hexParts, 4) === '01' ? 'on' : 'off'
+        case '1d01':{
+          // Swing Mode On
+          const newSwing = 'on'
+          this.cacheSwingCode = hexParts
           if (this.cacheSwing !== newSwing) {
             this.cacheSwing = newSwing
-            this.service.updateCharacteristic(this.hapChar.SwingMode, this.cacheSwing === 'on' ? 1 : 0)
+            this.service.updateCharacteristic(this.hapChar.SwingMode, 1)
             this.accessory.log(`${platformLang.curSwing} [${this.cacheSwing}]`)
           }
           break

package/lib/platform.js

@@ -200,6 +200,7 @@ export default class {
                     break
                   }
                   case 'awsBrightnessNoScale':
+                  case 'hideLight':
                   case 'hideModeGreenTea':
                   case 'hideModeOolongTea':
                   case 'hideModeCoffee':

package/lib/utils/constants.js

@@ -91,7 +91,7 @@ export default {
     ],
     leakDevices: ['label', 'deviceId', 'ignoreDevice', 'lowBattThreshold'],
     thermoDevices: ['label', 'deviceId', 'ignoreDevice', 'lowBattThreshold', 'showExtraSwitch'],
-    fanDevices: ['label', 'deviceId', 'ignoreDevice'],
+    fanDevices: ['label', 'deviceId', 'ignoreDevice', 'hideLight'],
     heaterDevices: ['label', 'deviceId', 'ignoreDevice', 'tempReporting'],
     humidifierDevices: ['label', 'deviceId', 'ignoreDevice'],
     dehumidifierDevices: ['label', 'deviceId', 'ignoreDevice'],

package/package.json

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