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

claude_mpm/services/agents/deployment/pipeline/__init__.py

@@ -22,11 +22,11 @@ from .steps import (
 )
 
 __all__ = [
+    "BaseDeploymentStep",
+    "ConfigurationLoadStep",
     "DeploymentPipelineBuilder",
     "DeploymentPipelineExecutor",
     "PipelineContext",
-    "BaseDeploymentStep",
     "StepResult",
-    "ConfigurationLoadStep",
     "TargetDirectorySetupStep",
 ]

claude_mpm/services/agents/deployment/pipeline/pipeline_builder.py

@@ -1,6 +1,6 @@
 """Pipeline builder for creating deployment pipelines."""
 
-from typing import Any, Dict, List, Type
+from typing import Dict, List, Type
 
 from claude_mpm.core.logger import get_logger
 

claude_mpm/services/agents/deployment/pipeline/pipeline_context.py

@@ -42,7 +42,9 @@ class PipelineContext:
     template_files: List[Path] = field(default_factory=list)
     base_agent_data: Optional[Dict[str, Any]] = None
     base_agent_version: Optional[tuple] = None
-    agent_sources: Dict[str, str] = field(default_factory=dict)  # Maps agent names to sources
+    agent_sources: Dict[str, str] = field(
+        default_factory=dict
+    )  # Maps agent names to sources
 
     # Deployment results
     results: Dict[str, Any] = field(default_factory=dict)
