feat!: experimental Rive runtime backend (iOS + Android) by mfazekas · Pull Request #134 · rive-app/rive-nitro-react-native

mfazekas · 2026-01-23T11:00:11Z

Adds a new native backend using Rive's experimental runtime APIs on both iOS and Android. The new backend is async-native — all ViewModel operations go through a CommandQueue, eliminating the need for blockingAsync/runBlocking wrappers on the non-deprecated API surface.

The experimental backend is now the default. Legacy backend files are moved to ios/legacy/ and android/src/legacy/ (identical to main except getEnums() stub and backend property). New implementations live in ios/new/ and android/src/new/. CI runs tests on both backends.

Release-please configured for beta prereleases (0.5.0-beta, published as @next).

Opting into the legacy backend

# iOS
USE_RIVE_LEGACY=1 pod install

# Android — add to gradle.properties
USE_RIVE_LEGACY=true

Without the flag, the experimental backend is used.

What works

ViewModel data binding (all property types: number, string, boolean, color, enum, trigger, image, list, artboard)
Property listeners (number, string, boolean, color, enum, trigger)
Nested ViewModels (viewModelAsync)
List operations (add, remove, swap, getInstanceAt)
Touch/pointer events (handled automatically by RiveUIView on iOS, custom implementation on Android)
play() / pause() (iOS: toggles isPaused; Android: fully implemented)
getEnums() for introspection
getPropertyCountAsync / getInstanceCountAsync

Known limitations

Android

defaultArtboardViewModel doesn't expose the ViewModel name — pending rive-android#443. This causes modelName/propertyCount/instanceCount to throw and viewModelAsync path validation to be skipped on those instances.
replaceViewModel is a no-op (not yet implemented)

iOS

reset() only pauses — doesn't actually reset the state machine

Both platforms

Deprecated legacy APIs (Rive Events, SMI inputs, text runs) are not available in the experimental runtime and throw when called
Image and list property listeners are no-ops (not yet available in experimental SDK)

github-actions

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

ktlint

🚫 [ktlint] standard:multiline-if-else _{reported by reviewdog 🐶}
Missing { ... }

rive-nitro-react-native/android/src/experimental/java/com/margelo/nitro/rive/ExperimentalAssetLoader.kt

Line 129 in dd06b3a

data[2] == 0x54.toByte() && data[3] == 0x4F.toByte()) return AssetType.FONT

🚫 [ktlint] standard:chain-method-continuation _{reported by reviewdog 🐶}
Expected newline before '.'

rive-nitro-react-native/android/src/experimental/java/com/margelo/nitro/rive/HybridRiveFile.kt

Line 120 in dd06b3a

enums.map { enum ->

🚫 [ktlint] standard:chain-method-continuation _{reported by reviewdog 🐶}
Expected newline before '.'

rive-nitro-react-native/android/src/experimental/java/com/margelo/nitro/rive/HybridRiveFile.kt

Line 160 in dd06b3a

file.getViewModelProperties(name)

🚫 [ktlint] standard:try-catch-finally-spacing _{reported by reviewdog 🐶}
Expected a newline after '{'

rive-nitro-react-native/android/src/experimental/java/com/margelo/nitro/rive/HybridRiveFile.kt

Line 167 in dd06b3a

} catch (_: Exception) { return vmNames.first() }

🚫 [ktlint] standard:statement-wrapping _{reported by reviewdog 🐶}
Missing newline after '{'

rive-nitro-react-native/android/src/experimental/java/com/margelo/nitro/rive/HybridRiveFile.kt

Line 167 in dd06b3a

} catch (_: Exception) { return vmNames.first() }

🚫 [ktlint] standard:statement-wrapping _{reported by reviewdog 🐶}
Missing newline before '}'

rive-nitro-react-native/android/src/experimental/java/com/margelo/nitro/rive/HybridRiveFile.kt

Line 167 in dd06b3a

} catch (_: Exception) { return vmNames.first() }

🚫 [ktlint] standard:try-catch-finally-spacing _{reported by reviewdog 🐶}
Expected a newline before '}'

rive-nitro-react-native/android/src/experimental/java/com/margelo/nitro/rive/HybridRiveFile.kt

Line 167 in dd06b3a

} catch (_: Exception) { return vmNames.first() }

🚫 [ktlint] standard:try-catch-finally-spacing _{reported by reviewdog 🐶}
Expected a newline after '{'

rive-nitro-react-native/android/src/experimental/java/com/margelo/nitro/rive/HybridRiveFile.kt

Line 174 in dd06b3a

} catch (_: Exception) { continue }

🚫 [ktlint] standard:statement-wrapping _{reported by reviewdog 🐶}
Missing newline after '{'

rive-nitro-react-native/android/src/experimental/java/com/margelo/nitro/rive/HybridRiveFile.kt

Line 174 in dd06b3a

} catch (_: Exception) { continue }

