WIP feat: experimental iOS / Android POC by mfazekas · Pull Request #134 · rive-app/rive-nitro-react-native

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

Summary

Converts entire Rive API from async (Promise-based) to synchronous
Adds optional SPM integration for RiveRuntime 6.15.0 (experimental API)
Experimental iOS backend uses blockingAsync helper to bridge async rive-ios API to sync interface
Legacy iOS and Android backends simply remove Promise.async {} wrappers (underlying SDK is already sync)
Simplifies React hooks (useViewModelInstance, useRiveProperty) — no more useState + useEffect for async loading
Properties use value getter/setter instead of getValue()/setValue()
Android experimental backend: Custom TextureView renderer with Choreographer render loop, CommandQueue polling, ViewModel data binding via reflection, and property validation with error logging

Usage

# iOS experimental
USE_RIVE_SPM=1 pod install

# Android experimental
# Set in example/android/gradle.properties:
USE_RIVE_NEW_API=true

Without the flags, the legacy backends are used.

TODO

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}")

Add new experimental iOS backend (ios/new/) with synchronous API, move legacy backend files to ios/legacy/, add getEnums() support, retry listener streams on missingData, and restore TestComponentOverlay. Generated with [Claude Code](https://claude.ai/code) via [Happy](https://happy.engineering)

Generated with [Claude Code](https://claude.ai/code) via [Happy](https://happy.engineering)

getEnums() in legacy now throws directing users to the experimental backend instead of creating throwaway Worker+File instances. Generated with [Claude Code](https://claude.ai/code) via [Happy](https://happy.engineering)

…essing Generated with [Claude Code](https://claude.ai/code) via [Happy](https://happy.engineering)

… binding Each Worker has its own C++ command server with its own m_artboards handle map. Creating separate Workers per file meant artboard handles from one file were invalid on another file's server. Using a shared singleton Worker fixes cross-file artboard property set. Also wires fit/alignment through experimental Fit enum and improves asset type detection with audio/font magic bytes. Generated with [Claude Code](https://claude.ai/code) via [Happy](https://happy.engineering) Co-Authored-By: Claude <noreply@anthropic.com> Co-Authored-By: Happy <yesreply@happy.engineering>

Passing fit to the Rive() constructor breaks layout mode because the MTKView drawable isn't ready yet. Set rive.fit after setupRiveUIView() instead.

Port Flutter data binding tests for VM access, enums, creation variants, list properties, artboard and image properties. Includes new .riv test assets and react-native-harness upgrade to alpha.25.

…ceByIndex Prevents fatal crash when passing negative numbers to Swift APIs expecting unsigned integers.

CocoaPods doesn't embed SPM-resolved dynamic frameworks automatically. Patches the embed script to include RiveRuntime when USE_RIVE_SPM=1.

Move 19 backend-specific files to src/legacy/java/, add Gradle sourceSets switching via USE_RIVE_NEW_API property, prepare empty experimental dirs.

Uses the new handle-based Rive Android SDK (app.rive.*) with CommandQueue, path-based ViewModelInstance property access, and Kotlin Flows for reactivity. Throws UnsupportedOperationException for SMI inputs, text runs, and events. Also fixes Gradle property resolution to use rootProject.findProperty.

Custom TextureView renderer with Choreographer render loop, CommandQueue polling infrastructure, ViewModel resolution and property validation, and example Compose test activities for manual testing.

- Always use SPM for RiveRuntime (remove CocoaPods fallback) - Add RiveSPMEmbedFix module to auto-embed RiveRuntime.framework - Pin SPM to exactVersion instead of upToNextMajorVersion - Replace USE_RIVE_SPM with USE_RIVE_EXPERIMENTAL_RUNTIME to select legacy vs experimental backend independently of SPM Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

…ener All backends now emit the current value as the first listener emission, so hooks no longer need a synchronous property.value read on mount. - useRiveProperty: always starts undefined; value arrives via the listener's first callback (not a sync read). Setter updater fn now uses tracked React state instead of property.value. - iOS legacy: Number/String/Boolean/Enum addListener now emits the current value synchronously on subscribe, matching the stream-based behaviour of the experimental backend (valueStream) and Android (Flow with drop=0). No native changes needed elsewhere. - Tests: mock addListener emits current value on subscribe; updated test description to reflect the new async-first semantics. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

…erimental iOS test issues - Rename USE_RIVE_EXPERIMENTAL_RUNTIME to USE_RIVE_NEW_API (matches Android) - Remove USE_RIVE_SPM and all SPM embedding hacks from podspec/Podfile - Use standard CocoaPods dependency for RiveRuntime - Emit initial value in experimental addListener (number/string/bool/enum/color) - Guard tests that crash on experimental iOS (list ops, autoPlay, artboard/image loading) - Handle createInstanceByName throwing on experimental backend

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

…leFactory ArrayBufferHolder is a C++ type; the Swift protocol uses ArrayBuffer. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Deprecate the sync properties in favour of non-blocking async methods: - `viewModelCount` → `getViewModelCountAsync()` - `artboardCount` → `getArtboardCountAsync()` - `artboardNames` → `getArtboardNamesAsync()` Implemented across all four backends (iOS new/legacy, Android experimental/legacy) and regenerated nitrogen bindings. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Add getValueAsync(), getLengthAsync(), and getInstanceAtAsync() to all ViewModel property types (Number, String, Boolean, Color, Enum, List) across all four backends (iOS new/legacy, Android experimental/legacy). Sync versions are kept but deprecated in favour of the new async methods. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

…s, extract shared async helpers - Add missing `import com.margelo.nitro.core.Promise` to all 6 experimental Android ViewModel property files (Boolean, Color, Enum, List, Number, String) - Extract private async helpers in iOS new Color and List to eliminate non-trivial code duplication between deprecated sync and async methods Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

The Android Kotlin Style Guide (android: true) is not used by rive-android itself and introduces rules like chain-method-continuation that don't apply well to React Native bridge/Nitro bindings code. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

- Auto-fixed code style violations via ktlint --format - Disabled if-else-wrapping and condition-wrapping rules in .editorconfig (non-bug style opinions inconsistent with project's permissive approach) - Added scripts/lint-fix-kotlin.sh and yarn lint:fix:kotlin for auto-fixing Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

…ading Adds an optional `type` field ('image' | 'font' | 'audio') to ResolvedReferencedAsset so callers can declare asset types explicitly, avoiding the deprecated extension/magic-byte inference fallback. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

…pe field Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

…nctions Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Replace the old string-based `AssetType(fromExplicit:)` initializer and call site with the new `AssetType(from: RiveAssetType)` initializer that takes the typed enum directly from the nitrogen-generated `asset.type` property. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

…pe enum Replace string-based explicit type comparison with a proper enum `when` expression now that nitrogen generates `asset.type` as `RiveAssetType?` instead of `String?`. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

…ame, set() method - Add `validate?: boolean` (default true) to `viewModelByNameAsync` on all 4 backends; experimental backends skip the getViewModelNames() round-trip when false, legacy backends ignore it (native call already returns nil/null for unknown names) - Rename `createInstanceAsync()` → `createBlankInstanceAsync()` across spec and all 4 backends to make the semantics (blank/empty instance) explicit - Add `set(value)` fire-and-forget method to all 5 property types (Number, String, Boolean, Color, Enum) as the recommended write path; fully deprecate `value` getter and setter - Deprecate `viewModelCount`, `viewModelByIndex`, `viewModelByName` in favour of `getViewModelNamesAsync()` + `viewModelByNameAsync()`; remove removed `getViewModelCountAsync` / `viewModelByIndexAsync` variants - Regenerate nitrogen/ outputs and apply ktlint auto-format Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Nitrogen generates `func set(value: T)` with an external `value:` label, but our implementations had `func set(_ value: T)` with no external label. Fixes Swift build error: "instance method 'set' has different argument labels from those required by protocol 'Hybrid*PropertySpec_protocol' ('set(value:)')". Also updates the deprecated `value` setter delegate to `try? set(value: newValue)`. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Adds an explicit test for the contract that `useRiveNumber` delivers its initial value purely from the first listener emission on subscribe — no prop change is required. Uses a tight 1 s timeout to make it fail fast if the hook ever accidentally regresses to relying on a prop update cycle. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Mirror of the existing Data Binding exerciser but with WithViewModelSetup replaced to use `defaultArtboardViewModelAsync()` and `createDefaultInstanceAsync()` instead of the deprecated sync counterparts. Useful for verifying the async path produces identical visual results. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Previously both the sync and async variants ignored `index` and always called createDefaultInstance(). Fix implements createInstanceByIndexImpl which calls getInstanceNames(of:) on the File and picks the name at the requested position, then creates via .name(name, from:). Also adds a post_install Podfile hook that strips the RiveRuntime.Swift submodule from RiveRuntime's modulemaps. Without it the project cannot be built with Xcode 26 / Swift 6.2 because Clang sees conflicting definitions of swift::Optional / swift::String between the pre-built RiveRuntime XCFramework (Swift 6.1 ABI) and the freshly-compiled NitroModules (Swift 6.2 ABI). Closes issue 1 (iOS) from .local/docs/issues.md. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

…Android Remove the async variant — callers should use createInstanceByNameAsync instead. The deprecated sync createInstanceByIndex now correctly uses getViewModelInstanceNames + index lookup on Android (was returning default instance). Tests updated to use the sync API.

Were hardcoded to 0. Now use file.getProperties(of:) and file.getInstanceNames(of:) respectively. Tests strengthened to assert > 0 instead of >= 0.

Was accepting both replaced and original values. Now strictly asserts the replacement took effect. Passes on iOS; Android experimental SDK lacks the native bridge method (known limitation).

viewModelImpl was passing the property path as instanceName, which is misleading. The SDK doesn't expose the actual instance name, so leave it as "" (consistent with Android and Issue 11 limitation).

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 from 44681b5 to 91c487b Compare March 11, 2026 10:46

mfazekas and others added 14 commits March 16, 2026 10:37

chore: move docs to .local/docs, remove from git

e897fbc

Generated with [Claude Code](https://claude.ai/code) via [Happy](https://happy.engineering)

refactor: remove ExperimentalFileSource from legacy backend

b67258a

getEnums() in legacy now throws directing users to the experimental backend instead of creating throwaway Worker+File instances. Generated with [Claude Code](https://claude.ai/code) via [Happy](https://happy.engineering)

fix: restore whitespace in legacy files, remove fragile asset type gu…

7049920

…essing Generated with [Claude Code](https://claude.ai/code) via [Happy](https://happy.engineering)

fix: wire up trigger property to experimental fire/stream API

1af91b8

fix: set fit after view setup to fix .layout mode on initial render

0b8751e

Passing fit to the Rive() constructor breaks layout mode because the MTKView drawable isn't ready yet. Set rive.fit after setupRiveUIView() instead.

test: add advanced data binding harness tests

976d9b8

Port Flutter data binding tests for VM access, enums, creation variants, list properties, artboard and image properties. Includes new .riv test assets and react-native-harness upgrade to alpha.25.

fix: guard negative index in legacy viewModelByIndex and createInstan…

b7bfe4f

…ceByIndex Prevents fatal crash when passing negative numbers to Swift APIs expecting unsigned integers.

fix: correct trigger property listener task type signature

ed98433

fix: embed SPM RiveRuntime.framework via Podfile post_install hook

75c660a

CocoaPods doesn't embed SPM-resolved dynamic frameworks automatically. Patches the embed script to include RiveRuntime when USE_RIVE_SPM=1.

refactor: split Android into legacy/experimental backend directories

165b68c

Move 19 backend-specific files to src/legacy/java/, add Gradle sourceSets switching via USE_RIVE_NEW_API property, prepare empty experimental dirs.

feat: implement Android experimental backend with CommandQueue API

c222601

Custom TextureView renderer with Choreographer render loop, CommandQueue polling infrastructure, ViewModel resolution and property validation, and example Compose test activities for manual testing.

mfazekas and others added 5 commits March 16, 2026 10:37

fix: skip autoPlay test on both experimental backends

b77cfa1

fix: remove unused Platform import

aa2fdf0

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

mfazekas and others added 24 commits March 16, 2026 10:53

chore: update nitrogen generated files

be5135e

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

fix: use ArrayBuffer instead of ArrayBufferHolder in new HybridRiveFi…

c787e09

…leFactory ArrayBufferHolder is a C++ type; the Swift protocol uses ArrayBuffer. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

ci: fail lint-kotlin job on ktlint errors

3f0a5da

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

fix: format import in ReferencedAssets.ts to satisfy Prettier

fa2c0f3

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

chore: re-run nitrogen to regenerate bridge code for RiveAssetType/ty…

b310191

…pe field Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

fix: disable function-naming ktlint rule for Composable PascalCase fu…

18f91c0

…nctions Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

fix: apply ktlint auto-format to ComposeTestActivity

934e68f

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

fix(ios): implement propertyCount and instanceCount using runtime APIs

5ecdf1e

Were hardcoded to 0. Now use file.getProperties(of:) and file.getInstanceNames(of:) respectively. Tests strengthened to assert > 0 instead of >= 0.

test: strengthen replaceViewModel assertion to verify actual replacement

a33ad6f

Was accepting both replaced and original values. Now strictly asserts the replacement took effect. Passes on iOS; Android experimental SDK lacks the native bridge method (known limitation).

fix(ios): don't set instanceName to property path for nested VMIs

8326192

viewModelImpl was passing the property path as instanceName, which is misleading. The SDK doesn't expose the actual instance name, so leave it as "" (consistent with Android and Issue 11 limitation).

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

WIP feat: experimental iOS / Android POC#134

WIP feat: experimental iOS / Android POC#134
mfazekas wants to merge 57 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!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

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

Uh oh!

Summary

Usage

TODO

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!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

mfazekas commented Jan 23, 2026 •

edited

Loading