homebridge-smartsystem 7.1.4 → 7.1.6

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "homebridge-smartsystem",
   "displayname": "Duotecno Bridge",
-  "version": "7.1.4",
+  "version": "7.1.6",
   "description": "SmartServer (Proxy TCP sockets to the cloud, Smappee MQTT, Duotecno IP Nodes, Homekit interface)",
   "main": "index.js",
   "author": "Johan Coppieters",

package/server/smartapp.js

@@ -760,10 +760,13 @@ class SmartApp extends webapp_1.WebApp {
     //////////////////////
     sendDeviceList(ws) {
         const devices = this.getDeviceList();
-        console.log("Sending device list to WebSocket client: " + JSON.stringify(devices));
         if (ws.readyState === WebSocket.OPEN) {
-            (0, logger_1.debug)("smartapp", devices);
-            ws.send(JSON.stringify(devices));
+            const json = JSON.stringify(devices);
+            (0, logger_1.log)("smartapp", "sendDeviceList -> " + json);
+            ws.send(json);
+        }
+        else {
+            (0, logger_1.log)("smartapp", "sendDeviceList -> WebSocket not open (state=" + ws.readyState + ")");
         }
     }
     getDeviceList() {
@@ -773,6 +776,7 @@ class SmartApp extends webapp_1.WebApp {
             if (device.unit) {
                 relayState = device.unit.status > 0;
             }
+            (0, logger_1.log)("smartapp", `getDeviceList: device "${device.name}" (id=${device.id}) powerSource=${device.powerSource} powerChannel="${device.powerChannel}" relay=${relayState}`);
             // Get power measurements based on power source
             let power = 0;
             let energy = 0;
@@ -811,6 +815,7 @@ class SmartApp extends webapp_1.WebApp {
                 power = relayState ? (device.estimatedPower || 0) : 0;
                 measured = false;
             }
+            (0, logger_1.log)("smartapp", `getDeviceList: -> power=${power} measured=${measured}`);
             return {
                 id: device.id,
                 name: device.name,
@@ -829,12 +834,18 @@ class SmartApp extends webapp_1.WebApp {
         try {
             // Channel format could be "0", "0+1", "0+1+2", etc.
             const channels = channel.split('+').map(c => parseInt(c.trim()));
+            const available = this.power.smappee.bindings.map(b => `ch=${b.channel}(${b.value})`).join(', ');
+            (0, logger_1.log)("smartapp", `getPowerFromSmappee: looking for channel(s) [${channels}], available bindings: [${available}]`);
             let totalPower = 0;
             for (const ch of channels) {
-                const binding = this.power.smappee.bindings.find(b => b.channel.includes(ch.toString()));
+                const binding = this.power.smappee.bindings.find(b => b.channel === ch.toString());
                 if (binding && binding.value !== undefined) {
+                    (0, logger_1.log)("smartapp", `getPowerFromSmappee: matched ch=${ch} -> value=${binding.value}`);
                     totalPower += binding.value;
                 }
+                else {
+                    (0, logger_1.log)("smartapp", `getPowerFromSmappee: no match for ch=${ch}`);
+                }
             }
             return totalPower;
         }
@@ -848,10 +859,14 @@ class SmartApp extends webapp_1.WebApp {
             return null;
         try {
             // P1 meter typically has single power value
-            const binding = this.power.p1.bindings.find(b => b.channel.includes(channel));
+            const available = this.power.p1.bindings.map(b => `ch=${b.channel}(${b.value})`).join(', ');
+            (0, logger_1.log)("smartapp", `getPowerFromP1: looking for channel "${channel}", available bindings: [${available}]`);
+            const binding = this.power.p1.bindings.find(b => b.channel === channel);
             if (binding && binding.value !== undefined) {
+                (0, logger_1.log)("smartapp", `getPowerFromP1: matched ch=${channel} -> value=${binding.value}`);
                 return binding.value;
             }
+            (0, logger_1.log)("smartapp", `getPowerFromP1: no match for channel "${channel}"`);
             return null;
         }
         catch (e) {
@@ -865,12 +880,18 @@ class SmartApp extends webapp_1.WebApp {
         try {
             // Shelly channel format similar to Smappee
             const channels = channel.split('+').map(c => parseInt(c.trim()));
+            const available = this.power.shelly.bindings.map(b => `ch=${b.channel}(${b.value})`).join(', ');
+            (0, logger_1.log)("smartapp", `getPowerFromShelly: looking for channel(s) [${channels}], available bindings: [${available}]`);
             let totalPower = 0;
             for (const ch of channels) {
-                const binding = this.power.shelly.bindings.find(b => b.channel.includes(ch.toString()));
+                const binding = this.power.shelly.bindings.find(b => b.channel === ch.toString());
                 if (binding && binding.value !== undefined) {
+                    (0, logger_1.log)("smartapp", `getPowerFromShelly: matched ch=${ch} -> value=${binding.value}`);
                     totalPower += binding.value;
                 }
+                else {
+                    (0, logger_1.log)("smartapp", `getPowerFromShelly: no match for ch=${ch}`);
+                }
             }
             return totalPower;
         }
@@ -881,6 +902,7 @@ class SmartApp extends webapp_1.WebApp {
     }
     handleDeviceMessage(ws, data) {
         return __awaiter(this, void 0, void 0, function* () {
+            (0, logger_1.log)("smartapp", "handleDeviceMessage: received -> " + JSON.stringify(data));
             if (!Array.isArray(data)) {
                 (0, logger_1.err)("smartapp", "WebSocket message must be an array");
                 return;
@@ -888,6 +910,7 @@ class SmartApp extends webapp_1.WebApp {
             // Process relay control commands if any
             for (const cmd of data) {
                 if (cmd.id && cmd.relay !== undefined) {
+                    (0, logger_1.log)("smartapp", `handleDeviceMessage: relay command id=${cmd.id} state=${cmd.relay}`);
                     yield this.setDeviceRelay(cmd.id, cmd.relay);
                 }
             }