search

vercel / nft

Node.js dependency tracing utility
https://npmjs.com/@vercel/nft
MIT License
1.31k stars 136 forks source link

`@vercel/nft` does not pick up platform-specific binaries for `sentry` #382

Open LukeHagar opened 9 months ago

LukeHagar commented 9 months ago

I am creating this issue as #371 was closed without the entire issue being addressed, this issue is still occurring with sentry and the sentry team was forced to implement a solution to address this issue manually.

This will no doubt occur with other packages and solutions and anyone using the previous version of sentry.

styfle commented 9 months ago

Can you share example code that isn't working with @vercel/nft?

I thought it was already fixed upstream in https://github.com/getsentry/sentry-cli/pull/1874 so we'll need to know which version of the package and the line of code that causes the failure.

There is an existing integration test for sentry here: https://github.com/vercel/nft/blob/main/test/integration/sentry.js