claude-mpm 4.3.11__py3-none-any.whl → 4.3.13__py3-none-any.whl

claude_mpm/services/agents/registry/modification_tracker.py

@@ -30,7 +30,7 @@ import shutil
 import time
 import uuid
 from dataclasses import asdict, dataclass, field
-from datetime import datetime
+from datetime import datetime, timezone
 from enum import Enum
 from typing import Any, Callable, Dict, List, Optional, Set, Tuple
 
@@ -90,7 +90,7 @@ class AgentModification:
     @property
     def modification_datetime(self) -> datetime:
         """Get modification timestamp as datetime."""
-        return datetime.fromtimestamp(self.timestamp)
+        return datetime.fromtimestamp(self.timestamp, tz=timezone.utc)
 
     @property
     def age_seconds(self) -> float:

claude_mpm/services/async_session_logger.py

@@ -22,7 +22,7 @@ import os
 import sys
 import time
 from dataclasses import asdict, dataclass
-from datetime import datetime
+from datetime import datetime, timezone
 from enum import Enum
 from queue import Full, Queue
 from threading import Lock, Thread
@@ -184,7 +184,7 @@ class AsyncSessionLogger:
                 return session_id
 
         # Generate timestamp-based session ID
-        session_id = datetime.now().strftime("%Y%m%d_%H%M%S")
+        session_id = datetime.now(timezone.utc).strftime("%Y%m%d_%H%M%S")
         logger.info(f"Generated session ID: {session_id}")
         return session_id
 
@@ -406,7 +406,7 @@ class AsyncSessionLogger:
             agent_name = metadata["agent"].replace(" ", "_").lower()
 
         # Create timestamp with microsecond precision
-        now = datetime.now()
+        now = datetime.now(timezone.utc)
         timestamp = now.isoformat()
         microseconds = now.microsecond
 

claude_mpm/services/claude_session_logger.py

@@ -13,7 +13,7 @@ Configuration via .claude-mpm/configuration.yaml.
 import json
 import logging
 import os
-from datetime import datetime
+from datetime import datetime, timezone
 from typing import Any, Dict, Optional
 
 # Import configuration manager
@@ -133,7 +133,7 @@ class ClaudeSessionLogger:
         # Generate a default based on timestamp if nothing found
         if not session_id:
             # Use a timestamp-based session ID as fallback
-            session_id = datetime.now().strftime("%Y%m%d_%H%M%S")
+            session_id = datetime.now(timezone.utc).strftime("%Y%m%d_%H%M%S")
             logger.info(f"No Claude session ID found, using generated: {session_id}")
         else:
             logger.info(f"Using Claude session ID: {session_id}")
@@ -156,7 +156,7 @@ class ClaudeSessionLogger:
         agent_name = agent_name.replace(" ", "_").lower()
 
         # Generate timestamp with microseconds for uniqueness
-        timestamp = datetime.now().strftime("%Y%m%d_%H%M%S_%f")
+        timestamp = datetime.now(timezone.utc).strftime("%Y%m%d_%H%M%S_%f")
 
         # Create filename: session_id-agent-timestamp.json
         return f"{self.session_id}-{agent_name}-{timestamp}.json"
