Skip to content

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