feat(sessions): Implement Event Sourcing Session Summary v2 - #6
Open
fulvian wants to merge 6 commits into
Open
Conversation
…ration - Create basic CI workflow for repository validation - Add branch protection settings for main branch - Configure protections for single developer project 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
…hases 1-2/6)
Resolve multi-session collision issues by introducing session-specific marker files
and enhanced registry schema. This is part of holistic session context persistence
architecture redesign (Task ID: 375c778977df733b517653ed6b859ca6).
**Phase 1: Registry Schema Enhancement**
- Enhanced SessionInfo dataclass with 6 new fields:
* ended_at: Session end timestamp
* marker_file_path: Path to session-specific marker file
* compaction_events: List of compaction events
* summary_displayed: Summary display tracking
* model_type: AI model type (sonnet-4.5, glm-4.6, unknown)
* session_name: Optional user-friendly name
- Implemented validate_registry_schema() for schema validation
- Implemented migrate_registry_schema() for automatic migration
- Automatic migration on coordinator initialization
**Phase 2: PreCompact Hook Refactoring**
- Added session_coordinator integration
- Implemented write_marker_file_session_specific():
* Creates ~/.claude/state/devstream_session_{session_id}.txt
* Prevents collision in multi-session environments
- Implemented update_registry_compaction_event():
* Updates registry with compaction events
* Thread-safe using fcntl locking
* Tracks timestamp, trigger, marker file status, DB storage status
- Modified process_pre_compact():
* Now uses session-specific marker files instead of shared file
* Maintains backward compatibility
**Testing Status**:
- Phase 1: ✅ PASS - Registry migration validated
- Phase 2: ⏳ PENDING - Requires /compact command test
**Remaining Phases** (for next session):
- Phase 3: SessionEnd Hook Refactoring
- Phase 4: SessionStart Hook Major Refactoring
- Phase 5: Fallback Strategies
- Phase 6: Testing & Validation
**Context7 Patterns Applied**:
- psutil: Cross-platform PID validation (research complete)
- fcntl: Thread-safe file locking (already implemented)
- aiosqlite: Async database operations (already implemented)
**Files Modified**:
- .claude/hooks/devstream/utils/session_coordinator.py (+180 lines)
- .claude/hooks/devstream/sessions/pre_compact.py (+140 lines)
**Architecture**:
Session-specific marker files prevent race conditions in multi-session
environments (Sonnet 4.5 + GLM-4.6 concurrent). Each session writes to its
own marker file, eliminating overwrites and data loss.
Task ID: 375c778977df733b517653ed6b859ca6
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
…er files (Phase 3/6)
**Phase 3 Complete**: SessionEnd hook now writes session-specific marker files
**Changes**:
- ✅ Add write_marker_file_session_specific() method to SessionEnd hook
- ✅ Add update_registry_session_end() method for thread-safe registry updates
- ✅ Session-specific marker files: ~/.claude/state/devstream_session_{session_id}.txt
- ✅ Registry updates: status="ended", ended_at timestamp, compaction_events tracking
- ✅ Import time module for timestamp generation
- ✅ Fix GLM-4.6 router config to allow /compact to use current model (Sonnet 4.5)
**Pattern Consistency**: Mirrors PreCompact implementation for uniformity
**Testing**: Requires manual session exit to validate (Phase 3 testing)
**Next**: Phase 4 - SessionStart hook refactoring (multi-summary display + cleanup)
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
…ay + cleanup (Phase 4/6) **Phase 4 Complete**: SessionStart hook now supports multi-session summaries **New Methods**: - ✅ display_all_pending_summaries() - Iterates ALL session-specific marker files - Finds all ~/.claude/state/devstream_session_*.txt files - Displays summaries for sessions with summary_displayed=False - Updates registry (summary_displayed=True) after display - Thread-safe via SessionCoordinator locking - Supports multi-session scenarios (Sonnet 4.5 + GLM-4.6) - ✅ cleanup_old_sessions() - Zombie + expired session cleanup - Uses psutil for PID validation (Context7 pattern) - Removes zombie sessions (PID doesn't exist) - Removes expired sessions (ended >7 days ago, configurable) - Deletes associated marker files - Thread-safe registry updates - ✅ migrate_legacy_marker_file() - Backward compatibility - Migrates devstream_last_session.txt to session-specific format - Creates synthetic session ID for legacy summaries - Updates registry with legacy session info - One-time migration, automatically removes legacy file **Refactored Methods**: - ✅ display_previous_summary() - Now orchestrates 3-step workflow: 1. Migrate legacy marker file (if exists) 2. Cleanup old/zombie sessions (proactive maintenance) 3. Display ALL pending summaries (multi-summary support) **Context7 Patterns**: - psutil.pid_exists() for zombie detection - glob.glob() for marker file iteration - Thread-safe locking via SessionCoordinator - Graceful degradation on errors **Testing**: Requires session restart to validate multi-summary display **Next**: Phase 5 - Fallback Strategies + Phase 6 - Testing & Validation 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Replace over-engineered session summary system (6655 LOC, 14 files) with Event Sourcing pattern (450 LOC, 3 files) - 93% reduction. Key improvements: - Zero database queries during session (vs 7-9 queries) - Real-time event capture instead of post-hoc inference - Eliminate timezone bugs (epoch timestamps only) - 100% session coverage (vs 90% dual-write strategy) - Thread-safe in-memory append-only log Implementation: - session_event_log.py: In-memory event storage (Context7 pattern) - session_end_v2.py: Event aggregation with Array.reduce() - post_tool_use.py: Real-time event capture integration Tests: - 42/42 unit tests PASSING (100%) - 2/8 integration tests PASSING (core workflow verified) - Remaining failures: MCP timeout issues (non-critical) Technical debt resolved: - Fix: Module import path mismatch causing separate registries - Pattern: Shared absolute import path for registry singleton Co-authored-by: GLM-4.6 <glm@zhipuai.cn> 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Merge pull request #5 from fulvian/release/v0.1.0-beta
fulvian
added a commit
that referenced
this pull request
Oct 14, 2025
Comprehensive repository cleanup following Context7 best practices and production of DevStream v0.2.0 with Direct DB architecture. Implementation Details: - Cleaned .gitignore violations (.DS_Store, .log, .bak files) - Removed deprecated MCP server directories (mcp-devstream-server/, node_modules/) - Managed Pull Requests: Merged PR #10, evaluated PR #6 - Generated comprehensive CHANGELOG.md for v0.2.0 - Created release v0.2.0 with Direct DB architecture highlights - Optimized repository size and structure Quality Validation: - ✅ Tests: 17 tests passing, 100% coverage - ✅ Type safety: mypy --strict passed - ✅ Repository health: All issues resolved - ✅ Release: v0.2.0 published successfully Task ID: a708b697-6291-40cd-af06-d3fb9ec8569f 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
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.
Summary
Replace over-engineered session summary system (6655 LOC, 14 files) with Event Sourcing pattern (450 LOC, 3 files) - 93% reduction.
Key Improvements
Implementation
Tests
Technical Debt Resolved
Co-Authors
🤖 Generated with Claude Code