recording: do not rotate the session id for hybrid SDKs (#253)

CHANGELOG.md

@@ -1,5 +1,7 @@
 ## Next
 
+- recording: do not rotate the session id for hybrid SDKs ([#253](https://github.com/PostHog/posthog-ios/pull/253))
+
 ## 3.15.2 - 2024-11-13
 
 - fix: allow changing person properties after identify ([#249](https://github.com/PostHog/posthog-ios/pull/249))

PostHog/PostHogSessionManager.swift

@@ -46,7 +46,17 @@ import Foundation
         timeNow - sessionLastTimestamp > sessionChangeThreshold
     }
 
+    private func isiOSNativeSdk() -> Bool {
+        // postHogSdkName will be set to eg posthog-react-native if not
+        postHogSdkName == postHogiOSSdkName
+    }
+
     func resetSessionIfExpired(_ completion: () -> Void) {
+        // for hybrid SDKs, the session is handled by the hybrid SDK
+        guard isiOSNativeSdk() else {
+            return
+        }
+
         sessionLock.withLock {
             let timeNow = now().timeIntervalSince1970
             if sessionId != nil,
@@ -79,6 +89,11 @@ import Foundation
     }
 
     func rotateSessionIdIfRequired(_ completion: @escaping (() -> Void)) {
+        // for hybrid SDKs, the session is handled by the hybrid SDK
+        guard isiOSNativeSdk() else {
+            return
+        }
+
         sessionLock.withLock {
             let timeNow = now().timeIntervalSince1970
 
@@ -94,6 +109,11 @@ import Foundation
     }
 
     func updateSessionLastTime() {
+        // for hybrid SDKs, the session is handled by the hybrid SDK
+        guard isiOSNativeSdk() else {
+            return
+        }
+
         sessionLock.withLock {
             sessionLastTimestamp = now().timeIntervalSince1970
         }

PostHog/PostHogVersion.swift

@@ -11,5 +11,6 @@ import Foundation
 // This property is internal only
 public var postHogVersion = "3.15.2"
 
+public let postHogiOSSdkName = "posthog-ios"
 // This property is internal only
-public var postHogSdkName = "posthog-ios"
+public var postHogSdkName = postHogiOSSdkName