Skip to content

feat(mcp): add structured tool output behind mcp.output_schema flag - #3136

Draft
asoorm wants to merge 3 commits into
mainfrom
ahmet/router-592-mcp-add-structured-tool-output-outputschema
Draft

feat(mcp): add structured tool output behind mcp.output_schema flag#3136
asoorm wants to merge 3 commits into
mainfrom
ahmet/router-592-mcp-add-structured-tool-output-outputschema

Merge branch 'main' into ahmet/router-592-mcp-add-structured-tool-out…

c7d38ab
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Aug 7, 2026 in 0s

62.57% (target 30.00%)

View this Pull Request on Codecov

62.57% (target 30.00%)

Details

Codecov Report

❌ Patch coverage is 82.20339% with 42 lines in your changes missing coverage. Please review.
✅ Project coverage is 62.57%. Comparing base (64eaf60) to head (c7d38ab).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
router/pkg/mcpserver/response_schema.go 79.80% 25 Missing and 17 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3136      +/-   ##
==========================================
+ Coverage   62.37%   62.57%   +0.20%     
==========================================
  Files         262      263       +1     
  Lines       31003    31232     +229     
==========================================
+ Hits        19337    19544     +207     
- Misses      10158    10165       +7     
- Partials     1508     1523      +15     
Files with missing lines Coverage Δ
router/core/router.go 71.00% <100.00%> (+0.01%) ⬆️
router/pkg/config/config.go 83.00% <ø> (ø)
router/pkg/mcpserver/server.go 74.91% <100.00%> (+4.35%) ⬆️
router/pkg/mcpserver/response_schema.go 79.80% <79.80%> (ø)

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.