Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

When selecting a package variant from an export map we should favor n… #27030

Merged
merged 1 commit into from
Jul 6, 2023

Conversation

gnoff
Copy link
Collaborator

@gnoff gnoff commented Jun 30, 2023

When selecting a package variant from an export map we should favor node over edge-light

edge-light represents a runtime with some minimal set of web apis generally found across edge runtimes. However some environments might be both edge-light compatible and node compatible and (node is adding many web APIs) and when both conditions exist we want to favor the node implementations. A followup to this change will add the web streams APIs to Flight and Fizz so the node version exports the same interfaces for web streams that edge does in addition to the node specific implementations.

@gnoff gnoff requested a review from sebmarkbage June 30, 2023 20:28
@facebook-github-bot facebook-github-bot added CLA Signed React Core Team Opened by a member of the React Core Team labels Jun 30, 2023
@react-sizebot
Copy link

Comparing: 1fdacbe...1487aa7

Critical size changes

Includes critical production bundles, as well as any change greater than 2%:

Name +/- Base Current +/- gzip Base gzip Current gzip
oss-stable/react-dom/cjs/react-dom.production.min.js = 164.22 kB 164.22 kB = 51.74 kB 51.74 kB
oss-experimental/react-dom/cjs/react-dom.production.min.js = 171.63 kB 171.63 kB = 53.95 kB 53.95 kB
facebook-www/ReactDOM-prod.classic.js = 566.78 kB 566.78 kB = 99.97 kB 99.97 kB
facebook-www/ReactDOM-prod.modern.js = 550.58 kB 550.58 kB = 97.13 kB 97.13 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against 1487aa7

@gnoff gnoff merged commit a19bbeb into facebook:main Jul 6, 2023
@gnoff gnoff deleted the favor-node branch July 6, 2023 17:51
EdisonVan pushed a commit to EdisonVan/react that referenced this pull request Apr 15, 2024
facebook#27030)

When selecting a package variant from an export map we should favor node
over edge-light

edge-light represents a runtime with some minimal set of web apis
generally found across edge runtimes. However some environments might be
both edge-light compatible and node compatible and (node is adding many
web APIs) and when both conditions exist we want to favor the node
implementations. A followup to this change will add the web streams APIs
to Flight and Fizz so the node version exports the same interfaces for
web streams that edge does in addition to the node specific
implementations.
bigfootjon pushed a commit that referenced this pull request Apr 18, 2024
#27030)

When selecting a package variant from an export map we should favor node
over edge-light

edge-light represents a runtime with some minimal set of web apis
generally found across edge runtimes. However some environments might be
both edge-light compatible and node compatible and (node is adding many
web APIs) and when both conditions exist we want to favor the node
implementations. A followup to this change will add the web streams APIs
to Flight and Fizz so the node version exports the same interfaces for
web streams that edge does in addition to the node specific
implementations.

DiffTrain build for commit a19bbeb.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed React Core Team Opened by a member of the React Core Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants