fix: narrow user-facing display formatting

Co-authored-by: GPT-5 Codex <codex@openai.com>

Author: liangmiQwQ

CHANGELOG.md

@@ -1,6 +1,6 @@
 # Changelog
 
-- **Fixed** Vite+ diagnostics now display paths and commands without Rust debug formatting such as escaped backslashes, wrapper types, or list syntax ([#534](https://github.com/voidzero-dev/vite-task/pull/534)).
+- **Fixed** Vite+ diagnostics now display individual paths and working directories without Rust debug formatting such as quoted paths or escaped Windows backslashes ([#534](https://github.com/voidzero-dev/vite-task/pull/534)).
 - **Fixed** Windows automatic input tracking now records executable image reads when process creation performs the lookup inside `NtCreateUserProcess` ([#518](https://github.com/voidzero-dev/vite-task/pull/518)).
 - **Fixed** Failures while waiting for a started task process to exit no longer incorrectly say the process failed to spawn ([#515](https://github.com/voidzero-dev/vite-task/pull/515)).
 - **Fixed** Missing env vars requested through `@voidzero-dev/vite-task-client` now return `undefined` instead of `null`, preserving Vite production `NODE_ENV` semantics when builds run through `vp run` ([#508](https://github.com/voidzero-dev/vite-task/pull/508)).

crates/fspy/src/error.rs

@@ -1,15 +1,11 @@
-use std::{
-    ffi::{OsStr, OsString},
-    fmt,
-    path::PathBuf,
-};
+use std::{ffi::OsString, path::PathBuf};
 
 #[derive(thiserror::Error, Debug)]
 pub enum SpawnError {
     #[error(
         "could not resolve the full path of program '{}' with PATH={} under cwd({})",
         .program.display(),
-        display_path_env(.path.as_deref()),
+        .path.as_deref().unwrap_or_else(|| std::ffi::OsStr::new("<not set>")).display(),
         .cwd.display()
     )]
     Which {
@@ -34,18 +30,3 @@ pub enum SpawnError {
     #[error("underlying os error: {0}")]
     OsSpawn(std::io::Error),
 }
-
-struct DisplayPathEnv<'a>(Option<&'a OsStr>);
-
-impl fmt::Display for DisplayPathEnv<'_> {
-    fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
-        match self.0 {
-            Some(path) => fmt::Display::fmt(&path.display(), f),
-            None => f.write_str("<not set>"),
-        }
-    }
-}
-
-const fn display_path_env(path: Option<&OsStr>) -> DisplayPathEnv<'_> {
-    DisplayPathEnv(path)
-}

crates/vite_task_graph/src/lib.rs

@@ -4,7 +4,7 @@ pub mod loader;
 pub mod query;
 mod specifier;
 
-use std::{convert::Infallible, fmt, sync::Arc};
+use std::{convert::Infallible, sync::Arc};
 
 use config::{
     ResolvedGlobalCacheConfig, ResolvedTaskConfig, UserRunConfig, UserTaskConfig,
@@ -138,10 +138,7 @@ pub enum TaskGraphLoadError {
 /// - When the specifier is from a CLI command, `UnknownPackageError` can be a real error type in case cwd is not in any package.
 #[derive(Debug, thiserror::Error, Serialize)]
 pub enum SpecifierLookupError<PackageUnknownError = Infallible> {
-    #[error(
-        "Package '{package_name}' is ambiguous among multiple packages: {}",
-        display_package_paths(.package_paths)
-    )]
+    #[error("Package '{package_name}' is ambiguous among multiple packages: {package_paths:?}")]
     AmbiguousPackageName { package_name: Str, package_paths: Box<[Arc<AbsolutePath>]> },
 
     #[error("Package '{package_name}' not found")]
@@ -161,24 +158,6 @@ pub enum SpecifierLookupError<PackageUnknownError = Infallible> {
     PackageUnknown { unspecifier_package_error: PackageUnknownError, task_name: Str },
 }
 
-struct DisplayPackagePaths<'a>(&'a [Arc<AbsolutePath>]);
-
-impl fmt::Display for DisplayPackagePaths<'_> {
-    fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
-        for (index, path) in self.0.iter().enumerate() {
-            if index > 0 {
-                f.write_str(", ")?;
-            }
-            fmt::Display::fmt(&path.as_path().display(), f)?;
-        }
-        Ok(())
-    }
-}
-
-const fn display_package_paths(paths: &[Arc<AbsolutePath>]) -> DisplayPackagePaths<'_> {
-    DisplayPackagePaths(paths)
-}
-
 /// newtype of `DefaultIx` for indices in task graphs
 #[derive(Debug, Default, Clone, Copy, PartialEq, Eq, PartialOrd, Ord, Hash, Serialize)]
 pub struct TaskIx(DefaultIx);

crates/vite_task_plan/src/error.rs

@@ -3,7 +3,7 @@
     reason = "Arc<Path> is used for non-UTF-8 path data in error types"
 )]
 use std::path::Path;
-use std::{env::JoinPathsError, ffi::OsStr, fmt, sync::Arc};
+use std::{env::JoinPathsError, ffi::OsStr, sync::Arc};
 
 use vite_path::{AbsolutePath, relative::InvalidPathDataError};
 use vite_str::Str;
@@ -124,8 +124,7 @@ pub enum Error {
     TaskRecursionDetected(#[from] TaskRecursionError),
 
     #[error(
-        "Invalid Vite+ command: {} under cwd {}",
-        display_command(.program.as_str(), .args),
+        "Invalid vite task command: {program} with args {args:?} under cwd {}",
         .cwd.as_path().display()
     )]
     ParsePlanRequest {
@@ -184,23 +183,3 @@ pub enum Error {
     #[error("Cycle dependency detected: {}", _0.iter().map(std::string::ToString::to_string).collect::<Vec<_>>().join(" -> "))]
     CycleDependencyDetected(Vec<TaskDisplay>),
 }
-
-struct DisplayCommand<'a> {
-    program: &'a str,
-    args: &'a [Str],
-}
-
-impl fmt::Display for DisplayCommand<'_> {
-    fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
-        fmt::Display::fmt(&shell_escape::escape(self.program.into()), f)?;
-        for arg in self.args {
-            f.write_str(" ")?;
-            fmt::Display::fmt(&shell_escape::escape(arg.as_str().into()), f)?;
-        }
-        Ok(())
-    }
-}
-
-const fn display_command<'a>(program: &'a str, args: &'a [Str]) -> DisplayCommand<'a> {
-    DisplayCommand { program, args }
-}