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

Completion adds more duplicate snippets as you type #97659

Closed
jdneo opened this issue May 13, 2020 · 6 comments
Closed

Completion adds more duplicate snippets as you type #97659

jdneo opened this issue May 13, 2020 · 6 comments
Assignees
Labels
author-verification-requested Issues potentially verifiable by issue author bug Issue identified by VS Code Team member as probable bug snippets suggest IntelliSense, Auto Complete verified Verification succeeded
Milestone

Comments

@jdneo
Copy link
Member

jdneo commented May 13, 2020

Version: 1.46.0-insider (user setup)
Commit: 288852d
Date: 2020-05-12T05:43:31.363Z
Electron: 7.2.4
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.18363
Steps to Reproduce:

Completion adds more duplicate snippets as you type

image

We have predefined some snippets in a json file and registered them in package.json. Meanwhile, the language server side will also contribute some other snippets.

This issue only happens in 1.46 insider version, in the stable version (1.45.0), it works fine. So I guess it's a regression in the insider?

Also, if I just start the client side (not launch the language server), the duplication won't be observed. Does that mean something wrong when merging the results from client & server when providing the completion list?

@jrieken jrieken added snippets suggest IntelliSense, Auto Complete labels May 13, 2020
@jrieken
Copy link
Member

jrieken commented May 13, 2020

Meanwhile, the language server side will also contribute some other snippets.

I can repro for Java but not for TypeScript... How does the language service contribute these other snippets? Does it tweak the snippet file on disk?

@jrieken jrieken added the info-needed Issue requires more information from poster label May 13, 2020
@jdneo
Copy link
Member Author

jdneo commented May 13, 2020

The language server will resolve some other snippets according to the context. It won't tweak the snippet json file.

And the snippets defined in the json file are different from the language server side, which means they have no intersection

@jrieken jrieken added bug Issue identified by VS Code Team member as probable bug and removed info-needed Issue requires more information from poster labels May 13, 2020
@jrieken
Copy link
Member

jrieken commented May 13, 2020

Ok - this only happens when using incomplete results list and is due to bogous provider filtering, e.g the snippet provider is asked again for suggestion despite its suggestion being kept...

@jrieken jrieken added this to the May 2020 milestone May 13, 2020
@jrieken jrieken added the author-verification-requested Issues potentially verifiable by issue author label May 13, 2020
@jdneo
Copy link
Member Author

jdneo commented May 13, 2020

@jrieken Thank you, I'll test it after new insider released and let you know if the problem fixed.

@jdneo
Copy link
Member Author

jdneo commented May 14, 2020

@jrieken Verified the issue has been fixed. Thank you!

@github-actions
Copy link

This bug has been fixed in to the latest release of VS Code Insiders!

@jdneo, you can help us out by commenting /verified if things are now working as expected.

If things still don't seem right, please ensure you're on version e1d8f51 of Insiders (today's or later - you can use Help: About in the command pallette to check), and leave a comment letting us know what isn't working as expected.

Happy Coding!

@github-actions github-actions bot locked and limited conversation to collaborators Jun 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
author-verification-requested Issues potentially verifiable by issue author bug Issue identified by VS Code Team member as probable bug snippets suggest IntelliSense, Auto Complete verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

2 participants