fix: inline value with export as (#11687)

Author: ahabhgk

crates/rspack_plugin_javascript/src/parser_plugin/esm_export_dependency_parser_plugin.rs

@@ -122,14 +122,14 @@ impl JavascriptParserPlugin for ESMExportDependencyParserPlugin {
       Box::new(dep) as BoxDependency
     } else {
       let inlinable = parser
-        .get_tag_data(export_name, INLINABLE_CONST_TAG)
+        .get_tag_data(local_id, INLINABLE_CONST_TAG)
         .map(InlinableConstData::downcast)
         .map(|data| data.value);
       let enum_value = parser
         .build_info
         .collected_typescript_info
         .as_ref()
-        .and_then(|info| info.exported_enums.get(export_name).cloned());
+        .and_then(|info| info.exported_enums.get(local_id).cloned());
       if enum_value.is_some() && !parser.compiler_options.experiments.inline_enum {
         parser.add_error(rspack_error::error!("inlineEnum is still an experimental feature. To continue using it, please enable 'experiments.inlineEnum'.").into());
       }

tests/rspack-test/configCases/inline-const/basic/constants.no-inline.js

@@ -0,0 +1,3 @@
+const INLINE_1 = 1;
+const NOINLINE_1 = {};
+export { NOINLINE_1 as INLINE_1 }

tests/rspack-test/configCases/inline-const/basic/index.js

@@ -6,6 +6,7 @@ import * as reexportedSideEffects from "./re-export.side-effects.js";
 import * as reexportedBarrelSideEffects from "./re-export.barrel-side-effects.js";
 import * as reexportedDestructingBarrelSideEffects from "./re-export.destructing-barrel-side-effects.js";
 import * as constantsCjs from "./constants.cjs";
+import * as constantsNoInline from "./constants.no-inline.js";
 
 const generated = /** @type {string} */ (__non_webpack_require__("fs").readFileSync(__filename, "utf-8"));
 
@@ -175,3 +176,7 @@ it("should keep the module if part of the exports is inlined and side effects fr
     })
   }
 })
+
+it("should not inline no-inlinable constants", () => {
+  expect(constantsNoInline.INLINE_1).toEqual({});
+})

tests/rspack-test/configCases/inline-enum/basic/enum.export-as.ts

@@ -0,0 +1,10 @@
+enum InlineE1 {
+  A,
+}
+
+const InlineE = {
+  A: {},
+}
+
+export { InlineE1 as InlineE }
+export { InlineE as InlineE1 }

tests/rspack-test/configCases/inline-enum/basic/index.js

@@ -4,6 +4,7 @@ import * as destructing from "./enum.destructing";
 import * as sideEffects from "./enum.side-effects";
 import * as reexportedSideEffects from "./re-export.side-effects";
 import * as notOnlyPropertiesUsed from "./enum.not-only-properties-used";
+import * as exportAs from "./enum.export-as";
 
 const generated = /** @type {string} */ (__non_webpack_require__("fs").readFileSync(__filename, "utf-8"));
 
@@ -117,3 +118,8 @@ it("should keep the module if part of the enum members are inlined and side effe
     })
   }
 })
+
+it("should not inline no-inlinable enums", () => {
+  expect(exportAs.InlineE1.A).toEqual({});
+  expect(exportAs.InlineE.A).toBe(0);
+})