fix(migrate): tolerate nested npm overrides during bootstrap detection
Les mainteneurs répondent en général sous 1 jour
Évaluation
Cette issue n'a pas encore été évaluée.
Description
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.viteoroverrides.vitestis 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.
- Langage dominant
- Rust
- Étoiles
- 6k
- Forks
- 274
- Merge moyen
- 1 j 11 h
- PR mergées (30 j)
- 203
Préparer son environnement
Lance le conteneur de développement du projet dans votre navigateur, avec votre propre compte GitHub.
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de voidzero-dev/vite-plus
-
`npx --package=vite-plus vp create` fails with exit 127 when npm is the chosen package managerPeut-être pris @Crimson341 l’a pris il y a 2 jours. Ouvertebug
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
voidzero-dev/vite-plus#2970 ·
Les mainteneurs répondent en général sous 1 jour
-
pending triage
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
voidzero-dev/vite-plus#2882 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
voidzero-dev/vite-plus#2097 · 10 commentaires · 2 réactions ·
Les mainteneurs répondent en général sous 1 jour
-
vp treats a nested project as part of the parent's pnpm workspace, ignoring its own packageManager and lockfilePeut-être pris @liangmiQwQ l’a pris il y a 1 jour. Ouvertebug enhancement
voidzero-dev/vite-plus#3013 · 1 commentaire · 1 personne assignée ·
Les mainteneurs répondent en général sous 1 jour
-
Windows: PowerShell installer (1.1.0) does not add vite-plus\bin to user PATH — 'vp' not recognizedPeut-être pris @fengmk2 l’a pris il y a 1 jour. Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 30/100
voidzero-dev/vite-plus#2984 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de voidzero-dev/vite-plus
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
storytold/photocraft#5260 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
extendr/extendr.github.io#134 ·
-
awaiting-response bug
Difficulté 1/5 Moins d'une heure Accessibilité débutants 72/100
wildcard/caro#1655 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour