fix: fix useEffect setup function check in web-api/no-leaked-event-listener, closes #1228

[Rel1cx]

What kind of change does this PR introduce?

• Bugfix
• Feature
• Docs
• Test
• New Binding issue #___
• Code style update
• Refactor
• Build-related changes
• Other, please describe:

Does this PR introduce a breaking change?

• Yes, and the changes were approved in issue #___
• No

Checklist

• When resolving issues, they are referenced in the PR's title (e.g fix: remove a typo, closes #___, #___)
• I have added a convincing reason for adding this feature, if necessary

Other information

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Preview	Comments	Updated (UTC)
eslint-react	Ready	Preview	Comment	Sep 10, 2025 9:11am

[Copilot]

Pull Request Overview

This PR fixes a bug in the web-api/no-leaked-event-listener rule where the useEffect setup function check was incorrectly filtering out valid cases. The fix removes overly restrictive conditions that were preventing the rule from detecting leaked event listeners in certain scenarios.

• Removes two restrictive conditions from the isFunctionOfUseEffectSetup function
• Adds a test case to verify the fix works for custom hooks using useEffect

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
packages/core/src/hook/hook-hierarchy.ts	Removes restrictive callee type and identity checks from useEffect setup function detection
packages/plugins/eslint-plugin-react-web-api/src/rules/no-leaked-event-listener.spec.ts	Adds test case for custom hook scenario that was previously not caught

---

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