Skip to content

Fix sync ordering for dependent resources#154

Merged
ahmedre merged 1 commit intomainfrom
sync_improvements
May 5, 2026
Merged

Fix sync ordering for dependent resources#154
ahmedre merged 1 commit intomainfrom
sync_improvements

Conversation

@ahmedre
Copy link
Copy Markdown
Contributor

@ahmedre ahmedre commented May 4, 2026

Build and apply sync plans in dependency-aware phases so bookmarks and
collections are synced before collection-bookmark links are planned.
This lets collection-bookmark links use the remote IDs returned for
their dependencies in the same sync run.

Defer shared sync-token publication until all phases complete, and cap
the published token at the initial fetch watermark so changes made
remotely during phase POSTs are picked up by the next sync.

Build and apply sync plans in dependency-aware phases so bookmarks and
collections are synced before collection-bookmark links are planned.
This lets collection-bookmark links use the remote IDs returned for
their dependencies in the same sync run.

Defer shared sync-token publication until all phases complete, and cap
the published token at the initial fetch watermark so changes made
remotely during phase POSTs are picked up by the next sync.
@ahmedre ahmedre merged commit 366a47f into main May 5, 2026
1 check passed
@ahmedre ahmedre deleted the sync_improvements branch May 5, 2026 21:01
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