homebridge-smartsystem 6.8.7 → 6.8.9

package/README.md

@@ -302,15 +302,17 @@ Homebridge UI version
 - 4: better up-down values to and from HB
 
 
-### v6.8.0 - 08/07/2025
+### v6.8.x - 08/07/2025
 - 0: first version including the proxy
 - 1: max/min values for hbValues
 - 2: HB up/down -> don't respond to motor unit's, only macro
 - 3: remember request before the login kicked in (and redirect to that one, once the login was succesful)
 - 4: now using only TCP sockets to the server
 - 5: making it more robust / inline with repo-service version
-- 6: removed double error handlers on cloudSocket
-- 7: restart if no free connections
+- 6: 26 aug / removed double error handlers on cloudSocket
+- 7: 28 aug / restart if no free connections
+- 8: 28 aug / less logging (through logConfig) for device set/get functions
+- 9: corrected reading the local config file + dump it in the logs
 
 ## Hardware 
 A Raspberry Pi that connects to a Duotecno IP Node 

package/accessories/accessory.js

@@ -86,7 +86,7 @@ class Accessory {
     }
     updateState() {
         this.me.getCharacteristic(this.homebridge.Characteristic.On).updateValue(this.unit.status);
-        // log("accessory", "Received status change -> update accessory -> " + this.unit.getName() + " -> on = " + this.unit.status);
+        (0, logger_1.log)("accessory", "Received status change -> update accessory -> " + this.unit.getName() + " -> on = " + this.unit.status);
     }
 }
 exports.Accessory = Accessory;

package/accessories/accessory.ts

@@ -107,7 +107,7 @@ export class Accessory {
 
   updateState() {
     this.me.getCharacteristic(this.homebridge.Characteristic.On).updateValue(this.unit.status);
-    // log("accessory", "Received status change -> update accessory -> " + this.unit.getName() + " -> on = " + this.unit.status);
+    log("accessory", "Received status change -> update accessory -> " + this.unit.getName() + " -> on = " + this.unit.status);
   }
 
 }

package/accessories/dimmer.js

@@ -69,7 +69,7 @@ class Dimmer extends accessory_1.Accessory {
     }
     updateState() {
         this.me.getCharacteristic(this.homebridge.Characteristic.Brightness).updateValue(this.unit.value);
-        // log("accessory", "Received status change -> update accessory -> " + this.unit.getName() + " -> Brightness = " + this.unit.value);
+        (0, logger_1.log)("accessory", "Received status change -> update accessory -> " + this.unit.getName() + " -> Brightness = " + this.unit.value);
     }
 }
 exports.Dimmer = Dimmer;

package/accessories/dimmer.ts

@@ -84,7 +84,7 @@ export class Dimmer extends Accessory {
 
   updateState() {
     this.me.getCharacteristic(this.homebridge.Characteristic.Brightness).updateValue(this.unit.value);
-    // log("accessory", "Received status change -> update accessory -> " + this.unit.getName() + " -> Brightness = " + this.unit.value);
+    log("accessory", "Received status change -> update accessory -> " + this.unit.getName() + " -> Brightness = " + this.unit.value);
   }
 
 }

package/accessories/garagedoor.js

@@ -10,7 +10,6 @@ const accessory_1 = require("./accessory");
 // - has security, so when opening you need to authenticate
 // - TargetDoorState -> Characteristic.CurrentDoorState.OPEN, .STOPPED, .CLOSED
 //
