fix: LEAP-2060: Dynamic choices are not displayed when Interactive View All FF is turned ON (#7468)

Co-authored-by: nass600 <nass600@users.noreply.github.com>
Co-authored-by: hlomzik <hlomzik@users.noreply.github.com>

Author: nass600

web/libs/editor/src/mixins/DynamicChildrenMixin.js

@@ -1,6 +1,9 @@
 import { getRoot, protect, types, unprotect } from "mobx-state-tree";
 import ProcessAttrsMixin from "./ProcessAttrs";
 import { parseValue } from "../utils/data";
+import { guidGenerator } from "../utils/unique";
+import { FF_DEV_3391 } from "../utils/feature-flags";
+import { ff } from "@humansignal/core";
 
 const DynamicChildrenMixin = types
   .model({})
@@ -12,11 +15,19 @@ const DynamicChildrenMixin = types
   }))
   .actions((self) => {
     const prepareDynamicChildrenData = (data, store, parent) => {
+      // Right now with the FF on, we are traverseing the Tree in the store::initRoot and the
+      // AnnotationStore::afterCreate which will generate duplicated children so we only need to
+      // run this once, when the annotation store is not yet initialized
+      if (ff.isActive(FF_DEV_3391) && store.annotationStore.initialized) return;
+
       if (data && data.length) {
         for (const obj of data) {
+          // No matter if using Interactive View mode or not, we add the ids for consistency
+          const id = obj.id ?? guidGenerator();
           parent.children.push({
             type: self.defaultChildType,
             ...obj,
+            id,
             children: [],
           });
 
@@ -51,9 +62,13 @@ const DynamicChildrenMixin = types
         // we may need to rewrite this, initRoot and the other related methods
         // (actually a lot of them) to work asynchronously as well
 
-        setTimeout(() => {
+        if (ff.isActive(FF_DEV_3391)) {
           self.updateDynamicChildren(store);
-        });
+        } else {
+          setTimeout(() => {
+            self.updateDynamicChildren(store);
+          });
+        }
       },
 
       updateDynamicChildren(store) {