expo-realtime-ivs-broadcast 0.2.1 → 0.2.3

package/README.md

@@ -8,7 +8,8 @@ This module provides React Native components and a comprehensive API to integrat
 
 | Library Version | Expo SDK | React Native | React   | Notes |
 |-----------------|----------|--------------|---------|-------|
-| 0.2.0           | 54       | 0.81.x       | 19.1.x  | **Added Picture-in-Picture support** |
+| 0.2.1           | 54       | 0.81.x       | 19.1.x  | **Fixed iOS PiP pre-warming reliability** |
+| 0.2.0           | 54       | 0.81.x       | 19.1.x  | Added Picture-in-Picture support |
 | 0.1.7           | 54       | 0.81.x       | 19.1.x  | |
 | 0.1.4           | 53       | 0.79.x       | 19.0.x  | |
 

package/ios/ExpoIVSRemoteStreamView.swift

@@ -103,9 +103,17 @@ class ExpoIVSRemoteStreamView: ExpoView {
             print("✅ [REMOTE VIEW] Manager commanded me to render URN: \(deviceUrn)")
             
             // Notify the stage manager that a stream started rendering (for PiP)
-            // Use a small delay to ensure the view is fully set up
-            DispatchQueue.main.asyncAfter(deadline: .now() + 0.1) { [weak self] in
-                self?.stageManager?.notifyRemoteStreamRendered()
+            // Use a longer delay to ensure the view hierarchy is fully set up
+            DispatchQueue.main.asyncAfter(deadline: .now() + 0.3) { [weak self] in
+                guard let self = self else { return }
+                print("✅ [REMOTE VIEW] Notifying PiP system - view in window: \(self.window != nil), isRenderingVideo: \(self.isRenderingVideo)")
+                self.stageManager?.notifyRemoteStreamRendered()
+            }
+            
+            // Also notify immediately for faster setup if view is already ready
+            if self.window != nil {
+                print("✅ [REMOTE VIEW] View already in window, notifying PiP immediately")
+                self.stageManager?.notifyRemoteStreamRendered()
             }
         } catch {
             print("❌ [REMOTE VIEW] Failed to create preview for URN \(deviceUrn): \(error)")

package/ios/IVSStageManager.swift

@@ -1057,13 +1057,27 @@ class IVSStageManager: NSObject, IVSStageStreamDelegate, IVSStageStrategy, IVSSt
                 let previewView = try device.previewView()
                 pipController.setupWithSourceView(previewView)
                 pipTargetView = previewView
+                print("🖼️ [PiP] Set up with device preview view")
             } catch {
                 print("🖼️ [PiP] Warning: Could not get preview view from device: \(error)")
                 // Try to find a remote view that's rendering this device
-                if let remoteView = remoteViews.compactMap({ $0.value }).first(where: { $0.currentRenderedDeviceUrn == device.descriptor().urn }),
-                   let previewViewForPiP = remoteView.previewViewForPiP {
+                if let remoteView = remoteViews.compactMap({ $0.value }).first(where: { $0.currentRenderedDeviceUrn == device.descriptor().urn }) {
+                    // Always set up with the remote view container - it's the visible video view
                     pipController.setupWithSourceView(remoteView as UIView)
-                    pipTargetView = previewViewForPiP
+                    pipTargetView = remoteView.previewViewForPiP ?? remoteView
+                    print("🖼️ [PiP] Set up with matching remote view container")
+                } else if let anyRenderingView = remoteViews.compactMap({ $0.value }).first(where: { $0.isRenderingVideo }) {
+                    // Fallback: use any view that's rendering
+                    pipController.setupWithSourceView(anyRenderingView as UIView)
+                    pipTargetView = anyRenderingView.previewViewForPiP ?? anyRenderingView
+                    print("🖼️ [PiP] Set up with fallback rendering view")
+                } else if let anyView = remoteViews.compactMap({ $0.value }).first {
+                    // Last resort: use any registered view
+                    pipController.setupWithSourceView(anyView as UIView)
+                    pipTargetView = anyView
+                    print("🖼️ [PiP] Set up with any available view (last resort)")
+                } else {
+                    print("🖼️ [PiP] ERROR: No source view found for PiP - controller will NOT be initialized!")
                 }
             }
         }
@@ -1111,20 +1125,43 @@ class IVSStageManager: NSObject, IVSStageStreamDelegate, IVSStageStrategy, IVSSt
             if currentPiPSourceDeviceUrn != imageDevice.descriptor().urn {
                 print("🖼️ [PiP] Found new remote video stream: \(imageDevice.descriptor().urn)")
                 
+                // Debug: Log all registered remote views and their URNs
+                print("🖼️ [PiP] Registered remote views: \(remoteViews.count)")
+                for (index, viewWrapper) in remoteViews.enumerated() {
+                    if let view = viewWrapper.value {
+                        print("🖼️ [PiP]   View \(index): URN=\(view.currentRenderedDeviceUrn ?? "nil"), isRendering=\(view.isRenderingVideo)")
+                    }
+                }
+                
                 // Try to find the remote view that's rendering this stream
                 if let remoteView = remoteViews.compactMap({ $0.value }).first(where: { $0.currentRenderedDeviceUrn == imageDevice.descriptor().urn }) {
-                    // Use the remote view (container) as the source view for Video Call API
-                    // Cast to UIView explicitly since ExpoIVSRemoteStreamView extends ExpoView -> UIView
                     candidateSourceView = remoteView as UIView
                     print("🖼️ [PiP] Found matching remote view for source")
+                } else {
+                    // Fallback: Use ANY remote view that's rendering video
+                    if let anyRenderingView = remoteViews.compactMap({ $0.value }).first(where: { $0.isRenderingVideo }) {
+                        candidateSourceView = anyRenderingView as UIView
+                        print("🖼️ [PiP] Using fallback remote view (URN didn't match but view is rendering)")
+                    } else if let anyView = remoteViews.compactMap({ $0.value }).first {
+                        // Last resort: use any registered view
+                        candidateSourceView = anyView as UIView
+                        print("🖼️ [PiP] Using any available remote view as last resort")
+                    }
                 }
                 
                 attachToDevice(imageDevice, sourceView: candidateSourceView)
             }
         } else {
-            // No candidate stream found
+            // No candidate stream found - log why
             if currentPiPDevice == nil {
                 print("🖼️ [PiP] No active remote video stream found yet")
+                print("🖼️ [PiP]   Total participants: \(participants.count)")
+                for p in participants {
+                    print("🖼️ [PiP]   Participant \(p.info.participantId ?? "nil"): \(p.streams.count) streams")
+                    for s in p.streams {
+                        print("🖼️ [PiP]     Stream type: \(s.device.descriptor().type.rawValue), URN: \(s.device.descriptor().urn)")
+                    }
+                }
             }
         }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "expo-realtime-ivs-broadcast",
-  "version": "0.2.1",
+  "version": "0.2.3",
   "description": "An Expo module for real-time broadcasting using Amazon IVS.",
   "main": "build/index.js",
   "types": "build/index.d.ts",