homebridge-deconz 1.0.1 → 1.0.3

package/lib/DeconzAccessory/Light.js

@@ -46,45 +46,108 @@ class Light extends DeconzAccessory {
     }
 
     const params = {}
-    if (this.values.serviceName === 'Outlet') {
-      this.service.addCharacteristicDelegate({
-        key: 'lockPhysicalControls',
-        Characteristic: this.Characteristics.hap.LockPhysicalControls
-      })
-      params.onDelegate = this.service.characteristicDelegate('on')
-      params.lastOnDelegate = this.service.addCharacteristicDelegate({
+
+    if (this.values.serviceName === 'Valve') {
+      // No history
+    } else if (
+      this.servicesByServiceName[this.values.serviceName].length > 1 ||
+      this.values.serviceName === 'Light'
+    ) {
+      params.lightOnDelegate = this.service.characteristicDelegate('on')
+      params.lastLightOnDelegate = this.service.addCharacteristicDelegate({
         key: 'lastActivation',
         Characteristic: this.Characteristics.eve.LastActivation,
         silent: true
       })
-    } else if (this.values.serviceName !== 'Valve') {
-      params.lightOnDelegate = this.service.characteristicDelegate('on')
-      params.lastLightOnDelegate = this.service.addCharacteristicDelegate({
+    } else { // Outlet or Switch
+      if (this.values.serviceName === 'Outlet') {
+        this.service.addCharacteristicDelegate({
+          key: 'lockPhysicalControls',
+          Characteristic: this.Characteristics.hap.LockPhysicalControls
+        })
+      }
+      params.onDelegate = this.service.characteristicDelegate('on')
+      params.lastOnDelegate = this.service.addCharacteristicDelegate({
         key: 'lastActivation',
         Characteristic: this.Characteristics.eve.LastActivation,
         silent: true
       })
     }
-    if (this.service.characteristicDelegate('totalConsumption') != null) {
-      params.totalConsumptionDelegate = this.service.characteristicDelegate('totalConsumption')
-      if (this.service.values.consumption === undefined) {
+
+    if (this.servicesByServiceName.Consumption?.length === 1) {
+      const service = this.servicesByServiceName.Consumption[0]
+      params.totalConsumptionDelegate = service.characteristicDelegate('totalConsumption')
+      if (service.values.consumption === undefined) {
         // Power to be computed by history if not exposed by device
-        params.computedConsumptionDelegate = this.service.addCharacteristicDelegate({
+        params.computedConsumptionDelegate = service.addCharacteristicDelegate({
           key: 'consumption',
           Characteristic: this.Characteristics.eve.Consumption,
           unit: ' W'
         })
       }
-    } else if (this.service.characteristicDelegate('consumption') != null) {
-      params.consumptionDelegate = this.service.characteristicDelegate('consumption')
+    } else if (this.servicesByServiceName.Power?.length === 1) {
+      const service = this.servicesByServiceName.Power[0]
+      params.consumptionDelegate = service.characteristicDelegate('consumption')
       // Total Consumption to be computed by history
-      params.computedTotalConsumptionDelegate = this.service.addCharacteristicDelegate({
+      params.computedTotalConsumptionDelegate = service.addCharacteristicDelegate({
         key: 'totalConsumption',
         Characteristic: this.Characteristics.eve.TotalConsumption,
         unit: ' kWh'
       })
     }
-    this.historyService = new ServiceDelegate.History(this, params)
+
+    if (Object.keys(params).length > 0) {
+      this.historyService = new ServiceDelegate.History(this, params)
+    }
+
+    if (this.servicesByServiceName[this.values.serviceName].length === 1) {
+      if (
+        this.values.serviceName === 'Outlet' &&
+        this.servicesByServiceName.Consumption == null &&
+        this.servicesByServiceName.Power == null
+      ) {
+        // Dumb Outlet
+        const service = new ServiceDelegate(this, {
+          name: this.name + ' Consumption',
+          Service: this.Services.eve.Consumption,
+          hidden: true
+        })
+        service.addCharacteristicDelegate({
+          key: 'dummyTotalConsumption',
+          Characteristic: this.Characteristics.eve.TotalConsumption,
+          props: {
+            perms: [
+              this.Characteristic.Perms.PAIRED_READ,
+              this.Characteristic.Perms.NOTIFY,
+              this.Characteristic.Perms.HIDDEN
+            ]
+          },
+          silent: true,
+          value: 0
+        })
+      }
+    } else {
+      for (const i in this.servicesByServiceName[this.values.serviceName]) {
+        const service = this.servicesByServiceName[this.values.serviceName][i]
+        service.addCharacteristicDelegate({
+          key: 'index',
+          Characteristic: this.Characteristics.hap.ServiceLabelIndex,
+          silent: true,
+          value: Number(i)
+        })
+        if (i === '0') {
+          continue
+        }
+        this.historyService?.addLastOnDelegate(
+          service.characteristicDelegate('on'),
+          service.addCharacteristicDelegate({
+            key: 'lastActivation',
+            Characteristic: this.Characteristics.eve.LastActivation,
+            silent: true
+          })
+        )
+      }
+    }
 
     setImmediate(() => {
       this.debug('initialised')

package/lib/DeconzAccessory/Sensor.js

@@ -106,10 +106,10 @@ class Sensor extends DeconzAccessory {
         params.vocDensityDelegate = service.characteristicDelegate('vocDensity')
       }
     }
-    if (this.servicesByServiceName.Flag != null) {
+    if (this.servicesByServiceName.Flag?.length === 1) {
       const service = this.servicesByServiceName.Flag[0]
-      params.switchOnDelegate = service.characteristicDelegate('on')
-      params.lastSwitchOnDelegate = service.addCharacteristicDelegate({
+      params.onDelegate = service.characteristicDelegate('on')
+      params.lastOnDelegate = service.addCharacteristicDelegate({
         key: 'lastActivation',
         Characteristic: this.Characteristics.eve.LastActivation,
         silent: true

package/lib/DeconzAccessory/WarningDevice.js

@@ -35,8 +35,8 @@ class WarningDevice extends DeconzAccessory {
 
     const params = {}
     if (this.servicesByServiceName.WarningDevice?.length === 1) {
-      params.switchOnDelegate = this.service.characteristicDelegate('on')
-      params.lastSwitchOnDelegate = this.service.addCharacteristicDelegate({
+      params.onDelegate = this.service.characteristicDelegate('on')
+      params.lastOnDelegate = this.service.addCharacteristicDelegate({
         key: 'lastActivation',
         Characteristic: this.Characteristics.eve.LastActivation,
         silent: true

package/lib/DeconzAccessory/index.js

@@ -164,16 +164,12 @@ class DeconzAccessory extends AccessoryDelegate {
     } else if (params.serviceName === 'Battery') {
       service = this.servicesByServiceName.Battery?.[0]
     } else if (params.serviceName === 'Consumption') {
-      service = this.servicesByServiceName.Outlet?.[0] ||
-        this.servicesByServiceName.Light?.[0] ||
-        this.servicesByServiceName.Power?.[0]
+      service = this.servicesByServiceName.Power?.[0]
       if (service != null) {
         service.addResource(resource)
       }
     } else if (params.serviceName === 'Power') {
-      service = this.servicesByServiceName.Outlet?.[0] ||
-        this.servicesByServiceName.Light?.[0] ||
-        this.servicesByServiceName.Consumption?.[0]
+      service = this.servicesByServiceName.Consumption?.[0]
       if (service != null) {
         service.addResource(resource)
       }
@@ -223,17 +219,6 @@ class DeconzAccessory extends AccessoryDelegate {
     return service
   }
 
-  settings = {
-    anyOn: { service: 'Lightbulb', type: 'bool' },
-    buttonRepeat: { service: 'Button', type: 'boolMap' },
-    effects: { service: 'Light', type: 'boolMap' },
-    expose: { type: 'bool' },
-    logLevel: { type: 'int', min: 0, max: 3 },
-    lowBatteryThreshold: { service: 'Battery', type: 'int', min: 10, max: 100, factor: 100 },
-    offset: { service: 'Temperature', type: 'number', min: -5, max: 5 },
-    splitLight: { service: 'Light' }
-  }
-
   onUiGet (details = false) {
     const resource = this.device.resourceBySubtype[this.device.primary]
     const body = {

package/lib/DeconzService/Consumption.js

@@ -56,7 +56,9 @@ class Consumption extends DeconzService.SensorsResource {
   }
 
   constructor (accessory, resource, params = {}) {
-    params.Service = accessory.Services.my.Resource
+    params.name = accessory.name + ' Consumption'
+    params.Service = accessory.Services.eve.Consumption
+    params.exposeConfiguredName = true
     super(accessory, resource, params)
 
     Consumption.addResource(this, resource)

package/lib/DeconzService/Power.js

@@ -66,7 +66,9 @@ class Power extends DeconzService.SensorsResource {
   }
 
   constructor (accessory, resource, params = {}) {
-    params.Service = accessory.Services.my.Resource
+    params.name = accessory.name + ' Consumption'
+    params.Service = accessory.Services.eve.Consumption
+    params.exposeConfiguredName = true
     super(accessory, resource, params)
 
     Power.addResource(this, resource)

package/lib/DeconzService/WarningDevice.js

@@ -6,6 +6,7 @@
 'use strict'
 
 const DeconzService = require('../DeconzService')
+const { timeout } = require('homebridge-lib')
 
 class WarningDevice extends DeconzService.LightsResource {
   constructor (accessory, resource, params = {}) {
@@ -33,11 +34,10 @@ class WarningDevice extends DeconzService.LightsResource {
             body = { alert: 'lselect', ontime: onTime }
           }
         }
-        this.put(body)
+        await this.put(this.statePath, body)
         if (value) {
-          this.timer = setTimeout(() => {
-            this.values.on = false
-          }, onTime * 1000)
+          await timeout(onTime * 1000)
+          this.values.on = false
         }
       }
     })

package/lib/DeconzService/WindowCovering.js

@@ -5,8 +5,8 @@
 
 'use strict'
 
-const { timeout } = require('homebridge-lib')
 const DeconzService = require('../DeconzService')
+const { timeout } = require('homebridge-lib')
 
 class WindowCovering extends DeconzService.LightsResource {
   constructor (accessory, resource, params = {}) {
@@ -34,7 +34,7 @@ class WindowCovering extends DeconzService.LightsResource {
         return
       }
       this.values.targetPosition = Math.round(this.values.targetPosition / 5) * 5
-      await this.setPosition()
+      return this.setPosition()
     })
 
     this.addCharacteristicDelegate({
@@ -46,8 +46,8 @@ class WindowCovering extends DeconzService.LightsResource {
     this.addCharacteristicDelegate({
       key: 'holdPosition',
       Characteristic: this.Characteristics.hap.HoldPosition
-    }).on('didSet', () => {
-      this.put({ stop: true })
+    }).on('didSet', async () => {
+      await this.put(this.statePath, { stop: true })
       this.values.positionState = this.Characteristics.hap.PositionState.STOPPED
     })
 
@@ -60,7 +60,7 @@ class WindowCovering extends DeconzService.LightsResource {
           return
         }
         if (this.values.currentPosition !== 100) {
-          await this.setPosition()
+          return this.setPosition()
         }
       })
     }
@@ -80,7 +80,7 @@ class WindowCovering extends DeconzService.LightsResource {
         if (!fromHomeKit) {
           return
         }
-        await this.put({ speed: value })
+        await this.put('/config', { speed: value })
       })
     }
 
@@ -90,7 +90,7 @@ class WindowCovering extends DeconzService.LightsResource {
         Characteristic: this.Characteristics.my.PositionChange
       }).on('didSet', async (value) => {
         if (value !== 0) {
-          this.put({ lift_inc: -value })
+          await this.put(this.statePath, { lift_inc: -value })
           await timeout(this.platform.config.waitTimeReset)
           this.values.positionChange = 0
         }
@@ -120,7 +120,7 @@ class WindowCovering extends DeconzService.LightsResource {
         ? this.Characteristics.hap.PositionState.INCREASING
         : this.Characteristics.hap.PositionState.DECREASING
     this.moving = new Date()
-    await this.put({ lift })
+    return this.put(this.statePath, { lift })
   }
 
   updateState (state) {

package/lib/DeconzService/index.js

@@ -49,7 +49,7 @@ class DeconzService extends ServiceDelegate {
   constructor (accessory, resource, params) {
     super(accessory, {
       id: resource.id,
-      name: resource.body.name,
+      name: params.name ?? resource.body.name,
       Service: params.Service,
       subtype: resource.subtype,
       primaryService: params.primaryService,

package/package.json

@@ -4,7 +4,7 @@
   "displayName": "Homebridge deCONZ",
   "author": "Erik Baauw",
   "license": "Apache-2.0",
-  "version": "1.0.1",
+  "version": "1.0.3",
   "keywords": [
     "homebridge-plugin",
     "homekit",
@@ -24,11 +24,11 @@
   "engines": {
     "deCONZ": "2.23.1",
     "homebridge": "^1.6.1",
-    "node": "20.7.0||^20||^18"
+    "node": "20.8.0||^20||^18"
   },
   "dependencies": {
-    "hb-deconz-tools": "~1.0.7",
-    "homebridge-lib": "~6.5.1"
+    "hb-deconz-tools": "~1.0.8",
+    "homebridge-lib": "~6.6.1"
   },
   "scripts": {
     "prepare": "standard && rm -rf out && jsdoc -c jsdoc.json",