Skip to content

Fix MCP argument contracts and batch limits - #3588

Merged
Widthdom merged 4 commits into
mainfrom
fix-issue3538-3436-3539-3406
Jun 11, 2026
Merged

Fix MCP argument contracts and batch limits#3588
Widthdom merged 4 commits into
mainfrom
fix-issue3538-3436-3539-3406

Conversation

@Widthdom

Copy link
Copy Markdown
Owner

Summary

  • Align MCP tool schemas and validation metadata for aliases, scalar excludePaths, and compatible JSON type hints.
  • Report MCP argument clamps/ignored values through warnings and argument_adjustments.
  • Add batch_query slot limits, response-budget controls, estimate mode, slot ids, and resumable truncation hints.
  • Warn on invalid MCP tool filter names and fail closed for empty explicit allowlists.

Validation

  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter "FullyQualifiedName~McpToolContractTests|FullyQualifiedName~McpServerTests"
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter "FullyQualifiedName~Mcp" (via required Codex adversarial review)
  • dotnet run --project tools/CodeIndex.Changelog -- check
  • git diff --check origin/main..HEAD
  • dotnet format CodeIndex.sln --verify-no-changes
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll index . --json
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll status --check --json
  • Required Codex adversarial review: No blocking/actionable issues found.

Full dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj was attempted locally earlier but cannot complete in this environment because the installed runtimes do not include Microsoft.NETCore.App 10.0.8-servicing.26229.119, which the current ILLink dependency asks for during publish-trimmed tests.

Fixes #3538
Fixes #3436
Fixes #3539
Fixes #3406

@Widthdom
Widthdom marked this pull request as ready for review June 11, 2026 23:24
@Widthdom
Widthdom merged commit 755f19d into main Jun 11, 2026
10 checks passed
@Widthdom
Widthdom deleted the fix-issue3538-3436-3539-3406 branch June 11, 2026 23:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant