fix(es/module): Do not determine filename if no exports

swc-project · Jul 27, 2023 · 75e8214 · 75e8214
1 parent 9daa0e2
commit 75e8214
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/crates/swc_ecma_transforms_module/src/umd.rs b/crates/swc_ecma_transforms_module/src/umd.rs
@@ -110,9 +110,6 @@ where
     fn visit_mut_module(&mut self, module: &mut Module) {
         let import_interop = self.config.config.import_interop();
 
-        let filename = self.cm.span_to_filename(module.span);
-        let exported_name = self.config.determine_export_name(filename);
-
         let module_items = &mut module.body;
 
         let mut strip = ModuleDeclStrip::new(self.const_var_kind);
@@ -170,7 +167,7 @@ where
         //  Emit
         // ====================
 
-        let (adapter_fn_expr, factory_params) = self.adapter(exported_name, is_export_assign);
+        let (adapter_fn_expr, factory_params) = self.adapter(module.span, is_export_assign);
 
         let factory_fn_expr: Expr = Function {
             params: factory_params,
@@ -341,7 +338,7 @@ where
     /// });
     /// ```
     /// Return: adapter expr and factory params
-    fn adapter(&mut self, exported_name: Ident, is_export_assign: bool) -> (FnExpr, Vec<Param>) {
+    fn adapter(&mut self, module_span: Span, is_export_assign: bool) -> (FnExpr, Vec<Param>) {
         macro_rules! js_typeof {
             ($test:expr =>! $type:expr) => {
                 Expr::Unary(UnaryExpr {
@@ -375,7 +372,6 @@ where
         let factory = private_ident!("factory");
 
         let module_exports = module.clone().make_member(quote_ident!("exports"));
-        let global_lib = global.clone().make_member(exported_name);
         let define_amd = define.clone().make_member(quote_ident!("amd"));
 
         let mut cjs_args = vec![];
@@ -385,6 +381,10 @@ where
         let mut factory_params = vec![];
 
         if !is_export_assign && self.exports.is_some() {
+            let filename = self.cm.span_to_filename(module_span);
+            let exported_name = self.config.determine_export_name(filename);
+            let global_lib = global.clone().make_member(exported_name);
+
             cjs_args.push(quote_ident!("exports").as_arg());
             amd_dep_list.push(Some(quote_str!("exports").as_arg()));
             browser_args.push(