Add site cartridge path management commands#295
Merged
Conversation
Add `sites cartridges list|add|remove|set` commands with SDK operations for managing the ordered list of active cartridges on a site. Includes `--bm` flag for Business Manager support and automatic fallback to site archive import/export when OCAPI permissions are unavailable.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
sites cartridges list|add|remove|setCLI commands and corresponding SDK operations (getCartridgePath,addCartridge,removeCartridge,setCartridgePath) for managing the ordered list of active cartridges on a site--bmflag (Sites-Site), which uses site archive import/export since OCAPI only supports reading for BM/sites/*/cartridges) are unavailable, with actionable error messages when both paths failaddisfirst; supportsfirst,last,before,afterwith--targetCartridgeUpdateOptionswithlogcallback for phase messages andwaitOptions.onProgressfor job polling, consistent withjob import/job exportidiomCloses #194
Test plan
b2c sites cartridges list --site-id <id>returns numbered cartridge pathb2c sites cartridges list --bmreturns BM cartridge pathb2c sites cartridges add <name> --site-id <id>adds via OCAPIb2c sites cartridges add <name> --bmadds via site archive import with progress outputb2c sites cartridges remove <name> --site-id <id>removes cartridgeb2c sites cartridges set "a:b:c" --bmreplaces entire BM path--jsonflag returns structuredCartridgePathResultfor all commandssites cartridge list)pnpm run test:agent