Allow bulk operations for all apps in an app catalog, Closes #442 by nicodecleyre · Pull Request #689 · pnp/vscode-viva

nicodecleyre · 2026-01-13T23:38:46Z

🎯 Aim

This issue allows bulk operations for all apps in an app catalog

📷 Result

✅ What was done

bulk operations to:
Deploy
Retract
Remove
Enable
Disable
Install
Uninstall
Upgrade

🔗 Related issue

Closes: #442

Adam-it

@nicodecleyre awesome work!
I played around with it a bit and checked most of the bulk options

What we forgot in this PR is updating the docs. Since docs are now part of our code we should update it together with the PR so when it gets shipped we wont miss to align the documenatation.

Also I would move the buld operations to a new SpgxAppCLIBulkAction.ts file to make for better maintability so we dont make the SpgxAppCLIAction to long

Adam-it

@nicodecleyre ready for another round?
I rechecked the PR and left a few comments we could recheck. Let me know if you need any help

Adam-it · 2026-02-15T20:17:04Z

We just had a new minor release. We should rebase this branch, as well, before we proceed

Adam-it

@nicodecleyre I added a few fixups but the PR is not ready yet.
I think during the refactor in the PR there was a logical error introduced in which in many places the url to site level app catalog is introduced instead of the url to the site

I am still investigating this but from what I tested most of the bulk operations won't work due to this error. Maybe you wanna check this case as well.

Also shouldn't those bulk operations be also visible on tenant level app catalog?
I think it should that is why in SpfxAppCLIBulActions.ts in lines 603-618 we handle the case in which the app catalog is null and then the user may specify the relative site URL to which the app should be installed/uninstalled 🤔

Adam-it · 2026-03-30T23:46:42Z

we also need to rebase this branch against latest dev since we just had a minor release

Adam-it · 2026-05-21T23:37:41Z

We just did a minor release of SPFx Toolkit, agian. Please rebase this branch to align with latest dev before we proceed

Adam-it · 2026-07-28T18:44:05Z

@nicodecleyre we had another minor release and we need to rebase this one again.
Also this PR is open for a long time and is quite inactive. Do you think it still makes sense? Maybe if we do not have time to commit to this feature lets close the PR and reopen the issue?

nicodecleyre · 2026-08-02T16:13:56Z

@nicodecleyre we had another minor release and we need to rebase this one again. Also this PR is open for a long time and is quite inactive. Do you think it still makes sense? Maybe if we do not have time to commit to this feature lets close the PR and reopen the issue?

Ready

Adam-it

@nicodecleyre unfortunately I still noticed a few issues with the PR a few of them quite important we need to address before we merge. Let's give this another go

Adam-it · 2026-10-02T14:04:47Z

hey we just had a new minor release which means we aligned out main branch with dev. Please perform a rebase so we may proceed with a PR. You Rock 🤩

Adds bulk deploy/retract/remove/enable/disable/install/uninstall/upgrade commands for app catalog apps, and fixes getAppCatalogUrlFromNode to strip the /AppCatalog suffix and return undefined for the tenant catalog node.

Adam-it

LGTM 🔥

nicodecleyre changed the base branch from main to dev January 13, 2026 23:38

nicodecleyre force-pushed the issue-442 branch from cb2b2f5 to d45edba Compare January 13, 2026 23:39

Adam-it self-assigned this Jan 15, 2026

nicodecleyre force-pushed the issue-442 branch from d45edba to 7bda5bb Compare January 15, 2026 22:18

Adam-it requested changes Jan 15, 2026

View reviewed changes

Adam-it marked this pull request as draft January 15, 2026 23:26

nicodecleyre force-pushed the issue-442 branch 3 times, most recently from fba1b4d to 609cf4a Compare February 3, 2026 19:29

nicodecleyre marked this pull request as ready for review February 3, 2026 19:43

Adam-it force-pushed the issue-442 branch from 609cf4a to db415ee Compare February 13, 2026 01:12

Adam-it requested changes Feb 13, 2026

