Fix multiple entries pointing to wrong bundle in dist

packages/core/core/src/utils.js

@@ -35,7 +35,7 @@ const base62 = baseX(
 );
 
 export function getBundleGroupId(bundleGroup: BundleGroup): string {
-  return 'bundle_group:' + bundleGroup.entryAssetId;
+  return 'bundle_group:' + bundleGroup.target.name + bundleGroup.entryAssetId;
 }
 
 export function assertSignalNotAborted(signal: ?AbortSignal): void {

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

@@ -2076,6 +2076,59 @@ describe('html', function () {
     }
   });
 
+  it('supports multiple dist targets', async function () {
+    let b = await bundle(
+      path.join(__dirname, '/integration/html-multi-targets/'),
+      {
+        mode: 'production',
+        defaultTargetOptions: {
+          shouldScopeHoist: false,
+          shouldOptimize: false,
+          sourceMaps: false,
+        },
+      },
+    );
+    assertBundles(b, [
+      {
+        name: 'index.html',
+        type: 'html',
+        assets: ['index.html'],
+      },
+      {
+        type: 'js',
+        assets: [
+          'bundle-manifest.js',
+          'bundle-url.js',
+          'cacheLoader.js',
+          'index.js',
+          'js-loader.js',
+        ],
+      },
+      {
+        type: 'js',
+        assets: ['esmodule-helpers.js', 'shared.js'],
+      },
+      {
+        type: 'js',
+        assets: ['esmodule-helpers.js', 'shared.js'],
+      },
+      {
+        name: 'index.html',
+        type: 'html',
+        assets: ['index.html'],
+      },
+      {
+        type: 'js',
+        assets: [
+          'bundle-manifest.js',
+          'bundle-url.js',
+          'cacheLoader.js',
+          'index.js',
+          'js-loader.js',
+        ],
+      },
+    ]);
+  });
   it('should isolate async scripts', async function () {
     let b = await bundle(
       path.join(__dirname, '/integration/html-async-script/index.html'),

packages/core/integration-tests/test/integration/html-multi-targets/package.json

@@ -0,0 +1,20 @@
+{
+  "name": "assets-in-confluence",
+  "version": "0.1.0",
+  "private": true,
+  "targets": {
+    "main1": {
+      "distDir": "./dist/main1",
+      "source": "./src/main1/index.html",
+      "publicUrl": "./"
+    },
+    "main2": {
+      "distDir": "./dist/main2",
+      "source": "./src/main2/index.html",
+      "publicUrl": "./"
+    }
+  },
+  "@parcel/bundler-default": {
+    "minBundleSize": 0
+  }
+}

packages/core/integration-tests/test/integration/html-multi-targets/src/main1/index.html

@@ -0,0 +1 @@
+<script type="module" src="index.js"></script>

packages/core/integration-tests/test/integration/html-multi-targets/src/main1/index.js

@@ -0,0 +1,3 @@
+module.exports = async function () {
+  return import('../shared');
+};

packages/core/integration-tests/test/integration/html-multi-targets/src/main2/index.html

@@ -0,0 +1 @@
+<script type="module" src="index.js"></script>

packages/core/integration-tests/test/integration/html-multi-targets/src/main2/index.js

@@ -0,0 +1,3 @@
+module.exports = async function () {
+  return import('../shared');
+};

packages/core/integration-tests/test/integration/html-multi-targets/src/shared.js

@@ -0,0 +1 @@
+export const stuff = () => 'lots and lots of stuff'