Merge pull request #1233 from NullVoxPopuli/hoist-transitive-imports

Default hoistTransitiveImports to false

packages/addon-dev/src/rollup.ts

@@ -65,8 +65,18 @@ export class Addon {
   // emitting ES modules, in your `destDir`, and their filenames are equal to
   // their bundle names (the bundle names get generated by `publicEntrypoints`
   // above).
+  //
+  // hoistTransitiveImports is disabled because the purpose of hoisting transitive imports
+  // is to improve performance of apps loading modules.
+  // Since v2 addons do not know exactly how they'll be used, this performance decision
+  // is left up to apps.
   output() {
-    return { dir: this.#destDir, format: 'es', entryFileNames: '[name]' };
+    return {
+      dir: this.#destDir,
+      format: 'es',
+      entryFileNames: '[name]',
+      hoistTransitiveImports: false,
+    };
   }
 
   dependencies() {