@st1ggy/linter-config 1.0.47 → 1.0.49

package/eslint.config.js

@@ -1,7 +1,9 @@
 import common from './src/eslint/eslint.config.common.js'
+import react from './src/eslint/eslint.config.react.js'
 
 export default [
-  ...common,
+  // ...common,
+  ...react,
   {
     rules: {
       'import/extensions': 'off',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@st1ggy/linter-config",
-  "version": "1.0.47",
+  "version": "1.0.49",
   "description": "ESLint, Stylelint and Prettier configs for React, Vue [WIP] and Svelte [WIP]",
   "private": false,
   "main": "index.js",

package/src/eslint/eslint.config.react.js

@@ -1,43 +1,20 @@
-import reactPlugin from 'eslint-plugin-react'
+import reactRecommended from 'eslint-plugin-react/configs/recommended.js'
 import reactHooksPlugin from 'eslint-plugin-react-hooks'
 
 import typescript from './eslint.config.typescript.js'
 
 export default [
   ...typescript,
+  reactRecommended,
+  // reactHooksPlugin.configs.recommended,
   {
-    plugins: {
-      react: reactPlugin,
-      'react-hooks': reactHooksPlugin,
-    },
+    plugins: { 'react-hooks': reactHooksPlugin },
     rules: {
-      'react/sort-comp': 'off',
-      'react/prop-types': 'off',
-      'react/jsx-curly-newline': 'off',
-      'react/state-in-constructor': 'off',
-      'react/require-default-props': 'off',
-      'react/jsx-props-no-spreading': 'off',
-      'react/jsx-filename-extension': 'off',
-      'react/destructuring-assignment': 'off',
-      'react/static-property-placement': 'off',
-      'react/jsx-one-expression-per-line': 'off',
-      'react/self-closing-comp': [
-        'error',
-        {
-          component: true,
-          html: true,
-        },
-      ],
-      'react/default-props-match-prop-types': [
-        'warn',
-        {
-          allowRequiredDefaults: true,
-        },
-      ],
-      'react/jsx-wrap-multilines': 'off',
-      'react/jsx-indent': 'off',
-      'react-hooks/rules-of-hooks': 'error',
-      'react-hooks/exhaustive-deps': 'error',
+      'react/boolean-prop-naming': 'error',
+      'react/button-has-type': 'error',
+      'react/checked-requires-onchange-or-readonly': 'error',
+      'react/default-props-match-prop-types': ['error', { allowRequiredDefaults: true }],
+      'react/destructuring-assignment': 'error',
       'react/function-component-definition': [
         'error',
         {
@@ -45,13 +22,35 @@ export default [
           unnamedComponents: 'arrow-function',
         },
       ],
-      'react/jsx-curly-brace-presence': [
-        'warn',
+      'react/hook-use-state': 'error',
+      'react/jsx-boolean-value': 'error',
+      'react/jsx-child-element-spacing': 'error',
+      'react/jsx-closing-bracket-location': 'error',
+      'react/jsx-closing-tag-location': 'error',
+      'react/jsx-curly-brace-presence': ['error', { props: 'never', children: 'never' }],
+      'react/jsx-curly-newline': ['error', { multiline: 'consistent', singleline: 'forbid' }],
+      'react/jsx-curly-spacing': [
+        'error',
         {
-          props: 'never',
-          children: 'never',
+          when: 'never',
+          attributes: { allowMultiline: false },
+          children: false,
         },
       ],
+      'react/jsx-equals-spacing': ['error', 'never'],
+      'react/jsx-first-prop-new-line': ['error', 'multiline'],
+      'react/jsx-fragments': 'error',
+      'react/jsx-handler-names': 'error',
+      'react/jsx-max-props-per-line': ['error', { when: 'multiline' }],
+      'react/jsx-no-constructed-context-values': 'error',
+      'react/jsx-no-leaked-render': 'error',
+      'react/jsx-no-useless-fragment': 'error',
+      'react/jsx-one-expression-per-line': ['error', { allow: 'single-child' }],
+      'react/jsx-props-no-multi-spaces': 'error',
+      'react/jsx-tag-spacing': 'error',
+      'react/no-multi-comp': 'error',
+      'react/self-closing-comp': ['error', { component: true, html: true }],
+      ...reactHooksPlugin.configs.recommended.rules,
     },
   },
 ]

package/src/eslint/eslint.config.typescript.js

@@ -2,8 +2,7 @@ import tseslint from 'typescript-eslint'
 
 import common from './eslint.config.common.js'
 
-// export default tseslint.config(...tseslint.configs.strict, ...tseslint.configs.stylistic, ...common, {
-export default tseslint.config(...tseslint.configs.recommended, ...common, {
+export default tseslint.config(...tseslint.configs.recommended, ...tseslint.configs.stylistic, ...common, {
   languageOptions: {
     parserOptions: {
       project: ['tsconfig.json'],
@@ -11,27 +10,12 @@ export default tseslint.config(...tseslint.configs.recommended, ...common, {
   },
   settings: {
     'import/resolver': {
-      typescript: {}, // this loads <rootdir>/tsconfig.json to eslint
+      typescript: {},
     },
   },
   rules: {
-    '@typescript-eslint/no-explicit-any': 'off',
-    '@typescript-eslint/ban-types': ['warn'],
-    '@typescript-eslint/no-shadow': [
-      'error',
-      {
-        ignoreTypeValueShadow: true,
-      },
-    ],
-    '@typescript-eslint/no-var-requires': ['off'],
-    '@typescript-eslint/no-empty-function': ['off'],
-    '@typescript-eslint/no-use-before-define': ['error'],
-    '@typescript-eslint/no-unused-vars': [
-      'error',
-      {
-        ignoreRestSiblings: true,
-      },
-    ],
+    '@typescript-eslint/no-shadow': ['error', { ignoreTypeValueShadow: true }],
+    '@typescript-eslint/no-unused-vars': ['error', { ignoreRestSiblings: true }],
     '@typescript-eslint/ban-ts-comment': [
       'error',
       {
@@ -40,11 +24,6 @@ export default tseslint.config(...tseslint.configs.recommended, ...common, {
       },
     ],
     '@typescript-eslint/switch-exhaustiveness-check': 'error',
-    '@typescript-eslint/consistent-type-imports': [
-      'error',
-      {
-        fixStyle: 'inline-type-imports',
-      },
-    ],
+    '@typescript-eslint/consistent-type-imports': ['error', { fixStyle: 'inline-type-imports' }],
   },
 })