feat: Implement multi-instance synchronization with Sepia file change hooks #23
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
…ia file change hooks This major feature enables real-time synchronization between multiple ToCry instances sharing the same data directory using Sepia's new file system watching capabilities. Key improvements: - Real-time file system monitoring using Sepia's watcher hooks with -Dinotify - Thread-safe cache coordination across multiple instances - Optimistic locking with version tracking for conflict prevention - WebSocket-based event broadcasting for real-time updates - Comprehensive CLI configuration with --multi-instance and --no-file-watching flags - Updated static builds to support inotify file system watching Technical changes: - Added FileChangeHandler for Sepia event processing with debouncing - Enhanced BoardManager with thread-safe cache invalidation using Mutex - Implemented ConflictResolver with version tracking and merge strategies - Updated build system (Makefile, build_static.sh) to use -Dinotify flag - Added multi-instance configuration logic to main.cr and initialization.cr - Created comprehensive integration tests for multi-instance scenarios All tests passing, including unit tests, integration tests, and E2E tests. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
…on system Replaced redundant custom version tracking with Sepia's built-in object generations, simplifying the multi-instance coordination while maintaining the same functionality. Key changes: - Replaced ConflictResolver with streamlined MultiInstanceCoordinator - Removed custom .version files in favor of Sepia's native generation tracking - Simplified conflict detection to rely on cache invalidation and file watching - Integrated debouncing infrastructure directly into MultiInstanceCoordinator - Removed redundant FileChangeHandler class - Updated initialization to use simplified MultiInstanceCoordinator The refactored approach relies on: - Sepia's native object generations for automatic versioning - File system watching with cache invalidation for real-time sync - Event debouncing to prevent echo loops - WebSocket broadcasting for client updates All tests pass (JavaScript, Crystal, and E2E), confirming the simplified approach maintains full multi-instance synchronization capabilities. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
- Remove --multi-instance and --no-file-watching CLI flags - Enable multi-instance coordination by default in all modes except demo - Simplify configuration logic by removing unnecessary flags - Update test scripts to work with default multi-instance behavior - File watching now only disabled in demo mode 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Updated Sepia to version 0.2.0 which includes proper echo loop prevention in the file system watcher. This eliminates potential infinite loops when Sepia performs file operations that would trigger its own watcher events. Benefits: - Cleaner multi-instance coordination - No internal event spam - More robust synchronization between instances - Eliminates need for debouncing workarounds 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
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 comprehensive multi-instance synchronization for ToCry, allowing multiple instances to share the same data directory and synchronize changes in real-time using Sepia's file system change hooks.
Features
Technical Implementation
-Dinotifyflag for proper file system monitoringKey Changes
scripts/build_static.sh: Updated to use-Dinotifyfor both AMD64 and ARM64 buildsMakefile: Updated build targets with proper inotify supportsrc/conflict_resolution.cr: New MultiInstanceCoordinator for coordinating file changessrc/board_manager.cr: Enhanced with thread-safe cache invalidationsrc/main.cr: Simplified configuration - multi-instance enabled by defaultsrc/initialization.cr: Integrated file system watcher initializationshard.lock: Updated to Sepia 0.2.0 with echo loop preventionTesting
test_multi_instance.shvalidates basic multi-instance functionalityTest Plan
Breaking Changes
None. Multi-instance mode is enabled by default with zero configuration required.
Migration Guide
No migration needed. Existing installations will automatically gain multi-instance capabilities when upgraded.
🤖 Generated with Claude Code