ci(test): restore VT_SKIP_E2E so non-e2e scope is --workspace

Reverts the previous switch to --exclude vite_task_bin. That exclusion
happened to be equivalent today (vite_task_bin has no lib/bin/doc
tests), but it isn't verifiable from the scope alone: adding a lib test
to vite_task_bin later would silently drop it from CI.

The non-e2e Windows shard now scopes to the full workspace and the e2e
harness self-skips when VT_SKIP_E2E is set. Coverage is now legible from
the scopes: e2e_snapshots is sharded across 5 jobs, --workspace catches
everything else (current and future).

Author: claude

.github/workflows/ci.yml

@@ -96,8 +96,9 @@ jobs:
             run_env: ''
           # Windows e2e fixtures dominate wall-clock (60s per PTY step vs 20s on
           # Unix). Sharded 5 ways at the trial level by the e2e harness via
-          # VT_SHARD_INDEX/VT_SHARD_TOTAL; the non-e2e shard excludes the
-          # vite_task_bin crate (which has no lib tests) and runs everything else.
+          # VT_SHARD_INDEX/VT_SHARD_TOTAL. The non-e2e shard uses --workspace
+          # (self-evidently covers everything) and relies on VT_SKIP_E2E in the
+          # e2e harness to suppress the already-sharded work.
           - os: windows-latest
             target: x86_64-pc-windows-msvc
             cargo_cmd: cargo
@@ -138,8 +139,8 @@ jobs:
             cargo_cmd: cargo
             build_target: x86_64-pc-windows-msvc
             shard: windows-non-e2e
-            scope: '--workspace --exclude vite_task_bin'
-            run_env: ''
+            scope: ''
+            run_env: 'VT_SKIP_E2E=1'
     runs-on: ${{ matrix.os }}
     steps:
       - uses: taiki-e/checkout-action@7d1e50e93dc4fb3bba58f85018fadf77898aee8b # v1.4.2

crates/vite_task_bin/tests/e2e_snapshots/main.rs

@@ -571,6 +571,13 @@ fn parse_shard_env() -> Option<(usize, usize)> {
 
 #[expect(clippy::disallowed_types, reason = "Path required for CARGO_MANIFEST_DIR path traversal")]
 fn main() {
+    // Bypass for the non-e2e Windows shard: cargo still launches this binary
+    // as part of `cargo test --workspace`, but the e2e cases are sharded by
+    // the windows-e2e-* jobs, so we exit immediately.
+    if std::env::var_os("VT_SKIP_E2E").is_some() {
+        return;
+    }
+
     let tmp_dir = tempfile::tempdir().unwrap();
     let tmp_dir_path = AbsolutePathBuf::new(tmp_dir.path().canonicalize().unwrap()).unwrap();