chore: update ESLint configuration and clean up unused rules

Author: AvivAbachi

eslint.config.js

@@ -24,10 +24,9 @@ export default [
       'test-results',
       'playwright-report',
       'storybook-static',
+      'public',
       '.nx',
-      'jest.config.ts',
       '**/*.js',
-      'public',
       'applitools.config.cjs',
     ],
   },
@@ -41,7 +40,10 @@ export default [
     },
   },
   {
-    settings: { 'import/resolver': { typescript: true }, react: { version: 'detect' } },
+    settings: {
+      'import/resolver': { typescript: true },
+      react: { version: 'detect' },
+    },
     languageOptions: {
       parser: typescriptEslintParser,
       parserOptions: {
@@ -54,21 +56,22 @@ export default [
   },
   {
     rules: {
-      ...typescriptEslintEslintPlugin.configs.recommended.rules,
-      ...typescriptEslintEslintPlugin.configs['recommended-requiring-type-checking'].rules,
-      ...eslintPluginReact.configs.recommended.rules,
+      // React
       'react-hooks/rules-of-hooks': 'error',
       'react/jsx-filename-extension': [1, { extensions: ['.tsx'] }],
       'react/react-in-jsx-scope': 'off',
+      // TypeScript
       '@typescript-eslint/no-explicit-any': 'warn',
       '@typescript-eslint/no-unsafe-call': 'off',
       '@typescript-eslint/no-unsafe-member-access': 'off',
       '@typescript-eslint/no-floating-promises': 'off',
       '@typescript-eslint/restrict-template-expressions': 'off',
       '@typescript-eslint/no-base-to-string': 'off',
       '@typescript-eslint/no-unsafe-assignment': 'off',
+      // Import
       'import/no-unused-modules': 'error',
       'import/order': 'error',
+      // Prettier
       'prettier/prettier': 'error',
     },
   },

src/pages/BugReportForm .tsx

@@ -86,7 +86,6 @@ const BugReportForm = () => {
         <Form
           form={form}
           name="bug-report"
-          // eslint-disable-next-line @typescript-eslint/no-misused-promises
           onFinish={onFinish}
           onFinishFailed={onFinishFailed}
           labelCol={{ span: 6 }}

src/pages/components/Preloader.stories.tsx

@@ -49,7 +49,6 @@ let promise: Promise<void> | null = null
 
 const useTimeout = (ms: number) => {
   if (!fulfilled) {
-    // eslint-disable-next-line @typescript-eslint/only-throw-error
     throw (promise ||= new Promise((resolve) => {
       setTimeout(() => {
         fulfilled = true

src/pages/components/map-related/MapLayers/BusToolTip.tsx

@@ -112,7 +112,6 @@ export function BusToolTip({ position, icon, children }: BusToolTipProps) {
               <li>
                 {`${t('sample_time')}: `}
                 <span>
-                  {/* eslint-disable-next-line @typescript-eslint/no-unnecessary-type-assertion */}
                   {dayjs(position.point!.recorded_at_time as string)
                     .tz('Israel')
                     .format(`l [${t('at_time')}] LT`)}

tests/utils.ts

@@ -1,4 +1,4 @@
-/* eslint-disable @typescript-eslint/no-explicit-any, @typescript-eslint/no-unsafe-return */
+/* eslint-disable @typescript-eslint/no-explicit-any */
 import * as fs from 'fs'
 import * as path from 'path'
 import * as crypto from 'crypto'