fix: bypass detail cache for updated notifications & await popup init

- Pass forceRefresh=true in createDetailFetchTask when updated_at changed,
  preventing stale PR/issue state from LRU cache
- Await popup init() to surface unhandled rejections cleanly

Author: euxx

src/background/service-worker.js

@@ -251,10 +251,10 @@ async function fetchWithConcurrencyLimit(tasks, limit = 5) {
  * @param {Array} detailedNotifications - Array to update with details
  * @returns {Function} Async function to fetch details
  */
-function createDetailFetchTask(notification, index, detailedNotifications) {
+function createDetailFetchTask(notification, index, detailedNotifications, forceRefresh = false) {
   return async () => {
     try {
-      const details = await github.getNotificationDetails(notification);
+      const details = await github.getNotificationDetails(notification, forceRefresh);
       updateNotificationDetails(detailedNotifications[index], details, notification.subject.type);
       return { success: true, id: notification.id, index };
     } catch (error) {
@@ -452,7 +452,7 @@ async function checkNotifications() {
       if (priorityNotifications.length > 0) {
         priorityResults = await fetchWithConcurrencyLimit(
           priorityNotifications.map(({ notification: n, index }) =>
-            createDetailFetchTask(n, index, detailedNotifications),
+            createDetailFetchTask(n, index, detailedNotifications, true),
           ),
           CONCURRENCY.PRIORITY,
         );
@@ -482,7 +482,7 @@ async function checkNotifications() {
       if (backgroundNotifications.length > 0) {
         fetchWithConcurrencyLimit(
           backgroundNotifications.map(({ notification: n, index }) =>
-            createDetailFetchTask(n, index, detailedNotifications),
+            createDetailFetchTask(n, index, detailedNotifications, true),
           ),
           CONCURRENCY.BACKGROUND,
         )

src/popup/popup.js

@@ -1080,5 +1080,5 @@ window.addEventListener("beforeunload", () => {
     document.body.classList.add("transitions-enabled");
   });
   // Then initialize (showView will set the correct width)
-  init();
+  await init();
 })();

tests/service-worker.test.js

@@ -900,6 +900,39 @@ describe("service-worker", () => {
         expect(call[0].map((n) => n.id)).not.toContain("A");
       });
     });
+
+    it("should pass forceRefresh=true when fetching details for updated notifications", async () => {
+      const oldNotif = makeRawNotif("X");
+      oldNotif.subject.url = "https://api.github.com/repos/owner/repo/issues/1";
+      const updatedNotif = {
+        ...oldNotif,
+        updated_at: "2024-06-01T00:00:00Z", // newer than stored
+      };
+
+      mockGithub.getNotifications.mockResolvedValue({
+        items: [updatedNotif],
+        hasMore: false,
+      });
+
+      // Stored version has old updated_at → needsUpdate = true
+      mockStorageFunctions.getNotifications
+        .mockResolvedValueOnce([makeStoredNotif("X")]) // existingIds
+        .mockResolvedValueOnce([makeStoredNotif("X")]) // safeBasic re-read
+        .mockResolvedValueOnce([makeStoredNotif("X")]); // mergeAndSaveIfCurrent
+
+      mockGithub.getNotificationDetails.mockResolvedValue({
+        state: "closed",
+        user: { login: "alice", avatar_url: "https://example.com/a.png", html_url: "" },
+      });
+
+      messageHandler({ action: "refresh" }, {}, vi.fn());
+      await new Promise((resolve) => setTimeout(resolve, 150));
+
+      // getNotificationDetails must be called with forceRefresh=true
+      expect(mockGithub.getNotificationDetails).toHaveBeenCalled();
+      const [, forceRefresh] = mockGithub.getNotificationDetails.mock.calls[0];
+      expect(forceRefresh).toBe(true);
+    });
   });
 });