[APP-1782] Analytics migration (#9734)
* WIP * Clean up growthbook code, integrate into init and sessions * Move everything out of React * Add metrics client * Move to separate file * Shared metadata cache * Ensure we update metadata when session ID changes * Ensure userMetadata is cleared when logging out * WIP revamp * Integrate feature gates into analytics context * Clean up old code * Fix useMeta util * Some comments and cleanup * Add logger to base analytics context * Refactor current route handling * Rip out LogEvent from navigation * Update tracking endpoint * Migrate toClout * Clear out statsig client * Add todo, reset logger readme * Ope fix statsig noop * Refactor logging in feed-feedback, add debug logging to metrics client * Remove LogEvents alias for Metrics * Prefer root package export * Remove Metrics alias from logger * [APP-1782] Migrate to new analytics APIs (#9735) * Migrate logEvent to useAnalytics * Migrate logger.metric to useAnalytics * Migrate tricky spot, fix types * Migrate remaining tricky spot * Missed one * Remove metric() from logger * Migrate useGate to useAnalytics * Remove all other StatSig mentions * Update event payload * Update logger tests * Mock expo method * Fix session ID bug * Add session ID test * Add test for metrics client * Clarify intent * Clean up core analytics file * Clean up the call once utils * Fix TODO * Fix TODO * Fix TODO * Fix TODO * Fix TODO * Remove debug code * Fix navigation context * OK nav context is not working, todo * Checkpoint: works but feels hacky * Fix navigation context issue * Improve feature API * Improve metric logging * Update logger tests
This commit is contained in:
+38
-40
@@ -11,7 +11,6 @@ import {type AppBskyFeedDefs} from '@atproto/api'
|
||||
import throttle from 'lodash.throttle'
|
||||
|
||||
import {PROD_FEEDS, STAGING_FEEDS} from '#/lib/constants'
|
||||
import {Logger} from '#/logger'
|
||||
import {
|
||||
type FeedSourceFeedInfo,
|
||||
type FeedSourceInfo,
|
||||
@@ -22,6 +21,7 @@ import {
|
||||
type FeedPostSliceItem,
|
||||
} from '#/state/queries/post-feed'
|
||||
import {getItemsForFeedback} from '#/view/com/posts/PostFeed'
|
||||
import {useAnalytics} from '#/analytics'
|
||||
import {useAgent} from './session'
|
||||
|
||||
export const FEEDBACK_FEEDS = [...PROD_FEEDS, ...STAGING_FEEDS]
|
||||
@@ -42,8 +42,6 @@ export const THIRD_PARTY_ALLOWED_INTERACTIONS = new Set<
|
||||
'app.bsky.feed.defs#interactionSeen',
|
||||
])
|
||||
|
||||
const logger = Logger.create(Logger.Context.FeedFeedback)
|
||||
|
||||
export type StateContext = {
|
||||
enabled: boolean
|
||||
onItemSeen: (item: any) => void
|
||||
@@ -65,6 +63,8 @@ export function useFeedFeedback(
|
||||
feedSourceInfo: FeedSourceInfo | undefined,
|
||||
hasSession: boolean,
|
||||
) {
|
||||
const ax = useAnalytics()
|
||||
const logger = ax.logger.useChild(ax.logger.Context.FeedFeedback)
|
||||
const agent = useAgent()
|
||||
|
||||
const feed =
|
||||
@@ -85,12 +85,45 @@ export function useFeedFeedback(
|
||||
WeakSet<FeedPostSliceItem | AppBskyFeedDefs.Interaction>
|
||||
>(new WeakSet())
|
||||
|
||||
const flushEvents = useCallback(
|
||||
(stats: AggregatedStats | null, feedDescriptor: string) => {
|
||||
if (stats === null) {
|
||||
return
|
||||
}
|
||||
|
||||
if (stats.clickthroughCount > 0) {
|
||||
ax.metric('feed:clickthrough', {
|
||||
count: stats.clickthroughCount,
|
||||
feed: feedDescriptor,
|
||||
})
|
||||
stats.clickthroughCount = 0
|
||||
}
|
||||
|
||||
if (stats.engagedCount > 0) {
|
||||
ax.metric('feed:engaged', {
|
||||
count: stats.engagedCount,
|
||||
feed: feedDescriptor,
|
||||
})
|
||||
stats.engagedCount = 0
|
||||
}
|
||||
|
||||
if (stats.seenCount > 0) {
|
||||
ax.metric('feed:seen', {
|
||||
count: stats.seenCount,
|
||||
feed: feedDescriptor,
|
||||
})
|
||||
stats.seenCount = 0
|
||||
}
|
||||
},
|
||||
[ax],
|
||||
)
|
||||
|
||||
const aggregatedStats = useRef<AggregatedStats | null>(null)
|
||||
const throttledFlushAggregatedStats = useMemo(
|
||||
() =>
|
||||
throttle(
|
||||
() =>
|
||||
flushToStatsig(
|
||||
flushEvents(
|
||||
aggregatedStats.current,
|
||||
feed?.feedDescriptor ?? 'unknown',
|
||||
),
|
||||
@@ -100,7 +133,7 @@ export function useFeedFeedback(
|
||||
trailing: true,
|
||||
},
|
||||
),
|
||||
[feed?.feedDescriptor],
|
||||
[feed?.feedDescriptor, flushEvents],
|
||||
)
|
||||
|
||||
const sendToFeedNoDelay = useCallback(() => {
|
||||
@@ -130,7 +163,6 @@ export function useFeedFeedback(
|
||||
)
|
||||
.catch(() => {}) // ignore upstream errors
|
||||
|
||||
// Send to Statsig
|
||||
if (aggregatedStats.current === null) {
|
||||
aggregatedStats.current = createAggregatedStats()
|
||||
}
|
||||
@@ -299,37 +331,3 @@ function sendOrAggregateInteractionsForStats(
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
function flushToStatsig(stats: AggregatedStats | null, feedDescriptor: string) {
|
||||
if (stats === null) {
|
||||
return
|
||||
}
|
||||
|
||||
if (stats.clickthroughCount > 0) {
|
||||
logger.metric('feed:clickthrough', {
|
||||
count: stats.clickthroughCount,
|
||||
feed: feedDescriptor,
|
||||
})
|
||||
stats.clickthroughCount = 0
|
||||
}
|
||||
|
||||
if (stats.engagedCount > 0) {
|
||||
logger.metric('feed:engaged', {
|
||||
count: stats.engagedCount,
|
||||
feed: feedDescriptor,
|
||||
})
|
||||
stats.engagedCount = 0
|
||||
}
|
||||
|
||||
if (stats.seenCount > 0) {
|
||||
logger.metric(
|
||||
'feed:seen',
|
||||
{
|
||||
count: stats.seenCount,
|
||||
feed: feedDescriptor,
|
||||
},
|
||||
{statsig: false},
|
||||
)
|
||||
stats.seenCount = 0
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user