the event handler really should not add any children (unless it recognizes the node as referencing some other menu and copying that all here - but that is triggering another run of a menu factory if the other menu is coming from storage). creating the item "as normal" could fail for some nodes, if they do not contain information that any factory can make sense of - and just containing them to create a dummy item that is removed afterwards feels wrong.

hm, appending a child at the end could make sense, and would not be possible in a clean way. maybe having a second event at the very end of the createFromNode method makes sense, yes. that one would get both node and item and allow to again alter the item by adding (or removing) certain children.