Remove vcpkg code action from missing include scenarios

[sean-mcmanus]

This used to work at some point with an internal build, but I'm not sure if it ever worked with a public release (i.e. the ones I tried all don't work).

The code action implementation is supposed to check if it's a system header and show different code actions.

You can set compilerPath to "" for a repro.

I reviewed the code and it seems to be setup to do the right thing, but it's going wrong at some point either in cpptools or the TypeScript that handles the result.

[bobbrow]

It would be nice to remove the "vcpkg" code action as part of this if possible.

[sean-mcmanus]

@bobbrow You mean remove the vcpkg code action entirely? Even for non-system includes?

[bobbrow]

No, just for system includes since it's not going to help for that case (and it's actually a "bad" suggestion).

[sean-mcmanus]

@bobbrow Yeah, the code already handles/checks that -- it's just a matter of figuring out why it's not working as expected (like it used to).

[browntarik]

The edit compiler path code action appearing is expected behavior. A fix for the vcpkg code action can be found here: #11280

[michelleangela]

Fix available in pre-release https://github.com/microsoft/vscode-cpptools/releases/tag/v1.17.1.