@homebridge-plugins/homebridge-govee 11.30.0-beta.0 → 11.30.0-beta.2

package/CHANGELOG.md

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

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)
@@ -777,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))
             }
@@ -791,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,
         )
       }
@@ -800,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,
         )
 
@@ -820,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,
         )
       }
@@ -1443,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/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/package.json

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