Skip to content
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

LSP client stops working on file after renaming it #7829

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

matthiasblaesing
Copy link
Contributor

The LSP protocol requires that when a file is renamed the client signals this to the server by sending a didClose for the old location and after that a didOpen with the new location.

The change assumes, that the IDE always goes through the DataObject to issue the rename/move. Changes through the raw FileObject are not tracked.

Closes: #7806

@matthiasblaesing matthiasblaesing added LSP [ci] enable Language Server Protocol tests ci:dev-build [ci] produce a dev-build zip artifact (7 days expiration, see link on workflow summary page) labels Oct 3, 2024
@matthiasblaesing matthiasblaesing added this to the NB24 milestone Oct 3, 2024
The LSP protocol requires that when a file is renamed the client
signals this to the server by sending a didClose for the old location
and after that a didOpen with the new location.

The change assumes, that the IDE always goes through the DataObject to
issue the rename/move. Changes through the raw FileObject are not
tracked.

Closes: apache#7806
@matthiasblaesing
Copy link
Contributor Author

Reverted to draft. This needs another iteration after merging the changes to the initialization order.

@matthiasblaesing matthiasblaesing modified the milestones: NB24, NB25 Oct 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci:dev-build [ci] produce a dev-build zip artifact (7 days expiration, see link on workflow summary page) LSP [ci] enable Language Server Protocol tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

LSP client stops working on file after renaming it
1 participant