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

Fails to render completion details #47988

Closed
ghost opened this issue Apr 16, 2018 · 4 comments
Closed

Fails to render completion details #47988

ghost opened this issue Apr 16, 2018 · 4 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug suggest IntelliSense, Auto Complete verified Verification succeeded
Milestone

Comments

@ghost
Copy link

ghost commented Apr 16, 2018

Issue Type: Bug

account

*Based on tsserver.log we are sending the details, they are just not being rendered:

Info 402  [11:42:52.145] response:
    {"seq":0,"type":"response","command":"completionEntryDetails","request_seq":60,"success":true,"body":[{"name":"Account","kindModifiers":"export","kind":"class","displayParts":[{"text":"class","kind":"keyword"},{"text":" ","kind":"space"},{"text":"Account","kind":"text"}],"documentation":[],"tags":[],"codeActions":[{"description":"Import 'Account' from module \"~/pages\"","changes":[{"fileName":"/home/andy/sample/vscode-auto-import/src/pages/Auth/SignIn.tsx","textChanges":[{"start":{"line":2,"offset":1},"end":{"line":2,"offset":1},"newText":"import { Account } from '~/pages';\n"}]}]}],"source":[{"text":"~/pages","kind":"text"}]}]}
  • Hit backspace and ctrl-space to bring completions up again, and it will work.

VS Code version: Code - Insiders 1.23.0-insider (2227c70, 2018-04-16T09:45:56.409Z)
OS version: Linux x64 4.13.0-37-generic

System Info
Item Value
CPUs Intel(R) Core(TM) i7-6700HQ CPU @ 2.60GHz (8 x 2600)
GPU Status 2d_canvas: enabled
checker_imaging: disabled_off
flash_3d: enabled
flash_stage3d: enabled
flash_stage3d_baseline: enabled
gpu_compositing: enabled
multiple_raster_threads: enabled_on
native_gpu_memory_buffers: disabled_software
rasterization: disabled_software
video_decode: unavailable_software
video_encode: unavailable_software
webgl: enabled
webgl2: enabled
Load (avg) 1, 1, 1
Memory (System) 7.63GB (1.85GB free)
Process Argv /usr/share/code-insiders/code-insiders .
Screen Reader no
VM 0%
Extensions (1)
Extension Author (truncated) Version
typescript-javascript-grammar ms- 0.0.39
@mjbvz
Copy link
Contributor

mjbvz commented Apr 18, 2018

@jrieken The TS extension is also never asked to resolve the completion item in this case. This seems related to emmet somehow. It does not happen in a normal *.ts file or if you disable emmet

@jrieken jrieken removed their assignment Apr 19, 2018
@jrieken jrieken added the suggest IntelliSense, Auto Complete label Apr 19, 2018
@jrieken
Copy link
Member

jrieken commented Apr 19, 2018

It the widget that calls resolve, not sure why it doesn't. @ramya-rao-a?

@ramya-rao-a ramya-rao-a added this to the April 2018 milestone Apr 23, 2018
@ramya-rao-a
Copy link
Contributor

ramya-rao-a commented Apr 23, 2018

@jrieken resolve is getting called on the suggestion item, but the call never makes it through the extension host to the relevant extension. It gets stopped at https://github.com/Microsoft/vscode/blob/73f64b93127e0d297c98db2901d879ce73325c7f/src/vs/workbench/api/node/extHostLanguageFeatures.ts#L620-L623

This only happens, when there are multiple completion providers with one of them returning an incomplete list. Also, doesnt occur for the first letter that's typed. Only occurs when the subsequent letters are typed which results in re-triggering of the incomplete list.

During this re-triggering, previous requests are cancelled in https://github.com/Microsoft/vscode/blob/4ebe8217293726edfe29f4437cde0975cbfa3152/src/vs/editor/contrib/suggest/suggestModel.ts#L344-L345 which ends up calling https://github.com/Microsoft/vscode/blob/73f64b93127e0d297c98db2901d879ce73325c7f/src/vs/workbench/api/node/extHostLanguageFeatures.ts#L643-L645

@ramya-rao-a ramya-rao-a added the bug Issue identified by VS Code Team member as probable bug label Apr 23, 2018
@ramya-rao-a ramya-rao-a assigned jrieken and unassigned mjbvz Apr 23, 2018
@jrieken jrieken modified the milestones: April 2018, May 2018 Apr 24, 2018
@jrieken
Copy link
Member

jrieken commented Apr 24, 2018

👏 that's one deep investigation. Thanks @ramya-rao-a

jrieken added a commit that referenced this issue Apr 30, 2018
@mjbvz mjbvz added the verified Verification succeeded label May 31, 2018
@vscodebot vscodebot bot locked and limited conversation to collaborators Jun 14, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug suggest IntelliSense, Auto Complete verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

3 participants