Skip to content

Fix durable suggestion writes - #2624

Merged
Widthdom merged 1 commit into
mainfrom
fix-issue1410
May 24, 2026
Merged

Fix durable suggestion writes#2624
Widthdom merged 1 commit into
mainfrom
fix-issue1410

Conversation

@Widthdom

@Widthdom Widthdom commented May 24, 2026

Copy link
Copy Markdown
Owner

Summary

Validation

  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter SuggestionStoreTests
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj -c Release --filter SuggestionStoreTests -p:UseSharedCompilation=false
  • dotnet run --project tools/CodeIndex.Changelog -- check
  • codex adversarial review: No blocking/actionable issues found.

Note: dotnet test CodeIndex.sln -c Release -p:UseSharedCompilation=false was attempted before rebasing onto the updated origin/main, but it produced no completion output for over 8 minutes and was stopped; focused Release tests passed afterward.

Documentation and changelog

  • Added changelog fragment: changelog.d/unreleased/1410.fixed.md
  • No documentation updates were needed; this preserves the existing suggestion-store interface and hardens persistence/recovery behavior.

Follow-up candidates

  • None.

Fixes #1410

@Widthdom
Widthdom merged commit 45a590f into main May 24, 2026
10 checks passed
@Widthdom
Widthdom deleted the fix-issue1410 branch May 24, 2026 11:45
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.SaveUnlocked has no fsync; power loss can silently wipe local suggestions

1 participant