🚫 [ktlint] standard:statement-wrapping _{reported by reviewdog 🐶}
Missing newline before '}'

rive-nitro-react-native/android/src/experimental/java/com/margelo/nitro/rive/HybridRiveFile.kt

Line 174 in dd06b3a

} catch (_: Exception) { continue }

🚫 [ktlint] standard:try-catch-finally-spacing _{reported by reviewdog 🐶}
Expected a newline before '}'

rive-nitro-react-native/android/src/experimental/java/com/margelo/nitro/rive/HybridRiveFile.kt

Line 174 in dd06b3a

} catch (_: Exception) { continue }

🚫 [ktlint] standard:no-unused-imports _{reported by reviewdog 🐶}
Unused import

rive-nitro-react-native/android/src/experimental/java/com/margelo/nitro/rive/HybridViewModel.kt

Line 7 in dd06b3a

import app.rive.ViewModelInstanceSource

🚫 [ktlint] standard:string-template _{reported by reviewdog 🐶}
Redundant curly braces

rive-nitro-react-native/android/src/experimental/java/com/rive/RiveReactNativeView.kt

Line 83 in dd06b3a

    
           Log.d(TAG, "onSurfaceTextureAvailable: ${w}x${h} worker=${this@RiveReactNativeView.riveWorker != null}")

🚫 [ktlint] standard:if-else-wrapping _{reported by reviewdog 🐶}
Expected a newline

rive-nitro-react-native/android/src/experimental/java/com/rive/RiveReactNativeView.kt

Line 117 in dd06b3a

val deltaTime = if (lastFrameTimeNs == 0L) Duration.ZERO

🚫 [ktlint] standard:multiline-if-else _{reported by reviewdog 🐶}
Missing { ... }

rive-nitro-react-native/android/src/experimental/java/com/rive/RiveReactNativeView.kt

Line 117 in dd06b3a

val deltaTime = if (lastFrameTimeNs == 0L) Duration.ZERO

🚫 [ktlint] standard:if-else-wrapping _{reported by reviewdog 🐶}
Expected a newline

rive-nitro-react-native/android/src/experimental/java/com/rive/RiveReactNativeView.kt

Line 118 in dd06b3a

else (frameTimeNanos - lastFrameTimeNs).nanoseconds

🚫 [ktlint] standard:multiline-if-else _{reported by reviewdog 🐶}
Missing { ... }

rive-nitro-react-native/android/src/experimental/java/com/rive/RiveReactNativeView.kt

Line 118 in dd06b3a

else (frameTimeNanos - lastFrameTimeNs).nanoseconds

🚫 [ktlint] standard:chain-method-continuation _{reported by reviewdog 🐶}
Expected newline before '.'

rive-nitro-react-native/android/src/experimental/java/com/rive/RiveReactNativeView.kt

Line 159 in dd06b3a

    
           Log.d(TAG, "configure: reload=$reload initialUpdate=$initialUpdate fit=$activeFit surfaceTexture=${surfaceTexture != null} surfaceW=${surfaceWidth} surfaceH=${surfaceHeight}")

🚫 [ktlint] standard:argument-list-wrapping _{reported by reviewdog 🐶}
Argument should be on a separate line (unless all arguments can fit a single line)

rive-nitro-react-native/android/src/experimental/java/com/rive/RiveReactNativeView.kt

Line 159 in dd06b3a

    
           Log.d(TAG, "configure: reload=$reload initialUpdate=$initialUpdate fit=$activeFit surfaceTexture=${surfaceTexture != null} surfaceW=${surfaceWidth} surfaceH=${surfaceHeight}")

🚫 [ktlint] standard:argument-list-wrapping _{reported by reviewdog 🐶}
Argument should be on a separate line (unless all arguments can fit a single line)

rive-nitro-react-native/android/src/experimental/java/com/rive/RiveReactNativeView.kt

Line 159 in dd06b3a

    
           Log.d(TAG, "configure: reload=$reload initialUpdate=$initialUpdate fit=$activeFit surfaceTexture=${surfaceTexture != null} surfaceW=${surfaceWidth} surfaceH=${surfaceHeight}")

🚫 [ktlint] standard:string-template _{reported by reviewdog 🐶}
Redundant curly braces

rive-nitro-react-native/android/src/experimental/java/com/rive/RiveReactNativeView.kt

Line 159 in dd06b3a

    
           Log.d(TAG, "configure: reload=$reload initialUpdate=$initialUpdate fit=$activeFit surfaceTexture=${surfaceTexture != null} surfaceW=${surfaceWidth} surfaceH=${surfaceHeight}")

🚫 [ktlint] standard:max-line-length _{reported by reviewdog 🐶}
Exceeded max line length (140)

rive-nitro-react-native/android/src/experimental/java/com/rive/RiveReactNativeView.kt

Line 159 in dd06b3a

    
           Log.d(TAG, "configure: reload=$reload initialUpdate=$initialUpdate fit=$activeFit surfaceTexture=${surfaceTexture != null} surfaceW=${surfaceWidth} surfaceH=${surfaceHeight}")

🚫 [ktlint] standard:string-template _{reported by reviewdog 🐶}
Redundant curly braces

rive-nitro-react-native/android/src/experimental/java/com/rive/RiveReactNativeView.kt

Line 159 in dd06b3a

    
           Log.d(TAG, "configure: reload=$reload initialUpdate=$initialUpdate fit=$activeFit surfaceTexture=${surfaceTexture != null} surfaceW=${surfaceWidth} surfaceH=${surfaceHeight}")

🚫 [ktlint] standard:argument-list-wrapping _{reported by reviewdog 🐶}
Missing newline before ")"

rive-nitro-react-native/android/src/experimental/java/com/rive/RiveReactNativeView.kt

Line 159 in dd06b3a

    
           Log.d(TAG, "configure: reload=$reload initialUpdate=$initialUpdate fit=$activeFit surfaceTexture=${surfaceTexture != null} surfaceW=${surfaceWidth} surfaceH=${surfaceHeight}")

🚫 [ktlint] standard:statement-wrapping _{reported by reviewdog 🐶}
Missing newline after '{'

rive-nitro-react-native/android/src/experimental/java/com/rive/RiveReactNativeView.kt

Line 218 in dd06b3a

val worker = riveWorker ?: run { Log.w(TAG, "touch: no worker"); return }

🚫 [ktlint] standard:statement-wrapping _{reported by reviewdog 🐶}
Missing newline after ';'

rive-nitro-react-native/android/src/experimental/java/com/rive/RiveReactNativeView.kt

Line 218 in dd06b3a

val worker = riveWorker ?: run { Log.w(TAG, "touch: no worker"); return }

🚫 [ktlint] standard:statement-wrapping _{reported by reviewdog 🐶}
Missing newline before '}'

rive-nitro-react-native/android/src/experimental/java/com/rive/RiveReactNativeView.kt

Line 218 in dd06b3a

val worker = riveWorker ?: run { Log.w(TAG, "touch: no worker"); return }

🚫 [ktlint] standard:statement-wrapping _{reported by reviewdog 🐶}
Missing newline after '{'

rive-nitro-react-native/android/src/experimental/java/com/rive/RiveReactNativeView.kt

Line 219 in dd06b3a

    
           val smHandle = stateMachineHandle ?: run { Log.w(TAG, "touch: no smHandle"); return }

🚫 [ktlint] standard:statement-wrapping _{reported by reviewdog 🐶}
Missing newline after ';'

rive-nitro-react-native/android/src/experimental/java/com/rive/RiveReactNativeView.kt

Line 219 in dd06b3a

    
           val smHandle = stateMachineHandle ?: run { Log.w(TAG, "touch: no smHandle"); return }

🚫 [ktlint] standard:statement-wrapping _{reported by reviewdog 🐶}
Missing newline before '}'

rive-nitro-react-native/android/src/experimental/java/com/rive/RiveReactNativeView.kt

Line 219 in dd06b3a

    
           val smHandle = stateMachineHandle ?: run { Log.w(TAG, "touch: no smHandle"); return }

🚫 [ktlint] standard:if-else-wrapping _{reported by reviewdog 🐶}
A single line if-statement should be kept simple. The 'THEN' may not be wrapped in a block.

rive-nitro-react-native/android/src/experimental/java/com/rive/RiveReactNativeView.kt

Line 222 in dd06b3a

    
           if (w <= 0 || h <= 0) { Log.w(TAG, "touch: invalid surface ${w}x${h}"); return }

🚫 [ktlint] standard:statement-wrapping _{reported by reviewdog 🐶}
Missing newline after '{'

rive-nitro-react-native/android/src/experimental/java/com/rive/RiveReactNativeView.kt

Line 222 in dd06b3a

    
           if (w <= 0 || h <= 0) { Log.w(TAG, "touch: invalid surface ${w}x${h}"); return }

🚫 [ktlint] standard:string-template _{reported by reviewdog 🐶}
Redundant curly braces

rive-nitro-react-native/android/src/experimental/java/com/rive/RiveReactNativeView.kt

Line 222 in dd06b3a

    
           if (w <= 0 || h <= 0) { Log.w(TAG, "touch: invalid surface ${w}x${h}"); return }

🚫 [ktlint] standard:statement-wrapping _{reported by reviewdog 🐶}
Missing newline after ';'

rive-nitro-react-native/android/src/experimental/java/com/rive/RiveReactNativeView.kt

Line 222 in dd06b3a

    
           if (w <= 0 || h <= 0) { Log.w(TAG, "touch: invalid surface ${w}x${h}"); return }

🚫 [ktlint] standard:statement-wrapping _{reported by reviewdog 🐶}
Missing newline before '}'

rive-nitro-react-native/android/src/experimental/java/com/rive/RiveReactNativeView.kt

Line 222 in dd06b3a

    
           if (w <= 0 || h <= 0) { Log.w(TAG, "touch: invalid surface ${w}x${h}"); return }

🚫 [ktlint] standard:chain-method-continuation _{reported by reviewdog 🐶}
Expected newline before '.'

rive-nitro-react-native/example/android/app/src/main/java/rive/example/ComposeTestActivity.kt

Line 56 in dd06b3a

val legacyFile = app.rive.runtime.kotlin.core.File(bytes)

🚫 [ktlint] standard:chain-method-continuation _{reported by reviewdog 🐶}
Expected newline before '.'

rive-nitro-react-native/example/android/app/src/main/java/rive/example/ComposeTestActivity.kt

Line 66 in dd06b3a

    
           Log.d("ComposeRiveTest", "[$label]   input[$j]: name=${input.name} isBoolean=${input.isBoolean} isTrigger=${input.isTrigger} isNumber=${input.isNumber}")

🚫 [ktlint] standard:argument-list-wrapping _{reported by reviewdog 🐶}
Argument should be on a separate line (unless all arguments can fit a single line)

rive-nitro-react-native/example/android/app/src/main/java/rive/example/ComposeTestActivity.kt

Line 66 in dd06b3a

    
           Log.d("ComposeRiveTest", "[$label]   input[$j]: name=${input.name} isBoolean=${input.isBoolean} isTrigger=${input.isTrigger} isNumber=${input.isNumber}")

🚫 [ktlint] standard:argument-list-wrapping _{reported by reviewdog 🐶}
Argument should be on a separate line (unless all arguments can fit a single line)

rive-nitro-react-native/example/android/app/src/main/java/rive/example/ComposeTestActivity.kt

Line 66 in dd06b3a

    
           Log.d("ComposeRiveTest", "[$label]   input[$j]: name=${input.name} isBoolean=${input.isBoolean} isTrigger=${input.isTrigger} isNumber=${input.isNumber}")

🚫 [ktlint] standard:max-line-length _{reported by reviewdog 🐶}
Exceeded max line length (140)

rive-nitro-react-native/example/android/app/src/main/java/rive/example/ComposeTestActivity.kt

Line 66 in dd06b3a

    
           Log.d("ComposeRiveTest", "[$label]   input[$j]: name=${input.name} isBoolean=${input.isBoolean} isTrigger=${input.isTrigger} isNumber=${input.isNumber}")

🚫 [ktlint] standard:argument-list-wrapping _{reported by reviewdog 🐶}
Missing newline before ")"

rive-nitro-react-native/example/android/app/src/main/java/rive/example/ComposeTestActivity.kt

Line 66 in dd06b3a

    
           Log.d("ComposeRiveTest", "[$label]   input[$j]: name=${input.name} isBoolean=${input.isBoolean} isTrigger=${input.isTrigger} isNumber=${input.isNumber}")

🚫 [ktlint] standard:function-naming _{reported by reviewdog 🐶}
Function name should start with a lowercase letter (except factory methods) and use camel case

rive-nitro-react-native/example/android/app/src/main/java/rive/example/ComposeTestActivity.kt

Line 82 in dd06b3a

fun RiveContent() {

🚫 [ktlint] standard:chain-method-continuation _{reported by reviewdog 🐶}
Expected newline before '.'

rive-nitro-react-native/example/android/app/src/main/java/rive/example/ComposeTestActivity.kt

Line 102 in dd06b3a

    
           Log.d("ComposeRiveTest", "artboard=${artboard.artboardHandle} sm=${stateMachine.stateMachineHandle} name=${artboard.name} smName=${stateMachine.name}")

🚫 [ktlint] standard:argument-list-wrapping _{reported by reviewdog 🐶}
Argument should be on a separate line (unless all arguments can fit a single line)

rive-nitro-react-native/example/android/app/src/main/java/rive/example/ComposeTestActivity.kt

Line 102 in dd06b3a

    
           Log.d("ComposeRiveTest", "artboard=${artboard.artboardHandle} sm=${stateMachine.stateMachineHandle} name=${artboard.name} smName=${stateMachine.name}")

🚫 [ktlint] standard:argument-list-wrapping _{reported by reviewdog 🐶}
Argument should be on a separate line (unless all arguments can fit a single line)

rive-nitro-react-native/example/android/app/src/main/java/rive/example/ComposeTestActivity.kt

Line 102 in dd06b3a

    
           Log.d("ComposeRiveTest", "artboard=${artboard.artboardHandle} sm=${stateMachine.stateMachineHandle} name=${artboard.name} smName=${stateMachine.name}")

🚫 [ktlint] standard:max-line-length _{reported by reviewdog 🐶}
Exceeded max line length (140)

rive-nitro-react-native/example/android/app/src/main/java/rive/example/ComposeTestActivity.kt

Line 102 in dd06b3a

    
           Log.d("ComposeRiveTest", "artboard=${artboard.artboardHandle} sm=${stateMachine.stateMachineHandle} name=${artboard.name} smName=${stateMachine.name}")

🚫 [ktlint] standard:argument-list-wrapping _{reported by reviewdog 🐶}
Missing newline before ")"

rive-nitro-react-native/example/android/app/src/main/java/rive/example/ComposeTestActivity.kt

Line 102 in dd06b3a

    
           Log.d("ComposeRiveTest", "artboard=${artboard.artboardHandle} sm=${stateMachine.stateMachineHandle} name=${artboard.name} smName=${stateMachine.name}")

github-actions

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

ktlint

🚫 [ktlint] standard:if-else-wrapping _{reported by reviewdog 🐶}
A single line if-statement should be kept simple. The 'THEN' may not be wrapped in a block.

rive-nitro-react-native/android/src/experimental/java/com/rive/RiveReactNativeView.kt

Line 222 in a5855c9

    
           if (w <= 0 || h <= 0) { Log.w(TAG, "touch: invalid surface ${w}x${h}"); return }

🚫 [ktlint] standard:statement-wrapping _{reported by reviewdog 🐶}
Missing newline after '{'

rive-nitro-react-native/android/src/experimental/java/com/rive/RiveReactNativeView.kt

Line 222 in a5855c9

    
           if (w <= 0 || h <= 0) { Log.w(TAG, "touch: invalid surface ${w}x${h}"); return }

🚫 [ktlint] standard:string-template _{reported by reviewdog 🐶}
Redundant curly braces

rive-nitro-react-native/android/src/experimental/java/com/rive/RiveReactNativeView.kt

Line 222 in a5855c9

    
           if (w <= 0 || h <= 0) { Log.w(TAG, "touch: invalid surface ${w}x${h}"); return }

🚫 [ktlint] standard:statement-wrapping _{reported by reviewdog 🐶}
Missing newline after ';'

rive-nitro-react-native/android/src/experimental/java/com/rive/RiveReactNativeView.kt

Line 222 in a5855c9

    
           if (w <= 0 || h <= 0) { Log.w(TAG, "touch: invalid surface ${w}x${h}"); return }

🚫 [ktlint] standard:statement-wrapping _{reported by reviewdog 🐶}
Missing newline before '}'

rive-nitro-react-native/android/src/experimental/java/com/rive/RiveReactNativeView.kt

Line 222 in a5855c9

    
           if (w <= 0 || h <= 0) { Log.w(TAG, "touch: invalid surface ${w}x${h}"); return }

🚫 [ktlint] standard:chain-method-continuation _{reported by reviewdog 🐶}
Expected newline before '.'

rive-nitro-react-native/example/android/app/src/main/java/rive/example/ComposeTestActivity.kt

Line 56 in a5855c9

val legacyFile = app.rive.runtime.kotlin.core.File(bytes)

🚫 [ktlint] standard:chain-method-continuation _{reported by reviewdog 🐶}
Expected newline before '.'

rive-nitro-react-native/example/android/app/src/main/java/rive/example/ComposeTestActivity.kt

Line 66 in a5855c9

    
           Log.d("ComposeRiveTest", "[$label]   input[$j]: name=${input.name} isBoolean=${input.isBoolean} isTrigger=${input.isTrigger} isNumber=${input.isNumber}")

🚫 [ktlint] standard:argument-list-wrapping _{reported by reviewdog 🐶}
Argument should be on a separate line (unless all arguments can fit a single line)

rive-nitro-react-native/example/android/app/src/main/java/rive/example/ComposeTestActivity.kt

Line 66 in a5855c9

    
           Log.d("ComposeRiveTest", "[$label]   input[$j]: name=${input.name} isBoolean=${input.isBoolean} isTrigger=${input.isTrigger} isNumber=${input.isNumber}")

🚫 [ktlint] standard:argument-list-wrapping _{reported by reviewdog 🐶}
Argument should be on a separate line (unless all arguments can fit a single line)

rive-nitro-react-native/example/android/app/src/main/java/rive/example/ComposeTestActivity.kt

Line 66 in a5855c9

    
           Log.d("ComposeRiveTest", "[$label]   input[$j]: name=${input.name} isBoolean=${input.isBoolean} isTrigger=${input.isTrigger} isNumber=${input.isNumber}")

🚫 [ktlint] standard:max-line-length _{reported by reviewdog 🐶}
Exceeded max line length (140)

rive-nitro-react-native/example/android/app/src/main/java/rive/example/ComposeTestActivity.kt

Line 66 in a5855c9

    
           Log.d("ComposeRiveTest", "[$label]   input[$j]: name=${input.name} isBoolean=${input.isBoolean} isTrigger=${input.isTrigger} isNumber=${input.isNumber}")

🚫 [ktlint] standard:argument-list-wrapping _{reported by reviewdog 🐶}
Missing newline before ")"

rive-nitro-react-native/example/android/app/src/main/java/rive/example/ComposeTestActivity.kt

Line 66 in a5855c9

    
           Log.d("ComposeRiveTest", "[$label]   input[$j]: name=${input.name} isBoolean=${input.isBoolean} isTrigger=${input.isTrigger} isNumber=${input.isNumber}")

🚫 [ktlint] standard:function-naming _{reported by reviewdog 🐶}
Function name should start with a lowercase letter (except factory methods) and use camel case

rive-nitro-react-native/example/android/app/src/main/java/rive/example/ComposeTestActivity.kt

Line 82 in a5855c9

fun RiveContent() {

🚫 [ktlint] standard:chain-method-continuation _{reported by reviewdog 🐶}
Expected newline before '.'

rive-nitro-react-native/example/android/app/src/main/java/rive/example/ComposeTestActivity.kt

Line 102 in a5855c9

    
           Log.d("ComposeRiveTest", "artboard=${artboard.artboardHandle} sm=${stateMachine.stateMachineHandle} name=${artboard.name} smName=${stateMachine.name}")

🚫 [ktlint] standard:argument-list-wrapping _{reported by reviewdog 🐶}
Argument should be on a separate line (unless all arguments can fit a single line)

rive-nitro-react-native/example/android/app/src/main/java/rive/example/ComposeTestActivity.kt

Line 102 in a5855c9

    
           Log.d("ComposeRiveTest", "artboard=${artboard.artboardHandle} sm=${stateMachine.stateMachineHandle} name=${artboard.name} smName=${stateMachine.name}")

🚫 [ktlint] standard:argument-list-wrapping _{reported by reviewdog 🐶}
Argument should be on a separate line (unless all arguments can fit a single line)

rive-nitro-react-native/example/android/app/src/main/java/rive/example/ComposeTestActivity.kt

Line 102 in a5855c9

    
           Log.d("ComposeRiveTest", "artboard=${artboard.artboardHandle} sm=${stateMachine.stateMachineHandle} name=${artboard.name} smName=${stateMachine.name}")

🚫 [ktlint] standard:max-line-length _{reported by reviewdog 🐶}
Exceeded max line length (140)

rive-nitro-react-native/example/android/app/src/main/java/rive/example/ComposeTestActivity.kt

Line 102 in a5855c9

    
           Log.d("ComposeRiveTest", "artboard=${artboard.artboardHandle} sm=${stateMachine.stateMachineHandle} name=${artboard.name} smName=${stateMachine.name}")

🚫 [ktlint] standard:argument-list-wrapping _{reported by reviewdog 🐶}
Missing newline before ")"

rive-nitro-react-native/example/android/app/src/main/java/rive/example/ComposeTestActivity.kt

Line 102 in a5855c9

    
           Log.d("ComposeRiveTest", "artboard=${artboard.artboardHandle} sm=${stateMachine.stateMachineHandle} name=${artboard.name} smName=${stateMachine.name}")

…roid-auto-databind

…ndroid When auto-binding, cppDefaultVMCreateDefaultVMI returns handle 1L (null sentinel) if the artboard has no default ViewModel. Guard against passing this sentinel to bindViewModelInstance.

…lock

… return type

…VE_LEGACY

… iOS

…ackends

…asks

…ailures

…E, check WebP explicitly

…ReactNativeView

…allable methods Nitro calls hybrid methods on the JS thread, not the main thread. MainActor.assumeIsolated crashes at runtime when not on main.

…recursion

…tch boundary

Copilot

Pull request overview

Copilot reviewed 74 out of 153 changed files in this pull request and generated 5 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-04-09T12:58:21Z

src/hooks/useViewModelInstance.ts

+    try {
+      vmi = source.createInstanceByName(instanceName);
+    } catch {
+      // experimental backend throws for non-existent names
+    }


The catch block around source.createInstanceByName() swallows all exceptions with no logging. This can hide unexpected backend errors (not just “instance not found”) and make debugging very difficult. Consider logging the caught error (like the RiveFile path above) or narrowing the catch to only the expected “not found” failure mode.

Copilot · 2026-04-09T12:58:22Z

src/specs/RiveFile.nitro.ts

+  /**
+   * Get all enums defined in this Rive file.
+   * Useful for debugging and building dynamic UIs.
+   * @experimental Uses the experimental Rive API on iOS
+   */
+  getEnums(): Promise<RiveEnumDefinition[]>;


The getEnums() doc comment says it uses the experimental iOS API, but the method is also implemented on Android (including legacy) and is stubbed to throw on iOS legacy. Consider updating the doc to accurately describe backend/platform support (e.g., supported on experimental iOS + Android; throws on iOS legacy).

Copilot · 2026-04-09T12:58:22Z

android/src/new/java/com/margelo/nitro/rive/HybridViewModel.kt

+  // Null when constructed via DefaultForArtboard — the Rive Android SDK does not expose
+  // the ViewModel name from a ViewModelInstance, so name-dependent operations are unavailable.
+  // Track: https://github.com/rive-app/rive-android/issues/XXX
+  private val viewModelName: String?,
+  private val parentFile: HybridRiveFile,
+  private val vmSource: ViewModelSource
+) : HybridViewModelSpec() {
+  companion object {
+    private const val TAG = "HybridViewModel"
+    private const val NO_NAME_ERROR =
+      "This operation requires the ViewModel name, which is unavailable for ViewModels " +
+      "obtained via defaultArtboardViewModel(). The Rive Android SDK does not yet expose " +
+      "the ViewModel name from a ViewModelInstance. Use a named ViewModel instead, or " +
+      "track the upstream fix: https://github.com/rive-app/rive-android/issues/XXX"


NO_NAME_ERROR (and the preceding comment) contains a placeholder upstream link (.../issues/XXX). Since this string is thrown to JS as an exception, it should reference the real upstream issue/PR (e.g. rive-android#443) or omit the link to avoid sending users to a dead URL.

Copilot · 2026-04-09T12:58:22Z

android/src/new/java/com/margelo/nitro/rive/HybridRiveFile.kt

+    val vmSource = ViewModelSource.DefaultForArtboard(artboard)
+    // Name is null because the Rive Android SDK does not expose the ViewModel name
+    // from a ViewModelInstance — name-dependent operations will throw UnsupportedOperationException.
+    // Track upstream: https://github.com/rive-app/rive-android/issues/XXX


This comment references a placeholder upstream URL (.../issues/XXX). Since it documents a known limitation, it should point to the real tracking issue/PR so future maintainers/users can follow progress.

Suggested change

// Track upstream: https://github.com/rive-app/rive-android/issues/XXX

Copilot · 2026-04-09T12:58:23Z

example/src/reproducers/local/Issue159.tsx

+/**
+ * Reproduces issue #159 — Rive graphics stutter when JS/UI thread is under heavy load.
+ *
+ * Loads vehicles.riv from URL (endless animation).
+ * Two buttons: block JS thread or block UI thread for ~60s.
+ * If the vehicles stop animating, rendering depends on that thread.
+ */
+
+const VEHICLES_URL = require('../../../assets/rive/rewards.riv');
+


The file/constant naming and comment are inconsistent: the header says this loads vehicles.riv from a URL, but VEHICLES_URL is a local require() of rewards.riv. Consider either switching to the intended remote vehicles URL or updating the comment/constant name so the reproducer matches what it actually loads.

mfazekas force-pushed the feat/rive-ios-experimental branch 2 times, most recently from 95816cf to 4fe9e12 Compare January 23, 2026 11:23

mfazekas force-pushed the feat/rive-ios-experimental branch 4 times, most recently from 8485f9f to 9b4acd8 Compare February 9, 2026 10:25

mfazekas changed the title ~~feat: experimental iOS API support (getEnums via SPM)~~ feat: experimental iOS API support Feb 9, 2026

github-actions bot reviewed Feb 16, 2026

View reviewed changes

mfazekas force-pushed the feat/rive-ios-experimental branch from dd06b3a to a5855c9 Compare February 16, 2026 10:41

github-actions bot reviewed Feb 16, 2026

View reviewed changes

mfazekas force-pushed the feat/rive-ios-experimental branch from 8134a07 to ec12673 Compare February 17, 2026 12:11

mfazekas changed the title ~~feat: experimental iOS API support~~ WIP feat: experimental iOS / Android POC Feb 17, 2026

mfazekas force-pushed the feat/rive-ios-experimental branch 2 times, most recently from 91e2fb6 to cfb2ff6 Compare February 19, 2026 13:50

mfazekas force-pushed the feat/rive-ios-experimental branch from cfb2ff6 to 44681b5 Compare February 26, 2026 13:57

mfazekas force-pushed the feat/rive-ios-experimental branch 2 times, most recently from f1b851e to aa2fdf0 Compare March 16, 2026 09:39

mfazekas force-pushed the feat/rive-ios-experimental branch 4 times, most recently from 35c6fea to 4f1ab3f Compare March 27, 2026 15:52

mfazekas changed the base branch from main to feat/hooks-undefined-initial-value March 27, 2026 15:52

mfazekas force-pushed the feat/rive-ios-experimental branch from 4e0c53a to be6d334 Compare March 30, 2026 12:39

mfazekas force-pushed the feat/hooks-undefined-initial-value branch from 164180e to 7116ac7 Compare March 30, 2026 18:17

Base automatically changed from feat/hooks-undefined-initial-value to main March 30, 2026 18:20

mfazekas force-pushed the feat/rive-ios-experimental branch from dcdde5e to 0a3d09c Compare March 31, 2026 05:33

mfazekas changed the title ~~WIP feat: experimental iOS / Android POC~~ feat: experimental Rive runtime backend (iOS + Android) Mar 31, 2026

mfazekas marked this pull request as ready for review March 31, 2026 07:31

mfazekas changed the title ~~feat: experimental Rive runtime backend (iOS + Android)~~ feat!: experimental Rive runtime backend (iOS + Android) Mar 31, 2026

mfazekas requested a review from HayesGordon March 31, 2026 14:18

mfazekas added 15 commits April 8, 2026 15:31

fix: use yarn nitrogen (with postprocess), fix ktlint blank line

29ecca9

chore: add Issue189 reproducer and nodefaultbouncing.riv from fix/and…

14ebb75

…roid-auto-databind

fix(ios-exp): skip auto-binding when artboard has no default ViewModel

84372f4

Fix "0x1 not found" error when artboard has no default ViewModel on A…

d58ce51

…ndroid When auto-binding, cppDefaultVMCreateDefaultVMI returns handle 1L (null sentinel) if the artboard has no default ViewModel. Guard against passing this sentinel to bindViewModelInstance.

fix: enable experimental API flag, await Worker init, update Podfile.…

5408112

…lock

fix: update test harnesses for useViewModelInstance {instance, error}…

9d956d5

… return type

fix: ktlint spacing between declarations with comments

34d275a

test: skip Android experimental tests that need rive-android#443

054ad1b

test: clarify skip reasons for Android experimental test guards

0547574

fix: clean up legacy files to match upstream, fix Issue159 error type

5d4a35c

fix: make legacy HybridViewModel identical to main

6346b4c

feat: make experimental runtime the default, legacy opt-in via USE_RI…

b8b9d78

…VE_LEGACY

refactor: rename android/src/experimental to android/src/new to match…

33b2c4a

… iOS

fix: add missing getPropertyCountAsync/getInstanceCountAsync to new b…

6ac0dd5

…ackends

chore: enable prerelease (beta) versioning for 0.5.0 release track

e9ee276

mfazekas force-pushed the feat/rive-ios-experimental branch from 5c7cd87 to e9ee276 Compare April 8, 2026 13:33

mfazekas added 11 commits April 8, 2026 16:46

fix(ios): add @mainactor to RiveReactNativeView, guard stale config t…

e2bb152

…asks

fix(ios): make awaitViewReady reentrant, support multiple callers

9afc78e

fix(android): use RuntimeException instead of Error for recoverable f…

c888aa7

…ailures

fix: log error instead of swallowing in createInstanceByName catch

bb06d88

fix(android): log warning when inferFromMagicBytes falls back to IMAG…

a708e08

…E, check WebP explicitly

fix(ios): add MainActor.assumeIsolated for calls into @mainactor Rive…

a5a7f01

…ReactNativeView

fix(ios): port Xcode 26 ODR workaround from main into podspec

018e92c

fix(ios): add missing try to MainActor.assumeIsolated in afterUpdate

a212ec7

fix(ios): use onMainSync instead of MainActor.assumeIsolated for JS-c…

5b09da0

…allable methods Nitro calls hybrid methods on the JS thread, not the main thread. MainActor.assumeIsolated crashes at runtime when not on main.

fix(ios): make onMainSync provide @mainactor isolation, fix infinite …

62c426b

…recursion

fix(ios): use Result to propagate errors from onMainSync across dispa…

c31b359

…tch boundary

mfazekas requested a review from Copilot April 9, 2026 12:51

Copilot started reviewing on behalf of mfazekas April 9, 2026 12:52 View session

Copilot AI reviewed Apr 9, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat!: experimental Rive runtime backend (iOS + Android)#134

feat!: experimental Rive runtime backend (iOS + Android)#134
mfazekas wants to merge 86 commits intomainfrom
feat/rive-ios-experimental

mfazekas commented Jan 23, 2026 •

edited

Loading

Uh oh!

github-actions bot left a comment

Uh oh!

github-actions bot left a comment

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Apr 9, 2026

Uh oh!

Copilot AI Apr 9, 2026

Uh oh!

Copilot AI Apr 9, 2026

Uh oh!

Copilot AI Apr 9, 2026

Uh oh!

Copilot AI Apr 9, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

mfazekas commented Jan 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Opting into the legacy backend

What works

Known limitations

Android

iOS

Both platforms

Uh oh!

github-actions bot left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions bot left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Apr 9, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 9, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 9, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 9, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 9, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

mfazekas commented Jan 23, 2026 •

edited

Loading