homebridge-zwave-usb 2.8.1 → 2.8.4

package/dist/accessories/ZWaveAccessory.js

@@ -39,13 +39,13 @@ class ZWaveAccessory {
             }
         }
         const existingAccessory = this.platform.accessories.find((accessory) => accessory.UUID === uuid);
+        const nodeName = this.node.name || this.node.deviceConfig?.label || `Node ${this.node.nodeId}`;
         if (existingAccessory) {
             this.platformAccessory = existingAccessory;
         }
         else {
-            const accessoryName = `Node ${this.node.nodeId}`;
-            this.platform.log.info(`Creating new accessory for ${accessoryName} (UUID: ${uuid})`);
-            this.platformAccessory = new this.platform.api.platformAccessory(accessoryName, uuid);
+            this.platform.log.info(`Creating new accessory for ${nodeName} (UUID: ${uuid})`);
+            this.platformAccessory = new this.platform.api.platformAccessory(nodeName, uuid);
             this.platform.api.registerPlatformAccessories('homebridge-zwave-usb', 'ZWaveUSB', [
                 this.platformAccessory,
             ]);
@@ -53,12 +53,16 @@ class ZWaveAccessory {
         }
         // 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)
+        const model = this.node.deviceConfig?.label || nodeName;
+        const infoService = this.platformAccessory.getService(this.platform.Service.AccessoryInformation);
+        infoService
             .setCharacteristic(this.platform.Characteristic.Manufacturer, manufacturer)
             .setCharacteristic(this.platform.Characteristic.Model, model)
             .setCharacteristic(this.platform.Characteristic.SerialNumber, `Node ${this.node.nodeId}`);
+        if (!infoService.testCharacteristic(this.platform.Characteristic.Name)) {
+            infoService.addOptionalCharacteristic(this.platform.Characteristic.Name);
+        }
+        infoService.updateCharacteristic(this.platform.Characteristic.Name, nodeName);
         /**
          * Helper to normalize UUIDs for reliable comparison during metadata pruning.
          */
@@ -96,11 +100,13 @@ class ZWaveAccessory {
         infoService.getCharacteristic(this.platform.Characteristic.Name)
             .setProps({
             // eslint-disable-next-line @typescript-eslint/no-explicit-any
-            format: settings_1.HAPFormat.STRING,
-            // eslint-disable-next-line @typescript-eslint/no-explicit-any
             perms: [settings_1.HAPPerm.PAIRED_READ, settings_1.HAPPerm.NOTIFY],
         });
         infoService.updateCharacteristic(this.platform.Characteristic.Name, newName);
+        // Force metadata invalidation by updating the revision
+        // eslint-disable-next-line @typescript-eslint/no-var-requires
+        const version = require('../../package.json').version;
+        infoService.updateCharacteristic(this.platform.Characteristic.SoftwareRevision, version);
         // Update the Model and Serial if they were using the generic name
         const model = this.node.deviceConfig?.label || newName;
         infoService.setCharacteristic(this.platform.Characteristic.Model, model);

package/dist/features/ZWaveFeature.js

@@ -81,12 +81,14 @@ class BaseFeature {
                     // eslint-disable-next-line @typescript-eslint/no-explicit-any
                     this.accessory.addService(new serviceType(serviceName));
         }
-        // Explicitly set the name characteristic to ensure it's displayed correctly
-        service
-            .getCharacteristic(this.platform.Characteristic.Name)
-            .setProps({
+        // Mark the first functional service as primary to help HomeKit with naming/tiles
+        if (this.managedServices.length === 0) {
             // eslint-disable-next-line @typescript-eslint/no-explicit-any
-            format: settings_1.HAPFormat.STRING,
+            service.setPrimaryService(true);
+        }
+        // Explicitly set the name characteristic with NOTIFY to ensure it's displayed correctly
+        service.getCharacteristic(this.platform.Characteristic.Name)
+            .setProps({
             // eslint-disable-next-line @typescript-eslint/no-explicit-any
             perms: [settings_1.HAPPerm.PAIRED_READ, settings_1.HAPPerm.NOTIFY],
         });
@@ -95,12 +97,9 @@ class BaseFeature {
         if (!service.testCharacteristic(this.platform.Characteristic.ConfiguredName)) {
             service.addOptionalCharacteristic(this.platform.Characteristic.ConfiguredName);
         }
-        service
-            .getCharacteristic(this.platform.Characteristic.ConfiguredName)
+        service.getCharacteristic(this.platform.Characteristic.ConfiguredName)
             .setProps({
             // eslint-disable-next-line @typescript-eslint/no-explicit-any
-            format: settings_1.HAPFormat.STRING,
-            // eslint-disable-next-line @typescript-eslint/no-explicit-any
             perms: [settings_1.HAPPerm.PAIRED_READ, settings_1.HAPPerm.NOTIFY],
         });
         service.updateCharacteristic(this.platform.Characteristic.ConfiguredName, serviceName);

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "homebridge-zwave-usb",
   "displayName": "Homebridge Z-Wave USB",
-  "version": "2.8.1",
+  "version": "2.8.4",
   "description": "A Homebridge dynamic platform plugin for Z-Wave USB controllers using zwave-js.",
   "license": "Polyform-Noncommercial-1.0.0",
   "funding": {