eslint-plugin-react-hooks: fix compatibility with @typescript-eslint/[email protected]+

[neocturne]

Summary

In addition to TSTypeQuery, dependency nodes with a TSTypeReference
parent need to be ignored as well. Without this fix, generic type
variables will be listed as missing dependencies.

Example:

export function useFoo<T>(): (foo: T) => boolean {
    return useCallback((foo: T) => false, []);
}

This will report the following issue:

React Hook useCallback has a missing dependency: 'T'. Either include
it or remove the dependency array

Closes: #19742

Test Plan

I built the eslint-plugin-react-hooks bundle and copied it into the node_modules of my test project, which fixed the incorrect lint reports.

I don't think there is a good way to add a unit test for this, as the issue is TypeScript-specific?

[codesandbox-ci]

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 016abef:

Sandbox	Source
React	Configuration

[sizebot]

No significant bundle size changes to report.

Size changes (experimental)

Generated by 🚫 dangerJS against 016abef

[sizebot]

No significant bundle size changes to report.

Size changes (stable)

Generated by 🚫 dangerJS against 016abef

[sushruth]

A much needed fix!

[eps1lon]

In #19792

export function useFoo<T>(): (foo: T) => boolean {
    return useCallback((foo: T) => false, []);
}

would fail but pass once the patch from this PR was applied while still working with 2.x and 3.x

[gaearon]

Looks good.

[gaearon]

[email protected] should have the fix.

[chang-ke]

[email protected] should have the fix.

good job