From ae0f84fde724255c8ec884eb19f30e261ecd35c5 Mon Sep 17 00:00:00 2001 From: Gustavo Henrique Santos Souza de Miranda Date: Wed, 21 Jan 2026 14:46:36 -0300 Subject: [PATCH] chore: mark US-027 as complete and update progress log Co-Authored-By: Claude Opus 4.5 --- prd.json | 2 +- progress.txt | 12 ++++++++++++ 2 files changed, 13 insertions(+), 1 deletion(-) diff --git a/prd.json b/prd.json index 8b12662..07200b2 100644 --- a/prd.json +++ b/prd.json @@ -483,7 +483,7 @@ "Verify in browser using dev-browser skill" ], "priority": 27, - "passes": false, + "passes": true, "notes": "" }, { diff --git a/progress.txt b/progress.txt index e8f8664..cb616df 100644 --- a/progress.txt +++ b/progress.txt @@ -397,3 +397,15 @@ - Variable nodes initialized with { variableName: '', operation: 'set', value: 0 } - All add node handlers share the same pattern and use the getViewportCenter helper --- + +## 2026-01-21 - US-027 +- What was implemented: Connect nodes with edges including arrow markers and smooth styling +- Files changed: + - src/app/editor/[projectId]/FlowchartEditor.tsx - added MarkerType import, updated onConnect to create edges with smoothstep type and arrow markers, updated toReactFlowEdges to apply same styling to loaded edges +- **Learnings for future iterations:** + - Use `type: 'smoothstep'` for cleaner edge curves instead of default bezier + - Use `markerEnd: { type: MarkerType.ArrowClosed }` to add directional arrows to edges + - Connection type has nullable source/target, but Edge requires non-null strings - guard with early return + - Apply consistent edge styling in both onConnect (new edges) and toReactFlowEdges (loaded edges) + - Generate unique edge IDs with nanoid in onConnect callback +---