Skip to content

Fix FileIndexer race and symlink handling - #2721

Merged
Widthdom merged 2 commits into
mainfrom
codex/fix-issue1654-1655-1656-1711-main
May 31, 2026
Merged

Fix FileIndexer race and symlink handling#2721
Widthdom merged 2 commits into
mainfrom
codex/fix-issue1654-1655-1656-1711-main

Conversation

@Widthdom

Copy link
Copy Markdown
Owner

Summary

  • Add mtime recheck/retry when indexing file content.
  • Treat mid-scan delete races as non-fatal skipped paths and keep purge authority.
  • Report dangling directory symlinks and expose dangling_symlinks_skipped in full-scan JSON.
  • Add --follow-symlinks none|internal|all with none as the default.

Validation

  • dotnet build CodeIndex.sln -c Release -p:UseSharedCompilation=false
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj -c Release -p:UseSharedCompilation=false --filter "FullyQualifiedName~FileIndexerTests"
  • dotnet run --project tools/CodeIndex.Changelog -- check
  • dotnet test CodeIndex.sln -c Release -p:UseSharedCompilation=false was attempted, but local git signing configuration caused git-commit-based tests to fail with an SSH signing key passphrase prompt.

Documentation and Changelog

  • Updated DEVELOPER_GUIDE.md.
  • Added changelog fragments:
    • changelog.d/unreleased/1654.fixed.md
    • changelog.d/unreleased/1655.fixed.md
    • changelog.d/unreleased/1656.fixed.md
    • changelog.d/unreleased/1711.added.md

Notes

Fixes #1654
Fixes #1655
Fixes #1656
Fixes #1711

@Widthdom
Widthdom force-pushed the codex/fix-issue1654-1655-1656-1711-main branch from b8e86a7 to 3016f90 Compare May 31, 2026 04:34
@Widthdom
Widthdom marked this pull request as ready for review May 31, 2026 05:30
@Widthdom
Widthdom merged commit a39f77d into main May 31, 2026
10 checks passed
@Widthdom
Widthdom deleted the codex/fix-issue1654-1655-1656-1711-main branch May 31, 2026 05:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment