homebridge-yoto 0.0.18 → 0.0.20

package/lib/auth.js

@@ -35,7 +35,7 @@ export class YotoAuth {
    * @returns {Promise<YotoApiDeviceCodeResponse>}
    */
   async initiateDeviceFlow () {
-    this.log.info(LOG_PREFIX.AUTH, 'Initiating device authorization flow...')
+    this.log.debug(LOG_PREFIX.AUTH, 'Initiating device authorization flow...')
     this.log.debug(LOG_PREFIX.AUTH, `Client ID: ${this.clientId}`)
     this.log.debug(LOG_PREFIX.AUTH, `Endpoint: ${YOTO_OAUTH_DEVICE_CODE_URL}`)
     this.log.debug(LOG_PREFIX.AUTH, `Scope: ${OAUTH_SCOPE}`)
@@ -97,7 +97,7 @@ export class YotoAuth {
     const startTime = Date.now()
     const timeout = OAUTH_DEVICE_CODE_TIMEOUT
 
-    this.log.info(LOG_PREFIX.AUTH, 'Waiting for user authorization...')
+    this.log.debug(LOG_PREFIX.AUTH, 'Waiting for user authorization...')
 
     while (Date.now() - startTime < timeout) {
       try {

package/lib/platform.js

@@ -219,7 +219,7 @@ export class YotoPlatform {
    */
   async performDeviceFlow () {
     this.log.warn(LOG_PREFIX.PLATFORM, ERROR_MESSAGES.NO_AUTH)
-    this.log.info(LOG_PREFIX.PLATFORM, 'Starting OAuth flow...')
+    this.log.debug(LOG_PREFIX.PLATFORM, 'Starting OAuth flow...')
 
     const tokenResponse = await this.auth.authorize()
 
@@ -258,7 +258,7 @@ export class YotoPlatform {
    * @param {number} expiresAt - New expiration timestamp
    */
   handleTokenRefresh (accessToken, refreshToken, expiresAt) {
-    this.log.info(LOG_PREFIX.PLATFORM, 'Token refreshed')
+    this.log.debug(LOG_PREFIX.PLATFORM, 'Token refreshed')
     this.config.accessToken = accessToken
     this.config.refreshToken = refreshToken
     this.config.tokenExpiresAt = expiresAt
@@ -329,7 +329,7 @@ export class YotoPlatform {
    * @param {PlatformAccessory<YotoAccessoryContext>} accessory - Cached accessory
    */
   configureAccessory (accessory) {
-    this.log.info(LOG_PREFIX.PLATFORM, 'Loading accessory from cache:', accessory.displayName)
+    this.log.debug(LOG_PREFIX.PLATFORM, 'Loading accessory from cache:', accessory.displayName)
 
     // Add to our tracking map
     this.accessories.set(accessory.UUID, accessory)
@@ -341,7 +341,7 @@ export class YotoPlatform {
    */
   async discoverDevices () {
     try {
-      this.log.info(LOG_PREFIX.PLATFORM, 'Discovering Yoto devices...')
+      this.log.debug(LOG_PREFIX.PLATFORM, 'Discovering Yoto devices...')
 
       // Fetch devices from API
       const devices = await this.yotoApi.getDevices()
@@ -381,7 +381,7 @@ export class YotoPlatform {
 
     if (existingAccessory) {
       // Accessory exists - update it
-      this.log.info(LOG_PREFIX.PLATFORM, 'Restoring existing accessory:', device.name)
+      this.log.debug(LOG_PREFIX.PLATFORM, 'Restoring existing accessory:', device.name)
 
       // Update context with fresh device data
       const typedAccessory = /** @type {PlatformAccessory<YotoAccessoryContext>} */ (existingAccessory)
@@ -403,7 +403,7 @@ export class YotoPlatform {
       })
     } else {
       // Create new accessory
-      this.log.info(LOG_PREFIX.PLATFORM, 'Adding new accessory:', device.name)
+      this.log.debug(LOG_PREFIX.PLATFORM, 'Adding new accessory:', device.name)
 
       // Create platform accessory
       // eslint-disable-next-line new-cap
@@ -447,7 +447,7 @@ export class YotoPlatform {
 
     for (const [uuid, accessory] of this.accessories) {
       if (!this.discoveredUUIDs.includes(uuid)) {
-        this.log.info(LOG_PREFIX.PLATFORM, 'Removing stale accessory:', accessory.displayName)
+        this.log.debug(LOG_PREFIX.PLATFORM, 'Removing stale accessory:', accessory.displayName)
         staleAccessories.push(accessory)
       }
     }
@@ -473,7 +473,7 @@ export class YotoPlatform {
    * Shutdown handler - cleanup connections
    */
   async shutdown () {
-    this.log.info(LOG_PREFIX.PLATFORM, 'Shutting down...')
+    this.log.debug(LOG_PREFIX.PLATFORM, 'Shutting down...')
 
     // Stop status polling
     this.stopStatusPolling()

package/lib/playerAccessory.js

@@ -38,6 +38,9 @@ export class YotoPlayerAccessory {
     this.currentEvents = accessory.context.lastEvents || null
     this.lastUpdateTime = Date.now()
 
+    // Cache for device config
+    this.deviceConfig = null
+
     // Create dedicated MQTT client for this device
     this.mqtt = new YotoMqtt(this.log)
 
@@ -110,6 +113,15 @@ export class YotoPlayerAccessory {
    * @returns {Promise<void>}
    */
   async initialize () {
+    // Fetch device config for cached access
+    try {
+      this.deviceConfig = await this.platform.yotoApi.getDeviceConfig(this.device.deviceId)
+      this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Device config loaded`)
+    } catch (error) {
+      this.log.warn(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Failed to load device config:`, error)
+      // Continue without config - getters will return defaults
+    }
+
     await this.connectMqtt()
     // Status polling is handled at platform level
   }
@@ -514,14 +526,14 @@ export class YotoPlayerAccessory {
    * @param {YotoDeviceStatus | {status: YotoDeviceStatus}} statusMessage - Status data or wrapped status
    */
   handleStatusUpdate (statusMessage) {
-    this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Raw MQTT status message:`, JSON.stringify(statusMessage, null, 2))
+    this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Raw MQTT status message:`, JSON.stringify(statusMessage, null, 2))
 
     // Unwrap status if it's nested under a 'status' property
     const status = /** @type {YotoDeviceStatus} */ ('status' in statusMessage ? statusMessage.status : statusMessage)
 
-    this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Unwrapped status - batteryLevel:`, status.batteryLevel)
-    this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Unwrapped status - userVolume:`, status.userVolume)
-    this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Unwrapped status - volume:`, status.volume)
+    this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Unwrapped status - batteryLevel:`, status.batteryLevel)
+    this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Unwrapped status - userVolume:`, status.userVolume)
+    this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Unwrapped status - volume:`, status.volume)
 
     this.currentStatus = status
     this.lastUpdateTime = Date.now()
@@ -536,12 +548,12 @@ export class YotoPlayerAccessory {
    * @param {YotoPlaybackEvents | {events: YotoPlaybackEvents}} eventsMessage - Playback events or wrapped events
    */
   handleEventsUpdate (eventsMessage) {
-    this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Raw MQTT events message:`, JSON.stringify(eventsMessage, null, 2))
+    this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Raw MQTT events message:`, JSON.stringify(eventsMessage, null, 2))
 
     // Unwrap events if it's nested under an 'events' property
     const events = /** @type {YotoPlaybackEvents} */ ('events' in eventsMessage ? eventsMessage.events : eventsMessage)
 
-    this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Unwrapped events - cardId:`, events.cardId)
+    this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Unwrapped events - cardId:`, events.cardId)
     this.currentEvents = events
     this.lastUpdateTime = Date.now()
     this.accessory.context.lastEvents = events
@@ -560,7 +572,7 @@ export class YotoPlayerAccessory {
    * @param {import('./types.js').MqttCommandResponse} response - Command response
    */
   handleCommandResponse (response) {
-    this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Command response:`, response)
+    this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Command response:`, response)
   }
 
   /**
@@ -799,16 +811,16 @@ export class YotoPlayerAccessory {
    * @returns {Promise<CharacteristicValue>}
    */
   async getVolume () {
-    this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] getVolume - currentEvents:`, this.currentEvents)
-    this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] getVolume - events.volume:`, this.currentEvents?.volume)
+    this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] getVolume - currentEvents:`, this.currentEvents)
+    this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] getVolume - events.volume:`, this.currentEvents?.volume)
 
     // Volume comes from events, not status
     if (!this.currentEvents || this.currentEvents.volume === undefined) {
-      this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] getVolume - no volume in events, returning default: 50`)
+      this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] getVolume - no volume in events, returning default: 50`)
       return 50
     }
     const volume = Number(this.currentEvents.volume) || 50
-    this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] getVolume - returning volume:`, volume)
+    this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] getVolume - returning volume:`, volume)
     return volume
   }
 
@@ -871,15 +883,15 @@ export class YotoPlayerAccessory {
    * @returns {Promise<CharacteristicValue>}
    */
   async getBatteryLevel () {
-    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)
+    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)
 
     if (!this.currentStatus || this.currentStatus.batteryLevel === undefined) {
-      this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] getBatteryLevel - returning default: 100`)
+      this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] getBatteryLevel - returning default: 100`)
       return 100
     }
     const battery = Number(this.currentStatus.batteryLevel) || 100
-    this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] getBatteryLevel - returning:`, battery)
+    this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] getBatteryLevel - returning:`, battery)
     return battery
   }
 
@@ -977,20 +989,23 @@ export class YotoPlayerAccessory {
     this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Set display on:`, value)
 
     try {
-      // Get current config to update brightness settings
-      const config = await this.platform.yotoApi.getDeviceConfig(this.device.deviceId)
+      // Refresh config if not cached
+      if (!this.deviceConfig) {
+        this.deviceConfig = await this.platform.yotoApi.getDeviceConfig(this.device.deviceId)
+      }
 
       if (value) {
-        // Turn on - restore to auto or previous brightness
-        config.config.dayDisplayBrightness = 'auto'
-        config.config.nightDisplayBrightness = 'auto'
+        // Turn on - restore to previous brightness or default
+        const brightness = this.currentStatus?.dnowBrightness || 100
+        this.deviceConfig.config.dayDisplayBrightness = String(brightness)
+        this.deviceConfig.config.nightDisplayBrightness = String(brightness)
       } else {
         // Turn off - set brightness to 0
-        config.config.dayDisplayBrightness = '0'
-        config.config.nightDisplayBrightness = '0'
+        this.deviceConfig.config.dayDisplayBrightness = '0'
+        this.deviceConfig.config.nightDisplayBrightness = '0'
       }
 
-      await this.platform.yotoApi.updateDeviceConfig(this.device.deviceId, config)
+      await this.platform.yotoApi.updateDeviceConfig(this.device.deviceId, this.deviceConfig)
     } catch (error) {
       this.log.error(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Failed to set display on:`, error)
       throw new this.platform.api.hap.HapStatusError(
@@ -1024,16 +1039,19 @@ export class YotoPlayerAccessory {
     this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Set display brightness:`, value)
 
     try {
-      const config = await this.platform.yotoApi.getDeviceConfig(this.device.deviceId)
+      // Refresh config if not cached
+      if (!this.deviceConfig) {
+        this.deviceConfig = await this.platform.yotoApi.getDeviceConfig(this.device.deviceId)
+      }
 
-      // Map 0-100 to brightness string
-      const brightness = String(Math.round(Number(value)))
+      // Clamp brightness to 0-100
+      const brightness = Math.max(0, Math.min(100, Number(value)))
 
       // Update both day and night brightness
-      config.config.dayDisplayBrightness = brightness
-      config.config.nightDisplayBrightness = brightness
+      this.deviceConfig.config.dayDisplayBrightness = String(brightness)
+      this.deviceConfig.config.nightDisplayBrightness = String(brightness)
 
-      await this.platform.yotoApi.updateDeviceConfig(this.device.deviceId, config)
+      await this.platform.yotoApi.updateDeviceConfig(this.device.deviceId, this.deviceConfig)
     } catch (error) {
       this.log.error(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Failed to set display brightness:`, error)
       throw new this.platform.api.hap.HapStatusError(
@@ -1058,12 +1076,16 @@ export class YotoPlayerAccessory {
    * @param {CharacteristicValue} value - Enabled state
    */
   async setBluetoothEnabled (value) {
-    this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Set Bluetooth enabled:`, value)
+    this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Set Bluetooth:`, value)
 
     try {
-      const config = await this.platform.yotoApi.getDeviceConfig(this.device.deviceId)
-      config.config.bluetoothEnabled = value ? '1' : '0'
-      await this.platform.yotoApi.updateDeviceConfig(this.device.deviceId, config)
+      // Refresh config if not cached
+      if (!this.deviceConfig) {
+        this.deviceConfig = await this.platform.yotoApi.getDeviceConfig(this.device.deviceId)
+      }
+
+      this.deviceConfig.config.bluetoothEnabled = value ? 'true' : 'false'
+      await this.platform.yotoApi.updateDeviceConfig(this.device.deviceId, this.deviceConfig)
     } catch (error) {
       this.log.error(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Failed to set Bluetooth:`, error)
       throw new this.platform.api.hap.HapStatusError(
@@ -1091,11 +1113,15 @@ export class YotoPlayerAccessory {
     this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Set repeat all:`, value)
 
     try {
-      const config = await this.platform.yotoApi.getDeviceConfig(this.device.deviceId)
-      config.config.repeatAll = Boolean(value)
-      await this.platform.yotoApi.updateDeviceConfig(this.device.deviceId, config)
+      // Refresh config if not cached
+      if (!this.deviceConfig) {
+        this.deviceConfig = await this.platform.yotoApi.getDeviceConfig(this.device.deviceId)
+      }
+
+      this.deviceConfig.config.repeatAll = Boolean(value)
+      await this.platform.yotoApi.updateDeviceConfig(this.device.deviceId, this.deviceConfig)
     } catch (error) {
-      this.log.error(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Failed to set repeat:`, error)
+      this.log.error(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Failed to set repeat all:`, error)
       throw new this.platform.api.hap.HapStatusError(
         this.platform.api.hap.HAPStatus.SERVICE_COMMUNICATION_FAILURE
       )
@@ -1118,12 +1144,16 @@ export class YotoPlayerAccessory {
    * @param {CharacteristicValue} value - Enabled state
    */
   async setBtHeadphonesEnabled (value) {
-    this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Set BT headphones enabled:`, value)
+    this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Set BT headphones:`, value)
 
     try {
-      const config = await this.platform.yotoApi.getDeviceConfig(this.device.deviceId)
-      config.config.btHeadphonesEnabled = Boolean(value)
-      await this.platform.yotoApi.updateDeviceConfig(this.device.deviceId, config)
+      // Refresh config if not cached
+      if (!this.deviceConfig) {
+        this.deviceConfig = await this.platform.yotoApi.getDeviceConfig(this.device.deviceId)
+      }
+
+      this.deviceConfig.config.btHeadphonesEnabled = Boolean(value)
+      await this.platform.yotoApi.updateDeviceConfig(this.device.deviceId, this.deviceConfig)
     } catch (error) {
       this.log.error(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Failed to set BT headphones:`, error)
       throw new this.platform.api.hap.HapStatusError(
@@ -1233,16 +1263,19 @@ export class YotoPlayerAccessory {
    * @returns {Promise<CharacteristicValue>}
    */
   async getDayVolumeLimit () {
+    if (!this.deviceConfig) {
+      return 100 // Default to max
+    }
+
     try {
-      const config = await this.platform.yotoApi.getDeviceConfig(this.device.deviceId)
-      const limit = parseInt(config.config.maxVolumeLimit || '16')
+      const limit = parseInt(this.deviceConfig.config.maxVolumeLimit || '16')
       if (isNaN(limit)) {
         return 100
       }
       // Map 0-16 to 0-100
       return Math.round((limit / 16) * 100)
     } catch (error) {
-      this.log.error(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Failed to get day volume limit:`, error)
+      this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Failed to parse day volume limit:`, error)
       return 100
     }
   }
@@ -1255,11 +1288,15 @@ export class YotoPlayerAccessory {
     this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Set day volume limit:`, value)
 
     try {
-      const config = await this.platform.yotoApi.getDeviceConfig(this.device.deviceId)
+      // Refresh config if not cached
+      if (!this.deviceConfig) {
+        this.deviceConfig = await this.platform.yotoApi.getDeviceConfig(this.device.deviceId)
+      }
+
       // Map 0-100 to 0-16
       const limit = Math.round((Number(value) / 100) * 16)
-      config.config.maxVolumeLimit = String(limit)
-      await this.platform.yotoApi.updateDeviceConfig(this.device.deviceId, config)
+      this.deviceConfig.config.maxVolumeLimit = String(limit)
+      await this.platform.yotoApi.updateDeviceConfig(this.device.deviceId, this.deviceConfig)
     } catch (error) {
       this.log.error(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Failed to set day volume limit:`, error)
       throw new this.platform.api.hap.HapStatusError(
@@ -1291,16 +1328,19 @@ export class YotoPlayerAccessory {
    * @returns {Promise<CharacteristicValue>}
    */
   async getNightVolumeLimit () {
+    if (!this.deviceConfig) {
+      return 100 // Default to max
+    }
+
     try {
-      const config = await this.platform.yotoApi.getDeviceConfig(this.device.deviceId)
-      const limit = parseInt(config.config.nightMaxVolumeLimit || '16')
+      const limit = parseInt(this.deviceConfig.config.nightMaxVolumeLimit || '16')
       if (isNaN(limit)) {
         return 100
       }
       // Map 0-16 to 0-100
       return Math.round((limit / 16) * 100)
     } catch (error) {
-      this.log.error(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Failed to get night volume limit:`, error)
+      this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Failed to parse night volume limit:`, error)
       return 100
     }
   }
@@ -1313,11 +1353,15 @@ export class YotoPlayerAccessory {
     this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Set night volume limit:`, value)
 
     try {
-      const config = await this.platform.yotoApi.getDeviceConfig(this.device.deviceId)
+      // Refresh config if not cached
+      if (!this.deviceConfig) {
+        this.deviceConfig = await this.platform.yotoApi.getDeviceConfig(this.device.deviceId)
+      }
+
       // Map 0-100 to 0-16
       const limit = Math.round((Number(value) / 100) * 16)
-      config.config.nightMaxVolumeLimit = String(limit)
-      await this.platform.yotoApi.updateDeviceConfig(this.device.deviceId, config)
+      this.deviceConfig.config.nightMaxVolumeLimit = String(limit)
+      await this.platform.yotoApi.updateDeviceConfig(this.device.deviceId, this.deviceConfig)
     } catch (error) {
       this.log.error(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Failed to set night volume limit:`, error)
       throw new this.platform.api.hap.HapStatusError(
@@ -1331,13 +1375,16 @@ export class YotoPlayerAccessory {
    * @returns {Promise<CharacteristicValue>}
    */
   async getAmbientLightOn () {
+    if (!this.deviceConfig) {
+      return false // Default to off
+    }
+
     try {
-      const config = await this.platform.yotoApi.getDeviceConfig(this.device.deviceId)
-      const color = config.config.ambientColour || '#000000'
-      // Off if color is black (#000000)
-      return color !== '#000000' && color !== 'off'
+      const color = this.deviceConfig.config.ambientColour || '000000'
+      // Consider light "on" if not pure black
+      return color !== '000000'
     } catch (error) {
-      this.log.error(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Failed to get ambient light state:`, error)
+      this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Failed to parse ambient light state:`, error)
       return false
     }
   }
@@ -1350,11 +1397,16 @@ export class YotoPlayerAccessory {
     this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Set ambient light on:`, value)
 
     try {
+      // Refresh config if not cached
+      if (!this.deviceConfig) {
+        this.deviceConfig = await this.platform.yotoApi.getDeviceConfig(this.device.deviceId)
+      }
+
       if (value) {
-        // Turn on - set to white or previous color
-        await this.mqtt.setAmbientLight(this.device.deviceId, 255, 255, 255)
+        // Restore to a default color (warm white) when turning on
+        await this.updateAmbientLightColor()
       } else {
-        // Turn off - set to black
+        // Turn off by setting to black
         await this.mqtt.setAmbientLight(this.device.deviceId, 0, 0, 0)
       }
     } catch (error) {
@@ -1370,16 +1422,19 @@ export class YotoPlayerAccessory {
    * @returns {Promise<CharacteristicValue>}
    */
   async getAmbientLightHue () {
+    if (!this.deviceConfig) {
+      return 0 // Default hue
+    }
+
     try {
-      const config = await this.platform.yotoApi.getDeviceConfig(this.device.deviceId)
-      const hex = config.config.ambientColour || '#000000'
+      const hex = this.deviceConfig.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)
+      this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Failed to parse ambient light hue:`, error)
       return 0
     }
   }
@@ -1400,16 +1455,19 @@ export class YotoPlayerAccessory {
    * @returns {Promise<CharacteristicValue>}
    */
   async getAmbientLightSaturation () {
+    if (!this.deviceConfig) {
+      return 0 // Default saturation
+    }
+
     try {
-      const config = await this.platform.yotoApi.getDeviceConfig(this.device.deviceId)
-      const hex = config.config.ambientColour || '#000000'
+      const hex = this.deviceConfig.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)
+      this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Failed to parse ambient light saturation:`, error)
       return 0
     }
   }
@@ -1430,18 +1488,21 @@ export class YotoPlayerAccessory {
    * @returns {Promise<CharacteristicValue>}
    */
   async getAmbientLightBrightness () {
+    if (!this.deviceConfig) {
+      return 0 // Default brightness
+    }
+
     try {
-      const config = await this.platform.yotoApi.getDeviceConfig(this.device.deviceId)
-      const hex = config.config.ambientColour
+      const hex = this.deviceConfig.config.ambientColour || '000000'
       const { v } = this.hexToHsv(hex)
       const brightness = Math.round(v)
       if (isNaN(brightness)) {
-        return 100
+        return 0
       }
       return brightness
     } catch (error) {
-      this.log.error(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Failed to get ambient brightness:`, error)
-      return 100
+      this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Failed to parse ambient brightness:`, error)
+      return 0
     }
   }
 
@@ -1461,9 +1522,12 @@ export class YotoPlayerAccessory {
    */
   async updateAmbientLightColor () {
     try {
-      // Get current config to read existing HSV values
-      const config = await this.platform.yotoApi.getDeviceConfig(this.device.deviceId)
-      const currentHex = config.config.ambientColour || '#ffffff'
+      // Refresh config if not cached
+      if (!this.deviceConfig) {
+        this.deviceConfig = await this.platform.yotoApi.getDeviceConfig(this.device.deviceId)
+      }
+
+      const currentHex = this.deviceConfig.config.ambientColour || '000000'
       const currentHsv = this.hexToHsv(currentHex)
 
       // Use pending values or current values
@@ -1599,7 +1663,7 @@ export class YotoPlayerAccessory {
       // Optionally update accessory display name with current content
       if (this.platform.config.updateAccessoryName) {
         this.accessory.displayName = `${this.device.name} - ${title}`
-        this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Updated display name`)
+        this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Updated display name`)
       }
     }
 

package/lib/yotoApi.js

@@ -59,7 +59,7 @@ export class YotoApi {
 
     // Check if token needs refresh
     if (this.auth.isTokenExpired(this.tokenExpiresAt)) {
-      this.log.info(LOG_PREFIX.API, ERROR_MESSAGES.TOKEN_EXPIRED)
+      this.log.debug(LOG_PREFIX.API, ERROR_MESSAGES.TOKEN_EXPIRED)
 
       if (!this.refreshToken) {
         throw new Error(ERROR_MESSAGES.TOKEN_REFRESH_FAILED)

package/lib/yotoMqtt.js

@@ -65,7 +65,7 @@ export class YotoMqtt extends EventEmitter {
     }
 
     return new Promise((resolve, reject) => {
-      this.log.info(LOG_PREFIX.MQTT, `Connecting to ${this.brokerUrl}...`)
+      this.log.debug(LOG_PREFIX.MQTT, `Connecting to ${this.brokerUrl}...`)
 
       const clientId = `DASH${deviceId}`
       const username = `${deviceId}?x-amz-customauthorizer-name=${YOTO_MQTT_AUTH_NAME}`
@@ -144,7 +144,7 @@ export class YotoMqtt extends EventEmitter {
 
       this.client.on('reconnect', () => {
         this.reconnectAttempts++
-        this.log.info(LOG_PREFIX.MQTT, `Reconnection attempt ${this.reconnectAttempts}/${this.maxReconnectAttempts}`)
+        this.log.debug(LOG_PREFIX.MQTT, `Reconnection attempt ${this.reconnectAttempts}/${this.maxReconnectAttempts}`)
 
         if (this.reconnectAttempts >= this.maxReconnectAttempts) {
           this.log.error(LOG_PREFIX.MQTT, 'Max reconnection attempts reached, stopping reconnection')
@@ -201,14 +201,14 @@ export class YotoMqtt extends EventEmitter {
     }
 
     return new Promise((resolve) => {
-      this.log.info(LOG_PREFIX.MQTT, 'Disconnecting from MQTT broker...')
+      this.log.debug(LOG_PREFIX.MQTT, 'Disconnecting from MQTT broker...')
 
       if (this.client) {
         this.client.end(false, {}, () => {
           this.connected = false
           this.client = null
           this.subscribedDevices.clear()
-          this.log.info(LOG_PREFIX.MQTT, 'Disconnected')
+          this.log.debug(LOG_PREFIX.MQTT, 'Disconnected')
           resolve()
         })
       } else {
@@ -246,7 +246,7 @@ export class YotoMqtt extends EventEmitter {
       return
     }
 
-    this.log.info(LOG_PREFIX.MQTT, `Resubscribing to ${this.subscribedDevices.size} device(s)...`)
+    this.log.debug(LOG_PREFIX.MQTT, `Resubscribing to ${this.subscribedDevices.size} device(s)...`)
 
     for (const deviceId of this.subscribedDevices) {
       const callbacks = this.deviceCallbacks.get(deviceId)
@@ -281,7 +281,7 @@ export class YotoMqtt extends EventEmitter {
       return
     }
 
-    this.log.info(LOG_PREFIX.MQTT, `Subscribing to device ${deviceId}...`)
+    this.log.debug(LOG_PREFIX.MQTT, `Subscribing to device ${deviceId}...`)
 
     const topics = [
       this.buildTopic(MQTT_TOPIC_DATA_STATUS, deviceId),
@@ -331,7 +331,7 @@ export class YotoMqtt extends EventEmitter {
       return
     }
 
-    this.log.info(LOG_PREFIX.MQTT, `Unsubscribing from device ${deviceId}...`)
+    this.log.debug(LOG_PREFIX.MQTT, `Unsubscribing from device ${deviceId}...`)
 
     const topics = [
       this.buildTopic(MQTT_TOPIC_DATA_STATUS, deviceId),

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.18",
+  "version": "0.0.20",
   "author": "Bret Comnes <bcomnes@gmail.com> (https://bret.io)",
   "bugs": {
     "url": "https://github.com/bcomnes/homebridge-yoto/issues"