chore: change the parallel warning show logic

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

@@ -644,10 +644,10 @@ export const useNodesInteractions = () => {
         newNode.data.isInIteration = true
         newNode.data.iteration_id = prevNode.parentId
         newNode.zIndex = ITERATION_CHILDREN_Z_INDEX
-        if (newNode.data.type === BlockEnum.Answer) {
+        if (newNode.data.type === BlockEnum.Answer || newNode.data.type === BlockEnum.Tool || newNode.data.type === BlockEnum.Assigner) {
           const parentIterNodeIndex = nodes.findIndex(node => node.id === prevNode.parentId)
-          if (nodes[parentIterNodeIndex].data._isFirstTime)
-            nodes[parentIterNodeIndex].data._isShowTips = true
+          const iterNodeData: IterationNodeType = nodes[parentIterNodeIndex].data
+          iterNodeData._isShowTips = true
         }
       }
 

web/app/components/workflow/nodes/iteration/default.ts

@@ -1,7 +1,10 @@
 import { BlockEnum, ErrorHandleMode } from '../../types'
 import type { NodeDefault } from '../../types'
 import type { IterationNodeType } from './types'
-import { ALL_CHAT_AVAILABLE_BLOCKS, ALL_COMPLETION_AVAILABLE_BLOCKS } from '@/app/components/workflow/constants'
+import {
+  ALL_CHAT_AVAILABLE_BLOCKS,
+  ALL_COMPLETION_AVAILABLE_BLOCKS,
+} from '@/app/components/workflow/constants'
 const i18nPrefix = 'workflow'
 
 const nodeDefault: NodeDefault<IterationNodeType> = {
@@ -10,7 +13,6 @@ const nodeDefault: NodeDefault<IterationNodeType> = {
     iterator_selector: [],
     output_selector: [],
     _children: [],
-    _isFirstTime: true,
     _isShowTips: false,
     is_parallel: false,
     parallel_nums: 10,
@@ -19,21 +21,37 @@ const nodeDefault: NodeDefault<IterationNodeType> = {
   getAvailablePrevNodes(isChatMode: boolean) {
     const nodes = isChatMode
       ? ALL_CHAT_AVAILABLE_BLOCKS
-      : ALL_COMPLETION_AVAILABLE_BLOCKS.filter(type => type !== BlockEnum.End)
+      : ALL_COMPLETION_AVAILABLE_BLOCKS.filter(
+        type => type !== BlockEnum.End,
+      )
     return nodes
   },
   getAvailableNextNodes(isChatMode: boolean) {
-    const nodes = isChatMode ? ALL_CHAT_AVAILABLE_BLOCKS : ALL_COMPLETION_AVAILABLE_BLOCKS
+    const nodes = isChatMode
+      ? ALL_CHAT_AVAILABLE_BLOCKS
+      : ALL_COMPLETION_AVAILABLE_BLOCKS
     return nodes
   },
   checkValid(payload: IterationNodeType, t: any) {
     let errorMessages = ''
 
-    if (!errorMessages && (!payload.iterator_selector || payload.iterator_selector.length === 0))
-      errorMessages = t(`${i18nPrefix}.errorMsg.fieldRequired`, { field: t(`${i18nPrefix}.nodes.iteration.input`) })
+    if (
+      !errorMessages
+      && (!payload.iterator_selector || payload.iterator_selector.length === 0)
+    ) {
+      errorMessages = t(`${i18nPrefix}.errorMsg.fieldRequired`, {
+        field: t(`${i18nPrefix}.nodes.iteration.input`),
+      })
+    }
 
-    if (!errorMessages && (!payload.output_selector || payload.output_selector.length === 0))
-      errorMessages = t(`${i18nPrefix}.errorMsg.fieldRequired`, { field: t(`${i18nPrefix}.nodes.iteration.output`) })
+    if (
+      !errorMessages
+      && (!payload.output_selector || payload.output_selector.length === 0)
+    ) {
+      errorMessages = t(`${i18nPrefix}.errorMsg.fieldRequired`, {
+        field: t(`${i18nPrefix}.nodes.iteration.output`),
+      })
+    }
 
     return {
       isValid: !errorMessages,

web/app/components/workflow/nodes/iteration/node.tsx

@@ -31,13 +31,13 @@ const Node: FC<NodeProps<IterationNodeType>> = ({
   useEffect(() => {
     if (nodesInitialized)
       handleNodeIterationRerender(id)
-    if (data.is_parallel && data._isShowTips && data._isFirstTime) {
+    if (data.is_parallel && data._isShowTips) {
       Toast.notify({
         type: 'warning',
         message: t(`${i18nPrefix}.answerNodeWarningDesc`),
         duration: 5000,
       })
-      data._isFirstTime = false
+      data._isShowTips = false
     }
   }, [nodesInitialized, id, handleNodeIterationRerender, data, t])
 

web/app/components/workflow/nodes/iteration/types.ts

@@ -16,6 +16,5 @@ export type IterationNodeType = CommonNodeType & {
   is_parallel: boolean // open the parallel mode or not
   parallel_nums: number // the numbers of parallel
   error_handle_mode: ErrorHandleMode // how to handle error in the iteration
-  _isShowTips?: boolean // when answer node in parallel mode iteration show tips
-  _isFirstTime?: boolean // is the first time to add parallel iteration node
+  _isShowTips: boolean // when answer node in parallel mode iteration show tips
 }

web/app/components/workflow/run/node.tsx

@@ -76,7 +76,11 @@ const NodePanel: FC<Props> = ({
     if (!details || details.length === 0)
       return 0
 
-    return details.flat().filter(item => item.status === 'failed').length
+    return details.reduce((acc, iteration) => {
+      if (iteration.some(item => item.status === 'failed'))
+        acc++
+      return acc
+    }, 0)
   }
   useEffect(() => {
     setCollapseState(!nodeInfo.expand)