Fix two undo steps on add node via context menu

[webfiltered]

Removes change tracker monkey patch on LGraph.onNodeAdded.

┆Issue is synchronized with this Notion page by Unito

[huchenlei]

I am not so sure whether teh onNodeAdded hook can be removed, as it seems to also detect the add node controlled by script. Can you further explain which other place is also checking the state when adding node via searchbox?

[webfiltered]

checking the state when adding node via searchbox

Not sure if I misread, but there's an addition in this PR on NodeSearchBoxPopover.vue - when the search box actually adds a node, that method now calls changeTracker immediately after.

As for anything else using onNodeAdded, it is very likely doubling up on the pointerup / keyup checks. Unless nodes are being added without any user interaction... but if that's the case, should an undo step be created every time graph.add(node) is called?