Skip to content

feat(cloudflare): support cf Build Output deployments - #3230

Draft
james-elicx wants to merge 14 commits into
mainfrom
codex/apps-web-cf-v2
Draft

feat(cloudflare): support cf Build Output deployments#3230
james-elicx wants to merge 14 commits into
mainfrom
codex/apps-web-cf-v2

Conversation

@james-elicx

@james-elicx james-elicx commented Sep 11, 2026

Copy link
Copy Markdown
Member

Summary

  • add Cloudflare Build Output support to the vinext deployment flow while preserving staged CDN warming and promotion
  • migrate apps/web to the Cloudflare Vite plugin v2 prerelease and the cf CLI
  • expose typed self-contained and service-binding Workers Response Store config helpers, with matching runtime factory names and compatibility aliases for the existing APIs
  • generate and deploy the apps/web Response Store service-binding Worker from cloudflare.config.ts, without a second Wrangler config
  • deploy named auxiliary Build Output Workers with cf before the application Worker
  • use generated Worker binding/runtime types and select preview deployment tooling from cloudflare.config.ts

Validation

  • Workers Response Store package suite: 46 tests
  • Cloudflare Response Store adapter suite: 30 tests
  • deploy and RSC plugin suites: 366 tests
  • repository and targeted formatting, lint, and type checks
  • apps/web TypeScript check
  • frozen-lockfile install validation
  • full apps/web cf build with Vite 8.3; default app Worker plus 7-module Response Store service-binding Worker
  • configless cf deploy --prebuilt --dry-run
  • live cf preview deployment with Response Store MISS to HIT and matching response hashes

@james-elicx

Copy link
Copy Markdown
Member Author

/bigbonk review for issues

@pkg-pr-new

pkg-pr-new Bot commented Sep 11, 2026

Copy link
Copy Markdown

Open in StackBlitz

@vinext/cloudflare

npm i https://pkg.pr.new/cloudflare/vinext/@vinext/cloudflare@3230

create-vinext-app

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

@vinext/types

npm i https://pkg.pr.new/cloudflare/vinext/@vinext/types@3230

vinext

npm i https://pkg.pr.new/cloudflare/vinext@3230

@cloudflare/workers-response-store

npm i https://pkg.pr.new/cloudflare/vinext/@cloudflare/workers-response-store@3230

commit: 652a743

@github-actions

github-actions Bot commented Sep 11, 2026

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
response-store-demo preview production
static-export preview production
web preview production

@github-actions

github-actions Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Performance benchmarks

Compared 652a743 against base 5dd43c0 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.6 KB 129.6 KB ⚫ +0.0%
Dev server cold start vinext 3.12 s 3.10 s ⚫ -0.6%
Production build time vinext 3.43 s 3.41 s ⚫ -0.4%
RSC entry closure size (gzip) vinext 121.0 KB 121.0 KB ⚫ +0.0%
Server bundle size (gzip) vinext 211.0 KB 211.0 KB ⚫ +0.0%

View detailed results and traces

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

Comment thread packages/cloudflare/src/deploy.ts
@ask-bonk

ask-bonk Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Posted 1 inline finding.

Validation: 456 focused tests passed; apps/web Build Output completed successfully.

github run

@james-elicx

Copy link
Copy Markdown
Member Author

/bigbonk review for issues

@james-elicx

Copy link
Copy Markdown
Member Author

/bigbonk review for issues

@james-elicx

Copy link
Copy Markdown
Member Author

/bigbonk review for issues

Comment thread packages/cloudflare/src/deploy.ts
Comment thread packages/cloudflare/src/deploy.ts Outdated
Comment thread packages/cloudflare/src/deploy.ts Outdated
Comment thread packages/cloudflare/src/deploy.ts
@ask-bonk

ask-bonk Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Posted 4 inline findings.

Validation: 456 focused tests passed; apps/web Build Output and cf --prebuilt --dry-run succeeded.

github run

@james-elicx

Copy link
Copy Markdown
Member Author

/bigbonk review for issues

@ask-bonk

ask-bonk Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

@james-elicx Bonk workflow was cancelled.

View workflow run · To retry, trigger Bonk again.

@james-elicx

Copy link
Copy Markdown
Member Author

/bigbonk review for issues

@james-elicx

Copy link
Copy Markdown
Member Author

/bigbonk review for issues. The workflow has a hard 10-minute timeout including setup. Prioritize actionable correctness findings, stop analysis early enough to post the verdict, and return the best completed review within the available time even if coverage is not exhaustive.

Comment thread packages/cloudflare/src/version-deploy.ts
@ask-bonk

ask-bonk Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Posted 1 actionable inline finding.

github run

@james-elicx

Copy link
Copy Markdown
Member Author

/bigbonk review for issues

Please review exact head 632dfb83e0f0ff74030c0633b85579123426da73. You have a 10-minute time limit; please post the most important actionable findings you can complete within that window rather than timing out without a result.

Comment thread packages/cloudflare/src/deploy.ts Outdated
@ask-bonk

ask-bonk Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Posted 1 actionable inline finding.

github run

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