feat(cli): add --write, --fix and --unsafe options to fixable commands

crates/biome_cli/src/commands/check.rs

@@ -14,13 +14,18 @@ use biome_service::configuration::PartialConfigurationExt;
 use biome_service::workspace::RegisterProjectFolderParams;
 use biome_service::{
     configuration::{load_configuration, LoadedConfiguration},
-    workspace::{FixFileMode, UpdateSettingsParams},
+    workspace::UpdateSettingsParams,
 };
 use std::ffi::OsString;
 
+use super::{determine_fix_file_mode, FixFileModeOptions};
+
 pub(crate) struct CheckCommandPayload {
     pub(crate) apply: bool,
     pub(crate) apply_unsafe: bool,
+    pub(crate) write: bool,
+    pub(crate) fix: bool,
+    pub(crate) unsafe_: bool,
     pub(crate) cli_options: CliOptions,
     pub(crate) configuration: Option<PartialConfiguration>,
     pub(crate) paths: Vec<OsString>,
@@ -41,6 +46,9 @@ pub(crate) fn check(
     let CheckCommandPayload {
         apply,
         apply_unsafe,
+        write,
+        fix,
+        unsafe_,
         cli_options,
         configuration,
         mut paths,
@@ -54,18 +62,13 @@ pub(crate) fn check(
     } = payload;
     setup_cli_subscriber(cli_options.log_level, cli_options.log_kind);
 
-    let fix_file_mode = if apply && apply_unsafe {
-        return Err(CliDiagnostic::incompatible_arguments(
-            "--apply",
-            "--apply-unsafe",
-        ));
-    } else if !apply && !apply_unsafe {
-        None
-    } else if apply && !apply_unsafe {
-        Some(FixFileMode::SafeFixes)
-    } else {
-        Some(FixFileMode::SafeAndUnsafeFixes)
-    };
+    let fix_file_mode = determine_fix_file_mode(FixFileModeOptions {
+        apply,
+        apply_unsafe,
+        write,
+        fix,
+        unsafe_,
+    })?;
 
     let loaded_configuration =
         load_configuration(&session.app.fs, cli_options.as_configuration_path_hint())?;

crates/biome_cli/src/commands/format.rs

@@ -20,6 +20,8 @@ use biome_service::configuration::{
 use biome_service::workspace::{RegisterProjectFolderParams, UpdateSettingsParams};
 use std::ffi::OsString;
 
+use super::check_fix_incompatible_arguments;
+
 pub(crate) struct FormatCommandPayload {
     pub(crate) javascript_formatter: Option<PartialJavascriptFormatter>,
     pub(crate) json_formatter: Option<PartialJsonFormatter>,
@@ -29,6 +31,7 @@ pub(crate) struct FormatCommandPayload {
     pub(crate) files_configuration: Option<PartialFilesConfiguration>,
     pub(crate) stdin_file_path: Option<String>,
     pub(crate) write: bool,
+    pub(crate) fix: bool,
     pub(crate) cli_options: CliOptions,
     pub(crate) paths: Vec<OsString>,
     pub(crate) staged: bool,
@@ -50,6 +53,7 @@ pub(crate) fn format(
         stdin_file_path,
         files_configuration,
         write,
+        fix,
         mut json_formatter,
         mut css_formatter,
         since,
@@ -58,6 +62,14 @@ pub(crate) fn format(
     } = payload;
     setup_cli_subscriber(cli_options.log_level, cli_options.log_kind);
 
+    check_fix_incompatible_arguments(super::FixFileModeOptions {
+        apply: false,
+        apply_unsafe: false,
+        write,
+        fix,
+        unsafe_: false,
+    })?;
+
     let loaded_configuration =
         load_configuration(&session.app.fs, cli_options.as_configuration_path_hint())?;
     validate_configuration_diagnostics(
@@ -208,7 +220,7 @@ pub(crate) fn format(
 
     let execution = Execution::new(TraversalMode::Format {
         ignore_errors: cli_options.skip_errors,
-        write,
+        write: write || fix,
         stdin,
     })
     .set_report(&cli_options);

crates/biome_cli/src/commands/lint.rs

@@ -14,12 +14,17 @@ use biome_deserialize::Merge;
 use biome_service::configuration::{
     load_configuration, LoadedConfiguration, PartialConfigurationExt,
 };
-use biome_service::workspace::{FixFileMode, RegisterProjectFolderParams, UpdateSettingsParams};
+use biome_service::workspace::{RegisterProjectFolderParams, UpdateSettingsParams};
 use std::ffi::OsString;
 
+use super::{determine_fix_file_mode, FixFileModeOptions};
+
 pub(crate) struct LintCommandPayload {
     pub(crate) apply: bool,
     pub(crate) apply_unsafe: bool,
+    pub(crate) write: bool,
+    pub(crate) fix: bool,
+    pub(crate) unsafe_: bool,
     pub(crate) cli_options: CliOptions,
     pub(crate) linter_configuration: Option<PartialLinterConfiguration>,
     pub(crate) vcs_configuration: Option<PartialVcsConfiguration>,
@@ -37,6 +42,9 @@ pub(crate) fn lint(session: CliSession, payload: LintCommandPayload) -> Result<(
     let LintCommandPayload {
         apply,
         apply_unsafe,
+        write,
+        fix,
+        unsafe_,
         cli_options,
         mut linter_configuration,
         mut paths,
@@ -50,18 +58,13 @@ pub(crate) fn lint(session: CliSession, payload: LintCommandPayload) -> Result<(
     } = payload;
     setup_cli_subscriber(cli_options.log_level, cli_options.log_kind);
 
-    let fix_file_mode = if apply && apply_unsafe {
-        return Err(CliDiagnostic::incompatible_arguments(
-            "--apply",
-            "--apply-unsafe",
-        ));
-    } else if !apply && !apply_unsafe {
-        None
-    } else if apply && !apply_unsafe {
-        Some(FixFileMode::SafeFixes)
-    } else {
-        Some(FixFileMode::SafeAndUnsafeFixes)
-    };
+    let fix_file_mode = determine_fix_file_mode(FixFileModeOptions {
+        apply,
+        apply_unsafe,
+        write,
+        fix,
+        unsafe_,
+    })?;
 
     let loaded_configuration =
         load_configuration(&session.app.fs, cli_options.as_configuration_path_hint())?;