Skip to content

Process explorer: remove node.js/electron dependencies #101832

Closed
@bpasero

Description

@bpasero

The process explorer depends on:

  • Electron clipboard
  • os node.js module (totalMem)

This should be cleaned up to be ready for sandbox by leveraging the IElectronService instead. You can take a look at the issue reporter for how to get a hold of the electron service:

this.electronService = new ElectronService(configuration.windowId, mainProcessService) as IElectronService;

I think it would be reasonable to add another method to the electron service for accessing os.totalMem().

Metadata

Metadata

Labels

insiders-releasedPatch has been released in VS Code InsiderssandboxRunning VSCode in a node-free environmentworkbench-diagnosticsGeneral VS Code built-in diagnostic issues

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions