fix: remove conflicting MISE_SHELL setting

Fixes #3280

settings.toml

@@ -768,16 +768,6 @@ type = "Path"
 optional = true
 description = "Path to the rustup home directory. Defaults to ~/.rustup or %USERPROFILE%\\.rustup"
 
-[shell]
-env = "MISE_SHELL"
-type = "String"
-optional = true
-description = "Sets the shell across all mise commands like `mise run`, `mise en`, and watch files."
-docs = """
-Sets the shell across all mise commands like `mise run`, `mise en`, and watch files.
-This overrides MISE_UNIX_DEFAULT_INLINE_SHELL_ARGS and MISE_WINDOWS_DEFAULT_INLINE_SHELL_ARGS if set.
-"""
-
 [shorthands_file]
 env = "MISE_SHORTHANDS_FILE"
 type = "Path"

src/cli/en.rs

@@ -19,7 +19,7 @@ pub struct En {
     /// Shell to start
     ///
     /// Defaults to $SHELL
-    #[clap(verbatim_doc_comment, long, short = 's', env = "MISE_SHELL")]
+    #[clap(verbatim_doc_comment, long, short = 's')]
     pub shell: Option<String>,
 }
 

src/cli/run.rs

@@ -101,7 +101,7 @@ pub struct Run {
     /// Defaults to `sh -c -o errexit -o pipefail` on unix, and `cmd /c` on Windows
     /// Can also be set with the setting `MISE_UNIX_DEFAULT_INLINE_SHELL_ARGS` or `MISE_WINDOWS_DEFAULT_INLINE_SHELL_ARGS`
     /// Or it can be overridden with the `shell` property on a task.
-    #[clap(long, short, verbatim_doc_comment, env = "MISE_SHELL")]
+    #[clap(long, short, verbatim_doc_comment)]
     pub shell: Option<String>,
 
     /// Tool(s) to run in addition to what is in mise.toml files

src/config/settings.rs

@@ -179,10 +179,6 @@ impl Settings {
         if let Some(false) = self.vfox {
             self.disable_backends.push("vfox".to_string());
         }
-        if let Some(shell) = &self.shell {
-            self.unix_default_inline_shell_args = shell.clone();
-            self.windows_default_inline_shell_args = shell.clone();
-        }
         if let Some(disable_default_shorthands) = self.disable_default_shorthands {
             self.disable_default_registry = disable_default_shorthands;
         }