refactor: move EXEC_HARNESS_VERSION and EXEC_HARNESS_COMMAND to orchestrator mod

Author: GuillaumeLagrange

CONTRIBUTING.md

@@ -41,7 +41,7 @@ After releasing `memtrack` or `exec-harness`, you **must** update the version re
    const MEMTRACK_CODSPEED_VERSION: &str = "X.Y.Z"; // Update to new version
    ```
 
-2. **For exec-harness**: Update `EXEC_HARNESS_VERSION` in `src/exec/mod.rs`:
+2. **For exec-harness**: Update `EXEC_HARNESS_VERSION` in `src/executor/orchestrator.rs`:
    ```rust
    const EXEC_HARNESS_VERSION: &str = "X.Y.Z"; // Update to new version
    ```
@@ -57,7 +57,7 @@ The main runner (`codspeed-runner`) should be released after ensuring all depend
 **Verify binary version references**: Check that version constants in the runner code match the released versions:
 
 - `MEMTRACK_CODSPEED_VERSION` in `src/executor/memory/executor.rs`
-- `EXEC_HARNESS_VERSION` in `src/exec/mod.rs`
+- `EXEC_HARNESS_VERSION` in `src/executor/orchestrator.rs`
 
 #### Release Command
 

src/cli/exec/mod.rs

@@ -18,9 +18,6 @@ pub mod multi_targets;
 /// We temporarily force this name for all exec runs
 pub const DEFAULT_REPOSITORY_NAME: &str = "local-runs";
 
-pub const EXEC_HARNESS_COMMAND: &str = "exec-harness";
-pub const EXEC_HARNESS_VERSION: &str = "1.2.0";
-
 #[derive(Args, Debug)]
 pub struct ExecArgs {
     #[command(flatten)]

src/cli/exec/multi_targets.rs

@@ -1,5 +1,5 @@
-use super::EXEC_HARNESS_COMMAND;
 use crate::executor::config::BenchmarkTarget;
+use crate::executor::orchestrator::EXEC_HARNESS_COMMAND;
 use crate::prelude::*;
 use crate::project_config::{Target, TargetCommand, WalltimeOptions};
 use exec_harness::BenchmarkCommand;

src/executor/orchestrator.rs

@@ -1,7 +1,7 @@
 use super::{ExecutionContext, ExecutorName, get_executor_from_mode, run_executor};
 use crate::api_client::CodSpeedAPIClient;
 use crate::binary_installer::ensure_binary_installed;
-use crate::cli::exec::{EXEC_HARNESS_COMMAND, EXEC_HARNESS_VERSION, multi_targets};
+use crate::cli::exec::multi_targets;
 use crate::cli::run::logger::Logger;
 use crate::config::CodSpeedConfig;
 use crate::executor::config::BenchmarkTarget;
@@ -17,6 +17,9 @@ use serde_json::Value;
 use std::collections::BTreeMap;
 use std::path::{Path, PathBuf};
 
+pub const EXEC_HARNESS_COMMAND: &str = "exec-harness";
+pub const EXEC_HARNESS_VERSION: &str = "1.2.0";
+
 /// Shared orchestration state created once per CLI invocation.
 ///
 /// Holds the run-level configuration, environment provider, system info, and logger.

src/executor/tests.rs

@@ -341,7 +341,7 @@ fi
         command: &[String],
     ) -> String {
         shell_words::join(
-            std::iter::once(crate::cli::exec::EXEC_HARNESS_COMMAND)
+            std::iter::once(crate::executor::orchestrator::EXEC_HARNESS_COMMAND)
                 .chain(walltime_args.to_cli_args().iter().map(|s| s.as_str()))
                 .chain(command.iter().map(|s| s.as_str())),
         )