1
66%
main: 66%

Ran 28 Apr 2026 07:59AM UTC

Files 671

Run time 19s

Badge

Embed ▾

Committed 28 Apr 2026 07:53AM UTC coverage: 66.216% (+0.004%) from 66.212%

Job # 25040852647.1

Build Type

push

github

Committed by

web-flow

Commit Message

Classify packager errors as 400 in skill build handler (#5076)

* Bump github.com/stacklok/toolhive-core to v0.0.17

v0.0.17 introduces sentinel errors in oci/skills (ErrSkillMDMissing,
ErrInvalidFrontmatter, ErrInvalidSkillDir, ErrInvalidSkillFile,
ErrTooManyFiles, ErrSkillTooLarge) so callers can classify packager
failures with errors.Is instead of matching error strings. Pulled in
ahead of the build-handler refactor that consumes them.

* Classify packager errors as 400 in skill build handler

POST /api/v1beta/skills/build previously surfaced every packager
failure — including user-input mistakes like a missing SKILL.md or a
malformed frontmatter — as 500 Internal Server Error with no body,
generating Sentry noise on what is plainly bad input.

Wrap the s.packager.Package error with an errors.Is switch over the
six toolhive-core v0.0.17 sentinels and map each one to 400 with the
packager's message preserved. Anything else stays a 500.

Add five TestBuild cases that mock the packager returning a wrapped
sentinel and assert both the HTTP code and errors.Is on the sentinel,
covering the limits the toolhive-side validator misses (symlink in
skill dir, oversized dir).

Coverage Stats

59388 of 89688 relevant lines covered (66.22%)

62.42 hits per line

stacklok / toolhive / 25040852647 / 1
66%
main: 66%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 25040852647.1

stacklok / toolhive / 25040852647 / 1 66% main: 66%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 25040852647.1

stacklok / toolhive / 25040852647 / 1
66%
main: 66%

README BADGES
x