-
Notifications
You must be signed in to change notification settings - Fork 51
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Quest: let's forward _all_ the ServerCapabilities to tsserver #626
Labels
Comments
@NullVoxPopuli I'd be happy to help with this. I have a little bit of experience with language servers, but not a ton. Could you point me in the right direction to get started? |
Hello!, given the list of server capabilities in protocol.ts (linked above), is there a particular one you'd like to start with implementing? we only want one server capabilities addition per PR (for ease of testing) |
6 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Goal:
Achieve parity between tsserver and glint so that folks don't feel the need to have both servers active (which would consume more RAM)
Each capability should be its own PR -- some PRs will be more involved from others, and some may be straight-forward.
Here is where we need to configure: https://github.com/typed-ember/glint/blob/main/packages/core/src/language-server/binding.ts#L19
"
ServerCapabilities
"Here is the full list:
https://github.com/microsoft/vscode-languageserver-node/blob/main/protocol/src/common/protocol.ts#L1068
Features we want?
(and probably more)
Here are the features from reported requests we can add
Semantic Token Scopes not provided #601
Note from discord:
TypeScript language server code-folding
Note from discord:
textDocument/documentSymbol
progress
Shows ui indicator in $editor for booting/loading, pending, etc
https://github.com/microsoft/vscode-languageserver-node/blob/main/protocol/src/common/protocol.ts#L1104
The text was updated successfully, but these errors were encountered: