stacklok / toolhive / 28119447922
67%

DEFAULT BRANCH: main
Ran
Jobs 1
Files 764
Run time 3min
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: 67.07% (+0.01%) from 67.056%
28119447922

push

github

web-flow 
Index referencing-workload lookups across config controllers (#5599)

* Index ToolConfig and Webhook referencing-workload lookups

Replace the list-all-and-filter scans in MCPToolConfig and MCPWebhookConfig
findReferencing* with field-indexed MatchingFields lookups, mirroring the
merged MCPOIDCConfig template. Both reference only MCPServers, so each
registers one index (spec.toolConfigRef / spec.webhookConfigRef) in
SetupWithManager and queries it directly.

Behavior unchanged; see #5607.

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

* Index Telemetry and Authz referencing-workload lookups

Rewrite MCPTelemetryConfig and MCPAuthzConfig findReferencingWorkloads to use
field-indexed MatchingFields lookups instead of listing all workloads and
filtering in memory, mirroring the merged MCPOIDCConfig template. Each
registers indexes for MCPServer, MCPRemoteProxy, and VirtualMCPServer in
SetupWithManager (telemetryConfigRef top-level on the VMCP spec; authzConfigRef
nested under incomingAuth for the VMCP).

Behavior unchanged; see #5607.

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

* Index ExternalAuth referencing-workload lookups

MCPExternalAuthConfig is referenced through two workload spec fields
(externalAuthConfigRef and authServerRef when Kind=MCPExternalAuthConfig), so
findReferencingMCPServers / findReferencingMCPRemoteProxies ran two list+filter
passes and merged with manual dedup. Replace each with a single field-indexed
MatchingFields query backed by one combined index per workload type whose
extractor returns both referenced names, deduplicated — preserving the
merge/dedup semantics while dropping the per-field scans.

Behavior unchanged; see #5607.

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

---------

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

131 of 235 new or added lines in 5 files covered. (55.74%)

321 existing lines in 11 files now uncovered.

69611 of 103789 relevant lines covered (67.07%)

64.17 hits per line

Uncovered Changes

Lines Coverage ∆ File
37
41.13
 -1.83% cmd/thv-operator/controllers/mcptelemetryconfig_controller.go
25
78.14
 -3.81% cmd/thv-operator/controllers/mcpauthzconfig_controller.go
22
65.18
 -2.31% cmd/thv-operator/controllers/mcpexternalauthconfig_controller.go
10
71.21
 -1.74% cmd/thv-operator/controllers/mcpwebhookconfig_controller.go
10
52.17
 1.4% cmd/thv-operator/controllers/toolconfig_controller.go

Coverage Regressions

Lines Coverage ∆ File
161
37.91
 -0.93% pkg/vmcp/config/zz_generated.deepcopy.go
79
84.79
 -0.27% pkg/vmcp/server/server.go
51
34.9
 -0.07% pkg/vmcp/cli/serve.go
11
82.44
 -8.4% cmd/thv-operator/pkg/controllerutil/config.go
6
74.42
 0.0% pkg/script/internal/builtins/parallel.go
3
41.13
 -1.83% cmd/thv-operator/controllers/mcptelemetryconfig_controller.go
3
76.7
 -2.91% pkg/state/local.go
2
65.18
 -2.31% cmd/thv-operator/controllers/mcpexternalauthconfig_controller.go
2
71.43
 0.0% pkg/state/runconfig.go
2
82.29
 -0.21% pkg/vmcp/composer/workflow_engine.go
1
78.14
 -3.81% cmd/thv-operator/controllers/mcpauthzconfig_controller.go
Jobs
ID Job ID Ran Files Coverage
1 28119447922.1 764
67.07
 GitHub Action Run
Source Files on build 28119447922