homebridge-lovesac-stealthtech 1.1.0 → 1.1.2

package/dist/accessory.js

@@ -473,7 +473,7 @@ class LovesacAccessory {
                         ? this.Characteristic.RotationSpeed : this.Characteristic.Brightness;
                     this.volumeService.getCharacteristic(levelChar).updateValue(percent);
                     this.volumeService.getCharacteristic(this.Characteristic.On)
-                        .updateValue(!this.device.state.mute && this.device.state.volume > 0);
+                        .updateValue(this.device.state.power && !this.device.state.mute && this.device.state.volume > 0);
                 }
                 break;
             case constants_1.ResponseCode.Mute:
@@ -481,7 +481,7 @@ class LovesacAccessory {
                     .updateValue(this.device.state.mute);
                 if (this.volumeService) {
                     this.volumeService.getCharacteristic(this.Characteristic.On)
-                        .updateValue(!this.device.state.mute && this.device.state.volume > 0);
+                        .updateValue(this.device.state.power && !this.device.state.mute && this.device.state.volume > 0);
                 }
                 break;
             case constants_1.ResponseCode.Source:
@@ -489,10 +489,12 @@ class LovesacAccessory {
                     .updateValue(this.device.state.source + 1);
                 break;
             case constants_1.ResponseCode.Preset:
-                this.updatePresetSwitches(this.device.state.preset);
+                if (this.device.state.power) {
+                    this.updatePresetSwitches(this.device.state.preset);
+                }
                 break;
             case constants_1.ResponseCode.QuietMode:
-                if (this.quietModeService) {
+                if (this.quietModeService && this.device.state.power) {
                     this.quietModeService.getCharacteristic(this.Characteristic.On)
                         .updateValue(this.device.state.quietMode);
                 }

package/dist/ble/BleClient.d.ts

@@ -19,6 +19,8 @@ export declare class BleClient implements IBleClient {
     constructor(log: Logger);
     get resolvedAddress(): string;
     connect(address?: string): Promise<void>;
+    private connectPeripheral;
+    private scanWithRetry;
     disconnect(): Promise<void>;
     isConnected(): boolean;
     write(characteristicUuid: string, data: Buffer): Promise<void>;

package/dist/ble/BleClient.js

@@ -25,26 +25,27 @@ class BleClient {
             return;
         }
         const gen = ++this.connectGeneration;
-        let peripheral;
-        if (address) {
-            this.log.debug('BLE: Starting scan for %s...', address);
-            peripheral = await this.scanForDevice(address);
-            if (!peripheral) {
-                throw new Error(`Device ${address} not found`);
+        // Fast path: reuse cached peripheral from a prior connection (skip scan)
+        if (this.peripheral) {
+            this.log.debug('BLE: Reconnecting to cached peripheral %s...', this._resolvedAddress);
+            try {
+                await this.connectPeripheral(this.peripheral, gen);
+                return;
             }
-        }
-        else {
-            this.log.debug('BLE: Starting auto-discovery scan...');
-            peripheral = await this.scanForAnyDevice();
-            if (!peripheral) {
-                throw new Error('No Lovesac StealthTech device found');
+            catch (err) {
+                this.log.debug('BLE: Cached peripheral connect failed: %s — falling back to scan', err.message);
+                this.peripheral = null;
             }
         }
+        // Slow path: scan for the device (with one retry)
+        const peripheral = await this.scanWithRetry(address, gen);
         const resolvedId = peripheral.address !== '' && peripheral.address !== 'unknown'
             ? peripheral.address
             : peripheral.id ?? peripheral.uuid ?? 'unknown';
-        this.log.debug('BLE: Connecting to %s...', resolvedId);
         this._resolvedAddress = resolvedId;
+        await this.connectPeripheral(peripheral, gen);
+    }
+    async connectPeripheral(peripheral, gen) {
         // Register disconnect handler BEFORE connecting to avoid race (P0-2).
         // Capture the generation so a stale handler from a timed-out attempt does
         // not clear state that belongs to a newer connection.
@@ -52,7 +53,6 @@ class BleClient {
             this.log.debug('BLE: Disconnected');
             if (gen === this.connectGeneration) {
                 this._connected = false;
-                this.peripheral = null;
                 this.characteristics = {};
             }
         });
@@ -79,6 +79,33 @@ class BleClient {
         }
         this.log.debug('BLE: Found %d characteristics', Object.keys(this.characteristics).length);
     }
+    async scanWithRetry(address, gen) {
+        const label = address ?? 'auto-discovery';
+        const scanFn = address
+            ? () => this.scanForDevice(address)
+            : () => this.scanForAnyDevice();
+        this.log.debug('BLE: Starting scan for %s...', label);
+        let peripheral = await scanFn();
+        if (peripheral) {
+            return peripheral;
+        }
+        // First scan missed — retry once after a short delay
+        if (gen !== this.connectGeneration) {
+            throw new Error('Connection attempt superseded');
+        }
+        this.log.info('BLE: Scan found nothing, retrying in %ds...', settings_1.BLE_SCAN_RETRY_DELAY / 1000);
+        await new Promise(r => setTimeout(r, settings_1.BLE_SCAN_RETRY_DELAY));
+        if (gen !== this.connectGeneration) {
+            throw new Error('Connection attempt superseded');
+        }
+        peripheral = await scanFn();
+        if (peripheral) {
+            return peripheral;
+        }
+        throw new Error(address
+            ? `Device ${address} not found`
+            : 'No Lovesac StealthTech device found');
+    }
     async disconnect() {
         if (this.peripheral && this._connected) {
             try {
@@ -89,8 +116,8 @@ class BleClient {
             }
         }
         this._connected = false;
-        this.peripheral = null;
         this.characteristics = {};
+        // Keep this.peripheral cached for fast reconnect
     }
     isConnected() {
         return this._connected;

package/dist/protocol/LovesacDevice.d.ts

@@ -30,6 +30,7 @@ export declare class LovesacDevice {
     private static readonly MAX_POLL_INTERVAL_MS;
     private onPollSuccess;
     private onPollFailure;
+    private backoffPollInterval;
     private markUnreachable;
     stopPolling(): void;
     getResolvedAddress(): string;

package/dist/protocol/LovesacDevice.js

@@ -147,18 +147,25 @@ class LovesacDevice {
     onPollFailure(message) {
         this.consecutiveFailures++;
         this.log.warn('Background poll failed (%d/%d): %s', this.consecutiveFailures, settings_1.UNREACHABLE_THRESHOLD, message);
-        if (this.consecutiveFailures >= settings_1.UNREACHABLE_THRESHOLD && this.reachable) {
-            this.markUnreachable();
+        // Apply backoff at each threshold multiple (6, 12, 18, …) while unreachable.
+        // First crossing also marks the device unreachable and resets cached state.
+        if (this.consecutiveFailures >= settings_1.UNREACHABLE_THRESHOLD
+            && this.consecutiveFailures % settings_1.UNREACHABLE_THRESHOLD === 0) {
+            this.backoffPollInterval();
+            if (this.reachable) {
+                this.markUnreachable();
+            }
         }
     }
-    markUnreachable() {
-        this.reachable = false;
-        // Exponential backoff: double the poll interval, capped at 10 minutes
+    backoffPollInterval() {
         const newInterval = Math.min(this.pollIntervalMs * 2, LovesacDevice.MAX_POLL_INTERVAL_MS);
         if (newInterval !== this.pollIntervalMs) {
             this.pollIntervalMs = newInterval;
             this.log.info('Poll interval backed off to %ds', this.pollIntervalMs / 1000);
         }
+    }
+    markUnreachable() {
+        this.reachable = false;
         this.log.warn('Device unreachable after %d consecutive poll failures — resetting cached state', this.consecutiveFailures);
         // Reset state to sentinels so next successful connection triggers full re-sync
         (0, responses_1.resetState)(this.state);

package/dist/settings.d.ts

@@ -21,7 +21,8 @@ export declare const BLE_CONNECT_TIMEOUT = 30000;
 export declare const BLE_WRITE_TIMEOUT = 10000;
 export declare const BLE_DISCONNECT_TIMEOUT = 10000;
 export declare const BLE_DISCOVER_TIMEOUT = 15000;
-export declare const UNREACHABLE_THRESHOLD = 3;
+export declare const BLE_SCAN_RETRY_DELAY = 2000;
+export declare const UNREACHABLE_THRESHOLD = 6;
 /**
  * Race a promise against a timeout. Rejects with a descriptive error if
  * the timeout fires first. The underlying promise is NOT cancelled — the

package/dist/settings.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.UNREACHABLE_THRESHOLD = exports.BLE_DISCOVER_TIMEOUT = exports.BLE_DISCONNECT_TIMEOUT = exports.BLE_WRITE_TIMEOUT = exports.BLE_CONNECT_TIMEOUT = exports.BLE_SCAN_TIMEOUT = exports.MAX_VOLUME = exports.CharUUID = exports.SOFA_SERVICE_UUID_SHORT = exports.SOFA_SERVICE_UUID = exports.PLATFORM_NAME = exports.PLUGIN_NAME = void 0;
+exports.UNREACHABLE_THRESHOLD = exports.BLE_SCAN_RETRY_DELAY = exports.BLE_DISCOVER_TIMEOUT = exports.BLE_DISCONNECT_TIMEOUT = exports.BLE_WRITE_TIMEOUT = exports.BLE_CONNECT_TIMEOUT = exports.BLE_SCAN_TIMEOUT = exports.MAX_VOLUME = exports.CharUUID = exports.SOFA_SERVICE_UUID_SHORT = exports.SOFA_SERVICE_UUID = exports.PLATFORM_NAME = exports.PLUGIN_NAME = void 0;
 exports.withTimeout = withTimeout;
 exports.errorMessage = errorMessage;
 exports.resolveDeviceConfig = resolveDeviceConfig;
@@ -29,8 +29,10 @@ exports.BLE_CONNECT_TIMEOUT = 30_000;
 exports.BLE_WRITE_TIMEOUT = 10_000;
 exports.BLE_DISCONNECT_TIMEOUT = 10_000;
 exports.BLE_DISCOVER_TIMEOUT = 15_000;
+// Delay between scan retry attempts (milliseconds)
+exports.BLE_SCAN_RETRY_DELAY = 2000;
 // Consecutive poll failures before marking device unreachable
-exports.UNREACHABLE_THRESHOLD = 3;
+exports.UNREACHABLE_THRESHOLD = 6;
 /**
  * Race a promise against a timeout. Rejects with a descriptive error if
  * the timeout fires first. The underlying promise is NOT cancelled — the

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "homebridge-lovesac-stealthtech",
-  "version": "1.1.0",
+  "version": "1.1.2",
   "description": "Homebridge plugin for Lovesac StealthTech Sound + Charge BLE control",
   "main": "dist/index.js",
   "files": [