@smarterhomeapp/homebridge-vantage 1.0.1 → 1.0.2

package/dist/platform.js

@@ -219,7 +219,7 @@ class VantagePlatform {
             const svc = acc.getService(S.Thermostat);
             if (!svc || !dev)
                 return;
-            this.log.info(`thermostatIndoorModeChange (VID=${vid}, Name=${dev.name}, Mode=${mode}, TargetTemp=${targetTemp})`);
+            this.log.debug(`thermostatIndoorModeChange (VID=${vid}, Name=${dev.name}, Mode=${mode}, TargetTemp=${targetTemp})`);
             // this.log.info(`Before update - dev.mode=${dev.mode}, dev.targetTemp=${dev.targetTemp}, dev.heating=${dev.heating}, dev.cooling=${dev.cooling}`);
             if (targetTemp === -1) {
                 // Only update mode when targetTemp is -1 (mode change without temperature)

package/dist/settings.d.ts

@@ -1,2 +1,2 @@
-export declare const PLUGIN_NAME = "homebridge-vantage";
+export declare const PLUGIN_NAME = "@smarterhomeapp/homebridge-vantage";
 export declare const PLATFORM_NAME = "VantageControls";

package/dist/settings.js

@@ -1,5 +1,5 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.PLATFORM_NAME = exports.PLUGIN_NAME = void 0;
-exports.PLUGIN_NAME = 'homebridge-vantage';
+exports.PLUGIN_NAME = '@smarterhomeapp/homebridge-vantage';
 exports.PLATFORM_NAME = 'VantageControls';

package/dist/vantageInfusion.js

@@ -93,8 +93,13 @@ class VantageInfusion extends events_1.EventEmitter {
             this.isInsecureCfg = false;
         }
         else {
-            this.isInsecureCmd = await this.portUsable(3001, 'STATUS ALL\n');
-            this.isInsecureCfg = await this.portUsable(2001, '<IIntrospection><GetInterfaces><call></call></GetInterfaces></IIntrospection>\n');
+            // Probe sequentially to avoid controller session limits
+            const cmdResult = await this.portUsable(3001, 'STATUS ALL\n');
+            // brief backoff before probing config port
+            await new Promise((r) => setTimeout(r, 150));
+            const cfgResult = await this.portUsable(2001, '<IIntrospection><GetInterfaces><call></call></GetInterfaces></IIntrospection>\n');
+            this.isInsecureCmd = !!cmdResult;
+            this.isInsecureCfg = !!cfgResult;
         }
         // Legacy-style port logs
         this.opts.log.info(this.isInsecureCmd ? 'Using insecure port: 3001' : 'Using SSL port: 3010');
@@ -104,21 +109,46 @@ class VantageInfusion extends events_1.EventEmitter {
     portUsable(port, probe) {
         return new Promise((resolve) => {
             let sawData = false;
+            this.opts.log.debug(`Testing ${this.opts.ipaddress}:${port} with probe: "${probe.trim()}"`);
             const sock = net.connect({ host: this.opts.ipaddress, port }, () => {
+                this.opts.log.debug(`Connected to ${this.opts.ipaddress}:${port}`);
+                // improve send behavior
+                sock.setNoDelay(true);
+                // keep a post-connect timeout; fail if no reply in time
+                sock.setTimeout(8000);
+                // write immediately
+                this.opts.log.debug(`Sending initial probe on port ${port}`);
                 sock.write(probe);
+                // send a second probe shortly after to coax a response
+                setTimeout(() => {
+                    if (!sawData) {
+                        this.opts.log.debug(`Re-sending probe on port ${port}`);
+                        sock.write(probe);
+                    }
+                }, 150);
+            });
+            sock.once('connect', () => {
+                this.opts.log.debug(`Port ${port} connection established`);
             });
-            sock.setTimeout(5000, () => sock.destroy());
-            sock.once('connect', () => sock.setTimeout(0));
-            sock.on('data', () => {
+            sock.on('data', (data) => {
+                const text = data.toString();
+                this.opts.log.debug(`Port ${port} received data: ${text.slice(0, 200).replace(/\r/g, '\\r').replace(/\n/g, '\\n')}`);
                 sawData = true;
                 resolve(true);
                 sock.destroy();
             });
+            sock.on('timeout', () => {
+                this.opts.log.debug(`Port ${port} post-connect timeout`);
+                sock.destroy();
+            });
             sock.on('close', () => {
+                this.opts.log.debug(`Port ${port} closed, sawData: ${sawData}`);
                 if (!sawData)
                     resolve(false);
             });
-            sock.on('error', () => {
+            sock.on('error', (err) => {
+                const msg = err instanceof Error ? `${err.code || ''} ${err.message}` : String(err);
+                this.opts.log.debug(`Port ${port} error: ${msg}`);
                 /* resolve on close path */
             });
         });
@@ -573,7 +603,7 @@ class VantageInfusion extends events_1.EventEmitter {
                     }
                 });
                 socket.on('close', () => {
-                    this.opts.log.info("closing" + shouldBreak);
+                    this.opts.log.debug("closing " + shouldBreak);
                     if (!finishOnce._done && readObjects.length) {
                         const xml = '<Project><Objects>' +
                             readObjects.map((o) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@smarterhomeapp/homebridge-vantage",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "description": "Homebridge plugin for Vantage InFusion (dynamic platform, HB 1.6+ / 2.0 ready)",
   "license": "MIT",
   "keywords": [