@@ -110,9 +112,9 @@ class PipelineContext:
         """Initialize the results dictionary with default structure."""
         if not self.results:
             self.results = {
-                "target_dir": str(self.actual_target_dir)
-                if self.actual_target_dir
-                else "",
+                "target_dir": (
+                    str(self.actual_target_dir) if self.actual_target_dir else ""
+                ),
                 "deployed": [],
                 "updated": [],
                 "migrated": [],

claude_mpm/services/agents/deployment/pipeline/pipeline_executor.py

@@ -6,7 +6,7 @@ from typing import Any, Dict, List
 from claude_mpm.core.logger import get_logger
 
 from .pipeline_context import PipelineContext
-from .steps.base_step import BaseDeploymentStep, StepResult, StepStatus
+from .steps.base_step import BaseDeploymentStep
 
 
 class DeploymentPipelineExecutor:
@@ -93,7 +93,7 @@ class DeploymentPipelineExecutor:
                 step_execution_time = time.time() - step_start_time
                 context.step_timings[step.name] = step_execution_time
 
-                error_msg = f"Unexpected error in step {step.name}: {str(e)}"
+                error_msg = f"Unexpected error in step {step.name}: {e!s}"
                 self.logger.error(error_msg, exc_info=True)
                 context.add_error(error_msg)
                 failed_steps.append(step.name)
@@ -152,7 +152,7 @@ class DeploymentPipelineExecutor:
 
         # Check for duplicate step names
         step_names = [step.name for step in steps]
-        duplicates = set([name for name in step_names if step_names.count(name) > 1])
+        duplicates = {name for name in step_names if step_names.count(name) > 1}
         if duplicates:
             errors.append(f"Duplicate step names found: {duplicates}")
 

claude_mpm/services/agents/deployment/pipeline/steps/__init__.py

@@ -11,9 +11,9 @@ from .configuration_step import ConfigurationLoadStep
 from .target_directory_step import TargetDirectorySetupStep
 
 __all__ = [
+    "AgentProcessingStep",
     "BaseDeploymentStep",
-    "StepResult",
     "ConfigurationLoadStep",
+    "StepResult",
     "TargetDirectorySetupStep",
-    "AgentProcessingStep",
 ]

claude_mpm/services/agents/deployment/pipeline/steps/agent_processing_step.py

@@ -1,7 +1,6 @@
 """Agent processing step for deployment pipeline."""
 
 import time
-
 from pathlib import Path
 
 from claude_mpm.services.agents.deployment.processors import (
@@ -57,7 +56,7 @@ class AgentProcessingStep(BaseDeploymentStep):
                         source_info = context.agent_sources[agent_name]
                     else:
                         source_info = self._determine_agent_source(template_file)
-                    
+
                     # Create agent deployment context
                     agent_context = AgentDeploymentContext.from_template_file(
                         template_file=template_file,
@@ -93,9 +92,7 @@ class AgentProcessingStep(BaseDeploymentStep):
                         failed_count += 1
 
                 except Exception as e:
-                    error_msg = (
-                        f"Failed to process agent {template_file.stem}: {str(e)}"
-                    )
+                    error_msg = f"Failed to process agent {template_file.stem}: {e!s}"
                     self.logger.error(error_msg, exc_info=True)
                     context.add_error(error_msg)
                     failed_count += 1
@@ -125,7 +122,7 @@ class AgentProcessingStep(BaseDeploymentStep):
             execution_time = time.time() - start_time
             context.step_timings[self.name] = execution_time
 
-            error_msg = f"Agent processing step failed: {str(e)}"
+            error_msg = f"Agent processing step failed: {e!s}"
             self.logger.error(error_msg, exc_info=True)
             context.add_error(error_msg)
 
@@ -207,32 +204,36 @@ class AgentProcessingStep(BaseDeploymentStep):
 
     def _determine_agent_source(self, template_path: Path) -> str:
         """Determine the source of an agent from its template path.
-        
+
         Args:
             template_path: Path to the agent template
-            
+
         Returns:
             Source string (system/project/user/unknown)
         """
         template_str = str(template_path.resolve())
-        
+
         # Check if it's a system template
-        if "/claude_mpm/agents/templates/" in template_str or "/src/claude_mpm/agents/templates/" in template_str:
+        if (
+            "/claude_mpm/agents/templates/" in template_str
+            or "/src/claude_mpm/agents/templates/" in template_str
+        ):
             return "system"
-        
+
         # Check if it's a project agent
         if "/.claude-mpm/agents/" in template_str:
             # Check if it's in the current working directory
             try:
                 from pathlib import Path
+
                 cwd = Path.cwd()
                 if str(cwd) in template_str:
                     return "project"
             except:
                 pass
-        
+
         # Check if it's a user agent
         if "/.claude/agents/" in template_str:
             return "user"
-            
+
         return "unknown"

claude_mpm/services/agents/deployment/pipeline/steps/base_step.py

@@ -76,7 +76,6 @@ class BaseDeploymentStep(ABC):
         Returns:
             Result of executing this step
         """
-        pass
 
     def should_execute(self, context) -> bool:
         """Check if this step should be executed.

claude_mpm/services/agents/deployment/pipeline/steps/configuration_step.py

@@ -63,7 +63,7 @@ class ConfigurationLoadStep(BaseDeploymentStep):
             execution_time = time.time() - start_time
             context.step_timings[self.name] = execution_time
 
-            error_msg = f"Failed to load configuration: {str(e)}"
+            error_msg = f"Failed to load configuration: {e!s}"
             self.logger.error(error_msg)
             context.add_error(error_msg)
 

claude_mpm/services/agents/deployment/pipeline/steps/target_directory_step.py

@@ -32,14 +32,13 @@ class TargetDirectorySetupStep(BaseDeploymentStep):
                 context.actual_target_dir = context.strategy.determine_target_directory(
                     context  # Pass the context which has the DeploymentContext interface
                 )
+            # Fallback logic if no strategy is set
+            elif context.target_dir:
+                context.actual_target_dir = context.target_dir
             else:
-                # Fallback logic if no strategy is set
-                if context.target_dir:
-                    context.actual_target_dir = context.target_dir
-                else:
-                    # MODIFIED: Default to project .claude/agents directory
-                    # All agents now deploy to the project level
-                    context.actual_target_dir = Path.cwd() / ".claude" / "agents"
+                # MODIFIED: Default to project .claude/agents directory
+                # All agents now deploy to the project level
+                context.actual_target_dir = Path.cwd() / ".claude" / "agents"
 
             # Create target directory if it doesn't exist
             context.actual_target_dir.mkdir(parents=True, exist_ok=True)
@@ -49,7 +48,7 @@ class TargetDirectorySetupStep(BaseDeploymentStep):
             try:
                 test_file.write_text("test")
                 test_file.unlink()
-            except Exception as e:
+            except Exception:
                 raise PermissionError(
                     f"Target directory is not writable: {context.actual_target_dir}"
                 )
@@ -69,7 +68,7 @@ class TargetDirectorySetupStep(BaseDeploymentStep):
             execution_time = time.time() - start_time
             context.step_timings[self.name] = execution_time
 
-            error_msg = f"Failed to set up target directory: {str(e)}"
+            error_msg = f"Failed to set up target directory: {e!s}"
             self.logger.error(error_msg)
             context.add_error(error_msg)
 

claude_mpm/services/agents/deployment/pipeline/steps/validation_step.py

@@ -68,7 +68,7 @@ class ValidationStep(BaseDeploymentStep):
             execution_time = time.time() - start_time
             context.step_timings[self.name] = execution_time
 
-            error_msg = f"Validation step failed: {str(e)}"
+            error_msg = f"Validation step failed: {e!s}"
             self.logger.error(error_msg, exc_info=True)
             context.add_error(error_msg)
 

claude_mpm/services/agents/deployment/processors/__init__.py

@@ -9,7 +9,7 @@ from .agent_deployment_result import AgentDeploymentResult
 from .agent_processor import AgentProcessor
 
 __all__ = [
-    "AgentProcessor",
     "AgentDeploymentContext",
     "AgentDeploymentResult",
+    "AgentProcessor",
 ]

claude_mpm/services/agents/deployment/processors/agent_processor.py

@@ -6,7 +6,7 @@ from claude_mpm.core.exceptions import AgentDeploymentError
 from claude_mpm.core.logger import get_logger
 
 from .agent_deployment_context import AgentDeploymentContext
-from .agent_deployment_result import AgentDeploymentResult, AgentDeploymentStatus
+from .agent_deployment_result import AgentDeploymentResult
 
 
 class AgentProcessor:
@@ -77,7 +77,7 @@ class AgentProcessor:
                     deployment_time_ms,
                     reason,
                 )
-            elif context.is_update():
+            if context.is_update():
                 self.logger.debug(f"Updated agent: {context.agent_name}")
                 return AgentDeploymentResult.updated(
                     context.agent_name,
@@ -86,21 +86,18 @@ class AgentProcessor:
                     deployment_time_ms,
                     reason,
                 )
-            else:
-                self.logger.debug(f"Deployed new agent: {context.agent_name}")
-                return AgentDeploymentResult.deployed(
-                    context.agent_name,
-                    context.template_file,
-                    context.target_file,
-                    deployment_time_ms,
-                )
+            self.logger.debug(f"Deployed new agent: {context.agent_name}")
+            return AgentDeploymentResult.deployed(
+                context.agent_name,
+                context.template_file,
+                context.target_file,
+                deployment_time_ms,
+            )
 
         except AgentDeploymentError as e:
             # Re-raise our custom exceptions
             deployment_time_ms = (time.time() - start_time) * 1000
-            self.logger.error(
-                f"Agent deployment error for {context.agent_name}: {str(e)}"
-            )
+            self.logger.error(f"Agent deployment error for {context.agent_name}: {e!s}")
             return AgentDeploymentResult.failed(
                 context.agent_name,
                 context.template_file,
@@ -151,11 +148,15 @@ class AgentProcessor:
                 # File doesn't exist, needs to be deployed
                 needs_update = True
                 reason = "New agent deployment"
-                self.logger.debug(f"Agent {context.agent_name} doesn't exist, will deploy")
+                self.logger.debug(
+                    f"Agent {context.agent_name} doesn't exist, will deploy"
+                )
             else:
                 # File exists, check version compatibility
                 needs_update, reason = self.version_manager.check_agent_needs_update(
-                    context.target_file, context.template_file, context.base_agent_version
+                    context.target_file,
+                    context.template_file,
+                    context.base_agent_version,
                 )
                 if needs_update:
                     # Check if this is a migration from old format
@@ -185,7 +186,10 @@ class AgentProcessor:
         """
         try:
             return self.template_builder.build_agent_markdown(
-                context.agent_name, context.template_file, context.base_agent_data, context.source_info
+                context.agent_name,
+                context.template_file,
+                context.base_agent_data,
+                context.source_info,
             )
         except Exception as e:
             raise AgentDeploymentError(

claude_mpm/services/agents/deployment/refactored_agent_deployment_service.py

@@ -3,7 +3,6 @@
 from pathlib import Path
 from typing import Any, Dict, List, Optional, Tuple
 
-from claude_mpm.core.config import Config
 from claude_mpm.core.interfaces import AgentDeploymentInterface
 from claude_mpm.core.logger import get_logger
 
@@ -12,10 +11,8 @@ from .facade import DeploymentFacade
 from .pipeline import (
     DeploymentPipelineBuilder,
     DeploymentPipelineExecutor,
-    PipelineContext,
 )
-from .processors import AgentProcessor
-from .results import DeploymentMetrics, DeploymentResultBuilder
+from .results import DeploymentResultBuilder
 
 # Import refactored components
 from .strategies import DeploymentContext, DeploymentStrategySelector
@@ -69,7 +66,7 @@ class RefactoredAgentDeploymentService(AgentDeploymentInterface):
             self.pipeline_builder, self.pipeline_executor
         )
 
-        self.logger.info(f"Refactored deployment service initialized")
+        self.logger.info("Refactored deployment service initialized")
         self.logger.info(f"Templates directory: {self.templates_dir}")
         self.logger.info(f"Base agent path: {self.base_agent_path}")
         self.logger.info(f"Working directory: {self.working_directory}")
@@ -167,7 +164,7 @@ class RefactoredAgentDeploymentService(AgentDeploymentInterface):
 
         except Exception as e:
             self.logger.error(f"Agent validation failed: {e}", exc_info=True)
-            return False, [f"Validation error: {str(e)}"]
+            return False, [f"Validation error: {e!s}"]
 
     def clean_deployment(self, preserve_user_agents: bool = True) -> bool:
         """Clean up deployed agents.
@@ -236,7 +233,7 @@ class RefactoredAgentDeploymentService(AgentDeploymentInterface):
             available_executors = self.deployment_facade.get_available_executors()
 
             # Build status information
-            status = {
+            return {
                 "service_version": "refactored-1.0.0",
                 "status": "ready",
                 "templates_dir": str(self.templates_dir),
@@ -254,8 +251,6 @@ class RefactoredAgentDeploymentService(AgentDeploymentInterface):
                 },
             }
 
-            return status
-
         except Exception as e:
             self.logger.error(f"Failed to get deployment status: {e}", exc_info=True)
             return {
@@ -292,7 +287,7 @@ class RefactoredAgentDeploymentService(AgentDeploymentInterface):
             strategy = self.strategy_selector.select_strategy(deployment_context)
 
             # Use facade with synchronous deployment for reliability
-            results = self.deployment_facade.deploy_agents(
+            return self.deployment_facade.deploy_agents(
                 templates_dir=self.templates_dir,
                 base_agent_path=self.base_agent_path,
                 working_directory=self.working_directory,
@@ -303,8 +298,6 @@ class RefactoredAgentDeploymentService(AgentDeploymentInterface):
                 use_async=False,  # Use synchronous deployment to ensure agents are ready when Claude Code launches
             )
 
-            return results
-
         except Exception as e:
             self.logger.error(f"Async deployment failed: {e}", exc_info=True)
             return {

claude_mpm/services/agents/deployment/results/__init__.py

@@ -8,6 +8,6 @@ from .deployment_metrics import DeploymentMetrics
 from .deployment_result_builder import DeploymentResultBuilder
 
 __all__ = [
-    "DeploymentResultBuilder",
     "DeploymentMetrics",
+    "DeploymentResultBuilder",
 ]

claude_mpm/services/agents/deployment/results/deployment_result_builder.py

@@ -1,7 +1,7 @@
 """Deployment result builder for creating structured deployment results."""
 
 from pathlib import Path
-from typing import Any, Dict, List, Optional
+from typing import Any, Dict, Optional
 
 from claude_mpm.core.logger import get_logger
 

claude_mpm/services/agents/deployment/strategies/__init__.py

@@ -18,8 +18,8 @@ from .user_strategy import UserAgentDeploymentStrategy
 __all__ = [
     "BaseDeploymentStrategy",
     "DeploymentContext",
-    "SystemAgentDeploymentStrategy",
+    "DeploymentStrategySelector",
     "ProjectAgentDeploymentStrategy",
+    "SystemAgentDeploymentStrategy",
     "UserAgentDeploymentStrategy",
-    "DeploymentStrategySelector",
 ]

claude_mpm/services/agents/deployment/strategies/base_strategy.py

@@ -3,7 +3,7 @@
 from abc import ABC, abstractmethod
 from dataclasses import dataclass
 from pathlib import Path
-from typing import Any, Dict, List, Optional
+from typing import List, Optional
 
 from claude_mpm.core.config import Config
 
@@ -48,7 +48,6 @@ class BaseDeploymentStrategy(ABC):
         Returns:
             True if this strategy should handle the deployment
         """
-        pass
 
     @abstractmethod
     def determine_target_directory(self, context: DeploymentContext) -> Path:
@@ -60,7 +59,6 @@ class BaseDeploymentStrategy(ABC):
         Returns:
             Path where agents should be deployed
         """
-        pass
 
     @abstractmethod
     def get_templates_directory(self, context: DeploymentContext) -> Path:
@@ -72,7 +70,6 @@ class BaseDeploymentStrategy(ABC):
         Returns:
             Path to the templates directory
         """
-        pass
 
     @abstractmethod
     def get_excluded_agents(self, context: DeploymentContext) -> List[str]:
@@ -84,7 +81,6 @@ class BaseDeploymentStrategy(ABC):
         Returns:
             List of agent names to exclude from deployment
         """
-        pass
 
     @abstractmethod
     def should_deploy_system_instructions(self, context: DeploymentContext) -> bool:
@@ -96,7 +92,6 @@ class BaseDeploymentStrategy(ABC):
         Returns:
             True if system instructions should be deployed
         """
-        pass
 
     @abstractmethod
     def get_deployment_priority(self) -> int:
@@ -108,7 +103,6 @@ class BaseDeploymentStrategy(ABC):
         Returns:
             Priority number (lower = higher priority)
         """
-        pass
 
     def __str__(self) -> str:
         """String representation of the strategy."""

claude_mpm/services/agents/deployment/strategies/project_strategy.py

@@ -58,10 +58,7 @@ class ProjectAgentDeploymentStrategy(BaseDeploymentStrategy):
                 pass
 
         # Check if deployment_mode is "project" - this should be sufficient
-        if context.deployment_mode == "project":
-            return True
-
-        return False
+        return context.deployment_mode == "project"
 
     def determine_target_directory(self, context: DeploymentContext) -> Path:
         """Determine target directory for project agents.

claude_mpm/services/agents/deployment/strategies/system_strategy.py

@@ -76,9 +76,8 @@ class SystemAgentDeploymentStrategy(BaseDeploymentStrategy):
         # Always deploy to project directory
         if context.working_directory:
             return context.working_directory / ".claude" / "agents"
-        else:
-            # Fallback to current working directory if not specified
-            return Path.cwd() / ".claude" / "agents"
+        # Fallback to current working directory if not specified
+        return Path.cwd() / ".claude" / "agents"
 
     def get_templates_directory(self, context: DeploymentContext) -> Path:
         """Get templates directory for system agents.

claude_mpm/services/agents/deployment/strategies/user_strategy.py

@@ -49,10 +49,7 @@ class UserAgentDeploymentStrategy(BaseDeploymentStrategy):
         # Check for user-specific environment variable
         import os
 
-        if "CLAUDE_MPM_USER_PWD" in os.environ:
-            return True
-
-        return False
+        return "CLAUDE_MPM_USER_PWD" in os.environ
 
     def determine_target_directory(self, context: DeploymentContext) -> Path:
         """Determine target directory for user agents.
@@ -70,9 +67,8 @@ class UserAgentDeploymentStrategy(BaseDeploymentStrategy):
         # Always deploy to project directory
         if context.working_directory:
             return context.working_directory / ".claude" / "agents"
-        else:
-            # Fallback to current working directory if not specified
-            return Path.cwd() / ".claude" / "agents"
+        # Fallback to current working directory if not specified
+        return Path.cwd() / ".claude" / "agents"
 
     def get_templates_directory(self, context: DeploymentContext) -> Path:
         """Get templates directory for user agents.

claude_mpm/services/agents/deployment/validation/__init__.py

@@ -11,8 +11,8 @@ from .template_validator import TemplateValidator
 from .validation_result import ValidationResult, ValidationSeverity
 
 __all__ = [
-    "DeploymentValidator",
     "AgentValidator",
+    "DeploymentValidator",
     "TemplateValidator",
     "ValidationResult",
     "ValidationSeverity",

claude_mpm/services/agents/deployment/validation/agent_validator.py

@@ -2,7 +2,7 @@
 
 import re
 from pathlib import Path
-from typing import Any, Dict, Optional
+from typing import Any, Dict
 
 import yaml
 

claude_mpm/services/agents/deployment/validation/template_validator.py

@@ -2,11 +2,11 @@
 
 import json
 from pathlib import Path
-from typing import Any, Dict, List
+from typing import Any, Dict
 
 from claude_mpm.core.logger import get_logger
 
-from .validation_result import ValidationResult, ValidationSeverity
+from .validation_result import ValidationResult
 
 
 class TemplateValidator:

claude_mpm/services/agents/deployment/validation/validation_result.py

@@ -175,12 +175,11 @@ class ValidationResult:
         Returns:
             New ValidationResult with merged issues
         """
-        merged = ValidationResult(
+        return ValidationResult(
             is_valid=self.is_valid and other.is_valid,
             issues=self.issues + other.issues,
             metadata={**self.metadata, **other.metadata},
         )
-        return merged
 
     def to_dict(self) -> Dict[str, Any]:
         """Convert validation result to dictionary.
@@ -208,10 +207,7 @@ class ValidationResult:
 
     def __str__(self) -> str:
         """String representation of the validation result."""
-        if self.is_valid:
-            status = "VALID"
-        else:
-            status = "INVALID"
+        status = "VALID" if self.is_valid else "INVALID"
 
         summary = f"Validation {status}"
 

claude_mpm/services/agents/loading/__init__.py

@@ -5,7 +5,7 @@ from .base_agent_manager import BaseAgentManager
 from .framework_agent_loader import FrameworkAgentLoader
 
 __all__ = [
-    "FrameworkAgentLoader",
     "AgentProfileLoader",
     "BaseAgentManager",
+    "FrameworkAgentLoader",
 ]