refactor(cli): use command runner to run predictable logic (#4176)

crates/biome_cli/src/changed.rs

@@ -7,14 +7,14 @@ use std::ffi::OsString;
 pub(crate) fn get_changed_files(
     fs: &DynRef<'_, dyn FileSystem>,
     configuration: &PartialConfiguration,
-    since: Option<String>,
+    since: Option<&str>,
 ) -> Result<Vec<OsString>, CliDiagnostic> {
     let default_branch = configuration
         .vcs
         .as_ref()
         .and_then(|v| v.default_branch.as_ref());
 
-    let base = match (since.as_ref(), default_branch) {
+    let base = match (since, default_branch) {
         (Some(since), Some(_)) => since,
         (Some(since), None) => since,
         (None, Some(branch)) => branch,

crates/biome_cli/src/commands/check.rs

@@ -168,8 +168,13 @@ pub(crate) fn check(
 
     let stdin = get_stdin(stdin_file_path, &mut *session.app.console, "check")?;
 
-    let vcs_targeted_paths =
-        get_files_to_process(since, changed, staged, &session.app.fs, &fs_configuration)?;
+    let vcs_targeted_paths = get_files_to_process(
+        since.as_deref(),
+        changed,
+        staged,
+        &session.app.fs,
+        &fs_configuration,
+    )?;
 
     session
         .app

crates/biome_cli/src/commands/ci.rs

@@ -142,7 +142,7 @@ pub(crate) fn ci(session: CliSession, payload: CiCommandPayload) -> Result<(), C
     }
 
     if changed {
-        paths = get_changed_files(&session.app.fs, &fs_configuration, since)?;
+        paths = get_changed_files(&session.app.fs, &fs_configuration, since.as_deref())?;
     }
 
     session