add appId to Telemetry (facebook#47099)

Summary: 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
EdmondChuiHW · Oct 17, 2024 · 8c8d54d · 8c8d54d
1 parent d992d40
commit 8c8d54d
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>