fix: skip reporting already-read notifications (#8848)

Author: huhuanming

packages/kit/src/views/Notifications/pages/NotificationList.tsx

@@ -492,6 +492,7 @@ function BaseNotificationList() {
                         item?.body?.extras?.params?.msgId ||
                         item?.body?.extras?.msgId ||
                         '',
+                      isRead: !!item.readed,
                     });
                     setTimeout(() => {
                       if (!item.readed) {

packages/shared/src/logger/scopes/app/scenes/page.ts

@@ -37,7 +37,7 @@ export class PageScene extends BaseScene {
   public notificationItemClicked(
     notificationId: string,
     type: ENotificationPushTopicTypes | 'unknown',
-    clickFrom: 'notificationClick' | 'notificationListClick',
+    clickFrom: 'app' | 'system',
   ) {
     return { notificationId, type, clickFrom };
   }

packages/shared/src/utils/notificationsUtils.ts

@@ -121,6 +121,7 @@ export interface INavigateToNotificationDetailParams {
   mode?: ENotificationPushMessageMode;
   payload?: string;
   topicType?: ENotificationPushTopicTypes;
+  isRead?: boolean;
 }
 
 export function parseNotificationPayload(
@@ -174,18 +175,21 @@ async function navigateToNotificationDetail({
   mode,
   payload,
   topicType,
+  isRead = false,
 }: INavigateToNotificationDetailParams) {
   let routes: string[] = [];
   let params: any = {};
   let shouldAckRead = true;
 
-  setTimeout(() => {
-    defaultLogger.app.page.notificationItemClicked(
-      notificationId,
-      topicType || 'unknown',
-      isFromNotificationClick ? 'notificationClick' : 'notificationListClick',
-    );
-  });
+  if (!isRead) {
+    setTimeout(() => {
+      defaultLogger.app.page.notificationItemClicked(
+        notificationId,
+        topicType || 'unknown',
+        isFromNotificationClick ? 'app' : 'system',
+      );
+    });
+  }
 
   if (isFromNotificationClick) {
     const statusRoutes = appGlobals.$navigationRef.current?.getState().routes;