feat: add OPENCODE_RESPECT_LSP_DIAGNOSTICS flag to ensure LSP diagnostics are always processed#12286
Open
lmn451 wants to merge 1 commit intoanomalyco:devfrom
Open
feat: add OPENCODE_RESPECT_LSP_DIAGNOSTICS flag to ensure LSP diagnostics are always processed#12286lmn451 wants to merge 1 commit intoanomalyco:devfrom
lmn451 wants to merge 1 commit intoanomalyco:devfrom
Conversation
…tics are always processed - Add new environment flag OPENCODE_RESPECT_LSP_DIAGNOSTICS - Remove TypeScript first-diagnostic skip when flag is enabled - Set diagnostic debounce to 0ms when flag is enabled - Increase diagnostic wait timeout to 30s when flag is enabled - Ensures all LSP servers' diagnostics are immediately processed when available
Contributor
|
Thanks for your contribution! This PR doesn't have a linked issue. All PRs must reference an existing issue. Please:
See CONTRIBUTING.md for details. |
Contributor
|
The following comment was made by an LLM, it may be inaccurate: No duplicate PRs found |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
OPENCODE_RESPECT_LSP_DIAGNOSTICS=trueBehavior
Usage
export OPENCODE_RESPECT_LSP_DIAGNOSTICS=true opencodeIssue Addressed
Fixes #12288 - This addresses the TypeScript first-diagnostic skip issue by ensuring that when LSP servers are running, their diagnostics are always processed rather than being skipped on first publication.
Technical Details
The flag addresses several conditions where LSP diagnostics were being skipped or delayed even when LSP servers were available:
Testing
Fixes issue where TypeScript and other language server diagnostics were skipped on first file edit, ensuring immediate feedback when LSP is available.