Add auto-fix for `no-array-reduce`

Repository metrics

Stars

5,022 stars (5,022 stars)

PR merge metrics

Avg merge 1d 16h (Avg merge 1d 16h)399 merged PRs in 30d (399 merged PRs in 30d)

Description

 const foo = array.reduce(callback, initValue);

let foo = initValue;
for (const [index, element] of array.entries()) {
  foo  = callback(foo, element, index, array);

  // Or safer
  // foo  = callback.call(array, foo, element, index, array); 
}

 const foo = array.reduce(callback); // No inital value

let foo;
for (const [index, element] of array.entries()) {
  if (index === 0) {
   foo = element;
   continue;
  }
  foo  = callback(foo, element, index, array);
}

 const foo = array.reduce((accumulator, element, index, array) => {
	return doThingsWith(accumulator, element, index, array)
}, initValue);

let foo = initValue;
for (const [index, element] of array.entries()) {
  foo  = doThingsWith(foo, element, index, array);
}

Contributor guide

Research direction

Implement the auto fix logic in the rule handler for `no array reduce`, converting reduce calls to for of loops with array entries.

Tech stack

javascript

Domain

developer experiencetooling

Issue type

Feature

DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.

3

Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.

Half day

Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.

Active

ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.

Clear

Prerequisites

JavaScriptESLintGit

Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.

60