chore: Fix new lint and type errors

Author: maxmilton

src/types.ts

@@ -3,7 +3,8 @@ import type { DEFAULT_SECTION_ORDER } from './utils';
 declare global {
   interface HTMLElement {
     /** `stage1` synthetic click event handler. */
-    __click?(event: MouseEvent): void;
+    // eslint-disable-next-line @typescript-eslint/no-invalid-void-type
+    __click?(event: MouseEvent): void | boolean | Promise<void>;
   }
 }
 

test/setup.ts

@@ -4,9 +4,7 @@ import { GlobalWindow, type Window } from 'happy-dom';
 /* eslint-disable no-var, vars-on-top */
 declare global {
   /** Real bun console. `console` is mapped to happy-dom's virtual console. */
-  // biome-ignore lint/style/noVar: define global
   var console2: Console;
-  // biome-ignore lint/style/noVar: define global
   var happyDOM: Window['happyDOM'];
 }
 /* eslint-enable */
@@ -74,7 +72,7 @@ function setupMocks(): void {
     },
     // @ts-expect-error - partial mock
     runtime: {
-      openOptionsPage: noop,
+      openOptionsPage: noopAsync,
     },
     // @ts-expect-error - partial mock
     sessions: {

test/unit/newtab.test.ts

@@ -74,7 +74,7 @@ describe('CSS', () => {
   test('does not contain any comments', () => {
     expect(css).not.toInclude('/*');
     expect(css).not.toInclude('*/');
-    expect(css).not.toInclude('//');
+    expect(css).not.toInclude('//'); // inline comments or URL protocol
     expect(css).not.toInclude('<!');
   });
 

test/unit/theme.test.ts

@@ -61,7 +61,7 @@ for (const theme of themeNames) {
     test('does not contain any comments', () => {
       expect(css).not.toInclude('/*');
       expect(css).not.toInclude('*/');
-      expect(css).not.toInclude('//');
+      expect(css).not.toInclude('//'); // inline comments or URL protocol
       expect(css).not.toInclude('<!');
     });