@@ -216,7 +216,7 @@ class ClaudeSessionLogger:
 
         # Prepare response data with standardized field names
         response_data = {
-            "timestamp": datetime.now().isoformat(),
+            "timestamp": datetime.now(timezone.utc).isoformat(),
             "session_id": self.session_id,
             "request": request_summary,  # Standardized field name
             "response": response_content,  # Already correct

claude_mpm/services/cli/agent_cleanup_service.py

@@ -138,6 +138,11 @@ class AgentCleanupService(IAgentCleanupService):
                 if not isinstance(result, dict):
                     result = {"success": bool(result)}
 
+                # Add success flag based on whether there were errors
+                if "success" not in result:
+                    # Consider it successful if no errors occurred
+                    result["success"] = not bool(result.get("errors"))
+
                 # Add cleaned_count for backward compatibility
                 if "cleaned_count" not in result:
                     removed_count = len(result.get("removed", []))

claude_mpm/services/cli/agent_listing_service.py

@@ -336,7 +336,7 @@ class AgentListingService(IAgentListingService):
             self.logger.error(f"Error listing agents by tier: {e}", exc_info=True)
             return AgentTierInfo(project=[], user=[], system=[])
 
-    def get_agent_details(self, agent_name: str) -> Optional[Dict[str, Any]]:
+    def get_agent_details(self, agent_name: str) -> Optional[Dict[str, Any]]:  # noqa: PLR0911
         """Get detailed information for a specific agent."""
         cache_key = f"agent_details_{agent_name}"
         cached = self._get_from_cache(cache_key)

claude_mpm/services/cli/agent_validation_service.py

@@ -1,3 +1,4 @@
+import re
 """
 Agent Validation Service
 ========================

claude_mpm/services/cli/memory_crud_service.py

@@ -17,7 +17,7 @@ DESIGN DECISIONS:
 
 import os
 from abc import ABC, abstractmethod
-from datetime import datetime
+from datetime import datetime, timezone
 from pathlib import Path
 from typing import Any, Dict, List, Optional
 
@@ -223,7 +223,9 @@ class MemoryCRUDService(IMemoryCRUDService):
                     stat = memory_file.stat()
                     file_stats = {
                         "size_kb": stat.st_size / 1024,
-                        "modified": datetime.fromtimestamp(stat.st_mtime).isoformat(),
+                        "modified": datetime.fromtimestamp(
+                            stat.st_mtime, tz=timezone.utc
+                        ).isoformat(),
                         "path": str(memory_file),
                     }
 
@@ -256,7 +258,7 @@ class MemoryCRUDService(IMemoryCRUDService):
                             "file_stats": {
                                 "size_kb": stat.st_size / 1024,
                                 "modified": datetime.fromtimestamp(
-                                    stat.st_mtime
+                                    stat.st_mtime, tz=timezone.utc
                                 ).isoformat(),
                                 "path": str(memory_file),
                             },
@@ -431,10 +433,10 @@ class MemoryCRUDService(IMemoryCRUDService):
                         {
                             "size_kb": stat.st_size / 1024,
                             "modified": datetime.fromtimestamp(
-                                stat.st_mtime
+                                stat.st_mtime, tz=timezone.utc
                             ).isoformat(),
                             "created": datetime.fromtimestamp(
-                                stat.st_ctime
+                                stat.st_ctime, tz=timezone.utc
                             ).isoformat(),
                         }
                     )
@@ -499,7 +501,10 @@ class MemoryCRUDService(IMemoryCRUDService):
                     continue
 
                 stat = memory_file.stat()
-                age_days = (datetime.now() - datetime.fromtimestamp(stat.st_mtime)).days
+                age_days = (
+                    datetime.now(timezone.utc)
+                    - datetime.fromtimestamp(stat.st_mtime, tz=timezone.utc)
+                ).days
 
                 # Identify files older than 30 days as candidates
                 if age_days > 30:

claude_mpm/services/cli/memory_output_formatter.py

@@ -200,7 +200,7 @@ class MemoryOutputFormatter(IMemoryOutputFormatter):
                 try:
                     dt = datetime.fromisoformat(last_modified.replace("Z", "+00:00"))
                     last_modified_str = dt.strftime("%Y-%m-%d %H:%M:%S")
-                except:
+                except Exception:
                     last_modified_str = last_modified
 
                 # Status indicator based on usage

claude_mpm/services/cli/session_manager.py

@@ -18,7 +18,7 @@ import json
 import uuid
 from abc import ABC, abstractmethod
 from dataclasses import dataclass, field
-from datetime import datetime, timedelta
+from datetime import datetime, timedelta, timezone
 from pathlib import Path
 from typing import Any, Dict, List, Optional
 
@@ -151,8 +151,12 @@ class SessionInfo:
 
     id: str
     context: str = "default"
-    created_at: str = field(default_factory=lambda: datetime.now().isoformat())
-    last_used: str = field(default_factory=lambda: datetime.now().isoformat())
+    created_at: str = field(
+        default_factory=lambda: datetime.now(timezone.utc).isoformat()
+    )
+    last_used: str = field(
+        default_factory=lambda: datetime.now(timezone.utc).isoformat()
+    )
     use_count: int = 0
     agents_run: List[Dict[str, Any]] = field(default_factory=list)
     metadata: Dict[str, Any] = field(default_factory=dict)
@@ -175,8 +179,8 @@ class SessionInfo:
         return cls(
             id=data["id"],
             context=data.get("context", "default"),
-            created_at=data.get("created_at", datetime.now().isoformat()),
-            last_used=data.get("last_used", datetime.now().isoformat()),
+            created_at=data.get("created_at", datetime.now(timezone.utc).isoformat()),
+            last_used=data.get("last_used", datetime.now(timezone.utc).isoformat()),
             use_count=data.get("use_count", 0),
             agents_run=data.get("agents_run", []),
             metadata=data.get("metadata", {}),
@@ -284,7 +288,7 @@ class SessionManager(ISessionManager):
         # Check session age
         try:
             created = datetime.fromisoformat(session.created_at)
-            age = datetime.now() - created
+            age = datetime.now(timezone.utc) - created
 
             if age > timedelta(days=7):
                 validation.warnings.append(f"Session is {age.days} days old")
@@ -355,10 +359,10 @@ class SessionManager(ISessionManager):
             {
                 "agent": agent,
                 "task": task[:100],  # Truncate long tasks
-                "timestamp": datetime.now().isoformat(),
+                "timestamp": datetime.now(timezone.utc).isoformat(),
             }
         )
-        session.last_used = datetime.now().isoformat()
+        session.last_used = datetime.now(timezone.utc).isoformat()
         session.use_count += 1
 
         self.save_session(session)
@@ -370,7 +374,7 @@ class SessionManager(ISessionManager):
 
         WHY: Prevents unbounded growth of session data and improves performance.
         """
-        now = datetime.now()
+        now = datetime.now(timezone.utc)
         max_age = timedelta(hours=max_age_hours)
 
         expired_ids = []
@@ -418,7 +422,7 @@ class SessionManager(ISessionManager):
             return True
 
         # Create timestamped archive file
-        timestamp = datetime.now().strftime("%Y%m%d_%H%M%S")
+        timestamp = datetime.now(timezone.utc).strftime("%Y%m%d_%H%M%S")
         archive_name = f"sessions_archive_{timestamp}.json.gz"
         archive_path = archive_dir / archive_name
 
@@ -509,5 +513,5 @@ class ManagedSession:
         """Exit session context with cleanup."""
         if self.session:
             # Update last used time
-            self.session.last_used = datetime.now().isoformat()
+            self.session.last_used = datetime.now(timezone.utc).isoformat()
             self.manager.save_session(self.session)

claude_mpm/services/cli/unified_dashboard_manager.py

@@ -81,7 +81,7 @@ class UnifiedDashboardManager(IUnifiedDashboardManager):
         background: bool = False,
         open_browser: bool = True,
         force_restart: bool = False,
-    ) -> Tuple[bool, bool]:
+    ) -> Tuple[bool, bool]:  # noqa: PLR0911
         """
         Start the dashboard using unified daemon.
 

claude_mpm/services/core/memory_manager.py

@@ -21,7 +21,7 @@ ARCHITECTURE:
 """
 
 import logging
