Skip to content

fix(offline): keep snapshot reads in atomic owner - #95

Merged
rdlabo merged 1 commit into
mainfrom
fix/offline-atomic-snapshot-reads
Aug 30, 2026
Merged

fix(offline): keep snapshot reads in atomic owner#95
rdlabo merged 1 commit into
mainfrom
fix/offline-atomic-snapshot-reads

Conversation

@rdlabo

@rdlabo rdlabo commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

Summary

  • route command, replica-row, and state reads through the repository owned by each atomic mutation
  • cover discard, retry, and discard-all recovery paths with regression tests
  • keep replacement and transport-state transitions inside the same atomic repository boundary

Context

Fixes the production error Use the repository passed to an atomic mutation for snapshot reads. observed in winecode on /main/list. The SQLite safety guard remains intact; callers now use the correct atomic owner instead of re-entering the root repository.

Verification

  • npx ng test kit --watch=false (985 tests)
  • npx ng test kit --watch=false --include ../offline/src/lib/offline-sync.service.spec.ts (175 tests)
  • npm run lint
  • npm run prebuild:kit

@rdlabo
rdlabo merged commit c320513 into main Aug 30, 2026
14 checks passed
@github-actions

Copy link
Copy Markdown

npm beta packages published

CI passed for the merge commit c320513779dc. Install the immutable package set with:

npm install @rdlabo/ionic-angular-kit@22.0.0-beta.pr95.shac320513779dc @rdlabo/ionic-angular-photo-editor@22.0.0-beta.pr95.shac320513779dc @rdlabo/ionic-angular-scroll-header@22.0.0-beta.pr95.shac320513779dc @rdlabo/ngx-cdk-scroll-strategies@22.0.0-beta.pr95.shac320513779dc

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