Conditionally use the esm package as a fallback, rather than default

Author: NullVoxPopuli

lib/load_brocfile.ts

@@ -1,8 +1,5 @@
 import path from 'path';
 import findup from 'findup-sync';
-import esm from 'esm';
-
-const esmRequire = esm(module);
 
 interface LoadBrocfileOptions {
   brocfilePath?: string;
@@ -37,8 +34,22 @@ function requireBrocfile(brocfilePath: string) {
     // Load brocfile via ts-node
     brocfile = require(brocfilePath);
   } else {
+    /**
+     * because 'esm' patches global modules,
+     * let's only load 'esm' if we absolutely have to.
+     * See context: https://github.com/broccolijs/broccoli/issues/498
+     *  (and related linkes)
+     */
+    try {
+      brocfile = require(brocfilePath);
+    } catch {
+      // eslint-disable-next-line @typescript-eslint/no-var-requires, node/no-missing-require
+      const esm = require('esm');
+      const esmRequire = esm(module);
+
+      brocfile = esmRequire(brocfilePath);
+    }
     // Load brocfile via esm shim
-    brocfile = esmRequire(brocfilePath);
   }
 
   // ESM `export default X` is represented as module.exports = { default: X }