You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When an export condition matches it doesn't use the "browser" main field to remap it just because "browser" was set.
Actual Behavior
In our case it matches on "worker" and then gets upgraded to the "browser" version unexpectedly.
Additional Information
solidjs/solid-start#263 Lays out the issue. It is talking about Vite. And I will take it up with them as well but it seems like this behavior originated by decisions taken by rollup.
This issue isn't only unexpected but it makes things very difficult. We have 3 things trying to balance:
Server vs Browser runtimes are different for SolidJS
Cloudflare Worker environments are Server environments for us but most tools call them browser, understandably because of Service/Web Workers. That's ok we can control this with export conditions.
Browser mainfield support is necessary for legacy tooling support. But its existence causes the work we do to map workers to be undone.
All these seem unmoveable. But perhaps main field overriding export conditions is not.
The text was updated successfully, but these errors were encountered:
Provide a minimal repository link (Read https://git.io/fNzHA for instructions).
Please use NPM for installing dependencies!
These may take more time to triage than the other options.
Expected Behavior
When an export condition matches it doesn't use the "browser" main field to remap it just because "browser" was set.
Actual Behavior
In our case it matches on "worker" and then gets upgraded to the "browser" version unexpectedly.
Additional Information
solidjs/solid-start#263 Lays out the issue. It is talking about Vite. And I will take it up with them as well but it seems like this behavior originated by decisions taken by rollup.
This issue isn't only unexpected but it makes things very difficult. We have 3 things trying to balance:
All these seem unmoveable. But perhaps main field overriding export conditions is not.
The text was updated successfully, but these errors were encountered: