Skip to content

test(cloudflare): prove deployed Pages CDN prewarming - #3041

Closed
james-elicx wants to merge 1 commit into
codex/rsc-prewarm-discovery-fail-closedfrom
codex/rsc-prewarm-pages-deployed-proof
Closed

test(cloudflare): prove deployed Pages CDN prewarming#3041
james-elicx wants to merge 1 commit into
codex/rsc-prewarm-discovery-fail-closedfrom
codex/rsc-prewarm-pages-deployed-proof

Conversation

@james-elicx

Copy link
Copy Markdown
Member

Summary

  • add one non-conflicting Pages ISR route to the existing hybrid workers-cache fixture
  • require the staged deploy warmup to discover and fill that HTML key
  • assert the first post-promotion Pages request is an untouched CDN HIT from the exact uploaded build before running the existing RSC browser proof

Scope

This reuses the existing Worker, seed deployment, strict warm command, production hostname, promotion readiness check, and Playwright job. It does not add a second deploy flow or change production behavior.

Validation

  • workers-cache hybrid App/Pages production build passed locally
  • 20 focused discovery tests passed
  • vp check passed
  • deployed strict warm/HIT proof runs in this PR CI

@james-elicx

Copy link
Copy Markdown
Member Author

/bigbonk review for issues

@pkg-pr-new

pkg-pr-new Bot commented Aug 22, 2026

Copy link
Copy Markdown

Open in StackBlitz

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

commit: e8079d2

@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

@github-actions

Copy link
Copy Markdown
Contributor

Performance benchmarks

Compared e8079d2 against base 22c8e57 using alternating same-runner rounds. Next.js was unchanged and skipped.

0 improved · 0 regressed · 6 within ±1.5%

Scenario Framework Baseline Current Change
Client bundle size (gzip) vinext 142.2 KB 142.2 KB ⚫ -0.0%
Client entry size (gzip) vinext 129.5 KB 129.5 KB ⚫ 0.0%
Dev server cold start vinext 2.38 s 2.37 s ⚫ -0.5%
Production build time vinext 2.60 s 2.61 s ⚫ +0.1%
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

@ask-bonk

ask-bonk Bot commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

LGTM!

github run

@james-elicx

Copy link
Copy Markdown
Member Author

Superseded by consolidated PR #3046. No commits were dropped or rewritten: #3046 now contains this exact linear commit as part of #3041#3046. Closing only the redundant draft PR.

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