fix(turbopack-ecmascript): use direct namespace re-export (#7963)

### Description

Also has a fix for async modules with `__turbopack_export_namespace__`.

Could still break if `__turbopack_export_namespace__` is called before
an `await`.


Closes PACK-2962

crates/turbopack-ecmascript-runtime/js/src/shared/runtime-utils.ts

@@ -147,7 +147,11 @@ function exportValue(module: Module, value: any) {
 }
 
 function exportNamespace(module: Module, namespace: any) {
- module.exports = module.namespaceObject = namespace;
+ if (isAsyncModuleExt(module.exports)) {
+ module.exports[turbopackExports] = namespace;
+ } else {
+ module.exports = module.namespaceObject = namespace;
+ }
 }
 
 function createGetter(obj: Record<string | symbol, any>, key: string | symbol) {
@@ -443,7 +447,7 @@ function asyncModule(
  if (err) {
  reject((promise[turbopackError] = err));
  } else {
- resolve(exports);
+ resolve(promise[turbopackExports]);
  }
 
  resolveQueue(queue);

crates/turbopack-ecmascript/src/references/external_module.rs

@@ -83,7 +83,7 @@ impl CachedExternalModule {
  if self.external_type == CachedExternalType::CommonJs {
  writeln!(code, "module.exports = mod;")?;
  } else {
- writeln!(code, "__turbopack_dynamic__(mod);")?;
+ writeln!(code, "__turbopack_export_namespace__(mod);")?;
  }
 
  Ok(EcmascriptModuleContent {

crates/turbopack-tests/tests/execution/turbopack/async-modules/esm-external/input/index.js

@@ -0,0 +1,8 @@
+import { a, b, default as def } from "esm-external/package";
+
+it(`should reexport all exports from an external esm module`, async () => {
+ expect(def).toBe("default");
+
+ expect(a).toBe("a");
+ expect(b).toBe("b");
+});