stacklok / toolhive / 23738160058

65%

Build Type

push

github

Committed by web-flow

Commit Message

Default to :latest tag for OCI skill install references (#4430)

When a user runs `thv skill install ghcr.io/org/skill` without a tag,
go-containerregistry defaults to "latest" internally but String() omits
it. The raw user input was passed to oras-go's registry.Pull which
requires an explicit tag, causing a confusing "Bad Gateway" error.

Build the qualified OCI reference from the parsed nameref.Reference,
appending `:latest` only when the user omitted a tag. Also change the
pull error status from 502 to 400 since a bad reference is a client
input error, not an upstream gateway failure.

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Coverage Stats

11 of 11 new or added lines in 1 file covered. (100.0%)

32 existing lines in 7 files now uncovered.

52654 of 80428 relevant lines covered (65.47%)

65.29 hits per line