beautifier/js-beautify

Beautification problem with JS Single Line Comment <!--

Open

#1,835 opened on 2020年8月31日

GitHub で見る
 (9 comments) (0 reactions) (1 assignee)JavaScript (8,342 stars) (1,460 forks)batch import
good first issuelanguage: javascripttype: bug

説明

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

コントリビューターガイド

技術スタック
javascript
領域
tooling
Issue 種別
bug
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3 hours
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
basic understanding of JavaScript parsing
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
50
調査方針
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.