Closed joadoumie closed 1 day ago
Here's a weird wacky thing today:
Every time we come back to the main page, we call IExtension.GetProvider
. And in the sample, we always instantiate a new one on the call to that. So every extension we have? Instantiates a new object each time we get to the main page.
Twofold fixes:
ICommandProvider
instance that they return out.TopLevelCommands
, but GetProvider
? We should already have that!
Description of the new feature / enhancement
We need to conduct a deep dive into how we manage extensions over the course of their lifetime.
Extension processes should not remain around for the entire lifecycle of the palette itself. As part of this investigation, we should consider:
Scenario when this would be used?
This will be abstracted away from extension developers, but will become core to the command palette
Supporting information
No response