fix: improve error handling and fix import cycle

- Fix import cycle by extracting NitroRiveView to separate file
- Make error types human-readable in logs (show enum names instead of numbers)
- Improve iOS error detection using RiveErrorCode enum values instead of string comparisons
- Rename NativeRiveError to NitroRiveError for consistency with Nitro Modules naming
- Make detectErrorType return tuple with both error type and message
- Add NitroRiveError.fileNotFound case for asset loading errors
- Remove non-existent error types (TextRunNotFoundError, IncorrectAnimationName)
- Map instanceNotFound errors to DataBindingError type

🤖 Generated with [Claude Code](https://claude.com/claude-code)

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

Author: mfazekas

ios/HybridRiveView.swift

@@ -194,8 +194,9 @@ extension HybridRiveView {
     do {
       return try fn()
     } catch (let e) {
-      let errorType = detectErrorType(e)
-      let errorMessage = "[RIVE] \(tag) \(note ?? "") \(e)"
+      let (errorType, errorDescription) = detectErrorType(e)
+      let noteString = note.map { " \($0)" } ?? ""
+      let errorMessage = "[RIVE] \(tag)\(noteString) \(errorDescription)"
 
       let riveError = RiveError(
         message: errorMessage,
@@ -205,35 +206,33 @@ extension HybridRiveView {
     }
   }
 
-  private func detectErrorType(_ error: Error) -> RiveErrorType {
-    if case NativeRiveError.instanceNotFound = error {
-      return .databindingerror
+  private func detectErrorType(_ error: Error) -> (RiveErrorType, String) {
+    switch error {
+    case NitroRiveError.instanceNotFound(let message):
+      return (.databindingerror, message)
+    case NitroRiveError.fileNotFound(let message):
+      return (.filenotfound, message)
+    default:
+      break
     }
 
     let nsError = error as NSError
-    guard let errorName = nsError.userInfo["name"] as? String else {
-      return .unknown
-    }
-
-    switch errorName {
-    case "NoArtboardFound":
-      return .incorrectartboardname
-    case "NoStateMachineFound":
-      return .incorrectstatemachinename
-    case "NoAnimationFound":
-      return .incorrectanimationname
-    case "Malformed":
-      return .malformedfile
-    case "FileNotFound":
-      return .filenotfound
-    case "NoStateMachineInputFound":
-      return .incorrectanimationname
-    case "TextRunNotFoundError":
-      return .textrunnotfounderror
-    case "DataBindingError":
-      return .databindingerror
+    let message = nsError.localizedDescription
+
+    // RiveErrorCode from RiveRuntime
+    switch nsError.code {
+    case RiveErrorCode.noArtboardFound.rawValue:
+      return (.incorrectartboardname, message)
+    case RiveErrorCode.noStateMachineFound.rawValue:
+      return (.incorrectstatemachinename, message)
+    case RiveErrorCode.noAnimationFound.rawValue:
+      return (.incorrectanimationname, message)
+    case RiveErrorCode.malformedFile.rawValue:
+      return (.malformedfile, message)
+    case RiveErrorCode.noStateMachineInputFound.rawValue:
+      return (.incorrectanimationname, message)
     default:
-      return .unknown
+      return (.unknown, message)
     }
   }
 }

ios/ReferencedAssetLoader.swift

@@ -23,18 +23,14 @@ func createIncorrectRiveURL(_ url: String) -> NSError {
     ])
 }
 
-func createAssetFileError(_ assetName: String) -> NSError {
-  return NSError(
-    domain: RiveErrorDomain, code: 801,
-    userInfo: [
-      NSLocalizedDescriptionKey: "Could not load Rive asset: \(assetName)", "name": "FileNotFound",
-    ])
+func createAssetFileError(_ assetName: String) -> NitroRiveError {
+  return NitroRiveError.fileNotFound(message: "Could not load Rive asset: \(assetName)")
 }
 
 final class ReferencedAssetLoader {
-  private func handleRiveError(error: NSError) {
+  private func handleRiveError(error: Error) {
     // TODO allow user to specify onError callback
-    RCTLogError(error.localizedDescription)
+    RCTLogError("\(error)")
   }
 
   private func handleInvalidUrlError(url: String) {

ios/RiveReactNativeView.swift

@@ -26,8 +26,9 @@ struct ViewConfiguration {
   let bindData: BindData
 }
 
-enum NativeRiveError: Error {
+enum NitroRiveError: Error {
   case instanceNotFound(message: String)
+  case fileNotFound(message: String)
 }
 
 class RiveReactNativeView: UIView, RiveStateMachineDelegate {
@@ -110,7 +111,7 @@ class RiveReactNativeView: UIView, RiveStateMachineDelegate {
         let viewModel = riveFile.defaultViewModel(for: artboard),
         let instance = viewModel.createInstance(fromName: name)
       else {
-        throw NativeRiveError.instanceNotFound(message: "\(name) instance not found")
+        throw NitroRiveError.instanceNotFound(message: "\(name) instance not found")
       }
       stateMachine?.bind(viewModelInstance: instance)
       // 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

src/core/Errors.ts

@@ -4,9 +4,7 @@ export enum RiveErrorType {
   MalformedFile = 2,
   IncorrectArtboardName = 3,
   IncorrectStateMachineName = 4,
-  IncorrectAnimationName = 5,
   DataBindingError = 6,
-  TextRunNotFoundError = 7,
 }
 
 export interface RiveError {

src/core/NitroRiveViewComponent.ts

@@ -0,0 +1,18 @@
+import {
+  getHostComponent,
+  type ReactNativeView,
+} from 'react-native-nitro-modules';
+import type {
+  RiveViewMethods,
+  RiveViewTSMethods,
+  RiveViewProps as NativeRiveViewProps,
+} from '../specs/RiveView.nitro';
+import RiveViewConfig from '../../nitrogen/generated/shared/json/RiveViewConfig.json';
+
+export const NitroRiveView = getHostComponent<
+  NativeRiveViewProps,
+  RiveViewMethods
+>('RiveView', () => RiveViewConfig) as ReactNativeView<
+  NativeRiveViewProps,
+  RiveViewTSMethods
+>;

src/core/RiveView.tsx

@@ -1,5 +1,5 @@
 import type { ComponentProps } from 'react';
-import { NitroRiveView } from '../index';
+import { NitroRiveView } from './NitroRiveViewComponent';
 import { RiveErrorType, type RiveError } from './Errors';
 
 export interface RiveViewProps

src/index.tsx

@@ -1,10 +1,4 @@
-import {
-  getHostComponent,
-  NitroModules,
-  type ReactNativeView,
-  type HybridView,
-} from 'react-native-nitro-modules';
-import type { Rive } from './specs/Rive.nitro';
+import { type HybridView } from 'react-native-nitro-modules';
 import {
   type RiveViewMethods,
   type RiveViewTSMethods,
@@ -20,15 +14,7 @@ export class DataBindByName implements DataBindByNameInterface {
   }
 }
 
-import RiveViewConfig from '../nitrogen/generated/shared/json/RiveViewConfig.json';
-
-export const NitroRiveView = getHostComponent<
-  NativeRiveViewProps,
-  RiveViewMethods
->('RiveView', () => RiveViewConfig) as ReactNativeView<
-  NativeRiveViewProps,
-  RiveViewTSMethods
->;
+export { NitroRiveView } from './core/NitroRiveViewComponent';
 
 export { RiveView, type RiveViewProps } from './core/RiveView';
 export type { RiveViewMethods };