Merge remote-tracking branch 'origin/main' into runner-aware-tools

# Conflicts:
#	crates/vite_task_graph/src/config/mod.rs

Author: wan9chi

.github/workflows/ci.yml

@@ -68,7 +68,7 @@ jobs:
   test:
     needs: detect-changes
     if: needs.detect-changes.outputs.code-changed == 'true'
-    name: Test
+    name: Test (${{ matrix.shard }})
     strategy:
       fail-fast: false
       matrix:
@@ -77,18 +77,71 @@ jobs:
             target: x86_64-unknown-linux-gnu
             cargo_cmd: cargo-zigbuild
             build_target: x86_64-unknown-linux-gnu.2.17
-          - os: windows-latest
-            target: x86_64-pc-windows-msvc
-            cargo_cmd: cargo
-            build_target: x86_64-pc-windows-msvc
+            shard: linux-gnu
+            scope: ''
+            run_env: ''
           - os: namespace-profile-mac-default
             target: aarch64-apple-darwin
             cargo_cmd: cargo
             build_target: aarch64-apple-darwin
+            shard: macos-arm64
+            scope: ''
+            run_env: ''
           - os: namespace-profile-mac-default
             target: x86_64-apple-darwin
             cargo_cmd: cargo
             build_target: x86_64-apple-darwin
+            shard: macos-x64
+            scope: ''
+            run_env: ''
+          # Windows e2e fixtures dominate wall-clock (60s per PTY step vs 20s on
+          # Unix). Coverage is partitioned by crate: the e2e shards run
+          # `-p vite_task_bin` and the non-e2e shard runs
+          # `--workspace --exclude vite_task_bin`; the union is the workspace
+          # by construction. The e2e_snapshots harness self-shards via
+          # VT_SHARD_INDEX/VT_SHARD_TOTAL across the 5 e2e jobs.
+          - os: windows-latest
+            target: x86_64-pc-windows-msvc
+            cargo_cmd: cargo
+            build_target: x86_64-pc-windows-msvc
+            shard: windows-e2e-1
+            scope: '-p vite_task_bin'
+            run_env: 'VT_SHARD_INDEX=1 VT_SHARD_TOTAL=5'
+          - os: windows-latest
+            target: x86_64-pc-windows-msvc
+            cargo_cmd: cargo
+            build_target: x86_64-pc-windows-msvc
+            shard: windows-e2e-2
+            scope: '-p vite_task_bin'
+            run_env: 'VT_SHARD_INDEX=2 VT_SHARD_TOTAL=5'
+          - os: windows-latest
+            target: x86_64-pc-windows-msvc
+            cargo_cmd: cargo
+            build_target: x86_64-pc-windows-msvc
+            shard: windows-e2e-3
+            scope: '-p vite_task_bin'
+            run_env: 'VT_SHARD_INDEX=3 VT_SHARD_TOTAL=5'
+          - os: windows-latest
+            target: x86_64-pc-windows-msvc
+            cargo_cmd: cargo
+            build_target: x86_64-pc-windows-msvc
+            shard: windows-e2e-4
+            scope: '-p vite_task_bin'
+            run_env: 'VT_SHARD_INDEX=4 VT_SHARD_TOTAL=5'
+          - os: windows-latest
+            target: x86_64-pc-windows-msvc
+            cargo_cmd: cargo
+            build_target: x86_64-pc-windows-msvc
+            shard: windows-e2e-5
+            scope: '-p vite_task_bin'
+            run_env: 'VT_SHARD_INDEX=5 VT_SHARD_TOTAL=5'
+          - os: windows-latest
+            target: x86_64-pc-windows-msvc
+            cargo_cmd: cargo
+            build_target: x86_64-pc-windows-msvc
+            shard: windows-non-e2e
+            scope: '--workspace --exclude vite_task_bin'
+            run_env: ''
     runs-on: ${{ matrix.os }}
     steps:
       - uses: taiki-e/checkout-action@7d1e50e93dc4fb3bba58f85018fadf77898aee8b # v1.4.2
@@ -124,13 +177,13 @@ jobs:
         if: ${{ matrix.target == 'x86_64-unknown-linux-gnu' }}
 
       - name: Build tests
-        run: ${{ matrix.cargo_cmd }} test --no-run --target ${{ matrix.build_target }}
+        run: ${{ matrix.cargo_cmd }} test --no-run ${{ matrix.scope }} --target ${{ matrix.build_target }}
 
       # Default `cargo test` runs only tests that need nothing beyond the
       # Rust toolchain; this step verifies that contract before Node.js
       # and pnpm enter the picture.
       - name: Run tests
-        run: ${{ matrix.cargo_cmd }} test --target ${{ matrix.build_target }}
+        run: ${{ matrix.run_env }} ${{ matrix.cargo_cmd }} test ${{ matrix.scope }} --target ${{ matrix.build_target }}
 
       # x86_64-apple-darwin runs on arm64 runner under Rosetta; install x64 Node
       # so fspy's x86_64 preload dylib can be injected into spawned node procs.
@@ -139,7 +192,7 @@ jobs:
           architecture: ${{ matrix.target == 'x86_64-apple-darwin' && 'x64' || '' }}
 
       - name: Run ignored tests
-        run: ${{ matrix.cargo_cmd }} test --target ${{ matrix.build_target }} -- --ignored
+        run: ${{ matrix.run_env }} ${{ matrix.cargo_cmd }} test ${{ matrix.scope }} --target ${{ matrix.build_target }} -- --ignored
 
   test-musl:
     needs: detect-changes
@@ -215,7 +268,7 @@ jobs:
       - run: cargo fmt --check
       - run: RUSTDOCFLAGS='-D warnings' cargo doc --no-deps --document-private-items
 
-      - uses: crate-ci/typos@5374cbf686e897b15713110e233094e2874de7ef # v1.46.1
+      - uses: crate-ci/typos@aca895bf05aec0cb7dffa6f94495e923224d9f17 # v1.46.2
         with:
           files: .
 

CHANGELOG.md

@@ -1,5 +1,6 @@
 # Changelog
 
+- **Added** task command shorthands for defining tasks as command strings or command string arrays ([#391](https://github.com/voidzero-dev/vite-task/pull/391))
 - **Changed** Cached logs are stored with colors intact (`FORCE_COLOR=1` is auto-injected into spawned tasks). Colors are then stripped at display time when the terminal does not support them. Other color-related env vars (`NO_COLOR`, `COLORTERM`, `TERM`, `TERM_PROGRAM`) are no longer passed through by default. Opt in via a task's `env`/`untrackedEnv` ([#378](https://github.com/voidzero-dev/vite-task/pull/378))
 - **Added** `output` field for cached tasks: archives matching files after a successful run and restores them on cache hit ([#375](https://github.com/voidzero-dev/vite-task/pull/375))
 - **Fixed** Windows cached tasks can now run package shims rewritten through PowerShell; default env passthrough now preserves `PATHEXT` ([#366](https://github.com/voidzero-dev/vite-task/pull/366))

Cargo.lock

@@ -49,9 +49,7 @@ base64 = "0.22.1"
 wincode = "0.5.2"
 bindgen = "0.72.1"
 bitflags = "2.10.0"
-# The newest released version (0.3.0) of brush-parser has a bug that reports incorrect locations for some ast nodes.
-# The latest commit has fixed it. TODO: switch back to released version when a new version is published.
-brush-parser = { git = "https://github.com/reubeno/brush", rev = "dcb760933b10ee0433d7b740a5709b06f5c67c6b" }
+brush-parser = "0.4.0"
 bstr = { version = "1.12.0", default-features = false, features = ["alloc", "std"] }
 bumpalo = { version = "3.17.0", features = ["allocator-api2"] }
 bytemuck = { version = "1.23.0", features = ["extern_crate_alloc", "must_cast"] }

Cargo.toml

@@ -1,7 +1,7 @@
 use std::{collections::BTreeMap, fmt::Display, ops::Range};
 
 use brush_parser::{
-    Parser, ParserOptions,
+    Parser, ParserImpl, ParserOptions,
     ast::{
         AndOr, Assignment, AssignmentName, AssignmentValue, Command, CommandPrefix,
         CommandPrefixOrSuffixItem, CommandSuffix, CompoundListItem, Pipeline, Program,
@@ -47,7 +47,9 @@ const PARSER_OPTIONS: ParserOptions = ParserOptions {
     enable_extended_globbing: false,
     posix_mode: true,
     sh_mode: true,
-    tilde_expansion: false,
+    tilde_expansion_at_word_start: false,
+    tilde_expansion_after_colon: false,
+    parser_impl: ParserImpl::Peg,
 };
 
 /// Remove shell quoting from a word value, respecting quoting context.