Upgraded eslint and build

eslint.config.mjs

@@ -6,14 +6,32 @@ export default tseslint.config(
     ignores: [
       '**/node_modules/',
       '.git/',
+      '.yarn/',
       'dist/',
       'jest.config.js',
       'media_data/',
       'tsup.config.js',
-      'packages/wasm_exec.js'
+      'packages/wasm_exec.js',
+      'eslint.config.mjs'
     ],
   },
   eslint.configs.recommended,
-  ...tseslint.configs.strict,
-  ...tseslint.configs.stylistic,
+  ...tseslint.configs.strictTypeChecked,
+  ...tseslint.configs.stylisticTypeChecked,
+  {
+    rules: {
+      '@typescript-eslint/no-confusing-void-expression': 'off',
+      '@typescript-eslint/restrict-template-expressions': ['error', {
+        allowNumber: true,
+      }]
+    }
+  },
+  {
+    languageOptions: {
+      parserOptions: {
+        projectService: true,
+        tsconfigRootDir: import.meta.dirname,
+      },
+    },
+  },
 );

package.json

@@ -24,29 +24,29 @@
     "test": "jest"
   },
   "devDependencies": {
-    "@eslint/js": "^9.10.0",
+    "@eslint/js": "^9.13.0",
     "@jest/globals": "^29.7.0",
-    "@types/chrome": "^0.0.271",
+    "@types/chrome": "^0.0.279",
     "@types/golang-wasm-exec": "^1.15.2",
-    "@types/jest": "^29.5.13",
-    "@types/node": "^20.16.5",
+    "@types/jest": "^29.5.14",
+    "@types/node": "^20.17.1",
     "@types/text-encoding": "^0.0.39",
-    "esbuild": "^0.23.1",
+    "esbuild": "^0.24.0",
     "esbuild-plugin-copy": "^2.1.1",
     "esbuild-plugin-json-merge": "^0.1.0",
     "esbuild-sass-plugin": "^3.3.1",
-    "eslint": "^9.10.0",
-    "globals": "^15.9.0",
+    "eslint": "^9.13.0",
+    "globals": "^15.11.0",
     "jest": "^29.7.0",
     "jest-environment-jsdom": "^29.7.0",
     "jest-transform-stub": "^2.0.0",
-    "sass-embedded": "^1.78.0",
+    "sass-embedded": "^1.80.4",
     "text-encoding": "^0.7.0",
     "ts-jest": "^29.2.5",
-    "tsup": "^8.2.4",
+    "tsup": "^8.3.5",
     "typed-assert": "^1.0.9",
-    "typescript": "^5.6.2",
-    "typescript-eslint": "^8.5.0"
+    "typescript": "^5.6.3",
+    "typescript-eslint": "^8.11.0"
   },
   "workspaces": [
     "tokenizer/**/*"

packages/chrome/index.ts

@@ -0,0 +1,4 @@
+// eslint-disable-next-line @typescript-eslint/no-deprecated
+const { getURL } = chrome.runtime;
+
+export { getURL };

packages/jq/index.ts

@@ -9,5 +9,6 @@ export const jq = async (data: string, query: string): Promise<TokenizerResponse
     await importWasm(go, 'jq.wasm');
   }
 
+  // eslint-disable-next-line @typescript-eslint/no-unsafe-return
   return window.___jq(data, query);
 };

packages/shared/import-wasm.ts

@@ -1,11 +1,12 @@
+import { getURL } from '@packages/chrome';
 import { loadWasm } from './wasm_helpers';
 
 export const importWasm = async (go: Go, wasmFile: string) => {
-  const webAssemblyInstance = await loadWasm(chrome.runtime.getURL(wasmFile), go.importObject);
+  const webAssemblyInstance = await loadWasm(getURL(wasmFile), go.importObject);
 
   void go.run(webAssemblyInstance.instance)
     .then(() => console.log(`Wasm module ${ wasmFile } loaded successfully`))
-    .catch(console.error)
+    .catch((error: unknown) => console.error(error))
     .finally(() => console.log('Wasm module loaded'));
 };
 

src/dom/build-dom.test.ts

