@shortkitsdk/react-native 0.2.33 → 0.2.35

package/android/src/main/java/com/shortkit/reactnative/ReactCarouselOverlayHost.kt

@@ -221,11 +221,16 @@ class ReactCarouselOverlayHost(context: Context) : FrameLayout(context), Carouse
 
     override fun activatePlayback() {
         isActive = true
+        // Source mute state from the SDK rather than hardcoding true so
+        // overlay icons reflect actual mute state even on image carousels
+        // (which have no audio of their own but should still match host
+        // expectations).
+        val isMutedNow = ShortKitBridge.shared?.sdk?.player?.isMuted?.value ?: true
         val params = com.facebook.react.bridge.Arguments.createMap().apply {
             putString("surfaceId", surfaceId)
             putBoolean("isActive", true)
             putString("playerState", "idle")
-            putBoolean("isMuted", true)
+            putBoolean("isMuted", isMutedNow)
             putDouble("playbackRate", 1.0)
             putBoolean("captionsEnabled", false)
             putNull("activeCue")

package/android/src/main/java/com/shortkit/reactnative/ReactOverlayHost.kt

@@ -172,7 +172,14 @@ class ReactOverlayHost(context: Context) : FrameLayout(context), FeedOverlay {
         cachedDuration = 0.0
         cachedBuffered = 0.0
         cachedPlayerState = "idle"
-        cachedIsMuted = true
+        // Read the SDK's actual current mute state instead of hardcoding
+        // `true`. The flow subscription below will keep this in sync going
+        // forward, but during the brief window between configure() and the
+        // first emission the overlay's initialProps would otherwise carry a
+        // stale `true`, causing the next-cell mute toggle to compute the
+        // wrong target value (the user's tap on the new cell would resolve
+        // to setMuted(false) regardless of actual state).
+        player?.isMuted?.value?.let { cachedIsMuted = it }
         cachedPlaybackRate = 1.0
         cachedCaptionsEnabled = false
         cachedActiveCue = null

package/android/src/main/java/com/shortkit/reactnative/ReactVideoCarouselOverlayHost.kt

@@ -136,6 +136,13 @@ class ReactVideoCarouselOverlayHost(context: Context) : FrameLayout(context), Vi
         cachedDuration = 0.0
         cachedBuffered = 0.0
         cachedPlayerState = "idle"
+        // Seed cachedIsMuted from the SDK's current value so the new
+        // surface's initialProps reflect actual mute state. The flow
+        // subscription (resubscribeToPlayer) will keep it in sync going
+        // forward, but its first emission can race with a host tap on the
+        // new cell, leading to a stale-true bug where the toggle target is
+        // computed wrong.
+        player?.isMuted?.value?.let { cachedIsMuted = it }
 
         val json = Json.encodeToString(item)
         carouselItemJSON = json

package/android/src/main/java/com/shortkit/reactnative/ShortKitPackage.kt

@@ -37,8 +37,6 @@ class ShortKitPackage : TurboReactPackage() {
     ): List<ViewManager<*, *>> {
         return listOf(
             ShortKitFeedViewManager(),
-            ShortKitPlayerViewManager(),
-            ShortKitWidgetViewManager(),
         )
     }
 }

package/ios/ShortKitSDK.xcframework/ios-arm64/ShortKitSDK.framework/Info.plist

@@ -11,9 +11,9 @@
     <key>CFBundlePackageType</key>
     <string>FMWK</string>
     <key>CFBundleVersion</key>
-    <string>0.2.33</string>
+    <string>0.2.35</string>
     <key>CFBundleShortVersionString</key>
-    <string>0.2.33</string>
+    <string>0.2.35</string>
     <key>MinimumOSVersion</key>
     <string>16.0</string>
 </dict>