nodejs
diff --git a/Collapse file
‎lib/internal/modules/package_json_reader.js‎
Copy file name to clipboardExpand all lines: lib/internal/modules/package_json_reader.js
+9-15Lines changed: 9 additions & 15 deletions b/Collapse file
‎lib/internal/modules/package_json_reader.js‎
Copy file name to clipboardExpand all lines: lib/internal/modules/package_json_reader.js
+9-15Lines changed: 9 additions & 15 deletions
diff --git a/Collapse file
‎test/sequential/test-module-loading.js‎
Copy file name to clipboardExpand all lines: test/sequential/test-module-loading.js
+1-1Lines changed: 1 addition & 1 deletion b/Collapse file
‎test/sequential/test-module-loading.js‎
Copy file name to clipboardExpand all lines: test/sequential/test-module-loading.js
+1-1Lines changed: 1 addition & 1 deletion
@@ -10,7 +10,7 @@ const {
 } = require('internal/errors').codes;
 const { internalModuleReadJSON } = internalBinding('fs');
 const { toNamespacedPath } = require('path');
-const { kEmptyObject } = require('internal/util');
+const { kEmptyObject, setOwnProperty } = require('internal/util');
 
 const { fileURLToPath, pathToFileURL } = require('internal/url');
 
@@ -73,20 +73,14 @@ function read(jsonPath, { base, specifier, isESM } = kEmptyObject) {
     let parsed;
     try {
       parsed = JSONParse(string);
-    } catch (error) {
-      if (isESM) {
-        throw new ERR_INVALID_PACKAGE_CONFIG(
-          jsonPath,
-          (base ? `"${specifier}" from ` : '') + fileURLToPath(base || specifier),
-          error.message,
-        );
-      } else {
-        // For backward compat, we modify the error returned by JSON.parse rather than creating a new one.
-        // TODO(aduh95): make it throw ERR_INVALID_PACKAGE_CONFIG in a semver-major with original error as cause
-        error.message = 'Error parsing ' + jsonPath + ': ' + error.message;
-        error.path = jsonPath;
-        throw error;
-      }
+    } catch (cause) {
+      const error = new ERR_INVALID_PACKAGE_CONFIG(
+        jsonPath,
+        isESM && (base ? `"${specifier}" from ` : '') + fileURLToPath(base || specifier),
+        cause.message,
+      );
+      setOwnProperty(error, 'cause', cause);
+      throw error;
     }
 
     result.exists = true;
 
@@ -128,7 +128,7 @@ assert.throws(
 
 assert.throws(
   function() { require('../fixtures/packages/unparseable'); },
-  /^SyntaxError: Error parsing/
+  { code: 'ERR_INVALID_PACKAGE_CONFIG' }
 );
 
 {
Original file line number	Diff line number	Diff line change
`@@ -128,7 +128,7 @@ assert.throws(`
`128`	`128`
`129`	`129`	`assert.throws(`
`130`	`130`	`function() { require('../fixtures/packages/unparseable'); },`
`131`		`- /^SyntaxError: Error parsing/`
	`131`	`+ { code: 'ERR_INVALID_PACKAGE_CONFIG' }`
`132`	`132`	`);`
`133`	`133`
`134`	`134`	`{`