fix: enable task selection for vt run --cache (#313)

- normalizes the run command to ignore if cache flag is specified or not
such that `vt run --cache` is treated like `vt run` and triggers
interactive task selection

Fixes #312

Author: HaasStefan

CHANGELOG.md

@@ -1,5 +1,6 @@
 # Changelog
 
+- **Fixed** `vp run --cache` now supports running without a task specifier and opens the interactive task selector, matching bare `vp run` behavior ([#312](https://github.com/voidzero-dev/vite-task/pull/313))
 - **Fixed** Ctrl-C now prevents future tasks from being scheduled and prevents caching of in-flight task results ([#309](https://github.com/voidzero-dev/vite-task/pull/309))
 - **Added** `--concurrency-limit` flag to limit the number of tasks running at the same time (defaults to 4) ([#288](https://github.com/voidzero-dev/vite-task/pull/288), [#309](https://github.com/voidzero-dev/vite-task/pull/309))
 - **Added** `--parallel` flag to ignore task dependencies and run all tasks at once with unlimited concurrency (unless `--concurrency-limit` is also specified) ([#309](https://github.com/voidzero-dev/vite-task/pull/309))

crates/vite_task/src/session/mod.rs

@@ -298,7 +298,12 @@ impl<'a> Session<'a> {
                     let bare = RunCommand::try_parse_from::<_, &str>([])
                         .expect("parsing hardcoded bare command should never fail")
                         .into_resolved();
-                    if run_command != bare {
+
+                    // Normalize the run_command for comparison by ignoring cache flags, which don't affect task selection.
+                    let mut normalized_run_command = run_command.clone();
+                    normalized_run_command.flags.cache = false;
+
+                    if normalized_run_command != bare {
                         return Err(vite_task_plan::Error::MissingTaskSpecifier.into());
                     }
                     let qpr = self.handle_no_task(is_interactive, &run_command).await?;

crates/vite_task_bin/tests/e2e_snapshots/fixtures/cache-task-select/package.json

@@ -0,0 +1,4 @@
+{
+  "name": "cache-task-select-test",
+  "private": true
+}

crates/vite_task_bin/tests/e2e_snapshots/fixtures/cache-task-select/snapshots.toml

@@ -0,0 +1,20 @@
+[[e2e]]
+name = "cache hit with interactive selector and --cache"
+steps = [
+  { argv = [
+    "vt",
+    "run",
+    "--cache",
+  ], interactions = [
+    { "expect-milestone" = "task-select::0" },
+    { "write-key" = "enter" },
+  ] },
+  { argv = [
+    "vt",
+    "run",
+    "--cache",
+  ], interactions = [
+    { "expect-milestone" = "task-select::0" },
+    { "write-key" = "enter" },
+  ] },
+]

crates/vite_task_bin/tests/e2e_snapshots/fixtures/cache-task-select/snapshots/cache hit with interactive selector and --cache.snap

@@ -0,0 +1,28 @@
+---
+source: crates/vite_task_bin/tests/e2e_snapshots/main.rs
+assertion_line: 441
+expression: e2e_outputs
+---
+> vt run --cache
+@ expect-milestone: task-select::0
+Select a task (↑/↓, Enter to run, type to search):
+
+  › build vtt print-file src/main.ts
+    lint  echo lint app
+@ write-key: enter
+Selected task: build
+$ vtt print-file src/main.ts
+export const main = 'initial';
+> vt run --cache
+@ expect-milestone: task-select::0
+Select a task (↑/↓, Enter to run, type to search):
+
+  › build vtt print-file src/main.ts
+    lint  echo lint app
+@ write-key: enter
+Selected task: build
+$ vtt print-file src/main.ts ◉ cache hit, replaying
+export const main = 'initial';
+
+---
+vt run: cache hit.

crates/vite_task_bin/tests/e2e_snapshots/fixtures/cache-task-select/src/main.ts

@@ -0,0 +1 @@
+export const main = 'initial';

crates/vite_task_bin/tests/e2e_snapshots/fixtures/cache-task-select/vite-task.json

@@ -0,0 +1,11 @@
+{
+  "tasks": {
+    "build": {
+      "command": "vtt print-file src/main.ts",
+      "cache": true
+    },
+    "lint": {
+      "command": "echo lint app"
+    }
+  }
+}