fix: resolve circular dependency in Analytics context

[VolodymyrBg]

What changed?

Fixed a critical issue in the Analytics context implementation where there was a circular dependency problem. The changes include:

1. Removed self-referential context usage in AnalyticsProvider
2. Added a new parentContext prop for proper context inheritance
3. Optimized context string concatenation using useMemo
4. Improved type safety and error handling

Key changes:

• Replaced useAnalytics() usage inside provider with direct prop-based context inheritance
• Added optional parentContext prop to AnalyticsProviderProps
• Implemented proper context string concatenation with memoization

Notes to reviewers

• The main issue was that the provider was trying to use its own context during initialization, which could lead to infinite loops and undefined behavior
• The new implementation follows React's context best practices
• Special attention was given to maintaining backward compatibility
• The TypeScript error about React imports needs to be addressed at the project configuration level

How has it been tested?

The changes have been tested in the following scenarios:

1. Basic context initialization

   • Verified provider initializes without circular dependency errors
   • Confirmed context values are properly passed to children

2. Context inheritance

   • Tested nested providers with different context values
   • Verified proper concatenation of parent and child contexts

3. Event logging

   • Confirmed logEventWithContext continues to work as expected
   • Verified event context strings are properly formatted

4. Error scenarios

   • Verified proper error handling when context is used outside provider
   • Tested with undefined/empty context values

No regression issues were found during testing.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
web-base-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 31, 2025 8:24pm

[cb-heimdall]

🟡 Heimdall Review Status

Requirement	Status	More Info
Reviews	🟡 0/1	Denominator calculation Show calculation 1 if user is bot 0 1 if user is external 0 From .codeflow.yml 1 Additional review requirements Show calculation Max 0 0 From CODEOWNERS 0 Global minimum 0 Max 1 1 1 if commit is unverified 0 Sum 1

[vercel]

@VolodymyrBg is attempting to deploy a commit to the Coinbase Team on Vercel.

A member of the Team first needs to authorize it.