-from datetime import datetime
+from datetime import datetime, timezone
 from pathlib import Path
 from typing import Any, Dict, List, Optional, Set
 
@@ -169,7 +169,7 @@ class MemoryManager(IMemoryManager):
             ]
 
         # Add new memory as a bullet point
-        timestamp = datetime.now().isoformat()
+        timestamp = datetime.now(timezone.utc).isoformat()
         lines.append(f"- [{timestamp}] {key}: {value}")
 
         # Write back
@@ -335,19 +335,37 @@ class MemoryManager(IMemoryManager):
         if pm_memories:
             aggregated_pm = self._aggregate_memories(pm_memories)
             result["actual_memories"] = aggregated_pm
-            memory_size = len(aggregated_pm.encode("utf-8"))
-            self.logger.info(
-                f"Aggregated PM memory ({memory_size:,} bytes) from {len(pm_memories)} source(s)"
-            )
+            # Count actual memory items in aggregated content
+            memory_items = [
+                line.strip()
+                for line in aggregated_pm.split("\n")
+                if line.strip().startswith("-")
+            ]
+            if memory_items:
+                self.logger.info(
+                    f"Aggregated PM memory: {len(memory_items)} total items from {len(pm_memories)} source(s)"
+                )
+            else:
+                self.logger.debug(
+                    f"Aggregated PM memory from {len(pm_memories)} source(s) (no items)"
+                )
 
         # Store agent memories (already aggregated per agent)
         if agent_memories_dict:
             result["agent_memories"] = agent_memories_dict
             for agent_name, memory_content in agent_memories_dict.items():
-                memory_size = len(memory_content.encode("utf-8"))
-                self.logger.debug(
-                    f"Aggregated {agent_name} memory: {memory_size:,} bytes"
-                )
+                # Count actual memory items
+                memory_items = [
+                    line.strip()
+                    for line in memory_content.split("\n")
+                    if line.strip().startswith("-")
+                ]
+                if memory_items:
+                    self.logger.debug(
+                        f"Aggregated {agent_name} memory: {len(memory_items)} items"
+                    )
+                else:
+                    self.logger.debug(f"Aggregated {agent_name} memory: no items")
 
         # Log summary
         if self._stats["loaded_count"] > 0 or self._stats["skipped_count"] > 0:
@@ -410,10 +428,21 @@ class MemoryManager(IMemoryManager):
                             "path": pm_memory_path,
                         }
                     )
-                    memory_size = len(loaded_content.encode("utf-8"))
-                    self.logger.info(
-                        f"Loaded {source} PM memory: {pm_memory_path} ({memory_size:,} bytes)"
-                    )
+                    # Count actual memory items (lines starting with "-")
+                    memory_items = [
+                        line.strip()
+                        for line in loaded_content.split("\n")
+                        if line.strip().startswith("-")
+                    ]
+                    if memory_items:
+                        self.logger.info(
+                            f"Loaded {source} PM memory: {len(memory_items)} items"
+                        )
+                    else:
+                        # Skip logging if no actual memory items
+                        self.logger.debug(
+                            f"Skipped {source} PM memory: {pm_memory_path} (no memory items)"
+                        )
                     self._stats["loaded_count"] += 1
             except Exception as e:
                 self.logger.error(
@@ -471,20 +500,47 @@ class MemoryManager(IMemoryManager):
                                 }
                             )
 
-                        memory_size = len(loaded_content.encode("utf-8"))
-                        self.logger.info(
-                            f"Loaded {source} memory for {agent_name}: {memory_file.name} ({memory_size:,} bytes)"
-                        )
+                        # Count actual memory items (lines starting with "-")
+                        memory_items = [
+                            line.strip()
+                            for line in loaded_content.split("\n")
+                            if line.strip().startswith("-")
+                        ]
+                        if memory_items:
+                            self.logger.info(
+                                f"Loaded {source} memory for {agent_name}: {len(memory_items)} items"
+                            )
+                        else:
+                            # Skip logging if no actual memory items
+                            self.logger.debug(
+                                f"Skipped {source} memory for {agent_name}: {memory_file.name} (no memory items)"
+                            )
                         self._stats["loaded_count"] += 1
                 except Exception as e:
                     self.logger.error(
                         f"Failed to load agent memory from {memory_file}: {e}"
                     )
             else:
-                # Log skipped memories
-                self.logger.info(
-                    f"Skipped {source} memory: {memory_file.name} (agent '{agent_name}' not deployed)"
-                )
+                # Log skipped memories only if they contain actual items
+                try:
+                    loaded_content = memory_file.read_text(encoding="utf-8")
+                    memory_items = [
+                        line.strip()
+                        for line in loaded_content.split("\n")
+                        if line.strip().startswith("-")
+                    ]
+                    if memory_items:
+                        self.logger.info(
+                            f"Skipped {source} memory: {memory_file.name} (agent '{agent_name}' not deployed, {len(memory_items)} items)"
+                        )
+                    else:
+                        # Don't log if file has no actual memory items
+                        self.logger.debug(
+                            f"Skipped {source} memory: {memory_file.name} (agent '{agent_name}' not deployed, no items)"
+                        )
+                except Exception:
+                    # If we can't read the file, just skip silently
+                    pass
 
                 # Detect naming mismatches
                 alt_name = (
@@ -570,7 +626,9 @@ class MemoryManager(IMemoryManager):
             lines.append("# Agent Memory")
 
         # Add latest timestamp
-        lines.append(f"<!-- Last Updated: {datetime.now().isoformat()}Z -->")
+        lines.append(
+            f"<!-- Last Updated: {datetime.now(timezone.utc).isoformat()}Z -->"
+        )
         lines.append("")
 
         # Add all unique items (sorted for consistency)

claude_mpm/services/core/path_resolver.py

@@ -163,7 +163,7 @@ class PathResolver(IPathResolver):
         self.logger.debug(f"No project root found from {start_path}")
         return None
 
-    def detect_framework_path(self) -> Optional[Path]:
+    def detect_framework_path(self) -> Optional[Path]:  # noqa: PLR0911
         """
         Auto-detect claude-mpm framework using unified path management.
 
@@ -322,8 +322,8 @@ class PathResolver(IPathResolver):
         """Detect framework path using unified path management."""
         try:
             # Import here to avoid circular dependencies
+            from ...core.unified_paths import DeploymentContext as UnifiedContext
             from ...core.unified_paths import (
-                DeploymentContext as UnifiedContext,
                 get_path_manager,
             )
 

claude_mpm/services/diagnostics/checks/installation_check.py

@@ -136,7 +136,7 @@ class InstallationCheck(BaseDiagnosticCheck):
                 details={"error": str(e)},
             )
 
-    def _check_installation_method(self) -> DiagnosticResult:
+    def _check_installation_method(self) -> DiagnosticResult:  # noqa: PLR0911
         """Detect how claude-mpm was installed."""
         methods_found = []
         details = {}

claude_mpm/services/event_aggregator.py

@@ -18,7 +18,7 @@ import sys
 import threading
 import time
 from collections import defaultdict
-from datetime import datetime
+from datetime import datetime, timezone
 from typing import Any, Dict, List, Optional
 
 try:
@@ -267,7 +267,9 @@ class EventAggregator:
         try:
             # Extract event metadata
             event_type = event_data.get("type", "unknown")
-            timestamp = event_data.get("timestamp", datetime.utcnow().isoformat() + "Z")
+            timestamp = event_data.get(
+                "timestamp", datetime.now(timezone.utc).isoformat() + "Z"
+            )
             data = event_data.get("data", {})
 
             # Update statistics

claude_mpm/services/event_bus/direct_relay.py

@@ -15,7 +15,7 @@ DO NOT use "event" or "type" fields - use "hook_event_name" instead!
 """
 
 import logging
-from datetime import datetime
+from datetime import datetime, timezone
 from typing import Any
 
 from .event_bus import EventBus
@@ -223,7 +223,9 @@ class DirectSocketIORelay:
                             event_type, broadcast_data
                         )
                         self.stats["events_relayed"] += 1
-                        self.stats["last_relay_time"] = datetime.now().isoformat()
+                        self.stats["last_relay_time"] = datetime.now(
+                            timezone.utc
+                        ).isoformat()
 
                         # Reset retry counter on successful broadcast
                         if self.connection_retries > 0:
@@ -258,7 +260,7 @@ class DirectSocketIORelay:
                                     logger.info(
                                         f"[DirectRelay] Retry successful for {event_type}"
                                     )
-                                except:
+                                except Exception:
                                     pass  # Already counted as failed
                 else:
                     # Enhanced logging when broadcaster is not available

claude_mpm/services/event_bus/event_bus.py

@@ -11,7 +11,7 @@ WHY pyee over alternatives:
 import asyncio
 import logging
 import threading
-from datetime import datetime
+from datetime import datetime, timezone
 from typing import Any, Callable, Dict, List, Optional, Set
 
 from pyee.asyncio import AsyncIOEventEmitter
@@ -210,7 +210,7 @@ class EventBus:
 
             # Update stats
             self._stats["events_published"] += 1
