reactflow-edge-routing 0.1.1 → 0.1.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "reactflow-edge-routing",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "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/routing-core.ts

@@ -748,7 +748,10 @@ function createConnections(
 
     if (splitNearHandle) {
       // Pin-based: edges spread out along the node border near handles
-      const srcHandle = edge.sourceHandle ?? (autoBestSide ? null : findDefaultHandle(src, "source"));
+      // When autoBestSide is on, ignore explicit sourceHandle/targetHandle on edges
+      // and let the router pick the optimal side. Only use explicit handles when
+      // autoBestSide is off (pin-based routing with fixed handle positions).
+      const srcHandle = autoBestSide ? null : (edge.sourceHandle ?? findDefaultHandle(src, "source"));
       if (srcShapeRef && srcHandle) {
         const pinId = pinRegistry.getOrCreate(edge.source, srcHandle);
         srcEnd = AvoidConnEnd.fromShapePin(srcShapeRef as any, pinId);
@@ -759,7 +762,7 @@ function createConnections(
         srcEnd = (() => { const pt = Geometry.getHandlePoint(srcBounds, side); return AvoidConnEnd.fromPoint(new AvoidPoint(pt.x, pt.y)); })();
       }
 
-      const tgtHandle = edge.targetHandle ?? (autoBestSide ? null : findDefaultHandle(tgt, "target"));
+      const tgtHandle = autoBestSide ? null : (edge.targetHandle ?? findDefaultHandle(tgt, "target"));
       if (tgtShapeRef && tgtHandle) {
         const pinId = pinRegistry.getOrCreate(edge.target, tgtHandle);
         tgtEnd = AvoidConnEnd.fromShapePin(tgtShapeRef as any, pinId);
@@ -848,9 +851,14 @@ export class RoutingEngine {
       }
     }
 
-    // Adjust spacing at shared handles (fan-out effect) — skip when splitNearHandle is off
+    // Adjust spacing at shared handles (fan-out effect).
+    // Only for splitNearHandle=false (convergence mode): edges all start from the same center
+    // point, libavoid nudges them apart by idealNudgingDistance, and HandleSpacing.adjust scales
+    // that spread by handleNudging/idealNudging → final fan-out = handleNudging (clean decoupling).
+    // For splitNearHandle=true (pin-based), pins already fix the fan-out; scaling would incorrectly
+    // compress or expand existing pin spread, causing edgeToEdgeSpacing to reduce visible spacing.
     const splitNearHandle = opts.shouldSplitEdgesNearHandle ?? true;
-    if (splitNearHandle) {
+    if (!splitNearHandle) {
       const idealNudging = opts.idealNudgingDistance ?? 10;
       const handleNudging = opts.handleNudgingDistance ?? idealNudging;
       if (handleNudging !== idealNudging && edgePoints.size > 0) {
@@ -1022,9 +1030,9 @@ export class PersistentRouter {
       }
     }
 
-    // Adjust spacing at shared handles (fan-out effect) — skip when splitNearHandle is off
+    // Adjust spacing at shared handles (fan-out effect) — only for splitNearHandle=false (convergence mode)
     const splitNearHandle = opts.shouldSplitEdgesNearHandle ?? true;
-    if (splitNearHandle && handleNudging !== idealNudging && edgePoints.size > 0) {
+    if (!splitNearHandle && handleNudging !== idealNudging && edgePoints.size > 0) {
       HandleSpacing.adjust(this.prevEdges, edgePoints, handleNudging, idealNudging);
     }
 

package/src/use-edge-routing.ts

@@ -130,7 +130,9 @@ function toRouterOptions(opts?: UseEdgeRoutingOptions): AvoidRouterOptions {
     diagramGridSize: opts?.diagramGridSize ?? DEFAULT_OPTIONS.diagramGridSize,
     shouldSplitEdgesNearHandle: opts?.shouldSplitEdgesNearHandle ?? DEFAULT_OPTIONS.shouldSplitEdgesNearHandle,
     stubSize: opts?.stubSize,
-    autoBestSideConnection: opts?.autoBestSideConnection ?? DEFAULT_OPTIONS.autoBestSideConnection,
+    // bezier defaults to autoBestSideConnection: true — explicit handles
+    // make no visual sense on curved paths, so auto-side is the right default.
+    autoBestSideConnection: opts?.autoBestSideConnection ?? (opts?.connectorType === "bezier" ? true : DEFAULT_OPTIONS.autoBestSideConnection),
     debounceMs: opts?.debounceMs ?? DEFAULT_OPTIONS.debounceMs,
   };
 }