homebridge-ac-freedom 2.3.7 → 2.3.8

package/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 2.3.8
+
+- **Fix Fetch stuck on "Fetching…"** (#2) — the config UI Fetch button called `getDeviceParams` for every discovered device (each a separate HTTP call with a 15 s timeout), causing the total request time to exceed the UI framework timeout and leaving the button permanently stuck. Device param fetching is now skipped during the UI fetch — only the device name and ID are needed at that point.
+
 ## 2.3.7
 
 - **Fix inverted swing mode on local connection** (#1) — local Broadlink protocol uses fixation `0` = swinging and `7` = fixed; the plugin had it reversed, so the HomeKit swing toggle showed the opposite of the actual louvre state. Cloud connection was unaffected.

package/homebridge-ui/server.js

@@ -36,7 +36,7 @@ class AcFreedomUiServer extends HomebridgePluginUiServer {
     const devices  = [];
 
     for (const fam of families) {
-      const devs = await api.getDevices(fam.familyid);
+      const devs = await api.getDevices(fam.familyid, { fetchParams: false });
       devices.push(...devs);
     }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "homebridge-ac-freedom",
-  "version": "2.3.7",
+  "version": "2.3.8",
   "displayName": "Homebridge AC Freedom",
   "description": "Homebridge plugin for AUX air conditioners via Broadlink (local UDP) and AUX Cloud API",
   "main": "index.js",

package/src/cloud-api.js

@@ -159,7 +159,7 @@ class AuxCloudAPI {
   }
 
   // ── Devices ────────────────────────────────────────────────────
-  async getDevices(familyId) {
+  async getDevices(familyId, { fetchParams = true } = {}) {
     const result = await this._request('POST', 'appsync/group/dev/query?action=select', {
       headers: this._headers({ familyid: familyId }),
       body: '{"pids":[]}',
@@ -171,13 +171,15 @@ class AuxCloudAPI {
 
     const devices = result.data?.endpoints || [];
 
-    for (const dev of devices) {
-      dev.params = {};
-      try {
-        const params = await this.getDeviceParams(dev);
-        if (params) dev.params = params;
-      } catch {
-        // Ignore param fetch errors on initial load
+    if (fetchParams) {
+      for (const dev of devices) {
+        dev.params = {};
+        try {
+          const params = await this.getDeviceParams(dev);
+          if (params) dev.params = params;
+        } catch {
+          // Ignore param fetch errors on initial load
+        }
       }
     }