fix a few more issues

Author: emma-sg

src/autoclick.ts

@@ -109,9 +109,9 @@ export class AutoClick extends BackgroundBehavior {
       });
     }
   }
-  // @ts-expect-error TODO: this looks like a typo, there's no associated `try` block for this `catch` block, so it ends up being a method
-  catch(e) {
-    this.debug(e.toString());
+  // TODO: this looks like maybe a typo, there's no associated `try` block for this `catch` block, so it ends up being a method
+  catch(e: unknown) {
+    this.debug((e as Error).toString());
   }
 
   async done() {

src/autofetcher.ts

@@ -134,7 +134,7 @@ export class AutoFetcher extends BackgroundBehavior {
 
       const reader = resp.body!.getReader();
 
-      let res = null;
+      let res: ReadableStreamReadResult<Uint8Array> | null = null;
 
       do {
         res = await reader.read();

src/autoscroll.ts

@@ -122,7 +122,6 @@ export class AutoScroll
     }
 
     if (
-      // @ts-expect-error TODO not sure what self.scrollHeight is here
       (self.window.scrollY + self["scrollHeight"]) /
         self.document.scrollingElement!.scrollHeight <
       0.9

src/index.ts

@@ -415,7 +415,7 @@ export class BehaviorManager {
       }
     });
 
-    const promises = [];
+    const promises: Promise<void>[] = [];
 
     for (const url of urls) {
       promises.push(addLink(url));

src/lib/utils.ts

@@ -39,7 +39,7 @@ export async function waitUntilNode(
   timeout = 1000,
   interval = waitUnit,
 ): Promise<Node | null> {
-  let node = null;
+  let node: Node | null = null;
   let stop = false;
   const waitP = waitUntil(() => {
     node = xpathNode(path, root);
@@ -209,12 +209,12 @@ export async function openWindow(
     const p = new Promise((resolve) => (self["__bx_openResolve"] = resolve));
     await callBinding(self["__bx_open"], { url });
 
-    let win = null;
+    let win: WindowProxy | null = null;
 
     try {
-      win = await p;
+      win = (await p) as WindowProxy | null;
       if (win) {
-        return win as WindowProxy;
+        return win;
       }
     } catch (e) {
       console.warn(e);
@@ -248,7 +248,7 @@ export class RestoreState {
   }
 
   async restore(rootPath: string, childMatch: string) {
-    let root = null;
+    let root: Node | null = null;
 
     while (((root = xpathNode(rootPath)), !root)) {
       await sleep(100);
@@ -314,7 +314,7 @@ export function* xpathNodes(path: string, root?: Node | null) {
     null,
     XPathResult.ORDERED_NODE_ITERATOR_TYPE,
   );
-  let result = null;
+  let result: Node | null = null;
   while ((result = iter.iterateNext()) !== null) {
     yield result;
   }

src/site/twitter.ts

@@ -179,7 +179,7 @@ export class TwitterTimelineBehavior
       return;
     }
 
-    let mediaTweetUrl = null;
+    let mediaTweetUrl: string | null = null;
 
     try {
       mediaTweetUrl = new URL(