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

Recommendation to search from Marketplace is not shown #180826

Closed
DanTup opened this issue Apr 25, 2023 · 2 comments · Fixed by #183962
Closed

Recommendation to search from Marketplace is not shown #180826

DanTup opened this issue Apr 25, 2023 · 2 comments · Fixed by #183962
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug extension-recommendations Extension recommendations issues insiders-released Patch has been released in VS Code Insiders regression Something that used to work is now broken verified Verification succeeded
Milestone

Comments

@DanTup
Copy link
Contributor

DanTup commented Apr 25, 2023

I opened VS Code with a brand new profile (--profile-temp --user-data-dir /foo/tmp) and opened .arb file. I was expected to see a notification "Marketplace has extensions that can help with '.arb' files" but nothing appeared.

I put a breakpoint inside promptRecommendations() in fileBasedRecommendations.ts and noticed that when this.recommendationsByPattern.get('.arb') is called, the set of extensions returned does not match those you get when yous earch "ext:arb" in the marketplace search. It returns a number of unrelated extensions, which are then discarded for not being ARB extensions, so ultimately no prompt is shown.

Screenshot 2023-04-25 at 14 38 31

For example, in the list fetched in code is the Go extension, the CSharp extension, the Python extension, the PowerShell extension. But when I search the marketplace I get:

Screenshot 2023-04-25 at 14 43 41

As a result, promptRecommendations never calls promptFromRecommendations, so promptRecommendedExtensionForFileType is also never called, which is where the file-type prompt is usually handled.

(@sandy081 FYI - I notice you made some changes in this area in #177181, so perhaps you have some context about how this works and whether this is a bug, or the behaviour is not exactly as I expect?).

@sandy081 sandy081 added bug Issue identified by VS Code Team member as probable bug extension-recommendations Extension recommendations issues regression Something that used to work is now broken labels May 11, 2023
@sandy081 sandy081 added this to the May 2023 milestone May 11, 2023
@sandy081 sandy081 changed the title File-based recommendations are not working correctly Recommendation to search from Marketplace is not shown May 11, 2023
sandy081 added a commit that referenced this issue May 31, 2023
sandy081 added a commit that referenced this issue May 31, 2023
@VSCodeTriageBot VSCodeTriageBot added unreleased Patch has not yet been released in VS Code Insiders insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels May 31, 2023
@joyceerhl joyceerhl added the verified Verification succeeded label May 31, 2023
@DanTup
Copy link
Contributor Author

DanTup commented Jun 1, 2023

Thanks! :)

@github-actions github-actions bot locked and limited conversation to collaborators Jul 16, 2023
@isidorn
Copy link
Contributor

isidorn commented Aug 14, 2023

We decided to remove the recommendation on file type extension feature. More details here #188467
Thank you!

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 extension-recommendations Extension recommendations issues insiders-released Patch has been released in VS Code Insiders regression Something that used to work is now broken verified Verification succeeded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants