fix: the new node name is generated based on the original node when duplicating (#4675)

web/app/components/workflow/hooks/use-nodes-interactions.ts

@@ -33,6 +33,7 @@ import {
   Y_OFFSET,
 } from '../constants'
 import {
+  genNewNodeTitleFromOld,
   generateNewNode,
   getNodesConnectedSourceOrTargetHandleIdsMap,
   getTopLeftNodePosition,
@@ -1067,7 +1068,6 @@ export const useNodesInteractions = () => {
       const offsetY = currentPosition.y - y
       clipboardElements.forEach((nodeToPaste, index) => {
         const nodeType = nodeToPaste.data.type
-        const nodesWithSameType = nodes.filter(node => node.data.type === nodeType)
 
         const newNode = generateNewNode({
           data: {
@@ -1077,7 +1077,7 @@ export const useNodesInteractions = () => {
             _isBundled: false,
             _connectedSourceHandleIds: [],
             _connectedTargetHandleIds: [],
-            title: nodesWithSameType.length > 0 ? `${t(`workflow.blocks.${nodeType}`)} ${nodesWithSameType.length + 1}` : t(`workflow.blocks.${nodeType}`),
+            title: genNewNodeTitleFromOld(nodeToPaste.data.title),
           },
           position: {
             x: nodeToPaste.position.x + offsetX,

web/app/components/workflow/utils.ts

@@ -289,6 +289,20 @@ export const generateNewNode = ({ data, position, id, zIndex, ...rest }: Omit<No
   } as Node
 }
 
+export const genNewNodeTitleFromOld = (oldTitle: string) => {
+  const regex = /^(.+?)\s*\((\d+)\)\s*$/
+  const match = oldTitle.match(regex)
+
+  if (match) {
+    const title = match[1]
+    const num = parseInt(match[2], 10)
+    return `${title} (${num + 1})`
+  }
+  else {
+    return `${oldTitle} (1)`
+  }
+}
+
 export const getValidTreeNodes = (nodes: Node[], edges: Edge[]) => {
   const startNode = nodes.find(node => node.data.type === BlockEnum.Start)