feat: accept simulation as runner mode

Author: GuillaumeLagrange

src/run/config.rs

@@ -72,6 +72,14 @@ impl TryFrom<RunArgs> for Config {
         let upload_url = Url::parse(&raw_upload_url)
             .map_err(|e| anyhow!("Invalid upload URL: {raw_upload_url}, {e}"))?;
 
+        // Emit deprecation warning if using "instrumentation" mode
+        if args.mode == RunnerMode::Instrumentation {
+            warn!(
+                "The 'instrumentation' runner mode is deprecated and will be removed in a future version. \
+                Please use 'simulation' instead."
+            );
+        }
+
         Ok(Self {
             upload_url,
             token: args.token,

src/run/mod.rs

@@ -138,6 +138,7 @@ pub struct RunArgs {
 #[serde(rename_all = "lowercase")]
 pub enum RunnerMode {
     Instrumentation,
+    Simulation,
     Walltime,
 }
 

src/run/runner/helpers/env.rs

@@ -6,6 +6,14 @@ pub fn get_base_injected_env(
     mode: RunnerMode,
     profile_folder: &Path,
 ) -> HashMap<&'static str, String> {
+    let runner_mode_internal_env_value = match mode {
+        RunnerMode::Instrumentation => "instrumentation",
+        // While the runner now deprecates the usage of instrumentation with a message, we
+        // internally still use instrumentation temporarily to give time to users to upgrade their
+        // integrations to a version that accepts both instrumentation and simulation.
+        RunnerMode::Simulation => "instrumentation",
+        RunnerMode::Walltime => "walltime",
+    };
     HashMap::from([
         ("PYTHONHASHSEED", "0".into()),
         (
@@ -18,7 +26,10 @@ pub fn get_base_injected_env(
         ),
         ("ARCH", ARCH.into()),
         ("CODSPEED_ENV", "runner".into()),
-        ("CODSPEED_RUNNER_MODE", mode.to_string()),
+        (
+            "CODSPEED_RUNNER_MODE",
+            runner_mode_internal_env_value.into(),
+        ),
         (
             "CODSPEED_PROFILE_FOLDER",
             profile_folder.to_string_lossy().to_string(),

src/run/runner/mod.rs

@@ -22,6 +22,7 @@ impl Display for RunnerMode {
     fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
         match self {
             RunnerMode::Instrumentation => write!(f, "instrumentation"),
+            RunnerMode::Simulation => write!(f, "simulation"),
             RunnerMode::Walltime => write!(f, "walltime"),
         }
     }
@@ -32,6 +33,7 @@ pub const EXECUTOR_TARGET: &str = "executor";
 pub fn get_executor_from_mode(mode: &RunnerMode) -> Box<dyn Executor> {
     match mode {
         RunnerMode::Instrumentation => Box::new(ValgrindExecutor),
+        RunnerMode::Simulation => Box::new(ValgrindExecutor),
         RunnerMode::Walltime => Box::new(WallTimeExecutor::new()),
     }
 }