Flatten cargoExtraArgs away from the runnable lsp extension

Author: Veykril

crates/rust-analyzer/src/handlers/request.rs

@@ -874,6 +874,7 @@ pub(crate) fn handle_runnables(
                 if all_targets {
                     cargo_args.push("--all-targets".to_owned());
                 }
+                cargo_args.extend(config.cargo_extra_args.iter().cloned());
                 res.push(lsp_ext::Runnable {
                     label: format!(
                         "cargo {cmd} -p {}{all_targets}",
@@ -887,7 +888,6 @@ pub(crate) fn handle_runnables(
                         cwd: cwd.into(),
                         override_cargo: config.override_cargo.clone(),
                         cargo_args,
-                        cargo_extra_args: config.cargo_extra_args.clone(),
                         executable_args: Vec::new(),
                         environment: Default::default(),
                     }),
@@ -897,6 +897,8 @@ pub(crate) fn handle_runnables(
         Some(TargetSpec::ProjectJson(_)) => {}
         None => {
             if !snap.config.linked_or_discovered_projects().is_empty() {
+                let mut cargo_args = vec!["check".to_owned(), "--workspace".to_owned()];
+                cargo_args.extend(config.cargo_extra_args.iter().cloned());
                 res.push(lsp_ext::Runnable {
                     label: "cargo check --workspace".to_owned(),
                     location: None,
@@ -905,8 +907,7 @@ pub(crate) fn handle_runnables(
                         workspace_root: None,
                         cwd: ".".into(),
                         override_cargo: config.override_cargo,
-                        cargo_args: vec!["check".to_owned(), "--workspace".to_owned()],
-                        cargo_extra_args: config.cargo_extra_args,
+                        cargo_args,
                         executable_args: Vec::new(),
                         environment: Default::default(),
                     }),

crates/rust-analyzer/src/lsp/ext.rs

@@ -468,8 +468,6 @@ pub struct CargoRunnableArgs {
     pub workspace_root: Option<Utf8PathBuf>,
     // command, --package and --lib stuff
     pub cargo_args: Vec<String>,
-    // user-specified additional cargo args, like `--release`.
-    pub cargo_extra_args: Vec<String>,
     // stuff after --
     pub executable_args: Vec<String>,
 }

crates/rust-analyzer/src/lsp/to_proto.rs

@@ -1391,7 +1391,6 @@ pub(crate) fn runnable(
                     override_cargo: config.override_cargo,
                     cargo_args,
                     cwd: cwd.into(),
-                    cargo_extra_args: config.cargo_extra_args,
                     executable_args,
                     environment: Default::default(),
                 }),
@@ -1435,7 +1434,6 @@ pub(crate) fn runnable(
                     override_cargo: config.override_cargo,
                     cargo_args,
                     cwd: Utf8PathBuf::from("."),
-                    cargo_extra_args: config.cargo_extra_args,
                     executable_args,
                     environment: Default::default(),
                 }),

crates/rust-analyzer/src/target_spec.rs

@@ -110,7 +110,8 @@ impl CargoTargetSpec {
         kind: &RunnableKind,
         cfg: &Option<CfgExpr>,
     ) -> (Vec<String>, Vec<String>) {
-        let extra_test_binary_args = snap.config.runnables().extra_test_binary_args;
+        let config = snap.config.runnables();
+        let extra_test_binary_args = config.extra_test_binary_args;
 
         let mut cargo_args = Vec::new();
         let mut executable_args = Vec::new();
@@ -196,6 +197,7 @@ impl CargoTargetSpec {
                 }
             }
         }
+        cargo_args.extend(config.cargo_extra_args.iter().cloned());
         (cargo_args, executable_args)
     }
 

docs/dev/lsp-extensions.md

@@ -392,11 +392,6 @@ rust-analyzer supports two `kind`s of runnables, `"cargo"` and `"shell"`. The `a
      * The cargo command to run.
      */
     cargoArgs: string[];
-    /**
-     * Extra arguments to pass to cargo.
-     */
-    // What is the point of this when cargoArgs exists?
-    cargoExtraArgs: string[];
     /**
      * Arguments to pass to the executable, these will be passed to the command after a `--` argument.
      */

editors/code/src/lsp_ext.ts

@@ -257,22 +257,19 @@ export type CargoRunnableArgs = {
      * The workspace root directory of the cargo project.
      */
     workspaceRoot?: string;
-    /**
-     * The cargo command to run.
-     */
-    cargoArgs: string[];
-    /**
-     * Extra arguments to pass to cargo.
-     */
-    // What is the point of this when cargoArgs exists?
-    cargoExtraArgs: string[];
     /**
      * Arguments to pass to the executable, these will be passed to the command after a `--` argument.
      */
     executableArgs: string[];
+    /**
+     * Arguments to pass to cargo.
+     */
+    cargoArgs: string[];
     /**
      * Command to execute instead of `cargo`.
      */
+    // This is supplied by the user via config. We could pull this through the client config in the
+    // extension directly, but that would prevent us from honoring the rust-analyzer.toml for it.
     overrideCargo?: string;
 } & CommonRunnableArgs;
 

editors/code/src/run.ts

@@ -165,9 +165,6 @@ export async function createTaskFromRunnable(
 
 export function createCargoArgs(runnableArgs: ra.CargoRunnableArgs): string[] {
     const args = [...runnableArgs.cargoArgs]; // should be a copy!
-    if (runnableArgs.cargoExtraArgs) {
-        args.push(...runnableArgs.cargoExtraArgs); // Append user-specified cargo options.
-    }
     if (runnableArgs.executableArgs.length > 0) {
         args.push("--", ...runnableArgs.executableArgs);
     }

editors/code/tests/unit/runnable_env.test.ts

@@ -12,7 +12,6 @@ function makeRunnable(label: string): ra.Runnable {
             cargoArgs: [],
             cwd: ".",
             executableArgs: [],
-            cargoExtraArgs: [],
             environment: {},
         },
     };