repl: no RegExp side effects

Métriques du dépôt

Stars

117 218 stars (117 218 stars)

Métriques de merge PR

Merge moyen 18j 17h (Merge moyen 18j 17h)219 PRs mergées en 30 j (219 PRs mergées en 30 j)

Description

$ ./out/Release/node 
> RegExp.$_
'\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000'

Guide contributeur

Direction de recherche

Examinez le code source du REPL de Node.js, en particulier la fonction qui gère l'évaluation et le nettoyage des propriétés statiques de RegExp (RegExp.$ et $1 $9). Le bogue est causé par la tentative du REPL de préserver les effets de bord entre les évaluations. Cherchez dans lib/repl.js où ces propriétés sont définies ou effacées après chaque commande. Vérifiez s'il manque une étape de réinitialisation. De plus, examinez tout travail existant de l'assigné sur une branche, si disponible. La correction implique probablement de garantir que ces propriétés ne soient pas polluées entre les évaluations du REPL.

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.

3

Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.

1-3 heures

Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.

Active

ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.

Claire

Prérequis

JavaScriptNode.js basics

Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.

65