I don't know what triggered such a horrible code reformation in canvas, actually this is just a very tiny modification. Eventually, I adopted another simpler optimization strategy other than what I proposed here. Please test it carefully before merging, it works on my side, but I'm not sure if any issue exists.

A brief summary:

1. Make all tooltips not draggable.
2. Rewrite store.setPodPosition and store.setPodParent, add a parameter dirty: bool, set pod.dirty = pod.dirty | dirty, that is, only triggers remote update request when the argument dirty=true. Note that we use | (or) operation instead of = (assign) to avoid a dirty pod cleaned by another local setPodPosition or setPodParent with dirty=false before the update is synced to the DB.
3. For codeNode and scopeNode, useEffect listens on the change of node position or change, call setPodPosition or setPodParent with dirty=false whenever the corresponding position or parent node changes. We don't need to write the change to DB for now, but we must sync store.pods with the nodes in canvas.
4. dirty=true is only used when stopping dragging nodes (in onNodeDragStop, that is, we only writes the update to DB when it is caused by operations by our current user. We don't need to write update pulled from the remote yjs server to the DB.

Possible issue:

• During dragging, the position of a node changes all the time, so we keep rendering the node and calling setPodPosition continually, even though it doesn't change store.pods[id].dirty and no data is written to the DB. One solution to mitigate the update to store: take a look at dragging as well, only update store.pods[id] after the dragging stops, like:

function codeNode({..., xPos, yPos, dragging}) {
// ....
useEffect( () => {
 if(!dragging) {
 let [x, y] = relativePostion(xPos, yPos); // transform XY coordinate
 setPodPosition({id, x, y, dirty: false})
}
}, [...,xPos, yPos, dragging])
// ....
}

But I think the performance of dragging is acceptable now, if you feel uncomfortable with the performance in the future, the solution is just for reference.

close #138