Merged
Conversation
- Add BackupManifest, ManifestEntry, ScannedFile types - Add scanBackupFolder() to recursively read markdown + assets - Add readManifest() and writeManifest() helpers - Add transformContentForImport() for reverse asset reference conversion - Add tests for scanner, manifest, and content transform
- Add bidirectional flag to BackupState (default true for new installs) - Add setBidirectional setter and reset handling - Add lastPullAt tracking for UI display - Add BACKUP_PULL_INTERVAL_MS constant (20s)
- Add hashContent(), syncFromBackup(), createDocFromFile(), updateDocFromFile() - Add pull loop with visibility detection and 20s interval - Update BackupSubscriber to include bidirectional sync (bidirectional flag) - Add setLastPullAt tracking for UI display - Support creating/updating/deleting docs from filesystem changes
- Add pull loop to SpaceBackupSubscriber with visibility detection - Space backups always bidirectional (respects user permissions for writes) - Check admin/writer role before allowing sync-from-backup writes - Add periodic pull with 20s interval - Remove unused lastPullAt state variable
- Add hashContent tests (consistency, different content, unicode, empty) - Add bidirectional sync export tests - Export hashContent and syncFromBackup for testing - Fix self aliasing lint issues in test mocks - Add readManifest and writeManifest exports
- Show warning in UI when File System Access API is not supported (Safari/Firefox) - Use FileSystemDirectoryHandle.watch() for real-time file change detection when available (Chrome 110+) - Fall back to polling (20s interval) when watch is not supported - Add supportsFileSystemWatch() detection helper
- Remove 20s polling interval fallback - Use FileSystemDirectoryHandle.watch() when available - Fall back to pull on visibility change and manual sync only - Remove unused _BACKUP_PULL_INTERVAL_MS constant
- Only enable bidirectional sync when watch API is available - Disable the sync toggle in UI when watch is not supported - Show appropriate message explaining browser requirement - Remove visibility-based pull fallback (only real-time watch)
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR enhances the filesystem backup functionality to optionally do two-way sync. This enables collaborating with an AI agent like opencode or claude code on the documents.
Only available on chromium browsers.