Don't include the default export in static reexports (parcel-bundler#…

…6701)

packages/core/integration-tests/test/integration/scope-hoisting/es6/no-reexport-default/a.js

@@ -0,0 +1,5 @@
+export {default} from './b.js';
+export * from './b.js';
+
+export {default as other} from './c.js';
+export * from './c.js';

packages/core/integration-tests/test/integration/scope-hoisting/es6/no-reexport-default/async.js

@@ -0,0 +1,5 @@
+import _default, {other} from './a.js';
+
+sideEffectNoop(_default, other);
+
+export default _default;

packages/core/integration-tests/test/integration/scope-hoisting/es6/no-reexport-default/b.js

@@ -0,0 +1 @@
+export default 42;

packages/core/integration-tests/test/integration/scope-hoisting/es6/no-reexport-default/c.js

@@ -0,0 +1 @@
+export default 99;

packages/core/integration-tests/test/integration/scope-hoisting/es6/no-reexport-default/index.js

@@ -0,0 +1,3 @@
+import _default, {other} from './a.js';
+
+output = import('./async').then(mod => mod.default);

packages/core/integration-tests/test/integration/scope-hoisting/es6/no-reexport-esmodule/a.js

@@ -0,0 +1,2 @@
+module.exports.foo = 42;
+module.exports.__esModule = true;

packages/core/integration-tests/test/integration/scope-hoisting/es6/no-reexport-esmodule/async.js

@@ -0,0 +1 @@
+export * from './a';

packages/core/integration-tests/test/integration/scope-hoisting/es6/no-reexport-esmodule/index.js

@@ -0,0 +1,3 @@
+import * as a from './a.js';
+
+output = import('./async').then(mod => mod.__esModule);

packages/core/integration-tests/test/scope-hoisting.js

@@ -3714,6 +3714,28 @@ describe('scope hoisting', function() {
       let test = await run(b);
       assert.equal(test({foo: 2}), 2);
     });
+
+    it('should not include default when reexporting * without $parcel$exportWildcard', async () => {
+      let b = await bundle(
+        path.join(
+          __dirname,
+          'integration/scope-hoisting/es6/no-reexport-default/index.js',
+        ),
+      );
+
+      assert.equal(await run(b), 42);
+    });
+
+    it('should not include __esModule when reexporting * without $parcel$exportWildcard', async () => {
+      let b = await bundle(
+        path.join(
+          __dirname,
+          'integration/scope-hoisting/es6/no-reexport-esmodule/index.js',
+        ),
+      );
+
+      assert.equal(await run(b), undefined);
+    });
   });
 
   describe('commonjs', function() {

packages/packagers/js/src/ScopeHoistingPackager.js

@@ -983,6 +983,13 @@ ${code}
               this.usedHelpers.add('$parcel$exportWildcard');
             } else {
               for (let symbol of this.bundleGraph.getUsedSymbols(dep)) {
+                if (
+                  symbol === 'default' || // `export * as ...` does not include the default export
+                  symbol === '__esModule'
+                ) {
+                  continue;
+                }
+
                 let resolvedSymbol = this.getSymbolResolution(
                   asset,
                   resolved,