fix(rome_js_analyze): noRedundantUseStrict check only 'use strict' di…

…rective (#4591)

* fix: noRedundantUseStrict check only 'use strict' directive

* docs: update changelog

* docs: update

CHANGELOG.md

@@ -117,6 +117,11 @@ parameter decorators:
   }
   ```
 
+- [noRedundantUseStrict](https://docs.rome.tools/lint/rules/noredundantusestrict/) check only `'use strict'` directive to resolve false positive diagnostics.
+
+  React introduce new directives, "use client" and "use server".
+  The rule raises false positive errors about these directives.
+
 ### Parser
 
 ### VSCode

crates/rome_js_analyze/src/analyzers/suspicious/no_redundant_use_strict.rs

@@ -91,6 +91,9 @@ impl Rule for NoRedundantUseStrict {
 
     fn run(ctx: &RuleContext<Self>) -> Self::Signals {
         let node = ctx.query();
+        if node.inner_string_text().ok()? != "use strict" {
+            return None;
+        }
         let mut outer_most: Option<AnyJsStrictModeNode> = None;
         let root = ctx.root();
         match root {

crates/rome_js_analyze/tests/specs/suspicious/noRedundantUseStrict/validReactDirectives.tsx

@@ -0,0 +1,14 @@
+'use client'
+
+import { useState } from "react"
+
+export default function Counter() {
+  const [count, setCount] = useState(0)
+
+  return (
+    <div>
+      <p>You clicked {count} times</p>
+      <button type="button" onClick={() => setCount(count + 1)}>Click me</button>
+    </div>
+  )
+}

crates/rome_js_analyze/tests/specs/suspicious/noRedundantUseStrict/validReactDirectives.tsx.snap

@@ -0,0 +1,24 @@
+---
+source: crates/rome_js_analyze/tests/spec_tests.rs
+expression: validReactDirectives.tsx
+---
+# Input
+```js
+'use client'
+
+import { useState } from "react"
+
+export default function Counter() {
+  const [count, setCount] = useState(0)
+
+  return (
+    <div>
+      <p>You clicked {count} times</p>
+      <button type="button" onClick={() => setCount(count + 1)}>Click me</button>
+    </div>
+  )
+}
+
+```
+
+