claude-mpm 3.9.11__py3-none-any.whl → 4.0.3__py3-none-any.whl

claude_mpm/core/session_manager.py

@@ -1,12 +1,12 @@
 """Session ID management for Claude subprocess optimization."""
 
-import uuid
-from typing import Optional, Dict, Any, List
-from datetime import datetime, timedelta
+import gzip
 import json
 import shutil
-import gzip
+import uuid
+from datetime import datetime, timedelta
 from pathlib import Path
+from typing import Any, Dict, List, Optional
 
 from ..core.logger import get_logger
 
@@ -15,10 +15,10 @@ logger = get_logger(__name__)
 
 class SessionManager:
     """Manages session IDs for Claude subprocess reuse."""
-    
+
     def __init__(self, session_dir: Optional[Path] = None):
         """Initialize session manager.
-        
+
         Args:
             session_dir: Directory to store session metadata
         """
@@ -26,46 +26,48 @@ class SessionManager:
         self.session_dir.mkdir(parents=True, exist_ok=True)
         self.active_sessions: Dict[str, Dict[str, Any]] = {}
         self._load_sessions()
-    
+
     def create_session(self, context: str = "default") -> str:
         """Create a new session ID.
-        
+
         Args:
             context: Context identifier (e.g., "pm_orchestration", "agent_delegation")
-            
+
         Returns:
             UUID session ID
         """
         session_id = str(uuid.uuid4())
-        
+
         self.active_sessions[session_id] = {
             "id": session_id,
             "context": context,
             "created_at": datetime.now().isoformat(),
             "last_used": datetime.now().isoformat(),
             "use_count": 0,
-            "agents_run": []
+            "agents_run": [],
         }
-        
+
         self._save_sessions()
         logger.info(f"Created session {session_id} for context: {context}")
-        
+
         return session_id
-    
-    def get_or_create_session(self, context: str = "default", max_age_minutes: int = 30) -> str:
+
+    def get_or_create_session(
+        self, context: str = "default", max_age_minutes: int = 30
+    ) -> str:
         """Get existing session or create new one.
-        
+
         Args:
             context: Context identifier
             max_age_minutes: Maximum age of session to reuse
-            
+
         Returns:
             Session ID
         """
         # Look for existing session in context
         now = datetime.now()
         max_age = timedelta(minutes=max_age_minutes)
-        
+
         for session_id, session_data in self.active_sessions.items():
             if session_data["context"] == context:
                 last_used = datetime.fromisoformat(session_data["last_used"])
@@ -76,95 +78,101 @@ class SessionManager:
                     self._save_sessions()
                     logger.info(f"Reusing session {session_id} for context: {context}")
                     return session_id
-        
+
         # No valid session found, create new one
         return self.create_session(context)
-    
+
     def record_agent_use(self, session_id: str, agent: str, task: str):
         """Record that an agent used this session.
-        
+
         Args:
             session_id: Session ID
             agent: Agent name
             task: Task description
         """
         if session_id in self.active_sessions:
-            self.active_sessions[session_id]["agents_run"].append({
-                "agent": agent,
-                "task": task[:100],  # Truncate long tasks
-                "timestamp": datetime.now().isoformat()
-            })
+            self.active_sessions[session_id]["agents_run"].append(
+                {
+                    "agent": agent,
+                    "task": task[:100],  # Truncate long tasks
+                    "timestamp": datetime.now().isoformat(),
+                }
+            )
             self.active_sessions[session_id]["last_used"] = datetime.now().isoformat()
             self._save_sessions()
-    
+
     def cleanup_old_sessions(self, max_age_hours: int = 24, archive: bool = True):
         """Remove sessions older than max_age_hours.
-        
+
         WHY: We archive old sessions instead of just deleting them to preserve
         conversation history while reducing active memory usage.
-        
+
         Args:
             max_age_hours: Maximum age in hours
             archive: Whether to archive sessions before removing
         """
         now = datetime.now()
         max_age = timedelta(hours=max_age_hours)
-        
+
         expired = []
         for session_id, session_data in self.active_sessions.items():
             created = datetime.fromisoformat(session_data["created_at"])
             if now - created > max_age:
                 expired.append(session_id)
-        
+
         # Archive sessions if requested
         if archive and expired:
             self._archive_sessions([self.active_sessions[sid] for sid in expired])
-        
+
         for session_id in expired:
             del self.active_sessions[session_id]
             logger.info(f"Cleaned up expired session: {session_id}")
-        
+
         if expired:
             self._save_sessions()
-    
-    def get_recent_sessions(self, limit: int = 10, context: Optional[str] = None) -> list:
+
+    def get_recent_sessions(
+        self, limit: int = 10, context: Optional[str] = None
+    ) -> list:
         """Get recent sessions sorted by last used time.
-        
+
         Args:
             limit: Maximum number of sessions to return
             context: Filter by context (optional)
-            
+
         Returns:
             List of session data dictionaries sorted by last_used descending
         """
         sessions = list(self.active_sessions.values())
-        
+
         # Filter by context if specified
         if context:
             sessions = [s for s in sessions if s.get("context") == context]
-        
+
         # Sort by last_used descending (most recent first)
-        sessions.sort(key=lambda s: datetime.fromisoformat(s["last_used"]), reverse=True)
-        
+        sessions.sort(
+            key=lambda s: datetime.fromisoformat(s["last_used"]), reverse=True
+        )
+
         return sessions[:limit]
-    
+
     def get_session_by_id(self, session_id: str) -> Optional[Dict[str, Any]]:
         """Get session data by ID.
-        
+
         Args:
             session_id: Session ID to look up
-            
+
         Returns:
             Session data dictionary or None if not found
         """
         return self.active_sessions.get(session_id)
-    
+
     def get_last_interactive_session(self) -> Optional[str]:
         """Get the most recently used interactive session ID.
-        
+
         WHY: For --resume without arguments, we want to resume the last
         interactive session (context="default" for regular Claude runs).
-        
+
         Returns:
             Session ID of most recent interactive session, or None if none found
         """
@@ -172,120 +180,122 @@ class SessionManager:
         if recent_sessions:
             return recent_sessions[0]["id"]
         return None
-    
+
     def _save_sessions(self):
         """Save sessions to disk."""
         session_file = self.session_dir / "active_sessions.json"
         try:
-            with open(session_file, 'w') as f:
+            with open(session_file, "w") as f:
                 json.dump(self.active_sessions, f, indent=2)
         except Exception as e:
             logger.error(f"Failed to save sessions: {e}")
-    
+
     def _load_sessions(self):
         """Load sessions from disk."""
         session_file = self.session_dir / "active_sessions.json"
         if session_file.exists():
             try:
-                with open(session_file, 'r') as f:
+                with open(session_file, "r") as f:
                     self.active_sessions = json.load(f)
-                    
+
                 # Clean up old sessions on load (archive by default)
                 self.cleanup_old_sessions(archive=True)
-                
+
                 # Also check and clean .claude.json if needed
                 self._check_claude_json_size()
             except Exception as e:
                 logger.error(f"Failed to load sessions: {e}")
                 self.active_sessions = {}
-    
+
     def _archive_sessions(self, sessions: List[Dict[str, Any]]):
         """Archive sessions to compressed files.
-        
+
         WHY: Archiving preserves conversation history while reducing the size
         of active memory files like .claude.json.
-        
+
         Args:
             sessions: List of session data dictionaries to archive
         """
         if not sessions:
             return
-        
+
         archive_dir = self.session_dir.parent / "archives" / "sessions"
         archive_dir.mkdir(parents=True, exist_ok=True)
-        
+
         # Create timestamped archive file
         timestamp = datetime.now().strftime("%Y%m%d_%H%M%S")
         archive_name = f"sessions_archive_{timestamp}.json.gz"
         archive_path = archive_dir / archive_name
-        
+
         try:
             # Compress and save sessions
