microsoft / microsoft/TypeScript

Automatic Brace Insertion (ABI) is undesirable

Offen
#34,876 4 Kommentare 2 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Awaiting More Feedback Suggestion
Vorherrschende Sprache
Go
Sterne
111k
Forks
14.3k
Ø Merge
2 T. 4 Std.
Gemergte PRs (30 T.)
132

Beschreibung

TypeScript Version: 3.7-beta

Search Terms: automatic brace insertion ABI syntax error

Code:

function a() {
    "Automatic Brace Insertion! (ABI)";

Expected behavior:

The input is syntactically invalid, so emit should fail regardless of the state of noEmitOnError. Other forms of un-parsable input already do this, e.g. function a(

Actual behavior:

A closing brace is inserted in the emitted JS code.

function a() {
    "Automatic Brace Insertion! (ABI)";
}

This means that users who are not using noEmitOnError are silently having invalid code transformed into valid code. I can confirm this actually happened to someone shipping something to production.

Playground Link: ABI Playground Link

Related Issues: https://github.com/microsoft/TypeScript/issues/34870 , https://github.com/microsoft/TypeScript/issues/34871 , Lazy Twitter report

I would suggest this bug deserves to be fixed based on Design Goal 7 (Preserve runtime behavior of all JavaScript code) and Design Non-Goal 7 (Introduce behaviour that is likely to surprise users).

Tagline: If the braces don't fit, you must not emit.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne mit dem ABI Playground Link und der im Issue gezeigten minimalen TypeScript-Eingabe und vergleiche anschließend ihr Emit-Verhalten mit dem verlinkten Beispiel einer nicht abgeschlossenen Funktion. Verfolge die Verarbeitung dieser fehlerhaften Eingabe durch den Compiler und überprüfe, dass Emit keine schließende Klammer hinzufügt und kein ausführbares JavaScript erzeugt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
typescript
Bereich
compilers
Issue-Typ
Bug
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.