further review findings

Author: dygabo

lib/internal/modules/esm/get_format.js

@@ -23,7 +23,6 @@ const { containsModuleSyntax } = internalBinding('contextify');
 const { getPackageScopeConfig, getPackageType } = require('internal/modules/package_json_reader');
 const { fileURLToPath } = require('internal/url');
 const { readFileSync } = require('fs');
-const { Buffer } = require('buffer');
 const { ERR_UNKNOWN_FILE_EXTENSION } = require('internal/errors').codes;
 
 const protocolHandlers = {

test/es-module/test-esm-detect-ambiguous.mjs

@@ -168,7 +168,7 @@ describe('--experimental-detect-module', { concurrency: !process.env.TEST_PARALL
       });
     }
 
-    it('should hint format correctly for extensionles modules resolve hook', async () => {
+    it('should hint format correctly for the resolve hook for extensionless modules', async () => {
       let writeSync;
       const { stdout, stderr, code, signal } = await spawnPromisified(process.execPath, [
         '--experimental-detect-module',

test/es-module/test-esm-loader-hooks.mjs

@@ -781,17 +781,27 @@ describe('Loader hooks', { concurrency: !process.env.TEST_PARALLEL }, () => {
     }
   });
 
-  describe('should use hooks', async () => {
-    const { code, signal, stdout, stderr } = await spawnPromisified(process.execPath, [
-      '--import',
-      fixtures.fileURL('es-module-loaders/builtin-named-exports.mjs'),
-      fixtures.path('es-modules/require-esm-throws-with-loaders.js'),
-    ]);
+  describe('should use hooks ', async () => {
+    for (const { testConfigName, additionalOptions } of [
+      { testConfigName: 'without --experimental-detect-module', additionalOptions: [] },
+      { testConfigName: 'with --experimental-detect-module', additionalOptions: ['--experimental-detect-module'] },
+    ]) {
+      it(testConfigName, async () => {
+        const { code, signal, stdout, stderr } = await spawnPromisified(process.execPath, [
+          ...additionalOptions,
+          '--import',
+          '../es-module-loaders/builtin-named-exports.mjs',
+          fixtures.path('es-modules/require-esm-throws-with-loaders.js'),
+        ], {
+          cwd: fixtures.path('es-modules'),
+        });
 
-    assert.strictEqual(stderr, '');
-    assert.strictEqual(stdout, '');
-    assert.strictEqual(code, 0);
-    assert.strictEqual(signal, null);
+        assert.strictEqual(stderr, '');
+        assert.strictEqual(stdout, '');
+        assert.strictEqual(code, 0);
+        assert.strictEqual(signal, null);
+      });
+    }
   });
 
   it('should support source maps in commonjs translator', async () => {