Deprecate in favor of @npmcli/package-json

[lukekarrys]

The functionality of this package has been moved to @npmcli/package-json as part of npm/package-json#32. We should consider deprecating and archiving this package after examining its use across the rest of our packages and the ecosystem.

[43081j]

for what its worth, i don't think we should consider the two equivalent

read-package-json-fast:
https://npmgraph.js.org/?q=read-package-json-fast

@npmcli/package-json:
https://npmgraph.js.org/?q=@npmcli/package-json

we will be pulling so many more packages in, most likely for functionality we're not using. whereas read-package-json-fast is very, very focused (does one thing).

maybe we can at least have that subset of functionality still exposed in its own package?

[wraithgar]

This decision is a balance against "pure single package for its own sake" and "consolidating package parsing logic in all of npm". The latter was deemed more important. We have had several bugs in the past crop up because one of the 4 package parsing libs inside npm got a change, but the others didn't. It invites errors.

[43081j]

while that does make sense, it means there won't be a light way of reading a package JSON anymore.

all good, though. we will just read the file directly without the normalisation 👍

[wraithgar]

Unless you need to normalize it like npm does, this library absolutely is not needed. A try/catch around json-parse-even-better-errors would get you all of the parsing this does, including indentation detection.

[43081j]

i figured as much. we were mostly using it for the errors and normalisation, but in this particular case we don't really need that

thanks for the info 👍 deprecate away 🎉