Issue with Node/Edge files - export conditions

[vicb]

We have issues when sometimes the bundle include Node files where Edge/Browser files would be better suited for workerd.

A few examples of this:

• og (issue, PR)
• Next Auth / jose (issue)
• OTel
• ...

My current understanding of the issue is that Next traces Node files but not edge/browser files.

We should check if there is way to trace edge/browser filesμ. This seems to be supported by nft but not sure if Next exposes this options. We might need to submit to PR to Next.

Ref: ESBuild docs about conditions

Related PRs:

• fix: Drop the module condition from ESBuild #339
• fix: inline optional dependencies when bundling the server #325
• fix build issues with @opentelemetry #302
• fix: patch @vercel/og usage to use the edge runtime version #283

[lujjjh]

We're encountering the same issue when integrating with @sentry/nextjs. Patching it to use the edge version resolved the problem.

[vicb]

@lujjjh do you a minimal repro by any chance (link to a GH repository)?

[lujjjh]

@vicb Here's the repository: https://github.com/lujjjh/opennextjs-sentry-repro

[vicb]

@lujjjh Your issue has been fixed by @conico974 in this PR (the fix will be part of the next release)

The build conditions have been fixed via #593 and #628