Skip to content

Comments

feat: bind a language client for the latest server#288

Open
kevineger wants to merge 1 commit intostandalone-ls-clientfrom
standalone-ls-ctrl
Open

feat: bind a language client for the latest server#288
kevineger wants to merge 1 commit intostandalone-ls-clientfrom
standalone-ls-ctrl

Conversation

@kevineger
Copy link
Member

There are two inputs that determine how a language client is bound/registered:

  1. Does the user have a valid auth session?
  2. What's the latest server?

This controller listens to those two events and manages the lifecycle of the corresponding server. Here we use LatestCancelable to do the async spin up in response to the aforementioned sync events. To facilitate testing and keep the logging in a centralized spot, when constructed we listen to the ServerChangeEvent this class emits.

There are two inputs that determine how a language client is bound/registered:

1. Does the user have a valid auth session?
2. What's the latest server?

This controller listens to those two events and manages the lifecycle of the corresponding server. Here we use `LatestCancelable` to do the `async` _spin up_ in response to the aforementioned sync events. To facilitate testing and keep the logging in a centralized spot, when constructed we listen to the `ServerChangeEvent` this class emits.
@kevineger kevineger requested a review from amtoney524 December 3, 2025 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants