0xMiden / 0xMiden/rust-sdk

Sync makes unnecesary storage and vault sync

Open
#2,217 0 comments 1 reaction 0 assignees View on GitHub
enhancement
Dominant language
Rust
Stars
78
Forks
129
Avg merge
4d 11h
Merged PRs (30d)
52

Description

The sync should handle vault and storage-map oversize independently. Today any oversize flag routes the whole account through the incremental delta path, which always fetches both `sync_storage_maps` and `sync_account_vault`, even if not needed.

https://github.com/0xMiden/miden-client/blob/0113b1cd88e3fd88eab46e26f326df92f7426728/crates/rust-client/src/sync/state_sync.rs#L676-L682

Contributor guide

Open the contributing guide

Research direction

The issue points to lines 676-682 in crates/rust-client/src/sync/state_sync.rs. Start by reading the sync logic around those lines to understand how oversize flags trigger incremental delta paths. Examine the functions sync_storage_maps and sync_account_vault to see when each is needed. The fix is to decouple the flags so only the necessary sync runs. Run existing sync tests to verify the change doesn't break anything.

Written by the indexing model from the issue text.

Assessment

Domain
backend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Clearly specified
Newbie friendliness
65/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.