Skip to content
This repository has been archived by the owner on Jul 22, 2019. It is now read-only.

Fixes #142: Move indexing logic entirely to the server-side #143

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

kaloyan-raev
Copy link

Indexing is now triggered as part of the onInitialize() request.

Any code on the client related to indexing (except registered VS Code
commands) is moved to the server. The user commands are just proxies -
they make a request to the server.

All the code related to building, querying and maintaining the index is
moved to the index.ts file.

Wherever possible, standard message request from the language server
protocol are used instead of custom messages. For example
onDidChangeContent() and onDidChangeWatchedFiles() instead of the custom
"buildObjectTreeForDocument", "saveTreeCache", etc.

Indexing is now triggered as part of the onInitialize() request.

Any code on the client related to indexing (except registered VS Code
commands) is moved to the server. The user commands are just proxies -
they make a request to the server.

All the code related to building, querying and maintaining the index is
moved to the index.ts file.

Wherever possible, standard message request from the language server
protocol are used instead of custom messages. For example
onDidChangeContent() and onDidChangeWatchedFiles() instead of the custom
"buildObjectTreeForDocument", "saveTreeCache", etc.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant