Skip to content

test(cloudflare): preserve independent RSC warm proof - #3020

Open
james-elicx wants to merge 2 commits into
codex/rsc-prewarm-canonical-dedupefrom
codex/rsc-prewarm-e2e-proof
Open

test(cloudflare): preserve independent RSC warm proof#3020
james-elicx wants to merge 2 commits into
codex/rsc-prewarm-canonical-dedupefrom
codex/rsc-prewarm-e2e-proof

Conversation

@james-elicx

Copy link
Copy Markdown
Member

Summary

  • make promotion readiness poll only the unique no-store version endpoint
  • include both the app build ID and RSC build ID in that endpoint
  • leave both canonical RSC cache keys untouched until their first HIT assertions
  • retry only responses positively identified as the stale seed Worker; current-build MISS, BYPASS, wrong content, or wrong build still fail immediately

Test plan

  • vp check examples/workers-cache/app/api/prewarm-version/route.ts tests/e2e/cloudflare-workers/rsc-prewarm.spec.ts
  • PLAYWRIGHT_PROJECT=cloudflare-workers vp exec playwright test tests/e2e/cloudflare-workers/rsc-prewarm.spec.ts --list
  • deployed Cloudflare proof in CI

Stacked on #3019.

@github-actions

github-actions Bot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Performance benchmarks

Compared 9d352a2 against base d7f6655 using alternating same-runner rounds. Next.js was unchanged and skipped.

1 improved · 0 regressed · 5 within ±1.5%

Scenario Framework Baseline Current Change
Client bundle size (gzip) vinext 142.0 KB 142.0 KB ⚫ -0.0%
Client entry size (gzip) vinext 129.3 KB 129.3 KB ⚫ -0.0%
Dev server cold start vinext 3.27 s 3.18 s 🟢 -2.7%
Production build time vinext 3.40 s 3.37 s ⚫ -0.8%
RSC entry closure size (gzip) vinext 116.0 KB 116.0 KB ⚫ 0.0%
Server bundle size (gzip) vinext 197.5 KB 197.5 KB ⚫ -0.0%

View detailed results and traces

🟢 improvement · 🔴 regression · ⚫ change below 1.5% · paired base/head

@pkg-pr-new

pkg-pr-new Bot commented Aug 21, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@vinext/cloudflare@3020
npm i https://pkg.pr.new/create-vinext-app@3020
npm i https://pkg.pr.new/@vinext/types@3020
npm i https://pkg.pr.new/vinext@3020

commit: 9d352a2

@github-actions

Copy link
Copy Markdown
Contributor
Example Preview Production Original
app-router-cloudflare preview production
pages-router-cloudflare preview production
app-router-playground preview production original
realworld-api-rest preview production
nextra-docs-template preview production
benchmarks preview production
hackernews preview production original
workers-cache preview production
web preview production

Sign up for free to 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.

1 participant