Update vite and eslint configs (#176)

* remove configuration for vitest, it's in vite.config.ts now

* use typescript in vite config for coherence

* detail: force error for exhaustive-deps (and don't rely on outdated 'recommended' config of react hooks)

* use vite.config.ts for coverage as well as for test

* load all the rules and only overrride the non default one

Author: severo

eslint.config.js

@@ -25,7 +25,7 @@ export default typescript.config(
     rules: {
       ...react.configs.recommended.rules,
       ...react.configs['jsx-runtime'].rules,
-      ...reactHooks.configs.recommended.rules,
+      ...reactHooks.configs['recommended-latest'].rules,
       'react-refresh/only-export-components': [
         'warn',
         { allowConstantExport: true },
@@ -78,6 +78,8 @@ export default typescript.config(
       '@typescript-eslint/restrict-template-expressions': 'off',
       '@typescript-eslint/no-unused-vars': 'warn',
       '@typescript-eslint/require-await': 'warn',
+      // react hooks
+      'react-hooks/exhaustive-deps': 'error',
     },
     settings: { react: { version: 'detect' } },
   },

package.json

@@ -28,10 +28,10 @@
   ],
   "scripts": {
     "build:types": "tsc -b",
-    "build:lib": "vite build -c vite.lib.config.js",
+    "build:lib": "vite build -c vite.lib.config.ts",
     "build:app": "vite build",
     "build": "run-s build:lib build:types build:app",
-    "coverage": "vitest run -c vite.lib.config.js --coverage --coverage.include=src --coverage.include=bin",
+    "coverage": "vitest run --coverage --coverage.include=src --coverage.include=bin",
     "dev": "run-p -l watch:ts watch:vite watch:serve",
     "lint": "eslint",
     "lint:fix": "eslint --fix",

vite.lib.config.js vite.lib.config.tsvite.lib.config.js renamed to vite.lib.config.ts

@@ -29,5 +29,4 @@ export default defineConfig({
     },
     sourcemap: true,
   },
-  test: { environment: 'jsdom', globals: true },
 })