Skip to content

Make export and report outputs atomic - #2948

Merged
Widthdom merged 7 commits into
mainfrom
fix-issue2839-2842-2886-2850
Jun 2, 2026
Merged

Make export and report outputs atomic#2948
Widthdom merged 7 commits into
mainfrom
fix-issue2839-2842-2886-2850

Conversation

@Widthdom

@Widthdom Widthdom commented Jun 2, 2026

Copy link
Copy Markdown
Owner

Summary

  • Stage cdidx export archives, cdidx export ctags tagfiles, and cdidx report bundles in same-directory temporary files before replacing requested outputs.
  • Replace imported databases before deleting destination SQLite WAL/SHM sidecars.
  • Add regression tests for preserving existing outputs and sidecars when replacement fails.

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 -c Release --filter "FullyQualifiedName~ExportImportCommandRunnerTests|FullyQualifiedName~ReportCommandRunnerTests" -p:UseSharedCompilation=false
  • dotnet format CodeIndex.sln --verify-no-changes
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll status --check --json

Documentation And Changelog

  • Added bilingual changelog fragments:
    • changelog.d/unreleased/2839.fixed.md
    • changelog.d/unreleased/2842.fixed.md
    • changelog.d/unreleased/2850.fixed.md
    • changelog.d/unreleased/2886.fixed.md

Follow-Up References

Fixes #2839
Fixes #2842
Fixes #2886
Fixes #2850

@Widthdom
Widthdom merged commit 3e007ab into main Jun 2, 2026
10 checks passed
@Widthdom
Widthdom deleted the fix-issue2839-2842-2886-2850 branch June 2, 2026 17:42
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