fix: some cleanujp

Author: Princesseuh

crates/oxc_ast/src/ast/mod.rs

@@ -182,13 +182,15 @@ pub use oxc_syntax::{
     },
 };
 
-pub(crate) mod astro;
 pub(crate) mod comment;
 pub(crate) mod js;
 pub(crate) mod jsx;
 pub(crate) mod literal;
 pub(crate) mod macros;
 pub(crate) mod ts;
+// Astro module is placed last so its AST types get higher discriminants,
+// avoiding changing the discriminants of existing JS/TS types.
+pub(crate) mod astro;
 
 pub use astro::*;
 pub use comment::*;

crates/oxc_ast/src/generated/assert_layouts.rs

@@ -34,7 +34,7 @@ const _: () = {
     assert!(offset_of!(AstroDoctype, span) == 0);
     assert!(offset_of!(AstroDoctype, value) == 8);
 
-    // Padding: 1 bytes
+    // Padding: 0 bytes
     assert!(size_of::<Program>() == 128);
     assert!(align_of::<Program>() == 8);
     assert!(offset_of!(Program, span) == 0);
@@ -1681,7 +1681,7 @@ const _: () = if cfg!(target_family = "wasm") || align_of::<u64>() == 8 {
     assert!(offset_of!(AstroDoctype, span) == 0);
     assert!(offset_of!(AstroDoctype, value) == 8);
 
-    // Padding: 1 bytes
+    // Padding: 0 bytes
     assert!(size_of::<Program>() == 88);
     assert!(align_of::<Program>() == 4);
     assert!(offset_of!(Program, span) == 0);

crates/oxc_codegen/tests/integration/snapshots/stacktrace_is_correct.snap

@@ -1,7 +1,7 @@
 ---
 source: crates/oxc_codegen/tests/integration/sourcemap.rs
 ---
-Node.js version: v25.3.0
+Node.js version: v24.12.0
 
 ## Input
 const fn = () => {

crates/oxc_linter/src/rules/eslint/no_unused_vars/mod.rs

@@ -222,7 +222,6 @@ impl Rule for NoUnusedVars {
         // 1. declarations have side effects (they get merged together)
         // 2. vue/svelte scripts declare variables that get used in the template, which
         //    we can't detect
-        // Note: Astro files ARE supported - semantic analysis tracks template usage
         !ctx.source_type().is_typescript_definition()
             && !ctx.file_extension().is_some_and(|ext| ext == "vue" || ext == "svelte")
     }

crates/oxc_linter/src/rules/react/no_unescaped_entities.rs

@@ -137,9 +137,6 @@ fn test() {
           },
         });
         ",
-        // Script tags should not trigger unescaped entity errors - their content is JS, not HTML
-        r#"<script>window.foo = "bar"</script>"#,
-        r#"<script>测试 " 测试</script>"#,
     ];
 
     let fail = vec![
@@ -171,10 +168,17 @@ fn test() {
         "#,
         // "var Hello = createReactClass({
         //     render: function() {
+        //       return <>foo & bar</>;
+        //     }
+        //   });",
+        // "        var Hello = createReactClass({
+        //     render: function() {
         //       return <span>foo & bar</span>;
         //     }
         //   });
         // ",
+        r#"<script>window.foo = "bar"</script>"#,
+        r#"<script>测试 " 测试</script>"#,
     ];
 
     Tester::new(NoUnescapedEntities::NAME, NoUnescapedEntities::PLUGIN, pass, fail)

crates/oxc_linter/src/snapshots/react_no_unescaped_entities.snap

@@ -1,6 +1,5 @@
 ---
 source: crates/oxc_linter/src/tester.rs
-assertion_line: 444
 ---
 
   × Unexpected token. Did you mean `{'>'}` or `>`?
@@ -34,3 +33,21 @@ assertion_line: 444
    ·                                                            ▲
  5 │             }
    ╰────
+
+  ⚠ eslint-plugin-react(no-unescaped-entities): `"` can be escaped with " or “ or " or ”
+   ╭─[no_unescaped_entities.tsx:1:22]
+ 1 │ <script>window.foo = "bar"</script>
+   ·                      ─
+   ╰────
+
+  ⚠ eslint-plugin-react(no-unescaped-entities): `"` can be escaped with &quot; or &ldquo; or &#34; or &rdquo;
+   ╭─[no_unescaped_entities.tsx:1:26]
+ 1 │ <script>window.foo = "bar"</script>
+   ·                          ─
+   ╰────
+
+  ⚠ eslint-plugin-react(no-unescaped-entities): `"` can be escaped with &quot; or &ldquo; or &#34; or &rdquo;
+   ╭─[no_unescaped_entities.tsx:1:16]
+ 1 │ <script>测试 " 测试</script>
+   ·              ─
+   ╰────