facebookarchive/prepack

Bad command-line crashes Prepack

Open

#2 509 ouverte le 30 août 2018

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)JavaScript (14 268 stars) (520 forks)batch import
bughelp wantedlevel 1 (super easy)priority: low

Description

This triggers an invariant violation.

node lib/prepack-cli.js --debugDiagnosticSeverity foo

It shouldn't. The command-line argument processing should rather print a user-friendly error message in such cases and exit, like it already happens for most or all other bad arguments.

Guide contributeur

Stack technique
javascriptnodejs
Domaine
clideveloper experience
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
JavaScriptNode.jsbasic CLI knowledge
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65
Direction de recherche
Identify where command line arguments are parsed in lib/prepack cli.js or similar. Determine how other invalid arguments produce user friendly errors. Find the specific invariant violation triggered by bad debugDiagnosticSeverity and replace it with a proper error message and exit.