Avoid spamming telemetry with repeated reactdevtools.node_not_found…

… events (#10541)

* Avoid spamming telemetry with repeated `reactdevtools.node_not_found` events

* rename to distinctOn

src/ui/components/SecondaryToolbox/react-devtools/ReplayWall.ts

@@ -168,7 +168,9 @@ export class ReplayWall implements Wall {
           // Get the first node ID we found for this fiber ID, if available.
           const [nodeId] = fiberIdsToNodeIds.get(fiberId) ?? [];
           if (!nodeId) {
-            sendTelemetryEvent("reactdevtools.node_not_found", payload);
+            sendTelemetryEvent("reactdevtools.node_not_found", payload, {
+              distinctOn: fiberId,
+            });
             return;
           }
 

src/ui/utils/telemetry.ts

@@ -112,11 +112,30 @@ export function setTelemetryContext({ id, email, internal }: TelemetryUser) {
   }
 }
 
-export function sendTelemetryEvent(event: string, tags: any = {}) {
+const sentEvents = new Set<string>();
+
+type TelemetryOptions = {
+  /** It can be used to avoid sending repeated events. It's always combined with `event` for that purpose */
+  distinctOn?: string;
+};
+
+export function sendTelemetryEvent(
+  event: string,
+  tags: any = {},
+  { distinctOn }: TelemetryOptions = {}
+) {
   if (userData.get("global_logTelemetryEvent")) {
     console.log("telemetry event", { event, tags });
   }
 
+  if (distinctOn) {
+    const key = `${event}:${distinctOn}`;
+    if (sentEvents.has(key)) {
+      return;
+    }
+    sentEvents.add(key);
+  }
+
   if (skipTelemetry()) {
     return;
   }