esm: use correct URL for error decoration

PR-URL: #37854
Reviewed-By: Guy Bedford <[email protected]>
Reviewed-By: Jan Krems <[email protected]>

Author: bmeck

lib/internal/modules/esm/module_job.js

@@ -109,7 +109,11 @@ class ModuleJob {
       if (StringPrototypeIncludes(e.message,
                                   ' does not provide an export named')) {
         const splitStack = StringPrototypeSplit(e.stack, '\n');
-        const parentFileUrl = splitStack[0];
+        const parentFileUrl = StringPrototypeReplace(
+          splitStack[0],
+          /:\d+$/,
+          ''
+        );
         const { 1: childSpecifier, 2: name } = StringPrototypeMatch(
           e.message,
           /module '(.*)' does not provide an export named '(.+)'/);

test/fixtures/policy/bad-main.mjs

@@ -0,0 +1 @@
+import {doesNotExist} from './dep.js';

test/fixtures/policy/dependencies/dependencies-missing-export-policy.json

@@ -0,0 +1,11 @@
+{
+  "resources": {
+    "../bad-main.mjs": {
+      "integrity": true,
+      "dependencies": true
+    },
+    "../dep.js": {
+      "integrity": true
+    }
+  }
+}

test/parallel/test-policy-dependencies.js

@@ -89,3 +89,23 @@ const dep = fixtures.path('policy', 'parent.js');
   );
   assert.strictEqual(status, 1);
 }
+{
+  // Regression test for https://github.com/nodejs/node/issues/37812
+  const depPolicy = fixtures.path(
+    'policy',
+    'dependencies',
+    'dependencies-missing-export-policy.json');
+  const { status, stderr } = spawnSync(
+    process.execPath,
+    [
+      '--experimental-policy',
+      depPolicy,
+      fixtures.path('policy', 'bad-main.mjs'),
+    ]
+  );
+  assert.strictEqual(status, 1);
+  assert.match(
+    `${stderr}`,
+    /SyntaxError: Named export 'doesNotExist' not found\./,
+    'Should give the real SyntaxError and position');
+}