homebridge-ac-freedom 2.3.6 → 2.3.8

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # 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.
+
 ## 2.3.6
 
 - **Config UI: feature toggles redesigned** — checkboxes removed; each feature is now a colour-coded pill: bright green when enabled, muted grey when disabled

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.6",
+  "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/ac-accessory.js

@@ -462,8 +462,9 @@ class AcFreedomAccessory {
     this.state.currentTemp = s.ambientTemp;
     // Normalise local fan speed → canonical cloud numbering
     this.state.fanSpeed    = FAN_REMAP[s.fanSpeed] ?? FAN_SPEED.AUTO;
-    this.state.swingV      = s.verticalFixation === 7;
-    this.state.swingH      = s.horizontalFixation === 7;
+    // Local fixation: 0 = swinging, 7 = fixed
+    this.state.swingV      = s.verticalFixation === 0;
+    this.state.swingH      = s.horizontalFixation === 0;
     this.state.sleep       = !!s.sleep;
     this.state.health      = !!s.health;
     this.state.eco         = !!s.mildew;
@@ -613,8 +614,9 @@ class AcFreedomAccessory {
     return this._send(
       () => {
         const a = this._localApi;
-        a.state.verticalFixation   = v ? 7 : 0;
-        a.state.horizontalFixation = h ? 7 : 0;
+        // Local fixation: 0 = swinging, 7 = fixed
+        a.state.verticalFixation   = v ? 0 : 7;
+        a.state.horizontalFixation = h ? 0 : 7;
         return a.setState();
       },
       () => this.cloudSet({ [CLOUD.SWING_V]: v ? 1 : 0, [CLOUD.SWING_H]: h ? 1 : 0 }),

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
+        }
       }
     }