Microsoft/TypeScript

commonjs export binding produces invalid code for increment/decrement in PrefixUnaryExpression

Open

#35.879 aberto em 27 de dez. de 2019

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)TypeScript (6.726 forks)batch import
BugDomain: JS EmitHelp Wanted

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

TypeScript Version: 3.7.2

Search Terms: export increment decrement syntax error

Code

// @module: commonjs
let foo = 1;
export { foo };
while (!--foo) { }

Expected behavior:

while (!(exports.foo = --foo)) { }

Actual behavior:

Because the module-transform uses onSubstituteNode, it doesn't use the auto-parenthesizing that's done by the factory API.

while (!exports.foo = --foo) { } // SyntaxError: `!exports.foo` is no valid assignment target

Playground Link: https://www.typescriptlang.org/play/?target=99&module=1&ssl=3&ssc=19&pln=1&pc=1#code/DYUwLgBAZg9jEF4IEYDcAoEAPADjATpAN7RwQC+GA7gBYCWoEAFAIQC0bsMAlBCeUA

Related Issues: #33044

Guia do colaborador

Direção de pesquisa
Investigue o transformador do módulo commonjs, especificamente o callback onSubstituteNode, e adicione parênteses para PrefixUnaryExpression quando o operando é uma atribuição de exportação.
Pilha de tecnologia
typescript
Domain
compilers
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.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
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.
Claro
Prerequisites
TypeScriptTypeScript compiler internals
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40