Remove unnecessary empty init() methods and autolinking entries for ViewModel classes

Author: mfazekas

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

@@ -75,7 +75,7 @@ class HybridRiveFile : HybridRiveFileSpec() {
 
     for ((key, assetData) in assetsData) {
       val asset = cache[key] ?: continue
-      loadJobs.add(loader.updateAsset(assetData, asset, context))
+      loadJobs.add(loader.updateAsset(assetData, asset))
     }
 
     if (loadJobs.isNotEmpty()) {

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

@@ -15,7 +15,7 @@ class HybridViewModelImageProperty(private val viewModelImage: ViewModelImagePro
   }
 
   override fun addListener(onChanged: () -> Unit) {
-    listeners.add(onChanged)
+    listeners.add { onChanged() }
     ensureValueListenerJob(viewModelImage.valueFlow.map { })
   }
 }

ios/HybridViewModel.swift

@@ -6,12 +6,7 @@ class HybridViewModel: HybridViewModelSpec {
   init(viewModel: RiveDataBindingViewModel) {
     self.viewModel = viewModel
   }
-  
-  override init() {
-    self.viewModel = nil
-    super.init()
-  }
-  
+
   var propertyCount: Double { Double(viewModel?.propertyCount ?? 0) }
 
   var instanceCount: Double { Double(viewModel?.instanceCount ?? 0) }

ios/HybridViewModelBooleanProperty.swift

@@ -9,15 +9,7 @@ class HybridViewModelBooleanProperty: HybridViewModelBooleanPropertySpec, Valued
     self.property = property
     super.init()
   }
-  
-  /// ⚠️ DO NOT REMOVE
-  /// Nitro requires a parameterless initializer for JS bridging.
-  /// This is invoked automatically during hybrid module construction.
-  /// Internally we always use `init(property:)`
-  override init() {
-    super.init()
-  }
-  
+
   var value: Bool {
     get {
       return property.value

ios/HybridViewModelColorProperty.swift

@@ -9,15 +9,7 @@ class HybridViewModelColorProperty: HybridViewModelColorPropertySpec, ValuedProp
     self.property = property
     super.init()
   }
-  
-  /// ⚠️ DO NOT REMOVE
-  /// Nitro requires a parameterless initializer for JS bridging.
-  /// This is invoked automatically during hybrid module construction.
-  /// Internally we always use `init(property:)`
-  override init() {
-    super.init()
-  }
-  
+
   var value: Double {
     get {
       return property.value.toHexDouble()

ios/HybridViewModelEnumProperty.swift

@@ -9,15 +9,7 @@ class HybridViewModelEnumProperty: HybridViewModelEnumPropertySpec, ValuedProper
     self.property = property
     super.init()
   }
-  
-  /// ⚠️ DO NOT REMOVE
-  /// Nitro requires a parameterless initializer for JS bridging.
-  /// This is invoked automatically during hybrid module construction.
-  /// Internally we always use `init(property:)`
-  override init() {
-    super.init()
-  }
-  
+
   var value: String {
     get {
       return property.value

ios/HybridViewModelInstance.swift

@@ -6,12 +6,7 @@ class HybridViewModelInstance: HybridViewModelInstanceSpec {
   init(viewModelInstance: RiveDataBindingViewModel.Instance) {
     self.viewModelInstance = viewModelInstance
   }
-  
-  override init() {
-    self.viewModelInstance = nil
-    super.init()
-  }
-  
+
   var instanceName: String { viewModelInstance?.name ?? "" }
 
   func numberProperty(path: String) throws -> (any HybridViewModelNumberPropertySpec)? {

ios/HybridViewModelNumberProperty.swift

@@ -9,14 +9,6 @@ class HybridViewModelNumberProperty: HybridViewModelNumberPropertySpec, ValuedPr
     super.init()
   }
 
-  /// ⚠️ DO NOT REMOVE
-  /// Nitro requires a parameterless initializer for JS bridging.
-  /// This is invoked automatically during hybrid module construction.
-  /// Internally we always use `init(property:)`
-  override init() {
-    super.init()
-  }
-
   var value: Double {
     get {
       return Double(property.value)

ios/HybridViewModelStringProperty.swift

@@ -9,15 +9,7 @@ class HybridViewModelStringProperty: HybridViewModelStringPropertySpec, ValuedPr
     self.property = property
     super.init()
   }
-  
-  /// ⚠️ DO NOT REMOVE
-  /// Nitro requires a parameterless initializer for JS bridging.
-  /// This is invoked automatically during hybrid module construction.
-  /// Internally we always use `init(property:)`
-  override init() {
-    super.init()
-  }
-  
+
   var value: String {
     get {
       return property.value

ios/HybridViewModelTriggerProperty.swift

@@ -15,15 +15,7 @@ class HybridViewModelTriggerProperty: HybridViewModelTriggerPropertySpec, Valued
     self.property = property
     super.init()
   }
-  
-  /// ⚠️ DO NOT REMOVE
-  /// Nitro requires a parameterless initializer for JS bridging.
-  /// This is invoked automatically during hybrid module construction.
-  /// Internally we always use `init(property:)`
-  override init() {
-    super.init()
-  }
-  
+
   func trigger() {
     property.trigger()
   }