feat(cloudflare): support cf Build Output deployments - #3230
Conversation
|
/bigbonk review for issues |
@vinext/cloudflare
create-vinext-app
@vinext/types
vinext
@cloudflare/workers-response-store
commit: |
|
Performance benchmarksCompared 1 improved · 0 regressed · 5 within ±1.5%
View detailed results and traces 🟢 improvement · 🔴 regression · ⚫ change below 1.5% · paired base/head |
![ask-bonk[bot]](https://avatars.githubusercontent.com/in/2454580?s=60&v=4){kind=small}
|
Posted 1 inline finding. Validation: 456 focused tests passed; apps/web Build Output completed successfully. |
|
/bigbonk review for issues |
|
/bigbonk review for issues |
|
/bigbonk review for issues |
|
Posted 4 inline findings. Validation: 456 focused tests passed; apps/web Build Output and |
|
/bigbonk review for issues |
|
@james-elicx Bonk workflow was cancelled. View workflow run · To retry, trigger Bonk again. |
|
/bigbonk review for issues |
|
/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. |
|
Posted 1 actionable inline finding. |
|
/bigbonk review for issues Please review exact head |
|
Posted 1 actionable inline finding. |
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| ); | ||
| writeFileSync( | ||
| path.join(plugin, "config.js"), | ||
| `export const bindings = { versionMetadata: () => ({ version: ${JSON.stringify(version)} }) }; |
| writeFileSync( | ||
| path.join(plugin, "config.js"), | ||
| `export const bindings = { versionMetadata: () => ({ version: ${JSON.stringify(version)} }) }; | ||
| export const exports = { worker: () => ({ version: ${JSON.stringify(version)} }) };`, |
2 participants
Summary
Add an optional
cfdeployment path for vinext projects usingcloudflare.config.tsand Cloudflare Vite plugin v2. Existing Wrangler projects keep their current deployment flow.apps/webusescf@1.0.0-beta.1and Cloudflare Vite plugin2.0.0-beta.sha-805ec1ff3, andvinext init --experimental-cfgenerates the typed configuration and matching dependencies/scripts for new projects.cf.@vinext/cloudflare; deploy named service-binding Workers with nativecf deploy --workerbefore the application Worker.cfcommands, including the defaultproductionmode required for prebuilt deployments..cloudflare/types/during install, dev, and build; keep them out of the PR. Select CI preview tooling by the presence ofcloudflare.config.ts.Validation
202 targeted init and typed-configuration tests passed with the updated Vite plugin. Removing generated types and running a frozen install recreates them;
apps/webtype-checks against both CLI-generated and build-generated declarations.647 targeted init, deployment, RSC-plugin, and multi-stage build tests passed after merging current main.
All five generated init configurations build: App and Pages service-binding mode, self-contained Response Store, Workers Cache, and Pages without CDN caching.
Full
apps/webbuild produces both the application and Response Store Workers; both passcfdeploy dry-runs.Live Workers Cache deployment with the beta CLI completed discovery, manifest generation, final version upload, HTML/RSC warming (2/2), and promotion. Repeated HTML and RSC requests returned cache hits with stable response hashes; the dynamic control route bypassed cache and returned changing content.
Frozen-lockfile installation and targeted formatting, lint, and type checks passed.