fix(apollo-react): compute node height from handle count to stop flicker [MST-11677]

[KodudulaAshishUiPath]

Summary

Fixes the Switch-node height flicker (MST-11677): deleting a case from an unconnected Switch made it oscillate between the 2-handle (96px) and 3-handle (128px) sizes. Root cause was a feedback loop where BaseNode derived its height from the measured height prop and wrote it back via updateNode inside a requestAnimationFrame.

Changes

• computedHeight is now a pure function of visible handle count + intrinsic footer/default height (getIntrinsicHeight). It never reads the measured height prop, so the value written back can't feed into its own input — the loop is structurally gone.
• A single, synchronous useEffect writes computedHeight to node.height (no rAF), only when it differs from the explicit getNode(id)?.height, then calls updateNodeInternals. Comparing against the explicit height (not the measured prop) means a lagging ResizeObserver measurement can't retrigger the write. Convergent and idempotent.
• Keeping node.height authoritative fixes the downstream model: measured height, edge anchoring, fit-view, selection bounds, and handle spacing all agree with the rendered body (fixes handles overflowing on high-fan-out nodes).
• animatedViewportManager now reads node.measured?.height ?? node.height ?? …, matching the package's dominant convention (previously node.height || 100, which went stale when height wasn't written).
• Removed the old baseHeightRef/syncedHeightRef bookkeeping and the dead getContainerHeight height param.

Testing

• pnpm lint (Biome) passes
• pnpm typecheck passes
• Tests pass — rewritten height suite asserts the write-once/converge behavior, the 128→96 deflation settles with no re-inflation, and no write when already correct (full apollo-react suite green, 1885)
• pnpm test:visual — runs in CI
• No as any / type suppressions added

[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 15, 2026, 11:43:13 PM
apollo-docs	🟢 Ready	Preview, Logs	Jul 15, 2026, 11:43:13 PM
apollo-landing	🟢 Ready	Preview, Logs	Jul 15, 2026, 11:43:13 PM
apollo-vertex	🟢 Ready	Preview, Logs	Jul 15, 2026, 11:43:13 PM

[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]

Pull request overview

Fixes a React Flow node-height oscillation in apollo-react canvas BaseNode caused by a deferred requestAnimationFrame height sync briefly stamping a “synced” height before the write actually committed, allowing a stale in-flight height to be treated as an external resize.

Changes:

• Move syncedHeightRef stamping into the rAF callback so it only records heights that were actually committed via updateNode.
• Tighten comments around syncedHeightRef / baseHeightRef semantics to reflect commit-time stamping and the external-vs-internal height distinction.
• Add a regression test that keeps rAF deferred and reproduces the stale interleaved render scenario, asserting the node deflates and remains stable.

Reviewed changes

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

File	Description
packages/apollo-react/src/canvas/components/BaseNode/BaseNode.tsx	Stamps syncedHeightRef at rAF commit time (paired with updateNode) to prevent stale in-flight heights from poisoning baseHeightRef.
packages/apollo-react/src/canvas/components/BaseNode/BaseNode.test.tsx	Adds a deferred-rAF regression test to reproduce and prevent the 96px ↔ 128px flicker loop.

[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%	37.9% (11/29)	7.34 MB	27.98 MB	−219 B
@uipath/apollo-wind	40.3%	—	395.1 KB	2.57 MB	−10 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]

Pull request overview

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

[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.

[SreedharAvvari]

https://apollo-design-qxvi3khj6-uipath.vercel.app/?path=/story/apollo-react-canvas-components-nodes-basenode--sizes story looks broken compared to https://apollo-design.vercel.app/?path=/story/apollo-react-canvas-components-nodes-basenode--sizes

[Copilot]

Pull request overview

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

[BenGSchulz]

Thanks for digging through for any custom height needs. I would just like to see the sizes story updated or removed so it doesn't cause confusion.

[KodudulaAshishUiPath]

Thanks for digging through for any custom height needs. I would just like to see the sizes story updated or removed so it doesn't cause confusion.

Yeah sure, will update that

[Copilot]

Pull request overview

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

[Copilot]

Pull request overview

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

[Copilot]

Pull request overview

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

[Copilot]

Pull request overview

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