The structured (switch-containing) renderer passed showOrdering = false, so the overflow menu held only Remove — no Move up/Move down existed for any step in such a workflow.
withStepMoved required linearActionNodesOrNull(), which is null for any graph holding a switch, so even a wired-up control would have been a silent no-op.

Moves are now lane-relative on the route projection: a step reorders within its own lane (trunk or branch), and the lane's edges are rebuilt from the new order — including the branch-tail edges that feed whatever follows the switch. A move that would place a step above the step whose output it reads is refused with a sentence naming both, leaving the draft unchanged.

Verified on emulator-5554: Move up on the switch reordered the trunk; Move up inside the "Matching" branch reordered that lane only; moving a step above its value source produced "Show notification reads a value from Make uppercase, which would no longer run before it." with the route unchanged. Covered by a step moves within a branch lane of a switched workflow, a trunk step moves even though the route holds a switch, and a move that would strand a value reference is refused with an explanation.

Fixed in `e4f3978` (committed locally, push blocked on a 1Password approval). Two causes, both gone: - The structured (switch-containing) renderer passed `showOrdering = false`, so the overflow menu held only **Remove** — no Move up/Move down existed for any step in such a workflow. - `withStepMoved` required `linearActionNodesOrNull()`, which is null for any graph holding a switch, so even a wired-up control would have been a silent no-op. Moves are now lane-relative on the route projection: a step reorders within its own lane (trunk or branch), and the lane's edges are rebuilt from the new order — including the branch-tail edges that feed whatever follows the switch. A move that would place a step above the step whose output it reads is refused with a sentence naming both, leaving the draft unchanged. Verified on emulator-5554: Move up on the switch reordered the trunk; Move up inside the "Matching" branch reordered that lane only; moving a step above its value source produced "Show notification reads a value from Make uppercase, which would no longer run before it." with the route unchanged. Covered by `a step moves within a branch lane of a switched workflow`, `a trunk step moves even though the route holds a switch`, and `a move that would strand a value reference is refused with an explanation`.

morten-olsen referenced this issue

2026-09-17 14:16:43 +02:00

Add drag reorder and clipboard operations to workflow editing #13

morten-olsen referenced this issue from a commit

2026-09-17 14:17:27 +02:00

feat(editor): reorder, extend and copy steps on a route projection

morten-olsen closed this issue

2026-09-17 14:17:35 +02:00