LSP: Show an error message on an invalid named module

[ghivert]

Hi!

Currently, when the LSP opens a Gleam file whose name is incorrect (something like my-module.gleam for instance), nothing happens. It could be nice to let the LSP indicates it as a bug (because gleam build will fail) with a message like "my-module is not a valid name for a Gleam module. You should switch to my_module instead."

[lpil]

Good idea! Thank you

[ahuangg]

hello @lpil, would I be able to pick up this issue?

[lpil]

That would be wonderful! Yes please!

[ahuangg]

I was thinking of implementing the module name validation in the language server engine.rs file within the compile function. The idea is to check if module names are valid similar to this and, if not, add them to the results. Before moving forward and submitting a pull request, I'd love to hear your thoughts on this approach and if there is an alternative solution I missed.

[lpil]

Wouldn't we want an error to be shown for the command line too? I could see it being useful in that case also