claude-mpm 4.1.0__py3-none-any.whl → 4.1.2__py3-none-any.whl

claude_mpm/services/framework_claude_md_generator/deployment_manager.py

@@ -95,7 +95,7 @@ class DeploymentManager:
 
         # Check if file exists and compare versions
         if target_file.exists() and not force:
-            with open(target_file, "r") as f:
+            with open(target_file) as f:
                 existing_content = f.read()
                 existing_fw_ver = self.version_manager.parse_current_version(
                     existing_content
@@ -119,7 +119,7 @@ class DeploymentManager:
 
             return True, f"Successfully deployed version {version_str}"
         except Exception as e:
-            return False, f"Deployment failed: {str(e)}"
+            return False, f"Deployment failed: {e!s}"
 
     def check_deployment_needed(self, parent_path: Path) -> Tuple[bool, str]:
         """
@@ -137,7 +137,7 @@ class DeploymentManager:
             return True, f"{self.target_filename} does not exist"
 
         try:
-            with open(target_file, "r") as f:
+            with open(target_file) as f:
                 existing_content = f.read()
                 existing_fw_ver = self.version_manager.parse_current_version(
                     existing_content
@@ -151,7 +151,7 @@ class DeploymentManager:
 
             return False, "Already up to date"
         except Exception as e:
-            return True, f"Error checking existing file: {str(e)}"
+            return True, f"Error checking existing file: {e!s}"
 
     def backup_existing(self, parent_path: Path) -> Optional[Path]:
         """

claude_mpm/services/framework_claude_md_generator/section_generators/__init__.py

@@ -32,7 +32,6 @@ class BaseSectionGenerator(ABC):
         Returns:
             str: Generated section content
         """
-        pass
 
     def get_timestamp(self) -> str:
         """Get current UTC timestamp."""

claude_mpm/services/framework_claude_md_generator/section_generators/agents.py

@@ -1,5 +1,3 @@
-from pathlib import Path
-
 """
 Agents section generator for framework CLAUDE.md.
 

claude_mpm/services/framework_claude_md_generator/version_manager.py

@@ -7,7 +7,7 @@ Handles version parsing, incrementing, and comparison operations.
 """
 
 import re
-from typing import Optional, Tuple
+from typing import Optional
 
 
 class VersionManager:
@@ -38,7 +38,7 @@ class VersionManager:
             version_path = package_path.parent / "framework" / "VERSION"
 
         if version_path.exists():
-            with open(version_path, "r") as f:
+            with open(version_path) as f:
                 version_content = f.read().strip()
                 # Framework VERSION file contains just the framework version number
                 try:
@@ -117,7 +117,6 @@ class VersionManager:
 
         if v1 < v2:
             return -1
-        elif v1 > v2:
+        if v1 > v2:
             return 1
-        else:
-            return 0
+        return 0

claude_mpm/services/hook_service.py

@@ -12,17 +12,15 @@ rather than a single list with type checking because:
 """
 
 import time
-from datetime import datetime
 from typing import Any, Dict, List, Optional
 
 from claude_mpm.core.config import Config
 from claude_mpm.core.interfaces import HookServiceInterface
-from claude_mpm.core.logging_config import get_logger, log_operation
+from claude_mpm.core.logging_config import get_logger
 from claude_mpm.hooks.base_hook import (
     BaseHook,
     HookContext,
     HookResult,
-    HookType,
     PostDelegationHook,
     PreDelegationHook,
 )
@@ -83,7 +81,7 @@ class HookService(HookServiceInterface):
                 )
                 return True
 
-            elif isinstance(hook, PostDelegationHook):
+            if isinstance(hook, PostDelegationHook):
                 self.post_delegation_hooks.append(hook)
                 # Sort by priority
                 self.post_delegation_hooks.sort(key=lambda h: h.priority)
@@ -92,11 +90,10 @@ class HookService(HookServiceInterface):
                 )
                 return True
 