-            with gzip.open(archive_path, 'wt', encoding='utf-8') as f:
+            with gzip.open(archive_path, "wt", encoding="utf-8") as f:
                 json.dump(sessions, f, indent=2)
-            
+
             logger.info(f"Archived {len(sessions)} sessions to {archive_path}")
         except Exception as e:
             logger.error(f"Failed to archive sessions: {e}")
-    
+
     def _check_claude_json_size(self):
         """Check .claude.json size and suggest cleanup if needed.
-        
+
         WHY: Large .claude.json files cause memory issues. This provides
         proactive monitoring and suggestions for cleanup.
         """
         claude_json_path = Path.home() / ".claude.json"
-        
+
         if not claude_json_path.exists():
             return
-        
+
         file_size = claude_json_path.stat().st_size
         warning_threshold = 500 * 1024  # 500KB
-        
+
         if file_size > warning_threshold:
             size_mb = file_size / (1024 * 1024)
-            logger.warning(f".claude.json is {size_mb:.1f}MB - consider running 'claude-mpm cleanup-memory'")
-    
+            logger.warning(
+                f".claude.json is {size_mb:.1f}MB - consider running 'claude-mpm cleanup-memory'"
+            )
+
     def archive_claude_json(self, keep_days: int = 30) -> bool:
         """Archive old conversations from .claude.json.
-        
+
         WHY: This is called by the cleanup command to reduce memory usage
         while preserving conversation history.
-        
+
         Args:
             keep_days: Number of days of history to keep
-            
+
         Returns:
             True if successful, False otherwise
         """
         claude_json_path = Path.home() / ".claude.json"
-        
+
         if not claude_json_path.exists():
             logger.info("No .claude.json file to archive")
             return True
-        
+
         try:
             # Create backup first
             archive_dir = Path.home() / ".claude-mpm" / "archives"
             archive_dir.mkdir(parents=True, exist_ok=True)
-            
+
             timestamp = datetime.now().strftime("%Y%m%d_%H%M%S")
             backup_name = f"claude_json_backup_{timestamp}.json.gz"
             backup_path = archive_dir / backup_name
-            
+
             # Compress and backup current file
-            with open(claude_json_path, 'rb') as f_in:
-                with gzip.open(backup_path, 'wb') as f_out:
+            with open(claude_json_path, "rb") as f_in:
+                with gzip.open(backup_path, "wb") as f_out:
                     shutil.copyfileobj(f_in, f_out)
-            
+
             logger.info(f"Created backup at {backup_path}")
-            
+
             # For now, we don't modify the original .claude.json
             # as we don't know its exact structure.
             # The cleanup command handles this.
-            
+
             return True
-            
+
         except Exception as e:
             logger.error(f"Failed to archive .claude.json: {e}")
             return False
@@ -293,10 +303,10 @@ class SessionManager:
 
 class OrchestrationSession:
     """Context manager for orchestration sessions."""
-    
+
     def __init__(self, session_manager: SessionManager, context: str = "orchestration"):
         """Initialize orchestration session.
-        
+
         Args:
             session_manager: SessionManager instance
             context: Session context
@@ -304,12 +314,12 @@ class OrchestrationSession:
         self.session_manager = session_manager
         self.context = context
         self.session_id: Optional[str] = None
-    
+
     def __enter__(self) -> str:
         """Enter session context, return session ID."""
         self.session_id = self.session_manager.get_or_create_session(self.context)
         return self.session_id
-    
+
     def __exit__(self, exc_type, exc_val, exc_tb):
         """Exit session context."""
         # Could add cleanup here if needed
@@ -328,7 +338,7 @@ with OrchestrationSession(session_manager, "pm_delegation") as session_id:
         session_id=session_id,  # First call creates context
         timeout=30
     )
-    
+
     # Run agents with same session (reuses context)
     for agent, task in delegations:
         stdout, stderr, returncode = self.launcher.launch_oneshot(
@@ -337,4 +347,4 @@ with OrchestrationSession(session_manager, "pm_delegation") as session_id:
             timeout=60
         )
         session_manager.record_agent_use(session_id, agent, task)
-"""
+"""