Skip to content

fix: gracefully handle LSP errors in OpenDeclarationHyperlinkDetector#1380

Merged
sebthom merged 1 commit intoeclipse-lsp4e:mainfrom
sebthom:issue-1169
Nov 13, 2025
Merged

fix: gracefully handle LSP errors in OpenDeclarationHyperlinkDetector#1380
sebthom merged 1 commit intoeclipse-lsp4e:mainfrom
sebthom:issue-1169

Conversation

@sebthom
Copy link
Copy Markdown
Member

@sebthom sebthom commented Nov 12, 2025

Wrap definition/declaration/typeDefinition/implementation requests with exceptionally(...) so per-call failures resolve to harmless results instead of breaking the merge.

Fixes #1169. Supersedes #1186

Wrap definition/declaration/typeDefinition/implementation requests with
exceptionally(...) so per-call failures resolve to harmless results
instead of breaking the merge.
@sebthom sebthom requested a review from rubenporras November 12, 2025 17:29
@sebthom sebthom merged commit d142f58 into eclipse-lsp4e:main Nov 13, 2025
11 checks passed
@sebthom sebthom deleted the issue-1169 branch November 13, 2025 13:35
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.

OpenDeclarationHyperlinkDetector.detectHyperlinks() handles invalid responses inappropriately, poisoning possibly good results

2 participants