Hacktoberfest 2026 : les issues que les mainteneurs ont marquées pour octobre, ouvertes et accessibles aux débutants. Parcourir les issues Hacktoberfest

[bug] update: exits 0 after the upgrade actually failed

Ouverte Adaptée aux débutants
#242 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

É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
node.js, typescript
Domaine
cli

Piste de recherche

Commencez par le chemin d’échec de bl update qui affiche « Automatic update failed » ; l’issue ne nomme aucun fichier source ni aucun test. Suivez la manière dont cette branche gère l’échec de npm et le code de sortie du processus, puis exécutez les tests de mise à jour concernés ou reproduisez le problème avec un préfixe global npm en lecture seule. Le résultat attendu est qu’une mise à niveau échouée renvoie un code différent de zéro, tandis qu’une mise à niveau réussie reste réussie.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

Environment

  • CLI: bl 2.0.1 at the time of failure (Skill: 2.0.1)
  • Node: v22.23.3
  • OS: Linux 5.15.0-198-generic x86_64 (Ubuntu)
  • Install channel: npm global (npm prefix -g → /home/lin/.npm-global)
  • Region: cn (domestic)

Reproduce

  1. Install bl globally via npm.
  2. Make the npm global prefix read-only (here: a read-only filesystem sandbox).
  3. With a newer version available (2.0.1 → 2.1.0), run:
bl update; echo "exit=$?"

Expected

A failed upgrade exits non-zero.

Actual

The upgrade failed with EROFS, the CLI printed Automatic update failed. Please run manually: — and still exited 0.

Any driver that checks only the exit code (CI, cron, agent) will record the
upgrade as successful while the old version is still installed. In this case the
caller believed 2.0.1 was the latest until it re-read bl --version.

Full output

Current version: 2.0.1
Install method: npm
Checking for updates...
Latest version: 2.1.0

Updating bailian-cli via npm...

npm error code EROFS
npm error syscall rename
npm error path /home/lin/.npm-global/lib/node_modules/bailian-cli
npm error dest /home/lin/.npm-global/lib/node_modules/.bailian-cli-RC0fMVNr
npm error errno -30
npm error rofs EROFS: read-only file system, rename '/home/lin/.npm-global/lib/node_modules/bailian-cli' -> '/home/lin/.npm-global/lib/node_modules/.bailian-cli-RC0fMVNr'
npm error rofs Often virtualized file systems, or other file systems
npm error rofs that don't support symlinks, give this error.
npm error A complete log of this run can be found in: <npm debug log>

Automatic update failed. Please run manually:
  npm install -g bailian-cli@latest
exit=0

Already tried

  • Confirmed the exit code with both echo $? and ${PIPESTATUS[0]} — both 0
  • The failure is not shell-related: bl update itself returns 0
  • Same behaviour regardless of whether npm's cache/logs directories are writable
  • After granting write access, the retry succeeded (2.0.1 → 2.1.0) and printed
    the skill-update result normally — so only the failure branch reports success

Notes

  • Frequency: always on the failure path
  • Related: #229 reports --config <profile> also exiting 0 on failure
    (exits 0 unauthenticated). Same class of defect: the process exit code does
    not reflect the outcome, and the CLI's own documented contract treats exit 0 as
    SUCCESS.
  • Suggested fix: propagate npm's exit code from the bl update failure branch
    (or return a dedicated non-zero code), so bl update && ... and agent drivers
    can detect the failure.
Langage dominant
TypeScript
Étoiles
540
Forks
35
Merge moyen
6 h 43 min
PR mergées (30 j)
23

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de modelstudioai/cli

Toutes les issues de modelstudioai/cli

Issues similaires

Plus d'issues TypeScript

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.