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

stacklok / toolhive / 20131515559
59%

Build:
DEFAULT BRANCH: main
Ran 11 Dec 2025 11:28AM UTC
Jobs 1
Files 401
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

11 Dec 2025 11:26AM UTC coverage: 49.489% (+0.003%) from 49.486%
20131515559

push

github

web-flow
Remove redundant ToolType field from Workload (#2932)

* Remove redundant ToolType field from Workload

The ToolType field in the Workload struct mapped to either "mcp"
(container workloads) or "remote" (remote workloads), but since MCP
is the only alternative type in practice, the field has become
unnecessary and adds no value.

This change eliminates the redundant field from the codebase,
simplifying the business logic that previously checked both ToolType
and Remote fields. The condition "ToolType != 'mcp' && !Remote" now
simplifies to just checking the Remote field directly.

Changes:
- Remove ToolType field from core.Workload struct
- Delete LabelToolType constant and GetToolType function from labels package
- Simplify shouldSkipWorkload logic in client manager
- Remove ToolType from CLI list command output
- Remove tool_type from backend metadata (workloads and vMCP)
- Update all affected tests
- Update API documentation (swagger.yaml) and regenerate docs

Fixes #2923

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude <noreply@anthropic.com>
Signed-off-by: carlos <carlosgn@protonmail.com>

* Remove ToolType property

Signed-off-by: carlos <21148423+carlos-gn@users.noreply.github.com>

---------

Signed-off-by: carlos <carlosgn@protonmail.com>
Signed-off-by: carlos <21148423+carlos-gn@users.noreply.github.com>
Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: Muhammad Amir Ejaz <amir@stacklok.com>
Co-authored-by: Eleftheria Stein-Kousathana <eleftheria@stacklok.com>
Co-authored-by: Jakub Hrozek <jakub.hrozek@posteo.se>

1 of 2 new or added lines in 2 files covered. (50.0%)

1 existing line in 1 file now uncovered.

26807 of 54168 relevant lines covered (49.49%)

57.98 hits per line

New Missed Lines in Diff

Lines Coverage ∆ File
1
0.0
0.0% pkg/client/manager.go

Uncovered Existing Lines

Lines Coverage ∆ File
1
80.11
-0.76% pkg/labels/labels.go
Jobs
ID Job ID Ran Files Coverage
1 20131515559.1 11 Dec 2025 11:28AM UTC 401
49.49
GitHub Action Run
Source Files on build 20131515559
  • Tree
  • List 401
  • Changed 9
  • Source Changed 8
  • Coverage Changed 8
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • Github Actions Build #20131515559
  • 6cfd20a7 on github
  • Prev Build on main (#20128217091)
  • Next Build on main (#20131526076)
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