@homebridge-plugins/homebridge-govee 11.32.0-beta.24 → 11.32.0-beta.26

package/CHANGELOG.md

@@ -27,6 +27,9 @@ All notable changes to `@homebridge-plugins/homebridge-govee` will be documented
 - fix: hide the H5089 night light tile until its commands are worked out, rather than showing a control that does nothing (#1323)
 - feat: add support for the H3001 solar string lights as an rgb light (#1328)
 - docs(changelog): list every unreleased commit in the pending section
+- fix(fans): rebuild the fan service once when it was cached under the old speed scale, so the top of the slider stops setting the slowest speed (#1326) (@cwhett)
+- fix: show the H5089 night light tile again, but refuse control commands, so it reports the real state instead of quietly ignoring taps (#1323)
+- fix(ice-makers): stop the home app showing an ice size of 0 when one has never been set (#1329) (@cwhett)
 
 ## v11.31.0 (2026-07-24)
 

package/lib/device/fan-H1310.js

@@ -40,10 +40,12 @@ export default class {
       this.accessory.removeService(this.accessory.getService(this.hapServ.Fan))
     }
 
-    // Migrate old %-rotation speed to unitless
+    // Rebuild the fan service if it was cached under an older speed scale, as
+    // HomeKit keeps the old props until the service itself is replaced
     const existingService = this.accessory.getService(this.hapServ.Fanv2)
     if (existingService) {
-      if (existingService.getCharacteristic(this.hapChar.RotationSpeed).props.unit === 'percentage') {
+      const { props } = existingService.getCharacteristic(this.hapChar.RotationSpeed)
+      if (props.unit === 'percentage' || props.maxValue !== 100) {
         this.accessory.removeService(existingService)
       }
     }

package/lib/device/fan-H1370.js

@@ -34,6 +34,16 @@ export default class {
     const deviceConf = platform.deviceConf[accessory.context.gvDeviceId]
     this.hideLight = deviceConf && deviceConf.hideLight
 
+    // Rebuild the fan service if it was cached under an older speed scale, as
+    // HomeKit keeps the old props until the service itself is replaced
+    const existingService = this.accessory.getService(this.hapServ.Fanv2)
+    if (existingService) {
+      const { props } = existingService.getCharacteristic(this.hapChar.RotationSpeed)
+      if (props.unit === 'percentage' || props.maxValue !== 100) {
+        this.accessory.removeService(existingService)
+      }
+    }
+
     // Add the fan service if it doesn't already exist
     this.service = this.accessory.getService(this.hapServ.Fanv2) || this.accessory.addService(this.hapServ.Fanv2)
 

package/lib/device/fan-H7100.js

@@ -40,10 +40,12 @@ export default class {
       this.accessory.removeService(this.accessory.getService(this.hapServ.Fan))
     }
 
-    // Migrate old %-rotation speed to unitless
+    // Rebuild the fan service if it was cached under an older speed scale, as
+    // HomeKit keeps the old props until the service itself is replaced
     const existingService = this.accessory.getService(this.hapServ.Fanv2)
     if (existingService) {
-      if (existingService.getCharacteristic(this.hapChar.RotationSpeed).props.unit === 'percentage') {
+      const { props } = existingService.getCharacteristic(this.hapChar.RotationSpeed)
+      if (props.unit === 'percentage' || props.maxValue !== 100) {
         this.accessory.removeService(existingService)
       }
     }

package/lib/device/fan-H7101.js

@@ -96,10 +96,12 @@ export default class {
       this.accessory.removeService(this.accessory.getService(this.hapServ.Fan))
     }
 
-    // Migrate old %-rotation speed to unitless
+    // Rebuild the fan service if it was cached under an older speed scale, as
+    // HomeKit keeps the old props until the service itself is replaced
     const existingService = this.accessory.getService(this.hapServ.Fanv2)
     if (existingService) {
-      if (existingService.getCharacteristic(this.hapChar.RotationSpeed).props.unit === 'percentage') {
+      const { props } = existingService.getCharacteristic(this.hapChar.RotationSpeed)
+      if (props.unit === 'percentage' || props.maxValue !== 100) {
         this.accessory.removeService(existingService)
       }
     }

package/lib/device/fan-H7102.js

@@ -40,10 +40,12 @@ export default class {
       this.accessory.removeService(this.accessory.getService(this.hapServ.Fan))
     }
 
-    // Migrate old %-rotation speed to unitless
+    // Rebuild the fan service if it was cached under an older speed scale, as
+    // HomeKit keeps the old props until the service itself is replaced
     const existingService = this.accessory.getService(this.hapServ.Fanv2)
     if (existingService) {
-      if (existingService.getCharacteristic(this.hapChar.RotationSpeed).props.unit === 'percentage') {
+      const { props } = existingService.getCharacteristic(this.hapChar.RotationSpeed)
+      if (props.unit === 'percentage' || props.maxValue !== 100) {
         this.accessory.removeService(existingService)
       }
     }

package/lib/device/fan-H7105.js

@@ -53,10 +53,12 @@ export default class {
       this.accessory.removeService(this.accessory.getService(this.hapServ.Fan))
     }
 
-    // Migrate old %-rotation speed to unitless
+    // Rebuild the fan service if it was cached under an older speed scale, as
+    // HomeKit keeps the old props until the service itself is replaced
     const existingService = this.accessory.getService(this.hapServ.Fanv2)
     if (existingService) {
-      if (existingService.getCharacteristic(this.hapChar.RotationSpeed).props.unit === 'percentage') {
+      const { props } = existingService.getCharacteristic(this.hapChar.RotationSpeed)
+      if (props.unit === 'percentage' || props.maxValue !== 100) {
         this.accessory.removeService(existingService)
       }
     }

package/lib/device/fan-H7106.js

@@ -40,10 +40,12 @@ export default class {
       this.accessory.removeService(this.accessory.getService(this.hapServ.Fan))
     }
 
-    // Migrate old %-rotation speed to unitless
+    // Rebuild the fan service if it was cached under an older speed scale, as
+    // HomeKit keeps the old props until the service itself is replaced
     const existingService = this.accessory.getService(this.hapServ.Fanv2)
     if (existingService) {
-      if (existingService.getCharacteristic(this.hapChar.RotationSpeed).props.unit === 'percentage') {
+      const { props } = existingService.getCharacteristic(this.hapChar.RotationSpeed)
+      if (props.unit === 'percentage' || props.maxValue !== 100) {
         this.accessory.removeService(existingService)
       }
     }

package/lib/device/fan-H7107.js

@@ -53,10 +53,12 @@ export default class {
       this.accessory.removeService(this.accessory.getService(this.hapServ.Fan))
     }
 
-    // Migrate old %-rotation speed to unitless
+    // Rebuild the fan service if it was cached under an older speed scale, as
+    // HomeKit keeps the old props until the service itself is replaced
     const existingService = this.accessory.getService(this.hapServ.Fanv2)
     if (existingService) {
-      if (existingService.getCharacteristic(this.hapChar.RotationSpeed).props.unit === 'percentage') {
+      const { props } = existingService.getCharacteristic(this.hapChar.RotationSpeed)
+      if (props.unit === 'percentage' || props.maxValue !== 100) {
         this.accessory.removeService(existingService)
       }
     }

package/lib/device/fan-H7111.js

@@ -101,10 +101,12 @@ export default class {
       this.accessory.removeService(this.accessory.getService(this.hapServ.Fan))
     }
 
-    // Migrate old %-rotation speed to unitless
+    // Rebuild the fan service if it was cached under an older speed scale, as
+    // HomeKit keeps the old props until the service itself is replaced
     const existingService = this.accessory.getService(this.hapServ.Fanv2)
     if (existingService) {
-      if (existingService.getCharacteristic(this.hapChar.RotationSpeed).props.unit === 'percentage') {
+      const { props } = existingService.getCharacteristic(this.hapChar.RotationSpeed)
+      if (props.unit === 'percentage' || props.maxValue !== 100) {
         this.accessory.removeService(existingService)
       }
     }

package/lib/device/ice-maker-H7172.js

@@ -52,6 +52,13 @@ export default class {
       .onSet(async value => this.internalSizeUpdate(value))
     this.cacheSize = this.service.getCharacteristic(this.hapChar.RotationSpeed).value || 2
 
+    // Keep the characteristic in step with the cache. Without this an accessory
+    // that has never had a size set reports Active=1 with RotationSpeed=0, which
+    // is contradictory and which the Home app renders as "0".
+    if (this.service.getCharacteristic(this.hapChar.RotationSpeed).value !== this.cacheSize) {
+      this.service.updateCharacteristic(this.hapChar.RotationSpeed, this.cacheSize)
+    }
+
     // Output the customised options to the log
     const opts = JSON.stringify({})
     platform.log('[%s] %s %s.', accessory.displayName, platformLang.devInitOpts, opts)
@@ -105,8 +112,15 @@ export default class {
 
   async internalSizeUpdate(value) {
     try {
-      // Don't continue if 0
+      // 0 is not a valid ice size. Put the slider back, otherwise HomeKit keeps 0
+      // while the device carries on at cacheSize - and that 0 is what gets written
+      // to the accessory cache, so it returns on the next restart. Deferred for the
+      // same reason as the error reverts below: an immediate update would be
+      // overwritten by the value HomeKit is currently writing.
       if (value === 0) {
+        setTimeout(() => {
+          this.service.updateCharacteristic(this.hapChar.RotationSpeed, this.cacheSize)
+        }, 1000)
         return
       }
 
@@ -145,8 +159,11 @@ export default class {
     if (params.workMode) {
       const mode = params.workMode.workMode
       if (mode > 0 && mode <= 3) {
-        this.cacheState = 'on'
-        this.service.updateCharacteristic(this.hapChar.Active, 1)
+        if (this.cacheState !== 'on') {
+          this.cacheState = 'on'
+          this.service.updateCharacteristic(this.hapChar.Active, 1)
+          this.accessory.log(`${platformLang.curState} [on]`)
+        }
         if (this.cacheSize !== mode) {
           this.cacheSize = mode
           this.service.updateCharacteristic(this.hapChar.RotationSpeed, this.cacheSize)
@@ -184,6 +201,14 @@ export default class {
           }
           break
         }
+        case 'aa05': {
+          // Ice size report. The payload encoding is deliberately NOT decoded here:
+          // sizeCodes runs small=0x03..large=0x01 while the OpenAPI path sends
+          // workMode un-inverted, and which is authoritative is unresolved (#1329).
+          // Claiming a mapping here could silently swap sizes, so this case exists
+          // only to stop a routine status report being logged as a new scene code.
+          break
+        }
         default:
           this.accessory.logWarn(`${platformLang.newScene}: [${command}] [${hexString}]`)
           break

package/lib/device/ice-maker-H717D.js

@@ -52,6 +52,13 @@ export default class {
       .onSet(async value => this.internalSizeUpdate(value))
     this.cacheSize = this.service.getCharacteristic(this.hapChar.RotationSpeed).value || 2
 
+    // Keep the characteristic in step with the cache. Without this an accessory
+    // that has never had a size set reports Active=1 with RotationSpeed=0, which
+    // is contradictory and which the Home app renders as "0".
+    if (this.service.getCharacteristic(this.hapChar.RotationSpeed).value !== this.cacheSize) {
+      this.service.updateCharacteristic(this.hapChar.RotationSpeed, this.cacheSize)
+    }
+
     // Output the customised options to the log
     const opts = JSON.stringify({})
     platform.log('[%s] %s %s.', accessory.displayName, platformLang.devInitOpts, opts)
@@ -105,8 +112,15 @@ export default class {
 
   async internalSizeUpdate(value) {
     try {
-      // Don't continue if 0
+      // 0 is not a valid ice size. Put the slider back, otherwise HomeKit keeps 0
+      // while the device carries on at cacheSize - and that 0 is what gets written
+      // to the accessory cache, so it returns on the next restart. Deferred for the
+      // same reason as the error reverts below: an immediate update would be
+      // overwritten by the value HomeKit is currently writing.
       if (value === 0) {
+        setTimeout(() => {
+          this.service.updateCharacteristic(this.hapChar.RotationSpeed, this.cacheSize)
+        }, 1000)
         return
       }
 
@@ -145,8 +159,11 @@ export default class {
     if (params.workMode) {
       const mode = params.workMode.workMode
       if (mode > 0 && mode <= 3) {
-        this.cacheState = 'on'
-        this.service.updateCharacteristic(this.hapChar.Active, 1)
+        if (this.cacheState !== 'on') {
+          this.cacheState = 'on'
+          this.service.updateCharacteristic(this.hapChar.Active, 1)
+          this.accessory.log(`${platformLang.curState} [on]`)
+        }
         if (this.cacheSize !== mode) {
           this.cacheSize = mode
           this.service.updateCharacteristic(this.hapChar.RotationSpeed, this.cacheSize)
@@ -184,6 +201,14 @@ export default class {
           }
           break
         }
+        case 'aa05': {
+          // Ice size report. The payload encoding is deliberately NOT decoded here:
+          // sizeCodes runs small=0x03..large=0x01 while the OpenAPI path sends
+          // workMode un-inverted, and which is authoritative is unresolved (#1329).
+          // Claiming a mapping here could silently swap sizes, so this case exists
+          // only to stop a routine status report being logged as a new scene code.
+          break
+        }
         default:
           this.accessory.logWarn(`${platformLang.newScene}: [${command}] [${hexString}]`)
           break

package/lib/device/ice-maker-H8120.js

@@ -52,6 +52,13 @@ export default class {
       .onSet(async value => this.internalSizeUpdate(value))
     this.cacheSize = this.service.getCharacteristic(this.hapChar.RotationSpeed).value || 2
 
+    // Keep the characteristic in step with the cache. Without this an accessory
+    // that has never had a size set reports Active=1 with RotationSpeed=0, which
+    // is contradictory and which the Home app renders as "0".
+    if (this.service.getCharacteristic(this.hapChar.RotationSpeed).value !== this.cacheSize) {
+      this.service.updateCharacteristic(this.hapChar.RotationSpeed, this.cacheSize)
+    }
+
     // Output the customised options to the log
     const opts = JSON.stringify({})
     platform.log('[%s] %s %s.', accessory.displayName, platformLang.devInitOpts, opts)
@@ -105,8 +112,15 @@ export default class {
 
   async internalSizeUpdate(value) {
     try {
-      // Don't continue if 0
+      // 0 is not a valid ice size. Put the slider back, otherwise HomeKit keeps 0
+      // while the device carries on at cacheSize - and that 0 is what gets written
+      // to the accessory cache, so it returns on the next restart. Deferred for the
+      // same reason as the error reverts below: an immediate update would be
+      // overwritten by the value HomeKit is currently writing.
       if (value === 0) {
+        setTimeout(() => {
+          this.service.updateCharacteristic(this.hapChar.RotationSpeed, this.cacheSize)
+        }, 1000)
         return
       }
 
@@ -145,8 +159,11 @@ export default class {
     if (params.workMode) {
       const mode = params.workMode.workMode
       if (mode > 0 && mode <= 3) {
-        this.cacheState = 'on'
-        this.service.updateCharacteristic(this.hapChar.Active, 1)
+        if (this.cacheState !== 'on') {
+          this.cacheState = 'on'
+          this.service.updateCharacteristic(this.hapChar.Active, 1)
+          this.accessory.log(`${platformLang.curState} [on]`)
+        }
         if (this.cacheSize !== mode) {
           this.cacheSize = mode
           this.service.updateCharacteristic(this.hapChar.RotationSpeed, this.cacheSize)
@@ -184,6 +201,14 @@ export default class {
           }
           break
         }
+        case 'aa05': {
+          // Ice size report. The payload encoding is deliberately NOT decoded here:
+          // sizeCodes runs small=0x03..large=0x01 while the OpenAPI path sends
+          // workMode un-inverted, and which is authoritative is unresolved (#1329).
+          // Claiming a mapping here could silently swap sizes, so this case exists
+          // only to stop a routine status report being logged as a new scene code.
+          break
+        }
         default:
           this.accessory.logWarn(`${platformLang.newScene}: [${command}] [${hexString}]`)
           break

package/lib/device/outlet-double.js

@@ -1,13 +1,12 @@
-import { base64ToHex, generateCodeFromHexValues, parseError } from '../utils/functions.js'
+import { base64ToHex, parseError } from '../utils/functions.js'
 import platformLang from '../utils/lang-en.js'
 
-// Models whose night light can be controlled. The H5089 reports its light as
-// `aa 1b 01 [on] [level]` and that reading is correct, but no command shape
-// tried so far moves it - neither mirroring its own status code nor the
-// `33 18` the purifiers use - so it is left out rather than shipping a tile
-// that reports properly and ignores every tap (#1323). Put the model back
-// here once a command is known to work.
-const nightLightModels = []
+// Models that expose a night light. The H5089 reports its light as
+// `aa 1b 01 [on] [level]` and that reading is correct, so the tile shows the
+// real state. No command shape tried moves it - neither mirroring its own
+// status code nor the `33 18` the purifiers use - so control is refused
+// outright rather than accepted and silently dropped (#1323).
+const nightLightModels = ['H5089']
 
 export default class {
   constructor(platform, accessory) {
@@ -77,13 +76,9 @@ export default class {
         this.lightService.addCharacteristic(this.hapChar.ConfiguredName)
         this.lightService.updateCharacteristic(this.hapChar.ConfiguredName, 'Night Light')
       }
-      this.lightService.getCharacteristic(this.hapChar.On).onSet(async (value) => {
-        await this.internalLightStateUpdate(value)
-      })
+      this.lightService.getCharacteristic(this.hapChar.On).onSet(() => this.refuseLightControl())
       this.cacheLightState = this.lightService.getCharacteristic(this.hapChar.On).value ? 'on' : 'off'
-      this.lightService.getCharacteristic(this.hapChar.Brightness).onSet(async (value) => {
-        await this.internalLightBrightnessUpdate(value)
-      })
+      this.lightService.getCharacteristic(this.hapChar.Brightness).onSet(() => this.refuseLightControl())
       this.cacheLightBright = this.lightService.getCharacteristic(this.hapChar.Brightness).value || 100
     } else if (this.accessory.getService('Night Light')) {
       this.accessory.removeService(this.accessory.getService('Night Light'))
@@ -136,75 +131,12 @@ export default class {
     }
   }
 
-  // Govee does not use one night light command across its range: the purifiers
-  // that already work here send `33 18 [on] [level]`, while this plug reports
-  // its light as `aa 1b 01 [on] [level]`. Mirroring the status code was not
-  // enough on its own, so send both shapes - they express the same thing, so
-  // whichever the plug ignores costs nothing. Once it is known which one lands
-  // this drops to that one alone (#1323)
-  async sendLightCommand(state, level) {
-    const shapes = [
-      [0x33, 0x1B, 0x01, state, level],
-      [0x33, 0x18, state, level],
-    ]
-    let lastError
-    for (const hexValues of shapes) {
-      try {
-        await this.platform.sendDeviceUpdate(this.accessory, {
-          // This plug acks the app's commands as `ptIot` and ignores `ptReal`
-          cmd: 'ptIot',
-          value: generateCodeFromHexValues(hexValues),
-        })
-      } catch (err) {
-        lastError = err
-      }
-    }
-    if (lastError) {
-      throw lastError
-    }
-  }
-
-  async internalLightStateUpdate(value) {
-    try {
-      const newValue = value ? 'on' : 'off'
-      if (this.cacheLightState === newValue) {
-        return
-      }
-
-      // Keep the current level when toggling, so turning the light on does not
-      // also change how bright it is
-      await this.sendLightCommand(value ? 0x01 : 0x00, this.cacheLightBright)
-
-      this.cacheLightState = newValue
-      this.accessory.log(`[Night Light] ${platformLang.curState} [${newValue}]`)
-    } catch (err) {
-      this.accessory.logWarn(`${platformLang.devNotUpdated} ${parseError(err)}`)
-      setTimeout(() => {
-        this.lightService.updateCharacteristic(this.hapChar.On, this.cacheLightState === 'on')
-      }, 2000)
-      throw new this.hapErr(-70402)
-    }
-  }
-
-  async internalLightBrightnessUpdate(value) {
-    try {
-      if (value === 0 || this.cacheLightBright === value) {
-        return
-      }
-
-      // Setting a level also turns the light on, matching how HomeKit sends
-      // brightness changes alongside an on command
-      await this.sendLightCommand(0x01, value)
-
-      this.cacheLightBright = value
-      this.accessory.log(`[Night Light] ${platformLang.curBright} [${value}%]`)
-    } catch (err) {
-      this.accessory.logWarn(`${platformLang.devNotUpdated} ${parseError(err)}`)
-      setTimeout(() => {
-        this.lightService.updateCharacteristic(this.hapChar.Brightness, this.cacheLightBright)
-      }, 2000)
-      throw new this.hapErr(-70402)
-    }
+  // The night light reading is correct, but every command shape tried has been
+  // ignored by the plug, so refuse the write. HomeKit reverts the tile rather
+  // than showing a state the plug was never given (#1323)
+  refuseLightControl() {
+    this.accessory.logWarn(`[Night Light] ${platformLang.devNotUpdated} this device does not accept night light commands`)
+    throw new this.hapErr(-70402)
   }
 
   externalUpdate(params) {

package/lib/device/switch-double.js

@@ -1,13 +1,12 @@
-import { base64ToHex, generateCodeFromHexValues, parseError } from '../utils/functions.js'
+import { base64ToHex, parseError } from '../utils/functions.js'
 import platformLang from '../utils/lang-en.js'
 
-// Models whose night light can be controlled. The H5089 reports its light as
-// `aa 1b 01 [on] [level]` and that reading is correct, but no command shape
-// tried so far moves it - neither mirroring its own status code nor the
-// `33 18` the purifiers use - so it is left out rather than shipping a tile
-// that reports properly and ignores every tap (#1323). Put the model back
-// here once a command is known to work.
-const nightLightModels = []
+// Models that expose a night light. The H5089 reports its light as
+// `aa 1b 01 [on] [level]` and that reading is correct, so the tile shows the
+// real state. No command shape tried moves it - neither mirroring its own
+// status code nor the `33 18` the purifiers use - so control is refused
+// outright rather than accepted and silently dropped (#1323).
+const nightLightModels = ['H5089']
 
 export default class {
   constructor(platform, accessory) {
@@ -77,13 +76,9 @@ export default class {
         this.lightService.addCharacteristic(this.hapChar.ConfiguredName)
         this.lightService.updateCharacteristic(this.hapChar.ConfiguredName, 'Night Light')
       }
-      this.lightService.getCharacteristic(this.hapChar.On).onSet(async (value) => {
-        await this.internalLightStateUpdate(value)
-      })
+      this.lightService.getCharacteristic(this.hapChar.On).onSet(() => this.refuseLightControl())
       this.cacheLightState = this.lightService.getCharacteristic(this.hapChar.On).value ? 'on' : 'off'
-      this.lightService.getCharacteristic(this.hapChar.Brightness).onSet(async (value) => {
-        await this.internalLightBrightnessUpdate(value)
-      })
+      this.lightService.getCharacteristic(this.hapChar.Brightness).onSet(() => this.refuseLightControl())
       this.cacheLightBright = this.lightService.getCharacteristic(this.hapChar.Brightness).value || 100
     } else if (this.accessory.getService('Night Light')) {
       this.accessory.removeService(this.accessory.getService('Night Light'))
@@ -136,75 +131,12 @@ export default class {
     }
   }
 
-  // Govee does not use one night light command across its range: the purifiers
-  // that already work here send `33 18 [on] [level]`, while this plug reports
-  // its light as `aa 1b 01 [on] [level]`. Mirroring the status code was not
-  // enough on its own, so send both shapes - they express the same thing, so
-  // whichever the plug ignores costs nothing. Once it is known which one lands
-  // this drops to that one alone (#1323)
-  async sendLightCommand(state, level) {
-    const shapes = [
-      [0x33, 0x1B, 0x01, state, level],
-      [0x33, 0x18, state, level],
-    ]
-    let lastError
-    for (const hexValues of shapes) {
-      try {
-        await this.platform.sendDeviceUpdate(this.accessory, {
-          // This plug acks the app's commands as `ptIot` and ignores `ptReal`
-          cmd: 'ptIot',
-          value: generateCodeFromHexValues(hexValues),
-        })
-      } catch (err) {
-        lastError = err
-      }
-    }
-    if (lastError) {
-      throw lastError
-    }
-  }
-
-  async internalLightStateUpdate(value) {
-    try {
-      const newValue = value ? 'on' : 'off'
-      if (this.cacheLightState === newValue) {
-        return
-      }
-
-      // Keep the current level when toggling, so turning the light on does not
-      // also change how bright it is
-      await this.sendLightCommand(value ? 0x01 : 0x00, this.cacheLightBright)
-
-      this.cacheLightState = newValue
-      this.accessory.log(`[Night Light] ${platformLang.curState} [${newValue}]`)
-    } catch (err) {
-      this.accessory.logWarn(`${platformLang.devNotUpdated} ${parseError(err)}`)
-      setTimeout(() => {
-        this.lightService.updateCharacteristic(this.hapChar.On, this.cacheLightState === 'on')
-      }, 2000)
-      throw new this.hapErr(-70402)
-    }
-  }
-
-  async internalLightBrightnessUpdate(value) {
-    try {
-      if (value === 0 || this.cacheLightBright === value) {
-        return
-      }
-
-      // Setting a level also turns the light on, matching how HomeKit sends
-      // brightness changes alongside an on command
-      await this.sendLightCommand(0x01, value)
-
-      this.cacheLightBright = value
-      this.accessory.log(`[Night Light] ${platformLang.curBright} [${value}%]`)
-    } catch (err) {
-      this.accessory.logWarn(`${platformLang.devNotUpdated} ${parseError(err)}`)
-      setTimeout(() => {
-        this.lightService.updateCharacteristic(this.hapChar.Brightness, this.cacheLightBright)
-      }, 2000)
-      throw new this.hapErr(-70402)
-    }
+  // The night light reading is correct, but every command shape tried has been
+  // ignored by the plug, so refuse the write. HomeKit reverts the tile rather
+  // than showing a state the plug was never given (#1323)
+  refuseLightControl() {
+    this.accessory.logWarn(`[Night Light] ${platformLang.devNotUpdated} this device does not accept night light commands`)
+    throw new this.hapErr(-70402)
   }
 
   externalUpdate(params) {

package/package.json

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