homebridge-yoto 0.0.20 → 0.0.21

package/lib/playerAccessory.js

@@ -467,7 +467,7 @@ export class YotoPlayerAccessory {
     try {
       // Check if device is online first
       if (!this.device.online) {
-        this.log.warn(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Device is offline, skipping MQTT connection`)
+        this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Device is offline, skipping MQTT connection`)
         return
       }
 
@@ -477,12 +477,7 @@ export class YotoPlayerAccessory {
         return
       }
 
-      // TEMPORARY: Debug logging for MQTT troubleshooting
-      this.log.warn(LOG_PREFIX.ACCESSORY, `[${this.device.name}] MQTT Connection Details:`)
-      this.log.warn(LOG_PREFIX.ACCESSORY, `  Device Online: ${this.device.online}`)
-      this.log.warn(LOG_PREFIX.ACCESSORY, `  Device ID: ${this.device.deviceId}`)
-      this.log.warn(LOG_PREFIX.ACCESSORY, `  Access Token: ${this.platform.config.accessToken}`)
-      this.log.warn(LOG_PREFIX.ACCESSORY, `  Token Length: ${this.platform.config.accessToken.length}`)
+      this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] Connecting to MQTT (device online: ${this.device.online})`)
 
       // Connect MQTT with device ID and access token
       await this.mqtt.connect(
@@ -503,12 +498,12 @@ export class YotoPlayerAccessory {
       // Set up MQTT event listeners
       this.mqtt.on('disconnected', () => {
         this.mqttConnected = false
-        this.log.warn(LOG_PREFIX.ACCESSORY, `[${this.device.name}] MQTT disconnected`)
+        this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] MQTT disconnected`)
       })
 
       this.mqtt.on('offline', () => {
         this.mqttConnected = false
-        this.log.warn(LOG_PREFIX.ACCESSORY, `[${this.device.name}] MQTT offline`)
+        this.log.debug(LOG_PREFIX.ACCESSORY, `[${this.device.name}] MQTT offline`)
       })
 
       this.mqtt.on('connected', () => {
@@ -616,12 +611,18 @@ export class YotoPlayerAccessory {
 
     // Update temperature
     if (this.temperatureService && this.currentStatus.temp) {
-      const temp = parseFloat(this.currentStatus.temp)
-      if (!isNaN(temp)) {
-        this.temperatureService.updateCharacteristic(
-          this.platform.Characteristic.CurrentTemperature,
-          temp
-        )
+      // Parse temp string format "x:temp:y" to get middle value
+      const tempParts = this.currentStatus.temp.split(':')
+      if (tempParts.length >= 2) {
+        const temp = Number(tempParts[1])
+        if (!isNaN(temp)) {
+          // Clamp to HomeKit's valid range: 0-100°C
+          const clampedTemp = Math.max(0, Math.min(100, temp))
+          this.temperatureService.updateCharacteristic(
+            this.platform.Characteristic.CurrentTemperature,
+            clampedTemp
+          )
+        }
       }
 
       // Update display brightness
@@ -929,11 +930,22 @@ export class YotoPlayerAccessory {
    */
   async getCurrentTemperature () {
     if (!this.currentStatus || !this.currentStatus.temp) {
+      return 20 // Default room temperature
+    }
+
+    // Parse temp string format "x:temp:y" to get middle value (actual temperature)
+    const tempParts = this.currentStatus.temp.split(':')
+    if (tempParts.length < 2) {
+      return 20
+    }
+
+    const temp = Number(tempParts[1])
+    if (isNaN(temp)) {
       return 20
     }
 
-    const temp = parseFloat(this.currentStatus.temp)
-    return isNaN(temp) ? 20 : temp
+    // Clamp to HomeKit's valid range: 0-100°C
+    return Math.max(0, Math.min(100, temp))
   }
 
   /**

package/lib/yotoMqtt.js

@@ -60,7 +60,7 @@ export class YotoMqtt extends EventEmitter {
    */
   async connect (accessToken, deviceId) {
     if (this.client) {
-      this.log.warn(LOG_PREFIX.MQTT, 'Already connected, disconnecting first...')
+      this.log.debug(LOG_PREFIX.MQTT, 'Already connected, disconnecting first...')
       await this.disconnect()
     }
 
@@ -70,21 +70,7 @@ export class YotoMqtt extends EventEmitter {
       const clientId = `DASH${deviceId}`
       const username = `${deviceId}?x-amz-customauthorizer-name=${YOTO_MQTT_AUTH_NAME}`
 
-      // TEMPORARY: Detailed debug logging for MQTT troubleshooting
-      this.log.warn(LOG_PREFIX.MQTT, '='.repeat(60))
-      this.log.warn(LOG_PREFIX.MQTT, 'MQTT CONNECTION DETAILS')
-      this.log.warn(LOG_PREFIX.MQTT, '='.repeat(60))
-      this.log.warn(LOG_PREFIX.MQTT, `Broker URL: ${this.brokerUrl}`)
-      this.log.warn(LOG_PREFIX.MQTT, `Client ID: ${clientId}`)
-      this.log.warn(LOG_PREFIX.MQTT, `Username: ${username}`)
-      this.log.warn(LOG_PREFIX.MQTT, `Device ID: ${deviceId}`)
-      this.log.warn(LOG_PREFIX.MQTT, `Token: ${accessToken}`)
-      this.log.warn(LOG_PREFIX.MQTT, `Token length: ${accessToken.length}`)
-      this.log.warn(LOG_PREFIX.MQTT, 'Port: 443')
-      this.log.warn(LOG_PREFIX.MQTT, 'Protocol: wss')
-      this.log.warn(LOG_PREFIX.MQTT, 'Keepalive: 300')
-      this.log.warn(LOG_PREFIX.MQTT, `Connect Timeout: ${MQTT_CONNECT_TIMEOUT}ms`)
-      this.log.warn(LOG_PREFIX.MQTT, '='.repeat(60))
+      this.log.debug(LOG_PREFIX.MQTT, `Connecting with client ID: ${clientId}`)
 
       this.client = mqtt.connect(this.brokerUrl, {
         keepalive: 300,
@@ -135,7 +121,7 @@ export class YotoMqtt extends EventEmitter {
         this.emit('disconnected')
 
         if (wasConnected) {
-          this.log.warn(LOG_PREFIX.MQTT, ERROR_MESSAGES.MQTT_DISCONNECTED)
+          this.log.info(LOG_PREFIX.MQTT, ERROR_MESSAGES.MQTT_DISCONNECTED)
           this.handleReconnect()
         } else {
           this.log.error(LOG_PREFIX.MQTT, 'Connection closed before establishing connection')
@@ -156,18 +142,18 @@ export class YotoMqtt extends EventEmitter {
 
       this.client.on('offline', () => {
         this.connected = false
-        this.log.warn(LOG_PREFIX.MQTT, 'MQTT client offline - connection failed or lost')
+        this.log.debug(LOG_PREFIX.MQTT, 'MQTT client offline - connection failed or lost')
 
         // Emit offline event
         this.emit('offline')
       })
 
       this.client.on('end', () => {
-        this.log.warn(LOG_PREFIX.MQTT, 'MQTT client ended')
+        this.log.debug(LOG_PREFIX.MQTT, 'MQTT client ended')
       })
 
       this.client.on('disconnect', (packet) => {
-        this.log.warn(LOG_PREFIX.MQTT, 'MQTT client disconnected:', packet)
+        this.log.debug(LOG_PREFIX.MQTT, 'MQTT client disconnected:', packet)
       })
 
       this.client.on('packetreceive', (packet) => {
@@ -309,10 +295,10 @@ export class YotoMqtt extends EventEmitter {
         // Request initial status after a short delay
         setTimeout(() => {
           this.requestStatus(deviceId).catch(err => {
-            this.log.warn(LOG_PREFIX.MQTT, `Failed to request initial status for ${deviceId}:`, err)
+            this.log.debug(LOG_PREFIX.MQTT, `Failed to request initial status for ${deviceId}:`, err)
           })
           this.requestEvents(deviceId).catch(err => {
-            this.log.warn(LOG_PREFIX.MQTT, `Failed to request initial events for ${deviceId}:`, err)
+            this.log.debug(LOG_PREFIX.MQTT, `Failed to request initial events for ${deviceId}:`, err)
           })
         }, INITIAL_STATUS_REQUEST_DELAY)
 
@@ -371,7 +357,7 @@ export class YotoMqtt extends EventEmitter {
       const deviceId = this.extractDeviceId(topic)
 
       if (!deviceId) {
-        this.log.warn(LOG_PREFIX.MQTT, `Could not extract device ID from topic: ${topic}`)
+        this.log.debug(LOG_PREFIX.MQTT, `Could not extract device ID from topic: ${topic}`)
         return
       }
 

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