fix(updates): show "Hermes updated" modal only once per release (#386)

The /api/update/status poll returns the same pendingReleaseNotes on every
poll. Workspace was running storeNotes() inside a useEffect keyed on those
notes, and storeNotes unconditionally cleared NOTES_SEEN_KEY. So every
time the user refreshed the page (or a poll fired), the seen marker was
dropped and the modal popped again.

Now we only clear the seen marker when the notes ID has actually changed
(i.e. a new release with different content). Identical payloads on
refresh keep the seen marker intact.

Closes #356

Co-authored-by: Aurora release bot <release@outsourc-e.com>

Author: outsourc-e

src/components/update-center-notifier.tsx

@@ -87,9 +87,27 @@ function notesId(sections: Array<ReleaseNoteSection>): string {
 
 function storeNotes(sections: Array<ReleaseNoteSection>): Notes | null {
   if (!sections.length) return null
-  const notes = { id: notesId(sections), sections, updatedAt: Date.now() }
+  const id = notesId(sections)
+  const notes = { id, sections, updatedAt: Date.now() }
+  // Only clear the "seen" marker when the release-notes payload actually
+  // changed. Without this guard the modal pops up on every page refresh
+  // because /api/update/status returns the same pendingReleaseNotes on every
+  // poll, useEffect fires, and we used to drop the seen marker every time.
+  // See #356.
+  let existingId: string | null = null
+  try {
+    const raw = localStorage.getItem(NOTES_KEY)
+    if (raw) {
+      const parsed = JSON.parse(raw) as Notes
+      existingId = parsed?.id ?? null
+    }
+  } catch {
+    existingId = null
+  }
+  if (existingId !== id) {
+    localStorage.removeItem(NOTES_SEEN_KEY)
+  }
   localStorage.setItem(NOTES_KEY, JSON.stringify(notes))
-  localStorage.removeItem(NOTES_SEEN_KEY)
   return notes
 }