esm,loaders: tidy ESMLoader internals

Author: JakobJingleheimer

lib/internal/modules/esm/loader.js

@@ -179,39 +179,37 @@ function nextHookFactory(chain, meta, { validateArgs, validateOutput }) {
  * the main module and everything in its dependency graph.
  */
 class ESMLoader {
-  /**
-   * Prior to ESM loading. These are called once before any modules are started.
-   * @private
-   * @property {KeyedHook[]} globalPreloaders Last-in-first-out
-   *  list of preload hooks.
-   */
-  #globalPreloaders = [];
-
-  /**
-   * Phase 2 of 2 in ESM loading.
-   * @private
-   * @property {KeyedHook[]} loaders Last-in-first-out
-   *  collection of loader hooks.
-   */
-  #loaders = [
-    {
-      fn: defaultLoad,
-      url: 'node:internal/modules/esm/load',
-    },
-  ];
-
-  /**
-   * Phase 1 of 2 in ESM loading.
-   * @private
-   * @property {KeyedHook[]} resolvers Last-in-first-out
-   *  collection of resolver hooks.
-   */
-  #resolvers = [
-    {
-      fn: defaultResolve,
-      url: 'node:internal/modules/esm/resolve',
-    },
-  ];
+  #hooks = {
+    __proto__: null,
+    /**
+     * Prior to ESM loading. These are called once before any modules are started.
+     * @private
+     * @property {KeyedHook[]} globalPreload Last-in-first-out list of preload hooks.
+     */
+    globalPreload: [],
+    /**
+     * Phase 2 of 2 in ESM loading.
+     * @private
+     * @property {KeyedHook[]} load Last-in-first-out collection of loader hooks.
+     */
+    load: [
+      {
+        fn: defaultLoad,
+        url: 'node:internal/modules/esm/load',
+      },
+    ],
+    /**
+     * Phase 1 of 2 in ESM loading.
+     * @private
+     * @property {KeyedHook[]} resolve Last-in-first-out collection of resolve hooks.
+     */
+    resolve: [
+      {
+        fn: defaultResolve,
+        url: 'node:internal/modules/esm/resolve',
+      },
+    ],
+  };
 
   #importMetaInitializer = initializeImportMeta;
 
@@ -305,13 +303,13 @@ class ESMLoader {
     );
 
     if (globalPreload) {
-      acceptedHooks.globalPreloader = globalPreload;
+      acceptedHooks.globalPreload = globalPreload;
     }
     if (resolve) {
-      acceptedHooks.resolver = resolve;
+      acceptedHooks.resolve = resolve;
     }
     if (load) {
-      acceptedHooks.loader = load;
+      acceptedHooks.load = load;
     }
 
     return acceptedHooks;
@@ -333,34 +331,34 @@ class ESMLoader {
         url,
       } = customLoaders[i];
       const {
-        globalPreloader,
-        resolver,
-        loader,
+        globalPreload,
+        resolve,
+        load,
       } = ESMLoader.pluckHooks(exports);
 
-      if (globalPreloader) {
+      if (globalPreload) {
         ArrayPrototypePush(
-          this.#globalPreloaders,
+          this.#hooks.globalPreload,
           {
-            fn: globalPreloader,
+            fn: globalPreload,
             url,
           },
         );
       }
-      if (resolver) {
+      if (resolve) {
         ArrayPrototypePush(
-          this.#resolvers,
+          this.#hooks.resolve,
           {
-            fn: resolver,
+            fn: resolve,
             url,
           },
         );
       }
-      if (loader) {
+      if (load) {
         ArrayPrototypePush(
-          this.#loaders,
+          this.#hooks.load,
           {
-            fn: loader,
+            fn: load,
             url,
           },
         );
@@ -411,8 +409,8 @@ class ESMLoader {
   async getModuleJob(specifier, parentURL, importAssertions) {
     let importAssertionsForResolve;
 
-    // By default, `this.#loaders` contains just the Node default load hook
-    if (this.#loaders.length !== 1) {
+    // By default, `this.#hooks.load` contains just the Node default load hook
+    if (this.#hooks.load.length !== 1) {
       // We can skip cloning if there are no user-provided loaders because
       // the Node.js default resolve hook does not use import assertions.
       importAssertionsForResolve = ObjectAssign(
@@ -555,7 +553,7 @@ class ESMLoader {
    * @returns {{ format: ModuleFormat, source: ModuleSource }}
    */
   async load(url, context = {}) {
-    const chain = this.#loaders;
+    const chain = this.#hooks.load;
     const meta = {
       chainFinished: null,
       context,
@@ -684,7 +682,7 @@ class ESMLoader {
   }
 
   preload() {
-    for (let i = this.#globalPreloaders.length - 1; i >= 0; i--) {
+    for (let i = this.#hooks.globalPreload.length - 1; i >= 0; i--) {
       const channel = new MessageChannel();
       const {
         port1: insidePreload,
@@ -695,19 +693,19 @@ class ESMLoader {
       insideLoader.unref();
 
       const {
-        fn: preloader,
+        fn: preload,
         url: specifier,
-      } = this.#globalPreloaders[i];
+      } = this.#hooks.globalPreload[i];
 
-      const preload = preloader({
+      const preloaded = preload({
         port: insideLoader,
       });
 
-      if (preload == null) { return; }
+      if (preloaded == null) { return; }
 
       const hookErrIdentifier = `${specifier} globalPreload`;
 
-      if (typeof preload !== 'string') { // [2]
+      if (typeof preloaded !== 'string') { // [2]
         throw new ERR_INVALID_RETURN_VALUE(
           'a string',
           hookErrIdentifier,
@@ -716,7 +714,7 @@ class ESMLoader {
       }
       const { compileFunction } = require('vm');
       const preloadInit = compileFunction(
-        preload,
+        preloaded,
         ['getBuiltin', 'port', 'setImportMetaCallback'],
         {
           filename: '<preload>',
@@ -804,7 +802,7 @@ class ESMLoader {
         parentURL,
       );
     }
-    const chain = this.#resolvers;
+    const chain = this.#hooks.resolve;
     const context = {
       conditions: DEFAULT_CONDITIONS,
       importAssertions,