Skip to content

feat!: kv-store on SQLite-wasm over OPFS (backport #22658)#22665

Merged
mverzilli merged 3 commits intobackport-to-v4-next-stagingfrom
claudebox/backport-22658-v4-next
Apr 20, 2026
Merged

feat!: kv-store on SQLite-wasm over OPFS (backport #22658)#22665
mverzilli merged 3 commits intobackport-to-v4-next-stagingfrom
claudebox/backport-22658-v4-next

Conversation

@AztecBot
Copy link
Copy Markdown
Collaborator

Backport of #22658 to v4-next.

Original PR summary

Experimental support for SQLite as a backend to kv-store, with the goal of eventually abandoning IndexedDB.

Conflicts

Single conflict in yarn-project/kv-store/package.json exports map:

  • v4-next keeps the ./config export (still imported by archiver, aztec-node, bot, node-lib, p2p, etc.) whereas that export had been removed on next.
  • Resolved by keeping all three exports: ./sqlite-opfs, ./stores, and ./config.

The rest of the cherry-pick (new sqlite-opfs/ module, wallet entrypoint wiring for walletDb.store, yarn.lock) applied cleanly.

Commits

  1. cherry-pick: ...(with conflicts) — original squash commit applied with the conflict marker preserved in package.json.
  2. fix: resolve cherry-pick conflicts — keeps the ./config export alongside the new ./sqlite-opfs entry.

No additional build-adaptation commit was required.

ClaudeBox log: https://claudebox.work/s/3c079f039c55d141?run=1

@AztecBot AztecBot added ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR. labels Apr 20, 2026
@socket-security
Copy link
Copy Markdown

socket-security Bot commented Apr 20, 2026

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addednpm/​@​sqlite.org/​sqlite-wasm@​3.50.4-build19810010094100

View full report

@mverzilli mverzilli marked this pull request as ready for review April 20, 2026 15:02
@mverzilli mverzilli enabled auto-merge April 20, 2026 15:02
The v4-next-staging line moved DataStoreConfig out of @aztec/stdlib/kv-store
into kv-store's own ./config subpath (the new './config' export in
package.json). Mirror the indexeddb backend's local import so sqlite-opfs
compiles on the backport branch.

Port-specific: on next/merge-train/fairies this import is still
@aztec/stdlib/kv-store.
@AztecBot
Copy link
Copy Markdown
Collaborator Author

Flakey Tests

🤖 says: This CI run detected 1 tests that failed, but were tolerated due to a .test_patterns.yml entry.

\033FLAKED\033 (8;;http://ci.aztec-labs.com/fc35ea524fa95ab2�fc35ea524fa95ab28;;�):  yarn-project/end-to-end/scripts/run_test.sh simple src/e2e_epochs/epochs_l1_reorgs.parallel.test.ts "updates L1 to L2 messages changed due to an L1 reorg" (66s) (code: 0) group:e2e-p2p-epoch-flakes

@mverzilli mverzilli merged commit d3fb022 into backport-to-v4-next-staging Apr 20, 2026
9 checks passed
@mverzilli mverzilli deleted the claudebox/backport-22658-v4-next branch April 20, 2026 17:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants