add appId to Telemetry (facebook#47099)

Summary: Pull Request resolved: facebook#47099 Changelog: [Internal] As discussed in D64547367, recent changes caused `description` to stop reporting `appId`. This impacted Telemetry's "top apps" dashboards. {F1936011487} Reviewed By: hoxyq Differential Revision: D64548348 fbshipit-source-id: 46af650a89e2bef7be000482e7b774f56f84cac3
discord · Oct 17, 2024 · d11954e · d11954e
1 parent 0d591d9
commit d11954e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/packages/dev-middleware/src/middleware/openDebuggerMiddleware.js b/packages/dev-middleware/src/middleware/openDebuggerMiddleware.js
@@ -156,6 +156,7 @@ export default function openDebuggerMiddleware({
           appId: appId ?? null,
           deviceId: device ?? null,
           resolvedTargetDescription: target.description,
+          resolvedTargetAppId: target.appId,
           prefersFuseboxFrontend: useFuseboxEntryPoint ?? false,
         });
         return;

diff --git a/packages/dev-middleware/src/types/EventReporter.js b/packages/dev-middleware/src/types/EventReporter.js
@@ -41,6 +41,7 @@ export type ReportableEvent =
             appId: string | null,
             deviceId: string | null,
             resolvedTargetDescription: string,
+            resolvedTargetAppId: string,
             prefersFuseboxFrontend: boolean,
           }>
         | ErrorResult<mixed>