dthree/vorpal

Hitting CTRL+C while in Inquirer prompt breaks everything

Open

#42 ouverte le 1 nov. 2015

Voir sur GitHub
 (16 commentaires) (0 réactions) (0 assignés)JavaScript (5 609 stars) (313 forks)batch import
bughelp wanted

Description

The output looks like this:

(^C again to quit)
function (str) {
  if (str === undefined) {
    return this._delimiter;
  }
  this._delimiter = String(str).trim() + ' ';
  if (this.isLocal()) {
    this.parent.ui.refresh();
  } else {
    this.parent._send('vantage-delimiter-downstream', 'downstream', {value: str, sessionId: this.id});
  }
  return this;
}
_

You can still type but nothing happens. Only way to recover is hitting CTRL+C again.

Guide contributeur

Stack technique
javascriptnodejs
Domaine
cli
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.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
half day
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
Basic JavaScriptNode.js CLI developmentUnderstanding of process signals
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50
Direction de recherche
Investigate how Vorpal sets up the Inquirer prompt and handles SIGINT signals. Look at the Inquirer library's behavior on Ctrl+C and how it interacts with Vorpal's internal state. The code snippet in the issue shows a function that might be from Vorpal's delimiter logic; check the refresh method and the send method. Consider implementing a proper cleanup or reset of the prompt state on Ctrl+C.