Skip to content

Offer interface keyword for classes in IntelliSense #15996

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

Open
psfinaki opened this issue Sep 18, 2023 · 5 comments
Open

Offer interface keyword for classes in IntelliSense #15996

psfinaki opened this issue Sep 18, 2023 · 5 comments
Labels
Area-LangService-AutoComplete autocomplete/intellisense Bug Impact-Medium (Internal MS Team use only) Describes an issue with moderate impact on existing code. Regression
Milestone

Comments

@psfinaki
Copy link
Member

Where is my interface keyword?
image

The word shows up, however, if there is something else out there:
image

@psfinaki
Copy link
Member Author

Don't know if this has ever worked, not marking this is a bug or regression therefore.

@auduchinok might be related to parser recovery so something interesting for you? I remember a few PRs in this area but the above repros in the latest main.

@auduchinok
Copy link
Member

Looking at the screenshots, it seems that in the first case the parser has recovered on the unfinished type declaration (look at the highlighted identifiers (i.e. they were analyzed after parsing), and on the second screenshot the whole analysis is broken (likely due to insufficient parser recovery, which indeed is interesting for me).

If this used to work previously, it may be that this code completion rule was written with an assumption of a broken tree. In that case the rule should be updated after the parser changes. I'm not familiar with the keywords code completion rules in Visual Studio or FCS, and I think currently @kerams is the most proficient with completion rules in FCS, so it might be something interesting for them too. 🙂

@psfinaki
Copy link
Member Author

Hmm sooo I dag up an old version of VS (like, last year or so) - this reproduces, hence most likely it never worked.

@inosik
Copy link

inosik commented Sep 20, 2023

FWIW, this works in VS 2019:

devenv_2023-09-20_08-28-47

@psfinaki
Copy link
Member Author

Hah, interesting. Thanks, good to know. So regression it is, although probably a long time regression.

@0101 0101 added Bug Impact-Medium (Internal MS Team use only) Describes an issue with moderate impact on existing code. Area-LangService-AutoComplete autocomplete/intellisense and removed Needs-Triage labels Oct 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-LangService-AutoComplete autocomplete/intellisense Bug Impact-Medium (Internal MS Team use only) Describes an issue with moderate impact on existing code. Regression
Projects
Status: New
Development

No branches or pull requests

4 participants