-            self._stats["last_event_time"] = datetime.now().isoformat()
+            self._stats["last_event_time"] = datetime.now(timezone.utc).isoformat()
 
             if self._debug:
                 logger.debug(f"Published event: {event_type}")
@@ -302,7 +302,7 @@ class EventBus:
             data: The event data
         """
         event_record = {
-            "timestamp": datetime.now().isoformat(),
+            "timestamp": datetime.now(timezone.utc).isoformat(),
             "type": event_type,
             "data": data,
         }

claude_mpm/services/event_bus/relay.py

@@ -11,7 +11,7 @@ WHY separate relay component:
 import logging
 import os
 import time
-from datetime import datetime
+from datetime import datetime, timezone
 from typing import Any, Dict, Optional
 
 # Socket.IO imports
@@ -151,7 +151,7 @@ class SocketIORelay:
                 # Verify connection is still alive
                 if self.client.connected:
                     return True
-            except:
+            except Exception:
                 pass
 
         # Need to create or reconnect
@@ -187,7 +187,9 @@ class SocketIORelay:
                     "subtype": (
                         event_type.split(".", 1)[1] if "." in event_type else "generic"
                     ),
-                    "timestamp": data.get("timestamp", datetime.now().isoformat()),
+                    "timestamp": data.get(
+                        "timestamp", datetime.now(timezone.utc).isoformat()
+                    ),
                     "data": data,
                     "source": "event_bus",
                 },
@@ -195,7 +197,7 @@ class SocketIORelay:
 
             # Update statistics
             self.stats["events_relayed"] += 1
-            self.stats["last_relay_time"] = datetime.now().isoformat()
+            self.stats["last_relay_time"] = datetime.now(timezone.utc).isoformat()
 
             if self.debug:
                 logger.debug(f"Relayed event to Socket.IO: {event_type}")

claude_mpm/services/events/consumers/dead_letter.py

@@ -6,7 +6,7 @@ Handles events that failed processing in other consumers.
 """
 
 import json
-from datetime import datetime
+from datetime import datetime, timezone
 from pathlib import Path
 from typing import Any, Dict, List, Optional
 
@@ -284,7 +284,7 @@ class DeadLetterConsumer(IEventConsumer):
 
     def _rotate_file(self) -> None:
         """Rotate to a new output file."""
-        timestamp = datetime.now().strftime("%Y%m%d-%H%M%S")
+        timestamp = datetime.now(timezone.utc).strftime("%Y%m%d-%H%M%S")
         self._current_file = self.output_dir / f"dead-letter-{timestamp}.jsonl"
         self._current_file_size = 0
         self._metrics["files_created"] += 1
@@ -293,7 +293,9 @@ class DeadLetterConsumer(IEventConsumer):
 
     async def _cleanup_old_files(self) -> None:
         """Remove files older than retention period."""
-        cutoff_time = datetime.now().timestamp() - (self.retention_days * 86400)
+        cutoff_time = datetime.now(timezone.utc).timestamp() - (
+            self.retention_days * 86400
+        )
 
         for file_path in self.output_dir.glob("dead-letter-*.jsonl"):
             try:

claude_mpm/services/events/core.py

@@ -12,7 +12,7 @@ import time
 import uuid
 from collections import defaultdict, deque
 from dataclasses import dataclass, field
-from datetime import datetime
+from datetime import datetime, timezone
 from enum import Enum
 from typing import Any, Deque, Dict, List, Optional, Set
 
@@ -225,7 +225,7 @@ class EventBus(IEventBus):
 
         # Add metadata
         if event.metadata:
-            event.metadata.published_at = datetime.now()
+            event.metadata.published_at = datetime.now(timezone.utc)
 
         # Queue event
         self._event_queues[event.priority].append(event)
@@ -353,7 +353,7 @@ class EventBus(IEventBus):
                 # Update metrics
                 if events_processed > 0:
                     self._metrics["events_processed"] += events_processed
-                    self._metrics["last_event_time"] = datetime.now()
+                    self._metrics["last_event_time"] = datetime.now(timezone.utc)
                     self._metrics["queue_size"] = sum(
                         len(q) for q in self._event_queues.values()
                     )