View reviewed changes

Adam-it marked this pull request as draft February 13, 2026 21:31

Adam-it force-pushed the dev branch 2 times, most recently from 2c84a0b to fd39e59 Compare February 15, 2026 20:15

nicodecleyre force-pushed the issue-442 branch 2 times, most recently from 33028e8 to a988805 Compare February 22, 2026 15:33

nicodecleyre marked this pull request as ready for review February 22, 2026 15:51

Adam-it force-pushed the issue-442 branch from a988805 to addd103 Compare March 17, 2026 21:31

Adam-it requested changes Mar 20, 2026

View reviewed changes

Adam-it marked this pull request as draft March 20, 2026 22:44

Adam-it force-pushed the dev branch from a06fbb6 to 6fc9093 Compare March 30, 2026 23:43

Adam-it force-pushed the dev branch from 6225314 to 50d7a0c Compare May 21, 2026 23:35

nicodecleyre force-pushed the issue-442 branch from c82a8e4 to a20694d Compare June 25, 2026 20:18

Adam-it force-pushed the dev branch 2 times, most recently from 90ba561 to 4d8d770 Compare July 28, 2026 18:40

nicodecleyre force-pushed the issue-442 branch from a20694d to 261d1d1 Compare August 2, 2026 16:05

nicodecleyre marked this pull request as ready for review August 2, 2026 16:13

Adam-it requested changes Sep 7, 2026

View reviewed changes

Adam-it marked this pull request as draft September 7, 2026 20:13

nicodecleyre force-pushed the issue-442 branch from 7a1d3f1 to 7c547ef Compare September 14, 2026 18:23

Adam-it force-pushed the dev branch from dc991c5 to 066379d Compare October 2, 2026 14:03

nicodecleyre force-pushed the issue-442 branch from d5f6fb7 to de5e698 Compare October 4, 2026 08:07

nicodecleyre force-pushed the issue-442 branch from de5e698 to 6efe423 Compare October 18, 2026 19:11

nicodecleyre marked this pull request as ready for review October 18, 2026 19:22

nicodecleyre force-pushed the issue-442 branch from 6efe423 to 0741bc6 Compare November 5, 2026 11:48

nicodecleyre force-pushed the dev branch from 8bc7207 to 3f99736 Compare November 5, 2026 12:00

nicodecleyre force-pushed the issue-442 branch from 0741bc6 to 4b22050 Compare November 5, 2026 12:06

nicodecleyre added 4 commits November 11, 2026 00:45

issue42

31fced0

Add bulk app catalog actions and fix site URL extraction

1a45e74

Adds bulk deploy/retract/remove/enable/disable/install/uninstall/upgrade commands for app catalog apps, and fixes getAppCatalogUrlFromNode to strip the /AppCatalog suffix and return undefined for the tenant catalog node.

Changes

86c75c8

Remove accidentally committed nested repo

fdcd8e0

Adam-it force-pushed the issue-442 branch from 4b22050 to fdcd8e0 Compare November 10, 2026 23:45

Adam-it added 2 commits November 11, 2026 22:43

fixup

2dcd751

fixup

38bdd22

Adam-it approved these changes Nov 11, 2026

View reviewed changes

Adam-it merged commit ca989e0 into pnp:dev Nov 11, 2026
1 check passed

Conversation

nicodecleyre commented Jan 13, 2026

🎯 Aim

📷 Result

✅ What was done

🔗 Related issue

Uh oh!

Adam-it left a comment

Choose a reason for hiding this comment

Uh oh!

Adam-it left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Adam-it commented Feb 15, 2026

Uh oh!

Adam-it left a comment

Choose a reason for hiding this comment

Uh oh!

Adam-it commented Mar 30, 2026

Uh oh!

Adam-it commented May 21, 2026

Uh oh!

Adam-it commented Jul 28, 2026

Uh oh!

nicodecleyre commented Aug 2, 2026

Uh oh!

Adam-it left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Adam-it commented Oct 2, 2026

Uh oh!

Adam-it left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants