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

package/CHANGELOG.md

@@ -29,6 +29,7 @@ All notable changes to `@homebridge-plugins/homebridge-govee` will be documented
 - 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/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/package.json

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