fix(bundler): Don't load dynamic imports (#1297)

swc_bundler: - Do not load dynamically imported files.
swc-project · Dec 28, 2020 · bc7ac45 · bc7ac45
1 parent ba13db5
commit bc7ac45
Show file tree

Hide file tree

Showing 6 changed files with 23 additions and 11 deletions.
diff --git a/bundler/Cargo.toml b/bundler/Cargo.toml
@@ -8,7 +8,7 @@ edition = "2018"
 license = "Apache-2.0/MIT"
 name = "swc_bundler"
 repository = "https://github.com/swc-project/swc.git"
-version = "0.18.1"
+version = "0.18.2"
 
 # See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
 [features]

diff --git a/bundler/src/bundler/import/mod.rs b/bundler/src/bundler/import/mod.rs
@@ -516,16 +516,19 @@ where
                         return Expr::Call(e);
                     }
 
-                    ExprOrSuper::Expr(ref e) => match &**e {
-                        Expr::Ident(Ident {
-                            sym: js_word!("import"),
-                            ..
-                        }) => {
-                            self.info.dynamic_imports.push(src.clone());
-                        }
-                        _ => {}
-                    },
-
+                    // TODO: Uncomment this after implementing an option to make swc_bundler
+                    // includes dynamic imports
+                    //
+                    //
+                    // ExprOrSuper::Expr(ref e) => match &**e {
+                    //     Expr::Ident(Ident {
+                    //         sym: js_word!("import"),
+                    //         ..
+                    //     }) => {
+                    //         self.info.dynamic_imports.push(src.clone());
+                    //     }
+                    //     _ => {}
+                    // },
                     _ => {}
                 }
 

diff --git a/bundler/tests/common/mod.rs b/bundler/tests/common/mod.rs
@@ -76,6 +76,7 @@ impl Load for Loader {
             Syntax::Typescript(TsConfig {
                 decorators: true,
                 tsx,
+                dynamic_import: true,
                 ..Default::default()
             }),
             JscTarget::Es2020,

diff --git a/bundler/tests/fixture/dynamic/input/dep.ts b/bundler/tests/fixture/dynamic/input/dep.ts
@@ -0,0 +1,3 @@
+console.log('Foo')
+
+export const a = 5;
diff --git a/bundler/tests/fixture/dynamic/input/entry.ts b/bundler/tests/fixture/dynamic/input/entry.ts
@@ -0,0 +1,3 @@
+const a = import('./dep')
+
+console.log(a)
diff --git a/bundler/tests/fixture/dynamic/output/entry.ts b/bundler/tests/fixture/dynamic/output/entry.ts
@@ -0,0 +1,2 @@
+const a = import('./dep');
+console.log(a);