Repository navigation
Conversation
ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Free Run ID: ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (38)
📝 WalkthroughWalkthroughThis pull request performs a comprehensive rebranding of the analytics SDK from Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~22 minutes Poem
Note 🎁 Summarized by CodeRabbit FreeThe PR author is not assigned a seat. To perform a comprehensive line-by-line review, please assign a seat to the pull request author through the subscription management page by visiting https://app.coderabbit.ai/login. Comment |
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 919eebff44
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| withdrawConsent, | ||
| EventTypes, | ||
| SDK_VERSION, | ||
| VERSION, |
There was a problem hiding this comment.
Preserve SDK_VERSION alias on default export
This change removes SDK_VERSION from the public default export and replaces it with VERSION, which introduces a runtime compatibility break for existing integrations that read sdk.SDK_VERSION (for diagnostics, telemetry, or version pinning). If the goal is package renaming, keeping SDK_VERSION as an alias to VERSION would avoid forcing immediate consumer code changes.
Useful? React with 👍 / 👎.
| MediaEventType, | ||
| } from './types'; | ||
| export type { AnalyticsConfig } from './config/types'; | ||
| export type { TanilyticsConfig } from './config/types'; |
There was a problem hiding this comment.
Re-export AnalyticsConfig as a compatibility alias
The public type export was renamed from AnalyticsConfig to TanilyticsConfig, which causes downstream TypeScript builds to fail for consumers that import AnalyticsConfig from the package. Since this commit is primarily a package rename, adding a temporary alias export (for example type AnalyticsConfig = TanilyticsConfig) would prevent a hard compile-time break during migration.
Useful? React with 👍 / 👎.



Summary by CodeRabbit
Refactor
@analytics-sdk/*totanilyticsand@tanilytics/*package namesDocumentation