Microsoft/TypeScript

Make private field name parsing ecma 262 compliant

Open

#50.883 aperta il 21 set 2022

Vedi su GitHub
 (1 commento) (1 reazione) (1 assegnatario)TypeScript (6726 fork)batch import
BugDomain: ParserEffort: ModerateHelp WantedRescheduled

Metriche repository

Star
 (48.455 star)
Metriche merge PR
 (Merge medio 6g 17h) (9 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Esamina il file test262 di riferimento (https://github.com/tc39/test262/blob/6f8e0f67cad29a92a23ed7b596a9be8d7ba34644/test/language/expressions/class/elements/after same line static async method rs privatename identifier alt.js) per comprendere l'analisi prevista dei nomi di campo privati con caratteri di non join a larghezza zero (U+200C e U+200D). Considera la PR #50856 che ha aggiunto il supporto per le sequenze di escape Unicode estese, e analizza perché convertire le sequenze di escape Unicode estese in quelle regolari non risolve gli errori. Esamina il codice del parser di TypeScript per l'analisi degli elementi di classe per allinearsi alla specifica ECMAScript 262, concentrandosi su come la modalità ES5 influisce sulla gestione delle sequenze di escape Unicode estese.
Tech stack
typescript
Dominio
tooling
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
GitTypeScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75