homebridge-ac-freedom 2.3.4 → 2.3.5

package/CHANGELOG.md

@@ -1,5 +1,25 @@
 # Changelog
 
+## 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`
+- **Fix device turns back on after power-off** — `sendPower(false)` no longer sends a separate sleep or fan command; UI reset is local-only, no extra cloud calls
+- **Fix fan not resetting on device** — `sendPower(false)` now includes `fanSpeed=0` in the same API call, so the device restores AUTO fan on next power-on
+- **Fix stale poll overriding command** — poll is suppressed for 5 s after any successful user command; a failed command no longer blocks state refresh
+- **Fix mode command turning device on** — `sendMode` uses `state.power` instead of hardcoded `power=1`; changing mode while AC is off no longer wakes the device
+- **Fix preset switches showing ON when AC is off** — `onGet` and `updateCharacteristics` now gate preset, fan and comfwind values on `state.power`
+- **Fix temperature step** — temperature is snapped in `onSet` and pushed back to HomeKit immediately; corrects slider behaviour even when HomeKit has a cached `minStep`
+- **Fix temperature out-of-range HAP error** — invalid temperature readings from cloud (e.g. 0°C or 100°C when AC is off) are ignored instead of forwarded to HomeKit
+- **Fix unhandled errors in onSet** — all send handlers now use `_trySend()` which catches errors and returns `SERVICE_COMMUNICATION_FAILURE` to HomeKit instead of logging "Unhandled error"
+- **Fix cloud error messages** — cloud API errors now extract `payload.message` instead of dumping the full JSON response
+- **Fix `connection: "cloud"` trying local** — local connection is only attempted when `connection` is explicitly `"hybrid"`
+- **Fix service tile order** — HomeKit climate card always shows tiles in the correct order (Fan → Sleep → Display → Health → Clean → Eco → Comf. Wind) regardless of when each service was first registered
+- **Fix crash on command failure** — `_trySend` now correctly references `this.platform.api.hap` instead of the undefined `this.api`
+- **Fix malformed device cookie** — cloud API now throws an actionable error instead of crashing with a JSON parse exception
+- **Remove Temperature Step and Poll Interval settings** — both are hardcoded (1 °C step, 30 s poll); removed from Config UI and schema to prevent misconfiguration
+- **Cleaner startup log** — reduced to one line per device; removed redundant "Restoring / Updating accessory" messages
+- **Code cleanup** — `LOCAL_MODE_MAP` and `FAN_REMAP` extracted as module-level constants; cloud API default params moved to `DEVICE_PARAMS` constant
+
 ## 2.3.4
 
 - **UI: badge colors** — Cloud (teal), Hybrid (green), Local (yellow) — clearly distinct from each other

package/config.schema.json

@@ -72,9 +72,9 @@
               "type": "object",
               "properties": {
                 "sleep":  { "title": "Sleep Mode",              "type": "boolean", "default": true },
-                "health": { "title": "Health / Ionizer",        "type": "boolean", "default": true },
-                "eco":    { "title": "Eco / Mildew Prevention", "type": "boolean", "default": true },
-                "clean":  { "title": "Self Clean",              "type": "boolean", "default": true }
+                "health": { "title": "Health / Ionizer",        "type": "boolean", "default": false },
+                "eco":    { "title": "Eco / Mildew Prevention", "type": "boolean", "default": false },
+                "clean":  { "title": "Self Clean",              "type": "boolean", "default": false }
               }
             },
             "showFan": {
@@ -91,19 +91,6 @@
               "title": "Display",
               "type": "boolean",
               "default": true
-            },
-            "pollInterval": {
-              "title": "Poll Interval (seconds)",
-              "type": "integer",
-              "default": 30,
-              "minimum": 5,
-              "maximum": 300
-            },
-            "tempStep": {
-              "title": "Temperature Step",
-              "type": "number",
-              "default": 0.5,
-              "enum": [0.5, 1]
             }
           }
         }
@@ -172,15 +159,7 @@
                 { "key": "devices[].showComfWind" },
                 { "key": "devices[].showDisplay" }
               ]
-            },
-            {
-              "key": "devices[].tempStep",
-              "titleMap": {
-                "0.5": "0.5 °C",
-                "1":   "1 °C"
-              }
-            },
-            { "key": "devices[].pollInterval" }
+            }
           ]
         }
       ]

package/homebridge-ui/public/index.html

