feat(tests): add inline comments to snapshot tests and fix test issues

- Add inline shell comments (# explanation) to all e2e test steps so
  they appear in generated snapshots for better readability
- Fix shared-caching-inputs: correct comments to reflect actual cache
  behavior (script2 gets cache hit from script1's identical command)
- Fix vite-task-smoke: correct comments (main.js IS an input file)
- Fix individual-cache-for-envs: add vite.config.json with envs config
  so different env values actually get separate cache entries
- Fix colon-in-name: rename test from "task with underscore in name"
  to "read file with colon in name" and delete stale snapshot
- Add file header comments explaining test purpose

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: branchseer

crates/vite_task_bin/tests/e2e_snapshots/fixtures/associate-existing-cache/snapshots.toml

@@ -1,8 +1,10 @@
+# Tests that tasks with identical commands share cache
+
 [[e2e]]
 name = "associate existing cache"
 steps = [
-  "vite run script1",
-  "vite run script2",
-  "json-edit package.json '_.scripts.script2 = \"print world\"'",
-  "vite run script2",
+  "vite run script1 # cache miss",
+  "vite run script2 # cache hit, same command as script1",
+  "json-edit package.json '_.scripts.script2 = \"print world\"' # change script2",
+  "vite run script2 # cache miss",
 ]

crates/vite_task_bin/tests/e2e_snapshots/fixtures/associate-existing-cache/snapshots/associate existing cache.snap

@@ -4,7 +4,7 @@ assertion_line: 203
 expression: e2e_outputs
 input_file: crates/vite_task_bin/tests/e2e_snapshots/fixtures/associate-existing-cache
 ---
-> vite run script1
+> vite run script1 # cache miss
 $ print hello
 hello
 
@@ -22,7 +22,7 @@ Task Details:
       → Cache miss: no previous cache entry found
 ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 
-> vite run script2
+> vite run script2 # cache hit, same command as script1
 $ print hello ✓ cache hit, replaying
 hello
 
@@ -40,9 +40,9 @@ Task Details:
       → Cache hit - output replayed - <duration> saved
 ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 
-> json-edit package.json '_.scripts.script2 = "print world"'
+> json-edit package.json '_.scripts.script2 = "print world"' # change script2
 
-> vite run script2
+> vite run script2 # cache miss
 $ print world ✗ cache miss: args changed, executing
 world
 

crates/vite_task_bin/tests/e2e_snapshots/fixtures/builtin-different-cwd/snapshots.toml

@@ -1,9 +1,11 @@
+# Tests that builtin commands have separate cache per cwd
+
 [[e2e]]
 name = "builtin different cwd"
 steps = [
-  "cd folder1 && vite lint",
-  "cd folder2 && vite lint",
-  "echo 'console.log(1);' > folder2/a.js",
-  "cd folder1 && vite lint",
-  "cd folder2 && vite lint",
+  "cd folder1 && vite lint # cache miss in folder1",
+  "cd folder2 && vite lint # cache miss in folder2",
+  "echo 'console.log(1);' > folder2/a.js # modify folder2",
+  "cd folder1 && vite lint # cache hit",
+  "cd folder2 && vite lint # cache miss",
 ]

crates/vite_task_bin/tests/e2e_snapshots/fixtures/builtin-different-cwd/snapshots/builtin different cwd.snap

@@ -4,7 +4,7 @@ assertion_line: 203
 expression: e2e_outputs
 input_file: crates/vite_task_bin/tests/e2e_snapshots/fixtures/builtin-different-cwd
 ---
-> cd folder1 && vite lint
+> cd folder1 && vite lint # cache miss in folder1
 
   ! eslint-plugin-unicorn(no-empty-file): Empty files are not allowed.
    ,-[a.js:1:1]
@@ -17,7 +17,7 @@ Found 1 warning and 0 errors.
 Finished in <duration> on 1 file with 90 rules using <n> threads.
 
 
-> cd folder2 && vite lint
+> cd folder2 && vite lint # cache miss in folder2
 
   ! eslint-plugin-unicorn(no-empty-file): Empty files are not allowed.
    ,-[a.js:1:1]
@@ -30,9 +30,9 @@ Found 1 warning and 0 errors.
 Finished in <duration> on 1 file with 90 rules using <n> threads.
 
 
-> echo 'console.log(1);' > folder2/a.js
+> echo 'console.log(1);' > folder2/a.js # modify folder2
 
-> cd folder1 && vite lint
+> cd folder1 && vite lint # cache hit
 ✓ cache hit, replaying
 
   ! eslint-plugin-unicorn(no-empty-file): Empty files are not allowed.
@@ -46,7 +46,7 @@ Found 1 warning and 0 errors.
 Finished in <duration> on 1 file with 90 rules using <n> threads.
 
 
-> cd folder2 && vite lint
+> cd folder2 && vite lint # cache miss
 ✗ cache miss: content of input 'folder2/a.js' changed, executing
 Found 0 warnings and 0 errors.
 Finished in <duration> on 1 file with 90 rules using <n> threads.

crates/vite_task_bin/tests/e2e_snapshots/fixtures/cache-disabled/snapshots.toml

@@ -1,13 +1,15 @@
+# Tests that cache: false in task config disables caching
+
 [[e2e]]
 name = "task with cache disabled"
 steps = [
-  "vite run no-cache-task",
-  "vite run no-cache-task",
+  "vite run no-cache-task # cache miss",
+  "vite run no-cache-task # cache disabled, runs again",
 ]
 
 [[e2e]]
 name = "task with cache enabled"
 steps = [
-  "vite run cached-task",
-  "vite run cached-task",
+  "vite run cached-task # cache miss",
+  "vite run cached-task # cache hit",
 ]

crates/vite_task_bin/tests/e2e_snapshots/fixtures/cache-disabled/snapshots/task with cache disabled.snap

@@ -1,9 +1,10 @@
 ---
-source: crates/vite_task_bin/tests/test_snapshots/main.rs
+source: crates/vite_task_bin/tests/e2e_snapshots/main.rs
+assertion_line: 203
 expression: e2e_outputs
-input_file: crates/vite_task_bin/tests/test_snapshots/fixtures/cache-disabled
+input_file: crates/vite_task_bin/tests/e2e_snapshots/fixtures/cache-disabled
 ---
-> vite run no-cache-task
+> vite run no-cache-task # cache miss
 $ print-file test.txt ⊘ cache disabled: no cache config
 test content
 
@@ -21,7 +22,7 @@ Task Details:
       → Cache disabled in task configuration
 ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 
-> vite run no-cache-task
+> vite run no-cache-task # cache disabled, runs again
 $ print-file test.txt ⊘ cache disabled: no cache config
 test content
 

crates/vite_task_bin/tests/e2e_snapshots/fixtures/cache-disabled/snapshots/task with cache enabled.snap

@@ -1,9 +1,10 @@
 ---
-source: crates/vite_task_bin/tests/test_snapshots/main.rs
+source: crates/vite_task_bin/tests/e2e_snapshots/main.rs
+assertion_line: 203
 expression: e2e_outputs
-input_file: crates/vite_task_bin/tests/test_snapshots/fixtures/cache-disabled
+input_file: crates/vite_task_bin/tests/e2e_snapshots/fixtures/cache-disabled
 ---
-> vite run cached-task
+> vite run cached-task # cache miss
 $ print-file test.txt
 test content
 
@@ -21,7 +22,7 @@ Task Details:
       → Cache miss: no previous cache entry found
 ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 
-> vite run cached-task
+> vite run cached-task # cache hit
 $ print-file test.txt ✓ cache hit, replaying
 test content
 

crates/vite_task_bin/tests/e2e_snapshots/fixtures/cache-miss-command-change/snapshots.toml

@@ -1,9 +1,11 @@
+# Tests cache behavior when command changes partially
+
 [[e2e]]
 name = "cache miss command change"
 steps = [
-  "vite run task",
-  "json-edit package.json '_.scripts.task = \"print baz && print bar\"'",
-  "vite run task",
-  "json-edit package.json '_.scripts.task = \"print bar\"'",
-  "vite run task",
+  "vite run task # cache miss",
+  "json-edit package.json '_.scripts.task = \"print baz && print bar\"' # change first subtask",
+  "vite run task # first: cache miss, second: cache hit",
+  "json-edit package.json '_.scripts.task = \"print bar\"' # remove first subtask",
+  "vite run task # cache hit",
 ]

crates/vite_task_bin/tests/e2e_snapshots/fixtures/cache-miss-command-change/snapshots/cache miss command change.snap

@@ -4,7 +4,7 @@ assertion_line: 203
 expression: e2e_outputs
 input_file: crates/vite_task_bin/tests/e2e_snapshots/fixtures/cache-miss-command-change
 ---
-> vite run task
+> vite run task # cache miss
 $ print foo
 foo
 
@@ -28,9 +28,9 @@ Task Details:
       → Cache miss: no previous cache entry found
 ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 
-> json-edit package.json '_.scripts.task = "print baz && print bar"'
+> json-edit package.json '_.scripts.task = "print baz && print bar"' # change first subtask
 
-> vite run task
+> vite run task # first: cache miss, second: cache hit
 $ print baz ✗ cache miss: args changed, executing
 baz
 
@@ -54,9 +54,9 @@ Task Details:
       → Cache hit - output replayed - <duration> saved
 ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 
-> json-edit package.json '_.scripts.task = "print bar"'
+> json-edit package.json '_.scripts.task = "print bar"' # remove first subtask
 
-> vite run task
+> vite run task # cache hit
 $ print bar ✓ cache hit, replaying
 bar
 

crates/vite_task_bin/tests/e2e_snapshots/fixtures/cache-miss-reasons/snapshots.toml

@@ -3,71 +3,55 @@
 [[e2e]]
 name = "env value changed"
 steps = [
-  # Initial run
-  "cross-env MY_ENV=1 vite run test",
-  # Cache miss: env value changed
-  "cross-env MY_ENV=2 vite run test",
+  "cross-env MY_ENV=1 vite run test # cache miss",
+  "cross-env MY_ENV=2 vite run test # cache miss: env value changed",
 ]
 
 [[e2e]]
 name = "env added"
 steps = [
-  # Initial run without env
-  "vite run test",
-  # Cache miss: env added
-  "cross-env MY_ENV=1 vite run test",
+  "vite run test # cache miss",
+  "cross-env MY_ENV=1 vite run test # cache miss: env added",
 ]
 
 [[e2e]]
 name = "env removed"
 steps = [
-  # Initial run with env
-  "cross-env MY_ENV=1 vite run test",
-  # Cache miss: env removed
-  "vite run test",
+  "cross-env MY_ENV=1 vite run test # cache miss",
+  "vite run test # cache miss: env removed",
 ]
 
 [[e2e]]
 name = "pass-through env added"
 steps = [
-  "vite run test",
-  # Add pass-through env
-  "json-edit vite.config.json \"_.tasks.test.passThroughEnvs = ['MY_PASSTHROUGH']\"",
-  # Cache miss: pass-through env added
-  "vite run test",
+  "vite run test # cache miss",
+  "json-edit vite.config.json \"_.tasks.test.passThroughEnvs = ['MY_PASSTHROUGH']\" # add pass-through env",
+  "vite run test # cache miss: pass-through env added",
 ]
 
 [[e2e]]
 name = "pass-through env removed"
 steps = [
-  # Setup: task with pass-through env
-  "json-edit vite.config.json \"_.tasks.test.passThroughEnvs = ['MY_PASSTHROUGH']\"",
-  "vite run test",
-  # Remove pass-through env
-  "json-edit vite.config.json \"delete _.tasks.test.passThroughEnvs\"",
-  # Cache miss: pass-through env removed
-  "vite run test",
+  "json-edit vite.config.json \"_.tasks.test.passThroughEnvs = ['MY_PASSTHROUGH']\" # setup",
+  "vite run test # cache miss",
+  "json-edit vite.config.json \"delete _.tasks.test.passThroughEnvs\" # remove pass-through env",
+  "vite run test # cache miss: pass-through env removed",
 ]
 
 [[e2e]]
 name = "cwd changed"
 steps = [
-  "vite run test",
-  # Create subfolder and copy test.txt there
+  "vite run test # cache miss",
   "mkdir -p subfolder",
   "cp test.txt subfolder/test.txt",
-  # Change cwd to subfolder
-  "json-edit vite.config.json \"_.tasks.test.cwd = 'subfolder'\"",
-  # Cache miss: working directory changed
-  "vite run test",
+  "json-edit vite.config.json \"_.tasks.test.cwd = 'subfolder'\" # change cwd",
+  "vite run test # cache miss: cwd changed",
 ]
 
 [[e2e]]
 name = "input content changed"
 steps = [
-  "vite run test",
-  # Modify input file
-  "replace-file-content test.txt initial modified",
-  # Cache miss: input content changed
-  "vite run test",
+  "vite run test # cache miss",
+  "replace-file-content test.txt initial modified # modify input",
+  "vite run test # cache miss: input changed",
 ]