Noir LSP often crashes with minor syntax errors

[critesjosh]

Problem

It is easy to make the Noir LSP crash. A missing curly brace will do it and it requires a window refresh to get it going again.

Will the nargo formatter help with this? Anything else we can/should do to improve the devex here?

Happy Case

Noir LSP doesn't crash, or auto formats

Alternatives Considered

No response

Additional Context

No response

Would you like to submit a PR for this Issue?

No

Support Needs

No response

[Savio-Sou]

How has your LSP been crashing?

Tried clicking the "Run Test" codelens with the missing } code snippet below. It prompted an error for sure, but pressing any arbitrary key dismissed the terminal and I'm good to go again:

Was that what you experienced, or was it something else?

[critesjosh]

@colinnielsen does this match the behavior you mentioned on Friday?

I'll keep an eye out for the next time the LSP crashes for me and report the details here.

[colinnielsen]

Yes. Missing a curly brace causes consistent crashing of the in-IDE lsp

[Savio-Sou]

Thanks Colin 🙌
Were you able to close the terminal and simply proceed with using other LSP features without e.g. restarting VS Code?

[colinnielsen]

Unfortunately no, I had to manually find the error and restart vscode

[colinnielsen]

It was a painpoint in my workflow

[TomAFrench]

Discussions should not be used for bug reports. This should be made as an issue with a reproducible example.

[jfecher]

@critesjosh or @colinnielsen can you re-open this as an issue ideally with a reproducible example please? Sorry to cause some friction, but this helps us organize work a bit better!