fix: attach path to json parse error

wraithgar · wraithgar · commit f5fbf383a1fa · 2025-09-23T13:44:01.000-07:00
With a TODO to move this to the right place eventually
diff --git a/workspaces/arborist/lib/arborist/load-actual.js b/workspaces/arborist/lib/arborist/load-actual.js
@@ -285,6 +285,10 @@ module.exports = cls => class ActualLoader extends cls {
           params.overrides = root.overrides.getNodeRule({ name: pkg.name, version: pkg.version })
         }
       } catch (err) {
+        if (err.code === 'EJSONPARSE') {
+          // TODO @npmcli/package-json should be doing this
+          err.path = join(real, 'package.json')
+        }
         params.error = err
       }
 

Original file line number	Diff line number	Diff line change
`@@ -285,6 +285,10 @@ module.exports = cls => class ActualLoader extends cls {`
`285`	`285`	`params.overrides = root.overrides.getNodeRule({ name: pkg.name, version: pkg.version })`
`286`	`286`	`}`
`287`	`287`	`} catch (err) {`
	`288`	`+ if (err.code === 'EJSONPARSE') {`
	`289`	`+ // TODO @npmcli/package-json should be doing this`
	`290`	`+ err.path = join(real, 'package.json')`
	`291`	`+ }`
`288`	`292`	`params.error = err`
`289`	`293`	`}`
`290`	`294`