Add tenant app catalog & add/remove site app catalog. Closes #360

[nicodecleyre]

Soooooo, i did a little oopsie woopsie and totally broke the other PR 😅

🎯 Aim

This feature adds the possibility to add a tenant app catalog of it doesn't exist and the possibility to add or remove a site app catalog

📷 Result

✅ What was done

clarify what was done and what still needs to be finished ex. [Remove this line]

• Adding feature to add a tenant app catalog - if none exist
• Adding feature to add a site app catalog
• Adding feature to remove a site app catalog
• A feature to choose from existing sites as dropdown when adding a site catalog - this would be a nice issue - currently i didn't succeed in this as i couldn't parse the looooong json getting back from the cli command sites list

🔗 Related issue

Closes: #360

[Copilot]

Pull Request Overview

This PR adds functionality to manage tenant and site app catalogs by allowing creation and removal via CLI commands. Key changes include:

• Registration and implementation of new commands for adding a tenant app catalog, adding a site app catalog, and removing a site app catalog.
• Updates to UI components in the CommandPanel to trigger these new commands.
• Addition of new context keys and command constants to support the new functionality.

Reviewed Changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.

File	Description
src/services/actions/CliActions.ts	Added new functions for adding/removing app catalogs and command registration.
src/panels/CommandPanel.ts	Updated command panel UI to support the new app catalog commands.
src/constants/ContextKeys.ts	Introduced a new context key for missing an app catalog.
src/constants/Commands.ts	Added new command constants for app catalog actions.

Files not reviewed (1)

• package.json: Language not supported

Comments suppressed due to low confidence (1)

src/panels/CommandPanel.ts:212

• The context key for the tenant app catalog node ('sp-tenant-app-catalog-url') differs from the site app catalog key ('sp-app-catalog-url'); please confirm that this naming distinction is intentional and document the rationale if needed.

new ActionTreeItem(tenantAppCatalogUrl.replace(origin, '...'), '', { name: 'globe', custom: false }, showExpandTreeIcon, 'vscode.open', `${Uri.parse(tenantAppCatalogUrl)}/AppCatalog`, 'sp-tenant-app-catalog-url', undefined,

[Adam-it]

@nicodecleyre Aweomse work 👏👏👏👏
I tested it out locally and I have one feed regarding the removal flow we could recheck before we proceed.
Also added some tiny comments to the code

Also for remove I think it would be best to use the trash icon

[Adam-it]

@nicodecleyre we just had another minor release due to SPFx version update. May I kindly ask you to rebase before we proceed

[Adam-it]

@nicodecleyre can you provide some update on this PR? Is there anything I may help you with or needs more clarity to proceed?