chore: make eslint run again and format the codebase

.eslintrc.cjs

@@ -2,20 +2,17 @@ module.exports = {
   env: {
     browser: true,
     es2021: true,
-    jest: true
+    jest: true,
   },
   extends: [
     'eslint:recommended',
     'plugin:react/recommended',
     'plugin:react/jsx-runtime',
+    'plugin:react-hooks/recommended',
+    'plugin:@typescript-eslint/recommended',
     '@electron-toolkit/eslint-config-ts/recommended',
     '@electron-toolkit/eslint-config-prettier',
-    'eslint:recommended',
-    'plugin:react/recommended',
-    'plugin:@typescript-eslint/recommended',
     'prettier',
-    'react-app',
-    'react-app/jest'
   ],
   rules: {
     // disable the rule for all files
@@ -27,25 +24,42 @@ module.exports = {
     '@typescript-eslint/no-explicit-any': 'warn',
     '@typescript-eslint/no-this-alias': 'warn',
     'react/react-in-jsx-scope': 'off',
-    camelcase: 'error',
-    'spaced-comment': 'error',
-    quotes: ['error', 'single'],
+    // the NethVoice API and the design tokens use snake_case on the wire:
+    // check declarations, not properties
+    camelcase: ['error', { properties: 'never' }],
+    // the '/' marker keeps TypeScript triple-slash directives intact:
+    // without it the autofix rewrites /// <reference ... /> into // / <reference ... />
+    'spaced-comment': ['error', 'always', { markers: ['/'] }],
+    quotes: ['error', 'single', { avoidEscape: true }],
     'no-duplicate-imports': 'error',
-    'react/display-name': false,
-    'import/no-anonymous-default-export': 'off'
+    'react/display-name': 'off',
+    // TypeScript prop types already cover what prop-types would check
+    'react/prop-types': 'off',
+    '@typescript-eslint/ban-ts-comment': 'warn',
   },
+  overrides: [
+    {
+      // main, preload and shared are plain TypeScript: a use* factory there is not
+      // a React hook, so the React rules can only produce false positives
+      files: ['src/main/**', 'src/preload/**', 'src/shared/**'],
+      rules: {
+        'react-hooks/rules-of-hooks': 'off',
+        'react-hooks/exhaustive-deps': 'off',
+      },
+    },
+  ],
   parser: '@typescript-eslint/parser',
   parserOptions: {
     ecmaFeatures: {
-      jsx: true
+      jsx: true,
     },
     ecmaVersion: 'latest',
-    sourceType: 'module'
+    sourceType: 'module',
   },
   plugins: ['react', 'react-hooks', '@typescript-eslint', 'prettier'],
   settings: {
-    'import/resolver': {
-      typescript: {}
-    }
-  }
+    react: {
+      version: 'detect',
+    },
+  },
 }

electron.vite.config.ts

@@ -10,16 +10,16 @@ export default defineConfig({
         '@/lib': resolve('src/main/lib'),
         '@/classes': resolve('src/main/classes'),
         '@shared': resolve('src/shared'),
-      }
-    }
+      },
+    },
   },
   preload: {
     plugins: [externalizeDepsPlugin()],
     resolve: {
       alias: {
-        '@shared': resolve('src/shared')
-      }
-    }
+        '@shared': resolve('src/shared'),
+      },
+    },
   },
   renderer: {
     assetsInclude: ['src/renderer/public/**/*'],
@@ -31,9 +31,9 @@ export default defineConfig({
         '@/assets': resolve('src/renderer/src/assets'),
         '@/pages': resolve('src/renderer/src/pages'),
         '@/components': resolve('src/renderer/src/components'),
-        '@/utils': resolve('src/renderer/src/utils')
-      }
+        '@/utils': resolve('src/renderer/src/utils'),
+      },
     },
-    plugins: [react()]
-  }
+    plugins: [react()],
+  },
 })

package.json

@@ -71,6 +71,7 @@
     "electron-vite": "^2.0.0",
     "eslint": "^8.56.0",
     "eslint-plugin-react": "^7.33.2",
+    "eslint-plugin-react-hooks": "^5.2.0",
     "framer-motion": "^11.11.8",
     "i18next": "^22.4.9",
     "i18next-browser-languagedetector": "^7.0.1",

postcss.config.js

@@ -3,6 +3,6 @@ module.exports = {
     'postcss-import': {},
     'tailwindcss/nesting': {},
     tailwindcss: {},
-    autoprefixer: {}
-  }
+    autoprefixer: {},
+  },
 }