From 3827dbbf74109a57283bf102a22ffdcd3244d6cb Mon Sep 17 00:00:00 2001 From: Abhishek Kumar Date: Tue, 26 May 2026 13:29:35 +0000 Subject: [PATCH] fix: don't auto-save on edge delete Persisting the workflow is a separate intent from "delete this edge", the same way it's separate from "delete this node" (useNodeHandlers doesn't auto-save either). The Save button in the edit dialog conveys the save semantics; trash buttons shouldn't piggy-back on them. After this, all delete paths (node toolbar trash, edge toolbar trash, Backspace on node, Backspace on edge) mark the workflow dirty and leave persistence to Cmd+S or the header Save button. Co-Authored-By: Claude Opus 4.7 (1M context) --- ui/src/components/flow/edges/CustomEdge.tsx | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/ui/src/components/flow/edges/CustomEdge.tsx b/ui/src/components/flow/edges/CustomEdge.tsx index fa158ec..f50f3df 100644 --- a/ui/src/components/flow/edges/CustomEdge.tsx +++ b/ui/src/components/flow/edges/CustomEdge.tsx @@ -260,10 +260,9 @@ export default function CustomEdge(props: CustomEdgeProps) { await saveWorkflow(); }, [id, updateEdge, saveWorkflow]); - const handleDeleteEdge = useCallback(async () => { + const handleDeleteEdge = useCallback(() => { deleteEdge(id); - await saveWorkflow(); - }, [id, deleteEdge, saveWorkflow]); + }, [id, deleteEdge]); return ( <>