Fix new ESM loader runtime by moving to abs URLs

packages/core/core/src/applyRuntimes.js

@@ -6,7 +6,6 @@ import type {SharedReference} from '@parcel/workers';
 import type {
   Asset,
   AssetGroup,
-  Bundle,
   Bundle as InternalBundle,
   Config,
   DevDepRequest,
@@ -41,6 +40,23 @@ type RuntimeConnection = {|
   isEntry: ?boolean,
 |};
 
+function nameRuntimeBundle(
+  bundle: InternalBundle,
+  siblingBundle: InternalBundle,
+) {
+  // We don't run custom namers on runtime bundles as the runtime assumes that they are
+  // located at the same nesting level as they're owning bundle. Custom naming could
+  // be added in future as long as the custom name is validated.
+  let {hashReference} = bundle;
+
+  let name = siblingBundle.name
+    .replace(`.${bundle.type}`, `.runtime.${hashReference}.${bundle.type}`)
+    .replace(siblingBundle.hashReference, 'sib-hash');
+
+  bundle.name = name;
+  bundle.displayName = name.replace(hashReference, '[hash]');
+}
+
 export default async function applyRuntimes<TResult>({
   bundleGraph,
   config,
@@ -51,7 +67,6 @@ export default async function applyRuntimes<TResult>({
   previousDevDeps,
   devDepRequests,
   configs,
-  nameRuntimeBundle,
 }: {|
   bundleGraph: InternalBundleGraph,
   config: ParcelConfig,
@@ -62,7 +77,6 @@ export default async function applyRuntimes<TResult>({
   previousDevDeps: Map<string, string>,
   devDepRequests: Map<string, DevDepRequest>,
   configs: Map<string, Config>,
-  nameRuntimeBundle: (bundle: Bundle) => Promise<void>,
 |}): Promise<Map<string, Asset>> {
   let runtimes = await config.getRuntimes();
   let connections: Array<RuntimeConnection> = [];
@@ -152,7 +166,7 @@ export default async function applyRuntimes<TResult>({
               }
               bundleGraph.createBundleReference(bundle, connectionBundle);
 
-              await nameRuntimeBundle(connectionBundle);
+              nameRuntimeBundle(connectionBundle, bundle);
             }
 
             connections.push({

packages/core/core/src/requests/BundleGraphRequest.js

@@ -411,8 +411,6 @@ class BundlerRunner {
         previousDevDeps: this.previousDevDeps,
         devDepRequests: this.devDepRequests,
         configs: this.configs,
-        nameRuntimeBundle: bundle =>
-          this.nameBundle(namers, bundle, internalBundleGraph),
       });
 
       // Add dev deps for namers, AFTER running them to account for lazy require().

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

@@ -2095,7 +2095,10 @@ describe('javascript', function () {
   it('should create a shared bundle between browser and worker contexts', async () => {
     let b = await bundle(
       path.join(__dirname, '/integration/html-shared-worker/index.html'),
-      {mode: 'production', defaultTargetOptions: {shouldScopeHoist: false}},
+      {
+        mode: 'production',
+        defaultTargetOptions: {shouldScopeHoist: false, shouldOptimize: false},
+      },
     );
 
     assertBundles(b, [
@@ -2109,11 +2112,10 @@ describe('javascript', function () {
           'get-worker-url.js',
           'lodash.js',
           'esmodule-helpers.js',
-          'bundle-url.js',
         ],
       },
       {
-        assets: ['bundle-manifest.js'],
+        assets: ['bundle-manifest.js', 'bundle-url.js'],
       },
       {
         assets: ['worker.js', 'lodash.js', 'esmodule-helpers.js'],

packages/core/test-utils/src/utils.js

@@ -968,7 +968,10 @@ export async function runESM(
 ): Promise<Array<{|[string]: mixed|}>> {
   let id = instanceId++;
   let cache = new Map();
-  function load(specifier, referrer, code = null) {
+  function load(inputSpecifier, referrer, code = null) {
+    // ESM can request bundles with an absolute URL. Normalize this to the baseDir.
+    let specifier = inputSpecifier.replace('http://localhost', baseDir);
+
     if (path.isAbsolute(specifier) || specifier.startsWith('.')) {
       let extname = path.extname(specifier);
       if (extname && extname !== '.js' && extname !== '.mjs') {

packages/runtimes/js/src/JSRuntime.js

@@ -390,17 +390,7 @@ function getLoaderRuntime({
       !needsDynamicImportPolyfill &&
       shouldUseRuntimeManifest(bundle, options)
     ) {
-      let params = [JSON.stringify(to.publicId)];
-
-      let relativeBase = getRelativeBasePath(
-        relativeBundlePath(bundle, to, {leadingDotSlash: false}),
-      );
-
-      if (relativeBase) {
-        params.push(relativeBase);
-      }
-
-      loaderModules.push(`load(${params.join(',')})`);
+      loaderModules.push(`load(${JSON.stringify(to.publicId)})`);
       needsEsmLoadPrelude = true;
       continue;
     }
@@ -425,10 +415,12 @@ function getLoaderRuntime({
       continue;
     }
 
-    let code = `require(${JSON.stringify(loader)})(${getAbsoluteUrlExpr(
-      relativePathExpr,
-      bundle,
-    )})`;
+    let absoluteUrlExpr = shouldUseRuntimeManifest(bundle, options)
+      ? `require('./helpers/bundle-manifest').resolve(${JSON.stringify(
+          to.publicId,
+        )})`
+      : getAbsoluteUrlExpr(relativePathExpr, bundle);
+    let code = `require(${JSON.stringify(loader)})(${absoluteUrlExpr})`;
 
     // In development, clear the require cache when an error occurs so the
     // user can try again (e.g. after fixing a build error).
@@ -632,39 +624,42 @@ function getRegisterCode(
   entryBundle: NamedBundle,
   bundleGraph: BundleGraph<NamedBundle>,
 ): string {
-  let idToName = {};
+  let idToName = [];
   bundleGraph.traverseBundles((bundle, _, actions) => {
     if (bundle.bundleBehavior === 'inline') {
       return;
     }
 
-    idToName[bundle.publicId] = path.basename(nullthrows(bundle.name));
+    idToName.push([
+      bundle.publicId,
+      relativeBundlePath(entryBundle, nullthrows(bundle), {
+        leadingDotSlash: false,
+      }),
+    ]);
 
     if (bundle !== entryBundle && isNewContext(bundle, bundleGraph)) {
       for (let referenced of bundleGraph.getReferencedBundles(bundle)) {
-        idToName[referenced.publicId] = path.basename(
-          nullthrows(referenced.name),
-        );
+        idToName.push([
+          referenced.publicId,
+          relativeBundlePath(entryBundle, nullthrows(referenced), {
+            leadingDotSlash: false,
+          }),
+        ]);
       }
       // New contexts have their own manifests, so there's no need to continue.
       actions.skipChildren();
     }
   }, entryBundle);
 
-  return (
-    "require('./helpers/bundle-manifest').register(JSON.parse(" +
-    JSON.stringify(JSON.stringify(idToName)) +
-    '));'
-  );
-}
+  let baseUrl =
+    entryBundle.env.outputFormat === 'esmodule' &&
+    entryBundle.env.supports('import-meta-url')
+      ? 'new __parcel__URL__("").toString()' // <-- this isn't ideal. We should use `import.meta.url` directly but it get's replaced currently
+      : `require('./helpers/bundle-url').getBundleURL('${entryBundle.publicId}')`;
 
-function getRelativeBasePath(relativePath: string) {
-  // Get the relative part of the path. This part is not in the manifest, only the basename is.
-  let relativeBase = path.posix.dirname(relativePath);
-  if (relativeBase === '.') {
-    return '';
-  }
-  return JSON.stringify(relativeBase + '/');
+  return `require('./helpers/bundle-manifest').register(${baseUrl},JSON.parse(${JSON.stringify(
+    JSON.stringify(idToName),
+  )}));`;
 }
 
 function getRelativePathExpr(
@@ -673,18 +668,6 @@ function getRelativePathExpr(
   options: PluginOptions,
 ): string {
   let relativePath = relativeBundlePath(from, to, {leadingDotSlash: false});
-  if (shouldUseRuntimeManifest(from, options)) {
-    let basePath = getRelativeBasePath(relativePath);
-    let resolvedPath = `require('./helpers/bundle-manifest').resolve(${JSON.stringify(
-      to.publicId,
-    )})`;
-
-    if (basePath) {
-      return `${basePath} + ${resolvedPath}`;
-    }
-    return resolvedPath;
-  }
-
   let res = JSON.stringify(relativePath);
   if (options.hmrOptions) {
     res += ' + "?" + Date.now()';

packages/runtimes/js/src/helpers/browser/esm-js-loader.js

@@ -1,9 +1,6 @@
-function load(id, base) {
-  let bundleId = require('../bundle-manifest').resolve(id);
-  let request = base ? './' + base + bundleId : './' + bundleId;
-
+function load(id) {
   // eslint-disable-next-line no-undef
-  return __parcel__import__(request);
+  return __parcel__import__(require('../bundle-manifest').resolve(id));
 }
 
 module.exports = load;

packages/runtimes/js/src/helpers/bundle-manifest.js

@@ -1,18 +1,17 @@
-var mapping = {};
+var mapping = new Map();
 
-function register(pairs) {
-  var keys = Object.keys(pairs);
-  for (var i = 0; i < keys.length; i++) {
-    mapping[keys[i]] = pairs[keys[i]];
+function register(baseUrl, pairs) {
+  for (var i = 0; i < pairs.length; i++) {
+    mapping.set(pairs[i][0], {baseUrl, path: pairs[i][1]});
   }
 }
 
 function resolve(id) {
-  var resolved = mapping[id];
+  var resolved = mapping.get(id);
   if (resolved == null) {
     throw new Error('Could not resolve bundle with id ' + id);
   }
-  return resolved;
+  return new URL(resolved.path, resolved.baseUrl).toString();
 }
 
 module.exports.register = register;