fix: improve diagnostics

Author: taga3s

crates/biome_js_analyze/src/lint/nursery/use_inline_script_id.rs

@@ -150,11 +150,11 @@ impl Rule for UseInlineScriptId {
                 rule_category!(),
                 state,
                 markup! {
-                    ""<Emphasis>"next/script"</Emphasis>" components with inline content or `dangerouslySetInnerHTML` must specify "<Emphasis>"id"</Emphasis>" attribute."
+                    ""<Emphasis>"next/script"</Emphasis>" components have inline content or `dangerouslySetInnerHTML` without "<Emphasis>"id"</Emphasis>" attribute."
                 },
             )
             .note(markup!(
-                "Without id attribute, Next.js cannot correctly track inline scripts and this can cause performance issues."
+                "Next.js requires "<Emphasis>"id"</Emphasis>" attribute to track and optimize inline scripts. Without it, performance issues may occur."
             ))
             .note(markup! {
                 "See the "<Hyperlink href="https://nextjs.org/docs/messages/inline-script-id">"Next.js docs"</Hyperlink>" for more details."

crates/biome_js_analyze/tests/specs/nursery/useInlineScriptId/invalid-01.jsx.snap

@@ -19,7 +19,7 @@ export default function Page() {
 ```
 invalid-01.jsx:6:5 lint/nursery/useInlineScriptId ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 
-  × next/script components with inline content or `dangerouslySetInnerHTML` must specify id attribute.
+  × next/script components have inline content or `dangerouslySetInnerHTML` without id attribute.
   
     4 │ export default function Page() {
     5 │   return (
@@ -28,7 +28,7 @@ invalid-01.jsx:6:5 lint/nursery/useInlineScriptId ━━━━━━━━━━
     7 │   )
     8 │ }
   
-  i Without id attribute, Next.js cannot correctly track inline scripts and this can cause performance issues.
+  i Next.js requires id attribute to track and optimize inline scripts. Without it, performance issues may occur.
   
   i See the Next.js docs for more details.
   

crates/biome_js_analyze/tests/specs/nursery/useInlineScriptId/invalid-02.jsx.snap

@@ -19,7 +19,7 @@ export default function Page() {
 ```
 invalid-02.jsx:6:5 lint/nursery/useInlineScriptId ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 
-  × next/script components with inline content or `dangerouslySetInnerHTML` must specify id attribute.
+  × next/script components have inline content or `dangerouslySetInnerHTML` without id attribute.
   
     4 │ export default function Page() {
     5 │   return (
@@ -28,7 +28,7 @@ invalid-02.jsx:6:5 lint/nursery/useInlineScriptId ━━━━━━━━━━
     7 │   )
     8 │ }
   
-  i Without id attribute, Next.js cannot correctly track inline scripts and this can cause performance issues.
+  i Next.js requires id attribute to track and optimize inline scripts. Without it, performance issues may occur.
   
   i See the Next.js docs for more details.
   

crates/biome_js_analyze/tests/specs/nursery/useInlineScriptId/invalid-03.jsx.snap

@@ -20,7 +20,7 @@ export default function Page() {
 ```
 invalid-03.jsx:7:5 lint/nursery/useInlineScriptId ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 
-  × next/script components with inline content or `dangerouslySetInnerHTML` must specify id attribute.
+  × next/script components have inline content or `dangerouslySetInnerHTML` without id attribute.
   
     5 │   const props = { strategy: "beforeInteractive" };
     6 │   return (
@@ -29,7 +29,7 @@ invalid-03.jsx:7:5 lint/nursery/useInlineScriptId ━━━━━━━━━━
     8 │   )
     9 │ }
   
-  i Without id attribute, Next.js cannot correctly track inline scripts and this can cause performance issues.
+  i Next.js requires id attribute to track and optimize inline scripts. Without it, performance issues may occur.
   
   i See the Next.js docs for more details.
   

crates/biome_js_analyze/tests/specs/nursery/useInlineScriptId/invalid-04.jsx.snap

@@ -19,7 +19,7 @@ export default function Page() {
 ```
 invalid-04.jsx:6:5 lint/nursery/useInlineScriptId ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 
-  × next/script components with inline content or `dangerouslySetInnerHTML` must specify id attribute.
+  × next/script components have inline content or `dangerouslySetInnerHTML` without id attribute.
   
     4 │ export default function Page() {
     5 │   return (
@@ -28,7 +28,7 @@ invalid-04.jsx:6:5 lint/nursery/useInlineScriptId ━━━━━━━━━━
     7 │   )
     8 │ }
   
-  i Without id attribute, Next.js cannot correctly track inline scripts and this can cause performance issues.
+  i Next.js requires id attribute to track and optimize inline scripts. Without it, performance issues may occur.
   
   i See the Next.js docs for more details.