Move getMetadataForLogger

src/analytics/index.tsx

@@ -16,13 +16,13 @@ import {
   useSessionId,
 } from '#/analytics/identifiers'
 import {
+  getMetadataForLogger,
   getNavigationMetadata,
   type MergeableMetadata,
   type Metadata,
 } from '#/analytics/metadata'
 import {type Metrics, metrics} from '#/analytics/metrics'
 import * as refParams from '#/analytics/misc/refParams'
-import {getMetadataForLogger} from '#/analytics/utils'
 import * as env from '#/env'
 import {useGeolocation} from '#/geolocation'
 import {device} from '#/storage'

src/analytics/metadata.ts

@@ -58,3 +58,22 @@ export function getNavigationMetadata() {
 export function setNavigationMetadata(meta: NavigationMetadata | undefined) {
   navigationMetadata = meta
 }
+
+/**
+ * We don't want or need to send all data to the logger
+ */
+export function getMetadataForLogger({
+  base,
+  geolocation,
+  session,
+}: Metadata): Record<string, any> {
+  return {
+    deviceId: base.deviceId,
+    sessionId: base.sessionId,
+    platform: base.platform,
+    appVersion: base.appVersion,
+    countryCode: geolocation.countryCode,
+    regionCode: geolocation.regionCode,
+    isBskyPds: session?.isBskyPds || 'anonymous',
+  }
+}

src/analytics/utils.ts

@@ -4,7 +4,6 @@ import {BSKY_SERVICE} from '#/lib/constants'
 import {type SessionAccount} from '#/state/session'
 import {
   type MergeableMetadata,
-  type Metadata,
   type SessionMetadata,
 } from '#/analytics/metadata'
 
@@ -32,19 +31,3 @@ export function accountToSessionMetadata(
     }
   }
 }
-
-export function getMetadataForLogger({
-  base,
-  geolocation,
-  session,
-}: Metadata): Record<string, any> {
-  return {
-    deviceId: base.deviceId,
-    sessionId: base.sessionId,
-    platform: base.platform,
-    appVersion: base.appVersion,
-    countryCode: geolocation.countryCode,
-    regionCode: geolocation.regionCode,
-    isBskyPds: session?.isBskyPds || 'anonymous',
-  }
-}