Skip to content

Add space and container sync#13598

Open
kristijanribaric wants to merge 18 commits into
zen-browser:space-syncfrom
kristijanribaric:add-space-and-container-sync
Open

Add space and container sync#13598
kristijanribaric wants to merge 18 commits into
zen-browser:space-syncfrom
kristijanribaric:add-space-and-container-sync

Conversation

@kristijanribaric
Copy link
Copy Markdown
Member

  • Created sync engine for syncing workspaces and containers
  • patched ContextualIdentityService to allow creating containers with specific IDs

… sys patch files

update moz.build to include sync dependency
modify session manager to seed and notify sidebar data changes
implement _applySyncChanges to reconcile workspace cache with pulled data
add methods to normalize and order workspaces by position
update workspace reordering to notify observers only on position change
add space and container sync
…etSidebarData in sync manager to clarify data retrieval
@dosubot dosubot Bot added size:XL This PR changes 500-999 lines, ignoring generated files. Feature labels May 7, 2026
@mr-cheffy mr-cheffy requested a review from Copilot May 7, 2026 16:53

This comment was marked as resolved.

Comment thread src/zen/sessionstore/ZenSessionManager.sys.mjs Outdated
Comment thread src/zen/sessionstore/ZenSessionManager.sys.mjs Outdated
Comment thread src/zen/sync/ZenWorkspacesSync.sys.mjs Outdated
Comment thread src/zen/sync/ZenSyncManager.sys.mjs Outdated
Comment thread src/zen/sync/ZenSyncManager.sys.mjs Outdated
@kristijanribaric kristijanribaric force-pushed the add-space-and-container-sync branch from d703d00 to 0bb8095 Compare May 7, 2026 19:22
Comment thread src/zen/sync/ZenSyncManager.sys.mjs Outdated
@kristijanribaric kristijanribaric requested a review from mr-cheffy May 8, 2026 09:11
Comment thread src/zen/common/emojis/ZenEmojiPicker.mjs
Comment thread src/zen/sync/ZenWorkspacesSync.sys.mjs Outdated
Comment thread src/zen/sync/ZenWorkspacesSync.sys.mjs Outdated
update emoji icon list formatting in ZenEmojiPicker.mjs
remove seedSnapshot call in ZenSessionManager.sys.mjs
add workspace change notifications in ZenSpaceManager.mjs
refactor workspace reordering observer notifications in ZenSpaceManager.mjs
update sidebar data handling in ZenSyncManager.sys.mjs
modify getAllIDs and existence check in ZenWorkspacesSync.sys.mjs
@kristijanribaric kristijanribaric force-pushed the add-space-and-container-sync branch from 930c515 to 160b88c Compare May 10, 2026 16:08
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 6 comments.

Comment thread src/zen/sync/ZenWorkspacesSync.sys.mjs
Comment thread src/zen/sync/ZenWorkspacesSync.sys.mjs
Comment thread src/zen/sync/ZenWorkspacesSync.sys.mjs Outdated
Comment thread src/zen/sync/ZenSyncManager.sys.mjs Outdated
Comment thread src/zen/spaces/ZenSpaceManager.mjs Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature size:XL This PR changes 500-999 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants