homebridge-ac-freedom 2.3.5 → 2.3.7

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## 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
+
 ## 2.3.5
 
 - **Fix double beep on power-on** — `TargetHeaterCoolerState` command is suppressed when fired within 200 ms of a power command; mode is already included in `sendPower`

package/README.md

@@ -31,13 +31,13 @@ A [Homebridge](https://homebridge.io) plugin for controlling **AUX-based air con
   - Self Clean
   - Comfortable Wind
 - **Display** switch — LED display on/off
-- **Auto-reset** — fan and sleep reset to default when AC turns on/off or mode changes
-- **Hybrid** — single cloud login at platform level; optional IP + MAC per device for local Broadlink UDP (local preferred, cloud fallback)
-- **Local** — Direct Broadlink UDP only, no internet required
+- **Auto-reset** — fan resets to Auto and Sleep turns off when the AC is powered off
+- **Consistent tile order** — Fan is always first, followed by Sleep, Display, Health, Clean, Eco, Comf. Wind
+- **Hybrid** — single cloud login at platform level; optional IP + MAC per device for direct local Broadlink UDP (local preferred, cloud fallback)
+- **Local** — direct Broadlink UDP only, no internet required
 - **Multi-device** — configure as many ACs as needed
 - **Custom config UI** — built-in Homebridge UI with Fetch Devices button
 - Homebridge v1 & v2 compatible
-- English UI
 
 ---
 
@@ -196,8 +196,6 @@ Required for Hybrid mode. Configured once, shared by all hybrid devices.
 | `name` | string | Yes | — | Device name in HomeKit |
 | `connection` | string | No | `"hybrid"` | `"hybrid"` or `"local"` |
 | `endpointId` | string | No | — | Cloud device ID (auto-detected if empty) |
-| `pollInterval` | integer | No | `30` | Polling interval in seconds (5–300) |
-| `tempStep` | number | No | `0.5` | Temperature step: `0.5` or `1` |
 
 ### Local Settings (`local` object, per device)
 
@@ -218,7 +216,7 @@ Required for Hybrid mode. Configured once, shared by all hybrid devices.
 | `showComfWind` | boolean | `false` | Comfortable Wind switch |
 | `showDisplay` | boolean | `false` | Display (LED) switch |
 
-> **Tip:** For a clean HomeKit appearance, enable **Fan** and only one preset mode — **Sleep** is the recommended choice.
+> **Tip:** For a clean HomeKit appearance, enable only the features you use. **Fan** and **Sleep** are enabled by default — most users need nothing else.
 
 ---
 
@@ -226,13 +224,11 @@ Required for Hybrid mode. Configured once, shared by all hybrid devices.
 
 Once configured, the AC appears as a **HeaterCooler** tile. Tapping it reveals:
 
-- Temperature control with heating/cooling thresholds
+- Temperature control (1 °C steps, 16–32 °C)
 - Mode selector — Auto / Heat / Cool
 - Fan speed slider — 0% Auto · 25% Low · 50% Medium · 75% High · 100% Turbo
 - Swing toggle
-- Preset switches — Sleep, Health, Eco, Clean, Comfortable Wind, Display
-
-All linked services appear as tiles inside the climate card.
+- Linked service tiles in a fixed order: Fan → Sleep → Display → Health → Clean → Eco → Comf. Wind
 
 ---
 
@@ -246,7 +242,7 @@ All linked services appear as tiles inside the climate card.
 | Works remotely | Yes | No |
 | Local fallback | Yes (when IP + MAC configured) | — |
 
-> ¹ **Hybrid + IP/MAC:** if internet is unavailable at startup, the device starts in local-only mode automatically. Full hybrid resumes on the next restart when internet is available.
+> ¹ **Hybrid + IP/MAC:** if internet is unavailable at Homebridge startup, the device starts in local-only mode automatically. Full hybrid resumes on the next restart when internet is available.
 
 ---
 
@@ -258,19 +254,22 @@ All linked services appear as tiles inside the climate card.
 - Check the `region` matches your account region
 - Logging into the AC Freedom app may invalidate the plugin session — restart Homebridge after using the app
 
-**Device not found**
+**Device not found after Fetch**
 - Leave `endpointId` empty for auto-detection (first discovered device is used)
 - Use the **Fetch** button in the Homebridge UI to discover and auto-fill your device IDs
+- If the device is missing from the cloud list, power cycle the AC and try again
 
-**"Server busy" errors**
-- These are transient and automatically suppressed
-- If persistent, increase `pollInterval` to 60 or higher
-
-**AC not responding**
+**AC not responding / "Service Communication Failure" in HomeKit**
 - Check the AC is powered on and connected to Wi-Fi
-- Local mode: Homebridge and AC must be on the same network
+- Local mode: Homebridge and the AC must be on the same network
 - Try restarting Homebridge
 
+**"Server busy" errors in logs**
+- These are transient cloud API errors and are automatically suppressed — no action needed
+
+**Tile order is wrong in HomeKit**
+- Remove the accessory from the Home app (the individual tile, not the whole bridge), then restart Homebridge — it will re-appear with the correct order
+
 ---
 
 ## Support

package/homebridge-ui/public/index.html

@@ -154,17 +154,17 @@
   .toggle-row { display: flex; gap: 6px; }
   .toggle-item {
     flex: 1; min-width: 0;
-    display: flex; align-items: center; justify-content: center; gap: 5px;
+    display: flex; align-items: center; justify-content: center;
     padding: 7px 4px; border-radius: 7px;
     border: 1px solid var(--border); background: var(--card-bg);
-    cursor: pointer; font-size: 11px; font-weight: 500; color: var(--text);
-    white-space: nowrap; transition: border-color .15s, background .15s;
+    cursor: pointer; font-size: 11px; font-weight: 500; color: var(--text-muted);
+    white-space: nowrap; transition: border-color .15s, background .15s, color .15s;
+    user-select: none;
   }
-  .toggle-item:hover { border-color: var(--accent); }
-  .toggle-item input[type="checkbox"] {
-    width: 13px; height: 13px; accent-color: var(--accent); flex-shrink: 0; cursor: pointer;
+  .toggle-item:not(.on):hover { border-color: #9ca3af; }
+  .toggle-item.on {
+    background: #16a34a; border-color: #16a34a; color: #fff; font-weight: 600;
   }
-  .toggle-item.checked { border-color: var(--accent-border); background: var(--accent-light); }
 
   /* ── Fieldset ── */
   .fieldset {
@@ -253,23 +253,22 @@
   function featToggles(d, i) {
     const presets = d.presets || {};
     const feats = [
-      { key: 'showFan',      label: t.fan||'Fan',             val: bool(d.showFan) },
-      { key: 'sleep',        label: t.sleep||'Sleep',          val: bool(presets.sleep),   preset: true },
-      { key: 'showDisplay',  label: t.display||'Display',      val: bool(d.showDisplay) },
-      { key: 'health',       label: t.health||'Health',        val: bool(presets.health),  preset: true },
-      { key: 'clean',        label: t.clean||'Clean',          val: bool(presets.clean),   preset: true },
-      { key: 'eco',          label: t.eco||'Eco',              val: bool(presets.eco),     preset: true },
-      { key: 'showComfWind', label: t.comfWind||'Comf. Wind',  val: bool(d.showComfWind) },
+      { key: 'showFan',      label: t.fan||'Fan',            val: bool(d.showFan) },
+      { key: 'sleep',        label: t.sleep||'Sleep',         val: bool(presets.sleep),   preset: true },
+      { key: 'showDisplay',  label: t.display||'Display',     val: bool(d.showDisplay) },
+      { key: 'health',       label: t.health||'Health',       val: bool(presets.health),  preset: true },
+      { key: 'clean',        label: t.clean||'Clean',         val: bool(presets.clean),   preset: true },
+      { key: 'eco',          label: t.eco||'Eco',             val: bool(presets.eco),     preset: true },
+      { key: 'showComfWind', label: t.comfWind||'Comf. Wind', val: bool(d.showComfWind) },
     ];
+
     return `
     <div class="toggle-row">
       ${feats.map(f => `
-        <label class="toggle-item ${f.val ? 'checked' : ''}">
-          <input type="checkbox" class="feat-cb"
-            data-i="${i}" data-k="${f.key}" data-preset="${f.preset||false}"
-            ${f.val ? 'checked' : ''}>
+        <div class="toggle-item ${f.val ? 'on' : ''}"
+          data-i="${i}" data-k="${f.key}" data-preset="${f.preset||false}">
           ${f.label}
-        </label>
+        </div>
       `).join('')}
     </div>
     `;
@@ -624,19 +623,18 @@
       });
     });
 
-    /* Feature checkboxes */
-    document.querySelectorAll('.feat-cb').forEach(cb => {
-      cb.closest('.toggle-item').classList.toggle('checked', cb.checked);
-      cb.addEventListener('change', () => {
-        const i = +cb.dataset.i, k = cb.dataset.k;
-        const isPreset = cb.dataset.preset === 'true';
+    /* Feature toggles */
+    document.querySelectorAll('.toggle-item').forEach(el => {
+      el.addEventListener('click', () => {
+        const isOn = el.classList.toggle('on');
+        const i = +el.dataset.i, k = el.dataset.k;
+        const isPreset = el.dataset.preset === 'true';
         if (isPreset) {
           if (!config.devices[i].presets) config.devices[i].presets = {};
-          config.devices[i].presets[k] = cb.checked;
+          config.devices[i].presets[k] = isOn;
         } else {
-          config.devices[i][k] = cb.checked;
+          config.devices[i][k] = isOn;
         }
-        cb.closest('.toggle-item').classList.toggle('checked', cb.checked);
         save();
       });
     });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "homebridge-ac-freedom",
-  "version": "2.3.5",
+  "version": "2.3.7",
   "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 }),