iobroker.zigbee2mqtt 3.1.5 → 3.2.0

package/lib/mqttServerController.js

@@ -1,7 +1,7 @@
 'use strict';
 
 const core = require('@iobroker/adapter-core');
-const { Aedes } = require('aedes');
+const Aedes = require('aedes');
 const net = require('node:net');
 
 /**
@@ -69,22 +69,24 @@ class MqttServerController {
             if (!db.setup) {
                 db.setup = async () => {};
             }
-            this.aedesBroker = await Aedes.createBroker({ persistence: db });
-            this.mqttServer  = net.createServer(this.aedesBroker.handle);
+            const aedes = Aedes({ persistence: db });
+            this.aedesBroker = aedes;
+            this.mqttServer  = net.createServer(aedes.handle);
             this._attachServerEvents('MQTT-Server');
 
             await new Promise((resolve, reject) => {
+                this.mqttServer.once('error', reject);
                 this.mqttServer.listen(
                     this.adapter.config.mqttServerPort,
                     this.adapter.config.mqttServerIPBind,
                     () => {
+                        this.mqttServer.removeListener('error', reject);
                         this.adapter.log.info(
                             `Starting MQTT-Server on IP ${this.adapter.config.mqttServerIPBind} and Port ${this.adapter.config.mqttServerPort}`
                         );
                         resolve();
                     }
                 );
-                this.mqttServer.once('error', reject);
             });
         } catch (err) {
             this.adapter.log.error(`createMQTTServer error: ${err && err.message ? err.message : String(err)}`);
@@ -102,22 +104,24 @@ class MqttServerController {
             return;
         }
         try {
-            this.aedesBroker = await Aedes.createBroker();
-            this.mqttServer  = net.createServer(this.aedesBroker.handle);
+             const aedes = Aedes();
+            this.aedesBroker = aedes;
+            this.mqttServer  = net.createServer(aedes.handle);
             this._attachServerEvents('DummyMQTT-Server');
 
             await new Promise((resolve, reject) => {
+                this.mqttServer.once('error', reject);
                 this.mqttServer.listen(
                     this.adapter.config.mqttServerPort,
                     this.adapter.config.mqttServerIPBind,
                     () => {
+                        this.mqttServer.removeListener('error', reject);
                         this.adapter.log.info(
                             `Starting DummyMQTT-Server on IP ${this.adapter.config.mqttServerIPBind} and Port ${this.adapter.config.mqttServerPort}`
                         );
                         resolve();
                     }
                 );
-                this.mqttServer.once('error', reject);
             });
         } catch (err) {
             this.adapter.log.error(`createDummyMQTTServer error: ${err && err.message ? err.message : String(err)}`);
@@ -130,6 +134,17 @@ class MqttServerController {
      * Wird beim Adapter-Stop oder nach einem Fehler beim Start aufgerufen.
      */
     closeServer() {
+        if (this.aedesBroker) {
+            const broker = this.aedesBroker;
+            this.aedesBroker = null;
+            try {
+                broker.close(() => {
+                    this.adapter.log.debug('Aedes broker closed.');
+                });
+            } catch (err) {
+                this.adapter.log.error(`closeServer aedesBroker.close() error: ${err}`);
+            }
+        }
         if (this.mqttServer) {
             const server = this.mqttServer;
             this.mqttServer = null;
@@ -143,21 +158,6 @@ class MqttServerController {
                 this.adapter.log.error(`closeServer mqttServer.close() error: ${err}`);
             }
         }
-        if (this.aedesBroker) {
-            const broker = this.aedesBroker;
-            this.aedesBroker = null;
-            try {
-                broker.close((err) => {
-                    if (err) {
-                        this.adapter.log.debug(`Aedes broker close callback error: ${err}`);
-                    } else {
-                        this.adapter.log.debug('Aedes broker closed.');
-                    }
-                });
-            } catch (err) {
-                this.adapter.log.error(`closeServer aedesBroker.close() error: ${err}`);
-            }
-        }
     }
 }
 

package/lib/states.js

@@ -365,7 +365,9 @@ const states = {
         unit: 'lux',
         getter: (payload) => {
             if (payload.illuminance == null) { return undefined; }
-            return Math.round(Math.pow(10, payload.illuminance / 10000));
+            const raw = payload.illuminance;
+            if (raw <= 0) { return 0; }
+            return Math.round(Math.pow(10, (raw - 1) / 10000));
         },
     },
 
