Skip to content

fix(build): resolve all CDN warm path owners - #3049

Closed
james-elicx wants to merge 1 commit into
codex/rsc-prewarm-page-ownershipfrom
codex/rsc-prewarm-union-ownership
Closed

fix(build): resolve all CDN warm path owners#3049
james-elicx wants to merge 1 commit into
codex/rsc-prewarm-page-ownershipfrom
codex/rsc-prewarm-union-ownership

Conversation

@james-elicx

Copy link
Copy Markdown
Member

Summary

  • resolve runtime ownership across the complete App+Pages discovered path union
  • exclude a Pages-discovered key when its runtime winner is an App route handler
  • add full/loading RSC variants when a Pages-discovered key resolves to an App page
  • retain HTML warming when the runtime winner is a Pages page

Scope

This only filters and classifies build-discovered CDN warm keys. Runtime routing and handler behavior are unchanged.

Validation

  • 34 focused discovery tests passed
  • targeted format, lint, and type checks passed
@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@3049
npm i https://pkg.pr.new/create-vinext-app@3049
npm i https://pkg.pr.new/@vinext/types@3049
npm i https://pkg.pr.new/vinext@3049

commit: 25e23c9

@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 25e23c9 against base 414c6a1 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 3.19 s 3.14 s ⚫ -1.5%
Production build time vinext 3.67 s 3.62 s ⚫ -1.4%
RSC entry closure size (gzip) vinext 116.1 KB 116.1 KB ⚫ +0.0%
Server bundle size (gzip) vinext 197.6 KB 197.6 KB ⚫ +0.0%

View detailed results and traces

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

@james-elicx

Copy link
Copy Markdown
Member Author

Superseded by consolidated PR #3057. No commits were dropped or rewritten: #3057 now contains this exact linear commit as part of #3047#3057. 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

1 participant