-
Notifications
You must be signed in to change notification settings - Fork 734
Create clickable links in quick info from @link JSDoc tags #1935
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR implements support for creating clickable links in quick info tooltips from JSDoc @link tags. The main change converts plain text JSDoc link references into Markdown-formatted links that allow navigation to declarations within the codebase or to external URLs.
Key Changes
- Refactored JSDoc link handling to generate clickable Markdown links instead of plain text
- Added support for both code declarations (file://... links) and HTTP/HTTPS URLs in JSDoc links
- Converted several utility functions to methods on
LanguageServiceto enable access to position converters
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| internal/ls/hover.go | Core implementation of JSDoc link conversion to Markdown links with declaration resolution |
| internal/ls/completions.go | Updated to use refactored method for getting completion details with documentation |
| internal/ls/string_completions.go | Updated to use refactored method for string literal completion details |
Fixes #1895.