Microsoft/TypeScript

Make private field name parsing ecma 262 compliant

Open

#50,883 opened on 2022年9月21日

GitHub で見る
 (1 comment) (1 reaction) (1 assignee)TypeScript (6,726 forks)batch import
BugDomain: ParserEffort: ModerateHelp WantedRescheduled

Repository metrics

Stars
 (48,455 stars)
PR merge metrics
 (平均マージ 6d 17h) (30d で 9 merged PRs)

説明

This file has parse errors, but probably shouldn't.

Considerations:

  • https://github.com/microsoft/TypeScript/pull/50856 added support for starting with a unicode escape
  • The names #ZW__NJ and #ZW__J contain \u200c and \u200d, respectively (not here, but in the file)
  • Support for extended unicode escapes (i.e. \u{6F}, rather than \u006F) depends on ES2015, but we appear to parse the file as ES5.
  • Converting the extended unicode escapes to regular unicode escapes does not seem to eliminate the errors

Discovered as part of https://github.com/microsoft/TypeScript/issues/50835

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

調査方針
参照された test262 ファイル（https://github.com/tc39/test262/blob/6f8e0f67cad29a92a23ed7b596a9be8d7ba34644/test/language/expressions/class/elements/after same line static async method rs privatename identifier alt.js）を調査し、ゼロ幅非接合子（U+200C および U+200D）を含むプライベートフィールド名の期待される解析を理解します。Unicode エスケープサポートを追加した PR #50856 を考慮し、拡張 Unicode エスケープを通常のエスケープに変換してもエラーが解決しない理由を分析します。TypeScript のパーサーコードでクラス要素の解析を調べ、ECMAScript 262 仕様に合わせ、ES5 モードが拡張 Unicode エスケープの処理にどのように影響するかに焦点を当てます。
技術スタック
typescript
領域
tooling
Issue 種別
バグ
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3時間
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
アクティブ
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
おおむね明確
前提条件
GitTypeScript
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
75