Fix Node Copy + Paste

[camielvs]

Description

Fixes a couple of issues relating to copy + paste nodes & node duplication. These issues were pre-existing to this PR stack (but made easier to solve using the new Node Manager).

Fixes:

• IO Nodes don't copy metadata (name, value, type etc)
• Can't copy + paste across multiple instances of the app
• Output nodes fail to connect when pasted

KNOWN ISSUE: copy + paste output nodes between Oasis instances does not copy over the connection to that node. This is because in the new instance we do not have access to the original graphSpec.outputValues data (from the old instance) which tells us what it is connected to. This is a complex issue which will be addressed separately. For now users will need to manually remake that connection. For more info see https://github.com/Shopify/oasis-frontend/issues/314

Related Issue and Pull requests

Closes https://github.com/Shopify/oasis-frontend/issues/250
Closes https://github.com/Shopify/oasis-frontend/issues/302

Type of Change

• Bug fix

Checklist

• I have tested this does not break current pipelines / runs functionality
• I have tested the changes on staging

Screenshots (if applicable)

Test Instructions

• Verify that copy + paste single nodes and groups of nodes works as expected. Including combinations of IO nodes and tasks
• Verify that it now also works when copying across different pipelines/tabs

Note: output nodes copied across tabs will not be connected due to technical limitations in our architecture

Additional Comments

NOT FOR THIS PR - but duplicateNodes method desperately needs a refactor.

[camielvs]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• Fix inability to delete edges #1040
• Add debug mode for viewing ReactFlow Node ids #1016
• Fix Disconnected Output Nodes when Copy + Paste between Tabs #1186
• Fix Node Copy + Paste #1114 👈 (View in Graphite)
• Refactor HandleConnection #1096
• Refactor addAndConnectNode #1089
• Fix Node Replacement Issues on IO Rename #1083 : 1 other dependent PR (#1087 )
• Implement Node Manager #1004 : 2 other dependent PRs (#1015 , #1017 )
• useNodeManager Hook #1082 : 1 other dependent PR (#930 )
• Provide Node Manager #1081
• Add Node Manager #929
• Groundwork for Id Conversions #1090
• Separate IONodeData type from TaskNodeData type #928
• Refactor Dynamic Node Callbacks #927
• master

This stack of pull requests is managed by Graphite. Learn more about stacking.

[camielvs]

This will remove instances where we end up with task, task 2, task 2 2, task 2 2 2 and so on. Currently this happens for input & output nodes duplicated via the toolbar.

Now the sequence will always be task, task (2), task (3) and so on. The brackets were added to minimize the chance of matching the regex to a non-duplicate task id with numbers in it.