Skip to content

[bench skill] Bench Edge builds of React too - #96234

Merged
gaearon merged 1 commit into
canaryfrom
bench/sandbox-bench-ssr
Jul 30, 2026
Merged

[bench skill] Bench Edge builds of React too#96234
gaearon merged 1 commit into
canaryfrom
bench/sandbox-bench-ssr

Conversation

@gaearon

Copy link
Copy Markdown
Contributor

Extends the bench to run the edge versions of React SSR fixture.

@github-actions

github-actionsBot commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

Stats skipped

Commit: 94cf70e
View workflow run

@github-actions

github-actionsBot commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

Tests Passed

Commit: 94cf70e

The e2e suite only measures the Node path through a Next.js app.
sandbox-ssr.mjs adds the react repo's flight-ssr-bench fixture as a
second suite: 8 variants (Fizz and Flight+Fizz x Node and Edge web
streams x sync/async), sequential-with-injection plus an HTTP server
at c=1/c=10, paired ABBA in the same VM, CPU profiles after the timed
runs, same results db and boot-level stats. The fixture is pinned to
one ref for both arms; PR edits to it are flagged, not silently
benched. React PRs run both suites.
bench-common.mjs carries the launcher-independent core (sandbox
plumbing, PR/arm resolution with the CI gate, cached react arm builds,
snapshots, recovery record, live estimates); sandbox-e2e's dry-run
verified byte-identical across the extraction. sbExec now verifies
the remote exit itself via an EXIT-trap marker (the CLI can return 0
after a remote failure — observed live). A/A validated with a two-run
criterion scaled to suite size (methodology.md).
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@gaearon
gaearonforce-pushed the bench/sandbox-bench-ssr branch from c0f5e46 to 94cf70eCompareJuly 25, 2026 23:43
gaearon added a commit to react/react that referenced this pull request Jul 26, 2026
Things I found while integrating this bench into
vercel/next.js#96234.
See code.
---------
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
@gaearon
gaearon marked this pull request as ready for review July 30, 2026 18:30
@gaearon
gaearon requested a review from timneutkensJuly 30, 2026 18:31
@gaearon
gaearon merged commit c53c0a5 into canaryJul 30, 2026
68 checks passed
@gaearon
gaearon deleted the bench/sandbox-bench-ssr branch July 30, 2026 18:55
m-kawafuji pushed a commit to m-kawafuji/next.js that referenced this pull request Aug 8, 2026
Extends the bench to run the edge versions of React SSR fixture.
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@gaearon@timneutkens