eslint-config-vylda-vanilla-react 4.3.0 → 5.0.0

package/Changelog.md

@@ -4,6 +4,20 @@ All notable changes to this component will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [5.0.0] - 2025-12-06
+### Added
+- some missing comments for rules [Vilda Lipold]
+### Removed
+- TODO comments for enabled rules [Vilda Lipold]
+### Updated
+- packages [Vilda Lipold]
+- Rules
+  - react/hook-use-state [Vilda Lipold]
+  - react/jsx-no-leaked-render [Vilda Lipold]
+  - react/no-adjacent-inline-elements [Vilda Lipold]
+  - react/no-object-type-as-default-prop [Vilda Lipold]
+  - react-hooks/void-use-memo [Vilda Lipold]
+
 ## [4.3.0] - 2025-12-06
 ### Updated
 - eslint-config-vylda-vanilla package [Vilda Lipold]

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-config-vylda-vanilla-react",
-  "version": "4.3.0",
+  "version": "5.0.0",
   "description": "Eslint rules for JS React projects",
   "main": "index.js",
   "type": "module",
@@ -28,12 +28,12 @@
   "homepage": "https://gitlab.com/Vylda/eslint-config-vylda-vanilla-react",
   "license": "MIT",
   "dependencies": {
-    "eslint": "^9.39.1",
-    "eslint-config-vylda-vanilla": "^4.6.0",
+    "eslint": "^9.39.2",
+    "eslint-config-vylda-vanilla": "^5.0.0",
     "eslint-plugin-jsx-a11y": "^6.10.2",
     "eslint-plugin-react": "^7.37.5",
     "eslint-plugin-react-hooks": "^7.0.1",
-    "eslint-plugin-react-refresh": "^0.4.20",
+    "eslint-plugin-react-refresh": "^0.4.26",
     "globals": "^16.5.0"
   },
   "devDependencies": {

package/rules/hooks.mjs

@@ -122,7 +122,7 @@ const config = [{
     // by the component/hook. See [`useMemo()` docs](https://react.dev/reference/react/useMemo)
     // for more information..
     // Missing URL in the official documentation.
-    'react-hooks/void-use-memo': 'off',
+    'react-hooks/void-use-memo': 'error',
   },
 }];
 

package/rules/react.mjs

@@ -85,8 +85,8 @@ const config = [
         },
       ],
 
+      // Requires that components wrapped with forwardRef must have a ref parameter.
       // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/forward-ref-uses-ref.md
-      // TODO: semver-major, enable
       'react/forward-ref-uses-ref': 'error',
 
       // Enforce a specific function type for function components
@@ -101,12 +101,15 @@ const config = [
 
       // Ensure destructuring and symmetric naming of useState hook value and setter variables
       // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/hook-use-state.md
-      // TODO: semver-major, enable
-      'react/hook-use-state': 'off',
+      'react/hook-use-state': [
+        'error',
+        {
+          allowDestructuredState: true,
+        },
+      ],
 
       // Enforce sandbox attribute on iframe elements
       // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/iframe-missing-sandbox.md
-      // TODO: semver-major, enable
       'react/iframe-missing-sandbox': 'error',
 
       // Enforce boolean attributes notation in JSX
@@ -232,8 +235,7 @@ const config = [
 
       // Prevent problematic leaked values from being rendered
       // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/jsx-no-leaked-render.md
-      // TODO: semver-major, enable
-      'react/jsx-no-leaked-render': 'off',
+      'react/jsx-no-leaked-render': 'error',
 
       // Prevent usage of unwrapped JSX strings
       // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/jsx-no-literals.md
@@ -270,8 +272,8 @@ const config = [
       // rule in stylistic.mjs
       'react/jsx-props-no-multi-spaces': 'error',
 
+      // Disallow JSX prop spreading the same identifier multiple times
       // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/jsx-props-no-spread-multi.md
-      // TODO: semver-major, enable
       'react/jsx-props-no-spread-multi': 'error',
 
       // Disallow JSX props spreading
@@ -307,8 +309,7 @@ const config = [
 
       // Prevent adjacent inline elements not separated by whitespace
       // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/no-adjacent-inline-elements.md
-      // TODO: enable? semver-major
-      'react/no-adjacent-inline-elements': 'off',
+      'react/no-adjacent-inline-elements': 'warn',
 
       // Prevent usage of Array index in keys
       // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/no-array-index-key.md
@@ -366,9 +367,9 @@ const config = [
       // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/no-namespace.md
       'react/no-namespace': 'error',
 
+      // Disallow usage of referential-type variables as default param in functional component
       // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/no-object-type-as-default-prop.md
-      // TODO: semver-major, enable
-      'react/no-object-type-as-default-prop': 'off',
+      'react/no-object-type-as-default-prop': 'warn',
 
       // Prevent usage of shouldComponentUpdate when extending React.PureComponent
       // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/no-redundant-should-component-update.md
@@ -533,8 +534,9 @@ const config = [
         },
       ],
 
+      // Enforce defaultProps declarations alphabetical sorting
       // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/sort-default-props.md
-      // TODO: semver-major, enable?
+      // off because defaultProps is deprecated
       'react/sort-default-props': ['off', { ignoreCase: true }],
 
       // Enforce propTypes declarations alphabetical sorting
@@ -552,12 +554,10 @@ const config = [
 
       // Enforce state initialization style
       // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/state-in-constructor.md
-      // TODO: set to "never" once babel-preset-airbnb supports public class fields
       'react/state-in-constructor': ['error', 'always'],
 
       // Enforces where React component static properties should be positioned
       // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/static-property-placement.md
-      // TODO: set to "static public field" once babel-preset-airbnb supports public class fields
       'react/static-property-placement': ['error', 'static public field'],
 
       // Require style prop value be an object or var