Skip to content

fix: remove forks pool to fix vitest onTaskUpdate timeout#13

Merged
eugenioenko merged 3 commits into
mainfrom
fix-vitest-timeout
May 28, 2026
Merged

fix: remove forks pool to fix vitest onTaskUpdate timeout#13
eugenioenko merged 3 commits into
mainfrom
fix-vitest-timeout

Conversation

@eugenioenko
Copy link
Copy Markdown
Owner

Summary

  • Remove pool: "forks" from vitest config, reverting to the default threads pool
  • The forks pool has a hardcoded 60s birpc RPC timeout that fires when a single test file runs longer than 60s (LSP tests take ~74s in CI)
  • The threads pool doesn't have this issue since communication happens in-process

Test plan

  • All 84 functional tests pass locally with no onTaskUpdate error
  • CI passes without the unhandled error

🤖 Generated with Claude Code

eugenioenko and others added 3 commits May 28, 2026 11:04
The forks pool causes birpc onTaskUpdate timeouts when test suites
run longer than 60s. The default threads pool handles this correctly.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
vitest 3.x had a hardcoded 60s birpc timeout that caused false failures
when test files ran longer than 60s. Fixed upstream in vitest PR #8297.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
clangd takes longer to initialize on CI, causing diagnostics and
signature help tests to timeout at 5s. Increased default to 10s and
added waitForLog("lsp initialized") before checking diagnostics.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@eugenioenko eugenioenko merged commit cfa7bfb into main May 28, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant