improve-mix-panel-fcp and its various impors across the components an…

…d pages

app/Analytics.tsx

@@ -1,20 +1,20 @@
 'use client';
 
 import { usePathname } from 'lib/i18n/navigation';
-import { init, track } from 'lib/utils/analytics';
+import { analytics } from 'lib/utils/analytics';
 import Script from 'next/script';
 import { useEffect } from 'react';
 
 const Analytics = () => {
   const path = usePathname();
 
   useEffect(() => {
-    init();
+    analytics.init();
   }, []);
 
   useEffect(() => {
     if (!path) return;
-    track('Viewed Page', { path });
+    analytics.track('Viewed Page', { path });
   }, [path]);
 
   // SimpleAnalytics

components/allowances/controls/batch-revoke/BatchRevokeControls.tsx

@@ -3,7 +3,7 @@ import TipSection from 'components/common/donate/TipSection';
 import { useDonate } from 'lib/hooks/ethereum/useDonate';
 import { useAddressPageContext } from 'lib/hooks/page-context/AddressPageContext';
 import { TokenAllowanceData } from 'lib/utils/allowances';
-import { track } from 'lib/utils/analytics';
+import { analytics } from 'lib/utils/analytics';
 import { useTranslations } from 'next-intl';
 import { useState } from 'react';
 import ControlsWrapper from '../ControlsWrapper';
@@ -33,7 +33,7 @@ const BatchRevokeControls = ({ selectedAllowances, isRevoking, isAllConfirmed, s
       return 'mid';
     };
 
-    track('Batch Revoked', {
+    analytics.track('Batch Revoked', {
       chainId: selectedChainId,
       address,
       allowances: selectedAllowances.length,

components/exploits/ExploitChecker.tsx

@@ -10,7 +10,7 @@ import {
 } from 'lib/hooks/page-context/AddressPageContext';
 import { isNullish } from 'lib/utils';
 import { getAllowanceKey } from 'lib/utils/allowances';
-import { track } from 'lib/utils/analytics';
+import { analytics } from 'lib/utils/analytics';
 import { getEventKey } from 'lib/utils/events';
 import { Exploit, getExploitStatus } from 'lib/utils/exploits';
 import ExploitStatus from './ExploitStatus';
@@ -29,7 +29,7 @@ const ExploitChecker = ({ exploit }: Props) => {
     queryKey: ['exploit-status', exploit.slug, allowances?.map(getAllowanceKey), events?.map(getEventKey)],
     queryFn: () => {
       const status = getExploitStatus(events!, allowances!, exploit);
-      track('Exploit Checked', { exploit: exploit.slug, account: address, chainId: selectedChainId, status });
+      analytics.track('Exploit Checked', { exploit: exploit.slug, account: address, chainId: selectedChainId, status });
       return status;
     },
     enabled: !isNullish(address) && !isNullish(events) && !isNullish(allowances) && !isNullish(selectedChainId),

components/footer/ColorThemeSelect.tsx

@@ -4,7 +4,7 @@ import { ComputerDesktopIcon, MoonIcon, SunIcon } from '@heroicons/react/24/outl
 import Select from 'components/common/select/Select';
 import { useColorTheme } from 'lib/hooks/useColorTheme';
 import { useMounted } from 'lib/hooks/useMounted';
-import { track } from 'lib/utils/analytics';
+import { analytics } from 'lib/utils/analytics';
 import { useTranslations } from 'next-intl';
 
 const ColorThemeSelect = () => {
@@ -19,7 +19,7 @@ const ColorThemeSelect = () => {
   ] as const;
 
   const selectTheme = (option: (typeof options)[number]) => {
-    track('Changed Color Theme', { theme: option.value });
+    analytics.track('Changed Color Theme', { theme: option.value });
     setTheme(option.value);
   };
 

components/footer/LanguageSelect.tsx

@@ -5,7 +5,7 @@ import Select from 'components/common/select/Select';
 import { Locale } from 'lib/i18n/config';
 import { useCsrRouter } from 'lib/i18n/csr-navigation';
 import { usePathname } from 'lib/i18n/navigation';
-import { track } from 'lib/utils/analytics';
+import { analytics } from 'lib/utils/analytics';
 import { useLocale } from 'next-intl';
 import { FormatOptionLabelMeta } from 'react-select';
 
@@ -36,7 +36,7 @@ const LanguageSelect = () => {
 
   const selectLanguage = (option: Option) => {
     const newLocale = option.value;
-    track('Changed language', { from: locale, to: newLocale });
+    analytics.track('Changed language', { from: locale, to: newLocale });
     router.replace(path, { locale: newLocale, scroll: false, showProgress: false, retainSearchParams: ['chainId'] });
     persistLocaleCookie(newLocale);
   };

components/signatures/cells/CancelMarketplaceCell.tsx

@@ -3,7 +3,7 @@ import { useHandleTransaction } from 'lib/hooks/ethereum/useHandleTransaction';
 import { useAddressPageContext } from 'lib/hooks/page-context/AddressPageContext';
 import { Marketplace, OnCancel, TransactionSubmitted, TransactionType } from 'lib/interfaces';
 import { waitForTransactionConfirmation } from 'lib/utils';
-import { track } from 'lib/utils/analytics';
+import { analytics } from 'lib/utils/analytics';
 import { usePublicClient, useWalletClient } from 'wagmi';
 import CancelCell from './CancelCell';
 
@@ -21,7 +21,7 @@ const CancelMarketplaceCell = ({ marketplace, onCancel }: Props) => {
   const sendCancelTransaction = async (): Promise<TransactionSubmitted> => {
     const hash = await marketplace?.cancelSignatures(walletClient!);
 
-    track('Cancelled Marketplace Signatures', {
+    analytics.track('Cancelled Marketplace Signatures', {
       chainId: selectedChainId,
       account: address,
       marketplace: marketplace.name,

components/signatures/cells/CancelPermitCell.tsx

@@ -4,7 +4,7 @@ import { useHandleTransaction } from 'lib/hooks/ethereum/useHandleTransaction';
 import { useAddressPageContext } from 'lib/hooks/page-context/AddressPageContext';
 import { OnCancel, TransactionSubmitted, TransactionType } from 'lib/interfaces';
 import { waitForTransactionConfirmation } from 'lib/utils';
-import { track } from 'lib/utils/analytics';
+import { analytics } from 'lib/utils/analytics';
 import { permit } from 'lib/utils/permit';
 import { isErc721Contract, PermitTokenData } from 'lib/utils/tokens';
 import { usePublicClient, useWalletClient } from 'wagmi';
@@ -25,7 +25,11 @@ const CancelPermitCell = ({ token, onCancel }: Props) => {
     if (isErc721Contract(token.contract)) return;
     const hash = await permit(walletClient!, token.contract, DUMMY_ADDRESS, 0n);
 
-    track('Cancelled Permit Signatures', { chainId: selectedChainId, account: address, token: token.contract.address });
+    analytics.track('Cancelled Permit Signatures', {
+      chainId: selectedChainId,
+      account: address,
+      token: token.contract.address,
+    });
 
     const waitForConfirmation = async () => {
       // TODO: Deduplicate this with the CancelMarketplaceCell

lib/hooks/ethereum/useAllowances.tsx

@@ -9,7 +9,7 @@ import {
   stripAllowanceData,
   type TokenAllowanceData,
 } from 'lib/utils/allowances';
-import { track } from 'lib/utils/analytics';
+import { analytics } from 'lib/utils/analytics';
 import { getEventKey, TimeLog, TokenEvent } from 'lib/utils/events';
 import { hasZeroBalance } from 'lib/utils/tokens';
 import { useLayoutEffect, useState } from 'react';
@@ -30,7 +30,7 @@ export const useAllowances = (address: Address, events: TokenEvent[] | undefined
     queryKey: ['allowances', address, chainId, events?.map(getEventKey)],
     queryFn: async () => {
       const allowances = getAllowancesFromEvents(address, events!, publicClient, chainId);
-      track('Fetched Allowances', { account: address, chainId });
+      analytics.track('Fetched Allowances', { account: address, chainId });
       return allowances;
     },
     // If events (transfers + approvals) don't change, derived allowances also shouldn't change, even if allowances

lib/hooks/ethereum/useDonate.tsx

@@ -4,7 +4,7 @@ import { DonateButtonType } from 'components/common/donate/DonateModal';
 import { DONATION_ADDRESS } from 'lib/constants';
 import { TransactionSubmitted, TransactionType } from 'lib/interfaces';
 import { getWalletAddress, waitForTransactionConfirmation } from 'lib/utils';
-import { track } from 'lib/utils/analytics';
+import { analytics } from 'lib/utils/analytics';
 import { getChainName, getChainNativeToken, getDefaultDonationAmount } from 'lib/utils/chains';
 import { parseEther } from 'viem';
 import { usePublicClient, useWalletClient } from 'wagmi';
@@ -39,7 +39,7 @@ export const useDonate = (chainId: number, type: DonateButtonType) => {
     const transactionSubmitted = await handleTransaction(sendDonation(amount), TransactionType.DONATE);
 
     if (transactionSubmitted) {
-      track('Donated', {
+      analytics.track('Donated', {
         chainId,
         chainName: getChainName(chainId),
         nativeToken,

lib/utils/allowances.ts

@@ -6,7 +6,7 @@ import { type TransactionSubmitted, TransactionType } from 'lib/interfaces';
 import { TransactionStore } from 'lib/stores/transaction-store';
 import { type Address, formatUnits, type PublicClient, type WalletClient, type WriteContractParameters } from 'viem';
 import { deduplicateArray, isNullish, waitForTransactionConfirmation, writeContractUnlessExcessiveGas } from '.';
-import { track } from './analytics';
+import { analytics } from './analytics';
 import { isNetworkError, isRevertedError, isUserRejectionError, parseErrorMessage, stringifyError } from './errors';
 import {
   Erc20ApprovalEvent,
@@ -510,7 +510,7 @@ const trackTransaction = (allowance: TokenAllowanceData, hash: string, newAmount
   if (!hash) return;
 
   if (isErc721Contract(allowance.contract)) {
-    track('Revoked ERC721 allowance', {
+    analytics.track('Revoked ERC721 allowance', {
       chainId: allowance.chainId,
       account: allowance.owner,
       spender: allowance.payload?.spender,
@@ -519,7 +519,7 @@ const trackTransaction = (allowance: TokenAllowanceData, hash: string, newAmount
     });
   }
 
-  track(newAmount === '0' ? 'Revoked ERC20 allowance' : 'Updated ERC20 allowance', {
+  analytics.track(newAmount === '0' ? 'Revoked ERC20 allowance' : 'Updated ERC20 allowance', {
     chainId: allowance.chainId,
     account: allowance.owner,
     spender: allowance.payload?.spender,

lib/utils/analytics.ts

@@ -1,15 +1,24 @@
 import mixpanel from 'mixpanel-browser';
 
-export const init = () => {
-  if (process.env.NEXT_PUBLIC_MIXPANEL_API_KEY) {
-    mixpanel.init(process.env.NEXT_PUBLIC_MIXPANEL_API_KEY, { ip: false });
-  }
-};
+export const analytics = {
+  isInitialized: false,
+  // init only when first used
+  init() {
+    if (this.isInitialized) return;
+    const apiKey = process.env.NEXT_PUBLIC_MIXPANEL_API_KEY;
+    if (apiKey && typeof window !== 'undefined') {
+      mixpanel.init(apiKey, { ip: false });
+      this.isInitialized = true;
+    }
+  },
 
-export const track = (eventName: string, eventProperties: any) => {
-  if (typeof window === 'undefined') return;
+  track(eventName: string, eventProperties?: Record<string, any>) {
+    if (typeof window === 'undefined' || !process.env.NEXT_PUBLIC_MIXPANEL_API_KEY) return;
+    // lazy initialize if not already done
+    if (!this.isInitialized) {
+      this.init();
+    }
 
-  if (process.env.NEXT_PUBLIC_MIXPANEL_API_KEY) {
     mixpanel.track(eventName, eventProperties);
-  }
+  },
 };

lib/utils/index.ts

@@ -15,7 +15,7 @@ import {
   WalletClient,
   WriteContractParameters,
 } from 'viem';
-import { track } from './analytics';
+import { analytics } from './analytics';
 import type { Log, TokenEvent } from './events';
 
 export const assertFulfilled = <T>(item: PromiseSettledResult<T>): item is PromiseFulfilledResult<T> => {
@@ -123,7 +123,7 @@ export const throwIfExcessiveGas = (chainId: number, address: Address, estimated
 
     // Track excessive gas usage so we can blacklist tokens
     // TODO: Use a different tool than analytics for this
-    track('Excessive gas limit', { chainId, address, estimatedGas: estimatedGas.toString() });
+    analytics.track('Excessive gas limit', { chainId, address, estimatedGas: estimatedGas.toString() });
 
     throw new Error(
       'This transaction has an excessive gas cost. It is most likely a spam token, so you do not need to revoke this approval.',

lib/utils/risk.tsx

@@ -1,6 +1,6 @@
 import { ExclamationCircleIcon, ExclamationTriangleIcon, InformationCircleIcon } from '@heroicons/react/24/solid';
 import { RiskFactor, RiskLevel } from 'lib/interfaces';
-import { track } from './analytics';
+import { analytics } from './analytics';
 
 export const RiskFactorScore: Record<string, number> = {
   blocklist: 100,
@@ -15,7 +15,7 @@ export const RiskFactorScore: Record<string, number> = {
 export const filterUnknownRiskFactors = (riskFactors: RiskFactor[]): RiskFactor[] => {
   return riskFactors.filter((riskFactor) => {
     if (RiskFactorScore[riskFactor.type] === undefined) {
-      track('Unknown Risk Factor', riskFactor);
+      analytics.track('Unknown Risk Factor', riskFactor);
       return false;
     }
 

lib/utils/tokens.ts

@@ -5,7 +5,7 @@ import ky from 'lib/ky';
 import { getTokenPrice } from 'lib/price/utils';
 import { Address, domainSeparator, getAbiItem, getAddress, pad, PublicClient, toHex, TypedDataDomain } from 'viem';
 import { deduplicateArray } from '.';
-import { track } from './analytics';
+import { analytics } from './analytics';
 import { isTransferTokenEvent, type TimeLog, type TokenEvent, TokenEventType } from './events';
 import { formatFixedPointBigInt } from './formatting';
 import { withFallback } from './promises';
@@ -335,7 +335,7 @@ export const getPermitDomain = async (contract: Erc20TokenContract): Promise<Typ
 
   if (!domain) {
     // If the domain separator is something else, we cannot generate a valid signature
-    track('Permit Domain Separator Mismatch', { name, verifyingContract, chainId });
+    analytics.track('Permit Domain Separator Mismatch', { name, verifyingContract, chainId });
     throw new Error('Could not determine Permit Signature data');
   }