fix(linter/exhaustive-deps): ignore empty strings passed to additionalHooks option

[holoflash]

Issue

Passing an empty string to additionalHooks in the react-hooks/exhaustive-deps rule:
"react-hooks/exhaustive-deps": ["error", { "additionalHooks": "" }],
causes the linter to consider all functions to be additionalHooks.

Behaviour in ESLint

Passing an empty string doesn't do anything.
Passing "()" causes the same problematic behaviour:
"react-hooks/exhaustive-deps": ["warn", { additionalHooks: "()" }],
The same is true for oxlint.

Suggested fix

Check for empty string.
Since ESLint is able to take an empty string without breaking - so should oxlint.
Ideally - handle the "()" case too, or leave it as is to match original behaviour.

How to test

Add:
"react-hooks/exhaustive-deps": ["error", { "additionalHooks": "" }],
to your .oxlintrc.json file
Lint the codebase.

Example:

function MyComponent(props) {
          const foo = bar.add();
    }

  ⚠ eslint-plugin-react-hooks(exhaustive-deps): React hook add requires an effect callback.
   ╭─[exhaustive_deps.tsx:2:23]
 1 │ function MyComponent(props) {
 2 │           const foo = bar.add();
   ·                       ─────────
 3 │         }
   ╰────
  help: Did you forget to pass a callback to the hook?

-------- rule config --------
[
  {
    "additionalHooks": ""
  }
]

[graphite-app]

How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

• 0-merge - adds this PR to the back of the merge queue
• hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

[Copilot]

Pull Request Overview

This PR fixes a bug in the react-hooks/exhaustive-deps linter rule where passing an empty string to the additionalHooks configuration causes all functions to be incorrectly treated as additional hooks. The fix adds validation to filter out empty strings before regex compilation.

• Adds empty string filtering to prevent incorrect regex matching behavior
• Includes a test case to verify the fix works correctly

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[camc314]

Thank you!

[codspeed-hq]

CodSpeed Performance Report

Merging #14464 will not alter performance

_{Comparing holoflash:bug/exhaustive-deps-additionalHooks (65969cc) with main (0f19be0)}

Summary

✅ 4 untouched
⏩ 33 skipped¹

Footnotes

1. 33 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