Merge pull request #50881 from callstack-internal/feat/extended-perf-…

…analytics-metadata

Extend perf analytics metadata

src/libs/Firebase/index.native.ts

@@ -2,10 +2,8 @@
 import crashlytics from '@react-native-firebase/crashlytics';
 import perf from '@react-native-firebase/perf';
 import * as Environment from '@libs/Environment/Environment';
-import * as PersonalDetailsUtils from '@libs/PersonalDetailsUtils';
-import * as ReportConnection from '@libs/ReportConnection';
-import * as SessionUtils from '@libs/SessionUtils';
-import type {FirebaseAttributes, Log, StartTrace, StopTrace, TraceMap} from './types';
+import type {Log, StartTrace, StopTrace, TraceMap} from './types';
+import utils from './utils';
 
 const traceMap: TraceMap = {};
 
@@ -19,7 +17,7 @@ const startTrace: StartTrace = (customEventName) => {
         return;
     }
 
-    const attributes = getAttributes();
+    const attributes = utils.getAttributes();
 
     perf()
         .startTrace(customEventName)
@@ -59,20 +57,6 @@ const log: Log = (action: string) => {
     crashlytics().log(action);
 };
 
-function getAttributes(): FirebaseAttributes {
-    const session = SessionUtils.getSession();
-
-    const accountId = session?.accountID?.toString() ?? 'N/A';
-    const reportsLength = ReportConnection.getAllReportsLength().toString();
-    const personalDetailsLength = PersonalDetailsUtils.getPersonalDetailsLength().toString();
-
-    return {
-        accountId,
-        reportsLength,
-        personalDetailsLength,
-    };
-}
-
 export default {
     startTrace,
     stopTrace,

src/libs/Firebase/index.web.ts

@@ -1,10 +1,8 @@
 import {trace} from '@firebase/performance';
 import * as Environment from '@libs/Environment/Environment';
-import * as PersonalDetailsUtils from '@libs/PersonalDetailsUtils';
-import * as ReportConnection from '@libs/ReportConnection';
-import * as SessionUtils from '@libs/SessionUtils';
 import {firebasePerfWeb} from './firebaseWebConfig';
-import type {FirebaseAttributes, Log, StartTrace, StopTrace, TraceMap} from './types';
+import type {Log, StartTrace, StopTrace, TraceMap} from './types';
+import utils from './utils';
 
 const traceMap: TraceMap = {};
 
@@ -21,7 +19,7 @@ const startTrace: StartTrace = (customEventName) => {
 
     const perfTrace = trace(firebasePerfWeb, customEventName);
 
-    const attributes = getAttributes();
+    const attributes = utils.getAttributes();
 
     Object.entries(attributes).forEach(([name, value]) => {
         perfTrace.putAttribute(name, value);
@@ -55,20 +53,6 @@ const log: Log = () => {
     // crashlytics is not supported on WEB
 };
 
-function getAttributes(): FirebaseAttributes {
-    const session = SessionUtils.getSession();
-
-    const accountId = session?.accountID?.toString() ?? 'N/A';
-    const reportsLength = ReportConnection.getAllReportsLength().toString();
-    const personalDetailsLength = PersonalDetailsUtils.getPersonalDetailsLength().toString();
-
-    return {
-        accountId,
-        reportsLength,
-        personalDetailsLength,
-    };
-}
-
 export default {
     startTrace,
     stopTrace,

src/libs/Firebase/types.ts

@@ -13,6 +13,10 @@ type FirebaseAttributes = {
     accountId: string;
     personalDetailsLength: string;
     reportsLength: string;
+    reportActionsLength: string;
+    transactionViolationsLength: string;
+    policiesLength: string;
+    transactionsLength: string;
 };
 
 export type {StartTrace, StopTrace, TraceMap, Log, FirebaseAttributes};

src/libs/Firebase/utils.ts

@@ -0,0 +1,33 @@
+import {getAllTransactions, getAllTransactionViolationsLength} from '@libs/actions/Transaction';
+import * as PersonalDetailsUtils from '@libs/PersonalDetailsUtils';
+import {getAllPoliciesLength} from '@libs/PolicyUtils';
+import {getReportActionsLength} from '@libs/ReportActionsUtils';
+import * as ReportConnection from '@libs/ReportConnection';
+import * as SessionUtils from '@libs/SessionUtils';
+import type {FirebaseAttributes} from './types';
+
+function getAttributes(): FirebaseAttributes {
+    const session = SessionUtils.getSession();
+
+    const accountId = session?.accountID?.toString() ?? 'N/A';
+    const reportsLength = ReportConnection.getAllReportsLength().toString();
+    const reportActionsLength = getReportActionsLength().toString();
+    const personalDetailsLength = PersonalDetailsUtils.getPersonalDetailsLength().toString();
+    const transactionViolationsLength = getAllTransactionViolationsLength().toString();
+    const policiesLength = getAllPoliciesLength().toString();
+    const transactionsLength = getAllTransactions().toString();
+
+    return {
+        accountId,
+        reportsLength,
+        reportActionsLength,
+        personalDetailsLength,
+        transactionViolationsLength,
+        policiesLength,
+        transactionsLength,
+    };
+}
+
+export default {
+    getAttributes,
+};

src/libs/PolicyUtils.ts

@@ -1050,6 +1050,10 @@ function hasPolicyFeedsError(feeds: Record<string, CardFeedData>, feedToSkip?: s
     return Object.entries(feeds).filter(([feedName, feedData]) => feedName !== feedToSkip && !!feedData.errors).length > 0;
 }
 
+function getAllPoliciesLength() {
+    return Object.keys(allPolicies ?? {}).length;
+}
+
 export {
     canEditTaxRate,
     extractPolicyIDFromPath,
@@ -1165,6 +1169,7 @@ export {
     hasUnsupportedIntegration,
     getWorkflowApprovalsUnavailable,
     getNetSuiteImportCustomFieldLabel,
+    getAllPoliciesLength,
 };
 
 export type {MemberEmailsToAccountIDs};

src/libs/ReportActionsUtils.ts

@@ -1769,6 +1769,10 @@ function getCardIssuedMessage(reportAction: OnyxEntry<ReportAction>, shouldRende
     }
 }
 
+function getReportActionsLength() {
+    return Object.keys(allReportActions ?? {}).length;
+}
+
 export {
     doesReportHaveVisibleActions,
     extractLinksFromMessageHtml,
@@ -1878,6 +1882,7 @@ export {
     getCardIssuedMessage,
     getRemovedConnectionMessage,
     getActionableJoinRequestPendingReportAction,
+    getReportActionsLength,
 };
 
 export type {LastVisibleMessage};

src/libs/actions/Transaction.ts

@@ -489,6 +489,14 @@ function getRecentWaypoints() {
     return recentWaypoints;
 }
 
+function getAllTransactionViolationsLength() {
+    return allTransactionViolations.length;
+}
+
+function getAllTransactions() {
+    return Object.keys(allTransactions ?? {}).length;
+}
+
 export {
     addStop,
     createInitialWaypoints,
@@ -504,4 +512,6 @@ export {
     openDraftDistanceExpense,
     getRecentWaypoints,
     sanitizeRecentWaypoints,
+    getAllTransactionViolationsLength,
+    getAllTransactions,
 };