homebridge-yoto 0.0.12 → 0.0.13

package/lib/playerAccessory.js

@@ -785,10 +785,10 @@ export class YotoPlayerAccessory {
    * @returns {Promise<CharacteristicValue>}
    */
   async getVolume () {
-    if (!this.currentStatus) {
+    if (!this.currentStatus || this.currentStatus.userVolume === undefined) {
       return 50
     }
-    return this.currentStatus.userVolume
+    return Number(this.currentStatus.userVolume) || 50
   }
 
   /**
@@ -813,10 +813,10 @@ export class YotoPlayerAccessory {
    * @returns {Promise<CharacteristicValue>}
    */
   async getMute () {
-    if (!this.currentStatus) {
+    if (!this.currentStatus || this.currentStatus.userVolume === undefined) {
       return false
     }
-    return this.currentStatus.userVolume === 0
+    return Number(this.currentStatus.userVolume) === 0
   }
 
   /**
@@ -849,10 +849,10 @@ export class YotoPlayerAccessory {
    * @returns {Promise<CharacteristicValue>}
    */
   async getBatteryLevel () {
-    if (!this.currentStatus) {
+    if (!this.currentStatus || this.currentStatus.batteryLevel === undefined) {
       return 100
     }
-    return this.currentStatus.batteryLevel
+    return Number(this.currentStatus.batteryLevel) || 100
   }
 
   /**
@@ -1527,6 +1527,15 @@ export class YotoPlayerAccessory {
     }
 
     this.activeContentCardId = cardId
+
+    // Handle no card inserted
+    if (!cardId || cardId === 'none' || cardId === 'null') {
+      this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] No card inserted`)
+      this.activeContentInfo = null
+      this.accessory.context.activeContentInfo = null
+      return
+    }
+
     this.log.info(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Active card changed: ${cardId}`)
 
     try {
@@ -1555,7 +1564,20 @@ export class YotoPlayerAccessory {
       // Store in context for persistence
       this.accessory.context.activeContentInfo = this.activeContentInfo
     } catch (error) {
-      this.log.error(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Failed to fetch content details:`, 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.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.activeContentInfo = null
+        this.accessory.context.activeContentInfo = null
+      } else {
+        // Other errors
+        this.log.error(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Failed to fetch content details:`, error)
+      }
     }
   }
 }

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