chore(showcase/scripts): bump vitest 3 → 4.1.3 to fix birpc onTaskUpdate timeout#4087
Merged
Conversation
jpr5
requested review from
marthakelly,
mme,
ranst91 and
tylerslaton
as code owners
April 19, 2026 01:58
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
Contributor
📣 Social Copy GeneratorGenerate social media copies (Twitter/X, LinkedIn, Blog Post) for this PR using Claude.
|
Fixes upstream birpc onTaskUpdate timeout (vitest-dev/vitest#8164, fixed by #8297, v4-only). Root package.json already on ^4.1.3; this aligns showcase/scripts, which held the only remaining ^3.0.0 pin and was therefore the only package affected by the bug. Local verification: 3 consecutive `pnpm nx run @copilotkit/showcase-scripts:test --skip-nx-cache` runs on Node 20.20.2, all green, 1061/1061 passing, zero onTaskUpdate errors, zero unhandled exceptions (wall times: 34s / 30s / 29s).
jpr5
force-pushed
the
chore/showcase-scripts-vitest-4
branch
from
April 19, 2026 02:10
b77cd5b to
aafa732
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
showcase/scriptstest runs intermittently fail with birpconTaskUpdatetimeouts, caused by an upstream bug in vitest 3.x's subprocess reporter transport (vitest-dev/vitest#8164). Fix landed upstream in #8297, shipped in v4.0.0-beta.4+.Why this is safe
package.jsonalready declares"vitest": "^4.1.3". The only holdout wasshowcase/scripts/package.jsonat"vitest": "^3.0.0". All 10 subprocess-spawning test files in the monorepo live inshowcase/scripts/__tests__/— zero elsewhere — so this is the only package actually affected by the bug.vi.mock, noexpect.extend, no.concurrent, no snapshots, no coverage config, no custom reporters. Onlyvi.spyOn(unchanged in v4) and standarddescribe/it/expect.pool: 'forks'+fileParallelismbehave identically in v3 and v4.showcase/scriptsimport resolution changes + a new@vitest/[email protected]snapshot. No other package was edited; no other package's resolution changed. vitest 3.2.4 and 2.1.9 continue to resolve side-by-side for the packages that still pin them.Verification
3 consecutive
pnpm nx run @copilotkit/showcase-scripts:test --skip-nx-cacheruns on Node 20.20.2:Zero
onTaskUpdateerrors, zero unhandled exceptions across all three runs.Test plan
Relationship to #4081
Complementary, not overlapping. #4081 splits / isolates the showcase-scripts test suite as a workaround at the Nx/pool-config layer. This PR fixes the root cause in the test runner itself. Once this merges, the
onTaskUpdatefailures #4081 has been chasing should clear; #4081 can then rebase on a clean main and its remaining work (the split/isolation improvements) can be evaluated on its own merits rather than as a bug workaround.