fix: eslint issues

Author: marabesi

package.json

@@ -98,8 +98,8 @@
     "package-macos": "npm run setup && electron-builder --macos -p never",
     "build-app-image": "npm run setup && electron-builder --linux AppImage -p never",
     "electron": "npm run setup && wait-on tcp:3000 && electron build/",
-    "lint": "eslint src/**/*.{ts,tsx}",
-    "lint:fix": "eslint src/**/*.{ts,tsx} --fix",
+    "lint": "eslint 'src/**/*.{ts,tsx}'",
+    "lint:fix": "eslint 'src/**/*.{ts,tsx}' --fix",
     "serve": "http-server ./build -p 3000",
     "prepare": "husky"
   },

src/components/ui/editor/JsonEditor.tsx

@@ -35,7 +35,7 @@ export default forwardRef(function JsonEditor(props: Props, ref: ForwardedRef<Re
 
   const basicSetup: BasicSetupOptions = {};
   if (editorOptions.options) {
-    // @ts-ignore
+    // @ts-expect-error dynamic key assignment into BasicSetupOptions
     editorOptions.options.forEach((item: Option) => basicSetup[item.title as keyof BasicSetupOptions] = item.active);
   }
 
@@ -46,7 +46,7 @@ export default forwardRef(function JsonEditor(props: Props, ref: ForwardedRef<Re
   };
 
   if (editorOptions.properties) {
-    // @ts-ignore
+    // @ts-expect-error dynamic key assignment into CSSProperties
     editorOptions.properties.forEach((item: Properties) => style[item.key] = item.value);
   }
 

src/components/ui/io/Button.tsx

@@ -1,7 +1,7 @@
-import { DetailedHTMLProps, ReactNode } from 'react';
+import { ButtonHTMLAttributes, DetailedHTMLProps, ReactNode } from 'react';
 
 type ButtonProps = {
-  onClick?: DetailedHTMLProps<any, any>
+  onClick?: DetailedHTMLProps<ButtonHTMLAttributes<HTMLButtonElement>, HTMLButtonElement>['onClick']
   children?: ReactNode
   className?: string
   title?: string

src/components/ui/menu/JsonMenu.tsx

@@ -1,13 +1,13 @@
-import { BaseSyntheticEvent, DetailedHTMLProps, useRef } from 'react';
+import { BaseSyntheticEvent, useRef } from 'react';
 import { FaRegClipboard, FaRegFileArchive, FaRegTrashAlt, FaSearch } from 'react-icons/fa';
 import Button from '../io/Button';
 import { useToolbarContext } from '../../../ToolbarContext';
 import { usePersistenceContext } from '../../../PersistenceContext';
 import { useClipboardContext } from '../../../ClipboardContext';
 
 interface Props {
-  onLoadedFile: DetailedHTMLProps<any, any>;
-  onSearch: DetailedHTMLProps<any, any>;
+  onLoadedFile: (content: string | ArrayBuffer | null) => void;
+  onSearch: () => void;
 }
 
 export default function JsonMenu({ onLoadedFile, onSearch } : Props) {
@@ -56,7 +56,7 @@ export default function JsonMenu({ onLoadedFile, onSearch } : Props) {
       <Button
         onClick={() => {
           if (fileContent.current) {
-            // @ts-ignore at the  time of this code, there were no options to reset without accessing the ref
+            // @ts-expect-error no typed API to reset a file input via ref
             fileContent.current.value = '';
           }
           deleteJson();

src/components/ui/menu/ResultMenu.tsx

@@ -1,13 +1,12 @@
 import { FaBackspace, FaRegCopy, FaSearch, FaTerminal, FaUserFriends } from 'react-icons/fa';
-import { DetailedHTMLProps } from 'react';
 import InputText from '../io/InputText';
 import Button from '../io/Button';
 import { useToolbarContext } from '../../../ToolbarContext';
 import { usePersistenceContext } from '../../../PersistenceContext';
 import { useClipboardContext } from '../../../ClipboardContext';
 
 interface Props {
-  onSearch: DetailedHTMLProps<any, any>;
+  onSearch: () => void;
 }
 
 export default function ResultMenu({ onSearch }: Props) {

src/setupTests.ts

@@ -6,16 +6,16 @@ import '@testing-library/jest-dom';
 import toast from 'react-hot-toast';
 import 'jsdom-worker';
 import { TextEncoder, TextDecoder } from 'util';
-
+// eslint-disable-next-line @typescript-eslint/no-require-imports
 const fmt2json = require('format-to-json');
 
 Object.assign(global, { TextDecoder, TextEncoder });
 
-// @ts-ignore
+// @ts-expect-error fmt2json is injected as a global for the worker simulation
 // eslint-disable-next-line no-undef
 globalThis.fmt2json = fmt2json;
 
-// @ts-ignore
+// @ts-expect-error importScripts is a worker-only global, stubbed here for jsdom
 // eslint-disable-next-line no-undef
 globalThis.importScripts = (url: string) => {
   console.log(`returning null for url ${url}`);