Merge pull request #1686 from embroider-build/require-resolve

Resolver bugfixes

Author: ef4

packages/compat/src/audit.ts

@@ -534,6 +534,12 @@ export class Audit {
   private async resolveDeps(deps: string[], fromFile: string): Promise<InternalModule['resolved']> {
     let resolved = new Map() as NonNullable<InternalModule['resolved']>;
     for (let dep of deps) {
+      if (['@embroider/macros'].includes(dep)) {
+        // the audit process deliberately removes the @embroider/macros babel
+        // plugins, so the imports are still present and should be left alone.
+        continue;
+      }
+
       let resolution = await this.resolver.nodeResolve(dep, fromFile);
       switch (resolution.type) {
         case 'virtual':
@@ -542,11 +548,6 @@ export class Audit {
           this.scheduleVisit(resolution.filename, fromFile);
           break;
         case 'not_found':
-          if (['@embroider/macros', '@ember/template-factory'].includes(dep)) {
-            // the audit process deliberately removes the @embroider/macros babel
-            // plugins, so the imports are still present and should be left alone.
-            continue;
-          }
           resolved.set(dep, { isResolutionFailure: true as true });
           break;
         case 'real':

packages/compat/src/compat-app-builder.ts

@@ -263,11 +263,6 @@ export class CompatAppBuilder {
     let renamePackages = Object.assign({}, ...this.allActiveAddons.map(dep => dep.meta['renamed-packages']));
     let renameModules = Object.assign({}, ...this.allActiveAddons.map(dep => dep.meta['renamed-modules']));
 
-    let activeAddons: CompatResolverOptions['activeAddons'] = {};
-    for (let addon of this.allActiveAddons) {
-      activeAddons[addon.name] = addon.root;
-    }
-
     let options: CompatResolverOptions['options'] = {
       staticHelpers: this.options.staticHelpers,
       staticModifiers: this.options.staticModifiers,
@@ -277,7 +272,6 @@ export class CompatAppBuilder {
 
     let config: CompatResolverOptions = {
       // this part is the base ModuleResolverOptions as required by @embroider/core
-      activeAddons,
       renameModules,
       renamePackages,
       resolvableExtensions: this.resolvableExtensions(),

packages/compat/tests/audit.test.ts

@@ -42,7 +42,6 @@ describe('audit', function () {
       activePackageRules: [],
       renamePackages: {},
       renameModules: {},
-      activeAddons: {},
       engines: [
         {
           packageName: 'audit-this-app',