Skip to content

feat(migrate): add warnings if certain packages and config files are found - #3275

Open
KTrain5169 wants to merge 8 commits into
voidzero-dev:mainfrom
KTrain5169:tool-recognisers
Open

KTrain5169 wants to merge 8 commits into
voidzero-dev:mainfrom
KTrain5169:tool-recognisers

Conversation

@KTrain5169

@KTrain5169 KTrain5169 commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor

vp migrate warns about rollup, rolldown, or jest dependencies with matching configurations during the first migration to Vite+. The warnings explain manual migration paths to Vite, tsdown, or Vitest.

Detection covers the project root and workspace packages, including shared dependencies declared at the root. It supports common configuration file extensions and the jest field in package.json. Each tool produces at most one warning per run.

Existing Vite+ projects show these warnings only with vp migrate --full. Ordinary vp migrate runs skip them. If no migration is needed, the command displays any requested warnings and exits.

Example output during a first migration or with vp migrate --full. This example has matching dependencies and configurations for all three tools:

! Warnings:
  - Rollup configuration detected. Rollup is API-compatible with Rolldown and can be migrated first before migrating further to tsdown or Vite. Once migrated to tsdown or Vite, re-run vp migrate.
  - Rolldown configuration detected. Rolldown can be migrated to tsdown or Vite, and it is recommended to do so. Once migrated, re-run vp migrate.
  - Jest configuration detected. Vitest is highly compatible with Jest, and so we recommend migrating. See the Vitest Jest migration guide: https://vitest.dev/guide/migration/jest, then re-run vp migrate.

Closes #2546

Comment thread packages/cli/src/migration/detector.ts
Comment thread packages/cli/src/migration/detector.ts Outdated
Comment thread packages/cli/src/migration/bin.ts Outdated
@fengmk2 fengmk2 self-assigned this Oct 30, 2026
@KTrain5169
KTrain5169 requested a review from fengmk2 October 30, 2026 06:30
@fengmk2 fengmk2 added test: e2e Auto run e2e tests test: create-e2e Run `vp create` e2e tests labels Oct 30, 2026

fengmk2 commented Oct 30, 2026

Copy link
Copy Markdown
Member

Updated in f9370a6. For existing Vite+ projects, tool migration warning detection now runs only when --full is set.

The Rollup, Rolldown, and Jest migration warnings still appear during the first migration to Vite+. Ordinary vp migrate reruns skip these warnings. vp migrate --full still shows them when matching dependencies and configurations are present.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test: create-e2e Run `vp create` e2e tests test: e2e Auto run e2e tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Warn on certain other tooling config files

2 participants