feat(eslint-config): tune eslint config

Author: dangreen

package.json

@@ -38,7 +38,7 @@
     "clean-publish": "^5.0.0",
     "commitizen": "^4.3.1",
     "del-cli": "^6.0.0",
-    "eslint": "^9.0.0",
+    "eslint": "9.31.0",
     "inquirer": "^9.3.7",
     "nano-staged": "^0.8.0",
     "npm-package-json-lint": "^9.0.0",

packages/eslint-config/package.json

@@ -54,15 +54,15 @@
   },
   "dependencies": {
     "@eslint/js": "^9.12.0",
-    "@stylistic/eslint-plugin": "^5.2.0",
+    "@stylistic/eslint-plugin": "^5.5.0",
     "@typescript-eslint/eslint-plugin": "^8.0.0",
     "@typescript-eslint/parser": "^8.0.0",
     "eslint-plugin-import": "^2.30.0",
     "eslint-plugin-jest": "^29.0.1",
     "eslint-plugin-jest-dom": "^5.4.0",
-    "eslint-plugin-jsdoc": "^51.3.3",
+    "eslint-plugin-jsdoc": "^61.2.1",
     "eslint-plugin-react": "^7.35.0",
-    "eslint-plugin-react-hooks": "^5.2.0",
+    "eslint-plugin-react-hooks": "^7.0.1",
     "eslint-plugin-testing-library": "^7.5.3",
     "globals": "^16.3.0",
     "typescript-eslint": "^8.35.1"

packages/eslint-config/src/subconfigs/basic.js

@@ -28,7 +28,7 @@ export default [
         {
           nestedBinaryExpressions: false,
           ignoreJSX: 'multi-line',
-          enforceForArrowConditionals: false,
+          ignoredNodes: ['ArrowFunctionExpression[body.type=ConditionalExpression]'],
           enforceForSequenceExpressions: false
         }
       ],
@@ -148,16 +148,14 @@ export default [
       // Stylistic Issues
       '@stylistic/array-bracket-newline': [
         'error',
-        {
-          multiline: true
-        }
+        'consistent'
       ],
       '@stylistic/array-bracket-spacing': ['error', 'never'],
       '@stylistic/array-element-newline': [
         'error',
         {
-          multiline: true,
-          minItems: 3
+          consistent: true,
+          multiline: true
         }
       ],
       '@stylistic/block-spacing': 'error',

packages/eslint-config/src/subconfigs/react.js

@@ -112,7 +112,6 @@ export default [
           maximum: 1
         }
       ],
-      '@stylistic/jsx-props-no-multi-spaces': 'error',
       '@stylistic/jsx-self-closing-comp': 'error',
       'react/jsx-no-bind': 'error',
       'react/jsx-no-comment-textnodes': 'error',

packages/eslint-config/src/subconfigs/typescript.js

@@ -12,6 +12,7 @@ import {
 
 export default [
   {
+    files: tsFiles,
     plugins: {
       '@typescript-eslint': typescriptPlugin,
       '@stylistic': stylisticPlugin
@@ -159,7 +160,7 @@ export default [
           allowDefinitionFiles: true
         }
       ],
-      '@typescript-eslint/no-require-imports': 'off',
+      '@typescript-eslint/no-require-imports': 'error',
       '@typescript-eslint/no-this-alias': [
         'error',
         {
@@ -190,12 +191,6 @@ export default [
       '@typescript-eslint/no-dupe-class-members': 'error'
     }
   },
-  {
-    files: tsFiles,
-    rules: {
-      '@typescript-eslint/no-require-imports': 'error'
-    }
-  },
   {
     files: dtsFiles,
     rules: {