🐛 Wrong documentation for useExhaustiveDependencies

[jer-sen]

Environment information

https://docs.rome.tools/lint/rules/useexhaustivedependencies/

What happened?

Options have nothing to do with hooks considered as stable, it's a list of custom hooks of which the dependencies will be checked.

Stable hooks are defined here

tools/crates/rome_js_analyze/src/semantic_analyzers/nursery/use_exhaustive_dependencies.rs

Line 180 in 392d188

let stable_config: HashSet<StableReactHookConfiguration> = HashSet::from_iter([

and are not configurable:

        let stable_config: HashSet<StableReactHookConfiguration> = HashSet::from_iter([
            StableReactHookConfiguration::new("useState", Some(1)),
            StableReactHookConfiguration::new("useReducer", Some(1)),
            StableReactHookConfiguration::new("useTransition", Some(1)),
            StableReactHookConfiguration::new("useRef", None),
            StableReactHookConfiguration::new("useContext", None),
            StableReactHookConfiguration::new("useId", None),
            StableReactHookConfiguration::new("useSyncExternalStore", None),
        ]);

Expected result

Something like:

Allows to specify custom hooks - from libraries or internal projects - that will also be checked.

By the way, it would be nice to make stable hooks configurable!

Code of Conduct

• I agree to follow Rome's Code of Conduct

[ematipico]

@jer-sen Rome is NOT maintained anymore

Check the README: https://github.com/rome/tools