chore: fix linter errors after new eslint config

Author: jamaljsr

lib/api/createRpc.ts

@@ -1,5 +1,5 @@
-import LNC from '../lnc';
 import { subscriptionMethods } from '@lightninglabs/lnc-core';
+import LNC from '../lnc';
 
 // capitalize the first letter in the string
 const capitalize = (s: string) => s && s[0].toUpperCase() + s.slice(1);
@@ -11,7 +11,7 @@ const capitalize = (s: string) => s && s[0].toUpperCase() + s.slice(1);
 export function createRpc<T extends object>(packageName: string, lnc: LNC): T {
   const rpc = {};
   return new Proxy(rpc, {
-    get(target, key, c) {
+    get(target, key) {
       const methodName = capitalize(key.toString());
       // the full name of the method (ex: lnrpc.Lightning.OpenChannel)
       const method = `${packageName}.${methodName}`;
@@ -27,7 +27,7 @@ export function createRpc<T extends object>(packageName: string, lnc: LNC): T {
         };
       } else {
         // call request for unary methods
-        return async (request: object): Promise<any> => {
+        return async (request: object): Promise<unknown> => {
           return await lnc.request(method, request);
         };
       }

lib/index.ts

@@ -1,4 +1,4 @@
-// tslint:disable-next-line
+// eslint-disable-next-line @typescript-eslint/no-require-imports
 require('./wasm_exec');
 
 import LNC from './lnc';

lib/lnc.ts

@@ -37,7 +37,7 @@ export default class LNC {
     instance: WebAssembly.Instance;
   };
 
-  _wasmClientCode: any;
+  _wasmClientCode: string;
   _namespace: string;
   credentials: CredentialStore;
 
@@ -84,7 +84,7 @@ export default class LNC {
     return lncGlobal[this._namespace] as WasmGlobal;
   }
 
-  private set wasm(value: any) {
+  private set wasm(value: WasmGlobal) {
     lncGlobal[this._namespace] = value;
   }
 
@@ -153,7 +153,7 @@ export default class LNC {
     // create the namespace object in the global scope if it doesn't exist
     // so that we can assign the WASM callbacks to it
     if (typeof this.wasm !== 'object') {
-      this.wasm = {};
+      this.wasm = {} as WasmGlobal;
     }
 
     // assign the WASM callbacks to the namespace object if they haven't
@@ -293,7 +293,7 @@ export default class LNC {
           const res = snakeKeysToCamel(rawRes);
           log.debug(`${method} response`, res);
           resolve(res as TRes);
-        } catch (error) {
+        } catch {
           log.debug(`${method} raw response`, response);
           reject(new Error(response));
           return;

lib/types/lnc.ts

@@ -28,8 +28,8 @@ export interface WasmGlobal {
    */
   wasmClientInvokeRPC: (
     rpcName: string,
-    request: any,
-    callback: (response: string) => any
+    request: string,
+    callback: (response: string) => void
   ) => void;
   /**
    * Returns true if client has specific permissions
@@ -79,7 +79,7 @@ export interface LncConfig {
    * Custom location for the WASM client code. Can be remote or local. If not
    * specified we’ll default to our instance on our CDN.
    */
-  wasmClientCode?: any; // URL or WASM client object
+  wasmClientCode?: string; // URL to WASM file
   /**
    * JavaScript namespace used for the main WASM calls. You can maintain multiple
    * connections if you use different namespaces. If not specified we'll default

lib/typings.d.ts

@@ -1,4 +1,4 @@
-declare var global: any;
+declare const global: unknown;
 
 interface GoInstance {
   run(instance: WebAssembly.Instance): Promise<void>;

lib/util/credentialStore.test.ts

@@ -131,7 +131,7 @@ describe('LncCredentialStore', () => {
     });
 
     it('should call _load method during construction', () => {
-      const store = new LncCredentialStore();
+      new LncCredentialStore();
 
       // Verify localStorage.getItem was called (from _load method)
       expect(mockSetup.localStorage.getItem).toHaveBeenCalledWith(

lib/util/encryption.test.ts

@@ -55,10 +55,9 @@ describe('Encryption Utilities', () => {
       const password = testData.password;
       const salt = 'testsalt12345678901234567890123456789012';
 
-      const result = encrypt(data, password, salt);
+      const result = encrypt(JSON.stringify(data), password, salt);
 
       expect(typeof result).toBe('string');
-      expect(result).toHaveLength(44); // AES encrypted strings are typically 44 chars
     });
 
     it('should produce different results for different data', () => {
@@ -99,10 +98,11 @@ describe('Encryption Utilities', () => {
       const password = testData.password;
       const salt = generateSalt();
 
-      const encrypted = encrypt(originalData, password, salt);
+      const actual = JSON.stringify(originalData);
+      const encrypted = encrypt(actual, password, salt);
       const decrypted = decrypt(encrypted, password, salt);
 
-      expect(decrypted).toEqual(originalData);
+      expect(decrypted).toEqual(actual);
     });
 
     it('should throw error for invalid encrypted data', () => {
@@ -214,10 +214,11 @@ describe('Encryption Utilities', () => {
       const password = testData.password;
       const salt = generateSalt();
 
-      const encrypted = encrypt(complexData, password, salt);
+      const actual = JSON.stringify(complexData);
+      const encrypted = encrypt(actual, password, salt);
       const decrypted = decrypt(encrypted, password, salt);
 
-      expect(decrypted).toEqual(complexData);
+      expect(decrypted).toEqual(actual);
     });
 
     it('should maintain data integrity through encrypt/decrypt cycle', () => {
@@ -238,10 +239,11 @@ describe('Encryption Utilities', () => {
       const salt = generateSalt();
 
       testCases.forEach((data) => {
-        const encrypted = encrypt(data, password, salt);
+        const actual = typeof data === 'string' ? data : JSON.stringify(data);
+        const encrypted = encrypt(actual, password, salt);
         const decrypted = decrypt(encrypted, password, salt);
 
-        expect(decrypted).toEqual(data);
+        expect(decrypted).toEqual(actual);
       });
     });
   });

lib/util/encryption.ts

@@ -5,18 +5,24 @@ const TEST_DATA = 'Irrelevant data for password verification';
 export const generateSalt = () => {
   const validChars =
     'ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789';
-  let array = new Uint8Array(32);
+  const array = new Uint8Array(32);
   globalThis.crypto.getRandomValues(array);
-  array = array.map((x) => validChars.charCodeAt(x % validChars.length));
-  const salt = String.fromCharCode.apply(null, array as any);
+  const numbers = Array.from(array, (x) =>
+    validChars.charCodeAt(x % validChars.length)
+  );
+  const salt = String.fromCharCode(...numbers);
   return salt;
 };
 
-export const encrypt = (data: any, password: string, salt: string) => {
+export const encrypt = (data: string, password: string, salt: string) => {
   return AES.encrypt(JSON.stringify(data), password + salt).toString();
 };
 
-export const decrypt = (data: any, password: string, salt: string): string => {
+export const decrypt = (
+  data: string,
+  password: string,
+  salt: string
+): string => {
   const decrypted = AES.decrypt(data, password + salt);
   return JSON.parse(decrypted.toString(enc.Utf8));
 };
@@ -33,7 +39,7 @@ export const verifyTestCipher = (
   try {
     const decrypted = decrypt(testCipher, password, salt);
     return decrypted === TEST_DATA;
-  } catch (error) {
+  } catch {
     return false;
   }
 };

lib/util/log.ts

@@ -47,25 +47,25 @@ export class Logger {
   /**
    * log a debug message
    */
-  debug = (message: string, ...args: any[]) =>
+  debug = (message: string, ...args: unknown[]) =>
     this._log(LogLevel.debug, message, args);
 
   /**
    * log an info message
    */
-  info = (message: string, ...args: any[]) =>
+  info = (message: string, ...args: unknown[]) =>
     this._log(LogLevel.info, message, args);
 
   /**
    * log a warn message
    */
-  warn = (message: string, ...args: any[]) =>
+  warn = (message: string, ...args: unknown[]) =>
     this._log(LogLevel.warn, message, args);
 
   /**
    * log an error message
    */
-  error = (message: string, ...args: any[]) =>
+  error = (message: string, ...args: unknown[]) =>
     this._log(LogLevel.error, message, args);
 
   /**
@@ -74,7 +74,7 @@ export class Logger {
    * @param message the message to log
    * @param args optional additional arguments to log
    */
-  private _log(level: LogLevel, message: string, args: any[]) {
+  private _log(level: LogLevel, message: string, args: unknown[]) {
     // don't log if the level to output is greater than the level of this message
     if (this._levelToOutput > level) return;
 

test/utils/mock-factory.ts

@@ -13,7 +13,7 @@ import { globalAccess } from './test-helpers';
 export interface MockSetup {
   localStorage: MockLocalStorage;
   wasmGlobal: WasmGlobal | null;
-  goInstance: any;
+  goInstance: ReturnType<typeof createGoInstanceMock>;
   namespace: string;
   cleanup: () => void;
 }