Edge should accept a generic type argument like Node

[kevinschaich]

Definition for Node:

export type Node<T = any, U extends string | undefined = string | undefined> = {
    id: string;
    data: T;
    type?: U;
    ...
}

Usage:

export type MyNode = Node<MyNodeData, 'my-node'>

Perfect. The def for Edge is inconsistent:

export type Edge<T = any> = DefaultEdge<T> | SmoothStepEdgeType<T> | BezierEdgeType<T>;

...

type DefaultEdge<T = any> = {
    id: string;
    data?: T;
    type?: string;
    ...
}

This means you can't enforce that generic props map to a discrete number of edge types throughout your app.

[kevinschaich]

Actually should mention I'm on v11 and these are the exported types I'm seeing. Maybe fixed in:

• ✨ @xyflow/react - React Flow 12 is out #3764

cc: @moklick ? Happy to close out if so

[moklick]

Yep, that's fixed in v12 https://github.com/xyflow/xyflow/blob/main/packages/system/src/types/edges.ts