fix(rome_js_analyze): suppress false positive when returning call exp…

…ressions of a hook for useHookAtTopLevel (#4538)

* fix: false positive for useHookAtTopLevel

* docs: update changelog

CHANGELOG.md

@@ -29,10 +29,14 @@ was defined [#4479](https://github.com/rome/tools/issues/4479)
 
 ### Formatter
 
+### Linter
+
 #### Other changes
 
+- Fix false positive diagnostics ([#4483](https://github.com/rome/tools/issues/4483)) that [`useHookAtTopLevel`](https://docs.rome.tools/lint/rules/usehookattoplevel/) caused to returning call expressions of a hook.
 - Revert [#4359](https://github.com/rome/tools/issues/4359)
 
+
 ### Parser
 
 #### Other changes

crates/rome_js_analyze/src/semantic_analyzers/nursery/use_hook_at_top_level.rs

@@ -57,6 +57,7 @@ fn enclosing_function_if_call_is_at_top_level(call: &JsCallExpression) -> Option
                 | JsSyntaxKind::JS_BLOCK_STATEMENT
                 | JsSyntaxKind::JS_VARIABLE_STATEMENT
                 | JsSyntaxKind::JS_EXPRESSION_STATEMENT
+                | JsSyntaxKind::JS_RETURN_STATEMENT
                 | JsSyntaxKind::JS_CALL_EXPRESSION
                 | JsSyntaxKind::JS_INITIALIZER_CLAUSE
                 | JsSyntaxKind::JS_VARIABLE_DECLARATOR

crates/rome_js_analyze/tests/specs/nursery/useHookAtTopLevel/valid.js

@@ -31,3 +31,7 @@ export function Component4() {
 export default function Component5() {
     useEffect();
 };
+
+const Component6 = () => {
+    return useState();
+};

crates/rome_js_analyze/tests/specs/nursery/useHookAtTopLevel/valid.js.snap

@@ -38,6 +38,10 @@ export default function Component5() {
     useEffect();
 };
 
+const Component6 = () => {
+    return useState();
+};
+
 ```