expo-realtime-ivs-broadcast 0.1.7 → 0.1.8

package/android/src/main/java/expo/modules/realtimeivsbroadcast/ExpoIVSStagePreviewView.kt

@@ -5,6 +5,8 @@ import android.os.Build
 import android.os.Handler
 import android.os.Looper
 import android.util.Log
+import android.view.View
+import android.view.ViewTreeObserver
 import android.widget.FrameLayout
 import androidx.annotation.RequiresApi
 import com.amazonaws.ivs.broadcast.BroadcastConfiguration.AspectMode
@@ -28,11 +30,52 @@ class ExpoIVSStagePreviewView(context: Context, appContext: AppContext) : ExpoVi
     // Props from React Native
     private var mirror: Boolean = false
     private var scaleMode: String = "fill"
+    
+    // Track view attachment state
+    @Volatile
+    private var isViewAttached = false
 
     init {
         Log.i("ExpoIVSStagePreviewView", "Initializing Stage Preview View...")
+        // Start initialization immediately - don't wait for onAttachedToWindow
+        // This ensures the preview is ready when the view becomes visible
         resolveStageManagerAndStreamWithRetry()
     }
+    
+    // Override onLayout to ensure child views are properly sized
+    override fun onLayout(changed: Boolean, left: Int, top: Int, right: Int, bottom: Int) {
+        super.onLayout(changed, left, top, right, bottom)
+        
+        // Manually layout children to fill the entire view
+        val childWidth = right - left
+        val childHeight = bottom - top
+        
+        for (i in 0 until childCount) {
+            val child = getChildAt(i)
+            child.layout(0, 0, childWidth, childHeight)
+        }
+        
+        Log.d("ExpoIVSStagePreviewView", "📐 onLayout: ${childWidth}x${childHeight}, children: $childCount")
+    }
+    
+    // Override onMeasure to properly measure children
+    override fun onMeasure(widthMeasureSpec: Int, heightMeasureSpec: Int) {
+        super.onMeasure(widthMeasureSpec, heightMeasureSpec)
+        
+        val width = MeasureSpec.getSize(widthMeasureSpec)
+        val height = MeasureSpec.getSize(heightMeasureSpec)
+        
+        // Measure all children with exact dimensions
+        val childWidthSpec = MeasureSpec.makeMeasureSpec(width, MeasureSpec.EXACTLY)
+        val childHeightSpec = MeasureSpec.makeMeasureSpec(height, MeasureSpec.EXACTLY)
+        
+        for (i in 0 until childCount) {
+            val child = getChildAt(i)
+            child.measure(childWidthSpec, childHeightSpec)
+        }
+        
+        Log.d("ExpoIVSStagePreviewView", "📐 onMeasure: ${width}x${height}")
+    }
 
     private fun resolveStageManagerAndStreamWithRetry() {
         Log.d("ExpoIVSStagePreviewView", "Attempting to resolve StageManager (attempt ${retryCount + 1})...")
@@ -60,7 +103,6 @@ class ExpoIVSStagePreviewView(context: Context, appContext: AppContext) : ExpoVi
         val cameraDevice = this.stageManager?.getLocalCameraDevice()
 
         if (cameraDevice == null) {
-            // Camera not ready yet, retry
             if (retryCount < maxRetries) {
                 retryCount++
                 Log.w("ExpoIVSStagePreviewView", "Camera device not ready, retrying in ${retryDelayMs}ms...")
@@ -78,8 +120,9 @@ class ExpoIVSStagePreviewView(context: Context, appContext: AppContext) : ExpoVi
         }
 
         val newDeviceUrn = cameraDevice.descriptor.urn
+        Log.i("ExpoIVSStagePreviewView", "📷 attachStreamWithRetry - Camera: ${cameraDevice.descriptor.friendlyName}, URN: $newDeviceUrn")
+        Log.i("ExpoIVSStagePreviewView", "📷 Current preview URN: $currentPreviewDeviceUrn, Has preview: ${ivsImagePreviewView != null}")
 
-        // If preview exists for the same device, do nothing.
         if (ivsImagePreviewView != null && this.currentPreviewDeviceUrn == newDeviceUrn) {
             Log.d("ExpoIVSStagePreviewView", "Already showing preview for device: $newDeviceUrn")
             return
@@ -88,9 +131,13 @@ class ExpoIVSStagePreviewView(context: Context, appContext: AppContext) : ExpoVi
         // Cleanup existing preview
         if (ivsImagePreviewView != null) {
             Log.d("ExpoIVSStagePreviewView", "Cleaning up existing preview")
-            removeView(ivsImagePreviewView)
+            try {
+                removeView(ivsImagePreviewView)
+            } catch (e: Exception) {
+                Log.w("ExpoIVSStagePreviewView", "Error removing old preview: ${e.message}")
+            }
             ivsImagePreviewView = null
-            this.currentPreviewDeviceUrn = null
+            currentPreviewDeviceUrn = null
         }
 
         try {
@@ -102,9 +149,16 @@ class ExpoIVSStagePreviewView(context: Context, appContext: AppContext) : ExpoVi
                 return
             }
             
-            val newPreview = imageDevice.previewView
+            // Use getPreviewView() with aspect mode to get a fresh preview view
+            // This is more reliable than using the previewView property after camera swap
+            val newPreview = try {
+                imageDevice.getPreviewView(AspectMode.FILL)
+            } catch (e: Exception) {
+                Log.w("ExpoIVSStagePreviewView", "getPreviewView(FILL) failed, trying previewView property: ${e.message}")
+                imageDevice.previewView
+            }
+            
             if (newPreview == null) {
-                // Preview view not ready yet, retry after delay
                 if (retryCount < maxRetries) {
                     retryCount++
                     Log.w("ExpoIVSStagePreviewView", "Preview view is null, retrying in ${retryDelayMs}ms (attempt $retryCount)...")
@@ -115,10 +169,22 @@ class ExpoIVSStagePreviewView(context: Context, appContext: AppContext) : ExpoVi
                 return
             }
             
-            // Check if preview is attached to another parent and detach it first
-            if (newPreview.parent != null) {
-                Log.w("ExpoIVSStagePreviewView", "Preview view already has a parent, detaching first...")
-                (newPreview.parent as? android.view.ViewGroup)?.removeView(newPreview)
+            Log.i("ExpoIVSStagePreviewView", "📷 Got preview view: ${newPreview.javaClass.simpleName}, hashCode: ${newPreview.hashCode()}")
+            
+            // IMPORTANT: Check if preview is attached to another parent and detach it first
+            val existingParent = newPreview.parent
+            if (existingParent != null && existingParent != this) {
+                Log.w("ExpoIVSStagePreviewView", "Preview view has another parent, detaching first...")
+                (existingParent as? android.view.ViewGroup)?.removeView(newPreview)
+            }
+            
+            // If already attached to this view, no need to re-add
+            if (newPreview.parent == this) {
+                Log.d("ExpoIVSStagePreviewView", "Preview already attached to this view")
+                this.ivsImagePreviewView = newPreview
+                this.currentPreviewDeviceUrn = newDeviceUrn
+                applyProps()
+                return
             }
             
             newPreview.layoutParams = FrameLayout.LayoutParams(
@@ -131,13 +197,46 @@ class ExpoIVSStagePreviewView(context: Context, appContext: AppContext) : ExpoVi
             this.currentPreviewDeviceUrn = newDeviceUrn
             
             applyProps()
+            
+            // Force layout update - measure and layout immediately with current dimensions
+            if (width > 0 && height > 0) {
+                val widthSpec = MeasureSpec.makeMeasureSpec(width, MeasureSpec.EXACTLY)
+                val heightSpec = MeasureSpec.makeMeasureSpec(height, MeasureSpec.EXACTLY)
+                newPreview.measure(widthSpec, heightSpec)
+                newPreview.layout(0, 0, width, height)
+                Log.i("ExpoIVSStagePreviewView", "📐 Forced layout: preview now ${newPreview.measuredWidth}x${newPreview.measuredHeight}")
+            } else {
+                // Parent not yet measured, use ViewTreeObserver to layout after measure
+                viewTreeObserver.addOnGlobalLayoutListener(object : ViewTreeObserver.OnGlobalLayoutListener {
+                    override fun onGlobalLayout() {
+                        viewTreeObserver.removeOnGlobalLayoutListener(this)
+                        val preview = ivsImagePreviewView ?: return
+                        if (this@ExpoIVSStagePreviewView.width > 0 && this@ExpoIVSStagePreviewView.height > 0) {
+                            val w = this@ExpoIVSStagePreviewView.width
+                            val h = this@ExpoIVSStagePreviewView.height
+                            val wSpec = MeasureSpec.makeMeasureSpec(w, MeasureSpec.EXACTLY)
+                            val hSpec = MeasureSpec.makeMeasureSpec(h, MeasureSpec.EXACTLY)
+                            preview.measure(wSpec, hSpec)
+                            preview.layout(0, 0, w, h)
+                            Log.i("ExpoIVSStagePreviewView", "📐 Deferred layout: preview now ${preview.measuredWidth}x${preview.measuredHeight}")
+                        }
+                    }
+                })
+            }
+            
+            // Also request layout for next frame
+            newPreview.requestLayout()
+            requestLayout()
+            invalidate()
+            
             Log.i("ExpoIVSStagePreviewView", "✅ Successfully attached camera preview for: ${cameraDevice.descriptor.friendlyName}")
+            Log.i("ExpoIVSStagePreviewView", "📷 Preview dimensions: ${newPreview.width}x${newPreview.height}, visibility: ${newPreview.visibility}")
+            Log.i("ExpoIVSStagePreviewView", "📷 Parent dimensions: ${this.width}x${this.height}, childCount: ${this.childCount}")
         } catch (e: Exception) {
             Log.e("ExpoIVSStagePreviewView", "❌ Failed to attach camera preview: ${e.message}", e)
             this.ivsImagePreviewView = null
             this.currentPreviewDeviceUrn = null
             
-            // Retry on exception as well
             if (retryCount < maxRetries) {
                 retryCount++
                 Log.w("ExpoIVSStagePreviewView", "Retrying after exception in ${retryDelayMs}ms (attempt $retryCount)...")
@@ -148,7 +247,8 @@ class ExpoIVSStagePreviewView(context: Context, appContext: AppContext) : ExpoVi
     
     // Public method for manager to notify when camera changes (e.g., after swapCamera)
     fun refreshPreview() {
-        Log.d("ExpoIVSStagePreviewView", "Refresh preview requested - clearing current preview and reattaching")
+        Log.i("ExpoIVSStagePreviewView", "🔄 Refresh preview requested - clearing current preview and reattaching")
+        Log.i("ExpoIVSStagePreviewView", "🔄 Current URN before refresh: $currentPreviewDeviceUrn")
         
         // Ensure we're on the main thread
         if (Looper.myLooper() != Looper.getMainLooper()) {
@@ -157,22 +257,26 @@ class ExpoIVSStagePreviewView(context: Context, appContext: AppContext) : ExpoVi
         }
         
         // Clear the current preview first
-        if (ivsImagePreviewView != null) {
+        val oldPreview = ivsImagePreviewView
+        if (oldPreview != null) {
+            Log.i("ExpoIVSStagePreviewView", "🔄 Removing old preview view")
             try {
-                removeView(ivsImagePreviewView)
+                removeView(oldPreview)
             } catch (e: Exception) {
                 Log.w("ExpoIVSStagePreviewView", "Error removing old preview view: ${e.message}")
             }
             ivsImagePreviewView = null
         }
+        
+        // IMPORTANT: Clear the URN so attachStreamWithRetry will fetch a fresh preview
         currentPreviewDeviceUrn = null
         
         retryCount = 0
-        // Add a small delay to ensure the old preview is fully detached 
-        // before attaching the new one (prevents green screen on camera swap)
+        // Add a delay to allow the new camera stream to initialize
         mainHandler.postDelayed({
+            Log.i("ExpoIVSStagePreviewView", "🔄 Now attaching new preview after delay")
             attachStreamWithRetry()
-        }, 50)
+        }, 200)
     }
     
     private fun applyProps() {
@@ -180,7 +284,6 @@ class ExpoIVSStagePreviewView(context: Context, appContext: AppContext) : ExpoVi
         setScaleMode(this.scaleMode)
     }
 
-    // Prop setters
     fun setMirror(mirror: Boolean) {
         this.mirror = mirror
         (ivsImagePreviewView as? ImagePreviewView)?.setMirrored(this.mirror)
@@ -204,20 +307,62 @@ class ExpoIVSStagePreviewView(context: Context, appContext: AppContext) : ExpoVi
 
     override fun onAttachedToWindow() {
         super.onAttachedToWindow()
+        isViewAttached = true
         Log.d("ExpoIVSStagePreviewView", "View attached to window")
-        // Re-attempt to attach if we don't have a preview yet
+        
+        // If we don't have a preview yet, try to attach one
         if (ivsImagePreviewView == null) {
             retryCount = 0
             resolveStageManagerAndStreamWithRetry()
+        } else {
+            // Re-add the preview if it was removed during detach
+            val preview = ivsImagePreviewView
+            if (preview != null && preview.parent == null) {
+                Log.d("ExpoIVSStagePreviewView", "Re-adding preview on attach")
+                try {
+                    addView(preview)
+                } catch (e: Exception) {
+                    Log.w("ExpoIVSStagePreviewView", "Error re-adding preview: ${e.message}")
+                    // Try fresh attach
+                    ivsImagePreviewView = null
+                    currentPreviewDeviceUrn = null
+                    retryCount = 0
+                    attachStreamWithRetry()
+                }
+            }
         }
     }
 
     override fun onDetachedFromWindow() {
-        super.onDetachedFromWindow()
         Log.d("ExpoIVSStagePreviewView", "View detached from window")
-        // Remove any pending retries
+        isViewAttached = false
+        
+        // Remove pending callbacks
         mainHandler.removeCallbacksAndMessages(null)
+        
+        // Remove the preview view from this parent so it can be reused
+        // Do this in a post to avoid conflicts with the current render cycle
+        val preview = ivsImagePreviewView
+        if (preview != null) {
+            mainHandler.post {
+                try {
+                    if (preview.parent == this) {
+                        removeView(preview)
+                        Log.d("ExpoIVSStagePreviewView", "Preview removed in post")
+                    }
+                } catch (e: Exception) {
+                    Log.w("ExpoIVSStagePreviewView", "Error removing preview in post: ${e.message}")
+                }
+            }
+        }
+        
+        // Clear references
+        ivsImagePreviewView = null
+        currentPreviewDeviceUrn = null
+        
         // Unregister from manager
         stageManager?.unregisterPreviewView(this)
+        
+        super.onDetachedFromWindow()
     }
 }

package/android/src/main/java/expo/modules/realtimeivsbroadcast/IVSStageManager.kt

@@ -165,16 +165,39 @@ class IVSStageManager(private val context: Context) : Stage.Strategy, StageRende
             return
         }
         
+        Log.i("ExpoIVSStageManager", "🔄 Swapping camera from ${currentDevice.descriptor.friendlyName} to ${newCamera.descriptor.friendlyName}")
+        
+        // First, notify preview views to clear their current preview
+        // This releases the old camera's preview before we switch
+        previewViews.mapNotNull { it.get() }.forEach { view ->
+            mainHandler.post {
+                try {
+                    // Force clear the preview by calling refreshPreview which removes the old one
+                    Log.i("ExpoIVSStageManager", "🔄 Clearing preview before swap")
+                } catch (e: Exception) {
+                    Log.w("ExpoIVSStageManager", "Error clearing preview: ${e.message}")
+                }
+            }
+        }
+        
+        // Update the local camera reference
         localCamera = newCamera
-        cameraStream = ImageLocalStageStream(newCamera)
+        
+        // Create new stream with video configuration
+        val oldStream = cameraStream
+        cameraStream = ImageLocalStageStream(newCamera, this.stageConfiguration.videoConfiguration)
+        
+        // Refresh the stage strategy to use the new stream
         stage?.refreshStrategy()
-        Log.i("ExpoIVSStageManager", "✅ Camera swapped to: ${newCamera.descriptor.friendlyName}")
+        
+        Log.i("ExpoIVSStageManager", "✅ Camera swapped to: ${newCamera.descriptor.friendlyName}, URN: ${newCamera.descriptor.urn}")
         
         // Delay the preview refresh to allow the new camera stream to fully initialize
-        // The green screen issue occurs when we try to get previewView before the stream is ready
+        // The back camera especially needs more time to warm up
         mainHandler.postDelayed({
+            Log.i("ExpoIVSStageManager", "🔄 Triggering preview refresh after camera swap")
             notifyPreviewViewsToRefresh()
-        }, 150) // 150ms delay for stream initialization
+        }, 500) // 500ms delay for camera to fully initialize
     }
     
     private fun notifyPreviewViewsToRefresh() {

package/package.json

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

package/android/build/kotlin/compileDebugKotlin/cacheable/caches-jvm/inputs/source-to-output.tab.values.s

@@ -1 +0,0 @@
-����

package/android/build/kotlin/compileDebugKotlin/cacheable/caches-jvm/jvm/kotlin/internal-name-to-source.tab.values.s

@@ -1 +0,0 @@
-��

package/android/build/kotlin/compileDebugKotlin/cacheable/caches-jvm/jvm/kotlin/proto.tab.values.s

@@ -1 +0,0 @@
-�3�3�4�1�7

package/android/build/kotlin/compileDebugKotlin/cacheable/caches-jvm/jvm/kotlin/source-to-classes.tab.values.s

@@ -1 +0,0 @@
-��