@homebridge-plugins/homebridge-tado 8.0.0 → 8.0.2

package/CHANGELOG.md

@@ -1,5 +1,8 @@
 # Changelog
 
+## v8.0.1 - 2025-06-12
+- Tado-API: Verify that refresh token is not empty before saving to file
+
 ## v8.0.0 - 2025-06-09
 - Add support for tado AC devices
 

package/package.json

@@ -1,10 +1,10 @@
 {
   "name": "@homebridge-plugins/homebridge-tado",
-  "version": "8.0.0",
+  "version": "8.0.2",
   "description": "Homebridge plugin for controlling tado° devices.",
   "main": "index.js",
   "scripts": {
-    "lint": "eslint --fix ."
+    "eslint": "eslint --fix ."
   },
   "type": "module",
   "repository": {
@@ -34,21 +34,21 @@
   "dependencies": {
     "@homebridge/plugin-ui-utils": "^2.1.0",
     "fakegato-history": "^0.6.7",
-    "form-data": "^4.0.3",
+    "form-data": "^4.0.4",
     "fs-extra": "^11.3.0",
     "got": "^14.4.7",
     "moment": "^2.30.1"
   },
   "devDependencies": {
-    "@babel/core": "7.27.4",
-    "@babel/eslint-parser": "7.27.5",
+    "@babel/core": "7.28.0",
+    "@babel/eslint-parser": "7.28.0",
     "@babel/eslint-plugin": "7.27.1",
-    "@eslint/js": "^9.28.0",
-    "eslint": "^9.28.0",
-    "eslint-config-prettier": "^10.1.5",
-    "eslint-plugin-import": "^2.31.0",
-    "eslint-plugin-prettier": "^5.4.1",
-    "globals": "^16.2.0",
-    "prettier": "^3.5.3"
+    "@eslint/js": "^9.31.0",
+    "eslint": "^9.31.0",
+    "eslint-config-prettier": "^10.1.8",
+    "eslint-plugin-import": "^2.32.0",
+    "eslint-plugin-prettier": "^5.5.3",
+    "globals": "^16.3.0",
+    "prettier": "^3.6.2"
   }
 }

package/src/tado/tado-api.js

@@ -93,7 +93,7 @@ export default class Tado {
         responseType: "json"
       });
       const { access_token, refresh_token } = response.body;
-      if (!access_token) throw new Error("Empty access token.");
+      if (!access_token || !refresh_token) throw new Error("Empty access/refresh token.");
       await fs.writeFile(this._tadoInternalTokenFilePath, JSON.stringify({ access_token, refresh_token }));
       this._tadoBearerToken = { access_token, refresh_token, timestamp: Date.now() };
     } catch (error) {
@@ -133,7 +133,7 @@ export default class Tado {
       }
       if (tokenResponse?.body) {
         const { access_token, refresh_token } = tokenResponse.body;
-        if (access_token) {
+        if (access_token && refresh_token) {
           await fs.writeFile(this._tadoInternalTokenFilePath, JSON.stringify({ access_token, refresh_token }));
           this._tadoBearerToken = { access_token, refresh_token, timestamp: Date.now() };
           Logger.info("Authentication successful!");
@@ -165,7 +165,7 @@ export default class Tado {
     throw new Error(`Failed to load from external file after ${maxRetries} attempts.`);
   }
 
-  async apiCall(path, method = 'GET', data = {}, params = {}, tado_url_dif, blockLog) {
+  async apiCall(path, method = 'GET', data = {}, params = {}, tado_url_dif) {
     Logger.debug('Get access token...', this.name);
     const access_token = await this.getToken();
 
@@ -380,7 +380,7 @@ export default class Tado {
 
   async setACZoneOverlay(home_id, zone_id, power, mode, temperature, fanSpeed, swing, termination, tempUnit) {
     // Note: fanSpeed parameter is kept for compatibility but ignored for AIR_CONDITIONING units
-    
+
     // Get current zone state to understand the structure
     let zone_state;
     try {
@@ -388,7 +388,7 @@ export default class Tado {
     } catch (error) {
       Logger.warn(`Could not get zone state: ${error.message}`, this.name);
     }
-    
+
     // Preserve existing termination settings if present
     const config = {
       setting: zone_state && zone_state.setting ? { ...zone_state.setting } : {},
@@ -404,8 +404,8 @@ export default class Tado {
         if (tempUnit && tempUnit.toLowerCase() === 'fahrenheit') {
           temperature = ((temperature - 32) * 5) / 9;
         }
-        
-        config.setting.temperature = { 
+
+        config.setting.temperature = {
           celsius: temperature,
           fahrenheit: Math.round((temperature * 1.8) + 32)
         };
@@ -433,7 +433,7 @@ export default class Tado {
     } else {
       config.termination.type = 'MANUAL';
     }
-    
+
     // Validate that config is not empty before making API call
     if (!config.setting || Object.keys(config.setting).length === 0) {
       Logger.error(`Config setting is empty! Power: ${power}, Mode: ${mode}, Temp: ${temperature}`, this.name);
@@ -461,7 +461,7 @@ export default class Tado {
         ...(config.termination.typeSkillBasedApp ? { typeSkillBasedApp: config.termination.typeSkillBasedApp } : {}),
       },
     };
-    
+
     return this.apiCall(`/api/v2/homes/${home_id}/zones/${zone_id}/overlay`, 'PUT', payload);
   }