.

Author: tung-hpzbook

src/helpers.js

@@ -481,94 +481,6 @@ async function synchronize({notes, deletedNoteIds, isSilent, credentials, nostrP
 	}
 }
 
-		// --- Step 5: Determine which notes to download or delete locally ---
-		const notesToDownload = [];
-		const notesToDeleteLocally = [];
-		const allLocalNotesMap = new Map(localNotes.map(n => [n.id, n]));
-
-		// Find notes updated remotely
-		for (const remoteMeta of remoteNoteMetadata) {
-			const localNote = allLocalNotesMap.get(remoteMeta.id);
-			if (!localNote) {
-				// Note exists on remote but not local, and isn't in the local delete list. Download it.
-				if (!deletedNoteIds.includes(remoteMeta.id)) {
-					notesToDownload.push(remoteMeta);
-				}
-			} else {
-				// Note exists on both. Download if remote is newer.
-				const remoteDate = new Date(remoteMeta.updatedAt);
-				const localDate = new Date(localNote.updatedAt);
-				if (remoteDate > localDate) {
-					notesToDownload.push(remoteMeta);
-				}
-			}
-		}
-
-		// Find notes deleted remotely
-		const remoteNoteIds = new Set(remoteNoteMetadata.map(m => m.id));
-		const locallyDeletedNoteIds = new Set(deletedNoteIds);
-		const uploadedNoteIds = new Set(notesToUpload.map(n => n.id));
-
-		// Create a set of all local note IDs for easier checking
-		const localNoteIds = new Set(allLocalNotesMap.keys());
-
-		// Identify notes that should be deleted locally:
-		// - Exist locally
-		// - Don't exist remotely
-		// - Weren't already marked for local deletion
-		// - Weren't just uploaded (which would mean they now exist remotely)
-		const remotelyDeletedNoteIds = [...localNoteIds].filter(noteId => 
-			!remoteNoteIds.has(noteId) && 
-			!locallyDeletedNoteIds.has(noteId) && 
-			!uploadedNoteIds.has(noteId)
-		);
-
-		notesToDeleteLocally.push(...remotelyDeletedNoteIds);
-
-		const downloadPromises = notesToDownload.map(remoteMeta => {
-			if (remoteMeta.source === 'git') {
-				// We already have the content from listNotesInGit? 
-				// listNotesInGit returns full note objects with content!
-				// So we don't need to download again. We just return it.
-				return Promise.resolve(remoteMeta);
-			} else if (remoteMeta.source === 'nostr') {
-				return Promise.resolve(remoteMeta); // Nostr also returns full event/note
-			} else {
-				return downloadNoteFromS3(remoteMeta.id, credentials);
-			}
-		});
-		const downloadResults = await Promise.allSettled(downloadPromises);
-
-		const updatedNotes = [];
-		downloadResults.forEach((result, idx) => {
-			if (result.status === 'fulfilled' && result.value) {
-				updatedNotes.push(result.value);
-			} else if (result.status === 'rejected') {
-				console.log('Sync download error:', notesToDownload[idx].id, result.reason.message);
-				// If a note was listed in metadata but fails to download with NoSuchKey,
-				// it means it was deleted between the list and get operations.
-				// We should treat it as a remote deletion.
-				if (result.reason && result.reason.code === 'NoSuchKey') {
-					notesToDeleteLocally.push(notesToDownload[idx].id);
-				}
-			}
-		});
-
-		return {
-			success: true,
-			uploadedCount: successfulUploadedCount,
-			deletedCount: successfulDeletedCount,
-			successfulDeletedIds, // downloaded notes
-			updatedNotes, // downloaded notes
-			notesToDeleteLocally,
-			finalRemoteIds: Array.from(remoteNoteIds)
-		};
-	} catch (error) {
-		console.error('synchronize error:', error);
-		return { success: false, error: error.message };
-	}
-}
-
 async function synchronizeImages({encryptedSettings, userId, nostrPrivateKey, nostrRelays}) {
 	try {
 		const credentials = await decryptSettings(encryptedSettings, userId);

src/index.js

@@ -936,13 +936,7 @@ document.addEventListener('alpine:init', () => { Alpine.data('mainApp', () => ({
 				if (!isSilent) {
 					let syncDescription = `Sync completed: ${result.uploadedCount || 0} uploaded, ${downloadedCount} downloaded, ${result.deletedCount || 0} remote deletes.`;
 					this.showToast({ title: 'Sync Successful', description: syncDescription });
-				}
-			} else {
-							title: 'Synced',
-							description: `Up ${result.uploadedCount} Down ${downloadedCount} Del ${result.deletedCount} Remote Del ${notesToDeleteLocally.length}`,
-						});
-					}
-
+				} else {
 					// S3/Nostr sync was successful, filter the pending deletions
 					// We only remove IDs from the deletion queue if they are NO LONGER present on the remote.
 					// If an ID is still in result.finalRemoteIds, it means the remote still had it when we started this sync.