• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

stacklok / toolhive / 23542795780
65%

Build:
DEFAULT BRANCH: main
Ran 25 Mar 2026 01:17PM UTC
Jobs 1
Files 578
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

25 Mar 2026 01:12PM UTC coverage: 65.307% (+0.05%) from 65.261%
23542795780

push

github

web-flow
Wire git resolver into skill install flow (#4338)

* Wire git resolver into skill install flow

The gitresolver package was fully implemented but not connected to the
main install path. This change integrates it so users can install skills
from git repositories via direct references or registry fallback.

Changes:
- Add WithGitResolver option and installFromGit method to skillsvc
- Detect git:// prefix in Install() before OCI reference check
- Extend resolveFromRegistry to handle "git" package types alongside OCI
- Add dev-mode SSRF bypass in validateHost for E2E testability
- Generate mock for gitresolver.Resolver interface
- Add unit tests for all git install paths (fresh, upgrade, no-op, errors)
- Add E2E tests with local dumb-HTTP git server infrastructure

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

* Reduce cyclomatic complexity in skill install functions

Extract helpers to bring Install, installFromGit, and resolveFromRegistry
below the gocyclo threshold of 15:
- installByName: plain-name install flow from Install
- installFromRegistryLookup: registry resolution dispatch
- applyGitInstall: store check + create/upgrade branching
- writeAndPersistGitSkill: shared write + persist logic
- resolveRegistryPackages: OCI/git package selection

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

* Roll back DB record on group registration failure and add missing tests

If registerSkillInGroup fails after a successful install, the skill was
left installed on disk and in the DB but not added to the expected group.
Add installAndRegister helper that wraps all five call sites with a
best-effort store.Delete rollback so retries start fresh.

Add unit tests for validateHost dev-mode SSRF bypass, git reference
URL conversion edge cases, and group registration rollback behavior.

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

* Extract repeated commit hash string into test constant

Fixes gocon... (continued)

204 of 237 new or added lines in 4 files covered. (86.08%)

8 existing lines in 3 files now uncovered.

51407 of 78716 relevant lines covered (65.31%)

71.72 hits per line

New Missed Lines in Diff

Lines Coverage ∆ File
4
13.94
-0.1% pkg/api/server.go
29
88.32
-1.1% pkg/skills/skillsvc/skillsvc.go

Uncovered Existing Lines

Lines Coverage ∆ File
2
94.67
-1.33% pkg/vmcp/composer/dag_executor.go
2
71.43
-1.68% pkg/vmcp/k8s/manager.go
4
80.8
-0.84% pkg/transport/proxy/transparent/transparent_proxy.go
Jobs
ID Job ID Ran Files Coverage
1 23542795780.1 25 Mar 2026 01:17PM UTC 578
65.31
GitHub Action Run
Source Files on build 23542795780
  • Tree
  • List 578
  • Changed 8
  • Source Changed 4
  • Coverage Changed 8
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • Github Actions Build #23542795780
  • f5056210 on github
  • Prev Build on main (#23541869769)
  • Next Build on main (#23545538692)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc