"Accessing non-existent property 'INVALID_ALT_NUMBER'" warning

[palfrey]

Every time I run sushi I get the following warning twice.

(node:39554) Warning: Accessing non-existent property 'INVALID_ALT_NUMBER' of module exports inside circular dependency

I don't think it's caused any issues, but looks like it should be at least looked at. Upstream bug is antlr/antlr4#2834

[cmoesel]

Hi @palfrey -- thanks for reporting this. We especially appreciate the link to the upstream bug report! We'll keep an eye on that report and upgrade ANTLR once a new version is released.

BTW -- we're currently targeting Node LTS versions, so if you want the most well tested experience of SUSHI, I'd recommend Node 12.x for now...

[palfrey]

antlr/antlr4#2931 should fix this, and got released as antlr 4.9

[cmoesel]

Yes, we saw that... but unfortunately, antlr 4.9 has breaking changes (apparently they do not follow semantic versioning). The most concerning for us is that antlr 4.9 requires Node 14 (it fails in Node 12). So given the choice between:

1. Keep antlr 4.8; which works in Node 12 and Node 14 (but logs harmless warnings in Node 14), OR
2. Move to antlr 4.9 and break SUSHI for all people / systems currently using Node 12

we're currently opting to keep antlr 4.8 for now.

[cmoesel]

This is no longer an issue since SUSHI now requires Node 18.