Sync makes unnecesary storage and vault sync
- 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
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