3140 - bug fix: paste multiple nested task dispatchers

Author: petrapaz

client/src/pages/platform/workflow-editor/utils/getFormattedName.ts

@@ -5,7 +5,7 @@ import {isPlainObject} from '../../cluster-element-editor/utils/clusterElementsU
 import useWorkflowDataStore from '../stores/useWorkflowDataStore';
 import getAllTasksRecursively from './getAllTasksRecursively';
 
-export default function getFormattedName(itemName: string): string {
+export default function getFormattedName(itemName: string, reservedNames?: Set<string>): string {
     const {nodes, workflow} = useWorkflowDataStore.getState();
 
     const nodeNames = nodes.map((node) => (node.data as NodeDataType).name);
@@ -66,7 +66,9 @@ export default function getFormattedName(itemName: string): string {
         return names?.includes(itemName) ? [names] : [];
     });
 
-    const allExistingNodes = [...existingNodes, ...existingClusterElementNodes];
+    const reservedMatchingNames = reservedNames ? [...reservedNames].filter((name) => name?.includes(itemName)) : [];
+
+    const allExistingNodes = [...existingNodes, ...existingClusterElementNodes, ...reservedMatchingNames];
 
     if (!allExistingNodes.length) {
         return `${itemName}_1`;

client/src/pages/platform/workflow-editor/utils/pasteNode.ts

@@ -27,18 +27,26 @@ const TASK_DISPATCHER_NAMES = new Set([
 
 const isTaskDispatcher = (componentName: string): boolean => TASK_DISPATCHER_NAMES.has(componentName);
 
-function renameNestedTasks(parameters: TaskParametersType, componentName: string): TaskParametersType {
+function renameNestedTasks(
+    parameters: TaskParametersType,
+    componentName: string,
+    reservedNames: Set<string>
+): TaskParametersType {
     const renameTask = (task: WorkflowTask) => {
         const subtaskComponentName = task.type?.split('/')?.[0];
 
         if (!subtaskComponentName) {
             return;
         }
 
-        task.name = getFormattedName(subtaskComponentName);
+        const newName = getFormattedName(subtaskComponentName, reservedNames);
+
+        reservedNames.add(newName);
+
+        task.name = newName;
 
         if (isTaskDispatcher(subtaskComponentName) && task.parameters) {
-            task.parameters = renameNestedTasks(task.parameters, subtaskComponentName);
+            task.parameters = renameNestedTasks(task.parameters, subtaskComponentName, reservedNames);
         }
     };
 
@@ -197,12 +205,16 @@ export default function pasteNode({
         }
     }
 
-    const newName = getFormattedName(copiedNode.componentName);
+    const reservedNames = new Set<string>();
+
+    const newName = getFormattedName(copiedNode.componentName, reservedNames);
+
+    reservedNames.add(newName);
 
     const clonedParameters = structuredClone(copiedNode.parameters ?? {});
 
     if (isTaskDispatcher(copiedNode.componentName)) {
-        renameNestedTasks(clonedParameters, copiedNode.componentName);
+        renameNestedTasks(clonedParameters, copiedNode.componentName, reservedNames);
     }
 
     const newNodeData = {