feat(cloudflare): automate Workers Response Store deployment - #3248
james-elicx wants to merge 20 commits into
Conversation
@vinext/cloudflare
create-vinext-app
@vinext/types
vinext
@cloudflare/workers-response-store
commit: |
|
Performance benchmarksCompared 0 improved · 0 regressed · 6 within ±1.5%
View detailed results and traces 🟢 improvement · 🔴 regression · ⚫ change below 1.5% · paired base/head |
|
/bigbonk review for issues |
|
/bigbonk review for issues |
|
/bigbonk review for issues |
|
@james-elicx Bonk workflow was cancelled. View workflow run · To retry, trigger Bonk again. |
|
/bigbonk review for in-scope correctness bugs only. You have an 8 minute time limit; finish and post your findings within 8 minutes. |
![ask-bonk[bot]](https://avatars.githubusercontent.com/in/2454580?s=60&v=4){kind=small}
|
Posted 2 inline correctness findings. |
|
/bigbonk review for in-scope correctness bugs only. You have an 8 minute time limit; finish and post your findings within 8 minutes. |
|
Posted 2 inline correctness findings. |
|
/bigbonk review for in-scope correctness bugs only. Review the latest head 7e547a5. You have an 8 minute time limit; finish and post your findings within 8 minutes. |
|
Posted 1 inline correctness finding. |
|
/bigbonk review for in-scope correctness bugs only. Review the latest head 0394a5f. You have an 8 minute time limit; finish and post your findings within 8 minutes. |
|
Posted 4 inline correctness findings. |
|
/bigbonk review for in-scope correctness bugs only. Review latest head 9183dd7. Hard 8 minute time limit: stop analysis and post your findings before 8 minutes elapse. |
|
Posted 2 inline correctness findings. |
|
/bigbonk review for in-scope correctness bugs only. Review latest head 2ab45b6. Hard 8 minute time limit: stop analysis and post your findings before 8 minutes elapse. |
|
Posted 2 inline correctness findings. Targeted tests passed: 368 tests. |
|
Posted 2 inline correctness findings. Targeted tests passed: 370 tests. |
|
/bigbonk re-review only the fixes for your prior comments in commit 7666376. Do not review the CI-only test commits. Hard 8 minute time limit: stop analysis and post findings before 8 minutes elapse. |
|
LGTM! |
|
Superseded by #3249, which implements the config-first design from a clean branch. |
1 participant
Keeps caching opt-in for generated Cloudflare apps, while making Workers Response Store the default choice after users enable caching and making service-binding mode a one-command deployment.\n\n- generates a private Response Store Worker, R2 binding, SQLite Durable Object migration, app service binding, and version metadata\n- deploys the generated cache Worker before the application through vinext-cloudflare deploy\n- supports stable custom service and R2 names, plus binding a separately managed compatible service\n- adds an explicit cache opt-in to vinext init and create-vinext-app, with Response Store preselected\n- migrates apps/web from Workers Cache plus KV to its own vinext-web-response-store\n- exercises generated service artifacts in local E2E and deployment CI\n\nValidation:\n- 490 focused unit tests\n- 30 @vinext/cloudflare tests, including service-binding and self-contained E2E\n- targeted format, lint, and type checks\n- actionlint for modified deployment workflows\n- response-store-demo live one-command deployment with service, R2, and Durable Object reuse\n- apps/web production build plus Wrangler dry-runs for both generated Workers