JS minify error: too many nested expressions

[jmooring]

Tested with v2.9.27.

too many nested expressions on line 1 and column 137156
    1: ...}(t,e))),Xe()}var o},Js.prototype.invoke_...
                              ^

test.js.txt

Reference:
https://discourse.gohugo.io/t/build-failure-with-0-92/36523

[jmooring]

This appears to be a regression introduced in v2.9.17.

[tdewolff]

Thanks Joe, I'm on it!

[tdewolff]

Looks like it is not so much a bug, but a hard limit to prevent abuse. The problem is that the file in question has an expression of comma-operators that has more than 1000 elements (ie. a,b,c,d,...,AA,BB of length 1000). Since this creates a tree of a 1000 branches and leaves, that is fairly inefficient (which is why the limit is there in the first place, to prevent memory trouble).

The solution is to parse comma-operators as a list and not a tree.

[tdewolff]

Should be fixed now, please advice if you run into additional problems

[jmooring]

Thanks!

[ylazy]

@tdewolff I'm facing this issue in the latest release (2.21.1):

Error minimizing index.js:
minify.exe --css-precision 0 --html-keep-conditional-comments --html-keep-default-attrvals --html-keep-document-tags --html-keep-end-tags --html-keep-quotes --html-keep-special-comments --js-keep-var-names --js-precision 0 --json-keep-numbers --json-precision 0 --svg-precision 0 --output "C:\Users\admin\Downloads\2000x1200\index.min.js" "index.js"
ERROR: cannot minify index.js: too many nested expressions on line 678 and column 15528
  678: ...y:162.2156}).wait(1).to({graphics:mask_gr...

index.zip

Please check this!
Thank you!

[tdewolff]

That is a huge concatenation of member operators (dot operators). I don't think this could be fixed like we did with the comma operator. But perhaps we could make the ExprLimit a variable you can change. Let me see if I can get this in this week.

[tdewolff]

I've added this in the parse package and should soon be used in minify.