Skip to content

[codex] Fix MCP validation contract issues - #3314

Merged
Widthdom merged 13 commits into
mainfrom
fix-issue3160-3182-3184-3186-3192-3194-3195-3196-3197-3198
Jun 6, 2026
Merged

[codex] Fix MCP validation contract issues#3314
Widthdom merged 13 commits into
mainfrom
fix-issue3160-3182-3184-3186-3192-3194-3195-3196-3197-3198

Conversation

@Widthdom

@Widthdom Widthdom commented Jun 6, 2026

Copy link
Copy Markdown
Owner

Summary

  • Tightened MCP argument validation for pagination bounds, since, impact query length, path/list bounds, required path handling, and project filter resolver failures.
  • Aligned tools/list schema constraints with runtime validation, including index path compatibility.
  • Added issue-specific regression coverage for MCP argument allowlist/schema/type contracts.
  • Added changelog fragments for each fixed issue.

Validation

  • dotnet build CodeIndex.sln
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter "FullyQualifiedName~McpServerTests|FullyQualifiedName~McpToolContractTests"
  • dotnet run --project tools/CodeIndex.Changelog -- check
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll status --check --json
  • Adversarial review completed; found schema/rate-limit ordering issues during review and fixed them before opening this PR.

Fixes #3198
Fixes #3197
Fixes #3196
Fixes #3195
Fixes #3194
Fixes #3192
Fixes #3186
Fixes #3184
Fixes #3182
Fixes #3160

@Widthdom
Widthdom marked this pull request as ready for review June 6, 2026 01:51
@Widthdom
Widthdom merged commit a30e9b9 into main Jun 6, 2026
10 checks passed
@Widthdom
Widthdom deleted the fix-issue3160-3182-3184-3186-3192-3194-3195-3196-3197-3198 branch June 6, 2026 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment