Compatibility with HTMX 2.0.10

[breun]

The current latest release of HtmlUnit (version 4.21.0) comes with HtmxTwoZeroSevenScriptPreProcessor which I've been successfully using with HTMX 2.0.7-2.0.9, but it fails when upgrading to HTMX 2.0.10.

It seems the variable names used in the (minified) HTMX release have changed with version 2.0.10.

• 2.0.9: i.push(...F(c.querySelectorAll(e))) → 2.0.10: i.push(...F(u.querySelectorAll(e))) (c → u)
• 2.0.9: r.push(...we(i,n)) → 2.0.10: r.push(...ve(i,n)) (we → ve)

So with HTMX 2.0.10, two of the three push(...) patterns replaced by HtmxTwoZeroSevenScriptPreProcessor#preProcess no longer match and HtmlUnit's JS engine fails when parsing htmx.min.js.

[rbri]

ok, will have a look, maybe i can make some kind of a hotfix for this.

[breun]

Adding these additional replacements indeed works:

// 2.0.10
patchedSourceCode = patchedSourceCode.replace(
    "i.push(...F(u.querySelectorAll(e)))",
    "i.push.apply(i,F(u.querySelectorAll(e)))");

patchedSourceCode = patchedSourceCode.replace(
    "r.push(...ve(i,n))",
    "r.push.apply(r,ve(i,n))");

Alternatively, using the non-minified htmx.js instead of htmx.min.js also works, because that doesn't suffer from the non-deterministic minified variable names, but of course users also lose the minification benefits then.

[rbri]

@breun do you need a fast fix? or is it ok if it takes a week or so?

[breun]

I have a local workaround, so no hurry!

[rbri]

I have a local workaround, so no hurry!

looks like there are more problems with running the 2.0.10 test suite - will investigate that.