fix(vite_task): pass raw args to dev command (#198)

### TL;DR

Fix argument handling for the `vite dev` command and improve debugging information.

### What changed?

- Fixed argument handling for the `vite dev` command by replacing `#[clap(last = true)]` with `#[arg(allow_hyphen_values = true, trailing_var_arg = true)]` to properly support hyphenated arguments
- Removed an unnecessary debug log in `TaskEnvs`
- Added detailed debug logging for executed commands, including duration and exit status
- Added a new snapshot test to verify that the `vite dev` command correctly handles port arguments

### How to test?

Run the new snapshot test to verify that the `vite dev` command correctly handles the `--port` argument:

```bash
cd packages/cli/snap-tests/command-dev-with-port
vite dev --port 12312312312
```

The command should properly pass the port argument to the dev server and show the expected RangeError.

### Why make this change?

The previous implementation didn't properly handle hyphenated arguments for the `vite dev` command, which could cause issues when users tried to pass options like `--port`. This change ensures that all arguments, including those with hyphens, are correctly passed to the underlying Vite process. The additional logging also helps with debugging task execution.

Author: fengmk2

crates/vite_task/src/execute.rs

@@ -311,8 +311,6 @@ impl TaskEnvs {
         ];
         *env_path = join_paths(node_modules_bin_paths.into_iter().chain(paths))?.into();
 
-        tracing::debug!("all_envs: {:?}", all_envs);
-
         Ok(Self { all_envs, envs_without_pass_through })
     }
 }

crates/vite_task/src/lib.rs

@@ -116,7 +116,7 @@ pub enum Commands {
         args: Vec<String>,
     },
     Dev {
-        #[clap(last = true)]
+        #[arg(allow_hyphen_values = true, trailing_var_arg = true)]
         /// Arguments to pass to vite dev
         args: Vec<String>,
     },

crates/vite_task/src/schedule.rs

@@ -223,6 +223,13 @@ async fn get_cached_or_execute<'a>(
                 let executed_task =
                     execute_task(execution_id, &task.resolved_command, base_dir).await?;
                 let exit_status = executed_task.exit_status;
+                tracing::debug!(
+                    "executed command `{}` finished, duration: {:?}, skip_cache: {}, {}",
+                    task.resolved_command.fingerprint.command,
+                    executed_task.duration,
+                    skip_cache,
+                    exit_status
+                );
                 if !skip_cache && exit_status.success() {
                     let cached_task = CommandCacheValue::create(executed_task, fs, base_dir)?;
                     cache.update(&task, cached_task).await?;

packages/cli/snap-tests/command-dev-with-port/package.json

@@ -0,0 +1,2 @@
+{
+}

packages/cli/snap-tests/command-dev-with-port/snap.txt

@@ -0,0 +1,2 @@
+> vite dev --port 12312312312 2>&1 | grep RangeError # intentionally use an invalid port (exceeds 0-65535) to trigger RangeError
+RangeError [ERR_SOCKET_BAD_PORT]: options.port should be >= 0 and < 65536. Received type number (12312312312).

packages/cli/snap-tests/command-dev-with-port/steps.json

@@ -0,0 +1,8 @@
+{
+  "env": {
+    "VITE_DISABLE_AUTO_INSTALL": "1"
+  },
+  "commands": [
+    "vite dev --port 12312312312 2>&1 | grep RangeError # intentionally use an invalid port (exceeds 0-65535) to trigger RangeError"
+  ]
+}