luau-lsp icon indicating copy to clipboard operation
luau-lsp copied to clipboard

isIgnoredFiles with default constructed path due to drive letter

Open MohamedBannona opened this issue 1 year ago • 3 comments

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

MohamedBannona avatar Aug 22 '24 16:08 MohamedBannona

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?

JohnnyMorganz avatar Sep 11 '24 17:09 JohnnyMorganz

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"
}

MohamedBannona avatar Sep 15 '24 07:09 MohamedBannona

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.

JohnnyMorganz avatar Sep 16 '24 02:09 JohnnyMorganz

This error still happens on Windows when we open a file "on the fly" with null workspace (opening a roblox game or a git project does not error), it just doesn't error on Linux so I assume that this is not expected

Image

lopi-py avatar Jan 25 '25 03:01 lopi-py