Address linter errors

src/helki_client.ts

@@ -1,22 +1,19 @@
 import axios, { AxiosInstance } from 'axios';
 import axiosRetry from 'axios-retry';
+import { Logger } from 'homebridge';
 
 interface TokenResponse {
   access_token: string;
   refresh_token: string;
   expires_in: number;
 }
 
-interface NodeResponse {
-  nodes: Node[];
-}
-
 interface StatusArgs {
-  [key: string]: any;
+  [key: string]: string | number;
 }
 
 interface SetupArgs {
-  [key: string]: any;
+  [key: string]: string | number;
 }
 
 interface PowerLimitResponse {
@@ -105,21 +102,23 @@ class HelkiClient {
   private axiosInstance: AxiosInstance;
   private accessToken: string;
   private expiresAt: Date;
+  private log: Logger;
 
   constructor(
     apiName: string,
     clientId: string,
     clientSecret: string,
     username: string,
     password: string,
-    retryAttempts = 5,
+    log: Logger,
   ) {
     this.apiHost = `https://${apiName}.helki.com`;
     this.clientId = clientId;
     this.clientSecret = clientSecret;
     this.username = username;
     this.password = password;
 
+    this.log = log;
     this.accessToken = '';
     this.expiresAt = new Date();
 
@@ -134,7 +133,7 @@ class HelkiClient {
     }, error => {
       if (error.response) {
         // Log any request error
-        console.log('Request error:', error);
+        this.log.error('Request error:', error);
       }
       return Promise.reject(error);
     });
@@ -146,7 +145,7 @@ class HelkiClient {
     }, error => {
       if (error.response) {
         // Log error details including the response from the server
-        console.log(`Error response from ${error.response.config.url}:`, error.response);
+        this.log.error(`Error response from ${error.response.config.url}:`, error.response);
       }
       return Promise.reject(error);
     });
@@ -181,7 +180,7 @@ class HelkiClient {
     this.expiresAt = new Date(Date.now() + expires_in * 1000);
 
     if (expires_in < MIN_TOKEN_LIFETIME) {
-      console.warn(`Token expires in ${expires_in}s, which is below the minimum lifetime of ${MIN_TOKEN_LIFETIME}s.`);
+      this.log.warn(`Token expires in ${expires_in}s, which is below the minimum lifetime of ${MIN_TOKEN_LIFETIME}s.`);
     }
   }
 
@@ -202,7 +201,8 @@ class HelkiClient {
     };
   }
 
-  private async apiRequest<T = any>(path: string, method: 'GET' | 'POST' = 'GET', data?: any): Promise<T> {
+  // eslint-disable-next-line
+  private async apiRequest<T>(path: string, method: 'GET' | 'POST' = 'GET', data?: any): Promise<T> {
     await this.checkRefresh();
     const url = `${this.apiHost}/api/v2/${path}`;
     const headers = this.getHeaders();
@@ -212,8 +212,11 @@ class HelkiClient {
         ? this.axiosInstance.get<T>(url, { headers })
         : this.axiosInstance.post<T>(url, data, { headers }));
       return response.data;
-    } catch (error: any) {
-      throw new Error(`API request to ${path} failed: ${error.message}`);
+    } catch (error: unknown) {
+      if (error instanceof Error) {
+        throw new Error(`API request to ${path} failed: ${error.message}`);
+      }
+      throw new Error(`API request to ${path} failed: ${error}`);
     }
   }
 
@@ -239,22 +242,23 @@ class HelkiClient {
     await this.apiRequest<void>(`devs/${deviceId}/${node.type}/${node.addr}/status`, 'POST', status);
   }
 
-  public async getSetup(deviceId: string, node: Node): Promise<any> {
+  public async getSetup(deviceId: string, node: Node): Promise<unknown> {
     return this.apiRequest(`devs/${deviceId}/${node.type}/${node.addr}/setup`);
   }
 
-  public async setSetup(deviceId: string, node: Node, setupArgs: SetupArgs): Promise<any> {
+  public async setSetup(deviceId: string, node: Node, setupArgs: SetupArgs): Promise<unknown> {
     let setupData = await this.getSetup(deviceId, node); // Assuming this returns the current setup in a directly usable format
     setupData = { ...setupData, ...setupArgs }; // Merge with new setup arguments
     return this.apiRequest(`devs/${deviceId}/${node.type}/${node.addr}/setup`, 'POST', setupData);
   }
 
-  public async getDeviceAwayStatus(deviceId: string): Promise<any> {
+  public async getDeviceAwayStatus(deviceId: string): Promise<unknown> {
     return this.apiRequest(`devs/${deviceId}/mgr/away_status`);
   }
 
-  public async setDeviceAwayStatus(deviceId: string, statusArgs: StatusArgs): Promise<any> {
-    const data = Object.fromEntries(Object.entries(statusArgs).filter(([_, v]) => v != null));
+  public async setDeviceAwayStatus(deviceId: string, statusArgs: StatusArgs): Promise<unknown> {
+    // eslint-disable-next-line @typescript-eslint/no-unused-vars
+    const data = Object.fromEntries(Object.entries(statusArgs).filter(([_, v]) => v !== null));
     return this.apiRequest(`devs/${deviceId}/mgr/away_status`, 'POST', data);
   }
 

src/platform.ts

@@ -55,7 +55,8 @@ export class Technotherm implements DynamicPlatformPlugin {
         this.config.clientId,
         this.config.clientSecret,
         this.config.username,
-        this.config.password);
+        this.config.password,
+        this.log);
 
       const homes = await helki.getGroupedDevices();
       for (const home of homes) {
@@ -73,7 +74,7 @@ export class Technotherm implements DynamicPlatformPlugin {
             new Radiator(this, existingAccessory, helki);
           } else {
             // Accessory doesn't exist, add new
-            const accessoryName = `${device.name} (${home.name})`
+            const accessoryName = `${device.name} (${home.name})`;
             this.log.info('Adding new accessory:', accessoryName); // Use device name for display name
             const accessory = new this.api.platformAccessory(accessoryName, uuid);
             accessory.context.device = device;
@@ -85,8 +86,12 @@ export class Technotherm implements DynamicPlatformPlugin {
         }
       }
 
-    } catch (error: any) {
-      this.log.error('Failed to discover devices:', error.message);
+    } catch (error: unknown) {
+      if (error instanceof Error) {
+        this.log.error(`Failed to discover devices: ${error.message}`);
+      } else {
+        this.log.error(`Failed to discover devices: ${error}`);
+      }
     }
   }
 

src/radiator.ts

@@ -1,6 +1,6 @@
 import { Service, PlatformAccessory, CharacteristicValue } from 'homebridge';
 import { Technotherm } from './platform';
-import { HelkiClient, Node, Status } from './helki_client';
+import { HelkiClient, Node } from './helki_client';
 
 export class Radiator {
   private service: Service;
@@ -9,7 +9,7 @@ export class Radiator {
   constructor(
     private readonly platform: Technotherm,
     private readonly accessory: PlatformAccessory,
-    private readonly helkiClient: HelkiClient
+    private readonly helkiClient: HelkiClient,
   ) {
     // Initialize node from accessory context
     this.node = this.accessory.context.node;
@@ -50,7 +50,7 @@ export class Radiator {
         {
           stemp: value.toString(),
           mode: 'manual',
-          units: 'C'
+          units: 'C',
         });
     } catch (error) {
       this.platform.log.error('Failed to set target temperature:', error);
@@ -84,7 +84,7 @@ export class Radiator {
     } else if (value === this.platform.Characteristic.TargetHeatingCoolingState.AUTO) {
       mode = 'auto'; // Assuming auto control
     } else {
-      mode = 'off'
+      mode = 'off';
     }
     // Set the mode accordingly
     try {