runtime list updates (#9855) f660dea549

* runtime list updates

Co-authored-by: hernan <hernan@rive.app>

Author: bodymovin

.rive_head

@@ -1 +1 @@
-b8bfaaf5e7d1b859c7be04d7139aeaea3eb474a1
+f660dea5494097df60419b00af6baf1c1e1b896d

js/src/rive.ts

@@ -3281,9 +3281,14 @@ export class ViewModelInstance {
   }
 
   public addParent(parent: ViewModelInstance) {
-    this._parents.push(parent);
-    if (this._propertiesWithCallbacks.length > 0 || this._children.length > 0) {
-      parent.addToViewModelCallbacks(this);
+    if (!this._parents.includes(parent)) {
+      this._parents.push(parent);
+      if (
+        this._propertiesWithCallbacks.length > 0 ||
+        this._children.length > 0
+      ) {
+        parent.addToViewModelCallbacks(this);
+      }
     }
   }
 
@@ -3905,6 +3910,7 @@ export class ViewModelInstanceList extends ViewModelInstanceValue {
       (this._viewModelInstanceValue as rc.ViewModelInstanceList).addInstance(
         instance.runtimeInstance!,
       );
+      instance.addParent(this._parentViewModel);
     }
   }
 
@@ -3913,6 +3919,7 @@ export class ViewModelInstanceList extends ViewModelInstanceValue {
       (this._viewModelInstanceValue as rc.ViewModelInstanceList).removeInstance(
         instance.runtimeInstance!,
       );
+      instance.removeParent(this._parentViewModel);
     }
   }
 
@@ -3927,12 +3934,19 @@ export class ViewModelInstanceList extends ViewModelInstanceValue {
       this._viewModelInstanceValue as rc.ViewModelInstanceList
     ).instanceAt(index);
     if (runtimeInstance != null) {
-      const viewModelInstance = new ViewModelInstance(runtimeInstance, null);
+      const viewModelInstance = new ViewModelInstance(
+        runtimeInstance,
+        this._parentViewModel,
+      );
       return viewModelInstance;
     }
     return null;
   }
 
+  public swap(a: number, b: number) {
+    (this._viewModelInstanceValue as rc.ViewModelInstanceList).swap(a, b);
+  }
+
   public internalHandleCallback(callback: Function) {
     callback();
   }

js/src/rive_advanced.mjs.d.ts

@@ -928,6 +928,7 @@ export declare class ViewModelInstanceList extends ViewModelInstanceValue {
   removeInstance(vmi: ViewModelInstance): void;
   removeInstanceAt(index: number): void;
   instanceAt(index: number): ViewModelInstance;
+  swap(a: number, b: number): ViewModelInstance;
 }
 export declare class ViewModelInstanceAssetImage extends ViewModelInstanceValue {
   value(image: ImageInternal):void;

wasm/src/bindings.cpp

@@ -1206,7 +1206,13 @@ EMSCRIPTEN_BINDINGS(RiveWASM)
                   optional_override([](rive::ViewModelInstanceListRuntime& self, int index) {
                       self.removeInstanceAt(index);
                   }),
-                  allow_raw_pointers());
+                  allow_raw_pointers())
+        .function(
+            "swap",
+            optional_override([](rive::ViewModelInstanceListRuntime& self, uint32_t a, uint32_t b) {
+                self.swap(a, b);
+            }),
+            allow_raw_pointers());
     class_<rive::ViewModelInstanceAssetImageRuntime, base<rive::ViewModelInstanceValueRuntime>>(
         "ViewModelInstanceAssetImage")
         .function(