@viclafouch/eslint-config-viclafouch 4.10.1-beta.1 → 4.12.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@viclafouch/eslint-config-viclafouch",
-  "version": "4.10.1-beta.1",
+  "version": "4.12.0",
   "description": "ESLint and Prettier Config from Victor de la Fouchardiere",
   "main": "index.js",
   "keywords": [
@@ -25,32 +25,33 @@
   "homepage": "https://github.com/viclafouch/eslint-config-viclafouch#readme",
   "peerDependencies": {
     "eslint": "^8.57.0",
-    "prettier": "^3.2.5",
-    "typescript": "^5.4.3"
+    "prettier": "^3.3.0",
+    "typescript": "^5.5.0"
   },
   "dependencies": {
-    "@babel/core": "^7.24.4",
-    "@babel/eslint-parser": "^7.24.1",
-    "@next/eslint-plugin-next": "^14.2.1",
-    "@rushstack/eslint-patch": "^1.10.2",
+    "@babel/core": "^7.25.2",
+    "@babel/eslint-parser": "^7.25.1",
+    "@next/eslint-plugin-next": "^14.2.5",
+    "@rushstack/eslint-patch": "^1.10.4",
     "@total-typescript/ts-reset": "^0.5.1",
-    "@typescript-eslint/eslint-plugin": "^7.6.0",
-    "@typescript-eslint/parser": "^7.6.0",
+    "@typescript-eslint/eslint-plugin": "^8.0.0",
+    "@typescript-eslint/parser": "^8.0.0",
     "app-root-path": "^3.1.0",
     "babel-loader": "^9.1.3",
     "eslint": "^8.57.0",
     "eslint-config-prettier": "^9.1.0",
     "eslint-plugin-import": "^2.29.1",
-    "eslint-plugin-jsx-a11y": "^6.8.0",
-    "eslint-plugin-prettier": "^5.1.3",
-    "eslint-plugin-promise": "^6.1.1",
-    "eslint-plugin-react": "^7.34.1",
-    "eslint-plugin-react-hooks": "^4.6.0",
-    "eslint-plugin-simple-import-sort": "^12.1.0",
-    "eslint-plugin-testing-library": "^6.2.1",
-    "get-tsconfig": "^4.7.3",
-    "prettier": "^3.2.5",
-    "typescript": "^5.4.5"
+    "eslint-plugin-jsx-a11y": "^6.9.0",
+    "eslint-plugin-prettier": "^5.2.1",
+    "eslint-plugin-promise": "^7.0.0",
+    "eslint-plugin-react": "^7.35.0",
+    "eslint-plugin-react-hooks": "^4.6.2",
+    "eslint-plugin-simple-import-sort": "^12.1.1",
+    "eslint-plugin-testing-library": "^6.2.2",
+    "get-tsconfig": "^4.7.6",
+    "prettier": "^3.3.3",
+    "typescript": "^5.5.4",
+    "prettier-plugin-curly": "^0.2.2"
   },
   "scripts": {
     "lint": "eslint .",

package/prettier.js

@@ -8,6 +8,7 @@ module.exports = {
     'prettier/prettier': [
       'error',
       {
+        plugins: ['prettier-plugin-curly'],
         semi: false,
         singleQuote: true,
         printWidth: 80,

package/rules/react.js

@@ -325,7 +325,7 @@ module.exports = {
     // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/function-component-definition.md
     'react/function-component-definition': [
       'error',
-      { namedComponents: 'arrow-function' }
+      { namedComponents: 'arrow-function', unnamedComponents: 'arrow-function' }
     ],
 
     // Prevent react contexts from taking non-stable values
@@ -368,7 +368,11 @@ module.exports = {
     ],
 
     // Don't require captions for video / audio
-    'jsx-a11y/media-has-caption': 'off'
+    'jsx-a11y/media-has-caption': 'off',
+
+    // Disable spreading props in components more than once
+    // https://github.com/jsx-eslint/eslint-plugin-react/blob/v7.35.0/docs/rules/jsx-props-no-spread-multi.md
+    'react/jsx-props-no-spread-multi': 'off'
   },
   overrides: [
     {

package/rules/typescript.js

@@ -60,12 +60,6 @@ module.exports = {
     'no-shadow': 'off',
     '@typescript-eslint/no-shadow': baseVariablesRules['no-shadow'],
 
-    // Replace @viclafouch/eslint 'no-throw-literal' rule with '@typescript-eslint' version
-    // https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/no-throw-literal.md
-    'no-throw-literal': 'off',
-    '@typescript-eslint/no-throw-literal':
-      baseBestPracticesRules['no-throw-literal'],
-
     // Replace @viclafouch/eslint 'no-unused-expressions' rule with '@typescript-eslint' version
     // https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/no-unused-expressions.md
     'no-unused-expressions': 'off',
@@ -114,10 +108,6 @@ module.exports = {
       }
     ],
 
-    // No useless template literals
-    // https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/no-useless-template-literals.md
-    '@typescript-eslint/no-useless-template-literals': 'error',
-
     // Require consistently using T[] instead of Array<T>
     // https://typescript-eslint.io/rules/array-type
     '@typescript-eslint/array-type': 'error',
@@ -140,7 +130,14 @@ module.exports = {
 
     // No more "as Record<any, any>" in Array.reduce initial value, use generics
     // https://typescript-eslint.io/rules/prefer-reduce-type-parameter
-    '@typescript-eslint/prefer-reduce-type-parameter': 'error'
+    '@typescript-eslint/prefer-reduce-type-parameter': 'error',
+
+    // Require Promise-like statements to be handled appropriately.
+    // https://typescript-eslint.io/rules/no-floating-promises
+    '@typescript-eslint/no-floating-promises': 'error',
+
+    // Prefer using nullish coalescing (??) over logical (||) when possible.
+    '@typescript-eslint/prefer-nullish-coalescing': 'error'
 
     // '@typescript-eslint/ban-types': [
     //   'error',

package/tsconfig.json

@@ -1,4 +1,5 @@
 {
+  "$schema": "https://json.schemastore.org/tsconfig",
   "include": ["./**/*"],
   "compilerOptions": {
     "target": "esnext",
@@ -9,12 +10,19 @@
     "forceConsistentCasingInFileNames": true,
     "noEmit": true,
     "esModuleInterop": true,
-    "module": "esnext",
+    // External bundlers should use preserve
+    "module": "preserve",
+    // Make indexing stricter, not accessing unknown properties
     "noUncheckedIndexedAccess": true,
     "noPropertyAccessFromIndexSignature": false,
     "moduleResolution": "node",
+    // No accidental global script
+    "moduleDetection": "force",
     "resolveJsonModule": true,
+    // Force to "type-only" imports. Erase the import {} statement on runtime.
+    "verbatimModuleSyntax": true,
     "isolatedModules": true,
+    // Forbid any
     "noImplicitAny": true,
     "jsx": "preserve",
     "incremental": true