sindresorhus/eslint-plugin-unicorn

Rule proposal: `prefer-iterator-to-array`

Open

Aperta il 20 feb 2025

Vedi su GitHub
 (2 commenti) (3 reazioni) (0 assegnatari)JavaScript (5022 star) (468 fork)user submission
help wantednew rule

Descrizione

Description

Iterator#toArray() is more clear to me that coverts iterator to array than [...iterator], especially when iterator is a long expression.

Examples

// ❌
[...map.values()].map(value => value * 2)

// ✅
map.values().toArray().map(value => value * 2)
map.values().map(value => value * 2).toArray()

Proposed rule name

prefer-iterator-to-array

Additional Info

Full list of Iterators https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Iterator#description

Guida contributor

Tech stack
javascripttypescript
Dominio
tooling
Tipo issue
feature
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
half day
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
fresh
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
clear
Prerequisiti
ESLint plugin developmentAST basicsJavaScript iterators
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70
Direzione di ricerca
Esamina l'implementazione di codice di regole simili come 'prefer array flat map' nel repository eslint plugin unicorn per i pattern di codice. La regola deve rilevare l'uso dell'operatore spread sugli iterabili (es. [...map.values()]) e suggerire l'uso di .toArray() invece. Utilizza selettori AST come 'SpreadElement' o 'CallExpression' per identificare il pattern. Il problema fornisce esempi chiari di ciò che dovrebbe essere segnalato e di ciò che è preferibile.