decentraland / builder / 32970542251

54%

master: 53%

Pull #3482

github

fix: infinite render loop in the Intercom component

analyticsReadyCallback depended on intercomUserData and set a new state
object on every invocation, so each render recreated the callback,
re-ran the effect, and re-registered analytics.ready() — which invokes
the callback again once analytics.js is loaded. The result was a
permanent ~1,300 commits/sec render loop calling Intercom('update') on
every cycle.

Latent since the callback was introduced, it started firing for
everyone once #3475 moved analytics.js behind a first-party proxy that
ad blockers no longer drop.

Fix: stable callback plus a functional update that returns the same
reference when the anonymous id hasn't changed, so React bails out.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

2598 of 5991 branches covered (43.37%)

Branch coverage included in aggregate %.

7743 of 13278 relevant lines covered (58.31%)

37.58 hits per line