feat(telemetry): split frontend setup from build duration in export event

Time `build.setup_frontend` separately from `build.build` and skip
additional keys already present in defaults so callers can't overwrite
them.

Author: FarhanAliRaza

reflex/utils/export.py

@@ -80,8 +80,9 @@ def _time_phase(name: str) -> Iterator[None]:
         if frontend:
             with _time_phase("compile_duration"):
                 prerequisites.get_compiled_app(prerender_routes=prerender_routes)
-            with _time_phase("build_duration"):
+            with _time_phase("setup_duration"):
                 build.setup_frontend(Path.cwd())
+            with _time_phase("build_duration"):
                 build.build()
         if zipping:
             with _time_phase("zip_duration"):
@@ -103,6 +104,7 @@ def _time_phase(name: str) -> Iterator[None]:
             detail=detail,
             duration=time.monotonic() - start,
             compile_duration=phase_durations.get("compile_duration"),
+            setup_duration=phase_durations.get("setup_duration"),
             build_duration=phase_durations.get("build_duration"),
             zip_duration=phase_durations.get("zip_duration"),
         )

reflex/utils/telemetry.py

@@ -293,6 +293,7 @@ def _prepare_event(event: str, **kwargs) -> _Event | None:
         "status",
         "duration",
         "compile_duration",
+        "setup_duration",
         "build_duration",
         "zip_duration",
     ]
@@ -301,6 +302,8 @@ def _prepare_event(event: str, **kwargs) -> _Event | None:
     properties = event_data["properties"].copy()
 
     for key in additional_keys:
+        if key in properties:
+            continue
         if key in kwargs and kwargs[key] is not None:
             properties[key] = kwargs[key]