Skip to content

Fix scan edge cases and canonical indexing - #2757

Merged
Widthdom merged 7 commits into
mainfrom
fix-issue1466-1467-1622
May 31, 2026
Merged

Fix scan edge cases and canonical indexing#2757
Widthdom merged 7 commits into
mainfrom
fix-issue1466-1467-1622

Conversation

@Widthdom

@Widthdom Widthdom commented May 31, 2026

Copy link
Copy Markdown
Owner

Summary

  • Fix directory traversal to skip already visited symlink targets and cap unexpectedly deep scans.
  • Compute checksums from canonical indexed content after line-ending and line-leading invisible normalization.
  • Skip real Git LFS pointer placeholders from body indexing and report lfs_pointer_skipped, while preserving normal text that only mentions the LFS version line.

Validation

  • dotnet run --project tools/CodeIndex.Changelog -- check
  • dotnet build CodeIndex.sln -c Release -p:UseSharedCompilation=false
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter "FullyQualifiedName~FileIndexerTests" -p:UseSharedCompilation=false
  • dotnet test CodeIndex.sln -c Release -p:UseSharedCompilation=false passed before the latest origin merge; after the latest origin merge, net8 passed and net9 failed 77 tests with System.ObjectDisposedException: Cannot write to a closed TextWriter in unrelated console/MCP test paths.
  • Adversarial review: No blocking/actionable issues found.

Related

Fixes #1466
Fixes #1467
Fixes #1622

@Widthdom
Widthdom merged commit 2d299c5 into main May 31, 2026
10 checks passed
@Widthdom
Widthdom deleted the fix-issue1466-1467-1622 branch May 31, 2026 15:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant