Skip to content

Fix offline atomic startup cleanup - #73

Merged
rdlabo merged 1 commit into
mainfrom
agent/fix-offline-atomic-startup-cleanup
Aug 13, 2026
Merged

Fix offline atomic startup cleanup#73
rdlabo merged 1 commit into
mainfrom
agent/fix-offline-atomic-startup-cleanup

Conversation

@rdlabo

@rdlabo rdlabo commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

What changed

  • include atomic mutation startup in the cleanup boundary
  • always release the reader lease and write gate when the initial SQLite revision read fails
  • add a regression test proving later writes remain usable

Why

A failure while reading the initial PRAGMA data_version happened before the existing try/finally. That stranded the repository reader lease and could block all later writes for the process lifetime.

Validation

  • npx ng test kit --watch=false (44 files / 823 tests)
  • targeted ESLint
  • npm run prebuild:kit
  • git diff --check

@rdlabo
rdlabo marked this pull request as ready for review August 13, 2026 15:33
@netlify

netlify Bot commented Aug 13, 2026

Copy link
Copy Markdown

Deploy Preview for rdlabo-ionic-angular-library ready!

Name Link
🔨 Latest commit 8f57127
🔍 Latest deploy log https://app.netlify.com/projects/rdlabo-ionic-angular-library/deploys/6a7de3cfd6b1820008f45904
😎 Deploy Preview https://deploy-preview-73--rdlabo-ionic-angular-library.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@rdlabo
rdlabo merged commit c86f66a into main Aug 13, 2026
12 checks passed

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

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