@open-xchange/linter-presets 1.14.1 → 1.15.0

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## `1.15.0` – 2025-Oct-24
+
+- removed: (ESLint) plugin `eslint-plugin-react-hooks-static-deps`
+- removed: (ESLint) options `additionalHooks` and `staticHooks` in `env.react`
+- added: (ESLint) reactivated rule `react-hooks/exhaustive-deps` in `env.react`
+
 ## `1.14.1` – 2025-Oct-23
 
 - chore: bump dependencies

package/dist/eslint/env/react.d.ts

@@ -9,34 +9,6 @@ export interface EnvReactOptions extends EnvBaseOptions {
      * Additional hooks that will be treated as effect hooks, like "useEffect".
      */
     effectHooks?: readonly string[];
-    /**
-     * Configuration for the rule "react-hooks-static-deps/exhaustive-deps".
-     *
-     * Additional hooks that will be checked for dependencies, as a dictionary
-     * mapping hook names to the parameter index of the callback function to be
-     * analyzed.
-     *
-     * @example
-     *  additionalHooks: {
-     *    useCustomHook: 0,   // usage: `useCustomHook(callback, [deps])`
-     *    useOtherHook: 1,    // usage: `useOtherHook(value, callback, [deps])`
-     *  }
-     */
-    additionalHooks?: Record<string, number>;
-    /**
-     * Configuration for the rule "react-hooks-static-deps/exhaustive-deps".
-     *
-     * Additional hooks that return stable values, as a dictionary mapping hook
-     * names to the declaration of the stable values the hook will return.
-     *
-     * @example
-     *  staticHooks: {
-     *    useCustomHook: true,              // entire return value is stable
-     *    useOtherHook: [false, true],      // second array element is stable
-     *    useThirdHook: { callback: true }, // property "callback" is stable
-     *  }
-     */
-    staticHooks?: Record<string, boolean | boolean[] | Record<string, boolean>>;
 }
 /**
  * Creates configuration objects with linter rules for ReactJS.
@@ -44,7 +16,6 @@ export interface EnvReactOptions extends EnvBaseOptions {
  * Wraps the following packages:
  * - `@eslint-react/eslint-plugin`
  * - `eslint-plugin-react-hooks`
- * - `eslint-plugin-react-hooks-static-deps`
  * - `eslint-plugin-react-refresh`
  * - `eslint-plugin-jsx-a11y`
  *

package/dist/eslint/env/react.js

@@ -1,9 +1,7 @@
 import reactPlugin from "@eslint-react/eslint-plugin";
 import reactHooksPluginModule from "eslint-plugin-react-hooks";
-import reactHooksStaticDepsPlugin from "eslint-plugin-react-hooks-static-deps";
 import reactRefreshPlugin from "eslint-plugin-react-refresh";
 import jsxA11yPlugin from "eslint-plugin-jsx-a11y";
-import { fixupPluginRules } from "@eslint/compat";
 import { parser } from "typescript-eslint";
 import { createConfig, customRules, convertRuleWarningsToErrors } from "../shared/env-utils.js";
 // workaround for broken typings
@@ -15,7 +13,6 @@ const reactHooksPlugin = reactHooksPluginModule;
  * Wraps the following packages:
  * - `@eslint-react/eslint-plugin`
  * - `eslint-plugin-react-hooks`
- * - `eslint-plugin-react-hooks-static-deps`
  * - `eslint-plugin-react-refresh`
  * - `eslint-plugin-jsx-a11y`
  *
@@ -63,19 +60,6 @@ export default function react(envOptions) {
         }),
         // "react-hooks" plugin and recommended rules, raise all recommended rules to "error" level
         createConfig("env.react.react-hooks", envOptions, convertRuleWarningsToErrors(reactHooksPlugin.configs.flat.recommended)),
-        // "react-hooks-static-deps" plugin
-        (envOptions.additionalHooks ?? envOptions.staticHooks) && createConfig("env.react.static-deps", envOptions, {
-            plugins: {
-                "react-hooks-static-deps": fixupPluginRules(reactHooksStaticDepsPlugin), // https://github.com/stoikio/eslint-plugin-react-hooks-static-deps/issues/1
-            },
-            rules: {
-                "react-hooks/exhaustive-deps": "off",
-                "react-hooks-static-deps/exhaustive-deps": ["error", {
-                        additionalHooks: envOptions.additionalHooks,
-                        staticHooks: envOptions.staticHooks,
-                    }],
-            },
-        }),
         // "react-refresh" plugin
         createConfig("env.react.react-refresh", envOptions, reactRefreshPlugin.configs.vite),
         // "jsx-a11y" plugin

package/docs/eslint/env/react.md

@@ -4,7 +4,6 @@ Creates configuration objects with linter rules for ReactJS. Adds the following
 
 - [@eslint-react/eslint-plugin](https://eslint-react.xyz/)
 - [eslint-plugin-react-hooks](https://github.com/facebook/react/tree/main/packages/eslint-plugin-react-hooks)
-- [eslint-plugin-react-hooks-static-deps](https://github.com/stoikio/eslint-plugin-react-hooks-static-deps)
 - [eslint-plugin-react-refresh](https://github.com/ArnaudBarre/eslint-plugin-react-refresh)
 - [eslint-plugin-jsx-a11y](https://github.com/jsx-eslint/eslint-plugin-jsx-a11y)
 
@@ -23,8 +22,6 @@ function react(options: EnvReactOptions): Linter.Config[];
 | `ignores` | `string[]` | `[]` | Glob patterns for source files matching `files` to be ignored. |
 | `rules` | `Linter.RulesRecord` | `{}` | Additional linter rules to be added to the configuration. |
 | `effectHooks` | `string[]` | `[]` | Additional hooks that will be treated as effect hooks, like "useEffect". |
-| `additionalHooks` | `Record<string,number>` | `{}` | Additional hooks that will be checked for dependencies. |
-| `staticHooks` | `Record<string,boolean\|boolean[]\|Record<string,boolean>>` | `{}` | Additional hooks that return stable values. |
 
 ## Example
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@open-xchange/linter-presets",
-  "version": "1.14.1",
+  "version": "1.15.0",
   "description": "Configuration presets for ESLint and StyleLint",
   "repository": {
     "type": "git",
@@ -62,7 +62,6 @@
     "eslint-plugin-no-unsanitized": "^4.1.4",
     "eslint-plugin-promise": "^7.2.1",
     "eslint-plugin-react-hooks": "^7.0.0",
-    "eslint-plugin-react-hooks-static-deps": "^1.0.7",
     "eslint-plugin-react-refresh": "^0.4.24",
     "eslint-plugin-regexp": "^2.10.0",
     "eslint-plugin-testing-library": "^7.13.3",