Skip to content

perf(cache): remove eager response store tag index - #3211

Closed
james-elicx wants to merge 1 commit into
codex/response-store-do-put-protocolfrom
codex/response-store-tag-index-efficiency
Closed

perf(cache): remove eager response store tag index#3211
james-elicx wants to merge 1 commit into
codex/response-store-do-put-protocolfrom
codex/response-store-tag-index-efficiency

Conversation

@james-elicx

Copy link
Copy Markdown
Member

Summary

  • stop writing one SQLite row for every cache tag on every publication
  • select tagged entries from stored metadata only during explicit refresh and purge mutations
  • retain tag invalidation timestamps for soft-tag correctness
  • preserve the old tables for safe rollback compatibility without backfilling or writing them

Validation

  • vp check 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:18
@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@3211

create-vinext-app

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

@vinext/types

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

vinext

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

@vinext/workers-response-store

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

commit: 24c93c4

@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 24c93c4 against base 63d8feb using alternating same-runner rounds. Next.js was unchanged and skipped.

1 improved · 0 regressed · 5 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.39 s 3.31 s 🟢 -2.5%
Production build time vinext 3.73 s 3.70 s ⚫ -0.8%
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