Skip to content

Commit 978e642

Browse files
authored
fix: should not lazy-barrel import-then-export (#11550)
1 parent 0f6f3d5 commit 978e642

File tree

2 files changed

+2
-10
lines changed

2 files changed

+2
-10
lines changed

crates/rspack_plugin_javascript/src/parser_plugin/esm_import_dependency_parser_plugin.rs

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ impl JavascriptParserPlugin for ESMImportDependencyParserPlugin {
7878
) -> Option<bool> {
7979
parser.last_esm_import_order += 1;
8080
let attributes = import_decl.with.as_ref().map(|obj| get_attributes(obj));
81-
let mut dependency = ESMImportSideEffectDependency::new(
81+
let dependency = ESMImportSideEffectDependency::new(
8282
source.into(),
8383
parser.last_esm_import_order,
8484
import_decl.span.into(),
@@ -88,14 +88,7 @@ impl JavascriptParserPlugin for ESMImportDependencyParserPlugin {
8888
Some(parser.source_map.clone()),
8989
false,
9090
);
91-
if parser.compiler_options.experiments.lazy_barrel
92-
&& parser
93-
.factory_meta
94-
.and_then(|meta| meta.side_effect_free)
95-
.unwrap_or_default()
96-
{
97-
dependency.set_lazy();
98-
}
91+
9992
parser.dependencies.push(Box::new(dependency));
10093

10194
parser

packages/rspack-test-tools/tests/configCases/lazy-barrel/basic/rspack.config.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@ const allModules = fs
1515
const lazyModules = new Set(
1616
[
1717
"named-barrel/b.js",
18-
"named-barrel/d.js",
1918
"mixed-barrel/a.js",
2019
"mixed-barrel/b.js",
2120
"star-barrel/c.js",

0 commit comments

Comments
 (0)