X Tutup
Skip to content

Recreate LSP Client to prevent out of sync state#872

Merged
DaelonSuzuka merged 1 commit intogodotengine:masterfrom
HolonProduction:recreate-client
Jul 31, 2025
Merged

Recreate LSP Client to prevent out of sync state#872
DaelonSuzuka merged 1 commit intogodotengine:masterfrom
HolonProduction:recreate-client

Conversation

@HolonProduction
Copy link
Member

Fixes #844

Over on the issue a rework of the client was mentioned. This PR is not meant to do this. It's a minimal fix to ensure consistent state, since consistent state is a blocker to move forward with certain refactorings and fixes in the server (e.g. godotengine/godot#105236)

(Also added one unrelated comment to a message filter, since it jumped into my eye)

@DaelonSuzuka DaelonSuzuka self-requested a review June 16, 2025 15:33
@DaelonSuzuka
Copy link
Collaborator

I tried doing this a while ago but got a lot of wierd behavior and decided revisit later. I'll try to make time to test this out today.

Thanks!

@DaelonSuzuka
Copy link
Collaborator

I need an AI that just slaps me in the face with a robot arm if I ever say I'm going to review something within a certain time window.

Sorry for the delay!

@DaelonSuzuka DaelonSuzuka merged commit dfe97cb into godotengine:master Jul 31, 2025
4 checks passed
@HolonProduction HolonProduction deleted the recreate-client branch October 27, 2025 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

LSP Client does not send textDocument/didOpen after reconnecting

2 participants

X Tutup