esm: remove support for deprecated hooks

Those have been deprecated for a while, it's time. PR-URL: #47580 Reviewed-By: Jacob Smith <[email protected]> Reviewed-By: Mohammed Keyvanzadeh <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Geoffrey Booth <[email protected]> Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: James M Snell <[email protected]>
nodejs · Apr 19, 2023 · 7dd3732 · 7dd3732
1 parent f509f10
commit 7dd3732
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 87 deletions.
diff --git a/lib/internal/modules/esm/hooks.js b/lib/internal/modules/esm/hooks.js
@@ -1,7 +1,6 @@
 'use strict';
 
 const {
-  ArrayPrototypeJoin,
   ArrayPrototypePush,
   FunctionPrototypeCall,
   Int32Array,
@@ -661,45 +660,9 @@ function pluckHooks({
   globalPreload,
   resolve,
   load,
-  // obsolete hooks:
-  dynamicInstantiate,
-  getFormat,
-  getGlobalPreloadCode,
-  getSource,
-  transformSource,
 }) {
-  const obsoleteHooks = [];
   const acceptedHooks = { __proto__: null };
 
-  if (getGlobalPreloadCode) {
-    globalPreload ??= getGlobalPreloadCode;
-
-    process.emitWarning(
-      'Loader hook "getGlobalPreloadCode" has been renamed to "globalPreload"',
-    );
-  }
-  if (dynamicInstantiate) {
-    ArrayPrototypePush(obsoleteHooks, 'dynamicInstantiate');
-  }
-  if (getFormat) {
-    ArrayPrototypePush(obsoleteHooks, 'getFormat');
-  }
-  if (getSource) {
-    ArrayPrototypePush(obsoleteHooks, 'getSource');
-  }
-  if (transformSource) {
-    ArrayPrototypePush(obsoleteHooks, 'transformSource');
-  }
-
-  if (obsoleteHooks.length) {
-    process.emitWarning(
-      `Obsolete loader hook(s) supplied and will be ignored: ${
-        ArrayPrototypeJoin(obsoleteHooks, ', ')
-      }`,
-      'DeprecationWarning',
-    );
-  }
-
   if (globalPreload) {
     acceptedHooks.globalPreload = globalPreload;
   }

diff --git a/test/es-module/test-esm-loader-obsolete-hooks.mjs b/test/es-module/test-esm-loader-obsolete-hooks.mjs
diff --git a/test/fixtures/es-module-loaders/hooks-obsolete.mjs b/test/fixtures/es-module-loaders/hooks-obsolete.mjs