homebridge-yoto 0.0.17 → 0.0.19

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

@@ -514,14 +514,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,19 +536,19 @@ 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
 
-    // Track active content changes
+    // Track active content changes with event data
     if (this.platform.config.exposeActiveContent !== false && events.cardId) {
-      this.handleActiveContentChange(events.cardId)
+      this.handleActiveContentChange(events.cardId, events)
     }
 
     // Update playback-related characteristics
@@ -560,7 +560,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 +799,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 +871,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
   }
 
@@ -1567,8 +1567,9 @@ export class YotoPlayerAccessory {
   /**
    * Handle active content change
    * @param {string} cardId - Card ID
+   * @param {YotoPlaybackEvents} [events] - Optional event data with track/chapter info
    */
-  async handleActiveContentChange (cardId) {
+  async handleActiveContentChange (cardId, events) {
     // Skip if same card
     if (this.activeContentCardId === cardId) {
       return
@@ -1584,25 +1585,38 @@ export class YotoPlayerAccessory {
       return
     }
 
-    // Check if we own this card before attempting to fetch details
+    this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Active card changed: ${cardId}`)
+
+    // Use event data if available for immediate content info
+    if (events?.trackTitle || events?.chapterTitle) {
+      const title = events.trackTitle || events.chapterTitle || cardId
+      this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Now playing: "${title}"`)
+
+      if (events.chapterKey && events.chapterKey !== events.chapterTitle) {
+        this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Chapter: ${events.chapterKey}`)
+      }
+
+      // Optionally update accessory display name with current content
+      if (this.platform.config.updateAccessoryName) {
+        this.accessory.displayName = `${this.device.name} - ${title}`
+        this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Updated display name`)
+      }
+    }
+
+    // Check if we own this card to fetch additional details
     if (!this.platform.isCardOwned(cardId)) {
-      this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Playing card ${cardId} (store content - details not available)`)
+      this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Card ${cardId} (not in library - using event data)`)
       this.activeContentInfo = null
       this.accessory.context.activeContentInfo = null
       return
     }
 
-    this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Active card changed: ${cardId}`)
-
+    // Attempt to fetch full card details for owned cards
     try {
-      // Fetch card details for owned cards
       const content = await this.platform.yotoApi.getContent(cardId)
       this.activeContentInfo = content
 
-      // Log content information
-      const title = content.card?.title || 'Unknown'
-      this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Now playing: "${title}"`)
-
+      // Log additional metadata from API
       if (content.card?.metadata?.author) {
         this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Author: ${content.card.metadata.author}`)
       }
@@ -1611,23 +1625,17 @@ export class YotoPlayerAccessory {
         this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Category: ${content.card.metadata.category}`)
       }
 
-      // Optionally update accessory display name with current content
-      if (this.platform.config.updateAccessoryName && content.card?.title) {
-        this.accessory.displayName = `${this.device.name} - ${content.card.title}`
-        this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Updated display name`)
-      }
-
       // Store in context for persistence
       this.accessory.context.activeContentInfo = this.activeContentInfo
     } catch (error) {
       // Handle 403 Forbidden (store-bought cards user doesn't own)
       if (error instanceof Error && error.message.includes('403')) {
-        this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Playing card ${cardId} (store content - details not available)`)
+        this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Card ${cardId} API access denied (using event data)`)
         this.activeContentInfo = null
         this.accessory.context.activeContentInfo = null
       } else if (error instanceof Error && error.message.includes('404')) {
         // Card not found - might be deleted or invalid
-        this.log.warn(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Card ${cardId} not found`)
+        this.log.warn(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Card ${cardId} not found in API`)
         this.activeContentInfo = null
         this.accessory.context.activeContentInfo = null
       } else {

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