Support private extension packs and private extension dependencies

[lachaib]

Hello,

While trying to bundle several private extensions along as extensionPack or extensionDependencies, I discovered that it's not possible to install private extensions through them. (Btw, while looking at the code I found a typo)

From what I read in installExtensionByPackage, what would need to be done is to extract from manifest for fields extensionPack and extensionDependencies, and for each element in the lists, try to see if they can be found in any registry, in which case they should be also be installed by installExtensionById. If not found in registry, call workbench.extensions.installExtension command as done currently.

This would also be a good opportunity for private registries to be a cache for large extensions.

Thanks

[kumar121193]

@joelspadin-garmin This would be a beneficial addition to the manager. In my company, we have multiple private extensions and want to create a private extension pack that bundles private and public extensions. As @lachaib is suggesting, the code changes would be to read through the extensionPack and extensionDependencies before executing workbench.extensions.installExtension, and installing any private extensions available in the registry.

[kumar121193]

@joelspadin-garmin Please let me know if this enhancement can be added. I am happy to contribute if needed.

[joelspadin-garmin]

Sorry for the late response. Yes, this does sound like something that could be useful. I don't have time to implement it though.

[lachaib]

I can make the contribution but I've changed company since and won't be able to test it. Anyone who'd be eager to test?

[kumar121193]

@lachaib @joelspadin-garmin Any update on this?

[josiezhan]

I can make the contribution but I've changed company since and won't be able to test it. Anyone who'd be eager to test?

I've tested it.It's useful。

[lachaib]

I can make the contribution but I've changed company since and won't be able to test it. Anyone who'd be eager to test?

I've tested it.It's useful。

Thanks @KINGZJS Can you mention your test in the pull request as well? Now we'll need @joelspadin-garmin to merge and release it.