use PreferExecutable enum for pixi run

Author: kajal-jotwani

crates/pixi/tests/integration_rust/common/mod.rs

@@ -38,7 +38,7 @@ use pixi_manifest::{EnvironmentName, FeatureName};
 use pixi_progress::global_multi_progress;
 use pixi_task::{
     ExecutableTask, RunOutput, SearchEnvironments, TaskExecutionError, TaskGraph, TaskGraphError,
-    TaskName, get_task_env,
+    TaskName, get_task_env, PreferExecutable,
 };
 use rattler_conda_types::{MatchSpec, ParseStrictness::Lenient, Platform};
 use rattler_lock::{LockFile, LockedPackageRef, UrlOrPath};
@@ -591,7 +591,7 @@ impl PixiControl {
                 .map(|e| e.best_platform())
                 .or(Some(Platform::current())),
         );
-        let task_graph = TaskGraph::from_cmd_args(&project, &search_env, args.task, false, false)
+        let task_graph = TaskGraph::from_cmd_args(&project, &search_env, args.task, false, PreferExecutable::Never)
             .map_err(RunError::TaskGraphError)?;
 
         // Iterate over all tasks in the graph and execute them.

crates/pixi_cli/src/run.rs

@@ -26,7 +26,7 @@ use pixi_manifest::{FeaturesExt, TaskName};
 use pixi_progress::global_multi_progress;
 use pixi_task::{
     AmbiguousTask, CanSkip, ExecutableTask, FailedToParseShellScript, InvalidWorkingDirectory,
-    SearchEnvironments, TaskAndEnvironment, TaskGraph, get_task_env,
+    SearchEnvironments, TaskAndEnvironment, TaskGraph, get_task_env, PreferExecutable,
 };
 use rattler_conda_types::Platform;
 use thiserror::Error;
@@ -183,9 +183,17 @@ pub async fn execute(args: Args) -> miette::Result<()> {
         SearchEnvironments::from_opt_env(&workspace, explicit_environment.clone(), search_platform)
             .with_disambiguate_fn(disambiguate_task_interactive);
 
-    let task_graph =
-        TaskGraph::from_cmd_args(&workspace, &search_environment, args.task, args.skip_deps, args.executable)?;
-
+    let task_graph = TaskGraph::from_cmd_args(
+        &workspace,
+        &search_environment,
+        args.task,
+        args.skip_deps,
+        if args.executable {
+            PreferExecutable::Always
+        } else {
+            PreferExecutable::Never
+        },
+    )?;
     tracing::debug!("Task graph: {}", task_graph);
 
     // Print dry-run message if dry-run mode is enabled

crates/pixi_task/src/task_graph.rs

@@ -43,6 +43,21 @@ fn join_args_with_single_quotes<'a>(args: impl IntoIterator<Item = &'a str>) ->
         .join(" ")
 }
 
+/// Controls whether to prefer resolving commands as executables over Pixi tasks
+#[derive(Debug, Clone, Copy, PartialEq, Eq)]
+pub enum PreferExecutable {
+    /// Always try to resolve as a Pixi task first (default behavior)
+    Never,
+    /// Always treat as an executable, skip task resolution
+    Always,
+}
+
+impl Default for PreferExecutable {
+    fn default() -> Self {
+        Self::Never
+    }
+}
+
 /// A task ID is a unique identifier for a [`TaskNode`] in a [`TaskGraph`].
 ///
 /// To get a task from a [`TaskGraph`], you can use the [`TaskId`] as an index.
@@ -199,7 +214,7 @@ impl<'p> TaskGraph<'p> {
         search_envs: &SearchEnvironments<'p, D>,
         args: Vec<String>,
         skip_deps: bool,
-        prefer_executable: bool,
+        prefer_executable: PreferExecutable,
     ) -> Result<Self, TaskGraphError> {
         // Split 'args' into arguments if it's a single string, supporting commands
         // like: `"test 1 == 0 || echo failed"` or `"echo foo && echo bar"` or
@@ -214,7 +229,7 @@ impl<'p> TaskGraph<'p> {
             (args, true)
         };
 
-        if !prefer_executable && let Some(name) = args.first() {
+        if prefer_executable == PreferExecutable::Never && let Some(name) = args.first() {
             match search_envs.find_task(TaskName::from(name.clone()), FindTaskSource::CmdArgs, None)
             {
                 Err(FindTaskError::MissingTask(_)) => {}
@@ -639,7 +654,7 @@ mod test {
             &search_envs,
             run_args.iter().map(|arg| arg.to_string()).collect(),
             skip_deps,
-            false,
+            PreferExecutable::Never,
         )
         .unwrap();