Move runner type from user property to event property

Signed-off-by: Wei-Chun, Chang <[email protected]>

recce/event/collector.py

@@ -6,12 +6,11 @@
 from contextlib import contextmanager
 from datetime import datetime
 from json import JSONDecodeError
-from typing import Union
 
 import portalocker
 import requests
 
-from recce import __version__, is_ci_env, get_runner
+from recce import __version__, is_ci_env
 
 
 class Collector:
@@ -24,7 +23,6 @@ def __init__(self):
         self._delete_threshold = 1000
         self._upload_threshold = 10
         self._is_ci: bool = is_ci_env()
-        self._runner: Union[str, None] = get_runner()
 
     def is_ready(self):
         if self._api_key is None or self._user_id is None:
@@ -61,7 +59,6 @@ def log_event(self, prop, event_type):
                 version=__version__,
                 python_version=python_version,
                 is_ci=self._is_ci,
-                runner=self._runner,
             ),
             event_properties=prop,
             platform=sys.platform,

recce/event/track.py

@@ -9,7 +9,7 @@
 from rich.console import Console
 from rich.markup import escape
 
-from recce import event
+from recce import event, get_runner
 from recce.core import load_context
 
 console = Console()
@@ -78,6 +78,7 @@ def invoke(self, ctx: Context) -> t.Any:
         finally:
             end_time = time.time()
             duration = end_time - start_time
+            runner = get_runner()
             command = ctx.command.name
             props = dict(
                 command=command,
@@ -86,6 +87,9 @@ def invoke(self, ctx: Context) -> t.Any:
                 duration=duration,
             )
 
+            if runner is not None:
+                props['runner_type'] = runner
+
             try:
                 recce_context = load_context()
             except Exception: