Merge branch 'main' into fix/rename-too-many-args

Author: yangshun

.claude/skills/update-changelog.md

@@ -0,0 +1,41 @@
+---
+description: Update CHANGELOG.md with a new entry for the current change
+user_invocable: true
+---
+
+# Update Changelog
+
+Add a new entry to `CHANGELOG.md` for the change being made in the current branch.
+
+## Instructions
+
+1. Read `CHANGELOG.md` to understand the current format.
+2. Determine the appropriate category for the change:
+   - **Added** — new user-facing feature or capability
+   - **Changed** — modification to existing user-facing behavior
+   - **Removed** — removal of user-facing feature or option
+   - **Fixed** — bug fix affecting users
+   - **Perf** — performance improvement noticeable to users
+3. Write a concise, user-facing description. Focus on what changed from the user's perspective, not implementation details.
+4. Include a PR link in the format `([#NNN](https://github.com/voidzero-dev/vite-task/pull/NNN))`. If the PR number is not yet known, leave a `([#???](https://github.com/voidzero-dev/vite-task/pull/???))` placeholder.
+5. Insert the new entry at the **top** of the existing list in `CHANGELOG.md` (newest first).
+6. If the current change is closely related to an existing entry (e.g., multiple PRs contributing to the same feature or fix), group them into a single item with multiple PR links rather than adding a separate entry.
+
+## What NOT to include
+
+Do not add entries for:
+
+- Internal refactors with no user-facing effect
+- CI/CD changes
+- Dependency bumps
+- Test-only fixes (flaky tests, test infrastructure)
+- Documentation changes (CLAUDE.md, README, etc.)
+- Chore/tooling changes
+
+The changelog is for **end-users only**.
+
+## Entry format
+
+```
+- **Category** description ([#NNN](https://github.com/voidzero-dev/vite-task/pull/NNN))
+```

.github/workflows/ci.yml

@@ -149,9 +149,6 @@ jobs:
       # -crt-static: vite-task is shipped as a NAPI module in vite+, and musl Node
       # with native modules links to musl libc dynamically, so we must do the same.
       RUSTFLAGS: --cfg tokio_unstable -D warnings -C target-feature=-crt-static
-      # On musl, concurrent PTY operations can trigger SIGSEGV in musl internals.
-      # Run test threads sequentially to avoid the race.
-      RUST_TEST_THREADS: 1
     steps:
       - name: Install Alpine dependencies
         shell: sh {0}
@@ -175,7 +172,14 @@ jobs:
           corepack enable
           pnpm install
 
-      - run: cargo test
+      # Use cargo-nextest: it runs each test in its own process, avoiding
+      # musl's fork()-in-multithreaded-process SIGSEGV while keeping parallel
+      # execution (across processes instead of threads within one process).
+      # Exclude packages with custom test harnesses (harness = false) since
+      # nextest can't discover their tests; run those with cargo test instead.
+      - run: cargo install cargo-nextest --locked
+      - run: cargo nextest run --workspace --exclude vite_task_bin --exclude vite_task_plan
+      - run: cargo test -p vite_task_bin -p vite_task_plan
 
   fmt:
     name: Format and Check Deps

CHANGELOG.md

@@ -0,0 +1,7 @@
+# Changelog
+
+- **Changed** default untracked env patterns to align with Turborepo, covering more CI and platform-specific variables ([#262](https://github.com/voidzero-dev/vite-task/pull/262))
+- **Added** `--log=interleaved|labeled|grouped` flag to control task output display: `interleaved` (default) streams directly, `labeled` prefixes lines with `[pkg#task]`, `grouped` buffers output per task ([#266](https://github.com/voidzero-dev/vite-task/pull/266))
+- **Added** musl target support (`x86_64-unknown-linux-musl`) ([#273](https://github.com/voidzero-dev/vite-task/pull/273))
+- **Changed** cache hit/miss indicators to use neutral symbols (◉/〇) instead of ✓/✗ to avoid confusion with success/error ([#268](https://github.com/voidzero-dev/vite-task/pull/268))
+- **Added** automatic skip of caching for tasks that modify their own inputs ([#248](https://github.com/voidzero-dev/vite-task/pull/248))

CLAUDE.md

@@ -135,6 +135,10 @@ All code must work on both Unix and Windows without platform skipping:
 - Platform differences should be handled gracefully, not skipped
 - After major changes to `fspy*` or platform-specific crates, run `just lint-linux` and `just lint-windows`
 
+## Changelog
+
+When a change is user-facing (new feature, changed behavior, bug fix, removal, or perf improvement), run `/update-changelog` to add an entry to `CHANGELOG.md`. Do not add entries for internal refactors, CI, dep bumps, test fixes, or docs changes.
+
 ## Quick Reference
 
 - **Task Format**: `package#task` (e.g., `app#build`, `@test/utils#lint`)

crates/vite_task/src/session/execute/fingerprint.rs

@@ -6,8 +6,7 @@
 use std::{
     collections::BTreeMap,
     fs::File,
-    hash::Hasher as _,
-    io::{self, BufRead, Read},
+    io::{self, BufRead},
     sync::Arc,
 };
 
@@ -182,20 +181,6 @@ fn determine_folder_change_kind<'a>(
     }
 }
 
-/// Hash file content using `xxHash3_64`
-fn hash_content(mut stream: impl Read) -> io::Result<u64> {
-    let mut hasher = twox_hash::XxHash3_64::default();
-    let mut buf = [0u8; 8192];
-    loop {
-        let n = stream.read(&mut buf)?;
-        if n == 0 {
-            break;
-        }
-        hasher.write(&buf[..n]);
-    }
-    Ok(hasher.finish())
-}
-
 /// Check if a directory entry should be ignored in fingerprinting
 fn should_ignore_entry(name: &[u8]) -> bool {
     matches!(name, b"." | b".." | b".DS_Store") || name.eq_ignore_ascii_case(b"dist")
@@ -251,7 +236,7 @@ pub fn fingerprint_path(
             return process_directory(std_path, path_read);
         }
     }
-    Ok(PathFingerprint::FileContentHash(hash_content(reader)?))
+    Ok(PathFingerprint::FileContentHash(super::hash::hash_content(reader)?))
 }
 
 /// Process a directory on Windows using `std::fs::read_dir`

crates/vite_task/src/session/execute/glob_inputs.rs

@@ -5,12 +5,7 @@
 //!
 //! All glob patterns are workspace-root-relative (resolved at task graph stage).
 
-use std::{
-    collections::BTreeMap,
-    fs::File,
-    hash::Hasher as _,
-    io::{self, Read},
-};
+use std::{collections::BTreeMap, fs::File, io};
 
 #[cfg(test)]
 use vite_path::AbsolutePathBuf;
@@ -114,21 +109,9 @@ pub fn compute_globbed_inputs(
     Ok(result)
 }
 
-/// Hash file content using `xxHash3_64`.
 #[expect(clippy::disallowed_types, reason = "receives std::path::Path from wax glob walker")]
 fn hash_file_content(path: &std::path::Path) -> io::Result<u64> {
-    let file = File::open(path)?;
-    let mut reader = io::BufReader::new(file);
-    let mut hasher = twox_hash::XxHash3_64::default();
-    let mut buf = [0u8; 8192];
-    loop {
-        let n = reader.read(&mut buf)?;
-        if n == 0 {
-            break;
-        }
-        hasher.write(&buf[..n]);
-    }
-    Ok(hasher.finish())
+    super::hash::hash_content(io::BufReader::new(File::open(path)?))
 }
 
 #[cfg(test)]

crates/vite_task/src/session/execute/hash.rs

@@ -0,0 +1,15 @@
+use std::{hash::Hasher as _, io};
+
+/// Hash content using 8 KiB buffered `xxHash3_64`.
+pub(super) fn hash_content(mut stream: impl io::Read) -> io::Result<u64> {
+    let mut hasher = twox_hash::XxHash3_64::default();
+    let mut buf = [0u8; 8192];
+    loop {
+        let n = stream.read(&mut buf)?;
+        if n == 0 {
+            break;
+        }
+        hasher.write(&buf[..n]);
+    }
+    Ok(hasher.finish())
+}

crates/vite_task/src/session/execute/mod.rs

@@ -1,5 +1,6 @@
 pub mod fingerprint;
 pub mod glob_inputs;
+mod hash;
 pub mod spawn;
 
 use std::{collections::BTreeMap, io::Write as _, process::Stdio, sync::Arc};