From 8dd06850ae9287d2171541f75654a56f34d356bd Mon Sep 17 00:00:00 2001
From: Bradley Farias <bradley.meck@gmail.com>
Date: Sun, 21 Mar 2021 14:35:49 -0500
Subject: [PATCH] esm: use correct URL for error decoration

PR-URL: https://github.com/nodejs/node/pull/37854
Reviewed-By: Guy Bedford <guybedford@gmail.com>
Reviewed-By: Jan Krems <jan.krems@gmail.com>
---
 lib/internal/modules/esm/module_job.js        |  6 +++++-
 test/fixtures/policy/bad-main.mjs             |  1 +
 .../dependencies-missing-export-policy.json   | 11 ++++++++++
 test/parallel/test-policy-dependencies.js     | 20 +++++++++++++++++++
 4 files changed, 37 insertions(+), 1 deletion(-)
 create mode 100644 test/fixtures/policy/bad-main.mjs
 create mode 100644 test/fixtures/policy/dependencies/dependencies-missing-export-policy.json

diff --git a/lib/internal/modules/esm/module_job.js b/lib/internal/modules/esm/module_job.js
index ebae1940502b4b..ba97a0e31784c2 100644
--- a/lib/internal/modules/esm/module_job.js
+++ b/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 '(.+)'/);
diff --git a/test/fixtures/policy/bad-main.mjs b/test/fixtures/policy/bad-main.mjs
new file mode 100644
index 00000000000000..db1938ad7c2e87
--- /dev/null
+++ b/test/fixtures/policy/bad-main.mjs
@@ -0,0 +1 @@
+import {doesNotExist} from './dep.js';
diff --git a/test/fixtures/policy/dependencies/dependencies-missing-export-policy.json b/test/fixtures/policy/dependencies/dependencies-missing-export-policy.json
new file mode 100644
index 00000000000000..5da0de13920936
--- /dev/null
+++ b/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
+    }
+  }
+}
diff --git a/test/parallel/test-policy-dependencies.js b/test/parallel/test-policy-dependencies.js
index 4486e0f8aa08c0..157da8295266db 100644
--- a/test/parallel/test-policy-dependencies.js
+++ b/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');
+}