Fix newer TS errors

Signed-off-by: Simon Bennetts <psiinon@gmail.com>

Author: psiinon

.github/workflows/ci.yaml

@@ -20,6 +20,8 @@ jobs:
 
       - name: Set Node.js environment
         uses: actions/setup-node@v4
+        with:
+          node-version: '20'
 
       - name: Install dependencies
         run: yarn install

source/Background/index.ts

@@ -262,15 +262,19 @@ async function handleMessage(
 }
 
 async function onMessageHandler(
-  message: MessageEvent,
+  message: unknown,
   _sender: Runtime.MessageSender
 ): Promise<number | ZestScriptMessage> {
   let val: number | ZestScriptMessage = 2;
   const items = await Browser.storage.sync.get({
     zapurl: 'http://zap/',
     zapkey: 'not set',
   });
-  const msg = await handleMessage(message, items.zapurl, items.zapkey);
+  const msg = await handleMessage(
+    message as MessageEvent,
+    items.zapurl as string,
+    items.zapkey as string
+  );
   if (!(typeof msg === 'boolean')) {
     val = msg;
   }
@@ -286,7 +290,11 @@ function cookieChangeHandler(
       zapkey: 'not set',
     })
     .then((items) => {
-      reportCookies(changeInfo.cookie, items.zapurl, items.zapkey);
+      reportCookies(
+        changeInfo.cookie,
+        items.zapurl as string,
+        items.zapkey as string
+      );
     });
 }
 

source/ContentScript/index.ts

@@ -278,15 +278,23 @@ function injectScript(): Promise<boolean> {
 
 injectScript();
 
+/* eslint-disable  @typescript-eslint/no-explicit-any */
 Browser.runtime.onMessage.addListener(
-  (message: MessageEvent, _sender: Runtime.MessageSender) => {
+  (
+    message: any,
+    _sender: Runtime.MessageSender,
+    _sendResponse: (response?: any) => void
+  ) => {
     if (message.type === ZAP_START_RECORDING) {
       configureExtension();
       recorder.recordUserInteractions();
       recorder.initializationScript();
     } else if (message.type === ZAP_STOP_RECORDING) {
       recorder.stopRecordingUserInteractions();
     }
+
+    // Returning `true` keeps the message channel open for async responses
+    return true;
   }
 );
 

source/Options/index.tsx

@@ -54,14 +54,14 @@ function restoreOptions(): void {
     })
     .then((items) => {
       (document.getElementById(ZAP_URL) as HTMLInputElement).value =
-        items.zapurl;
+        items.zapurl as string;
       (document.getElementById(ZAP_KEY) as HTMLInputElement).value =
-        items.zapkey;
+        items.zapkey as string;
       (document.getElementById(ZAP_ENABLE) as HTMLInputElement).checked =
-        items.zapenable;
+        items.zapenable as boolean;
       (
         document.getElementById('window-close-input') as HTMLInputElement
-      ).checked = items.zapclosewindowhandle;
+      ).checked = items.zapclosewindowhandle as boolean;
     });
 }
 document.addEventListener('DOMContentLoaded', restoreOptions);

source/Popup/index.tsx

@@ -29,6 +29,7 @@ import {
   ZAP_START_RECORDING,
   ZAP_STOP_RECORDING,
 } from '../utils/constants';
+import {ZestScriptMessage} from '../types/zestScript/ZestScript';
 
 const STOP = i18n.t('stop');
 const START = i18n.t('start');
@@ -97,7 +98,7 @@ async function restoreState(): Promise<void> {
       } else {
         stoppedAnimation();
       }
-      scriptNameInput.value = items.zapscriptname;
+      scriptNameInput.value = items.zapscriptname as string;
       if (items.zapclosewindowhandle) {
         done?.classList.remove('invisible');
       } else {
@@ -189,7 +190,8 @@ async function handleSaveScript(): Promise<void> {
     await Browser.runtime.sendMessage({type: STOP_RECORDING});
   }
   Browser.runtime.sendMessage({type: SAVE_ZEST_SCRIPT}).then((items) => {
-    downloadZestScript(items.script, items.title);
+    const msg = items as ZestScriptMessage;
+    downloadZestScript(msg.script, msg.title);
   });
 }
 

source/types/zestScript/ZestScript.ts

@@ -92,7 +92,7 @@ class ZestScript {
   getZestScript(): Promise<ZestScriptMessage> {
     return new Promise((resolve) => {
       Browser.storage.sync.get({zapscriptname: this.title}).then((items) => {
-        this.title = items.zapscriptname;
+        this.title = items.zapscriptname as string;
         resolve({script: this.toJSON(), title: this.title});
       });
     });