Wrong transform with @putout/plugin-for-of about putout HOT 15 CLOSED

Thanks! Just fixed 🎉 . Please re-install 🐊. Is it works for you?

from putout.

It's better now, but I don't see any transformation and the forEach still exists

❯ node tt.js
(a.forEach((aa) => d.push(aa)), b.forEach((bb) => d.push(bb)));

----------------------
(a.forEach((aa) => d.push(aa)), b.forEach((bb) => d.push(bb)));

from putout.

What kind of transformation you expect to see here?

from putout.

in my opinion, it should be the most appropriate

(function() {
    for (const aa of a)
        d.push(aa);
}(),
function() {
    for (const bb of b)
        d.push(bb);
}())

can correct me if you think otherwise

from putout.

not sure if this applies to this task but I'll write here first
for code

const js = `const z=a.forEach(aa=>d.push(aa))`;

result

❯ node tt.js
const z = a.forEach((aa) => d.push(aa));

----------------------
const z;

I expect something like

const z = function() {
    for (const bb of b)
        d.push(bb);
}()

from putout.

Just fixed 🎉 . Please re-install 🐊. Is it works for you?

Here is a new rule for merging loops: minify/merge-loops.

About const z=a.forEach(aa=>d.push(aa)), forEach() returns nothing so better to keep this code as is.

from putout.

But it removes the loop and without minification it causes a push
and after that it doesn't do anything

from putout.

Have you tried new version?

from putout.

Oh yes, the version was not the last
now I have a row without changes
I just thought that plugin-for-of must strictly change to for-of

but if it is skipped for minification then it seems it should just use a more complex condition than just replacing or skipping it

from putout.

Look, @putout/plugin-for-of used both in Putout linter and Minify, so when trapped on SequenceExpression it just skippes transformation, since we have @putout/plugin-extract-sequence-expression which will do the thing first.

About your case:

a.forEach(aa=>d.push(aa)),
b.forEach(bb=>d.push(bb))`;

Now with help of @putout/plugin-minify it will be transformed to:

for (const aa of [...a, ...b]) {
    d.push(aa);
}

And then all whitespaces will be reduced.

it seems it should just use a more complex condition than just replacing or skipping it

What do you suggest?

from putout.

Thank you, I understand now, thanks for the solution

from putout.

And could you update the minify version as well?

from putout.

It is updated, just re-install it

from putout.

I see that the last version was a month ago
https://www.npmjs.com/package/minify?activeTab=versions

from putout.

I’m using semantic versioning, just re-install minify and it will update all dependencies.

from putout.