Microsoft/monaco-editor

TS regex missed by Monarch grammar

Open

#2.512 geöffnet am 2. Juni 2021

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (1.283 Forks)batch import
buggrammarshelp wantedtypescript

Repository-Metriken

Stars
 (14.836 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 6h 26m) (15 gemergte PRs in 30 T)

Beschreibung

Extracted from https://github.com/microsoft/TypeScript-Website/issues/1870

const x = true ? /^([^"]*)"/ : /^([^']*)'/

/^([^"]*)"/ is not identified as a regex by the TS Monarch grammar, which breaks bracket matching:

image

Contributor Guide

Research-Richtung
Der Fehler tritt in der TypeScript Monarch Grammatik in Zeile 76 von typescript.ts auf (https://github.com/microsoft/monaco languages/blob/319a1ddf568438d59a3851784b092cdf1f88e29e/src/typescript/typescript.ts). Das Regex Muster nach einem ternären Operator wird nicht als Regex tokenisiert. Zur Behebung sollten die Tokenizer Regeln für die Behandlung von '/' nach bedingten Ausdrücken untersucht werden, um sicherzustellen, dass die Regex Erkennung für Muster wie `/^([^"]*)"/` und `/^([^']*)'/` funktioniert. Erwägen Sie, einen Zustand oder eine Regel hinzuzufügen, um in solchen Kontexten zwischen Division und Regex zu unterscheiden.
Tech Stack
typescript
Domain
frontend
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-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
TypeScriptMonarch grammar
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40