Skip to content

Conversation

@m-kus
Copy link
Member

@m-kus m-kus commented Sep 30, 2025

Replace App Server with Direct MMR Access

This PR refactors the bridge node architecture by removing the App Server layer and allowing the RPC server to directly access the MMR. Key changes:

  • Removed the app.rs module and its message-passing architecture
  • Updated the RPC server to directly instantiate and use the MMR
  • Added #[derive(Debug, Clone)] to BitcoinClient for easier sharing
  • Added a new get_block_height method to BlockMMRStore
  • Made the MMR ID configurable via CLI
  • Updated the Cargo.toml to use the m-kus fork of rust-accumulators
  • Removed the dependency on raito-spv-client

This simplifies the codebase by eliminating an unnecessary abstraction layer while maintaining the same functionality.

@m-kus m-kus mentioned this pull request Sep 30, 2025
Copy link
Member Author

m-kus commented Sep 30, 2025

@m-kus m-kus force-pushed the 09-30-thread_safe_mmr branch from 4383c3f to 76e9976 Compare September 30, 2025 10:57
@m-kus m-kus marked this pull request as ready for review September 30, 2025 10:57
@m-kus m-kus requested a review from maciejka as a code owner September 30, 2025 10:57
@m-kus m-kus force-pushed the 09-30-thread_safe_mmr branch from 76e9976 to 395c39d Compare September 30, 2025 11:00
@m-kus m-kus force-pushed the 09-29-expose_endpoint_to_fetch_multiple_block_headers branch from b1ec240 to a24ad49 Compare September 30, 2025 13:04
@m-kus m-kus force-pushed the 09-30-thread_safe_mmr branch 2 times, most recently from 4348b88 to 4899d0a Compare October 3, 2025 16:16
@m-kus m-kus force-pushed the 09-29-expose_endpoint_to_fetch_multiple_block_headers branch from a24ad49 to 228385b Compare October 3, 2025 16:16
@m-kus m-kus force-pushed the 09-30-thread_safe_mmr branch from 4899d0a to 6de9d4a Compare October 3, 2025 16:19
@m-kus m-kus force-pushed the 09-29-expose_endpoint_to_fetch_multiple_block_headers branch from 228385b to 4be00c7 Compare October 3, 2025 16:19
@m-kus m-kus mentioned this pull request Oct 3, 2025
@m-kus m-kus force-pushed the 09-29-expose_endpoint_to_fetch_multiple_block_headers branch from 4be00c7 to fa33177 Compare October 6, 2025 13:27
@m-kus m-kus force-pushed the 09-30-thread_safe_mmr branch from 6de9d4a to 907f75f Compare October 6, 2025 13:27
@m-kus m-kus force-pushed the 09-30-thread_safe_mmr branch from 907f75f to 4a0b9a3 Compare October 6, 2025 20:37
@m-kus m-kus force-pushed the 09-29-expose_endpoint_to_fetch_multiple_block_headers branch 2 times, most recently from e618821 to fe06826 Compare October 6, 2025 20:48
@m-kus m-kus force-pushed the 09-30-thread_safe_mmr branch from 4a0b9a3 to 5cb9acf Compare October 6, 2025 20:48
Base automatically changed from 09-29-expose_endpoint_to_fetch_multiple_block_headers to main October 6, 2025 21:28
@m-kus m-kus force-pushed the 09-30-thread_safe_mmr branch from 5cb9acf to 80547bc Compare October 6, 2025 21:29
@m-kus m-kus merged commit e26b9e6 into main Oct 6, 2025
9 checks passed
@m-kus m-kus deleted the 09-30-thread_safe_mmr branch October 6, 2025 21:37
m-kus added a commit that referenced this pull request Oct 8, 2025
maciejka pushed a commit that referenced this pull request Oct 10, 2025
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.

2 participants