homebridge-yoto 0.0.40 → 0.0.41

package/config.schema.json

@@ -61,6 +61,12 @@
             ],
             "description": "Choose how playback and volume controls are exposed. External Smart Speaker requires additional pairing steps and appears as a separate accessory."
           },
+          "battery": {
+            "title": "Battery",
+            "type": "boolean",
+            "default": true,
+            "description": "Expose battery status service."
+          },
           "temperature": {
             "title": "Temperature Sensor",
             "type": "boolean",
@@ -167,6 +173,7 @@
           "helpvalue": "<p><strong>Playback Controls:</strong> External Smart Speaker requires additional pairing steps in the Home app and will appear as a separate accessory.</p>"
         },
         "services.playbackAccessory",
+        "services.battery",
         "services.temperature",
         "services.nightlight",
         "services.cardSlot",

package/lib/accessory.js

@@ -23,6 +23,7 @@
  * @typedef {Object} YotoServiceToggles
  * @property {boolean} playback
  * @property {boolean} volume
+ * @property {boolean} battery
  * @property {boolean} temperature
  * @property {boolean} nightlight
  * @property {boolean} cardSlot
@@ -135,6 +136,7 @@ export class YotoPlayerAccessory {
     return {
       playback: playbackConfig.playbackEnabled,
       volume: playbackConfig.volumeEnabled,
+      battery: getBooleanSetting(serviceConfig['battery'], getServiceDefault('battery')),
       temperature: getBooleanSetting(serviceConfig['temperature'], getServiceDefault('temperature')),
       nightlight: getBooleanSetting(serviceConfig['nightlight'], getServiceDefault('nightlight')),
       cardSlot: getBooleanSetting(serviceConfig['cardSlot'], getServiceDefault('cardSlot')),
@@ -177,7 +179,9 @@ export class YotoPlayerAccessory {
       this.setupVolumeService()
     }
 
-    this.setupBatteryService()
+    if (serviceToggles.battery) {
+      this.setupBatteryService()
+    }
 
     // Setup optional services based on device capabilities
     if (serviceToggles.temperature && this.#deviceModel.capabilities.hasTemperatureSensor) {
@@ -917,7 +921,8 @@ export class YotoPlayerAccessory {
     })
 
     this.#deviceModel.on('error', (error) => {
-      this.#log.error(`[${this.#device.name}] Device error:`, error.message)
+      const details = error instanceof Error ? (error.stack || error.message) : String(error)
+      this.#log.error(`[${this.#device.name}] Device error:`, details)
     })
   }
 
@@ -1186,19 +1191,26 @@ export class YotoPlayerAccessory {
    * @param {CharacteristicValue} value
    */
   async setDayNightlightOn (value) {
-    if (value) {
-      // Turn ON - restore previous color or default to white
-      const colorToSet = this.#lastDayColor || '0xffffff'
-      this.#log.debug(LOG_PREFIX.ACCESSORY, `Turning day nightlight ON with color: ${colorToSet}`)
-      await this.#deviceModel.updateConfig({ ambientColour: colorToSet })
-    } else {
-      // Turn OFF - save current color and set to black
-      const currentColor = this.#deviceModel.config.ambientColour
-      if (!this.isColorOff(currentColor)) {
-        this.#lastDayColor = currentColor
+    try {
+      if (value) {
+        // Turn ON - restore previous color or default to white
+        const colorToSet = this.#lastDayColor || '0xffffff'
+        this.#log.debug(LOG_PREFIX.ACCESSORY, `Turning day nightlight ON with color: ${colorToSet}`)
+        await this.#deviceModel.updateConfig({ ambientColour: colorToSet })
+      } else {
+        // Turn OFF - save current color and set to black
+        const currentColor = this.#deviceModel.config.ambientColour
+        if (!this.isColorOff(currentColor)) {
+          this.#lastDayColor = currentColor
+        }
+        this.#log.debug(LOG_PREFIX.ACCESSORY, 'Turning day nightlight OFF')
+        await this.#deviceModel.updateConfig({ ambientColour: '0x000000' })
       }
-      this.#log.debug(LOG_PREFIX.ACCESSORY, 'Turning day nightlight OFF')
-      await this.#deviceModel.updateConfig({ ambientColour: '0x000000' })
+    } catch (error) {
+      this.#log.error(LOG_PREFIX.ACCESSORY, `[${this.#device.name}] Failed to set day nightlight:`, error)
+      throw new this.#platform.api.hap.HapStatusError(
+        this.#platform.api.hap.HAPStatus.SERVICE_COMMUNICATION_FAILURE
+      )
     }
   }
 
@@ -1228,10 +1240,17 @@ export class YotoPlayerAccessory {
 
     const brightnessValue = Math.max(0, Math.min(Math.round(rawBrightness), 100))
     this.#log.debug(LOG_PREFIX.ACCESSORY, `Setting day display brightness: ${brightnessValue}`)
-    await this.#deviceModel.updateConfig({
-      dayDisplayBrightness: brightnessValue,
-      dayDisplayBrightnessAuto: false
-    })
+    try {
+      await this.#deviceModel.updateConfig({
+        dayDisplayBrightness: brightnessValue,
+        dayDisplayBrightnessAuto: false
+      })
+    } catch (error) {
+      this.#log.error(LOG_PREFIX.ACCESSORY, `[${this.#device.name}] Failed to set day brightness:`, error)
+      throw new this.#platform.api.hap.HapStatusError(
+        this.#platform.api.hap.HAPStatus.SERVICE_COMMUNICATION_FAILURE
+      )
+    }
   }
 
   /**
@@ -1275,7 +1294,14 @@ export class YotoPlayerAccessory {
     const formattedColor = this.formatHexColor(newHex)
 
     this.#log.debug(LOG_PREFIX.ACCESSORY, `Setting day nightlight hue: ${hue}° → ${formattedColor}`)
-    await this.#deviceModel.updateConfig({ ambientColour: formattedColor })
+    try {
+      await this.#deviceModel.updateConfig({ ambientColour: formattedColor })
+    } catch (error) {
+      this.#log.error(LOG_PREFIX.ACCESSORY, `[${this.#device.name}] Failed to set day nightlight hue:`, error)
+      throw new this.#platform.api.hap.HapStatusError(
+        this.#platform.api.hap.HAPStatus.SERVICE_COMMUNICATION_FAILURE
+      )
+    }
   }
 
   /**
@@ -1319,7 +1345,14 @@ export class YotoPlayerAccessory {
     const formattedColor = this.formatHexColor(newHex)
 
     this.#log.debug(LOG_PREFIX.ACCESSORY, `Setting day nightlight saturation: ${saturation}% → ${formattedColor}`)
-    await this.#deviceModel.updateConfig({ ambientColour: formattedColor })
+    try {
+      await this.#deviceModel.updateConfig({ ambientColour: formattedColor })
+    } catch (error) {
+      this.#log.error(LOG_PREFIX.ACCESSORY, `[${this.#device.name}] Failed to set day nightlight saturation:`, error)
+      throw new this.#platform.api.hap.HapStatusError(
+        this.#platform.api.hap.HAPStatus.SERVICE_COMMUNICATION_FAILURE
+      )
+    }
   }
 
   // ---------- Night Nightlight Handlers ----------
@@ -1338,19 +1371,26 @@ export class YotoPlayerAccessory {
    * @param {CharacteristicValue} value
    */
   async setNightNightlightOn (value) {
-    if (value) {
-      // Turn ON - restore previous color or default to white
-      const colorToSet = this.#lastNightColor || '0xffffff'
-      this.#log.debug(LOG_PREFIX.ACCESSORY, `Turning night nightlight ON with color: ${colorToSet}`)
-      await this.#deviceModel.updateConfig({ nightAmbientColour: colorToSet })
-    } else {
-      // Turn OFF - save current color and set to black
-      const currentColor = this.#deviceModel.config.nightAmbientColour
-      if (!this.isColorOff(currentColor)) {
-        this.#lastNightColor = currentColor
+    try {
+      if (value) {
+        // Turn ON - restore previous color or default to white
+        const colorToSet = this.#lastNightColor || '0xffffff'
+        this.#log.debug(LOG_PREFIX.ACCESSORY, `Turning night nightlight ON with color: ${colorToSet}`)
+        await this.#deviceModel.updateConfig({ nightAmbientColour: colorToSet })
+      } else {
+        // Turn OFF - save current color and set to black
+        const currentColor = this.#deviceModel.config.nightAmbientColour
+        if (!this.isColorOff(currentColor)) {
+          this.#lastNightColor = currentColor
+        }
+        this.#log.debug(LOG_PREFIX.ACCESSORY, 'Turning night nightlight OFF')
+        await this.#deviceModel.updateConfig({ nightAmbientColour: '0x000000' })
       }
-      this.#log.debug(LOG_PREFIX.ACCESSORY, 'Turning night nightlight OFF')
-      await this.#deviceModel.updateConfig({ nightAmbientColour: '0x000000' })
+    } catch (error) {
+      this.#log.error(LOG_PREFIX.ACCESSORY, `[${this.#device.name}] Failed to set night nightlight:`, error)
+      throw new this.#platform.api.hap.HapStatusError(
+        this.#platform.api.hap.HAPStatus.SERVICE_COMMUNICATION_FAILURE
+      )
     }
   }
 
@@ -1380,10 +1420,17 @@ export class YotoPlayerAccessory {
 
     const brightnessValue = Math.max(0, Math.min(Math.round(rawBrightness), 100))
     this.#log.debug(LOG_PREFIX.ACCESSORY, `Setting night display brightness: ${brightnessValue}`)
-    await this.#deviceModel.updateConfig({
-      nightDisplayBrightness: brightnessValue,
-      nightDisplayBrightnessAuto: false
-    })
+    try {
+      await this.#deviceModel.updateConfig({
+        nightDisplayBrightness: brightnessValue,
+        nightDisplayBrightnessAuto: false
+      })
+    } catch (error) {
+      this.#log.error(LOG_PREFIX.ACCESSORY, `[${this.#device.name}] Failed to set night brightness:`, error)
+      throw new this.#platform.api.hap.HapStatusError(
+        this.#platform.api.hap.HAPStatus.SERVICE_COMMUNICATION_FAILURE
+      )
+    }
   }
 
   /**
@@ -1427,7 +1474,14 @@ export class YotoPlayerAccessory {
     const formattedColor = this.formatHexColor(newHex)
 
     this.#log.debug(LOG_PREFIX.ACCESSORY, `Setting night nightlight hue: ${hue}° → ${formattedColor}`)
-    await this.#deviceModel.updateConfig({ nightAmbientColour: formattedColor })
+    try {
+      await this.#deviceModel.updateConfig({ nightAmbientColour: formattedColor })
+    } catch (error) {
+      this.#log.error(LOG_PREFIX.ACCESSORY, `[${this.#device.name}] Failed to set night nightlight hue:`, error)
+      throw new this.#platform.api.hap.HapStatusError(
+        this.#platform.api.hap.HAPStatus.SERVICE_COMMUNICATION_FAILURE
+      )
+    }
   }
 
   /**
@@ -1471,7 +1525,14 @@ export class YotoPlayerAccessory {
     const formattedColor = this.formatHexColor(newHex)
 
     this.#log.debug(LOG_PREFIX.ACCESSORY, `Setting night nightlight saturation: ${saturation}% → ${formattedColor}`)
-    await this.#deviceModel.updateConfig({ nightAmbientColour: formattedColor })
+    try {
+      await this.#deviceModel.updateConfig({ nightAmbientColour: formattedColor })
+    } catch (error) {
+      this.#log.error(LOG_PREFIX.ACCESSORY, `[${this.#device.name}] Failed to set night nightlight saturation:`, error)
+      throw new this.#platform.api.hap.HapStatusError(
+        this.#platform.api.hap.HAPStatus.SERVICE_COMMUNICATION_FAILURE
+      )
+    }
   }
 
   // ==================== Nightlight Status ContactSensor Getters ====================
@@ -1594,7 +1655,14 @@ export class YotoPlayerAccessory {
   async setBluetoothState (value) {
     const enabled = Boolean(value)
     this.#log.debug(LOG_PREFIX.ACCESSORY, `Setting Bluetooth: ${enabled ? 'ON' : 'OFF'}`)
-    await this.#deviceModel.updateConfig({ bluetoothEnabled: enabled })
+    try {
+      await this.#deviceModel.updateConfig({ bluetoothEnabled: enabled })
+    } catch (error) {
+      this.#log.error(LOG_PREFIX.ACCESSORY, `[${this.#device.name}] Failed to set Bluetooth:`, error)
+      throw new this.#platform.api.hap.HapStatusError(
+        this.#platform.api.hap.HAPStatus.SERVICE_COMMUNICATION_FAILURE
+      )
+    }
   }
 
   // ==================== Card Control Switch Setter ====================
@@ -1683,7 +1751,14 @@ export class YotoPlayerAccessory {
       LOG_PREFIX.ACCESSORY,
       `[${this.#device.name}] Set day max volume limit raw=${value} normalizedPercent=${normalizedPercent} requestedSteps=${requestedSteps} -> steps=${limit} percent=${limitPercent}`
     )
-    await this.#deviceModel.updateConfig({ maxVolumeLimit: limit })
+    try {
+      await this.#deviceModel.updateConfig({ maxVolumeLimit: limit })
+    } catch (error) {
+      this.#log.error(LOG_PREFIX.ACCESSORY, `[${this.#device.name}] Failed to set day max volume limit:`, error)
+      throw new this.#platform.api.hap.HapStatusError(
+        this.#platform.api.hap.HAPStatus.SERVICE_COMMUNICATION_FAILURE
+      )
+    }
   }
 
   /**
@@ -1722,7 +1797,14 @@ export class YotoPlayerAccessory {
       LOG_PREFIX.ACCESSORY,
       `[${this.#device.name}] Set night max volume limit raw=${value} normalizedPercent=${normalizedPercent} requestedSteps=${requestedSteps} -> steps=${limit} percent=${limitPercent}`
     )
-    await this.#deviceModel.updateConfig({ nightMaxVolumeLimit: limit })
+    try {
+      await this.#deviceModel.updateConfig({ nightMaxVolumeLimit: limit })
+    } catch (error) {
+      this.#log.error(LOG_PREFIX.ACCESSORY, `[${this.#device.name}] Failed to set night max volume limit:`, error)
+      throw new this.#platform.api.hap.HapStatusError(
+        this.#platform.api.hap.HAPStatus.SERVICE_COMMUNICATION_FAILURE
+      )
+    }
   }
 
   // ==================== Characteristic Update Methods ====================
@@ -1746,6 +1828,8 @@ export class YotoPlayerAccessory {
    * @param {number} volumeSteps - Volume level (0-16)
    */
   updateVolumeCharacteristic (volumeSteps) {
+    if (!this.volumeService) return
+
     if (volumeSteps > 0) {
       this.#lastNonZeroVolume = Math.round((volumeSteps / 16) * 100)
     }
@@ -1757,7 +1841,6 @@ export class YotoPlayerAccessory {
       LOG_PREFIX.ACCESSORY,
       `[${this.#device.name}] Update volume characteristic rawSteps=${volumeSteps} percent=${percent}`
     )
-    if (!this.volumeService) return
 
     const { Characteristic } = this.#platform
     this.volumeService

package/lib/platform.js

@@ -127,13 +127,20 @@ export class YotoPlatform {
       }
     })
 
+    const formatError = (/** @type {unknown} */ error) => (
+      error instanceof Error ? (error.stack || error.message) : String(error)
+    )
+
     // Listen to account-level events
     this.yotoAccount.on('error', ({ error, context }) => {
+      const details = formatError(error)
       if (context.deviceId) {
         const label = this.formatDeviceLabel(context.deviceId)
-        log.error(`Device error [${label} ${context.operation} ${context.source}]:`, error.message)
+        log.error(`Device error [${label} ${context.operation} ${context.source}]:`, details)
+        log.debug('Device error context:', context)
       } else {
-        log.error('Account error:', error.message)
+        log.error('Account error:', details)
+        log.debug('Account error context:', context)
       }
     })
 

package/lib/speaker-accessory.js

@@ -247,7 +247,8 @@ export class YotoSpeakerAccessory {
     })
 
     this.#deviceModel.on('error', (error) => {
-      this.#log.error(`[${this.#device.name}] Speaker device error:`, error.message)
+      const details = error instanceof Error ? (error.stack || error.message) : String(error)
+      this.#log.error(`[${this.#device.name}] Speaker device error:`, details)
     })
   }
 

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