chore(compartment-mapper): do not call moduleSourceHook for "error"-type module sources

Author: boneskull

packages/compartment-mapper/src/import-hook.js

@@ -518,18 +518,12 @@ function* chooseModuleDescriptor(
  * @param {StrictlyRequiredFn} strictlyRequiredForCompartment
  * @param {string} packageLocation
  * @param {CompartmentSources} packageSources
- * @param {import('./types.js').ModuleSourceHook | undefined} moduleSourceHook
- * @param {string} canonicalName
- * @param {import('./types.js').LogFn} log
  * @returns {DeferErrorFn}
  */
 const makeDeferError = (
   strictlyRequiredForCompartment,
   packageLocation,
   packageSources,
-  moduleSourceHook,
-  canonicalName,
-  log,
 ) => {
   /**
    * @type {DeferErrorFn}
@@ -560,14 +554,6 @@ const makeDeferError = (
     };
     packageSources[specifier] = moduleSource;
 
-    if (moduleSourceHook) {
-      moduleSourceHook({
-        moduleSource: { error: moduleSource.deferredError },
-        canonicalName,
-        log,
-      });
-    }
-
     return record;
   };
   return deferError;
@@ -638,9 +624,6 @@ export const makeImportHookMaker = (
       strictlyRequiredForCompartment,
       packageLocation,
       packageSources,
-      moduleSourceHook,
-      compartmentDescriptor.label,
-      log,
     );
 
     /** @type {ImportHook} */
@@ -812,9 +795,6 @@ export function makeImportNowHookMaker(
       strictlyRequiredForCompartment,
       packageLocation,
       packageSources,
-      moduleSourceHook,
-      '', // no canonicalName for now
-      noop, // no-op log
     );
 
     /**