matterbridge-roborock-vacuum-plugin 1.1.1-rc02 → 1.1.1-rc05

package/.github/workflows/build.yml

@@ -13,7 +13,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        node-version: [18.x, 20.x, 22.x, 24.x]
+        node-version: [20.x, 22.x, 24.x]
         os: [ubuntu-latest, windows-latest, macos-latest]
 
     steps:

package/dist/platform.js

@@ -26,8 +26,8 @@ export class RoborockMatterbridgePlatform extends MatterbridgeDynamicPlatform {
     rrHomeId;
     constructor(matterbridge, log, config) {
         super(matterbridge, log, config);
-        if (this.verifyMatterbridgeVersion === undefined || typeof this.verifyMatterbridgeVersion !== 'function' || !this.verifyMatterbridgeVersion('3.2.0')) {
-            throw new Error(`This plugin requires Matterbridge version >= "3.2.0". Please update Matterbridge from ${this.matterbridge.matterbridgeVersion} to the latest version in the frontend.`);
+        if (this.verifyMatterbridgeVersion === undefined || typeof this.verifyMatterbridgeVersion !== 'function' || !this.verifyMatterbridgeVersion('3.2.2')) {
+            throw new Error(`This plugin requires Matterbridge version >= "3.2.2". Please update Matterbridge from ${this.matterbridge.matterbridgeVersion} to the latest version in the frontend.`);
         }
         this.log.info('Initializing platform:', this.config.name);
         if (config.whiteList === undefined)

package/dist/roborockCommunication/broadcast/listener/implementation/generalSyncMessageListener.js

@@ -0,0 +1,28 @@
+import { Protocol } from '../../model/protocol.js';
+export class GeneralSyncMessageListener {
+    duid;
+    handler;
+    timer;
+    constructor(duid) {
+        this.duid = duid;
+    }
+    waitFor() {
+        return new Promise((resolve, reject) => {
+            this.handler = resolve;
+            this.timer = setTimeout(() => {
+                reject('no ping response received for ' + this.duid + ' within ' + 30 + 'second');
+            }, 30 * 1000);
+        });
+    }
+    async onMessage(message) {
+        if (message.contain(Protocol.hello_response)) {
+            if (this.handler) {
+                this.handler(message);
+            }
+            if (this.timer) {
+                clearTimeout(this.timer);
+                this.timer.unref();
+            }
+        }
+    }
+}

package/matterbridge-roborock-vacuum-plugin.config.json

@@ -1,7 +1,7 @@
 {
   "name": "matterbridge-roborock-vacuum-plugin",
   "type": "DynamicPlatform",
-  "version": "1.1.1-rc02",
+  "version": "1.1.1-rc05",
   "whiteList": [],
   "blackList": [],
   "useInterval": true,

package/matterbridge-roborock-vacuum-plugin.schema.json

@@ -1,6 +1,6 @@
 {
   "title": "Matterbridge Roborock Vacuum Plugin",
-  "description": "matterbridge-roborock-vacuum-plugin v. 1.1.1-rc02 by https://github.com/RinDevJunior",
+  "description": "matterbridge-roborock-vacuum-plugin v. 1.1.1-rc05 by https://github.com/RinDevJunior",
   "type": "object",
   "required": ["username", "password"],
   "properties": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "matterbridge-roborock-vacuum-plugin",
-  "version": "1.1.1-rc02",
+  "version": "1.1.1-rc05",
   "description": "Matterbridge Roborock Vacuum Plugin",
   "author": "https://github.com/RinDevJunior",
   "license": "MIT",
@@ -31,7 +31,7 @@
     "apple home"
   ],
   "engines": {
-    "node": ">=22.0.0"
+    "node": ">=20.0.0 <21.0.0 || >=22.0.0 <23.0.0 || >=24.0.0 <25.0.0"
   },
   "dependencies": {
     "@log4js-node/log4js-api": "^1.0.2",

package/src/platform.ts

@@ -33,9 +33,9 @@ export class RoborockMatterbridgePlatform extends MatterbridgeDynamicPlatform {
     super(matterbridge, log, config);
 
     // Verify that Matterbridge is the correct version
-    if (this.verifyMatterbridgeVersion === undefined || typeof this.verifyMatterbridgeVersion !== 'function' || !this.verifyMatterbridgeVersion('3.2.0')) {
+    if (this.verifyMatterbridgeVersion === undefined || typeof this.verifyMatterbridgeVersion !== 'function' || !this.verifyMatterbridgeVersion('3.2.2')) {
       throw new Error(
-        `This plugin requires Matterbridge version >= "3.2.0". Please update Matterbridge from ${this.matterbridge.matterbridgeVersion} to the latest version in the frontend.`,
+        `This plugin requires Matterbridge version >= "3.2.2". Please update Matterbridge from ${this.matterbridge.matterbridgeVersion} to the latest version in the frontend.`,
       );
     }
     this.log.info('Initializing platform:', this.config.name);

package/src/roborockCommunication/broadcast/listener/implementation/generalSyncMessageListener.ts

@@ -0,0 +1,38 @@
+import { Protocol } from '../../model/protocol.js';
+import { ResponseMessage } from '../../model/responseMessage.js';
+import { AbstractMessageListener } from '../abstractMessageListener.js';
+
+export class GeneralSyncMessageListener implements AbstractMessageListener {
+  private readonly duid: string;
+
+  private handler?: (data: ResponseMessage) => void;
+  private timer?: NodeJS.Timeout;
+
+  constructor(duid: string) {
+    this.duid = duid;
+  }
+
+  public waitFor(): Promise<ResponseMessage> {
+    return new Promise<ResponseMessage>((resolve, reject) => {
+      this.handler = resolve;
+      this.timer = setTimeout(() => {
+        reject('no ping response received for ' + this.duid + ' within ' + 30 + 'second');
+      }, 30 * 1000);
+    });
+  }
+
+  public async onMessage(message: ResponseMessage): Promise<void> {
+    if (message.contain(Protocol.hello_response)) {
+      // trigger our waiters that we have received the response.
+      if (this.handler) {
+        this.handler(message);
+      }
+
+      // cleanup the timer
+      if (this.timer) {
+        clearTimeout(this.timer);
+        this.timer.unref();
+      }
+    }
+  }
+}

package/src/tests/platformRunner3.test.ts

@@ -2,14 +2,6 @@ import { PlatformRunner } from '../platformRunner';
 import { NotifyMessageTypes } from '../notifyMessageTypes';
 import { RoborockMatterbridgePlatform } from '../platform';
 import { RoborockVacuumCleaner } from '../rvc';
-import * as initialDataIndex from '../initialData/index';
-
-const getOperationalErrorState = jest.fn().mockReturnValue(2);
-
-jest.mock('./src/initialData/index', () => ({
-  ...initialDataIndex,
-  getOperationalErrorState,
-}));
 
 describe('PlatformRunner.updateRobot', () => {
   let platform: RoborockMatterbridgePlatform;

package/src/tests/roborockCommunication/broadcast/client/LocalNetworkClient.test.ts

@@ -52,6 +52,11 @@ describe('LocalNetworkClient', () => {
   afterEach(() => {
     jest.clearAllMocks();
     jest.useRealTimers();
+    // Clear keepConnectionAliveInterval to prevent open handle leaks
+    if (client['keepConnectionAliveInterval']) {
+      clearInterval(client['keepConnectionAliveInterval']);
+      client['keepConnectionAliveInterval'] = undefined;
+    }
   });
 
   it('should initialize fields in constructor', () => {