split StateWrapper.updateState overloads and mark UpdateMode unstable

Author: intergalacticspacehighway

packages/react-native/ReactAndroid/src/main/java/com/facebook/react/fabric/StateWrapperImpl.kt

@@ -14,6 +14,7 @@ import com.facebook.proguard.annotations.DoNotStripAny
 import com.facebook.react.bridge.NativeMap
 import com.facebook.react.bridge.ReadableNativeMap
 import com.facebook.react.bridge.WritableMap
+import com.facebook.react.common.annotations.UnstableReactNativeAPI
 import com.facebook.react.common.mapbuffer.ReadableMapBuffer
 import com.facebook.react.uimanager.ReferenceStateWrapper
 import com.facebook.react.uimanager.StateWrapper
@@ -24,6 +25,7 @@ import com.facebook.react.uimanager.StateWrapper
  */
 @SuppressLint("MissingNativeLoadLibrary")
 @DoNotStripAny
+@OptIn(UnstableReactNativeAPI::class)
 internal class StateWrapperImpl private constructor() : HybridClassBase(), ReferenceStateWrapper {
 
   private external fun initHybrid()
@@ -67,6 +69,10 @@ internal class StateWrapperImpl private constructor() : HybridClassBase(), Refer
     initHybrid()
   }
 
+  override fun updateState(map: WritableMap) {
+    updateState(map, StateWrapper.UpdateMode.Asynchronous)
+  }
+
   override fun updateState(map: WritableMap, updateMode: StateWrapper.UpdateMode) {
     if (!isValid) {
       FLog.e(TAG, "Race between StateWrapperImpl destruction and updateState")

packages/react-native/ReactAndroid/src/main/java/com/facebook/react/uimanager/StateWrapper.kt

@@ -9,6 +9,7 @@ package com.facebook.react.uimanager
 
 import com.facebook.react.bridge.ReadableNativeMap
 import com.facebook.react.bridge.WritableMap
+import com.facebook.react.common.annotations.UnstableReactNativeAPI
 import com.facebook.react.common.mapbuffer.ReadableMapBuffer
 
 /**
@@ -20,6 +21,7 @@ public interface StateWrapper {
   /**
    * Maps to EventQueue::UpdateMode in C++. Controls how state updates are flushed (Async or Sync).
    */
+  @UnstableReactNativeAPI
   public enum class UpdateMode(public val value: Int) {
     Asynchronous(0),
     unstable_Immediate(1),
@@ -39,9 +41,15 @@ public interface StateWrapper {
 
   /**
    * Pass a map of values back to the C++ layer. The operation is performed synchronously and cannot
-   * fail. updateMode controls whether the update is queued asynchronously or flushed immediately.
+   * fail.
    */
-  public fun updateState(map: WritableMap, updateMode: UpdateMode = UpdateMode.Asynchronous)
+  public fun updateState(map: WritableMap)
+
+  /**
+   * Pass a map of values back to the C++ layer. updateMode controls whether the update is queued
+   * asynchronously or flushed immediately.
+   */
+  @UnstableReactNativeAPI public fun updateState(map: WritableMap, updateMode: UpdateMode)
 
   /**
    * Mark state as unused and clean up in Java and in native. This should be called as early as

packages/react-native/ReactAndroid/src/test/java/com/facebook/react/uimanager/StateWrapperUpdateModeTest.kt

@@ -7,6 +7,7 @@
 
 package com.facebook.react.uimanager
 
+import com.facebook.react.common.annotations.UnstableReactNativeAPI
 import com.facebook.react.uimanager.StateWrapper.UpdateMode
 import org.assertj.core.api.Assertions.assertThat
 import org.junit.Test
@@ -18,6 +19,7 @@ import org.robolectric.RobolectricTestRunner
  * must match its order (Asynchronous = 0, unstable_Immediate = 1). A mismatch silently flips
  * async/sync state updates.
  */
+@OptIn(UnstableReactNativeAPI::class)
 @RunWith(RobolectricTestRunner::class)
 class StateWrapperUpdateModeTest {