homebridge-bedjet 0.1.5 → 0.1.7

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "homebridge-bedjet",
   "displayName": "BedJet",
-  "version": "0.1.5",
+  "version": "0.1.7",
   "description": "Homebridge plugin for BedJet V3 via Bluetooth LE",
   "license": "MIT",
   "main": "dist/index.js",

package/src/bedjet/BedJet.ts

@@ -149,22 +149,26 @@ export class BedJet extends EventEmitter {
 
     peripheral.once('disconnect', () => this._onDisconnected());
 
-    await peripheral.connectAsync();
+    try {
+      await peripheral.connectAsync();
+    } catch (err) {
+      throw new Error(`[${this.config.name}] connectAsync failed: ${err}`);
+    }
     this.log.info(`[${this.config.name}] Connected — discovering services…`);
 
-    // Wrap service discovery in a timeout — it hangs indefinitely on Linux/BlueZ
-    // if the adapter is busy or the device is in a bad state.
-    const { characteristics } = await Promise.race([
-      peripheral.discoverAllServicesAndCharacteristicsAsync(),
-      new Promise<never>((_, reject) =>
-        setTimeout(() => reject(new Error(`[${this.config.name}] Service discovery timed out`)), 10_000),
-      ),
-    ]);
+    // Use async discovery — works on Linux when the device is already bonded
+    let characteristics: Characteristic[];
+    try {
+      const result = await peripheral.discoverAllServicesAndCharacteristicsAsync();
+      characteristics = result.characteristics;
+    } catch (err) {
+      throw new Error(`[${this.config.name}] discoverAllServices failed: ${err}`);
+    }
 
     this.log.info(`[${this.config.name}] Found ${characteristics.length} characteristics`);
     for (const char of characteristics) {
       const uuid = normalize(char.uuid);
-      this.log.debug(`[${this.config.name}]   char: ${uuid}`);
+      this.log.info(`[${this.config.name}]   char: ${uuid}`);
       if (uuid === NORM_STATUS_UUID) {
         this.statusChar = char;
         this.log.info(`[${this.config.name}] Found status characteristic`);
@@ -174,25 +178,34 @@ export class BedJet extends EventEmitter {
       }
     }
 
+    this.log.info(`[${this.config.name}] Char check: status=${!!this.statusChar} command=${!!this.commandChar}`);
+
     if (!this.commandChar || !this.statusChar) {
-      throw new Error(`[${this.config.name}] Required characteristics not found (status=${!!this.statusChar} command=${!!this.commandChar})`);
+      throw new Error(`[${this.config.name}] Required characteristics not found`);
     }
 
     // Subscribe to notifications on the status characteristic
     this.log.info(`[${this.config.name}] Subscribing to notifications…`);
-    await Promise.race([
-      this.statusChar.subscribeAsync(),
-      new Promise<never>((_, reject) =>
-        setTimeout(() => reject(new Error(`[${this.config.name}] Subscribe timed out`)), 10_000),
-      ),
-    ]);
+    try {
+      await this.statusChar.subscribeAsync();
+    } catch (err) {
+      throw new Error(`[${this.config.name}] subscribeAsync failed: ${err}`);
+    }
     this.log.info(`[${this.config.name}] Subscribed — reading device status…`);
     this.statusChar.on('data', (data: Buffer) => this._handleNotification(data));
 
     // Read extended status and device name
-    await this._readDeviceStatus();
+    try {
+      await this._readDeviceStatus();
+    } catch (err) {
+      this.log.warn(`[${this.config.name}] _readDeviceStatus failed: ${err}`);
+    }
     this.log.info(`[${this.config.name}] Reading device name…`);
-    await this._readDeviceName();
+    try {
+      await this._readDeviceName();
+    } catch (err) {
+      this.log.warn(`[${this.config.name}] _readDeviceName failed: ${err}`);
+    }
 
     this.reconnectAttempts = 0;
     this._state = { ...this._state, isConnected: true };