Skip to content

fix(deps): make Vitest 5 available at the project root - #141

Merged
TamaraNocentini merged 1 commit into
mainfrom
fix/vitest-5-resolution
Oct 24, 2026
Merged

TamaraNocentini merged 1 commit into
mainfrom
fix/vitest-5-resolution

Conversation

@gambtho

@gambtho gambtho commented Oct 24, 2026

Copy link
Copy Markdown
Contributor

Picking up the Vitest 5 bump from #43. It ended up nested under Headlamp, so type-checking couldn't find vitest/globals and the test runner couldn't find the CLI.

This makes Vitest a direct dev dependency and points the override at it. Node 24 types satisfy its peer requirement; TypeScript stays on 5.9.3.

Checked a clean install, lint, types, all 123 tests, build, and formatting. /polish and the local CodeRabbit review came back clean.

AI-assisted.

@gambtho
gambtho requested a review from a team as a code owner October 24, 2026 13:31
Declare Vitest directly so the plugin's type checker and test runner can resolve it. Reference that version from the transitive override and add Node 24 types to satisfy Vitest 5's peer dependency.

Assisted-by: GPT-6.1
Signed-off-by: gambtho <Thomas.Gamble@microsoft.com>
@TamaraNocentini
TamaraNocentini force-pushed the fix/vitest-5-resolution branch from f875779 to ec5f0b7 Compare October 24, 2026 15:29
@TamaraNocentini
TamaraNocentini merged commit 8149d67 into main Oct 24, 2026
6 checks passed
@TamaraNocentini
TamaraNocentini deleted the fix/vitest-5-resolution branch October 24, 2026 15:41
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.

2 participants