homebridge-yoto 0.0.15 → 0.0.16

package/lib/playerAccessory.js

@@ -514,10 +514,10 @@ export class YotoPlayerAccessory {
    * @param {YotoDeviceStatus} status - Status data
    */
   handleStatusUpdate (status) {
-    this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Received status update:`, JSON.stringify(status, null, 2))
-    this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Status - batteryLevel:`, status.batteryLevel)
-    this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Status - userVolume:`, status.userVolume)
-    this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Status - volume:`, status.volume)
+    this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Received status update:`, JSON.stringify(status, null, 2))
+    this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Status - batteryLevel:`, status.batteryLevel)
+    this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Status - userVolume:`, status.userVolume)
+    this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Status - volume:`, status.volume)
 
     this.currentStatus = status
     this.lastUpdateTime = Date.now()
@@ -789,15 +789,15 @@ export class YotoPlayerAccessory {
    * @returns {Promise<CharacteristicValue>}
    */
   async getVolume () {
-    this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] getVolume - currentStatus:`, this.currentStatus)
-    this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] getVolume - userVolume:`, this.currentStatus?.userVolume)
+    this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] getVolume - currentStatus:`, this.currentStatus)
+    this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] getVolume - userVolume:`, this.currentStatus?.userVolume)
 
     if (!this.currentStatus || this.currentStatus.userVolume === undefined) {
-      this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] getVolume - returning default: 50`)
+      this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] getVolume - returning default: 50`)
       return 50
     }
     const volume = Number(this.currentStatus.userVolume) || 50
-    this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] getVolume - returning:`, volume)
+    this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] getVolume - returning:`, volume)
     return volume
   }
 
@@ -859,15 +859,15 @@ export class YotoPlayerAccessory {
    * @returns {Promise<CharacteristicValue>}
    */
   async getBatteryLevel () {
-    this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] getBatteryLevel - currentStatus:`, this.currentStatus)
-    this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] getBatteryLevel - batteryLevel:`, this.currentStatus?.batteryLevel)
+    this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] getBatteryLevel - currentStatus:`, this.currentStatus)
+    this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] getBatteryLevel - batteryLevel:`, this.currentStatus?.batteryLevel)
 
     if (!this.currentStatus || this.currentStatus.batteryLevel === undefined) {
-      this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] getBatteryLevel - returning default: 100`)
+      this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] getBatteryLevel - returning default: 100`)
       return 100
     }
     const battery = Number(this.currentStatus.batteryLevel) || 100
-    this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] getBatteryLevel - returning:`, battery)
+    this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] getBatteryLevel - returning:`, battery)
     return battery
   }
 
@@ -992,12 +992,16 @@ export class YotoPlayerAccessory {
    * @returns {Promise<CharacteristicValue>}
    */
   async getDisplayBrightness () {
-    if (!this.currentStatus) {
+    if (!this.currentStatus || this.currentStatus.dnowBrightness === undefined) {
       return 100
     }
 
     // Use current brightness value (0-100)
-    return Math.min(100, Math.max(0, this.currentStatus.dnowBrightness))
+    const brightness = Number(this.currentStatus.dnowBrightness)
+    if (isNaN(brightness)) {
+      return 100
+    }
+    return Math.min(100, Math.max(0, brightness))
   }
 
   /**
@@ -1220,6 +1224,9 @@ export class YotoPlayerAccessory {
     try {
       const config = await this.platform.yotoApi.getDeviceConfig(this.device.deviceId)
       const limit = parseInt(config.config.maxVolumeLimit || '16')
+      if (isNaN(limit)) {
+        return 100
+      }
       // Map 0-16 to 0-100
       return Math.round((limit / 16) * 100)
     } catch (error) {
@@ -1275,6 +1282,9 @@ export class YotoPlayerAccessory {
     try {
       const config = await this.platform.yotoApi.getDeviceConfig(this.device.deviceId)
       const limit = parseInt(config.config.nightMaxVolumeLimit || '16')
+      if (isNaN(limit)) {
+        return 100
+      }
       // Map 0-16 to 0-100
       return Math.round((limit / 16) * 100)
     } catch (error) {
@@ -1352,6 +1362,9 @@ export class YotoPlayerAccessory {
       const config = await this.platform.yotoApi.getDeviceConfig(this.device.deviceId)
       const hex = config.config.ambientColour || '#000000'
       const { h } = this.hexToHsv(hex)
+      if (isNaN(h)) {
+        return 0
+      }
       return h
     } catch (error) {
       this.log.error(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Failed to get ambient light hue:`, error)
@@ -1379,6 +1392,9 @@ export class YotoPlayerAccessory {
       const config = await this.platform.yotoApi.getDeviceConfig(this.device.deviceId)
       const hex = config.config.ambientColour || '#000000'
       const { s } = this.hexToHsv(hex)
+      if (isNaN(s)) {
+        return 0
+      }
       return s
     } catch (error) {
       this.log.error(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Failed to get ambient light saturation:`, error)
@@ -1404,11 +1420,15 @@ export class YotoPlayerAccessory {
   async getAmbientLightBrightness () {
     try {
       const config = await this.platform.yotoApi.getDeviceConfig(this.device.deviceId)
-      const hex = config.config.ambientColour || '#000000'
+      const hex = config.config.ambientColour
       const { v } = this.hexToHsv(hex)
-      return v
+      const brightness = Math.round(v)
+      if (isNaN(brightness)) {
+        return 100
+      }
+      return brightness
     } catch (error) {
-      this.log.error(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Failed to get ambient light brightness:`, error)
+      this.log.error(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Failed to get ambient brightness:`, error)
       return 100
     }
   }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "homebridge-yoto",
   "description": "Control your Yoto players through Apple HomeKit with real-time MQTT updates",
-  "version": "0.0.15",
+  "version": "0.0.16",
   "author": "Bret Comnes <bcomnes@gmail.com> (https://bret.io)",
   "bugs": {
     "url": "https://github.com/bcomnes/homebridge-yoto/issues"