@rfranzoi/scrypted-mqtt-securitysystem 1.0.18 → 1.0.20

package/dist/main.nodejs.js

@@ -34083,9 +34083,6 @@ function normalize(s) {
 function clamp(n, min, max) {
     return Math.max(min, Math.min(max, n));
 }
-function delay(ms) {
-    return new Promise(res => setTimeout(res, ms));
-}
 /** SecuritySystem outgoing defaults (PAI-like) */
 const DEFAULT_OUTGOING = {
     [sdk_1.SecuritySystemMode.Disarmed]: 'disarm',
@@ -34115,7 +34112,7 @@ class BaseMqttSensor extends sdk_1.ScryptedDeviceBase {
     constructor(nativeId, cfg) {
         super(nativeId);
         this.cfg = cfg;
-        this.online = this.online ?? true;
+        // ⚠️ Non toccare stati nel costruttore (discovery non ancora completata)
     }
     /** Called by parent on each MQTT message */
     handleMqtt(topic, payload) {
@@ -34208,7 +34205,7 @@ class ParadoxMqttSecuritySystem extends sdk_1.ScryptedDeviceBase {
         this.online = this.online ?? false;
         // Load sensors config and announce devices
         this.loadSensorsFromStorage();
-        this.discoverSensors();
+        this.discoverSensors().catch(e => this.console.error('discoverSensors error', e));
         // Connect on start
         this.connectMqtt().catch(e => this.console.error('MQTT connect error:', e));
         // chiusura pulita del client MQTT ai reload/stop del plugin
@@ -34276,7 +34273,7 @@ class ParadoxMqttSecuritySystem extends sdk_1.ScryptedDeviceBase {
     async getDevice(nativeId) {
         return this.devices.get(nativeId);
     }
-    async releaseDevice(id, nativeId) {
+    async releaseDevice(_id, nativeId) {
         try {
             const dev = this.devices.get(nativeId);
             if (dev) {
@@ -34303,64 +34300,64 @@ class ParadoxMqttSecuritySystem extends sdk_1.ScryptedDeviceBase {
             this.sensorsCfg = [];
         }
     }
-    /** ===== discoverSensors con batch/fallback (FIX type) ===== */
+    /** ===== discoverSensors: annuncia PRIMA, istanzia DOPO ===== */
     async discoverSensors() {
-        // Prepara i manifest e istanzia/aggiorna le classi locali
+        // 1) Prepara i manifest (niente istanze qui)
         const manifests = this.sensorsCfg.map(cfg => {
             const nativeId = `sensor:${cfg.id}`;
-            let interfaces = [
+            // interfacce dinamiche
+            const t = cfg.topics || {};
+            const interfaces = [
                 sdk_1.ScryptedInterface.Online,
                 sdk_1.ScryptedInterface.TamperSensor,
-                sdk_1.ScryptedInterface.Battery,
             ];
-            // Il tipo RESTA generico "Sensor" per compatibilità SDK,
-            // le capacità sono date dalle interfacce.
-            const type = sdk_1.ScryptedDeviceType.Sensor;
-            switch (cfg.kind) {
-                case 'contact':
-                    interfaces = [sdk_1.ScryptedInterface.EntrySensor, ...interfaces];
-                    break;
-                case 'motion':
-                    interfaces = [sdk_1.ScryptedInterface.MotionSensor, ...interfaces];
-                    break;
-                case 'occupancy':
-                    interfaces = [sdk_1.ScryptedInterface.OccupancySensor, ...interfaces];
-                    break;
-            }
-            let dev = this.devices.get(nativeId);
-            if (!dev) {
-                if (cfg.kind === 'contact')
-                    dev = new ContactMqttSensor(nativeId, cfg);
-                else if (cfg.kind === 'motion')
-                    dev = new MotionMqttSensor(nativeId, cfg);
-                else
-                    dev = new OccupancyMqttSensor(nativeId, cfg);
-                this.devices.set(nativeId, dev);
-            }
-            else {
-                dev.cfg = cfg;
+            if (cfg.kind === 'contact')
+                interfaces.unshift(sdk_1.ScryptedInterface.EntrySensor);
+            else if (cfg.kind === 'motion')
+                interfaces.unshift(sdk_1.ScryptedInterface.MotionSensor);
+            else
+                interfaces.unshift(sdk_1.ScryptedInterface.OccupancySensor);
+            // aggiungi Battery solo se previsto
+            if (t.batteryLevel || t.lowBattery) {
+                interfaces.push(sdk_1.ScryptedInterface.Battery);
             }
             return {
                 nativeId,
                 name: cfg.name,
-                type,
+                type: sdk_1.ScryptedDeviceType.Sensor,
                 interfaces,
             };
         });
-        // Annuncio in batch se disponibile, altrimenti uno per volta con un piccolo delay
-        const dm = deviceManager;
-        if (typeof dm.onDevicesChanged === 'function') {
-            dm.onDevicesChanged({ devices: manifests });
+        // 2) Annuncia i device
+        const dmAny = deviceManager;
+        if (typeof dmAny.onDevicesChanged === 'function') {
+            dmAny.onDevicesChanged({ devices: manifests });
             this.console.log('Annunciati (batch):', manifests.map(m => m.nativeId).join(', '));
         }
         else {
             for (const m of manifests) {
                 deviceManager.onDeviceDiscovered(m);
                 this.console.log('Annunciato:', m.nativeId);
-                await new Promise(res => setTimeout(res, 50));
             }
         }
-        // Rimuovi eventuali sensori non più presenti
+        // 3) Istanzia/aggiorna le classi DOPO l’annuncio
+        for (const cfg of this.sensorsCfg) {
+            const nativeId = `sensor:${cfg.id}`;
+            let dev = this.devices.get(nativeId);
+            if (!dev) {
+                if (cfg.kind === 'contact')
+                    dev = new ContactMqttSensor(nativeId, cfg);
+                else if (cfg.kind === 'motion')
+                    dev = new MotionMqttSensor(nativeId, cfg);
+                else
+                    dev = new OccupancyMqttSensor(nativeId, cfg);
+                this.devices.set(nativeId, dev);
+            }
+            else {
+                dev.cfg = cfg;
+            }
+        }
+        // 4) Rimuovi quelli spariti
         const announced = new Set(manifests.map(m => m.nativeId));
         for (const [nativeId] of this.devices) {
             if (!announced.has(nativeId)) {
@@ -34404,7 +34401,7 @@ class ParadoxMqttSecuritySystem extends sdk_1.ScryptedDeviceBase {
         }
         // sensors
         for (const s of this.sensorsCfg) {
-            const t = s.topics;
+            const t = s.topics || {};
             [t.contact, t.motion, t.occupancy, t.batteryLevel, t.lowBattery, t.tamper, t.online]
                 .filter(Boolean)
                 .forEach(x => subs.add(String(x)));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rfranzoi/scrypted-mqtt-securitysystem",
-  "version": "1.0.18",
+  "version": "1.0.20",
   "description": "Scrypted plugin: Paradox Security System via MQTT (PAI/PAI-MQTT style).",
   "license": "MIT",
   "main": "dist/main.nodejs.js",