fix: avoid renaming the export name when outputting a library

crates/rspack_plugin_library/src/assign_library_plugin.rs

@@ -401,20 +401,14 @@ async fn finish_modules(&self, compilation: &mut Compilation) -> Result<()> {
  }
 
  for (runtime, export, module_identifier) in runtime_info {
+ let mut module_graph = compilation.get_module_graph_mut();
  if let Some(export) = export {
- let exports_info = compilation
- .get_module_graph_mut()
- .get_export_info(module_identifier, &(export.as_str()).into());
- exports_info.set_used(
- &mut compilation.get_module_graph_mut(),
- UsageState::Used,
- Some(&runtime),
- );
+ let export_info = module_graph.get_export_info(module_identifier, &(export.as_str()).into());
+ export_info.set_used(&mut module_graph, UsageState::Used, Some(&runtime));
+ export_info.set_can_mangle_use(&mut module_graph, Some(false));
  } else {
- let exports_info = compilation
- .get_module_graph()
- .get_exports_info(&module_identifier);
- exports_info.set_used_in_unknown_way(&mut compilation.get_module_graph_mut(), Some(&runtime));
+ let exports_info = module_graph.get_exports_info(&module_identifier);
+ exports_info.set_used_in_unknown_way(&mut module_graph, Some(&runtime));
  }
  }
  Ok(())

packages/rspack-test-tools/tests/configCases/output/rspack-issue-8045/index.js

@@ -0,0 +1,5 @@
+export default 1;
+
+it('should work', () => {
+ expect(window.MyLibrary).toEqual(1);
+})

packages/rspack-test-tools/tests/configCases/output/rspack-issue-8045/rspack.config.js

@@ -0,0 +1,10 @@
+/** @type {import("@rspack/core").Configuration} */
+module.exports = {
+ output: {
+ library: {
+ name: "MyLibrary",
+ export: "default",
+ type: "window"
+ },
+ }
+}

packages/rspack-test-tools/tests/configCases/output/rspack-issue-8045/test.config.js

@@ -0,0 +1,6 @@
+/** @type {import("../../../..").TConfigCaseConfig} */
+module.exports = {
+ moduleScope(scope) {
+ scope.window = {};
+ }
+};