@homebridge-plugins/homebridge-ewelink 12.14.0 → 12.14.1-beta.0

package/CHANGELOG.md

@@ -2,6 +2,13 @@
 
 All notable changes to `@homebridge-plugins/homebridge-ewelink` will be documented in this file.
 
+## v12.14.1 (Pending Release)
+
+- fix(garage-one): send single-outlet payload on switchSCM devices (#774) (@nico81)
+- fix(ws): do not retry sendUpdate on timeout (#775) (@nico81)
+- fix(platform): do not serialize hidden accessories to the cache
+- chore: dependency updates
+
 ## v12.14.0 (2026-05-05)
 
 ### Changed

package/lib/connection/ws.js

@@ -322,7 +322,7 @@ export default class {
     }
   }
 
-  async sendUpdate(json, retries = 0) {
+  async sendUpdate(json) {
     // Generate the payload to send
     const toSend = {
       ...json,
@@ -368,13 +368,11 @@ export default class {
             throw new Error(`${this.lang.wsUnkRes} [${res.error}]`)
         }
       } catch (err) {
-        // Retry up to 3 times on timeout errors only
-        const errMsg = parseError(err)
-        if (retries < 3 && (errMsg.includes('504') || errMsg.includes('timeout') || errMsg.includes('Timeout'))) {
-          await sleep(1000)
-          return this.sendUpdate(json, retries + 1)
-        }
-        throw new Error(errMsg)
+        // Do not retry on timeout: the original command may have already been
+        // applied by the device but the cloud failed to deliver a formal ACK.
+        // Retrying non-idempotent writes (e.g. inching/momentary actuators)
+        // produces additional physical pulses, causing actuators to bounce.
+        throw new Error(parseError(err))
       }
     } else {
       // The web socket isn't currently open so log this

package/lib/device/simulation/garage-one.js

@@ -205,21 +205,33 @@ export default class {
       await sleep(delay)
       this.service.updateCharacteristic(this.hapChar.TargetDoorState, newPos)
       this.service.updateCharacteristic(this.hapChar.CurrentDoorState, newPos + 2)
+      // SCM (single-channel multi-format) devices use the switches[] array
+      // but only expose outlet 0. True dual-channel garage controllers
+      // (e.g. DUALR3) use separate outlets for open/close relay action.
+      const isScm = platformConsts.devices.switchSCM.includes(this.accessory.context.eweUIID)
+        || platformConsts.devices.switchSCMPower.includes(this.accessory.context.eweUIID)
+
       switch (this.setup) {
         case 'switchSingle':
           params.switch = 'on'
           break
         case 'switchMulti':
-          params.switches = [
-            {
-              switch: newPos === 0 ? 'on' : 'off',
-              outlet: 0,
-            },
-            {
-              switch: newPos === 1 ? 'on' : 'off',
-              outlet: 1,
-            },
-          ]
+          if (isScm) {
+            // SCM device: send a single momentary pulse on outlet 0
+            params.switches = [{ switch: 'on', outlet: 0 }]
+          } else {
+            // Dual-channel device with separate open/close relays
+            params.switches = [
+              {
+                switch: newPos === 0 ? 'on' : 'off',
+                outlet: 0,
+              },
+              {
+                switch: newPos === 1 ? 'on' : 'off',
+                outlet: 1,
+              },
+            ]
+          }
           break
         default:
           return

package/lib/platform.js

@@ -1427,8 +1427,7 @@ export default class {
           }
 
           // Update any changes to the sub accessory to the platform
-          this.api.updatePlatformAccessories([subAccessory])
-          devicesInHB.set(subAccessory.UUID, subAccessory)
+          this.saveAccessoryUpdate(subAccessory)
           if (i === 0) {
             accessory = subAccessory
           }
@@ -1567,8 +1566,7 @@ export default class {
           }
 
           // Update any changes to the sub accessory to the platform
-          this.api.updatePlatformAccessories([subAccessory])
-          devicesInHB.set(subAccessory.UUID, subAccessory)
+          this.saveAccessoryUpdate(subAccessory)
 
           // Increment the counter
           rfChlCounter += Object.keys(subDevice.buttons || {}).length
@@ -1826,8 +1824,7 @@ export default class {
       }
 
       // Update any changes to the device into our devicesInHB map
-      this.api.updatePlatformAccessories([accessory])
-      devicesInHB.set(accessory.UUID, accessory)
+      this.saveAccessoryUpdate(accessory)
       if (uiid === 5000) {
         this.log(
           '[%s] %s [%s] [%s].',
@@ -1947,6 +1944,20 @@ export default class {
     }
   }
 
+  saveAccessoryUpdate(accessory) {
+    // Persist any context/characteristic changes to Homebridge, then keep the
+    // accessory in our in-memory map. Hidden accessories are deliberately never
+    // registered (see addAccessory), so they have no associated plugin and would
+    // make Homebridge throw when it serialises its accessory cache to disk
+    // ('Cannot serialize accessory ... - missing associated plugin'). We still
+    // keep them in devicesInHB so lookups (e.g. the SW0 master of a multi-channel
+    // device) continue to work.
+    if (!accessory.context.hidden) {
+      this.api.updatePlatformAccessories([accessory])
+    }
+    devicesInHB.set(accessory.UUID, accessory)
+  }
+
   addExternalAccessory(device, hbDeviceId, category) {
     try {
       // Add the new accessory to Homebridge
@@ -2092,8 +2103,7 @@ export default class {
       // The update is from WS so update the WS online/offline status
       if (device.params.online !== accessory.context.reachableWAN) {
         accessory.context.reachableWAN = device.params.online
-        this.api.updatePlatformAccessories([accessory])
-        devicesInHB.set(accessory.UUID, accessory)
+        this.saveAccessoryUpdate(accessory)
 
         // Flag this true to update the sub accessories later
         reachableChange = true
@@ -2154,8 +2164,7 @@ export default class {
 
       // Update the accessory context if the device is back online or the IP changed
       if (reachableChange) {
-        this.api.updatePlatformAccessories([accessory])
-        devicesInHB.set(accessory.UUID, accessory)
+        this.saveAccessoryUpdate(accessory)
       }
     }
 
@@ -2183,8 +2192,7 @@ export default class {
           }
 
           // Save the sub accessory updates to the platform
-          this.api.updatePlatformAccessories([subAccessory])
-          devicesInHB.set(subAccessory.UUID, subAccessory)
+          this.saveAccessoryUpdate(subAccessory)
         }
       }
     }

package/package.json

@@ -3,7 +3,7 @@
   "displayName": "Homebridge eWeLink",
   "alias": "eWeLink",
   "type": "module",
-  "version": "12.14.0",
+  "version": "12.14.1-beta.0",
   "description": "Homebridge plugin to integrate eWeLink devices into HomeKit.",
   "author": {
     "name": "bwp91",
@@ -61,14 +61,14 @@
     "lint:fix": "npm run lint -- --fix"
   },
   "dependencies": {
-    "@homebridge/plugin-ui-utils": "^2.2.3",
-    "axios": "^1.16.0",
+    "@homebridge/plugin-ui-utils": "^2.2.4",
+    "axios": "^1.17.0",
     "node-persist": "^4.0.4",
-    "p-queue": "^9.2.0",
+    "p-queue": "^9.3.0",
     "websocket-as-promised": "^3.0.1",
-    "ws": "^8.20.0"
+    "ws": "^8.21.0"
   },
   "devDependencies": {
-    "@antfu/eslint-config": "^8.2.0"
+    "@antfu/eslint-config": "^9.0.0"
   }
 }