Skip to content

refactor(cache): reduce response store metadata calls - #3210

Closed
james-elicx wants to merge 1 commit into
codex/response-store-html-identityfrom
codex/response-store-do-put-protocol
Closed

refactor(cache): reduce response store metadata calls#3210
james-elicx wants to merge 1 commit into
codex/response-store-html-identityfrom
codex/response-store-do-put-protocol

Conversation

@james-elicx

Copy link
Copy Markdown
Member

Summary

  • collapse response-body writes into one reservation RPC and one publication RPC
  • retain pending-object rows as the durable R2 object registry
  • move orphan cleanup from every put to the metadata Durable Object alarm
  • return the published entry from the publication transaction

Validation

  • vp check packages/workers-response-store/src/binding.ts packages/workers-response-store/src/metadata-do.ts packages/workers-response-store/tests/e2e.test.mjs
  • pnpm --filter @vinext/workers-response-store run build:examples
  • pnpm --filter @vinext/workers-response-store exec vitest run tests/e2e.test.mjs tests/service-binding-e2e.test.mjs

@james-elicx
james-elicx added this pull request to stack #3195 September 10, 2026 10:15
@pkg-pr-new

pkg-pr-new Bot commented Sep 10, 2026

Copy link
Copy Markdown

Open in StackBlitz

@vinext/cloudflare

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

create-vinext-app

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

@vinext/types

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

vinext

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

@vinext/workers-response-store

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

commit: 63d8feb

@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
static-export preview production
web preview production

@github-actions

Copy link
Copy Markdown
Contributor

Performance benchmarks

Compared 63d8feb against base 13cbd88 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.24 s 3.22 s ⚫ -0.7%
Production build time vinext 3.66 s 3.61 s ⚫ -1.4%
RSC entry closure size (gzip) vinext 125.8 KB 125.8 KB ⚫ +0.0%
Server bundle size (gzip) vinext 216.5 KB 216.5 KB ⚫ +0.0%

View detailed results and traces

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

@james-elicx
james-elicx removed this pull request from stack #3195 September 10, 2026 10:31
@james-elicx

Copy link
Copy Markdown
Member Author

Superseded by #3213, which combines the related Durable Object efficiency changes into one PR above #3209.

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