fix: expand e2e coverage and docs for session sync compatibility#53
fix: expand e2e coverage and docs for session sync compatibility#53
Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces support for syncing the SQLite session database (opencode.db) and its associated sidecar files (-wal, -shm) while maintaining compatibility with legacy JSON session storage. Key changes include the implementation of a preserveWhenMissing flag in the sync logic to prevent accidental deletion of remote session data when local files are absent, and a new copySessionDbBundle utility to handle SQLite-specific file groups. The E2E testing framework has been significantly enhanced with new flags for secrets and sessions, retry logic for the sync-link command, and direct SQLite assertions to verify data integrity across synced instances. A bug was identified in the parse_jsonc helper function where a global regular expression could corrupt string values containing trailing comma patterns.
🤖 Review Jules RelayI found 1 Gemini suggestion so far. Type |
* feat: add Turso session backend with setup and migration commands * fix: make sync-link repo selection deterministic for e2e (#55) * fix: make sync-link repo selection deterministic for e2e * fix: harden turso runtime session sync safety * fix: harden repo reference parsing for PR feedback * refactor: reduce turso sync io overhead * fix: harden turso session sync transport and timeouts
Summary
opencode.dbopencode.dbTesting
bun run checkbun testbun run build