@fullstacksjs/eslint-config 13.6.0 → 13.8.0

package/cjs/modules/react.js

@@ -7,9 +7,11 @@ Object.defineProperty(exports, "__esModule", {
 var _eslintPlugin = _interopRequireDefault(require("@eslint-react/eslint-plugin"));
 var _eslintPluginJsxA11y = _interopRequireDefault(require("eslint-plugin-jsx-a11y"));
 var _eslintPluginReactHooks = _interopRequireDefault(require("eslint-plugin-react-hooks"));
+var _conditions = require("../utils/conditions.js");
 function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
 /** @return { import('eslint').Linter.Config } */
 function react(options = {}) {
+  const projectService = options.typescript && options.typescript.tsconfigRootDir && options.typescript.projectService;
   return {
     plugins: {
       ..._eslintPlugin.default.configs.all.plugins,
@@ -86,7 +88,9 @@ function react(options = {}) {
       '@eslint-react/no-unstable-default-props': 'error',
       '@eslint-react/no-unused-class-component-members': 'warn',
       '@eslint-react/no-unused-state': 'warn',
-      '@eslint-react/no-unused-props': 'off',
+      ...(0, _conditions.predicate)(projectService, {
+        '@eslint-react/no-unused-props': 'warn'
+      }),
       '@eslint-react/no-useless-forward-ref': 'off',
       '@eslint-react/no-useless-fragment': 'warn',
       '@eslint-react/no-use-context': 'warn',
@@ -109,8 +113,23 @@ function react(options = {}) {
       '@eslint-react/web-api/no-leaked-event-listener': 'error',
       '@eslint-react/web-api/no-leaked-interval': 'error',
       '@eslint-react/web-api/no-leaked-resize-observer': 'error',
-      'react-hooks/exhaustive-deps': 'warn',
       'react-hooks/rules-of-hooks': 'error',
+      'react-hooks/exhaustive-deps': 'warn',
+      'react-hooks/config': 'error',
+      'react-hooks/error-boundaries': 'error',
+      'react-hooks/component-hook-factories': 'error',
+      'react-hooks/gating': 'error',
+      'react-hooks/globals': 'error',
+      'react-hooks/immutability': 'error',
+      'react-hooks/preserve-manual-memoization': 'error',
+      'react-hooks/purity': 'error',
+      'react-hooks/refs': 'error',
+      'react-hooks/set-state-in-effect': 'error',
+      'react-hooks/set-state-in-render': 'error',
+      'react-hooks/static-components': 'error',
+      'react-hooks/unsupported-syntax': 'warn',
+      'react-hooks/use-memo': 'error',
+      'react-hooks/incompatible-library': 'warn',
       'jsx-a11y/alt-text': 'warn',
       'jsx-a11y/anchor-has-content': 'error',
       'jsx-a11y/anchor-is-valid': 'error',

package/cjs/modules/typescript.js

@@ -63,8 +63,6 @@ function typescript(options = {}) {
         prefix: ['is', 'should', 'has', 'can', 'did', 'will', 'enable', 'loading']
       }] : [])],
       '@typescript-eslint/no-array-constructor': 'error',
-      '@typescript-eslint/no-base-to-string': 'off',
-      // false negative
       '@typescript-eslint/no-confusing-non-null-assertion': 'error',
       '@typescript-eslint/no-dupe-class-members': 'error',
       '@typescript-eslint/no-duplicate-enum-values': 'error',
@@ -171,6 +169,7 @@ function typescript(options = {}) {
           allowProtectedClassPropertyAccess: false,
           allowIndexSignaturePropertyAccess: true
         }],
+        '@typescript-eslint/no-base-to-string': 'error',
         '@typescript-eslint/no-confusing-void-expression': ['off', {
           ignoreArrowShorthand: true,
           ignoreVoidOperator: true

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fullstacksjs/eslint-config",
-  "version": "13.6.0",
+  "version": "13.8.0",
   "license": "MIT",
   "author": "fullstacks <fullstacksjs@gmail.com>",
   "description": "fullstacks eslint config",
@@ -51,7 +51,7 @@
     "eslint-plugin-playwright": "2.2.2",
     "eslint-plugin-prettier": "5.5.4",
     "eslint-plugin-promise": "7.2.1",
-    "eslint-plugin-react-hooks": "6.1.1",
+    "eslint-plugin-react-hooks": "7.0.0",
     "eslint-plugin-regexp": "2.10.0",
     "eslint-plugin-storybook": "0.12.0",
     "globals": "16.4.0",

package/src/modules/react.mjs

@@ -2,8 +2,12 @@ import reactPlugin from '@eslint-react/eslint-plugin';
 import a11yPlugin from 'eslint-plugin-jsx-a11y';
 import hooksPlugin from 'eslint-plugin-react-hooks';
 
+import { predicate } from '../utils/conditions.mjs';
+
 /** @return { import('eslint').Linter.Config } */
 function react(options = {}) {
+  const projectService = options.typescript && options.typescript.tsconfigRootDir && options.typescript.projectService;
+
   return {
     plugins: {
       ...reactPlugin.configs.all.plugins,
@@ -82,7 +86,9 @@ function react(options = {}) {
       '@eslint-react/no-unstable-default-props': 'error',
       '@eslint-react/no-unused-class-component-members': 'warn',
       '@eslint-react/no-unused-state': 'warn',
-      '@eslint-react/no-unused-props': 'off',
+      ...predicate(projectService, {
+        '@eslint-react/no-unused-props': 'warn',
+      }),
       '@eslint-react/no-useless-forward-ref': 'off',
       '@eslint-react/no-useless-fragment': 'warn',
       '@eslint-react/no-use-context': 'warn',
@@ -107,8 +113,24 @@ function react(options = {}) {
       '@eslint-react/web-api/no-leaked-interval': 'error',
       '@eslint-react/web-api/no-leaked-resize-observer': 'error',
 
-      'react-hooks/exhaustive-deps': 'warn',
       'react-hooks/rules-of-hooks': 'error',
+      'react-hooks/exhaustive-deps': 'warn',
+
+      'react-hooks/config': 'error',
+      'react-hooks/error-boundaries': 'error',
+      'react-hooks/component-hook-factories': 'error',
+      'react-hooks/gating': 'error',
+      'react-hooks/globals': 'error',
+      'react-hooks/immutability': 'error',
+      'react-hooks/preserve-manual-memoization': 'error',
+      'react-hooks/purity': 'error',
+      'react-hooks/refs': 'error',
+      'react-hooks/set-state-in-effect': 'error',
+      'react-hooks/set-state-in-render': 'error',
+      'react-hooks/static-components': 'error',
+      'react-hooks/unsupported-syntax': 'warn',
+      'react-hooks/use-memo': 'error',
+      'react-hooks/incompatible-library': 'warn',
 
       'jsx-a11y/alt-text': 'warn',
       'jsx-a11y/anchor-has-content': 'error',

package/src/modules/typescript.mjs

@@ -68,7 +68,6 @@ function typescript(options = {}) {
           : []),
       ],
       '@typescript-eslint/no-array-constructor': 'error',
-      '@typescript-eslint/no-base-to-string': 'off', // false negative
       '@typescript-eslint/no-confusing-non-null-assertion': 'error',
       '@typescript-eslint/no-dupe-class-members': 'error',
       '@typescript-eslint/no-duplicate-enum-values': 'error',
@@ -156,6 +155,7 @@ function typescript(options = {}) {
             allowIndexSignaturePropertyAccess: true,
           },
         ],
+        '@typescript-eslint/no-base-to-string': 'error',
         '@typescript-eslint/no-confusing-void-expression': ['off', { ignoreArrowShorthand: true, ignoreVoidOperator: true }], // Annoying and conflict with @typescript-eslint/no-meaningless-void-operator
         '@typescript-eslint/no-duplicate-type-constituents': 'warn',
         '@typescript-eslint/no-for-in-array': 'error',