@homebridge-plugins/homebridge-govee 11.29.3 → 11.30.0-beta.1

package/CHANGELOG.md

@@ -2,6 +2,14 @@
 
 All notable changes to `@homebridge-plugins/homebridge-govee` will be documented in this file.
 
+## v11.30.0 (Pending Release)
+
+### Changed
+
+- chore(ble): let noble apply its own adapter timeout instead of racing a second one
+- fix: restore aws brightness scaling, so lights reach the brightness asked for (#1262) (@gradyg21)
+- fix: skip a periodic sync when the previous one is still running, so slow responses stop stacking up (#1256) (@schwaggy)
+
 ## v11.29.3 (2026-07-21)
 
 ### Changed

package/lib/connection/ble.js

@@ -174,20 +174,16 @@ export default class BLEConnection {
       return true
     }
 
-    let timeoutId
     try {
-      await Promise.race([
-        btClient.waitForPoweredOnAsync(),
-        new Promise((_, reject) => {
-          timeoutId = setTimeout(() => reject(new Error('timeout waiting for bluetooth adapter')), timeout)
-        }),
-      ])
+      // noble applies the timeout itself and removes its own state listener when
+      // it expires. Racing it against a second, shorter timer here just meant
+      // noble kept waiting - and holding its listener - for another five seconds
+      // after we had already given up.
+      await btClient.waitForPoweredOnAsync(timeout)
       return true
     } catch (err) {
       this.log.warn('[BLE] failed to power on adapter: %s.', err.message)
       return false
-    } finally {
-      clearTimeout(timeoutId)
     }
   }
 

package/lib/platform.js

@@ -57,6 +57,9 @@ export default class {
       this.deviceConf = {}
       this.ignoredDevices = []
 
+      // Periodic syncs currently running, so a slow one is not started twice
+      this.syncsInFlight = new Set()
+
       // Make sure user is running Homebridge v1.5 or above
       if (!api.versionGreaterOrEqual?.('1.5.0')) {
         throw new Error(platformLang.hbVersionFail)
@@ -217,6 +220,7 @@ export default class {
                     }
                     break
                   }
+                  case 'awsBrightnessNoScale':
                   case 'hideLight':
                   case 'hideModeGreenTea':
                   case 'hideModeOolongTea':
@@ -776,7 +780,7 @@ export default class {
 
           this.refreshBLEInterval = setInterval(async () => {
             try {
-              await this.goveeBLESync()
+              await this.runPeriodicSync('BLE', () => this.goveeBLESync())
             } catch (err) {
               this.log.warn('[BLE] sync failed: %s', parseError(err))
             }
@@ -790,7 +794,7 @@ export default class {
       if (this.httpClient && httpSyncNeeded) {
         this.goveeHTTPSync()
         this.refreshHTTPInterval = setInterval(
-          () => this.goveeHTTPSync(),
+          () => this.runPeriodicSync('HTTP', () => this.goveeHTTPSync()),
           this.config.httpRefreshTime * 1000,
         )
       }
@@ -799,7 +803,7 @@ export default class {
         const openApiRefresh = (this.config.openApiRefreshTime || this.config.httpRefreshTime) * 1000
         this.goveeOpenApiSync()
         this.refreshOpenApiInterval = setInterval(
-          () => this.goveeOpenApiSync(),
+          () => this.runPeriodicSync('OPENAPI', () => this.goveeOpenApiSync()),
           openApiRefresh,
         )
 
@@ -819,7 +823,7 @@ export default class {
         // No need for await as catches its own errors, we poll specific models that need it
         this.goveeAWSSync(true)
         this.refreshAWSInterval = setInterval(
-          () => this.goveeAWSSync(),
+          () => this.runPeriodicSync('AWS', () => this.goveeAWSSync()),
           60000,
         )
       }
@@ -1363,6 +1367,7 @@ export default class {
 
               if (this.awsClient) {
                 accessory.context.useAwsControl = true
+                accessory.context.awsBrightnessNoScale = deviceConf.awsBrightnessNoScale
                 accessory.context.awsColourMode = deviceConf.awsColourMode || platformConsts.defaultValues.awsColourMode
                 awsDevices.push(device.device)
 
@@ -1441,6 +1446,28 @@ export default class {
     }
   }
 
+  /**
+   * Run a periodic sync, skipping this run if the previous one is still going.
+   *
+   * The intervals fire on a timer regardless of how long a sync takes, and the
+   * http request timeout and the default refresh interval are both thirty
+   * seconds. A single slow response therefore guaranteed the next sync started
+   * before the last had finished, and those stacked up - piling more requests
+   * onto an endpoint that was already struggling (#1256).
+   */
+  async runPeriodicSync(name, sync) {
+    if (this.syncsInFlight.has(name)) {
+      this.log.debugWarn('[%s] %s', name, platformLang.syncStillRunning)
+      return
+    }
+    this.syncsInFlight.add(name)
+    try {
+      await sync()
+    } finally {
+      this.syncsInFlight.delete(name)
+    }
+  }
+
   async goveeAWSSync(allDevices = false) {
     const pollList = allDevices ? awsDevices : awsDevicesToPoll
     if (pollList.length === 0) {

package/lib/utils/command-builder.js

@@ -111,7 +111,15 @@ function buildBrightnessCommand(params, context) {
   return {
     awsParams: {
       cmd: 'brightness',
-      data: { val: params.value },
+      // Most devices take 0-254 over AWS, so HomeKit's 0-100 has to be scaled
+      // up. A handful expect 0-100 already and are opted out per device with
+      // `awsBrightnessNoScale`. Dropping this scaling made every AWS device
+      // land at roughly a third of the requested brightness (#1262).
+      data: {
+        val: context.awsBrightnessNoScale
+          ? params.value
+          : Math.round(params.value * 2.54),
+      },
     },
     bleParams: {
       cmd: CMD.BRIGHTNESS,

package/lib/utils/lang-en.js

@@ -130,6 +130,7 @@ export default {
   storageSetupErr: 'Could not setup storage client as',
   storageWriteErr: 'could not save accessory to file as',
   syncFail: 'sync failed as',
+  syncStillRunning: 'skipping this sync as the previous one has not finished yet',
   viaAL: 'via adaptive lighting',
   unknownCommand: 'unknown command in payload received',
 }

package/lib/utils/response-parser.js

@@ -49,8 +49,18 @@ export function parseDeviceUpdate(params, context) {
 
   // BRIGHTNESS
   if (params.state && hasProperty(params.state, 'brightness')) {
-    data.brightness = params.state.brightness
+    // AWS reports 0-254 on most devices, so it has to come back down to
+    // HomeKit's 0-100. Without this every AWS reading of 100 or more clamped
+    // to full brightness (#1262). The other sources already report 0-100.
+    if (params.source === 'AWS') {
+      data.brightness = context.awsBrightnessNoScale
+        ? params.state.brightness
+        : Math.round(params.state.brightness / 2.54)
+    } else {
+      data.brightness = params.state.brightness
+    }
   }
+  // Govee occasionally sends a value outside [0, 100] anyway
   if (hasProperty(data, 'brightness')) {
     data.brightness = Math.max(Math.min(data.brightness, 100), 0)
   }

package/package.json

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