Skip to content

fix: vtsls with source.organizeImports#6

Open
cubxx wants to merge 1 commit intofnune:mainfrom
cubxx:patch-1
Open

fix: vtsls with source.organizeImports#6
cubxx wants to merge 1 commit intofnune:mainfrom
cubxx:patch-1

Conversation

@cubxx
Copy link

@cubxx cubxx commented Feb 26, 2026

vtsls cannot execute source.organizeImports with this plugin

@fnune please review it

vtsls cannot execute source.organizeImports with this plugin
@fnune
Copy link
Owner

fnune commented Feb 27, 2026

@cubxx Thanks for the PR!

It's been a while since I last used this plugin myself. Could you expand your PR description to help me understand how passing kinds fixes your problem?

I am mostly trying to make sure this doesn't break anything else.

@cubxx
Copy link
Author

cubxx commented Feb 27, 2026

I added the only field to params.context to explicitly tell LSP server which code actions the client wants to execute.
If the only is omitted, some lsp server will return null or an empty array, even when code actions are actually available. This causes the client to mistakenly assume that no code actions are available, resulting in a failure to execute the intended code action.

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.

2 participants