Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: request cancellation #7054

Merged
merged 1 commit into from
Feb 14, 2025

Conversation

mhuisi
Copy link
Contributor

@mhuisi mhuisi commented Feb 12, 2025

This PR adds language server support for request cancellation to the following expensive requests: Code actions, auto-completion, document symbols, folding ranges and semantic highlighting. This means that when the client informs the language server that a request is stale (e.g. because it belongs to a previous state of the document), the language server will now prematurely cancel the computation of the response in order to reduce the CPU load for requests that will be discarded by the client anyways.

@mhuisi mhuisi added the changelog-server Language server, widgets, and IDE extensions label Feb 12, 2025
@mhuisi mhuisi requested a review from digama0 as a code owner February 12, 2025 12:57
@mhuisi mhuisi force-pushed the mhuisi/request-cancellation branch from 4d47dd6 to 0694047 Compare February 12, 2025 13:11
@github-actions github-actions bot added the toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN label Feb 12, 2025
@leanprover-community-bot
Copy link
Collaborator

Mathlib CI status (docs):

  • ❗ Batteries CI can not be attempted yet, as the nightly-testing-2025-02-12 tag does not exist there yet. We will retry when you push more commits. If you rebase your branch onto nightly-with-mathlib, Batteries CI should run now. (2025-02-12 13:41:20)

@mhuisi mhuisi added this pull request to the merge queue Feb 14, 2025
Merged via the queue into leanprover:master with commit 05fb67a Feb 14, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog-server Language server, widgets, and IDE extensions toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants