Hacktoberfest 2026: the issues maintainers tagged for October, open and beginner-friendly. Browse Hacktoberfest issues

fix(migrate): tolerate nested npm overrides during bootstrap detection

Open
#2,002 0 comments 0 reactions 1 assignee View on GitHub

@fengmk2 is already working on this.

Since Jul 1, 2026.

Assessment

This issue has not been assessed yet.

Description

bug

detectVitePlusBootstrapPending assumes npm overrides values are strings, but npm supports nested override objects. A valid package.json such as this can throw during migration because the bootstrap detection calls string methods like .startsWith() on the override value:

{
  "overrides": {
    "vite": {
      "rollup": "..."
    }
  }
}

Expected behavior:

  • Migration should not crash when overrides.vite or overrides.vitest is a nested object.
  • Non-string override values should be treated as not satisfying the Vite+ override and then handled safely by the migration rewrite logic.
  • Add a regression test covering a nested npm override under a managed key.

Suggested test:

Create an npm fixture with nested overrides:

{
  "devDependencies": {
    "vite-plus": "0.1.x"
  },
  "overrides": {
    "vite": {
      "rollup": "^4.0.0"
    }
  }
}

Run:

vp migrate --no-interactive

Assert that migration completes or reports an actionable migration error, but does not throw a TypeError from bootstrap detection.

Dominant language
Rust
Stars
5.8k
Forks
263
Avg merge
1d 4h
Merged PRs (30d)
148

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from voidzero-dev/vite-plus

All issues in voidzero-dev/vite-plus

Similar issues

More Rust issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.