Skip to content

Bound and redact detailed database diffs - #4962

Merged
Widthdom merged 8 commits into
mainfrom
fix-issue4859
Jul 28, 2026
Merged

Bound and redact detailed database diffs#4962
Widthdom merged 8 commits into
mainfrom
fix-issue4859

Conversation

@Widthdom

Copy link
Copy Markdown
Owner

Summary

  • emit deterministic structured records with named fields instead of opaque rows
  • redact text and paths by default, with explicit --include-content
  • enforce whole-record UTF-8 budgets with stable count, cursor, replay, and staleness metadata
  • keep import dry-run comparisons redacted without unusable cursor metadata

Validation

  • dotnet restore CodeIndex.sln
  • dotnet build CodeIndex.sln -c Release --no-restore (0 warnings, 0 errors)
  • focused Release tests: 114 passed on net8.0 and 114 passed on net9.0
  • full Release suite: net8.0 10,714 passed / 7 skipped with one unrelated timing failure; net9.0 10,240 passed / 419 skipped with two unrelated timing/synchronization failures; all three failures passed on isolated retry
  • dotnet format CodeIndex.sln --verify-no-changes --no-restore
  • dotnet run --project tools/CodeIndex.Changelog -- check (33 fragments)
  • Codex adversarial review: No blocking/actionable issues found.

Documentation and changelog

  • updated the English and Japanese sections in USER_GUIDE.md and TESTING_GUIDE.md
  • added changelog.d/unreleased/4859.security.md

Related issues

Fixes #4859

@Widthdom
Widthdom marked this pull request as ready for review July 28, 2026 18:19
@Widthdom
Widthdom merged commit d156f53 into main Jul 28, 2026
12 checks passed
@Widthdom
Widthdom deleted the fix-issue4859 branch July 28, 2026 18:20
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.

Bound and redact detailed database diffs

1 participant