homebridge-zwave-usb 1.2.3 → 1.2.6

package/dist/accessories/ControllerAccessory.js

@@ -14,7 +14,7 @@ class ControllerAccessory {
         this.platform = platform;
         this.controller = controller;
         const homeId = this.controller.homeId;
-        const uuid = this.platform.api.hap.uuid.generate(`ZWaveController-${homeId}`);
+        const uuid = this.platform.api.hap.uuid.generate(`homebridge-zwave-usb-controller-${homeId}`);
         const existingAccessory = this.platform.accessories.find((accessory) => accessory.UUID === uuid);
         if (existingAccessory) {
             this.platformAccessory = existingAccessory;
@@ -25,6 +25,11 @@ class ControllerAccessory {
             this.platform.api.registerPlatformAccessories('homebridge-zwave-usb', 'ZWaveUSB', [this.platformAccessory]);
             this.platform.accessories.push(this.platformAccessory);
         }
+        // Set accessory information
+        this.platformAccessory.getService(this.platform.Service.AccessoryInformation)
+            .setCharacteristic(this.platform.Characteristic.Manufacturer, 'Aeotec / Z-Wave JS')
+            .setCharacteristic(this.platform.Characteristic.Model, 'Z-Wave USB Controller')
+            .setCharacteristic(this.platform.Characteristic.SerialNumber, homeId?.toString() || 'Unknown');
         this.inclusionService =
             this.platformAccessory.getService('Inclusion Mode') ||
                 this.platformAccessory.addService(this.platform.Service.Switch, 'Inclusion Mode', 'Inclusion');

package/dist/accessories/ZWaveAccessory.js

@@ -24,6 +24,13 @@ class ZWaveAccessory {
             this.platform.api.registerPlatformAccessories('homebridge-zwave-usb', 'ZWaveUSB', [this.platformAccessory]);
             this.platform.accessories.push(this.platformAccessory);
         }
+        // Set accessory information
+        const manufacturer = this.node.deviceConfig?.manufacturer || 'Unknown';
+        const model = this.node.deviceConfig?.label || `Node ${this.node.nodeId}`;
+        this.platformAccessory.getService(this.platform.Service.AccessoryInformation)
+            .setCharacteristic(this.platform.Characteristic.Manufacturer, manufacturer)
+            .setCharacteristic(this.platform.Characteristic.Model, model)
+            .setCharacteristic(this.platform.Characteristic.SerialNumber, `Node ${this.node.nodeId}`);
     }
     addFeature(feature) {
         this.features.push(feature);

package/dist/features/MultilevelSwitchFeature.d.ts

@@ -1,6 +1,7 @@
 import { BaseFeature } from './ZWaveFeature';
 export declare class MultilevelSwitchFeature extends BaseFeature {
     private service;
+    private lastKnownBrightness;
     init(): void;
     update(): void;
     private handleGetOn;

package/dist/features/MultilevelSwitchFeature.js

@@ -4,6 +4,7 @@ exports.MultilevelSwitchFeature = void 0;
 const ZWaveFeature_1 = require("./ZWaveFeature");
 class MultilevelSwitchFeature extends ZWaveFeature_1.BaseFeature {
     service;
+    lastKnownBrightness = 0;
     init() {
         const subType = this.endpoint.index.toString();
         this.service = this.getService(this.platform.Service.Lightbulb, undefined, subType);
@@ -25,13 +26,15 @@ class MultilevelSwitchFeature extends ZWaveFeature_1.BaseFeature {
         if (typeof value === 'number') {
             const isOn = value > 0;
             this.service.updateCharacteristic(this.platform.Characteristic.On, isOn);
-            if (isOn && value <= 99) {
-                // Brightness in Z-Wave is 0-99, in HomeKit 0-100
+            if (value <= 99) {
+                this.lastKnownBrightness = value;
                 this.service.updateCharacteristic(this.platform.Characteristic.Brightness, value);
             }
-            else if (isOn && value === 255) {
+            else if (value === 255 && isOn) {
                 // 255 = restore previous level
-                // We don't necessarily know the previous level, so we leave it as is or default to 100
+                // Use our tracked brightness, default to 100 if unknown
+                const brightness = this.lastKnownBrightness > 0 ? this.lastKnownBrightness : 100;
+                this.service.updateCharacteristic(this.platform.Characteristic.Brightness, brightness);
             }
         }
     }

package/dist/platform/ZWaveUsbPlatform.js

@@ -68,18 +68,31 @@ class ZWaveUsbPlatform {
                 for (const acc of this.zwaveAccessories.values()) {
                     managedUuids.add(acc.platformAccessory.UUID);
                 }
-                const orphaned = this.accessories.filter(acc => !managedUuids.has(acc.UUID));
+                this.log.debug(`Reconciliation: ${managedUuids.size} managed accessories, ${this.accessories.length} in cache.`);
+                const orphaned = this.accessories.filter(acc => {
+                    const isOrphaned = !managedUuids.has(acc.UUID);
+                    if (isOrphaned) {
+                        this.log.info(`Found orphaned accessory in cache: ${acc.displayName} (${acc.UUID})`);
+                    }
+                    return isOrphaned;
+                });
                 if (orphaned.length > 0) {
                     this.log.info(`Removing ${orphaned.length} orphaned accessories from cache...`);
-                    this.api.unregisterPlatformAccessories('homebridge-zwave-usb', 'ZWaveUSB', orphaned);
-                    for (const orphan of orphaned) {
-                        const index = this.accessories.indexOf(orphan);
-                        if (index !== -1) {
-                            this.accessories.splice(index, 1);
+                    try {
+                        this.api.unregisterPlatformAccessories('homebridge-zwave-usb', 'ZWaveUSB', orphaned);
+                        for (const orphan of orphaned) {
+                            const index = this.accessories.indexOf(orphan);
+                            if (index !== -1) {
+                                this.accessories.splice(index, 1);
+                            }
                         }
+                        this.log.info('Successfully removed orphaned accessories.');
+                    }
+                    catch (err) {
+                        this.log.error('Failed to unregister orphaned accessories:', err);
                     }
                 }
-            }, 5000);
+            }, 10000);
         }
         catch (err) {
             this.log.error('Failed to connect to Z-Wave controller:', err);
@@ -90,6 +103,10 @@ class ZWaveUsbPlatform {
     }
     handleNodeReady(node) {
         this.log.info(`Node ${node.nodeId} ready`);
+        if (!node.ready) {
+            this.log.warn(`Node ${node.nodeId} reported ready but node.ready is false, skipping...`);
+            return;
+        }
         const homeId = this.zwaveController?.homeId;
         if (!homeId) {
             this.log.error(`Cannot create accessory for Node ${node.nodeId} - Home ID not available`);

package/dist/zwave/ZWaveController.js

@@ -32,27 +32,11 @@ class ZWaveController extends events_1.EventEmitter {
                 securityKeys.S2_AccessControl = Buffer.from(this.options.securityKeys.S2_AccessControl, 'hex');
             }
         }
-        // Redirect Z-Wave JS logs to Homebridge
-        const customLogTransport = {
-            log: (info, next) => {
-                const message = info.message || info;
-                const label = info.label ? `[${info.label}] ` : '';
-                const output = `${label}${message}`;
-                if (this.options.debug || info.level === 'error' || info.level === 'warn') {
-                    this.log.info(`[Z-Wave JS] ${output}`);
-                }
-                else {
-                    this.log.debug(`[Z-Wave JS] ${output}`);
-                }
-                if (next) {
-                    next();
-                }
-            }
-        };
+        // Z-Wave JS logging configuration - use default console logging to avoid winston compatibility issues
+        // Custom logging can be added in future if needed
         const logConfig = {
             enabled: true,
             level: this.options.debug ? 'debug' : 'info',
-            transports: [customLogTransport] // eslint-disable-line @typescript-eslint/no-explicit-any
         };
         const storagePath = this.options.storagePath || process.cwd();
         this.driver = new zwave_js_1.Driver(this.serialPort, {
@@ -190,6 +174,12 @@ class ZWaveController extends events_1.EventEmitter {
             if (listeners) {
                 node.off('ready', listeners.ready);
                 node.off('value updated', listeners.value);
+                if (listeners.interviewStageCompleted) {
+                    node.off('interview stage completed', listeners.interviewStageCompleted);
+                }
+                if (listeners.interviewFailed) {
+                    node.off('interview failed', listeners.interviewFailed);
+                }
             }
         }
         this.nodeListeners.clear();

package/dist/zwave/interfaces.d.ts

@@ -17,6 +17,7 @@ export interface IZWaveNode {
     name?: string;
     deviceConfig?: {
         label?: string;
+        manufacturer?: string;
     };
     ready: boolean;
     interviewStage: unknown;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "homebridge-zwave-usb",
   "displayName": "Z-Wave USB",
-  "version": "1.2.3",
+  "version": "1.2.6",
   "description": "A Homebridge dynamic platform plugin for Z-Wave USB controllers using zwave-js.",
   "repository": {
     "type": "git",