Skip to content

[codex] Fix SuggestionStore temp cleanup - #2255

Merged
Widthdom merged 3 commits into
mainfrom
fix-issue1574
May 16, 2026
Merged

[codex] Fix SuggestionStore temp cleanup#2255
Widthdom merged 3 commits into
mainfrom
fix-issue1574

Conversation

@Widthdom

Copy link
Copy Markdown
Owner

Summary

Validation

  • Adversarial review: No blocking/actionable issues found.
  • dotnet build CodeIndex.sln -c Release
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj -c Release --filter FullyQualifiedName~SuggestionStoreTests

Documentation and changelog

  • Added changelog.d/unreleased/1574.fixed.md.
  • No README or guide updates were needed because this is internal cleanup behavior for an existing save path.

Fixes #1574

@Widthdom
Widthdom marked this pull request as ready for review May 16, 2026 18:19
@Widthdom
Widthdom merged commit c69aaa6 into main May 16, 2026
7 checks passed
@Widthdom
Widthdom deleted the fix-issue1574 branch May 16, 2026 18:19
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.

SuggestionStore atomic save can leak .tmp file on Move failure

1 participant