Enforce `.replaceAll()` over `.split().join()` in `prefer-string-replace-all` rule

描述

foo.split('a').join('b');

foo.replaceAll('a', 'b');

// Not sure about this, maybe add `g` flag, result the same?
foo.split(/a+/).join('b');

// Different with foo.replaceAll('', 'b');
foo.split('').join('b');

貢獻者指南

技術棧

javascriptnodejs

領域

toolingdeveloper experience

議題類型

feature

難度面向新貢獻者的預計實作難度，1 表示很小改動，5 表示專家級工作。

3

預計時間有經驗貢獻者完成調查、實作、測試並準備 pull request 的粗略時間範圍。

half day

活動狀態議題目前的可參與程度：新鮮、活躍、陳舊、阻塞或等待維護者輸入。

fresh

清晰度議題是否清楚說明預期改動、驗收標準和下一步。

clear

前置要求

familiarity with ESLint rulesAST manipulationJavaScript

新手友善度1-100 的估計分數，表示該議題對首次貢獻者的友善程度。

40

研究方向

Examine the existing rule implementation in the eslint plugin unicorn repository, specifically the `prefer string replace all` rule file (`rules/prefer string replace all.js`). Understand how the rule currently detects `.replaceAll()` usage and how to add detection for `.split().join()` patterns. Review the test file for that rule to see existing test cases. Consider edge cases like using regex or empty string in split. The issue author suggested that `.split(/a+/).join('b')` might be equivalent, while `.split('').join('b')` is different. Investigate these and implement the check.