stacklok / toolhive / 20131515559

59%

Build Type

push

github

Committed by web-flow

Commit Message

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>

Run Details

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