Skip to content

[Flight] Log aborted await and component renders #33641

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

Merged
merged 4 commits into from
Jun 25, 2025

Conversation

sebmarkbage
Copy link
Collaborator

@sebmarkbage sebmarkbage commented Jun 25, 2025

Screenshot 2025-06-25 at 1 02 14 PM

The I/O entry doesn't show as aborted in the Server Request track because technically it wasn't. The end time is just made up. It's still going. It's not aborted until the abort signal propagates and if we do get that signal wired up before it emits, it instead would show up as rejected.

This ensures that if you stop the performance tracing before the stream ends
that at least the tracks that could be rendered show up in order.

Conversely we keep this at the end too in case you started in the middle.
This ensures that timed operations like errors aren't emitting the end time.
@react-sizebot
Copy link

react-sizebot commented Jun 25, 2025

Comparing: cee7939...fb07c57

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.js = 6.68 kB 6.68 kB +0.05% 1.83 kB 1.83 kB
oss-stable/react-dom/cjs/react-dom-client.production.js = 530.57 kB 530.57 kB = 93.67 kB 93.67 kB
oss-experimental/react-dom/cjs/react-dom.production.js = 6.69 kB 6.69 kB = 1.83 kB 1.83 kB
oss-experimental/react-dom/cjs/react-dom-client.production.js = 651.66 kB 651.66 kB = 114.78 kB 114.78 kB
facebook-www/ReactDOM-prod.classic.js = 674.81 kB 674.81 kB = 118.78 kB 118.78 kB
facebook-www/ReactDOM-prod.modern.js = 665.30 kB 665.30 kB = 117.19 kB 117.19 kB
oss-experimental/react-server-dom-esm/cjs/react-server-dom-esm-client.browser.development.js +4.03% 128.39 kB 133.56 kB +2.34% 23.65 kB 24.21 kB
oss-experimental/react-client/cjs/react-client-flight.development.js +4.00% 129.45 kB 134.63 kB +2.47% 23.30 kB 23.87 kB
oss-experimental/react-server-dom-parcel/cjs/react-server-dom-parcel-client.browser.development.js +4.00% 129.50 kB 134.67 kB +2.38% 23.67 kB 24.23 kB
oss-experimental/react-server-dom-turbopack/cjs/react-server-dom-turbopack-client.browser.development.js +3.96% 130.57 kB 135.75 kB +2.31% 24.04 kB 24.60 kB
oss-experimental/react-server-dom-webpack/cjs/react-server-dom-webpack-client.browser.development.js +3.95% 131.13 kB 136.31 kB +2.27% 24.19 kB 24.74 kB
oss-experimental/react-server-dom-esm/cjs/react-server-dom-esm-client.node.development.js +3.91% 132.38 kB 137.55 kB +2.45% 24.23 kB 24.82 kB
oss-experimental/react-server-dom-parcel/cjs/react-server-dom-parcel-client.edge.development.js +3.87% 133.66 kB 138.84 kB +2.30% 24.59 kB 25.15 kB
oss-experimental/react-server-dom-parcel/cjs/react-server-dom-parcel-client.node.development.js +3.83% 134.92 kB 140.09 kB +2.29% 24.70 kB 25.27 kB
oss-experimental/react-server-dom-turbopack/cjs/react-server-dom-turbopack-client.edge.development.js +3.78% 136.76 kB 141.94 kB +2.23% 25.07 kB 25.63 kB
oss-experimental/react-server-dom-webpack/cjs/react-server-dom-webpack-client.edge.development.js +3.78% 136.85 kB 142.03 kB +2.23% 25.11 kB 25.67 kB
oss-experimental/react-server-dom-turbopack/cjs/react-server-dom-turbopack-client.node.development.js +3.75% 137.85 kB 143.02 kB +2.25% 25.20 kB 25.77 kB
oss-experimental/react-server-dom-webpack/cjs/react-server-dom-webpack-client.node.unbundled.development.js +3.69% 140.20 kB 145.37 kB +2.23% 25.44 kB 26.01 kB
oss-experimental/react-server-dom-webpack/cjs/react-server-dom-webpack-client.node.development.js +3.66% 141.53 kB 146.71 kB +2.21% 25.69 kB 26.26 kB
oss-experimental/react-server-dom-esm/esm/react-server-dom-esm-client.browser.development.js +2.33% 165.82 kB 169.69 kB +1.11% 37.90 kB 38.32 kB

Significant size changes

Includes any change greater than 0.2%:

Expand to show
Name +/- Base Current +/- gzip Base gzip Current gzip
oss-experimental/react-server-dom-esm/cjs/react-server-dom-esm-client.browser.development.js +4.03% 128.39 kB 133.56 kB +2.34% 23.65 kB 24.21 kB
oss-experimental/react-client/cjs/react-client-flight.development.js +4.00% 129.45 kB 134.63 kB +2.47% 23.30 kB 23.87 kB
oss-experimental/react-server-dom-parcel/cjs/react-server-dom-parcel-client.browser.development.js +4.00% 129.50 kB 134.67 kB +2.38% 23.67 kB 24.23 kB
oss-experimental/react-server-dom-turbopack/cjs/react-server-dom-turbopack-client.browser.development.js +3.96% 130.57 kB 135.75 kB +2.31% 24.04 kB 24.60 kB
oss-experimental/react-server-dom-webpack/cjs/react-server-dom-webpack-client.browser.development.js +3.95% 131.13 kB 136.31 kB +2.27% 24.19 kB 24.74 kB
oss-experimental/react-server-dom-esm/cjs/react-server-dom-esm-client.node.development.js +3.91% 132.38 kB 137.55 kB +2.45% 24.23 kB 24.82 kB
oss-experimental/react-server-dom-parcel/cjs/react-server-dom-parcel-client.edge.development.js +3.87% 133.66 kB 138.84 kB +2.30% 24.59 kB 25.15 kB
oss-experimental/react-server-dom-parcel/cjs/react-server-dom-parcel-client.node.development.js +3.83% 134.92 kB 140.09 kB +2.29% 24.70 kB 25.27 kB
oss-experimental/react-server-dom-turbopack/cjs/react-server-dom-turbopack-client.edge.development.js +3.78% 136.76 kB 141.94 kB +2.23% 25.07 kB 25.63 kB
oss-experimental/react-server-dom-webpack/cjs/react-server-dom-webpack-client.edge.development.js +3.78% 136.85 kB 142.03 kB +2.23% 25.11 kB 25.67 kB
oss-experimental/react-server-dom-turbopack/cjs/react-server-dom-turbopack-client.node.development.js +3.75% 137.85 kB 143.02 kB +2.25% 25.20 kB 25.77 kB
oss-experimental/react-server-dom-webpack/cjs/react-server-dom-webpack-client.node.unbundled.development.js +3.69% 140.20 kB 145.37 kB +2.23% 25.44 kB 26.01 kB
oss-experimental/react-server-dom-webpack/cjs/react-server-dom-webpack-client.node.development.js +3.66% 141.53 kB 146.71 kB +2.21% 25.69 kB 26.26 kB
oss-experimental/react-server-dom-esm/esm/react-server-dom-esm-client.browser.development.js +2.33% 165.82 kB 169.69 kB +1.11% 37.90 kB 38.32 kB
oss-experimental/react-markup/cjs/react-markup.react-server.development.js +0.86% 609.34 kB 614.56 kB +0.50% 108.61 kB 109.15 kB

Generated by 🚫 dangerJS against fb07c57

Co-authored-by: Hendrik Liebau <mail@hendrik-liebau.de>
@sebmarkbage sebmarkbage merged commit bb6c9d5 into facebook:main Jun 25, 2025
15 of 17 checks passed
github-actions bot pushed a commit that referenced this pull request Jun 25, 2025
<img width="926" alt="Screenshot 2025-06-25 at 1 02 14 PM"
src="https://github.com/user-attachments/assets/1877d13d-5259-4cc4-8f48-12981e3073fe"
/>

The I/O entry doesn't show as aborted in the Server Request track
because technically it wasn't. The end time is just made up. It's still
going. It's not aborted until the abort signal propagates and if we do
get that signal wired up before it emits, it instead would show up as
rejected.

---------

Co-authored-by: Hendrik Liebau <mail@hendrik-liebau.de>

DiffTrain build for [bb6c9d5](bb6c9d5)
github-actions bot pushed a commit that referenced this pull request Jun 25, 2025
<img width="926" alt="Screenshot 2025-06-25 at 1 02 14 PM"
src="https://github.com/user-attachments/assets/1877d13d-5259-4cc4-8f48-12981e3073fe"
/>

The I/O entry doesn't show as aborted in the Server Request track
because technically it wasn't. The end time is just made up. It's still
going. It's not aborted until the abort signal propagates and if we do
get that signal wired up before it emits, it instead would show up as
rejected.

---------

Co-authored-by: Hendrik Liebau <mail@hendrik-liebau.de>

DiffTrain build for [bb6c9d5](bb6c9d5)
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