cache notification count, unvalidate on new notification

Author: imorland

framework/core/src/Notification/Command/ReadAllNotificationsHandler.php

@@ -48,6 +48,9 @@ public function handle(ReadAllNotifications $command)
 
         $this->notifications->markAllAsRead($actor);
 
+        // Invalidate notification count cache
+        resolve('cache.store')->forget("user.{$actor->id}.unread_notification_count");
+
         $this->events->dispatch(new ReadAll($actor, Carbon::now()));
     }
 }

framework/core/src/Notification/Command/ReadNotificationHandler.php

@@ -51,6 +51,9 @@ public function handle(ReadNotification $command)
 
         $notification->read_at = Carbon::now();
 
+        // Invalidate notification count cache
+        resolve('cache.store')->forget("user.{$actor->id}.unread_notification_count");
+
         $this->events->dispatch(new Read($actor, $notification, Carbon::now()));
 
         return $notification;

framework/core/src/Notification/Notification.php

@@ -233,6 +233,12 @@ public static function notify(array $recipients, BlueprintInterface $blueprint)
                 ];
             }, $recipients)
         );
+
+        // Invalidate notification count cache for all recipients
+        $cache = resolve('cache.store');
+        foreach ($recipients as $user) {
+            $cache->forget("user.{$user->id}.unread_notification_count");
+        }
     }
 
     /**

framework/core/src/User/User.php

@@ -431,7 +431,14 @@ public function getAlertableNotificationTypes()
      */
     public function getUnreadNotificationCount()
     {
-        return $this->unreadNotifications()->count();
+        // Cache notification count for 5 minutes to avoid expensive query on every page load
+        // Cache is actively invalidated when notifications are marked as read or created,
+        // so the TTL is mainly a safety net for edge cases (e.g., direct DB updates)
+        $cacheKey = "user.{$this->id}.unread_notification_count";
+
+        return resolve('cache.store')->remember($cacheKey, 300, function () {
+            return $this->unreadNotifications()->count();
+        });
     }
 
     /**