homebridge-yoto 0.0.17 → 0.0.18

package/lib/playerAccessory.js

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