Skip to content

Refactor cloud sync state ownership#992

Draft
ryokun6 wants to merge 3 commits intomainfrom
cursor/cloud-sync-engine-a056
Draft

Refactor cloud sync state ownership#992
ryokun6 wants to merge 3 commits intomainfrom
cursor/cloud-sync-engine-a056

Conversation

@ryokun6
Copy link
Copy Markdown
Owner

@ryokun6 ryokun6 commented Mar 16, 2026

Summary

  • extract the remaining cloud sync domains into src/utils/sync/engine/domains, including files metadata, individual blob domains, songs, videos, stickies, and calendar
  • keep cloudSync.ts as orchestration by delegating snapshot serialize/apply/merge behavior through the new engine modules
  • keep the single persisted sync metadata owner introduced earlier and preserve the existing logical/physical sync payload contracts

Testing

  • bun test tests/test-cloud-sync-utils.test.ts tests/test-cloud-sync-launch-and-store.test.ts
  • bun test tests/test-cloud-sync-domains.test.ts tests/test-sync-logical-domains.test.ts
  • bun run build
  • manual two-tab cloud sync verification on http://localhost:3001 using ryo / testtest: changed theme in tab 1 and confirmed it in tab 2 after refresh; created a new TextEdit document in tab 1 and confirmed it appeared in tab 2's Documents folder
Open in Web Open in Cursor 

cursoragent and others added 2 commits March 16, 2026 10:34
Co-authored-by: Ryo Lu <me@ryo.lu>
Co-authored-by: Ryo Lu <me@ryo.lu>
@ryos-deploy
Copy link
Copy Markdown

ryos-deploy bot commented Mar 16, 2026

The preview deployment for ryos-dev is ready. 🟢

Open Preview | Open Build Logs | Open Application Logs

Last updated at: 2026-03-16 15:14:19 CET

Co-authored-by: Ryo Lu <me@ryo.lu>
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