Skip to content

Fix checksum hex case drift - #2647

Merged
Widthdom merged 3 commits into
mainfrom
codex/fix-issue1722
May 24, 2026
Merged

Fix checksum hex case drift#2647
Widthdom merged 3 commits into
mainfrom
codex/fix-issue1722

Conversation

@Widthdom

Copy link
Copy Markdown
Owner

Summary

  • Enforce ordinal, case-sensitive checksum matching when resolving explicit project-local DB samples.
  • Standardize the NameFold fingerprint emitter on lowercase hex.
  • Document the lowercase SHA-256 hash convention and add focused regression coverage.

Validation

  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter "FullyQualifiedName~DbPathResolverTests|FullyQualifiedName~FileIndexerTests|FullyQualifiedName~NameFoldTests" -p:UseSharedCompilation=false
  • dotnet build CodeIndex.sln -c Release -p:UseSharedCompilation=false
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj -c Release --filter "FullyQualifiedName~DbPathResolverTests|FullyQualifiedName~FileIndexerTests|FullyQualifiedName~NameFoldTests" -p:UseSharedCompilation=false --no-restore
  • dotnet run --project tools/CodeIndex.Changelog -- check
  • codex exec adversarial review: No blocking/actionable issues found.

Full dotnet test CodeIndex.sln -c Release -p:UseSharedCompilation=false was attempted but was stopped after it ran without completion for an extended period in this local environment.

Documentation / Changelog

  • Updated DEVELOPER_GUIDE.md.
  • Added changelog.d/unreleased/1722.fixed.md.

Follow-up candidates

  • None.

Fixes #1722

@Widthdom
Widthdom marked this pull request as ready for review May 24, 2026 16:27
@Widthdom
Widthdom merged commit d7e4ed7 into main May 24, 2026
10 checks passed
@Widthdom
Widthdom deleted the codex/fix-issue1722 branch May 24, 2026 18:56
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.

Checksum hex comparison uses OrdinalIgnoreCase, masking case-format drift between writers

1 participant