@loyalytics/swan-react-native-sdk 2.7.1-beta.2 → 2.7.1-beta.4

package/lib/module/index.js

@@ -1876,6 +1876,12 @@ export default class SwanSDK {
         }
       }
       if (!clickData) return;
+
+      // Discard stale click data (>60s old) from a previous session
+      if (clickData.timestamp && Date.now() / 1000 - clickData.timestamp > 60) {
+        Logger.log('[SwanSDK] Stale carousel click data (>60s old), discarding');
+        return;
+      }
       Logger.log('[SwanSDK] Pending carousel click found:', JSON.stringify(clickData));
       const {
         messageId,
@@ -3823,12 +3829,6 @@ export function createNotificationOpenedHandler() {
       const customMessageId = notificationData.messageId;
       const dedupId = customMessageId || messageId;
 
-      // Deduplication: prevent double click handling
-      if (dedupId && !markClickProcessed(dedupId)) {
-        Logger.log('[SwanSDK] Click already processed for messageId:', dedupId);
-        return;
-      }
-
       // If onNotificationOpenedApp fires, it means iOS displayed the notification
       // natively (aps.alert present). Notifee handlers won't fire for iOS-native
       // notifications, so this handler MUST process the click.
@@ -3888,6 +3888,14 @@ export function createNotificationOpenedHandler() {
           Logger.warn('[SwanSDK] Failed to peek Content Extension click data:', err);
         }
       }
+
+      // Deduplication: prevent double click handling.
+      // Placed AFTER carousel defer check — if we defer to checkPendingCarouselClick,
+      // we must NOT mark the ID as processed or checkPendingCarouselClick will skip it.
+      if (dedupId && !markClickProcessed(dedupId)) {
+        Logger.log('[SwanSDK] Click already processed for messageId:', dedupId);
+        return;
+      }
       const deepLinkPayload = {
         ...notificationData,
         route,