🐛 useExhaustiveDependencies rule should handle preact

[ssssota]

Environment information

❯ pnpm exec biome rage
CLI:
  Version:                      1.6.0
  Color support:                true

Platform:
  CPU Architecture:             x86_64
  OS:                           windows

Environment:
  BIOME_LOG_DIR:                unset
  NO_COLOR:                     unset
  TERM:                         unset
  JS_RUNTIME_VERSION:           "v18.17.0"
  JS_RUNTIME_NAME:              "node"
  NODE_PACKAGE_MANAGER:         "pnpm/8.15.4"

Biome Configuration:
  Status:                       Loaded successfully
  Formatter disabled:           false
  Linter disabled:              false
  Organize imports disabled:    false
  VCS disabled:                 true

Workspace:
  Open Documents:               0

What happened?

No diagnostics report like importing from react.

#3 (comment)

import {useEffect} from 'preact/hooks';
// import {useEffect} from 'react';
function Component() {
  let a = 0;
  useEffect(() => {
    console.log(a);
  }, []);
  return null;
}

Playground

Expected result

Diagnostics report from useExhaustiveDependencies.

Code of Conduct

• I agree to follow Biome's Code of Conduct

[arendjr]

Thanks for reporting! As a workaround, you should be able to configure the hooks in the rule's configuration in the meantime.

[ematipico]

I am going to change the labels on this one.

I deem this as an enhancement. We have built those rules with React in mind, and I'm not sure of preact hooks work the same.

Even so, our logic didn't take preact in mind.

[arendjr]

Implemented here: #2105

I think this covers the places where React/Preact compatibility is relevant for us, but please let me know if I missed something.