feat(sync): add live bidirectional directory synchronization #17
Conversation
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
Add foundational sync module with file indexing, protocol extensions, and content-based change detection using xxHash64. - Add sync module with core types and file index system - Add 10 new protocol message types (0x70-0x7F) for sync operations - Add notify and globset dependencies - Add 35 comprehensive tests
…ring Implements cross-platform file watching using notify crate with: - Event debouncing to coalesce rapid file changes - Glob-based exclusion patterns for filtering - File size limits to prevent syncing oversized files - Async integration with tokio - Platform-specific quirk handling for Linux/macOS/Windows
…ciliation Implements bidirectional sync sessions with TLS 1.3 connections, file index exchange, and reconciliation logic. Includes conflict detection with four resolution strategies (LastWriteWins, KeepBoth, PreferLocal, PreferRemote) and operation ordering for safe execution. Integrates with existing discovery and transfer infrastructure for chunked file operations.
Adds continuous sync capability with concurrent task architecture: - File watcher monitors local directory changes with debouncing - Outbound processor sends sync operations to peer - Inbound processor receives and applies remote operations - Keep-alive task maintains connection health Implements graceful shutdown handling and event callback system for UI updates. Supports JSON output mode and all sync options including exclusion patterns, size limits, and deletion sync.
- Add directory sync feature to README core features list - Add comprehensive directory sync section with usage examples - Document sync command options (exclude, ignore-file, max-size, etc.) - Update npm README with sync command documentation - Fix dead_code warnings in session.rs with allow annotations - Remove unused imports and variables in test files
- Add underscores to numeric literals for readability - Use map_or/map_or_else instead of if let/else patterns - Replace redundant clones with direct ownership - Use div_ceil() instead of manual division ceiling - Inline format arguments where appropriate - Add necessary allow attributes for intentional casts - Rename _watcher field to watcher (field is used) - Use struct initialization instead of field reassignment - Apply let-else pattern where appropriate
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
Implements live bidirectional directory synchronization with automatic conflict resolution, enabling real-time file synchronization between devices on the same network.
Features
yoop synccommand with comprehensive options and progress reportingImplementation Details
Architecture
tokiofor async operationsSecurity
Performance