feat: integrate minijinja for tasks' command's rendering

crates/pixi_consts/src/consts.rs

@@ -89,6 +89,7 @@ pub const CLAP_GLOBAL_OPTIONS: &str = "Global Options";
 pub const CLAP_UPDATE_OPTIONS: &str = "Update Options";
 
 pub static TASK_STYLE: LazyLock<Style> = LazyLock::new(|| Style::new().blue());
+pub static TASK_ERROR_STYLE: LazyLock<Style> = LazyLock::new(|| Style::new().red());
 pub static PLATFORM_STYLE: LazyLock<Style> = LazyLock::new(|| Style::new().yellow());
 pub static ENVIRONMENT_STYLE: LazyLock<Style> = LazyLock::new(|| Style::new().magenta());
 pub static EXPOSED_NAME_STYLE: LazyLock<Style> = LazyLock::new(|| Style::new().yellow());

crates/pixi_manifest/Cargo.toml

@@ -44,9 +44,11 @@ rattler_virtual_packages = { workspace = true }
 rattler_lock = { workspace = true, optional = true }
 rattler_solve = { workspace = true, optional = true }
 
-# TODO: Remove these dependencies
+
 console = { workspace = true }
 miette = { workspace = true, features = ["fancy-no-backtrace"] }
+minijinja = { workspace = true, features = ["builtins"] }
+once_cell.workspace = true
 
 [dev-dependencies]
 assert_matches = { workspace = true }

crates/pixi_manifest/src/manifests/snapshots/pixi_manifest__manifests__workspace__tests__target_specific_tasks.snap

@@ -1,7 +1,7 @@
 ---
 source: crates/pixi_manifest/src/manifests/workspace.rs
-expression: "manifest.default_feature().targets.iter().flat_map(|(target, selector)|\n{\n    let selector_name =\n    selector.map_or_else(|| String::from(\"default\"), ToString::to_string);\n    target.tasks.iter().filter_map(move |(name, task)|\n    {\n        Some(format!(\"{}/{} = {}\", &selector_name, name.as_str(),\n        task.as_single_command()?))\n    })\n}).join(\"\\n\")"
+expression: "manifest.default_feature().targets.iter().flat_map(|(target, selector)|\n{\n    let selector_name =\n    selector.map_or_else(|| String::from(\"default\"), ToString::to_string);\n    target.tasks.iter().map(move |(name, task)|\n    {\n        format!(\"{}/{} = {:?}\", &selector_name, name.as_str(),\n        task.as_single_command().ok())\n    })\n}).join(\"\\n\")"
 ---
-default/test = test multi
-win-64/test = test win
-linux-64/test = test linux
+default/test = Some("test multi")
+win-64/test = Some("test win")
+linux-64/test = Some("test linux")

crates/pixi_manifest/src/manifests/workspace.rs

@@ -1214,13 +1214,16 @@ start = "python -m flask run --port=5050"
             .flat_map(|(target, selector)| {
                 let selector_name =
                     selector.map_or_else(|| String::from("default"), ToString::to_string);
-                target.tasks.iter().filter_map(move |(name, task)| {
-                    Some(format!(
-                        "{}/{} = {}",
+                target.tasks.iter().map(move |(name, task)| {
+                    format!(
+                        "{}/{} = {:?}",
                         &selector_name,
                         name.as_str(),
-                        task.as_single_command()?
-                    ))
+                        task.as_single_command(None)
+                            .ok()
+                            .flatten()
+                            .map(|c| c.to_string())
+                    )
                 })
             })
             .join("\n"));
@@ -2399,7 +2402,8 @@ platforms = ["linux-64", "win-64"]
                 .tasks
                 .get(&"warmup".into())
                 .unwrap()
-                .as_single_command()
+                .as_single_command(None)
+                .unwrap()
                 .unwrap(),
             "python warmup.py"
         );
@@ -2445,31 +2449,31 @@ test = "test initial"
         manifest
             .add_task(
                 "default".into(),
-                Task::Plain("echo default".to_string()),
+                Task::Plain("echo default".into()),
                 None,
                 &FeatureName::DEFAULT,
             )
             .unwrap();
         manifest
             .add_task(
                 "target_linux".into(),
-                Task::Plain("echo target_linux".to_string()),
+                Task::Plain("echo target_linux".into()),
                 Some(Platform::Linux64),
                 &FeatureName::DEFAULT,
             )
             .unwrap();
         manifest
             .add_task(
                 "feature_test".into(),
-                Task::Plain("echo feature_test".to_string()),
+                Task::Plain("echo feature_test".into()),
                 None,
                 &FeatureName::from("test"),
             )
             .unwrap();
         manifest
             .add_task(
                 "feature_test_target_linux".into(),
-                Task::Plain("echo feature_test_target_linux".to_string()),
+                Task::Plain("echo feature_test_target_linux".into()),
                 Some(Platform::Linux64),
                 &FeatureName::from("test"),
             )