Skip to content

fix(cloudflare): warm the deployed Worker route host - #3035

Closed
james-elicx wants to merge 1 commit into
codex/rsc-prewarm-cache-policy-retriesfrom
codex/rsc-prewarm-deploy-route-targets
Closed

fix(cloudflare): warm the deployed Worker route host#3035
james-elicx wants to merge 1 commit into
codex/rsc-prewarm-cache-policy-retriesfrom
codex/rsc-prewarm-deploy-route-targets

Conversation

@james-elicx

@james-elicx james-elicx commented Aug 22, 2026

Copy link
Copy Markdown
Member

Summary

  • derive the CDN warmup origin from the concrete catch-all Worker route reported by wrangler triggers deploy
  • prefer that exact deployed route over workers.dev fallback, including workers_dev: false, singular route, and { pattern, zone_name } route objects
  • preserve the declared HTTP/HTTPS scheme, reject wildcard or path-scoped routes that cannot provide one canonical site-wide warmup origin
  • do not infer warmup hosts from TPR's zone-oriented Wrangler config parser
  • describe Workers Caching tiered propagation accurately in deploy help
  • align the deploy-flow assertion with the preceding deterministic retry policy (no runtime retry change here)

This is stacked on #3034 and remains limited to RSC/HTML CDN prewarming target correctness. It does not change TPR or add Wrangler TOML parsing.

Validation

  • vp test run tests/deploy.test.ts tests/cloudflare-cdn-warm-deploy.test.ts (345 passed)
  • vp check packages/cloudflare/src/deploy-help.ts packages/cloudflare/src/deploy.ts packages/cloudflare/src/version-deploy.ts packages/cloudflare/src/worker-deployment-url.ts tests/cloudflare-cdn-warm-deploy.test.ts tests/deploy.test.ts
  • vp run vinext#build
  • vp run @vinext/cloudflare#build

Documentation checked

  • Cloudflare Workers route configuration and matching behavior
  • Wrangler 4.80 config schema for route, routes, pattern, and zone_name
  • Cloudflare Workers Caching tiered-cache behavior

@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@3035
npm i https://pkg.pr.new/create-vinext-app@3035
npm i https://pkg.pr.new/@vinext/types@3035
npm i https://pkg.pr.new/vinext@3035

commit: 58e25d2

@james-elicx
james-elicx force-pushed the codex/rsc-prewarm-deploy-route-targets branch from 020a03f to 58e25d2 Compare August 22, 2026 01:39
@github-actions

Copy link
Copy Markdown
Contributor

Performance benchmarks

Compared 58e25d2 against base be15b5e 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.18 s 3.17 s ⚫ -0.5%
Production build time vinext 3.64 s 3.63 s ⚫ -0.4%
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

@james-elicx

Copy link
Copy Markdown
Member Author

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