@@ -418,7 +420,9 @@ const states = {
         unit: 'lux',
         getter: (payload) => {
             if (payload.illuminance_raw == null) { return undefined; }
-            return Math.round(Math.pow(10, payload.illuminance_raw / 10000));
+            const raw = payload.illuminance_raw;
+            if (raw <= 0) { return 0; }
+            return Math.round(Math.pow(10, (raw - 1) / 10000));
         },
     },
 

package/lib/statesController.js

@@ -157,7 +157,7 @@ class StatesController {
                     if (typeof value === 'object') {
                         value = JSON.stringify(value);
                     }
-                    await this.adapter.setStateAsync(`${fullPath}.${key}`, value, true);
+                    this.adapter.setState(`${fullPath}.${key}`, value, true);
                 }
                 continue;
             }
@@ -233,7 +233,7 @@ class StatesController {
         if (value === undefined || value === null) {
             return;
         }
-        await this.adapter.setStateAsync(stateName, value, true);
+        this.adapter.setState(stateName, value, true);
     }
 
     /**
@@ -264,7 +264,7 @@ class StatesController {
             return;
         }
 
-        await this.adapter.setStateAsync(stateName, value, true);
+        this.adapter.setState(stateName, value, true);
 
         // Auto-Reset (false → nichts tun, true → nach timeout zurücksetzen)
         // Wenn value=false (z.B. Stop-Aktion im simpleMoveStopState-Modus),
@@ -276,7 +276,7 @@ class StatesController {
         if (value === true) {
             this.timeOutCache[stateName] = this.adapter.setTimeout(() => {
                 delete this.timeOutCache[stateName];
-                this.adapter.setStateAsync(stateName, false, true).catch((err) => {
+                this.adapter.setState(stateName, false, true).catch((err) => {
                     this.adapter.log.debug(`setStateWithTimeout reset error for ${stateName}: ${err}`);
                 });
             }, timeout);

package/lib/websocketController.js

@@ -133,8 +133,8 @@ class WebsocketController {
         this.adapter.log.debug(`WebSocket closed – code: ${code}, reason: ${reason ? reason.toString() : 'none'}`);
 
         try {
+            await this.adapter.setStateChangedAsync('info.connection', false, true);
             if (this.adapter.statesController) {
-                this.adapter.setStateChanged('info.connection', false, true);
                 await this.adapter.statesController.setAllAvailableToFalse();
             }
         } catch (err) {

package/lib/z2mController.js

@@ -87,7 +87,7 @@ class Z2mController {
         if (stateID === 'send_payload') {
             try {
                 controlObj.payload = JSON.parse(stateVal);
-                await this.adapter.setStateAsync(id, state.val, true);
+                this.adapter.setState(id, state.val, true);
             } catch (error) {
                 // Fix 9: rawStr als String direkt als payload – wird in main.js via
                 // JSON.stringify() nochmals gewrappt → würde doppelt quoten.
@@ -122,7 +122,7 @@ class Z2mController {
         if (deviceState.isOption) {
             // set optionsValues 'Cache'
             device.optionsValues[stateName] = state.val;
-            await this.adapter.setStateAsync(id, state.val, true);
+            this.adapter.setState(id, state.val, true);
             return;
         }
 
@@ -130,7 +130,7 @@ class Z2mController {
         const immediateAckRole = ['button'].includes(deviceState.role);
         const immediateAckId = ['brightness_move', 'colortemp_move', 'brightness_step', 'effect'].includes(deviceState.id);
         if (immediateAckRole || immediateAckId) {
-            await this.adapter.setStateAsync(id, state.val, true);
+            this.adapter.setState(id, state.val, true);
         }
 
         if (this.logCustomizations.debugDevices) {

package/main.js

@@ -71,7 +71,7 @@ class Zigbee2mqtt extends core.Adapter {
 
         adapterInfo(this.config, this.log);
 
-        await this.setStateAsync('info.connection', false, true);
+        this.setState('info.connection', false, true);
 
         const debugDevicesState = await this.getStateAsync('info.debugmessages');
         if (debugDevicesState && debugDevicesState.val) {
@@ -109,6 +109,7 @@ class Zigbee2mqtt extends core.Adapter {
                     clientId: `ioBroker.zigbee2mqtt_${Math.random().toString(16).slice(2, 8)}`,
                     clean: true,
                     reconnectPeriod: 500,
+                    connectTimeout: 10000,
                 };
 
                 if (this.config.externalMqttServerCredentials === true) {
@@ -130,6 +131,7 @@ class Zigbee2mqtt extends core.Adapter {
                     clientId: `ioBroker.zigbee2mqtt_${Math.random().toString(16).slice(2, 8)}`,
                     clean: true,
                     reconnectPeriod: 500,
+                    connectTimeout: 10000,
                 });
             }
 
@@ -156,18 +158,25 @@ class Zigbee2mqtt extends core.Adapter {
                 this.log.info('MQTT client reconnecting to Zigbee2MQTT...');
             });
 
-            this.mqttClient.on('offline', () => {
-                (async () => {
-                    this.log.warn('MQTT client offline – connection to Zigbee2MQTT lost.');
-                    await this.setStateChangedAsync('info.connection', false, true);
-                    try {
-                        if (this.statesController) {
-                            await this.statesController.setAllAvailableToFalse();
-                        }
-                    } catch (e) {
-                        this.log.error(`MQTT offline setAllAvailableToFalse error: ${e}`);
+            this.mqttClient.on('offline', async () => {
+                this.log.warn('MQTT client offline – connection to Zigbee2MQTT lost.');
+                await this.setStateChangedAsync('info.connection', false, true).catch((e) =>
+                    this.log.error(`MQTT offline setStateChangedAsync error: ${e}`)
+                );
+                try {
+                    if (this.statesController) {
+                        await this.statesController.setAllAvailableToFalse();
                     }
-                })().catch((e) => this.log.error(`MQTT offline handler error: ${e}`));
+                } catch (e) {
+                    this.log.error(`MQTT offline setAllAvailableToFalse error: ${e}`);
+                }
+            });
+
+            this.mqttClient.on('close', async () => {
+                this.log.debug('MQTT client connection closed.');
+                await this.setStateChangedAsync('info.connection', false, true).catch((e) =>
+                    this.log.error(`MQTT close setStateChangedAsync error: ${e}`)
+                );
             });
 
             this.mqttClient.on('error', (err) => {
@@ -513,7 +522,7 @@ class Zigbee2mqtt extends core.Adapter {
     async onUnload(callback) {
         try {
             if (['exmqtt', 'intmqtt'].includes(this.config.connectionType)) {
-                if (this.mqttClient && !this.mqttClient.disconnected) {
+                if (this.mqttClient && this.mqttClient.connected) {
                     try {
                         this.mqttClient.removeAllListeners();
                         this.mqttClient.end(true);
@@ -561,7 +570,7 @@ class Zigbee2mqtt extends core.Adapter {
                 this.log.error(e);
             }
 
-            await this.setStateAsync('info.connection', false, true);
+            this.setState('info.connection', false, true);
 
             // Schedule-Job beenden
             const job = schedule.scheduledJobs['coordinatorCheck'];
@@ -586,14 +595,14 @@ class Zigbee2mqtt extends core.Adapter {
         if (state && state.ack === false) {
             if (id.endsWith('info.debugmessages')) {
                 this.logCustomizations.debugDevices = state.val != null ? String(state.val) : '';
-                await this.setStateAsync(id, state.val, true);
+                this.setState(id, state.val, true);
                 return;
             }
             if (id.endsWith('info.logfilter')) {
                 this.logCustomizations.logfilter = state.val != null
                     ? String(state.val).split(';').filter((x) => x)
                     : [];
-                await this.setStateAsync(id, state.val, true);
+                this.setState(id, state.val, true);
                 return;
             }
 
@@ -608,7 +617,7 @@ class Zigbee2mqtt extends core.Adapter {
             }
 
             if (['exmqtt', 'intmqtt'].includes(this.config.connectionType)) {
-                if (!this.mqttClient || this.mqttClient.disconnected) {
+                if (!this.mqttClient || !this.mqttClient.connected) {
                     this.log.warn(`Cannot publish state, MQTT client not connected. (${id})`);
                     return;
                 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "iobroker.zigbee2mqtt",
-  "version": "3.1.5",
+  "version": "3.2.0",
   "description": "Zigbee2MQTT adapter for ioBroker",
   "author": {
     "name": "Dennis Rathjen and Arthur Rupp",
@@ -26,7 +26,7 @@
   "dependencies": {
     "@iobroker/adapter-core": "^3.3.2",
     "@iobroker/dm-utils": "^3.0.17",
-    "aedes": "^1.0.2",
+    "aedes": "^0.51.3",
     "aedes-persistence-nedb": "^2.0.3",
     "mqtt": "^5.15.1",
     "axios": "^1.14.0",