feat: add mobile sync client with outbox pattern (#40) #47

Merged

maximus merged 1 commit from issue-40-sync-mobile into master

2026-04-06 16:59:01 +00:00

maximus commented

2026-04-06 16:55:15 +00:00

Owner

Fixes #40

Summary

sync_outbox SQLite table (migration 0004) for offline-first mutations
Sync service (syncClient.ts): push outbox, pull server changes, apply locally
Outbox helper: writes to sync_outbox after each create/update/delete in repositories
Settings store extended: syncEnabled, lastSyncAt, userId
Sync polling: on launch + every 2 min + on return from background
Settings UI: Compte section with placeholder login, sync status, sync button
i18n keys for sync-related strings (FR + EN)

Notes

Logto OAuth integration (@logto/rn) is NOT included — just the UI scaffolding
Auth token handling is placeholder (reads userId from settings store)
The actual Logto flow + expo-secure-store adapter requires on-device testing

Depends on

PR #41 (shared types, not yet merged)

Fixes #40 ## Summary - `sync_outbox` SQLite table (migration 0004) for offline-first mutations - Sync service (`syncClient.ts`): push outbox, pull server changes, apply locally - Outbox helper: writes to sync_outbox after each create/update/delete in repositories - Settings store extended: `syncEnabled`, `lastSyncAt`, `userId` - Sync polling: on launch + every 2 min + on return from background - Settings UI: Compte section with placeholder login, sync status, sync button - i18n keys for sync-related strings (FR + EN) ## Notes - Logto OAuth integration (`@logto/rn`) is NOT included — just the UI scaffolding - Auth token handling is placeholder (reads userId from settings store) - The actual Logto flow + expo-secure-store adapter requires on-device testing ## Depends on - PR #41 (shared types, not yet merged)

maximus added 1 commit 2026-04-06 16:55:15 +00:00

feat: add mobile sync client with outbox pattern (#40 ) c496d9586c

- sync_outbox table in SQLite (migration 0004)
- Sync service: push/pull changes, fullSync, outbox cleanup
- Outbox writing in task/list/tag repositories after mutations
- Settings store: syncEnabled, lastSyncAt, userId
- Sync polling: on launch, every 2 min, on return from background
- Settings UI: Compte section with connect/sync/disconnect buttons
- i18n keys for sync strings (FR + EN)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

maximus commented

2026-04-06 16:55:48 +00:00

Author

Owner

Review — APPROVE

Summary: Solid sync scaffolding with outbox pattern, polling, and settings UI. Auth is correctly marked as placeholder.

sync_outbox table and migration correct
Outbox helper checks syncEnabled before writing
Push in batches of 50 with idempotency keys
Sync polling: launch + 2 min interval + background return
Settings UI with Compte section in both languages
TypeScript compiles
No secrets

Notes for integration:

Auth token is placeholder (userId, not JWT) — needs @logto/rn integration later
Sync pull response interface may need adjustment to match actual API response format (lists/tasks/tags arrays vs changes array)
These are expected for scaffolding and can be fixed during integration testing

## Review — APPROVE **Summary**: Solid sync scaffolding with outbox pattern, polling, and settings UI. Auth is correctly marked as placeholder. - [x] sync_outbox table and migration correct - [x] Outbox helper checks syncEnabled before writing - [x] Push in batches of 50 with idempotency keys - [x] Sync polling: launch + 2 min interval + background return - [x] Settings UI with Compte section in both languages - [x] TypeScript compiles - [x] No secrets **Notes for integration**: - Auth token is placeholder (userId, not JWT) — needs @logto/rn integration later - Sync pull response interface may need adjustment to match actual API response format (lists/tasks/tags arrays vs changes array) - These are expected for scaffolding and can be fixed during integration testing