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

Adding a folder to an Empty workspace seems to try to re-activate every extension #125430

Closed
TylerLeonhardt opened this issue Jun 3, 2021 · 8 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders verified Verification succeeded workspace-trust Trusted workspaces
Milestone

Comments

@TylerLeonhardt
Copy link
Member

Does this issue occur when all extensions are disabled?: N/A

Version: 1.57.0-insider
Commit: 20df877
Date: 2021-06-03T06:00:36.788Z
Electron: 12.0.9
Chrome: 89.0.4389.128
Node.js: 14.16.0
V8: 8.9.255.25-electron.0
OS: Darwin x64 20.5.0

Steps to Reproduce:

  1. Open an empty workspace
  2. Add a folder to the workspace via: Workspace: Add folder to workspace...
  3. Open dev tools and see a sea of red:
    image

This doesn't repro if the workspace is just a normal folder.

@TylerLeonhardt TylerLeonhardt added the bug Issue identified by VS Code Team member as probable bug label Jun 3, 2021
@lszomoru lszomoru added the workspace-trust Trusted workspaces label Jun 3, 2021
@lszomoru lszomoru added this to the May 2021 milestone Jun 3, 2021
@lszomoru
Copy link
Member

lszomoru commented Jun 3, 2021

@TylerLeonhardt, I am unable to reproduce the issue in Insiders and OSS.
Could you record a GIF as I have the feeling I am missing something. Thanks!

@TylerLeonhardt
Copy link
Member Author

I am adding a workspace that is trusted to an Empty Workspace… maybe that’s the key?

@sbatten
Copy link
Member

sbatten commented Jun 4, 2021

I could reproduce this with a trusted folder being added to a restricted mode empty workspace. However, I could only do so in Insiders but not in OSS. Did anything change in the last 24h?

@bpasero
Copy link
Member

bpasero commented Jun 4, 2021

See #69335

@lszomoru
Copy link
Member

lszomoru commented Jun 4, 2021

@bpasero, I am not seeing any commit from the last 24 hours linked on that issue.

@bpasero
Copy link
Member

bpasero commented Jun 4, 2021

@lszomoru yeah just wanted to point out why the extension host restarts. This has been like that since the beginning of multi root workspaces.

@lszomoru
Copy link
Member

lszomoru commented Jun 4, 2021

Got it. I was aware of the extension host restart.
Trying to figure out what changed in the last 24 hours that fixed the issue reported by @TylerLeonhardt

@lszomoru
Copy link
Member

lszomoru commented Jun 4, 2021

I got it to repro in Insiders but it does not repro in OSS. Will investigate the issue and try to get to the bottom of it.

@rzhao271 rzhao271 added the verified Verification succeeded label Jun 4, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Jul 19, 2021
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 insiders-released Patch has been released in VS Code Insiders verified Verification succeeded workspace-trust Trusted workspaces
Projects
None yet
Development

No branches or pull requests

7 participants
@bpasero @TylerLeonhardt @lszomoru @alexdima @sbatten @rzhao271 and others