js-tokens doesn't support private identifiers (#foo)

[o0101]

Hello @lifthrasiir

I ran

$ npx roadroller src/bang.js

on this file: https://github.com/i5ik/BANG/blob/799c5ed70bf0585b11024a1597f21fe5691ead74/src/bang.js

And received this output:

Error: invalid JS code in the input

The file src/bang.js runs correctly in the browser (as you can see)

Above, I ran roadroller on Node v14.17.6

Below, I used nvm to try node v16.9.1 and re-installed roadroller@latest and ran with -v and got the same error as well as

Actual memory usage: 145.9 MB (out of 150 MB)
Error: invalid JS code in the input

I tried with more memory --M and had

Actual memory usage: 600.8 MB (out of 1000 MB)
Error: invalid JS code in the input

---

Just leaving this report here for information.

Thank you again for this library!

[lifthrasiir]

It seems that the current version of js-tokens does not yet support private identifiers (#foo). I can update js-tokens if the upstream got a support for that. Note that Roadroller bundles its own version of js-tokens to aid the minification, so the update requires a manual process.

Meanwhile you can disable JS preprocessing by -t text -a eval, which should work for any input at the expense of compression ratio.

[lifthrasiir]

For now I'll keep this open because there might be a workaround without updating js-tokens.

[o0101]

thank you very much. I'll probably experiment with a couple of different workarounds including the ones you suggest as well I will try replacing the private member syntax with some other syntax. thank you very much!