-//
 class GarageDoor extends accessory_1.Accessory {
     constructor(homebridge, unit) {
         super(homebridge, unit);
@@ -50,7 +49,7 @@ class GarageDoor extends accessory_1.Accessory {
         else if (state == this.homebridge.Characteristic.CurrentDoorState.STOPPED)
             return types_1.UnitMotorCmd.kStop;
         else
-            return 0;
+            return null;
     }
     getDoorState(next) {
         try {
@@ -87,7 +86,7 @@ class GarageDoor extends accessory_1.Accessory {
     // in response to Duotecno status messages
     updateState() {
         const value = this.DT2HB(this.unit.status);
-        // log("accessory", "Received updateState -> Homekit GarageDoor for " + this.unit.node.getName() + " - " + this.unit.getName() + " -> " + this.unit.status + " / " + this.unit.value + " -> passing to HB: " + value);
+        (0, logger_1.log)("accessory", "Received updateState -> Homekit GarageDoor for " + this.unit.node.getName() + " - " + this.unit.getName() + " -> " + this.unit.status + " / " + this.unit.value + " -> passing to HB: " + value);
         this.me.getCharacteristic(this.homebridge.Characteristic.CurrentDoorState).updateValue(value);
     }
 }

package/accessories/garagedoor.ts

@@ -10,7 +10,6 @@ import { Accessory } from "./accessory";
 // - has security, so when opening you need to authenticate
 // - TargetDoorState -> Characteristic.CurrentDoorState.OPEN, .STOPPED, .CLOSED
 //
-//
 
 
 
@@ -61,7 +60,7 @@ export class GarageDoor extends Accessory {
       return this.homebridge.Characteristic.CurrentDoorState.STOPPED;  // ????
   }
 
-  HB2DT(state): UnitMotorCmd {
+  HB2DT(state): UnitMotorCmd | null {
     if (state == this.homebridge.Characteristic.CurrentDoorState.OPEN)
       return UnitMotorCmd.kOpen;
       
@@ -72,8 +71,7 @@ export class GarageDoor extends Accessory {
       return UnitMotorCmd.kStop;
 
     else 
-      return 0;
-
+      return null;
   }
 
   getDoorState(next) {
@@ -117,7 +115,7 @@ export class GarageDoor extends Accessory {
   // in response to Duotecno status messages
   updateState() {
     const value = this.DT2HB(this.unit.status);
-    // log("accessory", "Received updateState -> Homekit GarageDoor for " + this.unit.node.getName() + " - " + this.unit.getName() + " -> " + this.unit.status + " / " + this.unit.value + " -> passing to HB: " + value);
+    log("accessory", "Received updateState -> Homekit GarageDoor for " + this.unit.node.getName() + " - " + this.unit.getName() + " -> " + this.unit.status + " / " + this.unit.value + " -> passing to HB: " + value);
     this.me.getCharacteristic(this.homebridge.Characteristic.CurrentDoorState).updateValue(value);
   }
 

package/accessories/lock.js

@@ -69,7 +69,7 @@ class Lock extends accessory_1.Accessory {
     }
     // in response to Duotecno status messages
     updateState() {
-        // log("accessory", "Received updateState -> Homekit Lock for " + this.unit.node.getName() + " - " + this.unit.getName() + " -> " + this.unit.status);
+        (0, logger_1.log)("accessory", "Received updateState -> Homekit Lock for " + this.unit.node.getName() + " - " + this.unit.getName() + " -> " + this.unit.status);
         this.me.getCharacteristic(this.homebridge.Characteristic.LockCurrentState).updateValue(this.unit.status);
     }
 }

package/accessories/lock.ts

@@ -89,7 +89,7 @@ export class Lock extends Accessory {
 
   // in response to Duotecno status messages
   updateState() {
-    // log("accessory", "Received updateState -> Homekit Lock for " + this.unit.node.getName() + " - " + this.unit.getName() + " -> " + this.unit.status);
+    log("accessory", "Received updateState -> Homekit Lock for " + this.unit.node.getName() + " - " + this.unit.getName() + " -> " + this.unit.status);
     this.me.getCharacteristic(this.homebridge.Characteristic.LockCurrentState).updateValue(this.unit.status);
   }
 

package/accessories/mood.js

@@ -51,7 +51,7 @@ class Mood extends accessory_1.Accessory {
     }
     updateState() {
         this.me.getCharacteristic(this.homebridge.Characteristic.On).updateValue(!!this.unit.value);
-        // log("accessory", "Received status change -> update accessory -> " + this.unit.getName() + " -> On = " + !!this.unit.value);
+        (0, logger_1.log)("accessory", "Received status change -> update accessory -> " + this.unit.getName() + " -> On = " + !!this.unit.value);
     }
 }
 exports.Mood = Mood;

package/accessories/mood.ts

@@ -62,7 +62,7 @@ export class Mood extends Accessory {
   }
   updateState() {
     this.me.getCharacteristic(this.homebridge.Characteristic.On).updateValue(!!this.unit.value);
-    // log("accessory", "Received status change -> update accessory -> " + this.unit.getName() + " -> On = " + !!this.unit.value);
+    log("accessory", "Received status change -> update accessory -> " + this.unit.getName() + " -> On = " + !!this.unit.value);
   }
 
 }

package/accessories/temperature.js

@@ -34,7 +34,7 @@ class Temperature extends accessory_1.Accessory {
     }
     updateState() {
         this.me.getCharacteristic(this.homebridge.Characteristic.CurrentTemperature).updateValue(this.unit.value / 10.0);
-        // log("accessory", "Received updateState -> Homekit Temperature -> " + this.unit.getName() + " -> temp = " +(<number>this.unit.value) / 10.0);
+        (0, logger_1.log)("accessory", "Received updateState -> Homekit Temperature -> " + this.unit.getName() + " -> temp = " + this.unit.value / 10.0);
     }
 }
 exports.Temperature = Temperature;

package/accessories/temperature.ts

@@ -42,7 +42,7 @@ export class Temperature extends Accessory {
 
   updateState() {
     this.me.getCharacteristic(this.homebridge.Characteristic.CurrentTemperature).updateValue((<number>this.unit.value) / 10.0);
-    // log("accessory", "Received updateState -> Homekit Temperature -> " + this.unit.getName() + " -> temp = " +(<number>this.unit.value) / 10.0);
+    log("accessory", "Received updateState -> Homekit Temperature -> " + this.unit.getName() + " -> temp = " +(<number>this.unit.value) / 10.0);
   }
 
 }

package/accessories/windowcovering.js

@@ -134,7 +134,7 @@ class WindowCovering extends accessory_1.Accessory {
     // called in response to Duotecno status update
     updateState() {
         const value = this.DT2HB(this.unit.status);
-        // log("accessory", "Received updateState -> Homekit WindowCovering for " + this.unit.node.getName() + " - " + this.unit.getName() + " -> " + this.unit.status + " / " + this.unit.value + " -> passing: " + value);
+        (0, logger_1.log)("accessory", "Received updateState -> Homekit WindowCovering for " + this.unit.node.getName() + " - " + this.unit.getName() + " -> " + this.unit.status + " / " + this.unit.value + " -> passing: " + value);
         this.me.getCharacteristic(this.homebridge.Characteristic.CurrentPosition).updateValue(value);
         if (this.unit.status == types_1.UnitState.kClosing) {
             this.me.getCharacteristic(this.homebridge.Characteristic.TargetPosition).updateValue(0);

package/accessories/windowcovering.ts

@@ -161,7 +161,7 @@ export class WindowCovering extends Accessory {
   // called in response to Duotecno status update
   updateState() {
     const value = this.DT2HB(this.unit.status);
-    // log("accessory", "Received updateState -> Homekit WindowCovering for " + this.unit.node.getName() + " - " + this.unit.getName() + " -> " + this.unit.status + " / " + this.unit.value + " -> passing: " + value);
+    log("accessory", "Received updateState -> Homekit WindowCovering for " + this.unit.node.getName() + " - " + this.unit.getName() + " -> " + this.unit.status + " / " + this.unit.value + " -> passing: " + value);
 
     this.me.getCharacteristic(this.homebridge.Characteristic.CurrentPosition).updateValue(value);
 

package/duotecno/logger.js

@@ -16,6 +16,7 @@ exports.logSettings = {
     "*": LogLevel.log,
     "master": LogLevel.log,
     "protocol": LogLevel.noLog,
+    "accessory": LogLevel.noLog,
     "webapp": LogLevel.log,
     "p1": LogLevel.log,
     "shelly": LogLevel.log,

package/duotecno/logger.ts

@@ -11,6 +11,7 @@ export const logSettings = {
   "*": LogLevel.log,
   "master": LogLevel.log,
   "protocol": LogLevel.noLog,
+  "accessory": LogLevel.noLog,
   "webapp": LogLevel.log,
   "p1": LogLevel.log,
   "shelly": LogLevel.log,

package/duotecno/types.ts

@@ -17,7 +17,7 @@ export enum NodeType { kNoNode = 0, kStandardNode = 1, kGatewayNode = 4, kModemN
 
 // States
 export enum UnitState { kOpening = 4, kClosing = 3, kOpen = 2, kClosed = 1, kStopped = 0 };
-export enum UnitMotorCmd { kClose = 5, kOpen = 4, kStop = 3};
+export enum UnitMotorCmd { kClose = 5, kOpen = 4, kStop = 3 };
 
 export enum UnitType { 
   kNoType = 0, kDimmer = 1, kSwitch = 2, kInput = 3, 

package/package.json

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

package/server/proxy.js

@@ -13,6 +13,7 @@ exports.kEmptyProxy = {
     debug: false,
     kind: "solo" // default running under PM2 or other process manager -> don't restart yourself
 };
+const config = Object.assign({}, exports.kEmptyProxy);
 function setProxyConfig(c) {
     // set the proxy config
     if (c) {
@@ -24,19 +25,17 @@ function setProxyConfig(c) {
         config.debug = !!c.debug;
         config.kind = c.kind || exports.kEmptyProxy.kind;
     }
-    else {
-        config = Object.assign({}, exports.kEmptyProxy);
-    }
     return config;
 }
 exports.setProxyConfig = setProxyConfig;
-let config;
 try {
-    setProxyConfig(require('../config-proxy.json'));
+    const configPath = require.resolve('../config-proxy.json');
+    console.log(`Using config file at: ${configPath}`);
+    setProxyConfig(require(configPath));
 }
 catch (e) {
     setProxyConfig();
-    console.log("No config-proxy.json found, using default values from homebridge or other.");
+    log("No config-proxy.json found, using default values from homebridge or other.");
 }
 let connectionChecker = null;
 const gCloudConnections = [];
@@ -102,15 +101,19 @@ function cleanStart(restart = false) {
     }
     if (restart) {
         if (config.uniqueId) {
+            log(`[PROXY] → [CLOUD] Starting proxy with config: ${JSON.stringify(config, null, 2)}`);
             makeNewCloudConnection(config.masterAddress, config.masterPort, config.cloudServer, config.cloudPort, config.uniqueId);
-            // check every 5 second for a free connection
+            // check every 10 second for a free connection
             connectionChecker = setInterval(() => {
                 const freeConnection = gCloudConnections.find((ctx) => (!ctx.deviceSocket) && ctx.cloudSocket && (ctx.cloudSocket.readyState === 'open'));
-                if (!freeConnection) {
+                if (freeConnection) {
+                    log(`[PROXY] → [CLOUD] Found free connection #${freeConnection.uniqueId} to the Cloud -> we're still ok.`);
+                }
+                else {
                     error(`[PROXY] → [CLOUD] No free connections available, restarting proxy. ***************************`);
                     cleanStart(true);
                 }
-            }, 5000);
+            }, 10 * 1000);
         }
         else {
             log(`[PROXY] → [CLOUD] Not restarting, no unique ID configured, not starting the proxy.`);
@@ -309,8 +312,10 @@ function restart() {
 // Graceful shutdown handler
 function handleShutdown(signal) {
     log(`[PROXY] → [SYSTEM] Received ${signal}, shutting down gracefully...`);
-    if (connectionChecker)
+    if (connectionChecker) {
         clearInterval(connectionChecker);
+        connectionChecker = null;
+    }
     // Close all cloud connections
     gCloudConnections.forEach(ctx => ctx.cleanupSockets());
     gCloudConnections.splice(0, gCloudConnections.length);

package/server/proxy.ts

@@ -26,6 +26,8 @@ export const kEmptyProxy: ProxyConfig = {
   kind: "solo"          // default running under PM2 or other process manager -> don't restart yourself
 }
 
+const config = {...kEmptyProxy};
+
 export function setProxyConfig(c?: ProxyConfig): ProxyConfig {
   // set the proxy config
   if (c) {
@@ -36,22 +38,23 @@ export function setProxyConfig(c?: ProxyConfig): ProxyConfig {
     config.uniqueId = c.uniqueId || kEmptyProxy.uniqueId;
     config.debug = !!c.debug;
     config.kind = c.kind || kEmptyProxy.kind;
-  } else {
-    config = {...kEmptyProxy};
   }
   return config;
 }
 
-let config: ProxyConfig;
+
 try {
-  setProxyConfig(require('../config-proxy.json'));
+  const configPath = require.resolve('../config-proxy.json');
+  console.log(`Using config file at: ${configPath}`);
+  setProxyConfig(require(configPath));
+
 } catch (e) {
   setProxyConfig();
-  console.log("No config-proxy.json found, using default values from homebridge or other.");
+  log("No config-proxy.json found, using default values from homebridge or other.");
 }
 
 
-let connectionChecker: NodeJS.Timer | null = null;
+let connectionChecker: NodeJS.Timeout | null = null;
 
 const gCloudConnections: Array<Context> = [];
 const kDebug = config.debug || false; // enable debug mode from config
@@ -130,18 +133,21 @@ export function cleanStart(restart: boolean = false) {
 
   if (restart) {
     if (config.uniqueId) {
+      log(`[PROXY] → [CLOUD] Starting proxy with config: ${JSON.stringify(config, null, 2)}`);
       makeNewCloudConnection(config.masterAddress, config.masterPort, config.cloudServer, config.cloudPort, config.uniqueId);
 
-      // check every 5 second for a free connection
+      // check every 10 second for a free connection
       connectionChecker = setInterval(() => {
         const freeConnection = gCloudConnections.find((ctx: Context) =>
           (!ctx.deviceSocket) && ctx.cloudSocket && (ctx.cloudSocket.readyState === 'open')
         );
-        if (!freeConnection) {
+        if (freeConnection) {
+          log(`[PROXY] → [CLOUD] Found free connection #${freeConnection.uniqueId} to the Cloud -> we're still ok.`);
+        } else {
           error(`[PROXY] → [CLOUD] No free connections available, restarting proxy. ***************************`);
           cleanStart(true);
         }
-      }, 5000);
+      }, 10 * 1000);
 
     } else {
       log(`[PROXY] → [CLOUD] Not restarting, no unique ID configured, not starting the proxy.`);
@@ -385,7 +391,10 @@ function restart() {
 function handleShutdown(signal: string) {
   log(`[PROXY] → [SYSTEM] Received ${signal}, shutting down gracefully...`);
 
-  if (connectionChecker)clearInterval(connectionChecker);
+  if (connectionChecker) {
+    clearInterval(connectionChecker);
+    connectionChecker = null;
+  }
 
   // Close all cloud connections
   gCloudConnections.forEach(ctx => ctx.cleanupSockets());