reactflow-edge-routing 0.1.8 → 0.1.11

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "reactflow-edge-routing",
-  "version": "0.1.8",
+  "version": "0.1.11",
   "description": "Orthogonal edge routing for React Flow using obstacle-router. Edges route around nodes while pins stay fixed at their anchor points.",
   "main": "./src/index.ts",
   "types": "./src/index.ts",

package/src/index.ts

@@ -34,7 +34,7 @@ export { useEdgeRouting } from "./use-edge-routing";
 export type { UseEdgeRoutingOptions, UseEdgeRoutingResult } from "./use-edge-routing";
 
 export { useRoutedEdgePath } from "./use-routed-edge-path";
-export type { UseRoutedEdgePathParams } from "./use-routed-edge-path";
+export type { UseRoutedEdgePathParams, RoutePinPoints } from "./use-routed-edge-path";
 
 // Collision resolution
 export { resolveCollisions } from "./resolve-collisions";

package/src/routing-core.ts

@@ -200,7 +200,7 @@ function getConnType(connectorType: ConnectorType | undefined): number {
 /** Get the Router flags for the connector type. */
 function getRouterFlags(connectorType: ConnectorType | undefined): number {
   switch (connectorType) {
-    case "polyline": return PolyLineRouting | OrthogonalRouting;
+    case "polyline": return PolyLineRouting;
     default: return OrthogonalRouting;
   }
 }
@@ -737,13 +737,16 @@ function configureRouter(router: AvoidRouter, options: AvoidRouterOptions): void
   }
 
   // --- Routing options ---
-  router.setRoutingOption(nudgeOrthogonalSegmentsConnectedToShapesOpt, options.nudgeOrthogonalSegmentsConnectedToShapes ?? true);
-  router.setRoutingOption(nudgeSharedPathsWithCommonEndPointOpt, options.nudgeSharedPathsWithCommonEndPoint ?? true);
-  router.setRoutingOption(performUnifyingNudgingPreprocessingStepOpt, options.performUnifyingNudgingPreprocessingStep ?? true);
-  router.setRoutingOption(nudgeOrthogonalTouchingColinearSegmentsOpt, options.nudgeOrthogonalTouchingColinearSegments ?? false);
-  router.setRoutingOption(improveHyperedgeRoutesMovingJunctionsOpt, options.improveHyperedgeRoutesMovingJunctions ?? true);
-  router.setRoutingOption(penaliseOrthogonalSharedPathsAtConnEndsOpt, options.penaliseOrthogonalSharedPathsAtConnEnds ?? false);
-  router.setRoutingOption(improveHyperedgeRoutesMovingAddingAndDeletingJunctionsOpt, options.improveHyperedgeRoutesMovingAddingAndDeletingJunctions ?? false);
+  // Orthogonal nudging options only apply to orthogonal/bezier routing.
+  // Enabling them for polyline connectors corrupts the routed paths.
+  const isPolyline = options.connectorType === "polyline";
+  router.setRoutingOption(nudgeOrthogonalSegmentsConnectedToShapesOpt, isPolyline ? false : (options.nudgeOrthogonalSegmentsConnectedToShapes ?? true));
+  router.setRoutingOption(nudgeSharedPathsWithCommonEndPointOpt, isPolyline ? false : (options.nudgeSharedPathsWithCommonEndPoint ?? true));
+  router.setRoutingOption(performUnifyingNudgingPreprocessingStepOpt, isPolyline ? false : (options.performUnifyingNudgingPreprocessingStep ?? true));
+  router.setRoutingOption(nudgeOrthogonalTouchingColinearSegmentsOpt, isPolyline ? false : (options.nudgeOrthogonalTouchingColinearSegments ?? false));
+  router.setRoutingOption(improveHyperedgeRoutesMovingJunctionsOpt, isPolyline ? false : (options.improveHyperedgeRoutesMovingJunctions ?? true));
+  router.setRoutingOption(penaliseOrthogonalSharedPathsAtConnEndsOpt, isPolyline ? false : (options.penaliseOrthogonalSharedPathsAtConnEnds ?? false));
+  router.setRoutingOption(improveHyperedgeRoutesMovingAddingAndDeletingJunctionsOpt, isPolyline ? false : (options.improveHyperedgeRoutesMovingAddingAndDeletingJunctions ?? false));
 }
 
 // ---- Routing helpers ----

package/src/use-routed-edge-path.ts

@@ -56,8 +56,13 @@ function getFallback(
   return getStraightPath({ sourceX, sourceY, targetX, targetY });
 }
 
+export type RoutePinPoints = {
+  sourceX: number; sourceY: number;
+  targetX: number; targetY: number;
+};
+
 /**
- * Returns [path, labelX, labelY, wasRouted, controlPoints] for a routed edge.
+ * Returns [path, labelX, labelY, wasRouted, controlPoints, pinPoints] for a routed edge.
  *
  * - While a connected node is being dragged → dashed fallback
  * - If a routed path exists for this edge → solid routed path
@@ -67,10 +72,13 @@ function getFallback(
  * the source and target anchor points). Use them to build an editable edge on
  * top of auto-routing — e.g. render draggable handles at each waypoint.
  * When no route is available (fallback), `controlPoints` is an empty array.
+ *
+ * `pinPoints` are the exact source/target positions the router snapped to.
+ * Falls back to the passed-in sourceX/Y, targetX/Y when no route is available.
  */
 export function useRoutedEdgePath(
   params: UseRoutedEdgePathParams
-): [path: string, labelX: number, labelY: number, wasRouted: boolean, controlPoints: { x: number; y: number }[]] {
+): [path: string, labelX: number, labelY: number, wasRouted: boolean, controlPoints: { x: number; y: number }[], pinPoints: RoutePinPoints] {
   const {
     id, sourceX, sourceY, targetX, targetY,
     sourcePosition, targetPosition,
@@ -81,6 +89,8 @@ export function useRoutedEdgePath(
   const draggingNodeIds = useEdgeRoutingStore((s) => s.draggingNodeIds);
 
   return useMemo(() => {
+    const fallbackPins: RoutePinPoints = { sourceX, sourceY, targetX, targetY };
+
     // If a connected node is being dragged, show fallback
     const isDragging =
       draggingNodeIds.size > 0 &&
@@ -89,7 +99,7 @@ export function useRoutedEdgePath(
 
     if (isDragging) {
       const [path, lx, ly] = getFallback(sourceX, sourceY, targetX, targetY, sourcePosition, targetPosition, connectorType, params.borderRadius, offset);
-      return [path, lx, ly, false, []];
+      return [path, lx, ly, false, [], fallbackPins];
     }
 
     // If we have a routed path, use it (no stale check — trust the router)
@@ -99,11 +109,15 @@ export function useRoutedEdgePath(
       const controlPoints = route.points && route.points.length > 2
         ? route.points.slice(1, -1)
         : [];
-      return [route.path, route.labelX, route.labelY, true, controlPoints];
+      const pins: RoutePinPoints = {
+        sourceX: route.sourceX, sourceY: route.sourceY,
+        targetX: route.targetX, targetY: route.targetY,
+      };
+      return [route.path, route.labelX, route.labelY, true, controlPoints, pins];
     }
 
     // No route yet — show fallback
     const [path, lx, ly] = getFallback(sourceX, sourceY, targetX, targetY, sourcePosition, targetPosition, connectorType, params.borderRadius, offset);
-    return [path, lx, ly, false, []];
+    return [path, lx, ly, false, [], fallbackPins];
   }, [route, sourceX, sourceY, targetX, targetY, sourcePosition, targetPosition, offset, connectorType, params.borderRadius, source, target, draggingNodeIds]);
 }