Skip to content

Running Save Participants dialog is hanging on save and preventing the script from saving #2555

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
MarkKharitonov opened this issue Mar 13, 2020 · 13 comments
Labels

Comments

@MarkKharitonov
Copy link

Issue Description

Saving a ps1 script opens a "Running Save Participants ..." dialog that just hangs. Saving does not proceed until the dialog is closed.

Attached Logs

logs.zip
2020-03-12 21_26_04-Clipboard

Environment Information

Visual Studio Code

Name Version
Operating System Windows_NT x64 10.0.17763
VSCode 1.42.1
PowerShell Extension Version 2020.3.0

PowerShell Information

Name Value
PSVersion 5.1.17763.1007
PSEdition Desktop
PSCompatibleVersions 1.0 2.0 3.0 4.0 5.0 5.1.17763.1007
BuildVersion 10.0.17763.1007
CLRVersion 4.0.30319.42000
WSManStackVersion 3.0
PSRemotingProtocolVersion 2.3
SerializationVersion 1.1.0.1

Visual Studio Code Extensions

Visual Studio Code Extensions(Click to Expand)
Extension Author Version
powershell ms-vscode 2020.3.0
@ghost ghost added the Needs: Triage Maintainer attention needed! label Mar 13, 2020
@TylerLeonhardt
Copy link
Member

This is very strange... a quick search says it might be a vscode bug: https://github.com/search?q=%22Running+Save+Participants%22&type=Code

@MarkKharitonov
Copy link
Author

Another quick search yields this - microsoft/vscode#90542
Which is closed for they think it is extensions.
Indeed, without any extensions it does not seem to occur. I have only one extension - Powershell.

@rjmholt
Copy link
Contributor

rjmholt commented Mar 13, 2020

Do you have format on save enabled?

@MarkKharitonov
Copy link
Author

MarkKharitonov commented Mar 13, 2020 via email

@SydneyhSmith SydneyhSmith removed the Needs: Triage Maintainer attention needed! label Mar 17, 2020
@rjmholt
Copy link
Contributor

rjmholt commented Mar 17, 2020

My theory here is that:

  • The Save Participants dialogue represents a VSCode functionality that runs the formatter on save
  • It's waiting on the PowerShell extension to format something
  • For some reason, that formatting run is hanging

We haven't been able to reproduce this error yet, but if we can, that will make this much easier to fix

@SydneyhSmith
Copy link
Collaborator

It would also be great if you could provide the payload logs, information for how to do that is here:
https://github.com/PowerShell/vscode-powershell/blob/master/docs/troubleshooting.md#provide-language-server-protocol-payload-logs

@MarkKharitonov
Copy link
Author

@SydneyhSmith I do not use the preview version, so according to that link I cannot provide the payload logs:

NOTE: This currently only applies to the PowerShell Preview extension and only if you have version 2019.11.0 or higher.

Hmm, interesting that you cannot reproduce it. For me it reproduces right away. On the contrary, I cannot get rid of it. Had to downgrade the plugin version to the last of 2019. Still happens but more rarely.

I am open to demonstrate it live, if anyone wishes to send me a zoom meeting or anything like that.

@ghost ghost added the Needs: Maintainer Attention Maintainer attention needed! label Mar 17, 2020
@rjmholt
Copy link
Contributor

rjmholt commented Mar 17, 2020

I do not use the preview version, so according to that link I cannot provide the payload logs:

Ah the docs are out of date -- the stable extension will have these now

@bjbr-dev
Copy link

bjbr-dev commented May 29, 2020

This is happening to me very frequently - what can I do next time it happens to help diagnose it?

@ghost ghost added the Needs: Maintainer Attention Maintainer attention needed! label May 29, 2020
@MarkKharitonov
Copy link
Author

My advise - do not fight it, just downgrade the plugin to 2019.12.0. My empirical tests showed that this is the last version that allows to work sanely. I, personally, do not plan to upgrade until reports like this stop to appear.

@TylerLeonhardt
Copy link
Member

You know what, I think this is a duplicate of this issue:

#2627 (comment)

Can you all try the build there and let me know how it goes?

@SydneyhSmith
Copy link
Collaborator

We recently published an update to our preview extension which has addressed a number of underlying architectural issues, would you mind giving it a try and letting us know if you are still hitting this issue?

@SydneyhSmith SydneyhSmith removed the Needs: Maintainer Attention Maintainer attention needed! label Jun 2, 2020
@ghost
Copy link

ghost commented Aug 19, 2021

This issue was closed automatically as repro info was indicated as needed, but there has been no activity in over a week. Please feel free to reopen with any available information!

@ghost ghost closed this as completed Aug 19, 2021
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants