Skip to content

Cap JSON parsing depth across protocol inputs - #3274

Merged
Widthdom merged 7 commits into
mainfrom
fix-issue3012-3014-3016-3021-3022-3036-3043
Jun 5, 2026
Merged

Cap JSON parsing depth across protocol inputs#3274
Widthdom merged 7 commits into
mainfrom
fix-issue3012-3014-3016-3021-3022-3036-3043

Conversation

@Widthdom

@Widthdom Widthdom commented Jun 4, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add bounded JSON parsing for remaining protocol/control-plane inputs: MCP response frames, CLI envelope raw items, LSP messages, batch query lines, active workspace state, and MCP index lock metadata.
  • Add regression coverage for the requested issue cases, including HTTP MCP request-id probing behavior.
  • Add unreleased changelog fragments for each fixed issue.

Validation

  • dotnet build CodeIndex.sln -p:UseSharedCompilation=false
  • dotnet test CodeIndex.sln -p:UseSharedCompilation=false
  • dotnet run --project tools/CodeIndex.Changelog -- check
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll status --check --json
  • Codex adversarial review: no blocking/actionable issues found.

Fixes #3012
Fixes #3014
Fixes #3016
Fixes #3021
Fixes #3022
Fixes #3036
Fixes #3043

@Widthdom
Widthdom marked this pull request as ready for review June 5, 2026 00:50
@Widthdom
Widthdom merged commit 2519d66 into main Jun 5, 2026
10 checks passed
@Widthdom
Widthdom deleted the fix-issue3012-3014-3016-3021-3022-3036-3043 branch June 5, 2026 00:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment