fix(cjs-build): enable evaluating import.meta in cjs build

[nicotsx]

Purpose

A while ago, a PR was merged to disable import.meta evaluation altogether to support new versions of Webpack. This sometimes causes issues in CJS projects.

In the comments, there were some discussions about whether we should have a conditional and keep the previous behavior for CJS builds. This PR addresses exactly this by translating import.meta.url to a CJS compatible statement which results to the same output.

I have created a comparative build between two projects using the same code and a dependency (@sentry/node) that uses import.meta.url. You can see the difference in the output index.js

Before the change:

const importMetaUrl = typeof import.meta.url !== 'undefined' ? import.meta.url : undefined;

After:

const importMetaUrl = typeof require("url").pathToFileURL(__filename).href !== 'undefined' ? require("url").pathToFileURL(__filename).href : undefined;

Closes #1019

[styfle]

Thanks for the PR!

Please add a test to confirm this is working as expected.

[nicotsx]

@styfle Sorry for taking this long to add the unit tests, it should work as expected now

[styfle]

Great work, thanks!

[github-actions]

🎉 This PR is included in version 0.38.4 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