@@ -224,7 +224,6 @@
     fan: 'Fan',                  sleep: 'Sleep',            display: 'Display',
     health: 'Health',            clean: 'Clean',            eco: 'Eco',
     comfWind: 'Comf. Wind',
-    tempStep: 'Temperature Step', pollInterval: 'Poll Interval',
     addDevice: 'Add Device',     remove: 'Remove',
     hybrid: 'Hybrid',            cloudCredentials: 'Cloud Credentials',
     fetching: 'Fetching…',       devicesFound: 'devices found',
@@ -276,32 +275,6 @@
     `;
   }
 
-  function settingsRow(d, i) {
-    const step = d.tempStep !== undefined ? d.tempStep : 0.5;
-    const poll = d.pollInterval || 30;
-    return `
-      <div class="fieldset" style="margin-bottom:12px">
-        <div class="row-2">
-          <div class="fg" style="margin-bottom:0">
-            <label>${t.tempStep||'Temperature Step'}</label>
-            <select class="d-field" data-i="${i}" data-f="tempStep">
-              <option value="0.5" ${step==0.5?'selected':''}>0.5 °C</option>
-              <option value="1"   ${step==1  ?'selected':''}>1.0 °C</option>
-            </select>
-          </div>
-          <div class="fg" style="margin-bottom:0">
-            <label>${t.pollInterval||'Poll Interval'}</label>
-            <select class="d-field" data-i="${i}" data-f="pollInterval">
-              <option value="5"  ${poll==5 ?'selected':''}>5 seconds</option>
-              <option value="10" ${poll==10?'selected':''}>10 seconds</option>
-              <option value="20" ${poll==20?'selected':''}>20 seconds</option>
-              <option value="30" ${poll==30?'selected':''}>30 seconds</option>
-              <option value="60" ${poll==60?'selected':''}>60 seconds</option>
-            </select>
-          </div>
-        </div>
-      </div>`;
-  }
 
   /* ── Render cloud device card ─────────────────────────────── */
   function renderCloudDevice(d, i) {
@@ -310,8 +283,6 @@
     const conn      = (d.connection === 'local' || !d.connection) ? 'cloud' : d.connection;
     const isHybrid  = conn === 'hybrid';
     const local     = d.local || {};
-    const step      = d.tempStep !== undefined ? d.tempStep : 0.5;
-    const poll      = d.pollInterval || 30;
     const cardClass = isHybrid ? 'hybrid-card' : '';
     const badge     = isHybrid ? (t.hybrid||'Hybrid') : (t.cloud||'Cloud');
 
@@ -371,7 +342,6 @@
             </div>
           </div>
 
-          ${settingsRow(d, i)}
           ${featToggles(d, i)}
         </div>
       </div>`;
@@ -382,8 +352,6 @@
     const isOpen = expanded.has(i);
     const name   = d.name || `Local #${li + 1}`;
     const local  = d.local || {};
-    const step   = d.tempStep !== undefined ? d.tempStep : 0.5;
-    const poll   = d.pollInterval || 30;
 
     return `
       <div class="device-card" id="dev-card-${i}">
@@ -421,7 +389,6 @@
             </div>
           </div>
 
-          ${settingsRow(d, i)}
           ${featToggles(d, i)}
         </div>
       </div>`;
@@ -572,7 +539,6 @@
               local:      { ip: '', mac: '' },
               presets:    { sleep: true, health: false, eco: false, clean: false },
               showFan: true, showComfWind: false, showDisplay: false,
-              tempStep: 0.5, pollInterval: 30,
             });
             expanded.add(config.devices.length - 1);
             added++;
@@ -602,7 +568,6 @@
         local: { ip: '', mac: '' },
         presets: { sleep: true, health: false, eco: false, clean: false },
         showFan: true, showComfWind: false, showDisplay: false,
-        tempStep: 0.5, pollInterval: 30,
       });
       expanded.add(i);
       render(); save();
@@ -634,13 +599,11 @@
       });
     });
 
