The cloud sync functionality in src/utils/workers/syncing/worker.ts is completely disabled with a TODO comment. The syncLocalWithCloud function currently returns a mock empty response instead of performing actual synchronization.
Location:
File: src/utils/workers/syncing/worker.ts
Lines: 367-378
Current behavior:
typescript
export const syncLocalWithCloud = (
lastSyncDateTimeUTC: Date,
currentTags: Record<string, Tag>,
fullSync: boolean = false,
progressCallback?: (progress: number) => void,
): Promise< => {
// TODO: re-enable syncing when ready
console.log('[Sync] Syncing disabled temporarily');
return Promise.resolve({
status: 'Sync' as SyncOptions,
miscIdsToRemove: [],
miscObjectsToRemove: [],
miscItemsToUpsert: [],
noteIdsToRemove: [],
notesToUpsert: [],
tagIdsToRemove: [],
tagsToUpsert: [],
});
return syncByLastModified( // <-- This code is never reached
Expected behavior: The function should call syncByLastModified (or the appropriate sync implementation) to actually synchronize local data with the cloud, enabling users to sync their notes across devices.
Impact:
Users cannot sync their notes across multiple devices
Cloud backup functionality is non-functional
This contradicts the README which mentions sync capabilities
Labels: bug, feature, priority: high
The cloud sync functionality in src/utils/workers/syncing/worker.ts is completely disabled with a TODO comment. The syncLocalWithCloud function currently returns a mock empty response instead of performing actual synchronization.
Location:
File: src/utils/workers/syncing/worker.ts
Lines: 367-378
Current behavior:
typescript
export const syncLocalWithCloud = (
lastSyncDateTimeUTC: Date,
currentTags: Record<string, Tag>,
fullSync: boolean = false,
progressCallback?: (progress: number) => void,
): Promise< => {
// TODO: re-enable syncing when ready
console.log('[Sync] Syncing disabled temporarily');
return Promise.resolve({
status: 'Sync' as SyncOptions,
miscIdsToRemove: [],
miscObjectsToRemove: [],
miscItemsToUpsert: [],
noteIdsToRemove: [],
notesToUpsert: [],
tagIdsToRemove: [],
tagsToUpsert: [],
});
return syncByLastModified( // <-- This code is never reached
Expected behavior: The function should call syncByLastModified (or the appropriate sync implementation) to actually synchronize local data with the cloud, enabling users to sync their notes across devices.
Impact:
Users cannot sync their notes across multiple devices
Cloud backup functionality is non-functional
This contradicts the README which mentions sync capabilities
Labels: bug, feature, priority: high