fix(apollo-react): clear node faces so routed edges exit perpendicular [MST-11337]

[KodudulaAshishUiPath]

Summary

Routed (auto-layout) edges on multi-handle nodes rendered with a kink right at the source/target handle — no perpendicular exit offset. This shifts the bend nearest each node face out to a fixed perpendicular STUB_OFFSET so edges leave and enter cleanly.

ELK computes bend points from its port positions, which differ from the rendered React Flow handle positions on multi-handle nodes (Decision true/false, Script main/error). The first/last routed bend lands at or behind the handle, so the edge jogs immediately instead of exiting straight.

Demo

Before:

After:

Files Changed

• New clearNodeFace(waypoints, anchor, position) in geometry.ts: shifts the waypoint riser nearest a node face out to STUB_OFFSET (lands exactly STUB_OFFSET in front for any gap, including bends behind the face). Applied to both source and target faces in buildPathVertices.
• Gated behind a new autoRouted flag, threaded buildPathVertices → useEdgeGeometry → CanvasEdge, so it only affects router-produced routedWaypoints; manual waypoints render exactly where the user placed them.
• Shifting the stored waypoints (ids preserved) survives consolidateWaypoints, unlike a derived stub elbow.

Tests

New buildPathVertices / clearNodeFace cases: source + target shift, vertical (y-axis) face, behind-face forward-pull, and manual-not-shifted. 29 geometry / 85 Edges tests pass; biome + tsc clean.

🤖 Generated with Claude Code

[github-actions]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Review	Updated (PT)
apollo-design	🟢 Ready	Preview, Logs	Jul 16, 2026, 02:30:29 AM
apollo-docs	🟢 Ready	Preview, Logs	Jul 16, 2026, 02:30:29 AM
apollo-landing	🟢 Ready	Preview, Logs	Jul 16, 2026, 02:30:29 AM
apollo-vertex	🟢 Ready	Preview, Logs	Jul 16, 2026, 02:30:29 AM

[github-actions]

Dependency License Review

• ✅ 1950 package(s) scanned
• ✅ No license issues found
• ⚠️ 2 package(s) excluded (see details below)

License distribution

License	Packages
MIT	1720
ISC	89
Apache-2.0	55
BSD-3-Clause	27
BSD-2-Clause	23
BlueOak-1.0.0	8
MPL-2.0	4
MIT-0	3
CC0-1.0	3
MIT OR Apache-2.0	2
(MIT OR Apache-2.0)	2
Unlicense	2
LGPL-3.0-or-later	1
Python-2.0	1
CC-BY-4.0	1
(MPL-2.0 OR Apache-2.0)	1
Unknown	1
Artistic-2.0	1
(WTFPL OR MIT)	1
(BSD-2-Clause OR MIT OR Apache-2.0)	1
CC-BY-3.0	1
0BSD	1
(MIT OR CC0-1.0)	1
MIT AND ISC	1

Excluded packages

Package	Version	License	Reason
@img/sharp-libvips-linux-x64	1.2.4	LGPL-3.0-or-later	LGPL pre-built binary, not linked
khroma	2.1.0	Unknown	MIT per GitHub repo, missing license field in package.json

[Copilot]

✅ Ready to approve

The change is well-scoped behind an autoRouted flag, and the new geometry behavior is covered by targeted regression tests.

Note: this review does not count toward required approvals for merging.

Pull request overview

This PR improves auto-routed (graph router/ELK) Canvas edge rendering for multi-handle nodes by ensuring routed edges exit/enter node faces with a consistent perpendicular “stub” offset, eliminating immediate kinks at handles while keeping manually placed waypoints untouched.

Changes:

• Added clearNodeFace() to shift the nearest routed riser off node faces to EDGE_CONSTANTS.STUB_OFFSET, applied at both source and target in buildPathVertices.
• Threaded a new autoRouted flag from CanvasEdge → useEdgeGeometry → buildPathVertices to apply face-clearance only to router-produced waypoints.
• Added unit tests covering source/target clearance, vertical faces, behind-face pulls, and manual-waypoint non-shifting.

File summaries

File	Description
packages/apollo-react/src/canvas/components/Edges/shared/hooks/useEdgeGeometry.ts	Adds autoRouted arg and passes it through to buildPathVertices.
packages/apollo-react/src/canvas/components/Edges/shared/geometry.ts	Implements node-face clearance logic and applies it conditionally for auto-routed waypoints.
packages/apollo-react/src/canvas/components/Edges/shared/geometry.test.ts	Adds regression/unit coverage for the new clearance behavior.
packages/apollo-react/src/canvas/components/Edges/CanvasEdge.tsx	Sets autoRouted when rendering router-produced routedWaypoints (no manual waypoints).

Copilot's findings

• Files reviewed: 4/4 changed files
• Comments generated: 1

Note

Your feedback helps us improve the quality of this feature.
Please use 👍 or 👎 to tell us whether this assessment is correct.

[github-actions]

📊 Coverage + size by package

Per-package coverage and bundle size on this PR. New-line coverage = of the source lines this PR adds or changes, the % hit by tests.

Package	Coverage	New-line coverage	Packed (gzip)	Unpacked	vs main
@uipath/apollo-core	9.0%	—	43.82 MB	57.31 MB	±0
@uipath/apollo-react	35.7%	100.0% (17/17)	7.35 MB	27.99 MB	+835 B
@uipath/apollo-wind	40.3%	—	395.1 KB	2.57 MB	−15 B
@uipath/ap-chat	85.8%	—	43.43 MB	55.91 MB	±0

_{"Coverage" is each package's own coverage.include scope (e.g. apollo-core instruments only scripts/). "Packed"/"Unpacked" come from npm pack --dry-run and only cover built packages — "—" means not measured this run (package not affected / not built). "vs main" is the packed (gzipped) delta against the last successful main build (the package-sizes artifact from the Release workflow); "—" there means no main baseline was available this run. The baseline is main's latest build, not this PR's exact merge-base, so it includes any drift since the branch diverged. Packages with no vitest config are omitted.}

[Copilot]

⚠️ Not ready to approve

clearNodeFace currently determines the “nearest” riser via a global extremum across all waypoints, which can select the wrong segment for non-monotonic routed polylines and fail to clear the actual face-adjacent bend.

Copilot's findings

• Files reviewed: 4/4 changed files
• Comments generated: 2

Note

Your feedback helps us improve the quality of this feature.
Please use 👍 or 👎 to tell us whether this assessment is correct.

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

[snuziale]

do you even need this autoRouted
can you just check the waypoints length downstream?

[KodudulaAshishUiPath]

Downstream we only get effectiveWaypoints, which is already waypoints.length > 0 ? waypoints : routedWaypoints — so a non-empty array could be either manual (user-placed) or router-produced. A length check downstream can't tell them apart; autoRouted is what carries that distinction (face-clearance should apply only to router waypoints). We could instead pass both arrays down and check manual.length there, but that pushes the merge into the geometry layer.

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.