Microsoft/TypeScript

Make private field name parsing ecma 262 compliant

Open

#50.883 aberto em 21 de set. de 2022

Ver no GitHub
 (1 comment) (1 reaction) (1 assignee)TypeScript (6.726 forks)batch import
BugDomain: ParserEffort: ModerateHelp WantedRescheduled

Métricas do repositório

Stars
 (48.455 stars)
Métricas de merge de PR
 (Mesclagem média 6d 17h) (9 fundiu PRs em 30d)

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

Guia do colaborador

Direção de pesquisa
Investigue o arquivo test262 referenciado (https://github.com/tc39/test262/blob/6f8e0f67cad29a92a23ed7b596a9be8d7ba34644/test/language/expressions/class/elements/after same line static async method rs privatename identifier alt.js) para entender a análise esperada de nomes de campos privados com caracteres de não junção de largura zero (U+200C e U+200D). Considere o PR #50856 que adicionou suporte a escapes Unicode estendidos, e analise por que converter escapes Unicode estendidos para regulares não resolve os erros. Examine o código do analisador sintático do TypeScript para análise de elementos de classe para alinhar com a especificação ECMAScript 262, focando em como o modo ES5 afeta o tratamento de escapes Unicode estendidos.
Pilha de tecnologia
typescript
Domain
tooling
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
GitTypeScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75