Skip to content

Fix LSP textDocument URI length cap - #3299

Merged
Widthdom merged 7 commits into
mainfrom
fix-issue3129
Jun 5, 2026
Merged

Fix LSP textDocument URI length cap#3299
Widthdom merged 7 commits into
mainfrom
fix-issue3129

Conversation

@Widthdom

@Widthdom Widthdom commented Jun 5, 2026

Copy link
Copy Markdown
Owner

Summary

  • Reject oversized LSP textDocument.uri values before URI parsing and path normalization, using the MCP resource URI cap.
  • Add an LSP regression test for the bounded -32602 error response.
  • Document the LSP URI cap and add the unreleased changelog fragment.

Validation

  • dotnet build
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter FullyQualifiedName~LspServerTests
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter FullyQualifiedName~McpAuditLogTests
  • dotnet run --project tools/CodeIndex.Changelog -- check
  • dotnet format CodeIndex.sln --verify-no-changes
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll status --check --json
  • Codex adversarial review: No blocking/actionable issues found.

Fixes #3129

@Widthdom
Widthdom merged commit a5802af into main Jun 5, 2026
10 checks passed
@Widthdom
Widthdom deleted the fix-issue3129 branch June 5, 2026 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

LSP textDocument URIs should be length-capped before path resolution

1 participant