stacklok / toolhive / 24445492145

66%

Build Type

push

github

Committed by web-flow

Commit Message

Add GetContent API for on-the-fly SKILL.md retrieval from OCI and git sources (#4810)

* Add GetContent method to SkillService for on-the-fly SKILL.md retrieval

Add ContentOptions, SkillContent, and SkillFileEntry types to the skills
package and extend the SkillService interface with GetContent. Regenerate
the mock to match.

* Implement GetContent with OCI, git, and registry name resolution

GetContent resolves skill content without installing by trying, in order:
git:// and https:// references via the git resolver, the local OCI store
and remote OCI pull, and finally a registry catalog name lookup that
handles both plain names and qualified namespace/name identifiers.

Also fixes resolveRegistryPackages to append Subfolder as a #path
fragment to git URLs, and updates resolveFromRegistry to support
qualified namespace/name references.

* Add tests for GetContent git resolution and registry fallback

Cover git reference dispatch, git resolver failures, nil resolver guard,
registry name fallback to git, qualified namespace/name filtering,
splitQualifiedName, and resolveRegistryPackages subfolder handling.

* Update swagger.json with GetContent endpoint and schemas

* Fix gci import ordering, line length, and regenerate swagger docs

* Address code review: guard registry lookup, fix doc comment, add 502 to swagger

Skip registry name lookup for refs containing : or @ to avoid a wasted
network round-trip when a fully-qualified OCI reference fails. Fix the
GetContent doc comment to reflect the actual resolution order. Document
the 502 Bad Gateway response in the swagger annotation.

* Fix gci import grouping to match project config

Coverage Stats

138 of 196 new or added lines in 4 files covered. (70.41%)

3 existing lines in 1 file now uncovered.

57397 of 87369 relevant lines covered (65.69%)

61.62 hits per line