Skip to content

Notify Zed of error in LSP server#19

Merged
vittorius merged 7 commits intomainfrom
notify-zed-of-error-in-lsp
Jan 17, 2026
Merged

Notify Zed of error in LSP server#19
vittorius merged 7 commits intomainfrom
notify-zed-of-error-in-lsp

Conversation

@vittorius
Copy link
Owner

@vittorius vittorius commented Jan 15, 2026

  • accomplished unit tests for PathStore
  • now we notify LSP client if there is a file sync error or any kind of file watcher internal error
  • Zed will show such notifications as error pop-ups

An example of such notification:

image

PR train:

@vittorius vittorius force-pushed the tests-for-path-store branch from fffc384 to a0c329a Compare January 15, 2026 14:00
@vittorius vittorius force-pushed the notify-zed-of-error-in-lsp branch from 0dc7efd to 12721d0 Compare January 15, 2026 14:03
@vittorius vittorius force-pushed the tests-for-path-store branch from a0c329a to 2f6fe1f Compare January 15, 2026 14:07
@vittorius vittorius force-pushed the notify-zed-of-error-in-lsp branch from 12721d0 to e427076 Compare January 15, 2026 14:08
Copy link

@bloodfeast bloodfeast left a comment

Choose a reason for hiding this comment

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

I don't have anything to call out in this code really. Very nice job!

Base automatically changed from tests-for-path-store to main January 17, 2026 15:21
@vittorius vittorius force-pushed the notify-zed-of-error-in-lsp branch from e427076 to 3bceeb0 Compare January 17, 2026 15:25
@vittorius vittorius merged commit 3373bc2 into main Jan 17, 2026
8 checks passed
@vittorius vittorius deleted the notify-zed-of-error-in-lsp branch January 17, 2026 15:33
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