chore: add reset + fix: invoke play, pause to invoked from main (#53)

* chore: add reset + fix: invoke play, pause,

* fix: remove extra blank lines in Kotlin files

Author: mfazekas

android/src/main/java/com/margelo/nitro/rive/HybridRiveView.kt

@@ -112,9 +112,11 @@ class HybridRiveView(val context: ThemedReactContext) : HybridRiveViewSpec() {
     return HybridViewModelInstance(viewModelInstance)
   }
 
-  override fun play() = executeOnUiThread { view.play() }
+  override fun play() = asyncExecuteOnUiThread { view.play() }
 
-  override fun pause() = executeOnUiThread { view.pause() }
+  override fun pause() = asyncExecuteOnUiThread { view.pause() }
+
+  override fun reset() = asyncExecuteOnUiThread { view.reset() }
 
   override fun onEventListener(onEvent: (event: UnifiedRiveEvent) -> Unit) =
     executeOnUiThread { view.addEventListener(onEvent) }
@@ -185,6 +187,20 @@ class HybridRiveView(val context: ThemedReactContext) : HybridRiveViewSpec() {
     }
   }
 
+  private fun asyncExecuteOnUiThread(action: () -> Unit): Promise<Unit> {
+    return Promise.async {
+      context.currentActivity?.runOnUiThread() {
+        try {
+          action()
+        } catch (e: Exception) {
+          throw Error(e.message) // TODO: Correctly handling errors (https://nitro.margelo.com/docs/errors)
+        } catch (e: Error) {
+          throw Error(e.message)
+        }
+      }
+    }
+  }
+
   private fun executeOnUiThread(action: () -> Unit) {
     context.currentActivity?.runOnUiThread {
       try {

android/src/main/java/com/rive/RiveReactNativeView.kt

@@ -188,6 +188,8 @@ class RiveReactNativeView(context: ThemedReactContext) : FrameLayout(context) {
 
   fun pause() = riveAnimationView?.pause()
 
+  fun reset() = riveAnimationView?.reset()
+
   fun addEventListener(onEvent: (event: RNEvent) -> Unit) {
     val eventListener = object : RiveFileController.RiveEventListener {
       override fun notifyEvent(event: RiveEvent) {

ios/HybridRiveView.swift

@@ -66,6 +66,24 @@ where Wrapped == HybridDataBindMode {
 }
 
 class HybridRiveView: HybridRiveViewSpec {
+  func play() throws -> NitroModules.Promise<Void> {
+    return Promise.async {
+      try await self.getRiveView().play()
+    }
+  }
+
+  func pause() throws -> NitroModules.Promise<Void> {
+    return Promise.async {
+      try await self.getRiveView().pause()
+    }
+  }
+
+  func reset() throws -> NitroModules.Promise<Void> {
+    return Promise.async {
+      try await self.getRiveView().reset()
+    }
+  }
+
   // MARK: View Props
   var dataBind: HybridDataBindMode? {
     didSet {
@@ -103,10 +121,6 @@ class HybridRiveView: HybridRiveViewSpec {
     return HybridViewModelInstance(viewModelInstance: viewModelInstance)
   }
 
-  func play() throws { try getRiveView().play() }
-
-  func pause() throws { try getRiveView().pause() }
-
   func onEventListener(onEvent: @escaping (UnifiedRiveEvent) -> Void) throws {
     try getRiveView().addEventListener(onEvent)
   }

ios/RiveReactNativeView.swift

@@ -122,20 +122,22 @@ class RiveReactNativeView: UIView, RiveStateMachineDelegate {
       // this should be added if we support only playing artboards on their own - https://github.com/rive-app/rive-nitro-react-native/pull/23#discussion_r2534698281
       // artboard?.bind(viewModelInstance: instance)
     }
-    baseViewModel?.play()
+    play()
   }
 
+  @MainActor
   func play() {
     baseViewModel?.play()
   }
 
+  @MainActor
   func pause() {
     baseViewModel?.pause()
   }
 
   func refreshAfterAssetChange() {
     if baseViewModel?.isPlaying == false {
-      baseViewModel?.play()
+      play()
     }
   }