Skip to content

Add LSP test case - #292

Draft
nitely wants to merge 1 commit into
status-im:masterfrom
nitely:lsp_test
Draft

Add LSP test case#292
nitely wants to merge 1 commit into
status-im:masterfrom
nitely:lsp_test

Conversation

@nitely

@nitely nitely commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Tests for a (potential) stripped down LSP transport.

Things LSP needs to support:

  • Processing requests concurrently (ie: RPC handlers can run concurrently)
  • Sending requests to the connected client (ie: bidi communication)
  • Cancelling a given in progress RPC request by its request id (ie: RPC handlers can be cancelled)
  • stdio transport (not implemented)

@nitely

nitely commented Aug 24, 2026

Copy link
Copy Markdown
Contributor Author

mmh, but this does not add much that is not already tested. I was thinking if some of it could be implemented in json-rpc itself (ie: concurrent rpc handlers), the improvements could be tested with this instead of the LSP... but for now it does not seem worth to add it to the test suite.

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.

1 participant