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.

Bewertung

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

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.

Beschreibung

Awaiting More Feedback Suggestion

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.

Vorherrschende Sprache
Go
Sterne
111k
Forks
14.4k
Ø Merge
1 T. 19 Std.
Gemergte PRs (30 T.)
117

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus microsoft/TypeScript

Alle Issues in microsoft/TypeScript

Ähnliche Issues

Weitere Issues zu Go

Neue Issues direkt in Ihr Postfach

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