[cylewaitforit]

🔗 Linked issue

closes #3378

🧭 Context

This Pr migrates vitest imports to use vite-plus/test.

📚 Description

This Pr migrates vitest imports to use vite-plus/test as recommending in the vite-plus migration guide. #3377

This required patching @nuxt/test-utils to work with vite-plus/test. (This should be supported directly in the future version @nuxt/test-utils@4.4.0 nuxt/test-utils#1862 . Thanks to @danielroe 's nuxt/test-utils#1861)

It also includes a lint change to attempt to catch any usage of the previous pattern.

As stated by the vite-plus migration guide this can be helpful in reducing the number of direct imports required by a project.

[agentscanapp]

Thanks for opening this pull request! 🎉

We really appreciate you taking the time to contribute, @cylewaitforit.

A maintainer will take a look as soon as they can. In the meantime, please make sure that:

• the description explains what changed and why
• any related issues are linked
• existing tests still pass

If anything needs adjusting we'll leave comments here. Thanks again!

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated
npmx.dev	Ready	Preview	Oct 31, 2026 5:24pm UTC

2 Skipped Deployments

Project	Deployment	Actions	Updated
docs.npmx.dev	Ignored	Preview	Oct 31, 2026 5:24pm UTC
npmx-lunaria	Ignored		Oct 31, 2026 5:24pm UTC

[coderabbitai]

No actionable comments were generated in the recent review.

Recent review info

Run configuration

• Source mode: Auto
• Repository: npmx-dev/npmx.dev
• Branch: main
• File: .coderabbit.yaml
• Commit: 87724e6
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: bc705664-b3b0-4d0b-a864-b4da916369e0

Commits

Reviewing files that changed from the base of the PR and between d3f4931 and a1c4177.

Files ignored due to path filters: 1

• pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml

Files selected for processing: 2

• CONTRIBUTING.md
• vite.config.ts

Included review availability: This review used your included allowance. Your plan provides up to 8 included reviews per hour; 4 remain after this review.

Summary

Summary by CodeRabbit

• Chores
  • Updated the test suite to use vite-plus/test and removed the separate Vitest development dependency.
  • Added a lint check to prevent tests from importing directly from Vitest.
  • Improved compatibility with vite-plus/test in the Nuxt test setup.

The test suite now imports test APIs from vite-plus/test instead of vitest. The direct vitest development dependency is removed. The Nuxt mock plugin recognises the new import source, and the lint configuration rejects direct vitest imports.

Changes: Vite Plus test import migration

Layer / File(s)	Summary
Dependency and Nuxt mock updates package.json, patches/@nuxt__test-utils@4.1.0.patch	The direct vitest development dependency is removed. The Nuxt mock plugin recognises imports from vite-plus/test when detecting vi imports.
Test import updates test/nuxt/**/*.spec.ts, test/unit/**/*.spec.ts	Test utilities now come from vite-plus/test. Test cases and assertions are otherwise unchanged.
Lint rule and test documentation vite.config.ts, CONTRIBUTING.md	The lint configuration rejects imports from vitest and its subpaths. The unit-test example uses vite-plus/test.

Possibly related PRs

• npmx-dev/npmx.dev#2717: This PR adds Vitest tests in files whose test imports now use vite-plus/test.

Suggested reviewers: fengmk2

Merge Risk: Minimal · up to a1c41

The lint restriction and updated test example present no established merge-blocking issue. Merge after normal checks pass.

Pre-merge checks: 4

Check name	Status	Explanation
Title check		The title clearly and concisely describes the main change: migrating Vitest imports to vite-plus/test.
Description check		The description directly explains the migration, the @nuxt/test-utils compatibility patch, and the lint rule update. It is related to the changeset.
Linked Issues check		Issue #3378 requires migration from direct vitest imports to vite-plus/test. The reviewed changes update the repository test files and the CONTRIBUTING.md example. They remove the direct `vitest…
Out of Scope Changes check		The changes remain within issue #3378. Test import updates, the dependency removal, lint protection, documentation update, and the @nuxt/test-utils compatibility patch all support the import migrati…

Full details: Linked Issues check

Explanation

Issue #3378 requires migration from direct vitest imports to vite-plus/test. The reviewed changes update the repository test files and the CONTRIBUTING.md example. They remove the direct vitest development dependency and add lint errors for vitest and vitest/* imports. The @nuxt/test-utils patch also recognises vite-plus/test, which supports the migration. No other coding requirements are stated.

Full details: Out of Scope Changes check

Explanation

The changes remain within issue #3378. Test import updates, the dependency removal, lint protection, documentation update, and the @nuxt/test-utils compatibility patch all support the import migration. The summary does not establish an unrelated feature or behaviour change.

Finishing Touches

Generate unit tests (beta)

• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

• X
• Mastodon
• Reddit
• LinkedIn

[github-actions]

e18e dependency analysis

No dependency warnings found.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

[trueberryless]

Thanks for the PR, @cylewaitforit !

Everything looks good. I added one commit that also changes the CONTRIBUTING.md file and applies the new lint rule to the whole repo, not just test/.