Skip to content

SP-1297: Add download agent skill bundle command#396

Open
Thomas Neumüller (tneum-celonis) wants to merge 1 commit into
mainfrom
SP-1297-download-agent-skill-bundle
Open

SP-1297: Add download agent skill bundle command#396
Thomas Neumüller (tneum-celonis) wants to merge 1 commit into
mainfrom
SP-1297-download-agent-skill-bundle

Conversation

@tneum-celonis

@tneum-celonis Thomas Neumüller (tneum-celonis) commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Description

Adds content-cli asset-registry skills download --path <path> [--output <parentDir>], which installs a complete agent skill bundle (SKILL.md + all reference files) to the local filesystem in one command, preserving folder structure and overwriting existing files.

Builds on SP-348's single-file skills get: a new listSkillFiles API call to GET /pacman/api/core/asset-registry/skills/{path}/files fetches the manifest, then each entry is downloaded via the existing getSkillFile and written under <output>/<skillName>/<relativePath>. Includes a path-traversal guard so malformed manifest entries cannot escape the target directory. FileService.writeBufferToPath now mkdirs the file's parent (backwards-compatible for the existing basename-only callers).

Relevant links

Checklist

  • I have self-reviewed this PR
  • I have tested the change and proved that it works in different scenarios
  • I have updated docs if needed

@sonarqubecloud

Copy link
Copy Markdown

@tneum-celonis Thomas Neumüller (tneum-celonis) marked this pull request as ready for review July 13, 2026 14:53
Download a platform skill into the current directory:

```
content-cli asset-registry skills download --path platform/content-cli-setup

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We already have content-cli asset-registry skills get --path {path of the skill} --file {specific file}. Adding content-cli asset-registry skills download seems confusing, since we can just change the behavior of asset-registry skills get to download the whole bundle when the --file option is ommitted.

cc: Zgjim Haziri (@ZgjimHaziri) Kastriot Salihu (@ksalihu) Would appreciate your opinions here too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants