@putout/plugin-remove-useless-spread

Spread syntax can be used when all elements from an object or array need to be included in a list of some kind.

(c) MDN

????Putout plugin adds ability to remove useless spread syntax. Merged with @putout/plugin-spread.

Install

npm i @putout/plugin-remove-useless-spread

Rule

{
    "rules": {
        "remove-useless-spread/array": "on",
        "remove-useless-spread/object": "on",
        "remove-useless-spread/nested": "on"
    }
}

array

The thing is [...b] can be used for:

copying an array;
converting different value type like string to an array.

So better to be more concrete and use slice for copying and Array()/Array.from() for converting to decrease cognitive load. Also sometimes there is no need on any of this operations, and we can drop spread.

❌ Example of incorrect code

for (const a of [...b]) {}

const places = [...getPlaces()];

✅ Example of correct code

for (const a of b) {}

const places = getPlaces();

// Array constructor creates sparse array
[...Array(5)].map(Number);

object

❌ Example of incorrect code

const a = {
    ...fn(),
};

✅ Example of correct code

const a = fn();

nested

Checkout in ????Putout Editor.

❌ Example of incorrect code

[
    ...[
        ...a,
        ...b,
    ],
    ...x,
];

✅ Example of correct code

[
    ...a,
    ...b,
    ...x,
];

@putout/plugin-remove-useless-spread

Install

Rule

array

❌ Example of incorrect code

✅ Example of correct code

object

❌ Example of incorrect code

✅ Example of correct code

nested

❌ Example of incorrect code

✅ Example of correct code

License

Current Tags

30 Versions