fix(workflow): replace hardcoded trigger node logic with metadata-driven approach

- Add isStart: true to all trigger nodes (TriggerWebhook, TriggerSchedule, TriggerPlugin)
- Replace hardcoded BlockEnum checks in use-checklist.ts with metadata-driven logic
- Update trigger node tests to validate metadata instead of obsolete methods
- Add webhook URL validation to TriggerWebhook node
- Ensure backward compatibility with existing workflow configurations

This change migrates from hardcoded trigger node identification to a
centralized metadata-driven approach, improving maintainability and
consistency across the workflow system.

web/app/components/workflow/hooks/use-checklist.ts

@@ -27,10 +27,10 @@ import {
 } from '../constants'
 import {
   useGetToolIcon,
+  useNodesMetaData,
 } from '../hooks'
 import type { ToolNodeType } from '../nodes/tool/types'
 import type { DataSourceNodeType } from '../nodes/data-source/types'
-import { useNodesMetaData } from './use-nodes-meta-data'
 import { useToastContext } from '@/app/components/base/toast'
 import { useGetLanguage } from '@/context/i18n'
 import type { AgentNodeType } from '../nodes/agent/types'
@@ -132,10 +132,7 @@ export const useChecklist = (nodes: Node[], edges: Edge[]) => {
 
         // Start nodes and Trigger nodes should not show unConnected error if they have validation errors
         // or if they are valid start nodes (even without incoming connections)
-        const isStartNode = node.data.type === BlockEnum.Start
-          || node.data.type === BlockEnum.TriggerSchedule
-          || node.data.type === BlockEnum.TriggerWebhook
-          || node.data.type === BlockEnum.TriggerPlugin
+        const isStartNode = nodesExtraData?.[node.data.type as BlockEnum]?.metaData.isStart || false
 
         const isUnconnected = !validNodes.find(n => n.id === node.id)
         const shouldShowError = errorMessage || (isUnconnected && !isStartNode)

web/app/components/workflow/nodes/trigger-plugin/default.ts

@@ -6,6 +6,7 @@ import type { PluginTriggerNodeType } from './types'
 const metaData = genNodeMetaData({
   sort: 1,
   type: BlockEnum.TriggerPlugin,
+  isStart: true,
 })
 
 const nodeDefault: NodeDefault<PluginTriggerNodeType> = {

web/app/components/workflow/nodes/trigger-schedule/__tests__/default.test.ts

@@ -34,15 +34,11 @@ describe('Schedule Trigger Node Default', () => {
       expect(nodeDefault.defaultValue.frequency).toBe('daily')
     })
 
-    it('should have empty prev nodes', () => {
-      const prevNodes = nodeDefault.getAvailablePrevNodes(false)
-      expect(prevNodes).toEqual([])
-    })
-
-    it('should have available next nodes excluding Start', () => {
-      const nextNodes = nodeDefault.getAvailableNextNodes(false)
-      expect(nextNodes).toBeDefined()
-      expect(nextNodes.length).toBeGreaterThan(0)
+    it('should have correct metadata for trigger node', () => {
+      expect(nodeDefault.metaData).toBeDefined()
+      expect(nodeDefault.metaData.type).toBe('trigger-schedule')
+      expect(nodeDefault.metaData.sort).toBe(2)
+      expect(nodeDefault.metaData.isStart).toBe(true)
     })
   })
 

web/app/components/workflow/nodes/trigger-schedule/default.ts

@@ -107,6 +107,7 @@ const validateVisualConfig = (payload: ScheduleTriggerNodeType, t: any): string
 const metaData = genNodeMetaData({
   sort: 2,
   type: BlockEnum.TriggerSchedule,
+  isStart: true,
 })
 
 const nodeDefault: NodeDefault<ScheduleTriggerNodeType> = {

web/app/components/workflow/nodes/trigger-webhook/__tests__/default.test.ts

@@ -45,15 +45,11 @@ describe('Webhook Trigger Node Default', () => {
       expect(defaultValue.body).toHaveLength(0)
     })
 
-    it('should have empty prev nodes', () => {
-      const prevNodes = nodeDefault.getAvailablePrevNodes(false)
-      expect(prevNodes).toEqual([])
-    })
-
-    it('should have available next nodes excluding Start', () => {
-      const nextNodes = nodeDefault.getAvailableNextNodes(false)
-      expect(nextNodes).toBeDefined()
-      expect(nextNodes.length).toBeGreaterThan(0)
+    it('should have correct metadata for trigger node', () => {
+      expect(nodeDefault.metaData).toBeDefined()
+      expect(nodeDefault.metaData.type).toBe('trigger-webhook')
+      expect(nodeDefault.metaData.sort).toBe(3)
+      expect(nodeDefault.metaData.isStart).toBe(true)
     })
   })
 

web/app/components/workflow/nodes/trigger-webhook/default.ts

@@ -7,6 +7,7 @@ import { isValidParameterType } from './utils/parameter-type-utils'
 const metaData = genNodeMetaData({
   sort: 3,
   type: BlockEnum.TriggerWebhook,
+  isStart: true,
 })
 
 const nodeDefault: NodeDefault<WebhookTriggerNodeType> = {
@@ -23,6 +24,14 @@ const nodeDefault: NodeDefault<WebhookTriggerNodeType> = {
     response_body: '',
   },
   checkValid(payload: WebhookTriggerNodeType, t: any) {
+    // Require webhook_url to be configured
+    if (!payload.webhook_url || payload.webhook_url.trim() === '') {
+      return {
+        isValid: false,
+        errorMessage: t('workflow.nodes.triggerWebhook.validation.webhookUrlRequired'),
+      }
+    }
+
     // Validate parameter types for params and body
     const parametersWithTypes = [
       ...(payload.params || []),