refactor(analytics): migrate MetaMetrics events to AnalyticsController#43885
refactor(analytics): migrate MetaMetrics events to AnalyticsController#43885gauthierpetetin wants to merge 2 commits into
Conversation
|
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
✨ Files requiring CODEOWNER review ✨🔑 @MetaMask/accounts-engineers (8 files, +194 -182)
👨🔧 @MetaMask/core-extension-ux (10 files, +224 -210)
🫰 @MetaMask/core-platform (4 files, +0 -12)
👨🔧 @MetaMask/extension-platform (1 files, +0 -6)
👨🔧 @MetaMask/product-safety (1 files, +11 -12)
🔐 @MetaMask/web3auth (1 files, +37 -29)
|
Builds ready [7dc51a4]
⚡ Performance Benchmarks (Total: 🟢 20 pass · 🟡 4 warn · 🔴 0 fail)
Bundle size diffs [🚀 Bundle size reduced!]
|
Part of analytics migration umbrella PR #43885.
Part of analytics migration umbrella PR #43885.
Part of analytics migration umbrella PR #43885.
Part of analytics migration umbrella PR #43885.
Part of analytics migration umbrella PR #43885.
Part of analytics migration umbrella PR #43885.
Part of analytics migration umbrella PR #43885.
Part of analytics migration umbrella PR #43885.
Part of analytics migration umbrella PR #43885.
Part of analytics migration umbrella PR #43885.
Part of analytics migration umbrella PR #43885.
Part of analytics migration umbrella PR #43885.
Part of analytics migration umbrella PR #43885.
Part of analytics migration umbrella PR #43885.
Part of analytics migration umbrella PR #43885.
Part of analytics migration umbrella PR #43885.
Part of analytics migration umbrella PR #43885.
Part of analytics migration umbrella PR #43885.
Part of analytics migration umbrella PR #43885.
Part of analytics migration umbrella PR #43885.
Builds ready [0b2fcc1]
⚡ Performance Benchmarks (Total: 🟢 16 pass · 🟡 8 warn · 🔴 0 fail)
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
Builds ready [646fe2c]
⚡ Performance Benchmarks (Total: 🟢 15 pass · 🟡 9 warn · 🔴 0 fail)
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
|
Builds ready [fd68a24]
⚡ Performance Benchmarks (Total: 🟢 16 pass · 🟡 9 warn · 🔴 0 fail)
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
… migration Remove trackEvent, identify, and trackPage from MetaMetricsController and route remaining background and UI call sites through the analytics module and useAnalytics hook. Adds trackMetaMetricsPayload helper for legacy payload conversion and migrates the last MetaMetricsContext trackEvent consumers. Co-authored-by: Cursor <cursoragent@cursor.com>
Builds ready [2ce3041]
⚡ Performance Benchmarks (Total: 🟢 17 pass · 🟡 6 warn · 🔴 1 fail)
Bundle sizes
|
…aries Remove trackMetaMetricsPayload from the analytics module. Legacy MetaMetricsEventPayload conversion now lives in MetaMetricsController, token-detection init, and the metamask-controller UI RPC handler, matching the smart-transactions init pattern. Co-authored-by: Cursor <cursoragent@cursor.com>
Builds ready [1d9f894]
⚡ Performance Benchmarks (Total: 🟢 16 pass · 🟡 8 warn · 🔴 0 fail)
Bundle sizes
|


Umbrella tracker: analytics event migration
This PR tracks the full analytics migration. Do not merge as-is — land the sub-PRs below, then close this umbrella.
Merge order
mainvia #43731 (useAnalytics,trackAnalyticsEvent,AnalyticsController)trackAnalyticsPageRPC wiring andMetaMetricsProvidermigrationMetaMetricsControllerpublic shims, supersedes stale PR14)Sub-PR checklist
Description (full migration reference)
See sub-PRs above. Full diff is preserved on branch
refactor/analytics-migrate-events.Changelog
CHANGELOG entry: null
Related issues
Fixes:
Manual testing steps
Follow manual testing in each sub-PR after it lands.
Pre-merge author checklist
Pre-merge reviewer checklist