[bugfix] disable analytics manager on SSR

[franjohn21]

Seeing this error when using this package in a node app. Think we should just be safe and not save to local storage in the even the environment doesn't support it.

const cachedEventIdsValue = localStorage.getItem("tdk-analytics-event-ids");
                              ^
ReferenceError: localStorage is not defined

[changeset-bot]

🦋 Changeset detected

Latest commit: 654619b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages

Name	Type
@treasure-dev/tdk-react	Patch
@treasure-dev/tdk-core	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[vercel]

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

2 Skipped Deployments

Name	Status	Preview	Comments	Updated (UTC)
tdk-examples-connect	⬜️ Ignored (Inspect)	Visit Preview		Nov 13, 2024 9:52pm
tdk-examples-magicswap	⬜️ Ignored (Inspect)	Visit Preview		Nov 13, 2024 9:52pm

[franjohn21]

Actually now that I'm thinking more about this fix -- this is a bandaid. Should the AnalyticsManager even be running in SSR context? What do we want to track?

If i follow the code correctly I think there's technically a memory leak -- on every new render we'd initialize a new AnalyticsManager which invokes a new setInterval. This means there could be an ever increasing number of intervals added and will crash the server. This might not ever be a problem in vercel/lambda serverless functions architecture but if an app is a singular node server it's an issue and we shouldn't assume the SSR is running in a serverless function.

tdk-js/packages/core/src/analytics/AnalyticsManager.ts

Line 66 in cea79c5

setInterval(