Skip to content

Bound persisted JSON string-list metadata parsing - #3309

Merged
Widthdom merged 6 commits into
mainfrom
fix-issue3045
Jun 5, 2026
Merged

Bound persisted JSON string-list metadata parsing#3309
Widthdom merged 6 commits into
mainfrom
fix-issue3045

Conversation

@Widthdom

@Widthdom Widthdom commented Jun 5, 2026

Copy link
Copy Markdown
Owner

Summary

  • Bound persisted JSON string-list metadata parsing with raw-size, depth, item-count, and decoded-character caps.
  • Keep unknown-extension writer samples within the reader budget so status metadata remains readable even with very long paths.
  • Updated status contract docs and added a bilingual changelog fragment.

Validation

  • Tests added or updated for changed behavior, or not needed because: JsonStringListCodecTests covers codec bounds and writer/reader integration.
  • Relevant validation command(s) run:
    • dotnet build
    • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj
    • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter "FullyQualifiedName~JsonStringListCodecTests|FullyQualifiedName~DocumentationStatusContractTests|FullyQualifiedName~McpAuditLogTests.ToolsCall_TruncatesAuditRequestId_Issue3237|FullyQualifiedName~UnknownExtension"
    • 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 remaining blocking/actionable issues.

Documentation and changelog

  • Documentation updated, or not needed because: README.md, DEVELOPER_GUIDE.md, and AGENT_GUIDE.md clarify the bounded unknown-extension sample contract.
  • Changelog fragment added under changelog.d/unreleased/, or not needed because: changelog.d/unreleased/3045.security.md.
  • Fragment follows .codex/workflows/changelog-fragment.md for category, issue front matter, and bilingual sections.

Issues

Fixes #3045

Follow-up issues

@Widthdom
Widthdom merged commit e7373fd into main Jun 5, 2026
11 checks passed
@Widthdom
Widthdom deleted the fix-issue3045 branch June 5, 2026 14:18
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.

JsonStringListCodec should cap JSON depth and list size

1 participant