refactor(ast): move ModuleRecord to oxc_syntax

crates/oxc_ast/src/lib.rs

@@ -15,7 +15,6 @@ mod serialize;
 pub mod ast;
 mod ast_builder;
 mod ast_kind;
-pub mod module_record;
 mod span;
 pub mod syntax_directed_operations;
 mod trivia;

crates/oxc_semantic/src/builder.rs

@@ -3,9 +3,10 @@
 use std::{cell::RefCell, rc::Rc};
 
 #[allow(clippy::wildcard_imports)]
-use oxc_ast::{ast::*, module_record::ModuleRecord, AstKind, Trivias, Visit};
+use oxc_ast::{ast::*, AstKind, Trivias, Visit};
 use oxc_diagnostics::Error;
 use oxc_span::{Atom, SourceType, Span};
+use oxc_syntax::module_record::ModuleRecord;
 use rustc_hash::FxHashMap;
 
 use crate::{

crates/oxc_semantic/src/checker/javascript.rs

@@ -1,7 +1,6 @@
 #[allow(clippy::wildcard_imports)]
 use oxc_ast::{
     ast::*,
-    module_record::ExportLocalName,
     syntax_directed_operations::{BoundNames, IsSimpleParameterList, PropName},
     AstKind,
 };
@@ -11,6 +10,7 @@ use oxc_diagnostics::{
 };
 use oxc_span::{Atom, GetSpan, ModuleKind, Span};
 use oxc_syntax::{
+    module_record::ExportLocalName,
     operator::{BinaryOperator, LogicalOperator, UnaryOperator},
     NumberBase,
 };

crates/oxc_semantic/src/lib.rs

@@ -15,9 +15,10 @@ use std::rc::Rc;
 
 pub use builder::{SemanticBuilder, SemanticBuilderReturn};
 pub use jsdoc::{JSDoc, JSDocComment, JSDocTag};
-use oxc_ast::{ast::IdentifierReference, module_record::ModuleRecord, AstKind, Trivias};
+use oxc_ast::{ast::IdentifierReference, AstKind, Trivias};
 use oxc_span::SourceType;
 pub use oxc_syntax::{
+    module_record::ModuleRecord,
     scope::{ScopeFlags, ScopeId},
     symbol::{SymbolFlags, SymbolId},
 };

crates/oxc_semantic/src/module_record/builder.rs

@@ -1,6 +1,8 @@
 #[allow(clippy::wildcard_imports)]
-use oxc_ast::{ast::*, module_record::*, syntax_directed_operations::BoundNames};
+use oxc_ast::{ast::*, syntax_directed_operations::BoundNames};
 use oxc_span::{Atom, GetSpan, Span};
+#[allow(clippy::wildcard_imports)]
+use oxc_syntax::module_record::*;
 
 #[derive(Debug, Default)]
 pub struct ModuleRecordBuilder {

crates/oxc_semantic/src/module_record/mod.rs

@@ -5,10 +5,10 @@ pub use builder::ModuleRecordBuilder;
 #[cfg(test)]
 mod module_record_tests {
     use oxc_allocator::Allocator;
-    #[allow(clippy::wildcard_imports)]
-    use oxc_ast::module_record::*;
     use oxc_parser::Parser;
     use oxc_span::{SourceType, Span};
+    #[allow(clippy::wildcard_imports)]
+    use oxc_syntax::module_record::*;
 
     use crate::SemanticBuilder;
 

crates/oxc_syntax/Cargo.toml

@@ -15,7 +15,10 @@ default = []
 serde   = ["dep:serde"]
 
 [dependencies]
+oxc_index        = { workspace = true }
+oxc_span         = { workspace = true }
+
 unicode-id-start = { workspace = true }
 serde            = { workspace = true, features = ["derive"], optional = true }
-oxc_index        = { workspace = true }
 bitflags         = { workspace = true }
+rustc-hash       = { workspace = true }

crates/oxc_syntax/src/lib.rs

@@ -1,6 +1,7 @@
 //! Common code for JavaScript Syntax
 
 pub mod identifier;
+pub mod module_record;
 pub mod operator;
 pub mod precedence;
 pub mod scope;