Skip to content

Conversation

@ILDaviz
Copy link
Member

@ILDaviz ILDaviz commented Sep 6, 2025

Introduces a visual indicator in the CodeView status bar to
show the connection status to the Language Server (LSP).

The indicator changes colour and icon depending on the status:

  • Green: Connected
  • Yellow: Connecting/reconnecting
  • Red: Disconnected

When the status is disconnected (red), the user can click on the icon
to manually attempt to re-establish the connection.

Status management has been centralised via a new Pinia store
(useLspStore) to decouple the logic from the UI.

@ILDaviz ILDaviz changed the title [draft] Add LSP status management and reconnect functionality [Draft] Add LSP status management and reconnect functionality Sep 6, 2025
@ILDaviz
Copy link
Member Author

ILDaviz commented Sep 6, 2025

@ILDaviz ILDaviz changed the title [Draft] Add LSP status management and reconnect functionality [Draft] Feature Add LSP status management and reconnect functionality Sep 6, 2025
@ILDaviz ILDaviz mentioned this pull request Sep 6, 2025
@ILDaviz ILDaviz changed the title [Draft] Feature Add LSP status management and reconnect functionality [Draft] Feature add LSP status management and reconnect functionality Sep 6, 2025
@ILDaviz ILDaviz changed the title [Draft] Feature add LSP status management and reconnect functionality [Test] Feature add LSP status management and reconnect functionality Sep 6, 2025
export const useLspStore = defineStore('lsp', () => {
const status = ref<LspStatus>('disconnected')

function setStatus(newStatus: LspStatus) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This method is unused.

@ILDaviz ILDaviz changed the title [Test] Feature add LSP status management and reconnect functionality Feature add LSP status management and reconnect functionality Sep 8, 2025
@ILDaviz ILDaviz self-assigned this Sep 10, 2025
@saeedvaziry saeedvaziry merged commit ed05760 into tweakphp:main Sep 13, 2025
4 checks passed
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.

3 participants