Skip to content

fix(session): make Save As sidecar cleanup transactional - #354

Merged
devmobasa merged 1 commit into
mainfrom
fix/save-as-sidecar-transaction
Aug 24, 2026
Merged

fix(session): make Save As sidecar cleanup transactional#354
devmobasa merged 1 commit into
mainfrom
fix/save-as-sidecar-transaction

Conversation

@devmobasa

Copy link
Copy Markdown
Owner

Summary

  • quarantine stale non-lock sidecars before replacing a Save As target
  • restore quarantined recovery data when primary replacement fails
  • clean quarantine contents only after the new primary is committed and synced
  • retain recovery bytes in a private quarantine if post-commit cleanup fails
  • cover canonical backup, recovery, marker, clear, and preserved-version artifacts while retaining the active lock

Validation

  • transactional Save As failure-path tests
  • runtime Save As behavior tests
  • both focused suites with --no-default-features
  • cargo fmt --all -- --check
  • cargo clippy --workspace --all-targets --all-features -- -D warnings
  • git diff --check

@devmobasa
devmobasa merged commit d70c520 into main Aug 24, 2026
2 checks passed
@devmobasa
devmobasa deleted the fix/save-as-sidecar-transaction branch August 24, 2026 09:43
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.

1 participant