node is required to build Conjure Oxide (due to treesitter grammar)

[niklasdewally]

node is required when building treesitter-essence and conjure-oxide from source; however, this is not documented anywhere.

We should either document that node is required, or try to remove it as a dependency.

CC: @ozgurakgun @leiamcalisteryoung

[TAswan]

I don't believe it's possible to remove that node dependency without providing tree sitter essence as a binary (unsure how supported that is for js). But would it be worth it to add a compiler flag to ignore tree sitter and only use the legacy parser?

[ozgurakgun]

I think we may be able to avoid this by commiting the generated parser code in the repo. As long as we have an easy way (probably a script?) to regenerate the parser code this might be ok. Though I am not sure of the specifics, and won't be suprised if I am missing something.

My understanding is that tree-sitter-cli requires node and it is only used to generate a bunch of files?

[niklasdewally]

As long as we have an easy way (probably a script?) to regenerate the parser code this might be ok.

We could keep the existing build script, but make it trigger only when the parser code is changed?

[ozgurakgun]

We could keep the existing build script, but make it trigger only when the parser code is changed?

That sounds good. If it's hard to set that up, we can even have it as a manual step.