claude-mpm 4.4.3__py3-none-any.whl → 4.4.5__py3-none-any.whl

claude_mpm/services/unified/interfaces.py

@@ -120,7 +120,6 @@ class IDeploymentService(ABC):
         Returns:
             ServiceMetadata: Complete service metadata
         """
-        pass
 
     @abstractmethod
     def validate_deployment(
@@ -136,7 +135,6 @@ class IDeploymentService(ABC):
         Returns:
             List[str]: List of validation errors (empty if valid)
         """
-        pass
 
     @abstractmethod
     def deploy(
@@ -158,7 +156,6 @@ class IDeploymentService(ABC):
         Returns:
             DeploymentResult: Result of the deployment operation
         """
-        pass
 
     @abstractmethod
     def rollback(
@@ -174,7 +171,6 @@ class IDeploymentService(ABC):
         Returns:
             DeploymentResult: Result of the rollback operation
         """
-        pass
 
     @abstractmethod
     def list_deployments(
@@ -189,7 +185,6 @@ class IDeploymentService(ABC):
         Returns:
             List[Dict[str, Any]]: List of deployment information
         """
-        pass
 
     @abstractmethod
     def get_deployment_status(self, deployment_id: str) -> Dict[str, Any]:
@@ -202,7 +197,6 @@ class IDeploymentService(ABC):
         Returns:
             Dict[str, Any]: Deployment status information
         """
-        pass
 
 
 class IAnalyzerService(ABC):
@@ -222,7 +216,6 @@ class IAnalyzerService(ABC):
         Returns:
             ServiceMetadata: Complete service metadata
         """
-        pass
 
     @abstractmethod
     def analyze(
@@ -240,7 +233,6 @@ class IAnalyzerService(ABC):
         Returns:
             AnalysisResult: Result of the analysis
         """
-        pass
 
     @abstractmethod
     def batch_analyze(
@@ -258,7 +250,6 @@ class IAnalyzerService(ABC):
         Returns:
             List[AnalysisResult]: Results for each target
         """
-        pass
 
     @abstractmethod
     def get_metrics(self, target: Union[str, Path, Any]) -> Dict[str, Any]:
@@ -271,7 +262,6 @@ class IAnalyzerService(ABC):
         Returns:
             Dict[str, Any]: Analysis metrics
         """
-        pass
 
     @abstractmethod
     def compare(
@@ -291,7 +281,6 @@ class IAnalyzerService(ABC):
         Returns:
             Dict[str, Any]: Comparison results
         """
-        pass
 
     @abstractmethod
     def get_recommendations(
@@ -306,7 +295,6 @@ class IAnalyzerService(ABC):
         Returns:
             List[Dict[str, Any]]: List of recommendations
         """
-        pass
 
 
 class IConfigurationService(ABC):
@@ -326,7 +314,6 @@ class IConfigurationService(ABC):
         Returns:
             ServiceMetadata: Complete service metadata
         """
-        pass
 
     @abstractmethod
     def load_config(
@@ -341,7 +328,6 @@ class IConfigurationService(ABC):
         Returns:
             ConfigurationResult: Loaded configuration result
         """
-        pass
 
     @abstractmethod
     def save_config(
@@ -357,7 +343,6 @@ class IConfigurationService(ABC):
         Returns:
             ConfigurationResult: Save operation result
         """
-        pass
 
     @abstractmethod
     def validate_config(self, config: Dict[str, Any]) -> List[str]:
@@ -370,7 +355,6 @@ class IConfigurationService(ABC):
         Returns:
             List[str]: List of validation errors (empty if valid)
         """
-        pass
 
     @abstractmethod
     def merge_configs(
@@ -386,7 +370,6 @@ class IConfigurationService(ABC):
         Returns:
             Dict[str, Any]: Merged configuration
         """
-        pass
 
     @abstractmethod
     def get_config_value(
@@ -403,7 +386,6 @@ class IConfigurationService(ABC):
         Returns:
             Any: Configuration value
         """
-        pass
 
     @abstractmethod
     def set_config_value(
@@ -420,7 +402,6 @@ class IConfigurationService(ABC):
         Returns:
             ConfigurationResult: Result of set operation
         """
-        pass
 
     @abstractmethod
     def get_schema(self) -> Dict[str, Any]:
@@ -430,7 +411,6 @@ class IConfigurationService(ABC):
         Returns:
             Dict[str, Any]: Configuration schema
         """
-        pass
 
     @abstractmethod
     def apply_defaults(self, config: Dict[str, Any]) -> Dict[str, Any]:
@@ -443,7 +423,6 @@ class IConfigurationService(ABC):
         Returns:
             Dict[str, Any]: Configuration with defaults applied
         """
-        pass
 
 
 class IUnifiedService(ABC):
@@ -462,14 +441,12 @@ class IUnifiedService(ABC):
         Returns:
             bool: True if initialization successful
         """
-        pass
 
     @abstractmethod
     async def shutdown(self) -> None:
         """
         Gracefully shutdown the service.
         """
-        pass
 
     @abstractmethod
     def health_check(self) -> Dict[str, Any]:
@@ -479,7 +456,6 @@ class IUnifiedService(ABC):
         Returns:
             Dict[str, Any]: Health status information
         """
-        pass
 
     @abstractmethod
     def get_metrics(self) -> Dict[str, Any]:
@@ -489,11 +465,9 @@ class IUnifiedService(ABC):
         Returns:
             Dict[str, Any]: Service metrics
         """
-        pass
 
     @abstractmethod
     def reset(self) -> None:
         """
         Reset service to initial state.
         """
-        pass

claude_mpm/services/unified/migration.py

@@ -21,10 +21,9 @@ Migration Strategy:
 """
 
 import inspect
-from dataclasses import dataclass, field
+from dataclasses import dataclass
 from enum import Enum
-from pathlib import Path
-from typing import Any, Callable, Dict, List, Optional, Set, Type, TypeVar
+from typing import Any, Dict, List, Optional, Type, TypeVar
 
 from claude_mpm.core.logging_utils import get_logger
 
@@ -131,9 +130,7 @@ class ServiceMapper:
     def __init__(self):
         """Initialize service mapper."""
         self._mappings: Dict[str, ServiceMapping] = {}
-        self._feature_flags: Dict[FeatureFlag, bool] = {
-            flag: False for flag in FeatureFlag
-        }
+        self._feature_flags: Dict[FeatureFlag, bool] = dict.fromkeys(FeatureFlag, False)
         self._metrics = MigrationMetrics()
         self._logger = get_logger(f"{__name__}.ServiceMapper")
         self._initialize_mappings()
@@ -199,9 +196,7 @@ class ServiceMapper:
         """
         return self._mappings.get(legacy_service)
 
-    def get_unified_service(
-        self, legacy_path: str, legacy_class: str
-    ) -> Optional[str]:
+    def get_unified_service(self, legacy_path: str, legacy_class: str) -> Optional[str]:
         """
         Get unified service for a legacy service.
 
@@ -216,9 +211,7 @@ class ServiceMapper:
         mapping = self._mappings.get(key)
         return mapping.unified_service if mapping else None
 
-    def update_status(
-        self, legacy_service: str, status: MigrationStatus
-    ) -> bool:
+    def update_status(self, legacy_service: str, status: MigrationStatus) -> bool:
         """
         Update migration status for a service.
 
@@ -247,9 +240,7 @@ class ServiceMapper:
             elif status == MigrationStatus.REMOVED:
                 self._metrics.removed_services += 1
 
-        self._logger.info(
-            f"Updated {legacy_service} status: {old_status} -> {status}"
-        )
+        self._logger.info(f"Updated {legacy_service} status: {old_status} -> {status}")
         return True
 
     def set_feature_flag(self, flag: FeatureFlag, enabled: bool) -> None:
@@ -336,9 +327,7 @@ def create_compatibility_wrapper(
         def __init__(self, *args, **kwargs):
             """Initialize wrapper with unified service."""
             self._unified_service = unified_service
-            self._logger = get_logger(
-                f"{__name__}.{legacy_class.__name__}Wrapper"
-            )
+            self._logger = get_logger(f"{__name__}.{legacy_class.__name__}Wrapper")
             self._logger.debug(
                 f"Created compatibility wrapper for {legacy_class.__name__}"
             )
@@ -375,9 +364,7 @@ def create_compatibility_wrapper(
                 return attr
 
             # Fallback to legacy implementation if allowed
-            if ServiceMapper().is_feature_enabled(
-                FeatureFlag.ALLOW_LEGACY_FALLBACK
-            ):
+            if ServiceMapper().is_feature_enabled(FeatureFlag.ALLOW_LEGACY_FALLBACK):
                 if hasattr(legacy_class, name):
                     self._logger.warning(
                         f"Falling back to legacy implementation for {name}"
@@ -434,26 +421,22 @@ class MigrationValidator:
 
         # Get public methods from legacy class
         legacy_methods = {
-            name for name, _ in inspect.getmembers(
-                legacy_class, inspect.ismethod
-            )
+            name
+            for name, _ in inspect.getmembers(legacy_class, inspect.ismethod)
             if not name.startswith("_")
         }
 
         # Get public methods from unified class
         unified_methods = {
-            name for name, _ in inspect.getmembers(
-                unified_class, inspect.ismethod
-            )
+            name
+            for name, _ in inspect.getmembers(unified_class, inspect.ismethod)
             if not name.startswith("_")
         }
 
         # Check for missing methods
         missing = legacy_methods - unified_methods
         if missing:
-            issues.append(
-                f"Missing methods in unified service: {', '.join(missing)}"
-            )
+            issues.append(f"Missing methods in unified service: {', '.join(missing)}")
 
         # Check method signatures
         for method_name in legacy_methods & unified_methods:
@@ -494,14 +477,10 @@ class MigrationValidator:
 
             try:
                 # Execute on legacy service
-                legacy_result = getattr(legacy_instance, method_name)(
-                    *args, **kwargs
-                )
+                legacy_result = getattr(legacy_instance, method_name)(*args, **kwargs)
 
                 # Execute on unified service
-                unified_result = getattr(unified_instance, method_name)(
-                    *args, **kwargs
-                )
+                unified_result = getattr(unified_instance, method_name)(*args, **kwargs)
 
                 # Compare results
                 if legacy_result != unified_result:
@@ -511,9 +490,7 @@ class MigrationValidator:
                     )
 
             except Exception as e:
-                differences.append(
-                    f"Error testing {method_name}: {str(e)}"
-                )
+                differences.append(f"Error testing {method_name}: {e!s}")
 
         return differences
 
@@ -529,4 +506,4 @@ def get_service_mapper() -> ServiceMapper:
     Returns:
         ServiceMapper: Global mapper instance
     """
-    return _global_mapper
+    return _global_mapper

claude_mpm/services/unified/strategies.py

@@ -24,7 +24,7 @@ from abc import ABC, abstractmethod
 from dataclasses import dataclass, field
 from enum import Enum
 from pathlib import Path
-from typing import Any, Callable, Dict, List, Optional, Set, Type, TypeVar, Union
+from typing import Any, Dict, List, Optional, Set, Type, TypeVar, Union
 
 from claude_mpm.core.logging_utils import get_logger
 
@@ -104,9 +104,7 @@ class BaseStrategy(ABC):
         Args:
             metadata: Strategy metadata
         """
-        self.metadata = metadata or StrategyMetadata(
-            name=self.__class__.__name__
-        )
+        self.metadata = metadata or StrategyMetadata(name=self.__class__.__name__)
         self._logger = get_logger(f"{__name__}.{self.metadata.name}")
 
     @abstractmethod
@@ -120,7 +118,6 @@ class BaseStrategy(ABC):
         Returns:
             bool: True if strategy can handle context
         """
-        pass
 
     @abstractmethod
     def validate_input(self, input_data: Any) -> List[str]:
@@ -133,7 +130,6 @@ class BaseStrategy(ABC):
         Returns:
             List[str]: List of validation errors (empty if valid)
         """
-        pass
 
     def pre_execute(self, input_data: Any) -> Any:
         """
@@ -186,7 +182,6 @@ class DeploymentStrategy(BaseStrategy):
         Returns:
             Dict[str, Any]: Deployment result
         """
-        pass
 
     @abstractmethod
     def prepare_rollback(self, deployment_info: Dict[str, Any]) -> Dict[str, Any]:
@@ -199,7 +194,6 @@ class DeploymentStrategy(BaseStrategy):
         Returns:
             Dict[str, Any]: Rollback information
         """
-        pass
 
     @abstractmethod
     def cleanup(self, target: Union[str, Path]) -> bool:
@@ -212,7 +206,6 @@ class DeploymentStrategy(BaseStrategy):
         Returns:
             bool: True if cleanup successful
         """
-        pass
 
 
 class AnalyzerStrategy(BaseStrategy):
@@ -237,7 +230,6 @@ class AnalyzerStrategy(BaseStrategy):
         Returns:
             Dict[str, Any]: Analysis results
         """
-        pass
 
     @abstractmethod
     def extract_metrics(self, analysis_result: Dict[str, Any]) -> Dict[str, Any]:
@@ -250,7 +242,6 @@ class AnalyzerStrategy(BaseStrategy):
         Returns:
             Dict[str, Any]: Extracted metrics
         """
-        pass
 
     @abstractmethod
     def generate_report(
@@ -266,7 +257,6 @@ class AnalyzerStrategy(BaseStrategy):
         Returns:
             Union[str, Dict[str, Any]]: Generated report
         """
-        pass
 
 
 class ConfigStrategy(BaseStrategy):
@@ -288,7 +278,6 @@ class ConfigStrategy(BaseStrategy):
         Returns:
             Dict[str, Any]: Loaded configuration
         """
-        pass
 
     @abstractmethod
     def validate(self, config: Dict[str, Any]) -> List[str]:
@@ -301,7 +290,6 @@ class ConfigStrategy(BaseStrategy):
         Returns:
             List[str]: Validation errors (empty if valid)
         """
-        pass
 
     @abstractmethod
     def transform(self, config: Dict[str, Any]) -> Dict[str, Any]:
@@ -314,7 +302,6 @@ class ConfigStrategy(BaseStrategy):
         Returns:
             Dict[str, Any]: Transformed configuration
         """
-        pass
 
     @abstractmethod
     def get_schema(self) -> Dict[str, Any]:
@@ -324,7 +311,6 @@ class ConfigStrategy(BaseStrategy):
         Returns:
             Dict[str, Any]: Configuration schema
         """
-        pass
 
 
 class StrategyRegistry:
@@ -372,7 +358,8 @@ class StrategyRegistry:
         # Remove old strategy if overriding
         if override and strategy_name in self._strategy_metadata:
             self._strategies[strategy_class] = [
-                s for s in self._strategies[strategy_class]
+                s
+                for s in self._strategies[strategy_class]
                 if s.metadata.name != strategy_name
             ]
 
@@ -399,7 +386,8 @@ class StrategyRegistry:
         # Remove from all strategy lists
         for strategy_class in self._strategies:
             self._strategies[strategy_class] = [
-                s for s in self._strategies[strategy_class]
+                s
+                for s in self._strategies[strategy_class]
                 if s.metadata.name != strategy_name
             ]
 
@@ -435,16 +423,11 @@ class StrategyRegistry:
                 candidates.append(strategy)
 
         if not candidates:
-            self._logger.warning(
-                f"No suitable strategy found for context: {context}"
-            )
+            self._logger.warning(f"No suitable strategy found for context: {context}")
             return None
 
         # Sort by priority (highest first)
-        candidates.sort(
-            key=lambda s: s.metadata.priority.value,
-            reverse=True
-        )
+        candidates.sort(key=lambda s: s.metadata.priority.value, reverse=True)
 
         selected = candidates[0]
         self._logger.debug(
@@ -548,4 +531,4 @@ def select_strategy(
     Returns:
         Optional[BaseStrategy]: Selected strategy or None
     """
-    return _global_registry.select_strategy(strategy_class, context)
+    return _global_registry.select_strategy(strategy_class, context)

claude_mpm/services/unified/unified_analyzer.py

@@ -22,7 +22,6 @@ Features:
 - Recommendation generation
 """
 
-import asyncio
 from pathlib import Path
 from typing import Any, Dict, List, Optional, Union
 
@@ -111,7 +110,7 @@ class UnifiedAnalyzer(IAnalyzerService, IUnifiedService):
             return True
 
         except Exception as e:
-            self._logger.error(f"Failed to initialize: {str(e)}")
+            self._logger.error(f"Failed to initialize: {e!s}")
             return False
 
     async def shutdown(self) -> None:
@@ -228,9 +227,7 @@ class UnifiedAnalyzer(IAnalyzerService, IUnifiedService):
                 )
 
             # Execute analysis using strategy
-            self._logger.info(
-                f"Analyzing {target} using {strategy.metadata.name}"
-            )
+            self._logger.info(f"Analyzing {target} using {strategy.metadata.name}")
 
             # Validate input
             validation_errors = strategy.validate_input(target)
@@ -264,11 +261,11 @@ class UnifiedAnalyzer(IAnalyzerService, IUnifiedService):
             return result
 
         except Exception as e:
-            self._logger.error(f"Analysis error: {str(e)}")
+            self._logger.error(f"Analysis error: {e!s}")
             self._metrics["analysis_errors"] += 1
             return AnalysisResult(
                 success=False,
-                summary=f"Analysis failed: {str(e)}",
+                summary=f"Analysis failed: {e!s}",
                 severity="error",
             )
 
@@ -380,7 +377,7 @@ class UnifiedAnalyzer(IAnalyzerService, IUnifiedService):
             }
 
         except Exception as e:
-            self._logger.error(f"Comparison error: {str(e)}")
+            self._logger.error(f"Comparison error: {e!s}")
             return {"success": False, "error": str(e)}
 
     def get_recommendations(
@@ -399,31 +396,37 @@ class UnifiedAnalyzer(IAnalyzerService, IUnifiedService):
 
         # Basic recommendations from result
         for rec in analysis_result.recommendations:
-            recommendations.append({
-                "type": "general",
-                "description": rec,
-                "priority": "medium",
-            })
+            recommendations.append(
+                {
+                    "type": "general",
+                    "description": rec,
+                    "priority": "medium",
+                }
+            )
 
         # Add severity-based recommendations
         if analysis_result.severity == "critical":
-            recommendations.insert(0, {
-                "type": "urgent",
-                "description": "Critical issues found - immediate attention required",
-                "priority": "high",
-            })
+            recommendations.insert(
+                0,
+                {
+                    "type": "urgent",
+                    "description": "Critical issues found - immediate attention required",
+                    "priority": "high",
+                },
+            )
         elif analysis_result.severity == "error":
-            recommendations.insert(0, {
-                "type": "important",
-                "description": "Errors found - should be addressed soon",
-                "priority": "high",
-            })
+            recommendations.insert(
+                0,
+                {
+                    "type": "important",
+                    "description": "Errors found - should be addressed soon",
+                    "priority": "high",
+                },
+            )
 
         # Add metric-based recommendations
         if analysis_result.metrics:
-            metric_recs = self._generate_metric_recommendations(
-                analysis_result.metrics
-            )
+            metric_recs = self._generate_metric_recommendations(analysis_result.metrics)
             recommendations.extend(metric_recs)
 
         return recommendations
@@ -436,9 +439,7 @@ class UnifiedAnalyzer(IAnalyzerService, IUnifiedService):
         # This would be extended with actual strategy implementations
         self._logger.debug("Default strategies registered")
 
-    def _determine_analysis_type(
-        self, target: Any, options: Dict[str, Any]
-    ) -> str:
+    def _determine_analysis_type(self, target: Any, options: Dict[str, Any]) -> str:
         """
         Determine analysis type from target and options.
 
@@ -460,7 +461,7 @@ class UnifiedAnalyzer(IAnalyzerService, IUnifiedService):
                 # Determine by file extension
                 if path.suffix in [".py", ".js", ".ts", ".java"]:
                     return "code"
-                elif path.suffix in [".json", ".yaml", ".yml"]:
+                if path.suffix in [".json", ".yaml", ".yml"]:
                     return "config"
             elif path.is_dir():
                 return "project"
@@ -482,7 +483,6 @@ class UnifiedAnalyzer(IAnalyzerService, IUnifiedService):
     def _aggregate_batch_metrics(self, results: List[AnalysisResult]) -> None:
         """Aggregate metrics from batch analysis."""
         # Implementation would aggregate metrics across results
-        pass
 
     def _compare_metrics(
         self, metrics1: Dict[str, Any], metrics2: Dict[str, Any]
@@ -492,8 +492,8 @@ class UnifiedAnalyzer(IAnalyzerService, IUnifiedService):
         all_keys = set(metrics1.keys()) | set(metrics2.keys())
 
         for key in all_keys:
-            val1 = metrics1.get(key, None)
-            val2 = metrics2.get(key, None)
+            val1 = metrics1.get(key)
+            val2 = metrics2.get(key)
 
             if val1 != val2:
                 diff[key] = {"target1": val1, "target2": val2}
@@ -518,17 +518,21 @@ class UnifiedAnalyzer(IAnalyzerService, IUnifiedService):
 
         # Example metric-based recommendations
         if metrics.get("complexity", 0) > 10:
-            recommendations.append({
-                "type": "complexity",
-                "description": "Consider refactoring to reduce complexity",
-                "priority": "medium",
-            })
+            recommendations.append(
+                {
+                    "type": "complexity",
+                    "description": "Consider refactoring to reduce complexity",
+                    "priority": "medium",
+                }
+            )
 
         if metrics.get("code_duplication", 0) > 20:
-            recommendations.append({
-                "type": "duplication",
-                "description": "High code duplication detected - consider extracting common functionality",
-                "priority": "medium",
-            })
+            recommendations.append(
+                {
+                    "type": "duplication",
+                    "description": "High code duplication detected - consider extracting common functionality",
+                    "priority": "medium",
+                }
+            )
 
-        return recommendations
+        return recommendations