sindresorhus/eslint-plugin-unicorn

Rule proposal: Enforce `.toArray()` at the end of `Iterator` methods chain

Open

#2,767 opened on 2025年9月25日

GitHub で見る
 (5 comments) (1 reaction) (0 assignees)JavaScript (5,022 stars) (468 forks)user submission
help wantednew rule

説明

Description

Note that it's a good idea to call toArray() as a last step of your processing. For example, fibonacci().take(10).toArray().filter(...) is less efficient, because iterator helpers are lazy and avoids creating a temporary array.

From https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Iterator/toArray

Examples

// ❌
foo.toArray().filter(...)
//            ^^^^^^ Array#filter()

// ✅
foo.filter(...).toArray()
//  ^^^^^^ Iterator#filter()

Proposed rule name

not sure

Additional Info

No response

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

技術スタック
javascript
領域
tooling
Issue 種別
feature
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
half day
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
fresh
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
mostly clear
前提条件
Understanding of ESLint plugin architectureFamiliarity with AST (Abstract Syntax Tree)Basic knowledge of JavaScript iterators
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
70
調査方針
Review existing rule implementations in the plugin (e.g., 'prefer array flat map') as a reference. Examine the AST for calls like `foo.toArray().filter(...)` and determine how to detect when `.toArray()` is called before other array methods instead of after iterator methods. Consider using ESLint's `MemberExpression` and `CallExpression` nodes. The issue lacks a proposed rule name; comment to suggest a name based on naming conventions.
Rule proposal: Enforce `.toArray()` at the end of `Iterator` methods chain · sindresorhus/eslint-plugin-unicorn#2767 | Good First Issue