fix: avoid double initialization of node context menu items

[willeastcott]

Summary

Fixes the runtime error thrown when adding a node to a Graph:

Failed to execute 'structuredClone' on 'Window':
() => this._createUnconnectedEdgeForNode(node, item.edgeType) could not be cloned.

This breaks every story in the published Storybook (and any consumer that builds nodes with contextMenuItems in the schema, e.g. the React example).

Root cause

Graph.createNode was initializing context menu items and then handing them to view.addNodeContextMenu, which eventually calls GraphViewNode.addContextMenu — which initializes them again:

• First call: clones the schema items (no functions yet) and attaches onSelect callbacks.
• Second call: hits structuredClone on items that now contain onSelect function references → throws.

The regression was exposed by #136, which replaced the custom deepCopyFunction with structuredClone. The previous custom deep-copy iterated with for...in and silently preserved function references, masking this latent double-initialization bug.

The other call site (GraphViewNode.updateNodeType) already passes raw schema items directly, confirming addContextMenu is the correct (single) place to initialize.

Fix

In Graph.createNode, pass nodeSchema.contextMenuItems directly to view.addNodeContextMenu. Initialization now happens exactly once inside GraphViewNode.addContextMenu.

if (nodeSchema.contextMenuItems) {
    this.view.addNodeContextMenu(node.id, nodeSchema.contextMenuItems);
}

No public API changes.

Fixes #141

Test plan

• npm run lint passes
• npm run storybook opens without the structuredClone error
• Right-click a node in Directed Graph and Visual Programming Graph stories → context menu appears, items (e.g. "Delete Node", "Add ... Edge") are functional
• The React example (https://playcanvas.github.io/pcui-graph/storybook/) renders without errors after release

[Copilot]

Pull request overview

Fixes a runtime structuredClone error when creating nodes with contextMenuItems by ensuring node context menu items are initialized exactly once (in GraphViewNode.addContextMenu), restoring Storybook and consumer functionality.

Changes:

• Stop initializing/cloning nodeSchema.contextMenuItems inside Graph.createNode.
• Pass raw schema contextMenuItems directly to view.addNodeContextMenu, avoiding double-initialization and cloning functions.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.