`file.cwd` is not correctly set about unified-language-server HOT 5 CLOSED

Maybe the workspace folders(s) could be used? https://microsoft.github.io/language-server-protocol/specifications/specification-3-17/#workspace_workspaceFolders

from unified-language-server.

Isn’t it set to process.cwd() by default? Typically the client determines the cwd.

Also does it make sense if unified-engine detects this? I imagine a good cwd to set would be the directory containing the configuration file.

from unified-language-server.

Yes, but the value provided by process.cwd does not always make sense.
Atom sets it to / (which is why remark-parse can’t be resolved and why linter-remark doesn’t work with the PR)
I’m also guessing the Node.js language server runs in a cwd that is different from the “workspace”, as they‘re separate processed?

unified-engine has an option for it: options.cwd, which can be given the CWD. This can (should?) be set too.

I imagine a good cwd to set would be the directory containing the configuration file.

Files are found first, configuration files are found from them. That means that there can be a) multiple configuration files, b) none, c) one in say, /Users/

from unified-language-server.

Nice that LSP also has that. For Atom, the earlier linked lines (https://github.com/unifiedjs/unified-engine-atom/blob/605326d08a6d04b66aa8abf3143313f24c507699/index.js#L44-L53) uses something like that too.

from unified-language-server.

Yes, I think using rootUri of the InitializeParams should be used.

from unified-language-server.