@@ -171,17 +171,17 @@ describe('buildDom', () => {
     const result = buildDom(parsedJson);
 
     const invisibleElements = [
-      ...result.querySelectorAll('.toggle') ?? [],
-      ...result.querySelectorAll('.ellipsis') ?? [],
-      ...result.querySelectorAll('.properties-count') ?? [],
-      ...result.querySelectorAll('.items-count') ?? [],
+      ...result.querySelectorAll('.toggle'),
+      ...result.querySelectorAll('.ellipsis'),
+      ...result.querySelectorAll('.properties-count'),
+      ...result.querySelectorAll('.items-count'),
     ];
 
     invisibleElements.forEach((element) => element.remove());
 
     const text = result.textContent;
     assert(!!text, 'No text content found');
-    const parsed = JSON.parse(text);
+    const parsed = JSON.parse(text) as unknown;
 
     expect(parsed).toEqual(expected);
   });

src/dom/build-error-node.test.ts

@@ -5,7 +5,7 @@ describe('error-node', () => {
     Object.assign(global, {
       chrome: {
         runtime: {
-          getURL: jest.fn((a) => a),
+          getURL: jest.fn((a: string) => a),
         },
       },
     });

src/dom/build-error-node.ts

@@ -1,10 +1,11 @@
+import { getURL } from '@packages/chrome';
 import { element } from './helpres';
 
 export const buildErrorNode = (header: string, ...lines: string[]) => {
   const wrapper = document.createElement('div');
   wrapper.className = 'error';
   const image = document.createElement('img');
-  image.src = chrome.runtime.getURL('invalid.svg');
+  image.src = getURL('invalid.svg');
   image.alt = 'Error';
   const message = document.createElement('div');
   message.className = 'message';

src/extension.ts

@@ -46,24 +46,23 @@ export const runExtension = async () => {
     queryInput.style.display = 'none';
   });
 
-  queryButton.addEventListener('click', async () => {
+  queryButton.addEventListener('click', () => {
     rootContainer.classList.remove('raw', 'formatted');
     rootContainer.classList.add('query');
     queryInput.style.display = 'inline-block';
   });
 
-  queryInput.addEventListener('keydown', async (event) => {
-    if (event.key !== 'Enter') {
-      return;
-    }
-
-    const { jq } = await import('@packages/jq');
+  // eslint-disable-next-line @typescript-eslint/no-misused-promises
+  queryInput.addEventListener('keydown', async event => {
+    if (event.key === 'Enter') {
+      const { jq } = await import('@packages/jq');
 
-    const info = await jq(data.innerText, queryInput.value);
-    queryContainer.innerHTML = '';
-    queryContainer.appendChild(
-      prepareResponse(info),
-    );
+      const info = await jq(data.innerText, queryInput.value);
+      queryContainer.innerHTML = '';
+      queryContainer.appendChild(
+        prepareResponse(info),
+      );
+    }
   });
 
   formatContainer.appendChild(
@@ -77,7 +76,7 @@ const prepareResponse = (response: TokenizerResponse): HTMLElement => {
     : buildDom(response);
 };
 
-const extractLines = (response: ErrorNode) => response.error ?? 'Please check the file and try again.';
+const extractLines = (response: ErrorNode) => response.error;
 
 const extractHeader = (response: ErrorNode): string => response.scope === 'tokenizer'
   ? 'Invalid JSON file.'

src/helpres.ts

@@ -2,6 +2,7 @@ export const throws = (value?: string): never => {
   throw new Error(value ?? 'Unexpected value');
 };
 
+// eslint-disable-next-line @typescript-eslint/no-unnecessary-type-parameters
 export const query = <T extends HTMLElement = HTMLElement, D extends HTMLElement = HTMLElement>(
   value: D,
   selector: string,

src/main.ts

@@ -1,4 +1,4 @@
 import '../packages/wasm_exec.js';
 import { runExtension } from './extension';
 
-runExtension().catch((error) => console.error('Error running extension:', error))
+runExtension().catch((error: unknown) => console.error('Error running extension:', error));

tsconfig.json

@@ -14,7 +14,7 @@
     "strict": true,
     "skipLibCheck": true,
     "sourceRoot": ".",
-    "strictNullChecks": false,
+    "strictNullChecks": true,
     "paths": {
       "@packages/*": ["packages/*"]
     }