wip

packages/vite/src/esbuild-resolver.ts

@@ -66,6 +66,16 @@ export function esBuildResolver(): EsBuildPlugin {
         switch (resolution.type) {
           case 'found':
           case 'ignored':
+            // _app_ files in addons are part of the app, they should not be externalized
+            if (resolution.result.path) {
+              let pkg = resolverLoader.resolver.packageCache.ownerOfFile(resolution.result.path);
+              if (pkg?.isV2Addon()) {
+                resolverLoader.resolver.reverseSearchAppTree(pkg, resolution.result.path);
+                if (resolution.result.path?.includes('_app_')) {
+                  resolution.result.external = false;
+                }
+              }
+            }
             return resolution.result;
           case 'not_found':
             return resolution.err;

packages/vite/tests/optimize-deps.test.ts

@@ -2,7 +2,7 @@ import { optimizeDeps } from '../src/optimize-deps';
 
 describe('optimizeDeps', function () {
   test('should produce default output when invoked without arguments', function () {
-    const actual = optimizeDeps();
+    const actual = optimizeDeps({ appExtensions: ['.hbs'] });
 
     expect(actual).toMatchInlineSnapshot(
       {

test-packages/support/audit-assertions.ts

@@ -158,7 +158,7 @@ export class ExpectModule {
   }
 
   private emitMissingModule() {
-    const showNearMisses = 4;
+    const showNearMisses = 10;
     let actuals = sortBy(Object.keys(this.expectAudit.result.modules), candidate =>
       distance(candidate, this.outputName)
     );
@@ -180,14 +180,14 @@ export class ExpectModule {
     });
   }
 
-  withContents(fn: (src: string, imports: Import[]) => boolean, message?: string) {
+  withContents(fn: (src: string, imports: Import[]) => boolean, message?: string): PublicAPI<this> {
     if (!this.module) {
       this.emitMissingModule();
-      return;
+      return this;
     }
     if (this.module.type === 'unparseable') {
       this.emitUnparsableModule(message);
-      return;
+      return this;
     }
     const result = fn(this.module.content, this.module.imports);
     this.expectAudit.assert.pushResult({
@@ -196,6 +196,7 @@ export class ExpectModule {
       expected: true,
       message: message ?? `Expected passed function to return true for the contents of ${this.inputName}`,
     });
+    return this;
   }
 
   private emitUnparsableModule(message?: string) {
@@ -344,7 +345,9 @@ class EmptyExpectModule implements PublicAPI<ExpectModule> {
   doesNotExist() {}
   codeEquals() {}
   codeContains() {}
-  withContents() {}
+  withContents() {
+    return this;
+  }
 
   resolves(): PublicAPI<ExpectResolution> {
     return new EmptyExpectResolution() as PublicAPI<ExpectResolution>;

test-packages/support/rebuilder.ts

@@ -58,7 +58,6 @@ export class Rebuilder {
   }
 
   private captureStream(data: string) {
-    console.log(data.toString());
     this.combinedBuffer.push(data);
   }
 

tests/scenarios/vite-dep-optimizer-test.ts

@@ -160,8 +160,8 @@ app.forEachScenario(scenario => {
           let pageTitleImports = imports.filter(imp => /page-title/.test(imp.source));
           assert.strictEqual(pageTitleImports.length, 2, 'found two uses of page-title addon');
           assert.ok(
-            pageTitleImports.every(isOptimizedImport),
-            `every page-title module is optimized but we saw ${pageTitleImports.map(i => i.source).join(', ')}`
+            pageTitleImports.every(x => !isOptimizedImport(x)),
+            `every page-title module is not optimized but we saw ${pageTitleImports.map(i => i.source).join(', ')}`
           );
           return true;
         });