-    /* Device-level fields (name, tempStep, pollInterval) */
+    /* Device-level fields (name) */
     document.querySelectorAll('.d-field').forEach(el => {
       el.addEventListener('change', () => {
         const i = +el.dataset.i, f = el.dataset.f;
-        config.devices[i][f] = (f === 'tempStep')    ? (parseFloat(el.value) || 0.5)
-          : (f === 'pollInterval') ? (parseInt(el.value)   || 30)
-          : el.value;
+        config.devices[i][f] = el.value;
         save();
       });
       el.addEventListener('input', () => {

package/package.json

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

@@ -39,9 +39,17 @@ const CLOUD_MODE = { AUTO: 4, COOL: 0, HEAT: 1, DRY: 2, FAN: 3 };
 // Conversion tables applied in pollLocal / sendFanSpeed
 const FAN_SPEED = { AUTO: 0, LOW: 1, MEDIUM: 2, HIGH: 3, TURBO: 4 };
 
-// Local ↔ canonical (cloud) fan speed conversion
-const LOCAL_TO_CLOUD_FAN = { 0: 0, 1: 3, 2: 2, 3: 1, 4: 4 };
-const CLOUD_TO_LOCAL_FAN = { 0: 0, 1: 3, 2: 2, 3: 1, 4: 4 };
+// Local ↔ cloud fan speed remap (mapping is self-inverse: 1↔3, others unchanged)
+const FAN_REMAP = { 0: 0, 1: 3, 2: 2, 3: 1, 4: 4 };
+
+// Cloud mode → local protocol mode byte (AUTO=0 avoids 3-bit overflow at value 8)
+const LOCAL_MODE_MAP = {
+  [4]: 0, // AUTO
+  [0]: 1, // COOL
+  [1]: 4, // HEAT
+  [2]: 2, // DRY
+  [3]: 6, // FAN
+};
 
 class AcFreedomAccessory {
   constructor(platform, accessory, config, deviceApi) {
@@ -84,9 +92,10 @@ class AcFreedomAccessory {
     this.setupPresetSwitches();
     this.setupComfWindSwitch();
     this.setupDisplaySwitch();
+    this._reorderServices();
 
     // Start polling
-    const interval = (config.pollInterval || 30) * 1000;
+    const interval = 30000;
     this.pollTimer = setInterval(() => this.pollState(), interval);
     this.pollState();
   }
@@ -122,9 +131,13 @@ class AcFreedomAccessory {
     this.heaterCooler.getCharacteristic(C.Active)
       .onGet(() => this.state.power ? C.Active.ACTIVE : C.Active.INACTIVE)
       .onSet(async (value) => {
-        this.state.power = value === C.Active.ACTIVE;
-        await this.sendPower(this.state.power);
-        if (!this.state.power) { this.resetFanToAuto(); this.resetSleep(); }
+        const newPower = value === C.Active.ACTIVE;
+        if (newPower === this.state.power) return;
+        this.state.power = newPower;
+        if (!newPower) this._powerOffReset();
+        this._powerJustChanged = true;
+        setTimeout(() => { this._powerJustChanged = false; }, 300);
+        await this._trySend(() => this.sendPower(newPower));
       });
 
     this.heaterCooler.getCharacteristic(C.CurrentHeaterCoolerState)
@@ -160,30 +173,43 @@ class AcFreedomAccessory {
           default:                             mode = CLOUD_MODE.AUTO; break;
         }
         this.state.mode = mode;
-        await this.sendMode(mode);
-        this.resetFanToAuto(true);
-        this.resetSleep();
+        // Wait for Active.onSet to fire first — HomeKit sends both in the same
+        // batch but order is not guaranteed; 200 ms covers the race window.
+        await new Promise(r => setTimeout(r, 200));
+        if (this._powerJustChanged) return;
+        await this._trySend(() => this.sendMode(mode));
       });
 
     this.heaterCooler.getCharacteristic(C.CurrentTemperature)
       .setProps({ minValue: -20, maxValue: 60 })
       .onGet(() => this.state.currentTemp);
 
-    const step = this.config.tempStep || 0.5;
+    const snapTemp = (v) => Math.round(v);
+
     this.heaterCooler.getCharacteristic(C.CoolingThresholdTemperature)
-      .setProps({ minValue: 16, maxValue: 32, minStep: step })
-      .onGet(() => this.state.targetTemp)
+      .setProps({ minValue: 16, maxValue: 32, minStep: 1 })
+      .onGet(() => snapTemp(this.state.targetTemp))
       .onSet(async (value) => {
-        this.state.targetTemp = value;
-        await this.sendTemperature(value);
+        const snapped = snapTemp(value);
+        this.state.targetTemp = snapped;
+        if (snapped !== value) {
+          this.heaterCooler.updateCharacteristic(C.CoolingThresholdTemperature, snapped);
+          this.heaterCooler.updateCharacteristic(C.HeatingThresholdTemperature, snapped);
+        }
+        await this._trySend(() => this.sendTemperature(snapped));
       });
 
     this.heaterCooler.getCharacteristic(C.HeatingThresholdTemperature)
-      .setProps({ minValue: 16, maxValue: 32, minStep: step })
-      .onGet(() => this.state.targetTemp)
+      .setProps({ minValue: 16, maxValue: 32, minStep: 1 })
+      .onGet(() => snapTemp(this.state.targetTemp))
       .onSet(async (value) => {
-        this.state.targetTemp = value;
-        await this.sendTemperature(value);
+        const snapped = snapTemp(value);
+        this.state.targetTemp = snapped;
+        if (snapped !== value) {
+          this.heaterCooler.updateCharacteristic(C.CoolingThresholdTemperature, snapped);
+          this.heaterCooler.updateCharacteristic(C.HeatingThresholdTemperature, snapped);
+        }
+        await this._trySend(() => this.sendTemperature(snapped));
       });
 
     this.heaterCooler.getCharacteristic(C.SwingMode)
@@ -194,7 +220,7 @@ class AcFreedomAccessory {
         const on = value === C.SwingMode.SWING_ENABLED;
         this.state.swingV = on;
         this.state.swingH = on;
-        await this.sendSwing(on, on);
+        await this._trySend(() => this.sendSwing(on, on));
       });
   }
 
@@ -227,18 +253,22 @@ class AcFreedomAccessory {
     this.fanService.getCharacteristic(C.Active)
       .onGet(() => this.state.power ? C.Active.ACTIVE : C.Active.INACTIVE)
       .onSet(async (value) => {
-        this.state.power = value === C.Active.ACTIVE;
-        await this.sendPower(this.state.power);
-        if (!this.state.power) { this.resetFanToAuto(); this.resetSleep(); }
+        const newPower = value === C.Active.ACTIVE;
+        if (newPower === this.state.power) return;
+        this.state.power = newPower;
+        if (!newPower) this._powerOffReset();
+        this._powerJustChanged = true;
+        setTimeout(() => { this._powerJustChanged = false; }, 300);
+        await this._trySend(() => this.sendPower(newPower));
       });
 
     // 0=Auto  25=Low  50=Medium  75=High  100=Turbo
     this.fanService.getCharacteristic(C.RotationSpeed)
       .setProps({ minValue: 0, maxValue: 100, minStep: 25 })
-      .onGet(() => this.fanSpeedToPercent(this.state.fanSpeed))
+      .onGet(() => this.state.power ? this.fanSpeedToPercent(this.state.fanSpeed) : 0)
       .onSet(async (value) => {
         this.state.fanSpeed = this.percentToFanSpeed(value);
-        await this.sendFanSpeed(this.state.fanSpeed);
+        await this._trySend(() => this.sendFanSpeed(this.state.fanSpeed));
       });
 
     this.heaterCooler.addLinkedService(this.fanService);
@@ -262,7 +292,7 @@ class AcFreedomAccessory {
       svc.setCharacteristic(this.Characteristic.Name, cfg.label);
 
       svc.getCharacteristic(this.Characteristic.On)
-        .onGet(() => this.state[key])
+        .onGet(() => this.state.power && this.state[key])
         .onSet(async (value) => {
           if (value) {
             for (const otherKey of Object.keys(this.presetConfigs)) {
@@ -270,7 +300,7 @@ class AcFreedomAccessory {
             }
           }
           this.state[key] = value;
-          await this.sendPreset(key, value);
+          await this._trySend(() => this.sendPreset(key, value));
           this.refreshPresetSwitches(key);
         });
 
@@ -301,10 +331,10 @@ class AcFreedomAccessory {
     this.comfWindSwitch.setCharacteristic(this.Characteristic.Name, 'Comfortable Wind');
 
     this.comfWindSwitch.getCharacteristic(this.Characteristic.On)
-      .onGet(() => this.state.comfwind)
+      .onGet(() => this.state.power && this.state.comfwind)
       .onSet(async (value) => {
         this.state.comfwind = value;
-        await this.sendComfWind(value);
+        await this._trySend(() => this.sendComfWind(value));
       });
 
     this.heaterCooler.addLinkedService(this.comfWindSwitch);
@@ -327,28 +357,22 @@ class AcFreedomAccessory {
       .onGet(() => this.state.display)
       .onSet(async (value) => {
         this.state.display = value;
-        await this.sendDisplay(value);
+        await this._trySend(() => this.sendDisplay(value));
       });
 
     this.heaterCooler.addLinkedService(this.displaySwitch);
   }
 
-  // ── Fan speed helpers ──────────────────────────────────────────
-  resetFanToAuto(sendCommand = false) {
+  // ── Power-off UI reset ─────────────────────────────────────────
+  // Updates HomeKit display only. No cloud commands — avoids interfering
+  // with the power-off command and prevents spurious device wake-up.
+  _powerOffReset() {
     this.state.fanSpeed = FAN_SPEED.AUTO;
-    if (this.fanService) {
+    this.state.sleep    = false;
+    if (this.fanService)
       this.fanService.updateCharacteristic(this.Characteristic.RotationSpeed, 0);
-    }
-    if (sendCommand) this.sendFanSpeed(FAN_SPEED.AUTO).catch(() => {});
-  }
-
-  resetSleep() {
-    if (!this.state.sleep) return;
-    this.state.sleep = false;
-    if (this.presetSwitches?.sleep) {
+    if (this.presetSwitches?.sleep)
       this.presetSwitches.sleep.updateCharacteristic(this.Characteristic.On, false);
-    }
-    this.sendPreset('sleep', false).catch(() => {});
   }
 
   // Fan speed ↔ HomeKit percent (canonical cloud numbering)
@@ -368,6 +392,7 @@ class AcFreedomAccessory {
 
   // ── Poll state ─────────────────────────────────────────────────
   async pollState() {
+    if (this._pollLockUntil && Date.now() < this._pollLockUntil) return;
     try {
       const t = this.deviceApi.type;
       if (t === 'local') {
@@ -403,8 +428,10 @@ class AcFreedomAccessory {
 
       this.state.power       = !!params[CLOUD.POWER];
       this.state.mode        = params[CLOUD.MODE]       ?? CLOUD_MODE.AUTO;
-      this.state.targetTemp  = (params[CLOUD.TEMP_TARGET]  ?? 240) / 10;
-      this.state.currentTemp = (params[CLOUD.TEMP_AMBIENT] ?? 240) / 10;
+      const rawTarget  = (params[CLOUD.TEMP_TARGET]  ?? 240) / 10;
+      const rawAmbient = (params[CLOUD.TEMP_AMBIENT] ?? 240) / 10;
+      if (rawTarget  >= 16 && rawTarget  <= 32) this.state.targetTemp  = rawTarget;
+      if (rawAmbient >= 0  && rawAmbient <= 60) this.state.currentTemp = rawAmbient;
       this.state.fanSpeed    = params[CLOUD.FAN_SPEED]  ?? FAN_SPEED.AUTO;
       this.state.swingV      = !!params[CLOUD.SWING_V];
       this.state.swingH      = !!params[CLOUD.SWING_H];
@@ -434,7 +461,7 @@ class AcFreedomAccessory {
     this.state.targetTemp  = s.temperature;
     this.state.currentTemp = s.ambientTemp;
     // Normalise local fan speed → canonical cloud numbering
-    this.state.fanSpeed    = LOCAL_TO_CLOUD_FAN[s.fanSpeed] ?? FAN_SPEED.AUTO;
+    this.state.fanSpeed    = FAN_REMAP[s.fanSpeed] ?? FAN_SPEED.AUTO;
     this.state.swingV      = s.verticalFixation === 7;
     this.state.swingH      = s.horizontalFixation === 7;
     this.state.sleep       = !!s.sleep;
@@ -466,9 +493,10 @@ class AcFreedomAccessory {
       this.heaterCooler.updateCharacteristic(C.CurrentHeaterCoolerState, hcState);
     }
 
+    const snapped = Math.round(this.state.targetTemp);
     this.heaterCooler.updateCharacteristic(C.CurrentTemperature,           this.state.currentTemp);
-    this.heaterCooler.updateCharacteristic(C.CoolingThresholdTemperature,  this.state.targetTemp);
-    this.heaterCooler.updateCharacteristic(C.HeatingThresholdTemperature,  this.state.targetTemp);
+    this.heaterCooler.updateCharacteristic(C.CoolingThresholdTemperature,  snapped);
+    this.heaterCooler.updateCharacteristic(C.HeatingThresholdTemperature,  snapped);
     this.heaterCooler.updateCharacteristic(C.SwingMode,
       (this.state.swingV || this.state.swingH) ? C.SwingMode.SWING_ENABLED : C.SwingMode.SWING_DISABLED);
 
@@ -480,11 +508,11 @@ class AcFreedomAccessory {
     }
 
     for (const [key, svc] of Object.entries(this.presetSwitches || {})) {
-      svc.updateCharacteristic(C.On, this.state[key]);
+      svc.updateCharacteristic(C.On, this.state.power && this.state[key]);
     }
 
     if (this.comfWindSwitch) {
-      this.comfWindSwitch.updateCharacteristic(C.On, this.state.comfwind);
+      this.comfWindSwitch.updateCharacteristic(C.On, this.state.power && this.state.comfwind);
     }
     if (this.displaySwitch) {
       this.displaySwitch.updateCharacteristic(C.On, this.state.display);
@@ -512,28 +540,50 @@ class AcFreedomAccessory {
     return cloudFn(); // legacy 'cloud' type
   }
 
+  // Wraps a send call for use inside onSet handlers.
+  // Suppresses poll for 5 s after any command to avoid stale cloud state overriding UI.
+  // Logs the error and throws HapStatusError so HomeKit shows failure correctly.
+  async _trySend(fn) {
+    try {
+      await fn();
+      this._pollLockUntil = Date.now() + 5000;
+    } catch (err) {
+      this.log.warn('%s: command failed: %s', this.config.name, err.message);
+      throw new this.platform.api.hap.HapStatusError(
+        this.platform.api.hap.HAPStatus.SERVICE_COMMUNICATION_FAILURE,
+      );
+    }
+  }
+
   // ── Send commands ──────────────────────────────────────────────
   async sendPower(on) {
     return this._send(
-      () => { const a = this._localApi; a.state.power = on ? 1 : 0; return a.setState(); },
-      () => this.cloudSet({ [CLOUD.POWER]: on ? 1 : 0 }),
+      () => {
+        const a = this._localApi;
+        a.state.power = on ? 1 : 0;
+        if (on)  a.state.mode     = LOCAL_MODE_MAP[this.state.mode] ?? 0;
+        if (!on) a.state.fanSpeed = 0;
+        return a.setState();
+      },
+      () => {
+        const p = { [CLOUD.POWER]: on ? 1 : 0 };
+        if (on)  p[CLOUD.MODE]      = this.state.mode;
+        if (!on) p[CLOUD.FAN_SPEED] = 0;
+        return this.cloudSet(p);
+      },
     );
   }
 
   async sendMode(mode) {
-    // AUTO is 0 in local protocol (mode=8 would overflow the 3-bit field)
-    const localModeMap = {
-      [CLOUD_MODE.AUTO]: 0, [CLOUD_MODE.COOL]: 1,
-      [CLOUD_MODE.HEAT]: 4, [CLOUD_MODE.DRY]: 2, [CLOUD_MODE.FAN]: 6,
-    };
+    const pwr = this.state.power ? 1 : 0;
     return this._send(
       () => {
         const a = this._localApi;
-        a.state.power = 1;
-        a.state.mode  = localModeMap[mode] ?? 0;
+        a.state.power = pwr;
+        a.state.mode  = LOCAL_MODE_MAP[mode] ?? 0;
         return a.setState();
       },
-      () => this.cloudSet({ [CLOUD.POWER]: 1, [CLOUD.MODE]: mode }),
+      () => this.cloudSet({ [CLOUD.POWER]: pwr, [CLOUD.MODE]: mode }),
     );
   }
 
@@ -546,7 +596,7 @@ class AcFreedomAccessory {
 
   async sendFanSpeed(speed) {
     // speed is canonical cloud numbering; convert for local
-    const localSpeed = CLOUD_TO_LOCAL_FAN[speed] ?? 0;
+    const localSpeed = FAN_REMAP[speed] ?? 0;
     return this._send(
       () => {
         const a = this._localApi;
@@ -609,6 +659,40 @@ class AcFreedomAccessory {
     await this._cloudApi.setDeviceParams(this._cloudDevice, params);
   }
 
+  // ── Service ordering ───────────────────────────────────────────
+  // Sorts accessory.services so HomeKit always shows tiles in the
+  // correct order regardless of when each service was first added.
+  _reorderServices() {
+    const rank = (svc) => {
+      const s = svc.subtype;
+      if (svc.UUID === this.Service.AccessoryInformation.UUID) return 0;
+      if (svc.UUID === this.Service.HeaterCooler.UUID)         return 1;
+      if (s === 'fan')      return 2;
+      if (s === 'sleep')    return 3;
+      if (s === 'display')  return 4;
+      if (s === 'health')   return 5;
+      if (s === 'clean')    return 6;
+      if (s === 'eco')      return 7;
+      if (s === 'comfwind') return 8;
+      return 99;
+    };
+    this.accessory.services.sort((a, b) => rank(a) - rank(b));
+
+    // Rebuild linkedServices explicitly in the desired order so the HAP
+    // "linked" array is sent correctly regardless of IID values.
+    const linked = [
+      this.fanService,
+      this.presetSwitches?.sleep,
+      this.displaySwitch,
+      this.presetSwitches?.health,
+      this.presetSwitches?.clean,
+      this.presetSwitches?.eco,
+      this.comfWindSwitch,
+    ].filter(Boolean);
+    this.heaterCooler.linkedServices.length = 0;
+    for (const svc of linked) this.heaterCooler.linkedServices.push(svc);
+  }
+
   // ── Cleanup ────────────────────────────────────────────────────
   destroy() {
     if (this.pollTimer) { clearInterval(this.pollTimer); this.pollTimer = null; }

package/src/cloud-api.js

@@ -80,6 +80,13 @@ function httpRequest(url, options, body) {
   });
 }
 
+// Default device parameter keys fetched on every poll
+const DEVICE_PARAMS = [
+  'pwr', 'ac_mode', 'temp', 'envtemp', 'ac_mark',
+  'ac_vdir', 'ac_hdir', 'ac_slp', 'ac_health',
+  'mldprf', 'ac_clean', 'comfwind', 'scrdisp',
+];
+
 // ── AuxCloudAPI ──────────────────────────────────────────────────
 class AuxCloudAPI {
   constructor(region = 'eu') {
@@ -139,7 +146,7 @@ class AuxCloudAPI {
       this.userid = result.userid;
       return true;
     }
-    throw new Error(`Login failed: ${JSON.stringify(result)}`);
+    throw new Error(`Login failed: ${result.msg || result.status || 'unknown error'}`);
   }
 
   // ── Families ───────────────────────────────────────────────────
@@ -148,7 +155,7 @@ class AuxCloudAPI {
       headers: this._headers(),
     });
     if (result.status === 0) return result.data.familyList || [];
-    throw new Error(`Get families failed: ${JSON.stringify(result)}`);
+    throw new Error(`Get families failed: ${result.msg || result.status || 'unknown error'}`);
   }
 
   // ── Devices ────────────────────────────────────────────────────
@@ -159,7 +166,7 @@ class AuxCloudAPI {
     });
 
     if (result.status !== 0) {
-      throw new Error(`Get devices failed: ${JSON.stringify(result)}`);
+      throw new Error(`Get devices failed: ${result.msg || result.status || 'unknown error'}`);
     }
 
     const devices = result.data?.endpoints || [];
@@ -179,7 +186,12 @@ class AuxCloudAPI {
 
   // ── Get / Set device params ────────────────────────────────────
   async _actDeviceParams(device, act, params = [], vals = []) {
-    const cookieRaw = JSON.parse(Buffer.from(device.cookie, 'base64').toString());
+    let cookieRaw;
+    try {
+      cookieRaw = JSON.parse(Buffer.from(device.cookie, 'base64').toString());
+    } catch {
+      throw new Error('Device cookie is malformed — re-fetch devices in plugin settings');
+    }
     const mappedCookie = Buffer.from(JSON.stringify({
       device: {
         id: cookieRaw.terminalid,
@@ -239,17 +251,12 @@ class AuxCloudAPI {
       return out;
     }
 
-    throw new Error(`Device param ${act} failed: ${JSON.stringify(result)}`);
+    const errPayload = result?.event?.payload || {};
+    const reason = errPayload.message || errPayload.type || result?.event?.header?.name || 'unknown error';
+    throw new Error(`Device unreachable: ${reason}`);
   }
 
-  async getDeviceParams(device, params) {
-    if (!params) {
-      params = [
-        'pwr', 'ac_mode', 'temp', 'envtemp', 'ac_mark',
-        'ac_vdir', 'ac_hdir', 'ac_slp', 'ac_health',
-        'mldprf', 'ac_clean', 'comfwind', 'scrdisp',
-      ];
-    }
+  async getDeviceParams(device, params = DEVICE_PARAMS) {
     return this._actDeviceParams(device, 'get', params);
   }
 

package/src/platform.js

@@ -34,13 +34,11 @@ class AcFreedomPlatform {
     if (!api || !config) return;
 
     this.api.on('didFinishLaunching', () => {
-      this.log.info('AC Freedom platform loaded');
       this.discoverDevices().catch(err => this.log.error('Fatal: %s', err.message));
     });
   }
 
   configureAccessory(accessory) {
-    this.log.info('Restoring cached accessory: %s', accessory.displayName);
     this.accessories.set(accessory.UUID, accessory);
   }
 
@@ -72,7 +70,6 @@ class AcFreedomPlatform {
     const api = new AuxCloudAPI(cloudCreds.region || 'eu');
     try {
       await api.login(cloudCreds.email, cloudCreds.password);
-      this.log.info('Cloud login successful: %s', cloudCreds.email);
 
       const families = await api.getFamilies();
       const allDevices = [];
@@ -80,7 +77,7 @@ class AcFreedomPlatform {
         const devs = await api.getDevices(fam.familyid);
         allDevices.push(...devs);
       }
-      this.log.info('Found %d cloud device(s)', allDevices.length);
+      this.log.info('Cloud: %s — %d device(s)', cloudCreds.email, allDevices.length);
       return { api, devices: allDevices };
     } catch (err) {
       this.log.error('Cloud login failed: %s', err.message);
@@ -114,12 +111,10 @@ class AcFreedomPlatform {
 
     const existingAccessory = this.accessories.get(uuid);
     if (existingAccessory) {
-      this.log.info('Updating existing accessory: %s', deviceConfig.name);
       const instance = new AcFreedomAccessory(this, existingAccessory, deviceConfig, deviceApi);
       this.instances.set(uuid, instance);
       this.api.updatePlatformAccessories([existingAccessory]);
     } else {
-      this.log.info('Adding new accessory: %s', deviceConfig.name);
       const accessory = new this.api.platformAccessory(deviceConfig.name, uuid);
       const instance = new AcFreedomAccessory(this, accessory, deviceConfig, deviceApi);
       this.instances.set(uuid, instance);
@@ -184,40 +179,25 @@ class AcFreedomPlatform {
       }
     }
 
-    // Try local connection (optional)
+    // Try local connection only when explicitly set to hybrid mode
     let localApi = null;
     const localCfg = deviceConfig.local;
-    if (localCfg?.ip && localCfg?.mac) {
+    if (deviceConfig.connection === 'hybrid' && localCfg?.ip && localCfg?.mac) {
       const broadlink = new BroadlinkAcApi(localCfg.ip, localCfg.mac);
       try {
         const connected = await broadlink.connect();
         if (connected) {
-          this.log.info(
-            'Hybrid device %s: local connected at %s (preferred)',
-            deviceConfig.name, localCfg.ip,
-          );
           localApi = broadlink;
         } else {
-          this.log.warn(
-            'Hybrid device %s: local connect failed at %s — using cloud fallback',
-            deviceConfig.name, localCfg.ip,
-          );
+          this.log.warn('%s: local connect failed at %s — using cloud', deviceConfig.name, localCfg.ip);
         }
       } catch (err) {
-        this.log.warn(
-          'Hybrid device %s: local connect error (%s) — using cloud fallback',
-          deviceConfig.name, err.message,
-        );
+        this.log.warn('%s: local connect error (%s) — using cloud', deviceConfig.name, err.message);
       }
     }
 
-    this.log.info(
-      'Hybrid device %s: %s (%s) via %s',
-      deviceConfig.name,
-      cloudDevice.friendlyName || 'AUX AC',
-      cloudDevice.endpointId,
-      localApi ? 'local+cloud' : 'cloud only',
-    );
+    const mode = localApi ? `local+cloud (${localCfg.ip})` : 'cloud';
+    this.log.info('%s: %s', deviceConfig.name, mode);
 
     return {
       type:        'hybrid',
@@ -239,13 +219,13 @@ class AcFreedomPlatform {
     try {
       const connected = await api.connect();
       if (!connected) {
-        this.log.error('Failed to connect to local device at %s', local.ip);
+        this.log.error('%s: local connect failed at %s', config.name, local.ip);
         return null;
       }
-      this.log.info('Local device connected: %s', local.ip);
+      this.log.info('%s: local (%s)', config.name, local.ip);
       return { type: 'local', api };
     } catch (err) {
-      this.log.error('Local connection failed: %s', err.message);
+      this.log.error('%s: local connect error — %s', config.name, err.message);
       return null;
     }
   }