feat(agentFlow): Fix duplicate node and add test cases for delete and duplicate nodes (#5840)

* Test cases added for delete node

* Fix incorrect node id being assigned to duplicate node

* Test cases added for duplicate node

* Fix for bug FLOWISE-257

* lint errors fix

* Fix gemini comments

* FIx comment

* Apply suggestion from @gemini-code-assist[bot]

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

* Fix lint error

* Fixed review comments

---------

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

Author: j-sanaa

packages/agentflow/TESTS.md

@@ -34,7 +34,7 @@ These modules carry the highest risk. Test in the same PR when modifying.
 | `src/infrastructure/api/client.ts` | `createApiClient` — headers, auth token, 401 interceptor | ✅ Done |
 | `src/infrastructure/api/chatflows.ts` | All CRUD + `generateAgentflow` + `getChatModels`, FlowData serialization | ✅ Done |
 | `src/infrastructure/api/nodes.ts` | `getAllNodes`, `getNodeByName`, `getNodeIconUrl` | ✅ Done |
-| `src/infrastructure/store/AgentflowContext.tsx` | `agentflowReducer` (all actions), `normalizeNodes`. Remaining: `deleteNode()`, `duplicateNode()`, `updateNodeData()`, `getFlowData()` | 🟡 Partial |
+| `src/infrastructure/store/AgentflowContext.tsx` | `agentflowReducer` (all actions), `normalizeNodes`, `deleteNode()`, `duplicateNode()`. Remaining:  `updateNodeData()`, `getFlowData()` | 🟡 Partial|
 | `src/useAgentflow.ts` | `getFlow()`, `toJSON()`, `validate()`, `addNode()`, `clear()` | ⬜ Not yet — thin wrapper |
 | `src/features/canvas/hooks/useFlowHandlers.ts` | `handleConnect`, `handleNodesChange`, `handleEdgesChange`, `handleAddNode` — synchronous `onFlowChange` callbacks, dirty tracking, viewport resolution, change filtering | ✅ Done |
 
@@ -130,6 +130,7 @@ Key features:
     -   `./src/features/canvas/hooks/useFlowHandlers.ts`
     -   `./src/features/node-palette/search.ts`
     -   `./src/infrastructure/api/`
+    -   ⏳ `./src/infrastructure/store/AgentflowContext.tsx` — will be added when coverage reaches 80%
 -   **Coverage exclusions**:
     -   `src/__test_utils__/**` — test utilities
     -   `src/__mocks__/**` — module mocks

packages/agentflow/jest.config.js

@@ -40,7 +40,8 @@ module.exports = {
         './src/core/': { branches: 80, functions: 80, lines: 80, statements: 80 },
         './src/features/canvas/hooks/useFlowHandlers.ts': { branches: 80, functions: 80, lines: 80, statements: 80 },
         './src/features/node-palette/search.ts': { branches: 80, functions: 80, lines: 80, statements: 80 },
-        './src/infrastructure/api/': { branches: 80, functions: 80, lines: 80, statements: 80 }
+        './src/infrastructure/api/': { branches: 80, functions: 80, lines: 80, statements: 80 },
+        './src/infrastructure/store/AgentflowContext.tsx': { branches: 80, functions: 80, lines: 80, statements: 80 }
     },
     projects: [
         // .test.ts → node (fast, no DOM)

packages/agentflow/src/Agentflow.tsx

@@ -51,7 +51,15 @@ function AgentflowCanvas({
     renderHeader?: AgentflowProps['renderHeader']
     renderNodePalette?: AgentflowProps['renderNodePalette']
 }) {
-    const { state, setNodes, setEdges, setDirty, setReactFlowInstance, closeEditDialog, registerLocalStateSetters } = useAgentflowContext()
+    const {
+        state,
+        syncNodesFromReactFlow,
+        syncEdgesFromReactFlow,
+        setDirty,
+        setReactFlowInstance,
+        closeEditDialog,
+        registerLocalStateSetters
+    } = useAgentflowContext()
     const { isDarkMode } = useConfigContext()
     const agentflow = useAgentflow()
     const reactFlowWrapper = useRef<HTMLDivElement>(null)
@@ -82,12 +90,12 @@ function AgentflowCanvas({
 
     // Sync local ReactFlow state to context (when user interacts with canvas)
     useEffect(() => {
-        setNodes(nodes as FlowNode[])
-    }, [nodes, setNodes])
+        syncNodesFromReactFlow(nodes as FlowNode[])
+    }, [nodes, syncNodesFromReactFlow])
 
     useEffect(() => {
-        setEdges(edges as FlowEdge[])
-    }, [edges, setEdges])
+        syncEdgesFromReactFlow(edges as FlowEdge[])
+    }, [edges, syncEdgesFromReactFlow])
 
     // Flow handlers
     const { handleConnect, handleNodesChange, handleNodeDragStop, handleEdgesChange, handleAddNode } = useFlowHandlers({