Bug: [eslint-plugin-react-hooks] Constructions should be caught in default props/args

<!-- Please provide a clear and concise description of what the bug is. Include screenshots if needed. Please test using the latest version of the relevant React packages to make sure your issue has not already been fixed. --> React version: 17.0.1 ## Steps To Reproduce 1. ```js function useMyHook1(arr = []) { useEffect(() => { console.log(arr); }, [arr]); } function useMyHook2({ arr = [] }) { useEffect(() => { console.log(arr); }, [arr]); } function MyComponent({ arr = [] }) { useEffect(() => { console.log(arr); }, [arr]); return null; } ``` <!-- Your bug will get fixed much faster if we can run your code and it doesn't have dependencies other than React. Issues without reproduction steps or code examples may be immediately closed as not actionable. --> Link to code example: -- <!-- Please provide a CodeSandbox (https://codesandbox.io/s/new), a link to a repository on GitHub, or provide a minimal code example that reproduces the problem. You may provide a screenshot of the application if you think it is relevant to your bug report. Here are some tips for providing a minimal example: https://stackoverflow.com/help/mcve. --> ## The current behavior No warning appears for `arr` being potentially constructed on every render. ## The expected behavior Should warn that `arr` can be constructed on each render. The lint rules already warn for the following, so this is just an extension of the existing behavior: ```js const arr = props.arr ?? []; // The 'arr' logical expression could make the dependencies of useEffect Hook (at line 54) change on every render. Move it inside the useEffect callback. Alternatively, wrap the initialization of 'arr' in its own useMemo() Hook. ``` Reference: https://github.com/facebook/react/pull/19590