Now this is what the call stacks look like;

Global rename completed!

From what I found out, yield is indeed handled correctly. The bug could be before yield is encountered.

Global rename completed!

To add to what @Krinkle stated earlier, using mw.loader.getScript() doesn’t completely solve this, although you no longer have randomly generated URLs, this still goes to eval(), so the browser still doesn’t see it as a linked script with URL.

Currently, JavaScriptMinifier only supports ES2017 tokens. When you try to minify unsupported tokens, it attempts to interpret them and often results in bugs, similar to the one reported.

@pmiazga I have created a task here T362647 to investigate the CSS difference in import-reference test. Looks like it's no longer failing because of the deep selector :-)

Oh, I see you already did here https://gerrit.wikimedia.org/r/c/mediawiki/libs/less.php/+/1006154 :-) Thank you!

Good to know everything works fine now :-) @Krinkle Hmm, hoping I'm done with the current task this week, and we can tag a new minor version with the updates!

Done @Krinkle :-)

The !important flag on mixin call prevented proper evaluation of variables inside nested mixins. See #2394

While working with @Krinkle on this, Here are a few things we’d like to document for future reference.

This patch added the duplicate. Right now, there's no need for the duplicate function since we no longer use the old cache method which was removed Here.