Microsoft/TypeScript

Make private field name parsing ecma 262 compliant

Open

#50 883 ouverte le 21 sept. 2022

Voir sur GitHub
 (1 commentaire) (1 réaction) (1 assigné)TypeScript (6 726 forks)batch import
BugDomain: ParserEffort: ModerateHelp WantedRescheduled

Métriques du dépôt

Stars
 (48 455 stars)
Métriques de merge PR
 (Merge moyen 6j 17h) (9 PRs mergées en 30 j)

Description

This file has parse errors, but probably shouldn't.

Considerations:

  • https://github.com/microsoft/TypeScript/pull/50856 added support for starting with a unicode escape
  • The names #ZW__NJ and #ZW__J contain \u200c and \u200d, respectively (not here, but in the file)
  • Support for extended unicode escapes (i.e. \u{6F}, rather than \u006F) depends on ES2015, but we appear to parse the file as ES5.
  • Converting the extended unicode escapes to regular unicode escapes does not seem to eliminate the errors

Discovered as part of https://github.com/microsoft/TypeScript/issues/50835

Guide contributeur

Direction de recherche
Examinez le fichier test262 référencé (https://github.com/tc39/test262/blob/6f8e0f67cad29a92a23ed7b596a9be8d7ba34644/test/language/expressions/class/elements/after same line static async method rs privatename identifier alt.js) pour comprendre l'analyse attendue des noms de champs privés avec des caractères de non jonction de largeur nulle (U+200C et U+200D). Considérez la PR #50856 qui a ajouté la prise en charge des séquences d'échappement Unicode étendues, et analysez pourquoi la conversion des séquences d'échappement Unicode étendues en séquences régulières ne résout pas les erreurs. Examinez le code de l'analyseur syntaxique de TypeScript pour l'analyse des éléments de classe afin de se conformer à la spécification ECMAScript 262, en se concentrant sur la façon dont le mode ES5 affecte le traitement des séquences d'échappement Unicode étendues.
Stack technique
typescript
Domaine
tooling
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 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.
Plutôt claire
Prérequis
GitTypeScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75