ref(nextjs): Prefer esm/index.server.js over CJS version

[mischnic]

This makes bundlers prefer the ESM version of index.server.js, for better tree shaking, as ESM reexports can be statically analyzed and skipped when not needed, unlike the CJS version

The ESM versions are already listed first for the edge versions

[Lms24]

Hey @mischnic thanks for opening this PR!

To assess priority: Is the current exports declaration blocking you in some way? Could you elaborate a bit more what the specific issue is that this PR solves? Thanks!

I started our CI so that we check if our NextJS E2E tests are still passing. Unfortunately, exports changes are always a bit risky.

Also gonna ask @chargome to review

[mischnic]

Sorry for the short description, this is the fix for #16121 as with this change, importing @sentry/nextjs in a Node.js context resolves to an ESM file and bundlers including Turbopack are able to treeshake away everything except the required export (before, this import lead to withSentryConfig to be included in the user's project).

[chargome]

Closing, as this seems to be resolved on the Vercel side. If I misunderstood this feel free to re-open!

[Lms24]

Hey @mischnic apologies for the long wait. Currently going through some older PRs in this repo. Is this still relevant to get merged? (cc @chargome when you're back)

[mischnic]

Yeah, this is still relevant, I talked to Charly in the meantime as well.
But the fix as implement doesn't work right now.