fix(loadable-components): honor local name for source-less default imports (#585)

Author: kdy1

packages/loadable-components/src/lib.rs

@@ -682,7 +682,17 @@ where
             for specifier in import_decl.specifiers.iter() {
                 match specifier {
                     ImportSpecifier::Default(default_spec) => {
-                        if signature.is_default_specifier() {
+                        // For source-less signatures (`from: None`), default imports
+                        // must match the configured local identifier to avoid
+                        // over-matching unrelated default imports.
+                        let is_match = if signature.from.is_none() {
+                            default_spec.local.sym == signature.name
+                        } else {
+                            signature.is_default_specifier()
+                                || default_spec.local.sym == signature.name
+                        };
+
+                        if is_match {
                             self.specifiers.insert(default_spec.local.sym.clone());
                         }
                     }

packages/loadable-components/tests/fixture.rs

@@ -68,7 +68,7 @@ fn fixture_aliased_import(input: PathBuf) {
                 t.comments.clone(),
                 vec![
                     Signature {
-                        name: "default".into(),
+                        name: "loadable".into(),
                         from: None,
                     },
                 ],

packages/loadable-components/tests/fixture/aliased import/should not transform other default imports/input.js

@@ -0,0 +1,3 @@
+import foo from 'my-vendored-loadable';
+
+foo(() => import('./SomeComponent'));

packages/loadable-components/tests/fixture/aliased import/should not transform other default imports/output.js

@@ -0,0 +1,3 @@
+import foo from 'my-vendored-loadable';
+
+foo(() => import('./SomeComponent'));