Microsoft/TypeScript

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

Open

#35.879 geöffnet am 27. Dez. 2019

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)TypeScript (6.726 Forks)batch import
BugDomain: JS EmitHelp Wanted

Repository-Metriken

Stars
 (48.455 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 6T 17h) (9 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuche den CommonJS Modul Transformer, insbesondere den onSubstituteNode Callback, und füge für PrefixUnaryExpression Klammern hinzu, wenn der Operand eine Exportzuweisung ist.
Tech Stack
typescript
Domain
compilers
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
TypeScriptTypeScript compiler internals
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40