@homebridge-plugins/homebridge-ecovacs 7.3.0 → 7.4.0

package/CHANGELOG.md

@@ -2,6 +2,24 @@
 
 All notable changes to `@homebridge-plugins/homebridge-ecovacs` will be documented in this file.
 
+## v7.4.0 (2026-04-05)
+
+### Changes
+
+- dependency updates
+- remove support for node v20
+- fix: shallow copy default config/device objects
+- fix: correct poll interval config access
+- fix: remove extra brace in clean update log
+- fix: move accessory null check before first use
+- fix: clear cache after updateValue calls, not before
+- fix: correct error message for predefined area failures
+- fix: guard `refreshIntervals` in `pluginShutdown`
+- fix: use logDebug for `GetChargeState` in `externalReadyUpdate`
+- fix: use object over sparse array for virtual boundaries
+- fix: remove unused `refreshTime` config
+- fix: use `trim()` in `parseError` stack trace cleanup
+
 ## v7.3.0 (2026-02-16)
 
 ### Changes

package/README.md

@@ -3,7 +3,7 @@
 </p>
 <span align="center">
 
-# homebridge-ecovacs
+## homebridge-ecovacs
 
 Homebridge plugin to integrate Ecovacs Deebot or Yeedi devices into HomeKit
 
@@ -27,7 +27,7 @@ Homebridge plugin to integrate Ecovacs Deebot or Yeedi devices into HomeKit
 ### Prerequisites
 
 - To use this plugin, you will need to already have:
-  - [Node](https://nodejs.org): latest version of `v20`, `v22` or `v24` - any other major version is not supported.
+  - [Node](https://nodejs.org): latest version of `v22` or `v24` - any other major version is not supported.
   - [Homebridge](https://homebridge.io): `v1.6` - refer to link for more information and installation instructions.
 
 ### Setup

package/config.schema.json

@@ -700,7 +700,7 @@
       "type": "fieldset",
       "title": "Optional Settings",
       "expandable": true,
-      "items": ["disableDeviceLogging", "refreshTime"]
+      "items": ["disableDeviceLogging"]
     },
     {
       "key": "devices",

package/lib/platform.js

@@ -57,7 +57,7 @@ export default class {
       }
 
       // Apply the user's configuration
-      this.config = platformConsts.defaultConfig
+      this.config = { ...platformConsts.defaultConfig }
       this.applyUserConfig(config)
 
       // Create further variables needed by the plugin
@@ -136,7 +136,7 @@ export default class {
                 logRemove(`${key}.${id}`)
                 return
               }
-              this.deviceConf[id] = platformConsts.defaultDevice
+              this.deviceConf[id] = { ...platformConsts.defaultDevice }
               entries.forEach((subEntry) => {
                 const [k, v] = subEntry
                 switch (k) {
@@ -417,7 +417,7 @@ export default class {
     // A function that is called when the plugin fails to load or Homebridge restarts
     try {
       // Stop the refresh intervals
-      Object.keys(this.refreshIntervals).forEach((id) => {
+      Object.keys(this.refreshIntervals || {}).forEach((id) => {
         clearInterval(this.refreshIntervals[id])
       })
 
@@ -458,13 +458,13 @@ export default class {
       // Get the cached accessory or add to Homebridge if it doesn't exist
       const accessory = devicesInHB.get(uuid) || this.addAccessory(loadedDevice)
 
-      accessory.context.rawConfig = this.deviceConf?.[device.did] || platformConsts.defaultDevice
-
       // Final check the accessory now exists in Homebridge
       if (!accessory) {
         throw new Error(platformLang.accNotFound)
       }
 
+      accessory.context.rawConfig = this.deviceConf?.[device.did] || platformConsts.defaultDevice
+
       // Sort out some logging functions per accessory
       if (this.isBeta) {
         accessory.log = msg => this.log('[%s] %s.', accessory.displayName, msg)
@@ -487,21 +487,6 @@ export default class {
       accessory.context.isOnline = false
       accessory.context.lastMsg = ''
 
-      // Remove the existing cache values from the context
-      // accessory.context.cacheClean
-      // accessory.context.cacheCharge
-      // accessory.context.cacheAirDrying
-      // accessory.context.cacheSpeed
-      // accessory.context.cacheBattery
-      const cacheKeys = [
-        'cacheClean',
-        'cacheCharge',
-        'cacheAirDrying',
-        'cacheSpeed',
-        'cacheBattery',
-      ]
-      cacheKeys.forEach(key => delete accessory.context[key])
-
       // Add the 'clean' switch service if it doesn't already exist
       const cleanService = accessory.getService('Clean') || accessory.addService(this.hapServ.Switch, 'Clean', 'clean')
       if (!cleanService.testCharacteristic(this.hapChar.ConfiguredName)) {
@@ -618,6 +603,16 @@ export default class {
         accessory.logDebug('no air drying available or not configured')
       }
 
+      // Remove the existing cache values from the context now that services have been updated
+      const cacheKeys = [
+        'cacheClean',
+        'cacheCharge',
+        'cacheAirDrying',
+        'cacheSpeed',
+        'cacheBattery',
+      ]
+      cacheKeys.forEach(key => delete accessory.context[key])
+
       // TrueDetect service
       if (accessory.context.rawConfig.supportTrueDetect) {
         // Custom Eve characteristic like MaxSpeed
@@ -744,16 +739,16 @@ export default class {
         if (vbs) {
           accessory.logDebug(`MapVirtualBoundaries: ${JSON.stringify(vbs)}`)
           const vbsCombined = [...vbs.mapVirtualWalls, ...vbs.mapNoMopZones]
-          const virtualBoundaryArray = []
-          Object.keys(vbsCombined).forEach((key) => {
-            virtualBoundaryArray[vbsCombined[key].mapVirtualBoundaryID] = vbsCombined[key]
+          const virtualBoundaryMap = {}
+          vbsCombined.forEach((vb) => {
+            virtualBoundaryMap[vb.mapVirtualBoundaryID] = vb
           })
-          Object.keys(virtualBoundaryArray).forEach((key) => {
+          Object.values(virtualBoundaryMap).forEach((vb) => {
             accessory.control.run(
               'GetVirtualBoundaryInfo',
               vbs.mapID,
-              virtualBoundaryArray[key].mapVirtualBoundaryID,
-              virtualBoundaryArray[key].mapVirtualBoundaryType,
+              vb.mapVirtualBoundaryID,
+              vb.mapVirtualBoundaryType,
             )
           })
         }
@@ -770,7 +765,7 @@ export default class {
 
       // Refresh the current state of all the accessories
       this.refreshAccessory(accessory)
-      const { pollInterval } = accessory.context.rawConfig[device] || platformConsts.defaultValues
+      const { pollInterval } = accessory.context.rawConfig || platformConsts.defaultValues
       if (pollInterval > 0) {
         this.refreshIntervals[device.did] = setInterval(() => {
           devicesInHB.get(this.api.hap.uuid.generate(device.did)).control?.refresh()
@@ -956,7 +951,7 @@ export default class {
       const order = value ? `Clean${accessory.context.commandSuffix}` : 'Stop'
 
       // Log the update
-      accessory.log(`${platformLang.curCleaning} [${value ? platformLang.cleaning : platformLang.stop}}]`)
+      accessory.log(`${platformLang.curCleaning} [${value ? platformLang.cleaning : platformLang.stop}]`)
 
       // Send the command
       accessory.logDebug(`${platformLang.sendCmd} [${order}]`)
@@ -1096,7 +1091,7 @@ export default class {
       }, 2000)
     } catch (err) {
       // Catch any errors during the process
-      accessory.logWarn(`${platformLang.speedFail} ${parseError(err, [platformLang.errNotInit, platformLang.errNotReady])}`)
+      accessory.logWarn(`${platformLang.cleanFail} ${parseError(err, [platformLang.errNotInit, platformLang.errNotReady])}`)
 
       // Throw a 'no response' error and set a timeout to revert this after 2 seconds
       setTimeout(() => {
@@ -1220,7 +1215,7 @@ export default class {
       accessory.logDebug(`${platformLang.sendCmd} [GetBatteryState]`)
       accessory.control.run('GetBatteryState')
 
-      accessory.log(`${platformLang.sendCmd} [GetChargeState]`)
+      accessory.logDebug(`${platformLang.sendCmd} [GetChargeState]`)
       accessory.control.run('GetChargeState')
 
       accessory.logDebug(`${platformLang.sendCmd} [GetCleanState${accessory.context.commandSuffix}]`)

package/lib/utils/constants.js

@@ -6,7 +6,6 @@ export default {
     username: '',
     password: '',
     disableDeviceLogging: false,
-    refreshTime: 120,
     devices: [],
     platform: 'Deebot',
   },

package/lib/utils/functions.js

@@ -3,7 +3,7 @@ function parseError(err, hideStack = []) {
   if (err?.stack?.length > 0 && !hideStack.includes(err.message)) {
     const stack = err.stack.split('\n')
     if (stack[1]) {
-      toReturn += stack[1].replace('   ', '')
+      toReturn += stack[1].trim()
     }
   }
   return toReturn

package/package.json

@@ -3,7 +3,7 @@
   "displayName": "Homebridge Ecovacs",
   "alias": "Deebot",
   "type": "module",
-  "version": "7.3.0",
+  "version": "7.4.0",
   "description": "Homebridge plugin to integrate Ecovacs Deebot/Yeedi devices into HomeKit.",
   "author": {
     "name": "bwp91",
@@ -54,7 +54,7 @@
   "main": "lib/index.js",
   "engines": {
     "homebridge": "^1.6.0 || ^2.0.0-beta.0",
-    "node": "^20 || ^22 || ^24"
+    "node": "^22 || ^24"
   },
   "scripts": {
     "lint": "eslint . --max-warnings=0",
@@ -62,20 +62,21 @@
     "postinstall": "patch-package"
   },
   "dependencies": {
-    "@homebridge/plugin-ui-utils": "^2.2.0",
+    "@homebridge/plugin-ui-utils": "^2.2.3",
     "ecovacs-deebot": "^0.9.6-beta.12",
     "patch-package": "^8.0.1"
   },
   "devDependencies": {
-    "@antfu/eslint-config": "^7.4.3"
+    "@antfu/eslint-config": "^8.0.0"
   },
   "overrides": {
     "ecovacs-deebot": {
-      "axios": "^1.13.5",
+      "@xmldom/xmldom": "^0.9.9",
+      "axios": "^1.14.0",
       "canvas": {
         ".": "^2.11.1",
         "@mapbox/node-pre-gyp": {
-          "tar": "^7.5.9"
+          "tar": "^7.5.13"
         }
       },
       "simple-xmpp": {