Skip to content

completionItem/resolve provider is doubling some module's prefixes when attempting Get-Help #1242

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

Closed
nakosenk opened this issue Mar 25, 2020 · 4 comments · Fixed by #1243
Closed
Labels

Comments

@nakosenk
Copy link

nakosenk commented Mar 25, 2020

This happens on VS Code 1.43.2 using both the PowerShell 2020.03.0 and PowerShell Preview 2020.03.0 with PowerShell 5.1 on Windows 10 1909.

To reproduce the problem, start typing Test-RD and PSES will call Get-Help to search for Test-RDRDOUAccess when the actual Cmdlet is Test-RDOUAccess. This also happens with a script module I've created, but the RemoteDesktop module is one that comes from Microsoft.

Strangely, the only other module that I have that uses a Prefix is the ServerManagerTasks module, also from Microsoft, but that one does not cause the problem.

It's possible this could be a PowerShell bug, since I've seen that with my script module using the variable $MyInvocation.MyCommand results in the double prefix, but $MyInvocation.MyCommand.Name does not.

@TylerLeonhardt
Copy link
Member

Thanks for the info! Can you also attach the logs here so we can get a better idea of what's going on?

@ghost ghost added the Needs: Maintainer Attention Maintainer attention needed! label Mar 26, 2020
@nakosenk
Copy link
Author

I've attached a zip file with the logs and the Editor Service trace file.

> $psEditor.EditorServicesVersion

Major  Minor  Build  Revision
-----  -----  -----  --------
2      1      0      0       

> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      5.1.18362.628
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.18362.628
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           1.1.0.1

test-rdouaccess.zip

@TylerLeonhardt
Copy link
Member

PR out (for the extension) ^

@ghost ghost added the Needs: Maintainer Attention Maintainer attention needed! label Mar 26, 2020
@nakosenk
Copy link
Author

Created PowerShell/PowerShell#12213

@ghost ghost added Status: Fixed and removed Status: In PR labels Mar 26, 2020
@SydneyhSmith SydneyhSmith removed the Needs: Maintainer Attention Maintainer attention needed! label Mar 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants