beautifier/js-beautify

Beautification problem with JS Single Line Comment <!--

Open

#1.835 geöffnet am 31. Aug. 2020

Auf GitHub ansehen
 (9 Kommentare) (0 Reaktionen) (1 zugewiesene Person)JavaScript (8.342 Stars) (1.460 Forks)batch import
good first issuelanguage: javascripttype: bug

Beschreibung

In Javascript <!-- is treated as a single-line comment just like //.

js-beautify works as expected when the line starts with <!--:

Input: <!--alert(1)commented

Output: <!--alert(1)commented

Otherwise it will break the syntax:

Input: alert(1)<!--commented

Output: alert(1) < !--commented

Contributor Guide

Tech Stack
javascript
Domain
tooling
Issue Type
bug
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 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
basic understanding of JavaScript parsing
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
50
Research-Richtung
Investigate the tokenizer or comment handling in js beautify, likely in files like 'js/index.js' or 'core/tokenizer.js'. The issue occurs when `<! ` is not at the start of a line, so the regex or logic that identifies single line comments might need to be extended to include `<! ` as a comment starter. Look at how `//` is handled and add similar handling for `<! `. Check existing tests for comment handling and write a failing test for this case.