PowerShell `vp` wrapper hides failures: `$?` is `True` and `&&` keeps going after `vp` fails
Les mainteneurs répondent en général sous 1 jour
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 72/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- Active
- Stack technique
- powershell, rust, typescript
- Domaine
- cli
Piste de recherche
Commencez par lire la fonction vp dans env.ps1 et comparez son comportement en matière de statut dans PowerShell avec l’exécution directe de vp.exe. Reproduisez l’échec avec une commande qui fait échouer vp pack, puis vérifiez $LASTEXITCODE, $? et une chaîne &&. Le travail est terminé lorsque le wrapper signale l’échec via $? et empêche la chaîne de continuer.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Describe the bug
In PowerShell, the vp function defined in env.ps1 loses the exit status of the vp.exe binary. When a command fails, $LASTEXITCODE is set correctly, but $? is True.
Reproduction
see below
Steps to reproduce
example vite.config.ts:
export default defineConfig({
pack: {
entry: 'src/does-not-exist.js'
},
});
In powershell:
> vp pack
VITE+ - The Unified Toolchain for the Web
ℹ entry: src/does-not-exist.js
...
error: Build failed with 1 error:
[UNRESOLVED_ENTRY] Cannot resolve entry module src/fail.js.
at ...
> "$LastExitCode $?"
1 True
vp pack returns with exit code 1 (good), and with status True (bad).
running vp.exe directly does not have this issue:
> vp.exe pack
VITE+ - The Unified Toolchain for the Web
ℹ entry: src/does-not-exist.js
...
error: Build failed with 1 error:
[UNRESOLVED_ENTRY] Cannot resolve entry module src/fail.js.
at ...
> "$LastExitCode $?"
1 False
with vp.exe, $? correctly outputs False, which correctly breaks && chains for example
System Info
> vp env current
VITE+ - The Unified Toolchain for the Web
Node.js:
Version 26.11.0
Source devEngines.runtime
Bin Path C:\Users\me\AppData\Local\vite-plus\data\js_runtime\node\26.11.0\node.exe
Installed true
Mode managed
Package Manager:
Name npm
Version 12.2.0
Source devEngines.packageManager
Bin Paths
npm C:\Users\me\AppData\Local\vite-plus\data\package_manager\npm\12.2.0\npm\bin\npm.cmd
npx C:\Users\me\AppData\Local\vite-plus\data\package_manager\npm\12.2.0\npm\bin\npx.cmd
Installed true
Mode managed
> vp --version
VITE+ - The Unified Toolchain for the Web
vp v1.1.0
Local vite-plus:
vite-plus v1.1.0
Tools:
vite v8.3.3
rolldown v1.2.12
vitest v5.0.3
oxfmt v0.72.0
oxlint v1.87.0
oxlint-tsgolint v7.0.2003
tsdown v0.23.0
Environment:
Package manager npm ^12.2.0
Node.js v26.11.0 (devEngines.runtime)
Used Package Manager
npm
Logs
Validations
- Read the Contributing Guidelines.
- Check that there isn't already an issue for the same bug.
- Confirm this is a Vite+ issue and not an upstream issue (Vite, Vitest, tsdown, Rolldown, or Oxc).
- The provided reproduction is a minimal reproducible example.
- Langage dominant
- Rust
- Étoiles
- 6.5k
- Forks
- 296
- Merge moyen
- 1 j 13 h
- PR mergées (30 j)
- 232
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
-
Docker guide: the runtime stage needs libatomic1 for Node.js 25 and laterPeut-être pris @kshanxs l’a pris aujourd’hui. Ouvertedocumentation
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
voidzero-dev/vite-plus#3219 · 1 commentaire · 1 réaction ·
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
-
vp run -r executes each package task twice when the root package has a matching recursive scriptPeut-être pris @wan9chi l’a pris il y a 1 jour. Ouvertepending triage
voidzero-dev/vite-plus#3221 · 1 commentaire · 1 personne assignée ·
Les mainteneurs répondent en général sous 1 jour
-
pending triage
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
voidzero-dev/vite-plus#3218 ·
Les mainteneurs répondent en général sous 1 jour
-
LSP loads vite.config.ts with the server's cwd as rootPeut-être pris @leaysgur l’a pris il y a 2 jours. Ouvertepending triage
voidzero-dev/vite-plus#3217 · 1 personne assignée ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de voidzero-dev/vite-plus
Issues similaires
-
bug easy good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
lacs-project/sysknife#659 ·
Les mainteneurs répondent en général sous 2 jours
-
E_FIT_INPUT_MISSING (mixture overrides): the message says ferx-r bundles lack mixture_overridesOuverteP3
Difficulté 1/5 Moins d'une heure Accessibilité débutants 83/100
Les mainteneurs répondent en général sous 1 jour
-
Document multi-value claims with rowBoundaries operator:in (and their per-record application)Ouvertedocumentation
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
registrystack/registry-stack#2431 ·
Les mainteneurs répondent en général sous 1 jour
-
contribution
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
farion1231/cc-switch#8863 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour