search

JohnnyMorganz / luau-lsp

Language Server Implementation for Luau
MIT License
238 stars 58 forks source link

isIgnoredFiles with default constructed path due to drive letter #752

Open MohamedBannona opened 1 month ago

MohamedBannona commented 1 month ago

I'm not sure this happens cause of the sourcemap, but I did some tests and I only get this error in scripts included in the sourcemap, maybe because I'm using argon. image

JohnnyMorganz commented 1 month ago

In the next release I'm going to improve the error message for this. Can you show me the updated error message when that happens?

MohamedBannona commented 1 month ago

D:/Mopr/test is project root image

LSP[luau-lsp]: Error INVALID_SERVER_MESSAGE: {
  error = {
    code = -32603,
    message = "isIgnoredFile failed: relative path is default-constructed when constructing d:/Mopr/test/src/Server/Main.server.luau against D:/Mopr/test"
  },
  jsonrpc = "2.0"
}
JohnnyMorganz commented 1 month ago

Ah the age-old problem of case sensitivity of the drive letter. It seems that Neovim uses a capital letter while VSCode uses lowercase (for some historic reasons apparently)

We need to normalise the case letter here.