Merge pull request #114 from djedi/fix/SSE-notification-bug

Fix false SSE warning on local save

Author: djedi

client/src/views/Day.vue

@@ -172,10 +172,16 @@ const saveDay = async (isAutoSave: boolean = false) => {
   isSaving.value = true;
   const updatedDay = Object.assign(day.value, { data: modifiedText.value });
 
+  // Mark as locally updated BEFORE the API call to prevent SSE race condition
+  // (SSE event can arrive before the await returns)
+  if (day.value.uuid) {
+    markNoteUpdatedLocally(day.value.uuid);
+  }
+
   try {
     const res = await NoteService.saveDay(updatedDay);
-    // Mark as locally updated to prevent SSE duplicate processing
-    if (res.uuid) {
+    // Also mark after save in case it's a new note that just got a uuid
+    if (res.uuid && res.uuid !== day.value.uuid) {
       markNoteUpdatedLocally(res.uuid);
     }
     text.value = modifiedText.value;

client/src/views/Note.vue

@@ -127,12 +127,14 @@ const saveNote = async (isAutoSave: boolean = false) => {
   isSaving.value = true;
   const updatedNote = Object.assign(note.value, { data: modifiedText.value });
 
+  // Mark as locally updated BEFORE the API call to prevent SSE race condition
+  // (SSE event can arrive before the await returns)
+  if (note.value.uuid) {
+    markNoteUpdatedLocally(note.value.uuid);
+  }
+
   try {
     note.value = await NoteService.saveNote(updatedNote);
-    // Mark as locally updated to prevent SSE duplicate processing
-    if (note.value.uuid) {
-      markNoteUpdatedLocally(note.value.uuid);
-    }
     text.value = modifiedText.value;
     // Don't reset modifiedText - it's already correct and resetting causes editor glitches
     headerOptions.title = note.value.title || '';