From fb612eb32eb1a98c4d1cece52d32e01a1d82a15d Mon Sep 17 00:00:00 2001 From: mlogclub Date: Tue, 30 Jun 2026 11:34:41 +0800 Subject: [PATCH] refactor: remove onDelete prop from ConditionBranchConfigPanel and associated delete logic in WorkflowConfigPanel --- .../ai-workflows/_components/node-config-panel.tsx | 14 -------------- .../_components/workflow-config-sidebar.tsx | 14 -------------- .../ai-workflows/_components/workflow-editor.tsx | 1 - 3 files changed, 29 deletions(-) diff --git a/web/app/dashboard/ai-workflows/_components/node-config-panel.tsx b/web/app/dashboard/ai-workflows/_components/node-config-panel.tsx index c1410e3..f4a1819 100644 --- a/web/app/dashboard/ai-workflows/_components/node-config-panel.tsx +++ b/web/app/dashboard/ai-workflows/_components/node-config-panel.tsx @@ -207,14 +207,12 @@ export function ConditionBranchConfigPanel({ branchId, variables, onChange, - onDelete, }: { node: AIWorkflowDefinition["nodes"][number] nodes: AIWorkflowDefinition["nodes"] branchId: string variables: WorkflowVariableRef[] onChange: (nodeId: string, data: AIWorkflowDefinition["nodes"][number]["data"]) => void - onDelete: (branchId: string) => void }) { const config = normalizeNodeConfig(node.data?.config) const branches = config.branches ?? [] @@ -279,18 +277,6 @@ export function ConditionBranchConfigPanel({ )} - - {branch.default ? null : ( - - )} ) } diff --git a/web/app/dashboard/ai-workflows/_components/workflow-config-sidebar.tsx b/web/app/dashboard/ai-workflows/_components/workflow-config-sidebar.tsx index 083761f..a9240cd 100644 --- a/web/app/dashboard/ai-workflows/_components/workflow-config-sidebar.tsx +++ b/web/app/dashboard/ai-workflows/_components/workflow-config-sidebar.tsx @@ -27,7 +27,6 @@ export function WorkflowConfigPanel({ selectedNodeId, selectedBranch, onClose, - onSelectBranch, onChangeNodeData, onDeleteNode, }: { @@ -36,7 +35,6 @@ export function WorkflowConfigPanel({ selectedNodeId: string selectedBranch: SelectedWorkflowBranch | null onClose: () => void - onSelectBranch: (branch: SelectedWorkflowBranch | null) => void onChangeNodeData: (nodeId: string, data: WorkflowNodeData) => void onDeleteNode: (nodeId: string) => void }) { @@ -63,17 +61,6 @@ export function WorkflowConfigPanel({ return null } - const deleteSelectedBranch = (branchId: string) => { - const config = normalizeNodeConfig(selectedNode.data?.config) - onChangeNodeData(selectedNode.id, { - ...(selectedNode.data ?? {}), - config: { - ...config, - branches: (config.branches ?? []).filter((branch) => branch.id !== branchId), - }, - }) - onSelectBranch(null) - } const updatePanelTitle = (title: string) => { if (selectedBranchItem && selectedBranch) { const config = normalizeNodeConfig(selectedNode.data?.config) @@ -170,7 +157,6 @@ export function WorkflowConfigPanel({ branchId={selectedBranch.branchId} variables={availableVariables} onChange={onChangeNodeData} - onDelete={deleteSelectedBranch} /> ) : (