fix(cli): enable cache support for vp check in task runner

When `vp check` was used as a script (e.g., `"check": "vp check"`) and
run via `vp run check` with `run: { cache: true }`, cache was always
disabled because the command handler forced `UserCacheConfig::disabled()`.

Change the Check handler to use `UserCacheConfig::with_config(...)` with
auto input tracking and task-cache directory exclusions, matching the
pattern used by other synthesized commands (build, lint, fmt).

Author: fengmk2

packages/cli/binding/src/cli.rs

@@ -504,19 +504,30 @@ fn exclude_glob(pattern: &str, base: InputBase) -> UserInputEntry {
     UserInputEntry::GlobWithBase(GlobWithBase { pattern: Str::from(pattern), base })
 }
 
-/// Common cache input entries for build/pack commands.
-/// Excludes .vite-temp config files and dist output files that are both read and written.
-/// TODO: The hardcoded `!dist/**` exclusion is a temporary workaround. It will be replaced
-/// by a runner-aware approach that automatically excludes task output directories.
-fn build_pack_cache_inputs() -> Vec<UserInputEntry> {
+/// Base cache input entries shared by synthesized commands.
+/// Includes auto-tracking and excludes directories that cause false cache invalidation:
+/// - `.vite-temp`: config compilation cache, read+written during vp CLI startup
+/// - `.vite/task-cache`: task runner state files that change after each run
+fn base_cache_inputs() -> Vec<UserInputEntry> {
     vec![
         UserInputEntry::Auto(AutoInput { auto: true }),
         exclude_glob("!node_modules/.vite-temp/**", InputBase::Workspace),
         exclude_glob("!node_modules/.vite-temp/**", InputBase::Package),
-        exclude_glob("!dist/**", InputBase::Package),
+        exclude_glob("!node_modules/.vite/task-cache/**", InputBase::Workspace),
+        exclude_glob("!node_modules/.vite/task-cache/**", InputBase::Package),
     ]
 }
 
+/// Cache input entries for build/pack commands.
+/// Extends base inputs with dist output exclusion.
+/// TODO: The hardcoded `!dist/**` exclusion is a temporary workaround. It will be replaced
+/// by a runner-aware approach that automatically excludes task output directories.
+fn build_pack_cache_inputs() -> Vec<UserInputEntry> {
+    let mut inputs = base_cache_inputs();
+    inputs.push(exclude_glob("!dist/**", InputBase::Package));
+    inputs
+}
+
 fn merge_resolved_envs(
     envs: &Arc<FxHashMap<Arc<OsStr>, Arc<OsStr>>>,
     resolved_envs: Vec<(String, String)>,
@@ -587,10 +598,14 @@ impl CommandHandler for VitePlusCommandHandler {
         };
         match cli_args {
             CLIArgs::Synthesizable(SynthesizableSubcommand::Check { .. }) => {
-                // Check is a composite command — run as a subprocess in task scripts
-                Ok(HandledCommand::Synthesized(
-                    command.to_synthetic_plan_request(UserCacheConfig::disabled()),
-                ))
+                // Check is a composite command (fmt + lint) — run as a subprocess in task scripts
+                Ok(HandledCommand::Synthesized(command.to_synthetic_plan_request(
+                    UserCacheConfig::with_config(EnabledCacheConfig {
+                        env: Some(Box::new([Str::from("OXLINT_TSGOLINT_PATH")])),
+                        untracked_env: None,
+                        input: Some(base_cache_inputs()),
+                    }),
+                )))
             }
             CLIArgs::Synthesizable(subcmd) => {
                 let resolved =

packages/cli/snap-tests/check-cache-disabled/package.json

@@ -0,0 +1,7 @@
+{
+  "name": "check-cache-disabled-test",
+  "type": "module",
+  "scripts": {
+    "check": "vp check"
+  }
+}

packages/cli/snap-tests/check-cache-disabled/snap.txt

@@ -0,0 +1,5 @@
+> vp run check # vp check should have cache disabled without run.cache
+$ vp check ⊘ cache disabled
+pass: All 3 files are correctly formatted (<variable>ms, <variable> threads)
+pass: Found no warnings or lint errors in 1 file (<variable>ms, <variable> threads)
+

packages/cli/snap-tests/check-cache-disabled/src/index.js

@@ -0,0 +1,5 @@
+function hello() {
+  return "hello";
+}
+
+export { hello };

packages/cli/snap-tests/check-cache-disabled/steps.json

@@ -0,0 +1,3 @@
+{
+  "commands": ["vp run check # vp check should have cache disabled without run.cache"]
+}

packages/cli/snap-tests/check-cache-enabled/package.json

@@ -0,0 +1,7 @@
+{
+  "name": "check-cache-enabled-test",
+  "type": "module",
+  "scripts": {
+    "check": "vp check"
+  }
+}

packages/cli/snap-tests/check-cache-enabled/snap.txt

@@ -0,0 +1,20 @@
+> vp run check # first run should be cache miss
+$ vp check
+pass: All 4 files are correctly formatted (<variable>ms, <variable> threads)
+pass: Found no warnings or lint errors in 2 files (<variable>ms, <variable> threads)
+
+
+> vp run check # second run should be cache hit
+$ vp check ◉ cache hit, replaying
+pass: All 4 files are correctly formatted (<variable>ms, <variable> threads)
+pass: Found no warnings or lint errors in 2 files (<variable>ms, <variable> threads)
+
+---
+vp run: cache hit, <variable>ms saved.
+
+> echo 'export const foo = 1;' > src/foo.js
+> vp run check # third run should be cache miss after new file added
+$ vp check ○ cache miss: 'foo.js' added in 'src', executing
+pass: All 5 files are correctly formatted (<variable>ms, <variable> threads)
+pass: Found no warnings or lint errors in 3 files (<variable>ms, <variable> threads)
+

packages/cli/snap-tests/check-cache-enabled/src/index.js

@@ -0,0 +1,5 @@
+function hello() {
+  return "hello";
+}
+
+export { hello };

packages/cli/snap-tests/check-cache-enabled/steps.json

@@ -0,0 +1,8 @@
+{
+  "commands": [
+    "vp run check # first run should be cache miss",
+    "vp run check # second run should be cache hit",
+    "echo 'export const foo = 1;' > src/foo.js",
+    "vp run check # third run should be cache miss after new file added"
+  ]
+}

packages/cli/snap-tests/check-cache-enabled/vite.config.ts

@@ -0,0 +1,5 @@
+export default {
+  run: {
+    cache: true,
+  },
+};