stacklok / toolhive / 23951051179
65%

DEFAULT BRANCH: main
Ran
Jobs 1
Files 601
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
coverage: 65.338% (+0.03%) from 65.309%
23951051179

push

github

web-flow 
Extract shared WorkloadReference helpers into controllerutil (#4520)

* Extract shared WorkloadReference helpers into controllerutil

The ReferencingWorkloads pattern was duplicated across four config
controllers with inconsistent sorting behavior: MCPTelemetryConfig and
MCPExternalAuthConfig sorted refs, but MCPToolConfig and MCPOIDCConfig
did not. This caused unnecessary API server writes when the same set of
workloads was discovered in a different list order across reconcile runs.

Extract SortWorkloadRefs, WorkloadRefsEqual, and
FindWorkloadRefsFromMCPServers into the shared controllerutil package
so all controllers use deterministic, consistent ordering. Each
controller's findReferencingWorkloads now delegates to the shared
helper, removing ~50 lines of duplicated list-filter-build logic.

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

* Add WorkloadKind constants to replace string literals

Define WorkloadKindMCPServer, WorkloadKindVirtualMCPServer, and
WorkloadKindMCPRemoteProxy constants in the API types and use them
across all controllers. This fixes a goconst lint violation where
"MCPServer" appeared as a raw string literal 4+ times.

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

---------

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

57 of 67 new or added lines in 7 files covered. (85.07%)

11 existing lines in 4 files now uncovered.

54399 of 83258 relevant lines covered (65.34%)

63.59 hits per line

Uncovered Changes

Lines Coverage ∆ File
4
46.23
 0.51% cmd/thv-operator/controllers/mcpoidcconfig_controller.go
2
93.18
 -0.66% cmd/thv-operator/pkg/controllerutil/config.go
1
56.9
 2.86% cmd/thv-operator/controllers/mcpexternalauthconfig_controller.go
1
60.87
 -0.46% cmd/thv-operator/controllers/mcptelemetryconfig_controller.go
1
50.79
 0.27% cmd/thv-operator/controllers/toolconfig_controller.go
1
60.64
 0.0% cmd/thv-operator/controllers/virtualmcpserver_controller.go

Coverage Regressions

Lines Coverage ∆ File
5
82.81
 0.0% pkg/vmcp/composer/workflow_engine.go
3
93.33
 -5.0% pkg/vmcp/server/sessionmanager/cache.go
2
73.63
 -0.64% pkg/runner/config.go
1
46.23
 0.51% cmd/thv-operator/controllers/mcpoidcconfig_controller.go
Jobs
ID Job ID Ran Files Coverage
1 23951051179.1 601
65.34
 GitHub Action Run
Source Files on build 23951051179