Skip to content

Fix MCP diagnostic sanitization and protocol bounds - #3596

Merged
Widthdom merged 8 commits into
mainfrom
fix-issue3370-3425-3469-3505-3506
Jun 12, 2026
Merged

Fix MCP diagnostic sanitization and protocol bounds#3596
Widthdom merged 8 commits into
mainfrom
fix-issue3370-3425-3469-3505-3506

Conversation

@Widthdom

Copy link
Copy Markdown
Owner

Summary

  • Sanitized MCP tool, worker, and batch diagnostics so raw exception details are not echoed on the affected paths.
  • Coarsened HTTP MCP auth failure telemetry and normalized stdio/HTTP bearer token shape validation with matching docs.
  • Added bounded line reads for MCP stdio and worker protocols while preserving large-file protocol opt-ins.

Issues

Fixes #3370
Fixes #3425
Fixes #3469
Fixes #3505
Fixes #3506

Validation

  • dotnet build -p:UseSharedCompilation=false
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter "FullyQualifiedName~Issue3370|FullyQualifiedName~Issue3425|FullyQualifiedName~Issue3469|FullyQualifiedName~Issue3505|FullyQualifiedName~Issue3506|FullyQualifiedName~Issue3429|FullyQualifiedName~Issue3432|FullyQualifiedName~Issue3461|FullyQualifiedName~RunBatch_TooDeepJsonLine_ReturnsUsageError_Issue3022|FullyQualifiedName~StartInfo" -p:UseSharedCompilation=false
  • dotnet run --project tools/CodeIndex.Changelog -- check
  • git diff --check origin/main..HEAD
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll status --check --json
  • Codex adversarial review: final targeted re-review reported no blocking findings.

Notes

@Widthdom
Widthdom marked this pull request as ready for review June 12, 2026 05:10
@Widthdom
Widthdom merged commit 6b50d1f into main Jun 12, 2026
10 checks passed
@Widthdom
Widthdom deleted the fix-issue3370-3425-3469-3505-3506 branch June 12, 2026 05:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment