Skip to content

feat: integrate connector-based storage API#87

Merged
prestwich merged 1 commit intodevelopfrom
feat/connector-based-storage
Feb 18, 2026
Merged

feat: integrate connector-based storage API#87
prestwich merged 1 commit intodevelopfrom
feat/connector-based-storage

Conversation

@prestwich
Copy link
Member

Summary

  • Replace manual MDBX backend construction in StorageConfig::build_storage() with the new connector-based StorageBuilder from signet-storage 0.6.4
  • build_storage() is now async, using MdbxConnector for hot and cold MDBX backends
  • Add cold_sql_url field to StorageConfig with SIGNET_COLD_SQL_URL env var support
  • Add postgres and sqlite feature flags that enable SQL cold storage via SqlConnector
  • Remove direct dependencies on signet-cold, signet-cold-mdbx, signet-hot-mdbx (all re-exported from signet-storage)

Closes ENG-1911

Test plan

  • cargo clippy -p signet-node-config --all-targets — clean
  • cargo clippy -p signet-node-config --all-features --all-targets — clean
  • cargo clippy -p signet-node-config --no-default-features --all-targets — clean
  • cargo +nightly fmt — clean
  • cargo t -p signet-node-config — 2 unit tests + 1 doc test pass

🤖 Generated with Claude Code

Replace manual MDBX backend construction with the new connector-based
StorageBuilder from signet-storage 0.6.4. build_storage() is now async
and uses MdbxConnector/SqlConnector, with feature-gated postgres/sqlite
support for SQL cold backends via SIGNET_COLD_SQL_URL.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@prestwich prestwich requested a review from a team as a code owner February 18, 2026 14:00
@prestwich prestwich enabled auto-merge (squash) February 18, 2026 21:29
@prestwich prestwich merged commit a2e43de into develop Feb 18, 2026
7 of 12 checks passed
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.

3 participants

Comments