chore: 🤖 update

web-infra-dev · Sep 19, 2023 · 464cfea · 464cfea
1 parent 3c6a439
commit 464cfea
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 54 deletions.
diff --git a/crates/rspack/tests/tree-shaking/bb/snapshot/new_treeshaking.snap b/crates/rspack/tests/tree-shaking/bb/snapshot/new_treeshaking.snap
@@ -6,10 +6,6 @@ source: crates/rspack_testing/src/run_fixture.rs
 "./a.js": function (__unused_webpack_module, __webpack_exports__, __webpack_require__) {
 'use strict';
 __webpack_require__.r(__webpack_exports__);
-/* harmony import */var _c_js__WEBPACK_IMPORTED_MODULE_0_ = __webpack_require__(/* ./c.js */"./c.js");
-__webpack_require__.es(_c_js__WEBPACK_IMPORTED_MODULE_0_, __webpack_exports__);
-/* harmony import */var _b_js__WEBPACK_IMPORTED_MODULE_1_ = __webpack_require__(Object(function webpackMissingModule() { var e = new Error('Cannot find module './b.js''); e.code = 'MODULE_NOT_FOUND'; throw e; }()));
-__webpack_require__.es(_b_js__WEBPACK_IMPORTED_MODULE_1_, __webpack_exports__);
 // import { d, c } from "./b.js";
 
  // export const a = 3;
@@ -28,7 +24,6 @@ __webpack_require__.d(__webpack_exports__, {
 "./index.js": function (__unused_webpack_module, __webpack_exports__, __webpack_require__) {
 'use strict';
 __webpack_require__.r(__webpack_exports__);
-/* harmony import */var _a_js__WEBPACK_IMPORTED_MODULE_0_ = __webpack_require__(/* ./a.js */"./a.js");
 
 _a_js__WEBPACK_IMPORTED_MODULE_0_.ccc;
 },

diff --git a/crates/rspack/tests/tree-shaking/bb/snapshot/new_treeshaking.snap.new b/crates/rspack/tests/tree-shaking/bb/snapshot/new_treeshaking.snap.new
diff --git a/crates/rspack_plugin_javascript/src/dependency/esm/harmony_import_dependency.rs b/crates/rspack_plugin_javascript/src/dependency/esm/harmony_import_dependency.rs
@@ -61,6 +61,14 @@ impl DependencyTemplate for HarmonyImportDependency {
     let compilation = &code_generatable_context.compilation;
     let module = &code_generatable_context.module;
 
+    let connection = compilation.module_graph.connection_by_dependency(&self.id);
+    if let Some(con) = connection {
+      // TODO: runtime opt
+      let active = con.is_target_active(&compilation.module_graph, None);
+      if !active {
+        return;
+      }
+    }
     let ref_mgm = compilation
       .module_graph
       .module_graph_module_by_dependency_id(&self.id)
@@ -174,14 +182,14 @@ impl DependencyTemplate for HarmonyImportDependency {
       )));
     }
     if self.export_all {
-      let connection = compilation.module_graph.connection_by_dependency(&self.id);
-      if let Some(con) = connection {
-        // TODO: runtime opt
-        let active = con.is_target_active(&compilation.module_graph, None);
-        if !active {
-          return;
-        }
-      }
+      // let connection = compilation.module_graph.connection_by_dependency(&self.id);
+      // if let Some(con) = connection {
+      //   // TODO: runtime opt
+      //   let active = con.is_target_active(&compilation.module_graph, None);
+      //   if !active {
+      //     return;
+      //   }
+      // }
       runtime_requirements.insert(RuntimeGlobals::EXPORT_STAR);
       let exports_argument = compilation
         .module_graph