vMCP: Kubernetes - VirtualMCPCompositeToolDefinition CRD

[yrobla]

Design and implement the VirtualMCPCompositeToolDefinition CRD for defining reusable composite workflows that can be referenced by multiple VirtualMCPServer instances.

Closes: #2433

[Copilot]

Pull Request Overview

This PR introduces a new Kubernetes CRD VirtualMCPCompositeToolDefinition that enables defining reusable composite workflows for Virtual MCP Servers, allowing complex multi-step operations to be referenced across multiple VirtualMCPServer instances.

• Adds comprehensive documentation guide for the new CRD with examples and best practices
• Implements CRD definition with validation rules for workflow structure, parameters, steps, and dependencies
• Creates validation webhook with cycle detection, template validation, and error handling checks

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
docs/operator/virtualmcpcompositetooldefinition-guide.md	Comprehensive user guide with examples, validation rules, and troubleshooting
deploy/charts/operator-crds/crds/toolhive.stacklok.dev_virtualmcpservers.yaml	Adds compositeToolRefs field to reference composite tool definitions
deploy/charts/operator-crds/crds/toolhive.stacklok.dev_virtualmcpcompositetooldefinitions.yaml	New CRD definition with OpenAPI schema validation
cmd/thv-operator/api/v1alpha1/virtualmcpserver_types.go	Adds CompositeToolRefs field to VirtualMCPServerSpec
cmd/thv-operator/api/v1alpha1/virtualmcpcompositetooldefinition_types.go	Type definitions for the new CRD including spec, status, and supporting types
cmd/thv-operator/api/v1alpha1/virtualmcpcompositetooldefinition_webhook.go	Validation logic for workflow definitions including cycle detection
cmd/thv-operator/api/v1alpha1/virtualmcpcompositetooldefinition_webhook_test.go	Comprehensive test coverage for validation logic
cmd/thv-operator/api/v1alpha1/zz_generated.deepcopy.go	Generated DeepCopy methods for new types

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[codecov]

Codecov Report

❌ Patch coverage is 59.43060% with 114 lines in your changes missing coverage. Please review.
✅ Project coverage is 54.87%. Comparing base (e2c67bb) to head (5a6af7b).
⚠️ Report is 28 commits behind head on main.

Files with missing lines	Patch %	Lines
...lpha1/virtualmcpcompositetooldefinition_webhook.go	71.73%	54 Missing and 11 partials ⚠️
cmd/thv-operator/main.go	0.00%	39 Missing ⚠️
...-operator/api/v1alpha1/virtualmcpserver_webhook.go	0.00%	10 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2412      +/-   ##
==========================================
+ Coverage   54.54%   54.87%   +0.33%     
==========================================
  Files         264      274      +10     
  Lines       25351    26001     +650     
==========================================
+ Hits        13828    14269     +441     
- Misses      10210    10401     +191     
- Partials     1313     1331      +18     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Copilot]

Pull Request Overview

Copilot reviewed 15 out of 15 changed files in this pull request and generated 2 comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.