Properly visit member expressions (#7228)

packages/core/integration-tests/test/integration/js-import-member/foo.js

@@ -0,0 +1,6 @@
+export const foo = {
+	a: "b",
+  bar: "bar"
+};
+
+export const bar = "a";

packages/core/integration-tests/test/integration/js-import-member/index.js

@@ -0,0 +1,3 @@
+import { foo, bar } from "./foo.js";
+
+export default [bar, foo[bar], foo.bar];

packages/core/integration-tests/test/javascript.js

@@ -4982,6 +4982,14 @@ describe('javascript', function () {
     assert.deepEqual(res.default, 123);
   });
 
+  it('should replace imported values in member expressions', async function () {
+    let b = await bundle(
+      path.join(__dirname, 'integration/js-import-member/index.js'),
+    );
+    let res = await run(b);
+    assert.deepEqual(res.default, ['a', 'b', 'bar']);
+  });
+
   it('should not freeze live default imports', async function () {
     let b = await bundle(
       path.join(__dirname, 'integration/js-import-default-live/index.js'),

packages/transformers/js/core/src/utils.rs

@@ -317,11 +317,10 @@ macro_rules! fold_member_expr_skip_prop {
       &mut self,
       mut node: swc_ecmascript::ast::MemberExpr,
     ) -> swc_ecmascript::ast::MemberExpr {
-      node.obj = node.obj.fold_children_with(self);
+      node.obj = node.obj.fold_with(self);
 
-      // To ensure that fold_expr doesn't replace `require` in non-computed member expressions
       if node.computed {
-        node.prop = node.prop.fold_children_with(self);
+        node.prop = node.prop.fold_with(self);
       }
 
       node