-            else:
-                self.logger.warning(
-                    f"Attempted to register unsupported hook type: {type(hook).__name__}"
-                )
-                return False
+            self.logger.warning(
+                f"Attempted to register unsupported hook type: {type(hook).__name__}"
+            )
+            return False
 
         except Exception as e:
             self.logger.error(f"Failed to register hook {hook.name}: {e}")

claude_mpm/services/infrastructure/__init__.py

@@ -19,8 +19,8 @@ from .memory_guardian import MemoryGuardian
 from .monitoring import AdvancedHealthMonitor
 
 __all__ = [
-    "LoggingService",
     "AdvancedHealthMonitor",  # For SocketIO server monitoring
     "HealthMonitor",  # For Memory Guardian system monitoring
+    "LoggingService",
     "MemoryGuardian",
 ]

claude_mpm/services/infrastructure/context_preservation.py

@@ -14,14 +14,12 @@ Design Principles:
 
 import gzip
 import json
-import os
 import shutil
-import tempfile
-from datetime import datetime, timedelta
-from typing import Any, Dict, Iterator, List, Optional, Tuple
+from dataclasses import dataclass, field
+from datetime import datetime
+from typing import Any, Dict, List, Optional
 
 import ijson  # For streaming JSON parsing
-from dataclasses import dataclass, field
 
 from claude_mpm.services.core.base import BaseService
 
@@ -136,8 +134,7 @@ class ContextPreservationService(BaseService):
             if file_size_mb > self.large_file_threshold_mb:
                 self.log_info("Using streaming parser for large file")
                 return await self._parse_large_claude_json(extract_full)
-            else:
-                return await self._parse_standard_claude_json(extract_full)
+            return await self._parse_standard_claude_json(extract_full)
 
         except Exception as e:
             self.log_error(f"Failed to parse Claude JSON: {e}")
@@ -158,7 +155,6 @@ class ContextPreservationService(BaseService):
                 parser = ijson.parse(f)
 
                 active_conv_id = None
-                in_conversations = False
                 current_conv = {}
 
                 for prefix, event, value in parser:
@@ -169,7 +165,7 @@ class ContextPreservationService(BaseService):
                     # Parse conversations array
                     elif prefix.startswith("conversations.item"):
                         if event == "map_key":
-                            current_key = value
+                            pass
                         elif (
                             active_conv_id and current_conv.get("id") == active_conv_id
                         ):
@@ -204,12 +200,12 @@ class ContextPreservationService(BaseService):
             self.log_info(f"Compressing conversation history: {file_size_mb:.2f}MB")
 
             # Create backup first
-            backup_path = await self._create_backup()
+            await self._create_backup()
 
             # Load and filter conversations
             cutoff_time = datetime.now().timestamp() - (keep_recent_days * 86400)
 
-            with open(self.claude_json_path, "r") as f:
+            with open(self.claude_json_path) as f:
                 data = json.load(f)
 
             original_count = len(data.get("conversations", []))
@@ -341,7 +337,7 @@ class ContextPreservationService(BaseService):
     ) -> ConversationState:
         """Parse Claude JSON using standard JSON parser."""
         try:
-            with open(self.claude_json_path, "r") as f:
+            with open(self.claude_json_path) as f:
                 data = json.load(f)
 
             return await self._extract_conversation_state(data, extract_full)

claude_mpm/services/infrastructure/monitoring.py

@@ -15,9 +15,9 @@ Design Principles:
 """
 
 import asyncio
+import contextlib
 import logging
 import socket
-import threading
 import time
 from abc import ABC, abstractmethod
 from collections import deque
@@ -120,12 +120,10 @@ class HealthChecker(ABC):
     @abstractmethod
     def get_name(self) -> str:
         """Get the name of this health checker."""
-        pass
 
     @abstractmethod
     async def check_health(self) -> List[HealthMetric]:
         """Perform health check and return metrics."""
-        pass
 
 
 class ProcessResourceChecker(HealthChecker):
@@ -220,9 +218,11 @@ class ProcessResourceChecker(HealthChecker):
                 HealthMetric(
                     name="process_status",
                     value=status,
-                    status=HealthStatus.HEALTHY
-                    if process_healthy
-                    else HealthStatus.CRITICAL,
+                    status=(
+                        HealthStatus.HEALTHY
+                        if process_healthy
+                        else HealthStatus.CRITICAL
+                    ),
                     message=f"Process status: {status}",
                 )
             )
@@ -612,9 +612,9 @@ class ServiceHealthChecker(HealthChecker):
                 HealthMetric(
                     name="total_errors",
                     value=errors,
-                    status=HealthStatus.HEALTHY
-                    if errors == 0
-                    else HealthStatus.WARNING,
+                    status=(
+                        HealthStatus.HEALTHY if errors == 0 else HealthStatus.WARNING
+                    ),
                 )
             )
         except Exception as e:
@@ -843,7 +843,7 @@ class AdvancedHealthMonitor:
             return HealthStatus.UNKNOWN
 
         # Count metrics by status
-        status_counts = {status: 0 for status in HealthStatus}
+        status_counts = dict.fromkeys(HealthStatus, 0)
         for metric in metrics:
             status_counts[metric.status] += 1
 
@@ -890,10 +890,8 @@ class AdvancedHealthMonitor:
         self.monitoring = False
         if self.monitor_task:
             self.monitor_task.cancel()
-            try:
+            with contextlib.suppress(asyncio.CancelledError):
                 await self.monitor_task
-            except asyncio.CancelledError:
-                pass
             self.monitor_task = None
 
         self.logger.info("Stopped health monitoring")
@@ -966,7 +964,7 @@ class AdvancedHealthMonitor:
             }
 
         # Aggregate statistics
-        status_counts = {status: 0 for status in HealthStatus}
+        status_counts = dict.fromkeys(HealthStatus, 0)
         total_metrics = 0
         total_errors = 0
         total_duration_ms = 0
@@ -1019,18 +1017,18 @@ class AdvancedHealthMonitor:
                 "callbacks_count": len(self.health_callbacks),
             },
             "checkers": [checker.get_name() for checker in self.checkers],
-            "current_status": self.last_check_result.to_dict()
-            if self.last_check_result
-            else None,
+            "current_status": (
+                self.last_check_result.to_dict() if self.last_check_result else None
+            ),
             "aggregated_status": self.get_aggregated_status(),
             "monitoring_stats": dict(self.monitoring_stats),
             "history_summary": {
                 "total_checks": len(self.health_history),
-                "oldest_check": self.health_history[0].timestamp
-                if self.health_history
-                else None,
-                "newest_check": self.health_history[-1].timestamp
-                if self.health_history
-                else None,
+                "oldest_check": (
+                    self.health_history[0].timestamp if self.health_history else None
+                ),
+                "newest_check": (
+                    self.health_history[-1].timestamp if self.health_history else None
+                ),
             },
         }

claude_mpm/services/mcp_gateway/__init__.py

@@ -34,89 +34,89 @@ def __getattr__(name):
         from .core.interfaces import IMCPGateway
 
         return IMCPGateway
-    elif name == "IMCPToolRegistry":
+    if name == "IMCPToolRegistry":
         from .core.interfaces import IMCPToolRegistry
 
         return IMCPToolRegistry
-    elif name == "IMCPConfiguration":
+    if name == "IMCPConfiguration":
         from .core.interfaces import IMCPConfiguration
 
         return IMCPConfiguration
-    elif name == "IMCPToolAdapter":
+    if name == "IMCPToolAdapter":
         from .core.interfaces import IMCPToolAdapter
 
         return IMCPToolAdapter
-    elif name == "BaseMCPService":
+    if name == "BaseMCPService":
         from .core.base import BaseMCPService
 
         return BaseMCPService
 
     # Gateway implementations
-    elif name == "MCPGateway":
+    if name == "MCPGateway":
         from .server.mcp_gateway import MCPGateway
 
         return MCPGateway
-    elif name == "StdioHandler":
+    if name == "StdioHandler":
         from .server.stdio_handler import StdioHandler
 
         return StdioHandler
-    elif name == "AlternativeStdioHandler":
+    if name == "AlternativeStdioHandler":
         from .server.stdio_handler import AlternativeStdioHandler
 
         return AlternativeStdioHandler
 
     # Tool registry and adapters
-    elif name == "ToolRegistry":
+    if name == "ToolRegistry":
         from .registry.tool_registry import ToolRegistry
 
         return ToolRegistry
-    elif name == "BaseToolAdapter":
+    if name == "BaseToolAdapter":
         from .tools.base_adapter import BaseToolAdapter
 
         return BaseToolAdapter
-    elif name == "EchoToolAdapter":
+    if name == "EchoToolAdapter":
         from .tools.base_adapter import EchoToolAdapter
 
         return EchoToolAdapter
-    elif name == "CalculatorToolAdapter":
+    if name == "CalculatorToolAdapter":
         from .tools.base_adapter import CalculatorToolAdapter
 
         return CalculatorToolAdapter
-    elif name == "SystemInfoToolAdapter":
+    if name == "SystemInfoToolAdapter":
         from .tools.base_adapter import SystemInfoToolAdapter
 
         return SystemInfoToolAdapter
 
     # Configuration management
-    elif name == "MCPConfiguration":
+    if name == "MCPConfiguration":
         from .config.configuration import MCPConfiguration
 
         return MCPConfiguration
-    elif name == "MCPConfigLoader":
+    if name == "MCPConfigLoader":
         from .config.config_loader import MCPConfigLoader
 
         return MCPConfigLoader
 
     # Service registry
-    elif name == "MCPServiceRegistry":
+    if name == "MCPServiceRegistry":
         from .registry.service_registry import MCPServiceRegistry
 
         return MCPServiceRegistry
 
     # Exceptions
-    elif name == "MCPException":
+    if name == "MCPException":
         from .core.exceptions import MCPException
 
         return MCPException
-    elif name == "MCPConfigurationError":
+    if name == "MCPConfigurationError":
         from .core.exceptions import MCPConfigurationError
 
         return MCPConfigurationError
-    elif name == "MCPToolNotFoundError":
+    if name == "MCPToolNotFoundError":
         from .core.exceptions import MCPToolNotFoundError
 
         return MCPToolNotFoundError
-    elif name == "MCPServerError":
+    if name == "MCPServerError":
         from .core.exceptions import MCPServerError
 
         return MCPServerError
@@ -126,30 +126,30 @@ def __getattr__(name):
 
 # Public API exports
 __all__ = [
-    # Core interfaces
-    "IMCPGateway",
-    "IMCPToolRegistry",
-    "IMCPConfiguration",
-    "IMCPToolAdapter",
-    "BaseMCPService",
-    # Gateway implementations
-    "MCPGateway",
-    "StdioHandler",
     "AlternativeStdioHandler",
-    # Tool management
-    "ToolRegistry",
+    "BaseMCPService",
     "BaseToolAdapter",
-    "EchoToolAdapter",
     "CalculatorToolAdapter",
-    "SystemInfoToolAdapter",
+    "EchoToolAdapter",
+    "IMCPConfiguration",
+    # Core interfaces
+    "IMCPGateway",
+    "IMCPToolAdapter",
+    "IMCPToolRegistry",
+    "MCPConfigLoader",
     # Configuration
     "MCPConfiguration",
-    "MCPConfigLoader",
-    # Service registry
-    "MCPServiceRegistry",
+    "MCPConfigurationError",
     # Exceptions
     "MCPException",
-    "MCPConfigurationError",
-    "MCPToolNotFoundError",
+    # Gateway implementations
+    "MCPGateway",
     "MCPServerError",
+    # Service registry
+    "MCPServiceRegistry",
+    "MCPToolNotFoundError",
+    "StdioHandler",
+    "SystemInfoToolAdapter",
+    # Tool management
+    "ToolRegistry",
 ]