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

claude_mpm/services/unified/unified_config.py

@@ -22,7 +22,6 @@ Features:
 - Version control and rollback capabilities
 """
 
-import asyncio
 import json
 from pathlib import Path
 from typing import Any, Dict, List, Optional, Union
@@ -121,7 +120,7 @@ class UnifiedConfigManager(IConfigurationService, 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:
@@ -266,16 +265,18 @@ class UnifiedConfigManager(IConfigurationService, IUnifiedService):
                 success=True,
                 config=config_with_defaults,
                 validation_errors=[],
-                applied_defaults=self._get_applied_defaults(config, config_with_defaults),
+                applied_defaults=self._get_applied_defaults(
+                    config, config_with_defaults
+                ),
                 source=str(source),
             )
 
         except Exception as e:
-            self._logger.error(f"Failed to load configuration: {str(e)}")
+            self._logger.error(f"Failed to load configuration: {e!s}")
             self._metrics["validation_errors"] += 1
             return ConfigurationResult(
                 success=False,
-                validation_errors=[f"Load failed: {str(e)}"],
+                validation_errors=[f"Load failed: {e!s}"],
             )
 
     def save_config(
@@ -352,10 +353,10 @@ class UnifiedConfigManager(IConfigurationService, IUnifiedService):
             )
 
         except Exception as e:
-            self._logger.error(f"Failed to save configuration: {str(e)}")
+            self._logger.error(f"Failed to save configuration: {e!s}")
             return ConfigurationResult(
                 success=False,
-                validation_errors=[f"Save failed: {str(e)}"],
+                validation_errors=[f"Save failed: {e!s}"],
             )
 
     def validate_config(self, config: Dict[str, Any]) -> List[str]:
@@ -414,13 +415,12 @@ class UnifiedConfigManager(IConfigurationService, IUnifiedService):
         # Implement merge based on strategy
         if strategy == "deep":
             return self._deep_merge(*configs)
-        elif strategy == "shallow":
+        if strategy == "shallow":
             return self._shallow_merge(*configs)
-        elif strategy == "override":
+        if strategy == "override":
             return self._override_merge(*configs)
-        else:
-            self._logger.warning(f"Unknown merge strategy: {strategy}, using deep")
-            return self._deep_merge(*configs)
+        self._logger.warning(f"Unknown merge strategy: {strategy}, using deep")
+        return self._deep_merge(*configs)
 
     def get_config_value(
         self, key: str, default: Any = None, config: Optional[Dict[str, Any]] = None
@@ -595,9 +595,9 @@ class UnifiedConfigManager(IConfigurationService, IUnifiedService):
             # Infer from filename
             if "project" in path.name.lower():
                 return "project"
-            elif "agent" in path.name.lower():
+            if "agent" in path.name.lower():
                 return "agent"
-            elif "env" in path.name.lower():
+            if "env" in path.name.lower():
                 return "environment"
 
         return "generic"
@@ -639,9 +639,7 @@ class UnifiedConfigManager(IConfigurationService, IUnifiedService):
                 applied[key] = value
         return applied
 
-    def _add_to_version_history(
-        self, config_id: str, config: Dict[str, Any]
-    ) -> None:
+    def _add_to_version_history(self, config_id: str, config: Dict[str, Any]) -> None:
         """Add configuration to version history."""
         if config_id not in self._config_versions:
             self._config_versions[config_id] = []
@@ -663,7 +661,11 @@ class UnifiedConfigManager(IConfigurationService, IUnifiedService):
 
         for config in configs:
             for key, value in config.items():
-                if key in result and isinstance(result[key], dict) and isinstance(value, dict):
+                if (
+                    key in result
+                    and isinstance(result[key], dict)
+                    and isinstance(value, dict)
+                ):
                     result[key] = self._deep_merge(result[key], value)
                 else:
                     result[key] = value
@@ -685,4 +687,4 @@ class UnifiedConfigManager(IConfigurationService, IUnifiedService):
         """Get current timestamp."""
         from datetime import datetime
 
-        return datetime.now().isoformat()
+        return datetime.now().isoformat()

claude_mpm/services/unified/unified_deployment.py

@@ -21,7 +21,6 @@ Features:
 - Metrics and monitoring integration
 """
 
-import asyncio
 from pathlib import Path
 from typing import Any, Dict, List, Optional, Union
 
@@ -114,7 +113,7 @@ class UnifiedDeploymentService(IDeploymentService, 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:
@@ -215,7 +214,9 @@ class UnifiedDeploymentService(IDeploymentService, IUnifiedService):
         strategy = self._registry.select_strategy(DeploymentStrategy, context)
 
         if not strategy:
-            errors.append(f"No strategy available for deployment type: {deployment_type}")
+            errors.append(
+                f"No strategy available for deployment type: {deployment_type}"
+            )
         else:
             # Delegate validation to strategy
             strategy_errors = strategy.validate_input(config)
@@ -314,20 +315,19 @@ class UnifiedDeploymentService(IDeploymentService, IUnifiedService):
                     metadata=result,
                     rollback_info=rollback_info,
                 )
-            else:
-                self._metrics["failed_deployments"] += 1
-                return DeploymentResult(
-                    success=False,
-                    message=result.get("error", "Deployment failed"),
-                    metadata=result,
-                )
+            self._metrics["failed_deployments"] += 1
+            return DeploymentResult(
+                success=False,
+                message=result.get("error", "Deployment failed"),
+                metadata=result,
+            )
 
         except Exception as e:
-            self._logger.error(f"Deployment error: {str(e)}")
+            self._logger.error(f"Deployment error: {e!s}")
             self._metrics["failed_deployments"] += 1
             return DeploymentResult(
                 success=False,
-                message=f"Deployment failed: {str(e)}",
+                message=f"Deployment failed: {e!s}",
             )
 
     def rollback(
@@ -355,9 +355,7 @@ class UnifiedDeploymentService(IDeploymentService, IUnifiedService):
         try:
             # Get the strategy used for deployment
             strategy_name = deployment["strategy"]
-            strategy = self._registry.get_strategy(
-                DeploymentStrategy, strategy_name
-            )
+            strategy = self._registry.get_strategy(DeploymentStrategy, strategy_name)
 
             if not strategy:
                 return DeploymentResult(
@@ -380,17 +378,16 @@ class UnifiedDeploymentService(IDeploymentService, IUnifiedService):
                     success=True,
                     message=f"Successfully rolled back deployment {deployment_id}",
                 )
-            else:
-                return DeploymentResult(
-                    success=False,
-                    message="Rollback failed",
-                )
+            return DeploymentResult(
+                success=False,
+                message="Rollback failed",
+            )
 
         except Exception as e:
-            self._logger.error(f"Rollback error: {str(e)}")
+            self._logger.error(f"Rollback error: {e!s}")
             return DeploymentResult(
                 success=False,
-                message=f"Rollback failed: {str(e)}",
+                message=f"Rollback failed: {e!s}",
             )
 
     def list_deployments(
@@ -409,9 +406,7 @@ class UnifiedDeploymentService(IDeploymentService, IUnifiedService):
 
         if target:
             target_str = str(target)
-            deployments = [
-                d for d in deployments if d["target"] == target_str
-            ]
+            deployments = [d for d in deployments if d["target"] == target_str]
 
         return deployments
 
@@ -467,4 +462,4 @@ class UnifiedDeploymentService(IDeploymentService, IUnifiedService):
         """Get current timestamp."""
         from datetime import datetime
 
-        return datetime.now().isoformat()
+        return datetime.now().isoformat()

claude_mpm/storage/state_storage.py

@@ -39,6 +39,7 @@ class StateStorage:
 
         # Logging
 from claude_mpm.core.logging_utils import get_logger
+
 logger = get_logger(__name__)
 
         # File locking support (Unix-like systems)

claude_mpm/utils/agent_dependency_loader.py

@@ -149,7 +149,9 @@ class AgentDependencyLoader:
 
             # Check if this is an optional package that already failed
             if package_name in self.optional_failed:
-                logger.debug(f"Skipping optional package {package_name} (previously failed)")
+                logger.debug(
+                    f"Skipping optional package {package_name} (previously failed)"
+                )
                 return True, "optional-skipped"
 
             # Try to import and check version
@@ -174,14 +176,18 @@ class AgentDependencyLoader:
                         for alternative in self.OPTIONAL_DB_PACKAGES[package_name]:
                             try:
                                 alt_version = importlib.metadata.version(alternative)
-                                logger.info(f"Using {alternative} as alternative to {package_name}")
+                                logger.info(
+                                    f"Using {alternative} as alternative to {package_name}"
+                                )
                                 self.checked_packages.add(package_name)
                                 return True, f"{alternative}:{alt_version}"
                             except importlib.metadata.PackageNotFoundError:
                                 continue
                         # If no alternatives work, mark as optional failure
                         self.optional_failed[package_name] = "No alternatives available"
-                        logger.warning(f"Optional package {package_name} not found, marking as optional")
+                        logger.warning(
+                            f"Optional package {package_name} not found, marking as optional"
+                        )
                         return True, "optional-not-found"
                     return False, None
 
@@ -202,14 +208,20 @@ class AgentDependencyLoader:
                     if package_name in self.OPTIONAL_DB_PACKAGES:
                         for alternative in self.OPTIONAL_DB_PACKAGES[package_name]:
                             try:
-                                alt_version = pkg_resources.get_distribution(alternative).version
-                                logger.info(f"Using {alternative} as alternative to {package_name}")
+                                alt_version = pkg_resources.get_distribution(
+                                    alternative
+                                ).version
+                                logger.info(
+                                    f"Using {alternative} as alternative to {package_name}"
+                                )
                                 self.checked_packages.add(package_name)
                                 return True, f"{alternative}:{alt_version}"
                             except pkg_resources.DistributionNotFound:
                                 continue
                         self.optional_failed[package_name] = "No alternatives available"
-                        logger.warning(f"Optional package {package_name} not found, marking as optional")
+                        logger.warning(
+                            f"Optional package {package_name} not found, marking as optional"
+                        )
                         return True, "optional-not-found"
                     return False, None
 

claude_mpm/utils/common.py

@@ -46,7 +46,7 @@ def load_json_safe(
     file_path = Path(file_path)
 
     try:
-        with open(file_path, "r", encoding=encoding) as f:
+        with open(file_path, encoding=encoding) as f:
             return json.load(f)
     except FileNotFoundError:
         logger.debug(f"JSON file not found: {file_path}")
@@ -112,7 +112,7 @@ def load_yaml_safe(
     file_path = Path(file_path)
 
     try:
-        with open(file_path, "r", encoding=encoding) as f:
+        with open(file_path, encoding=encoding) as f:
             return yaml.safe_load(f) or default or {}
     except FileNotFoundError:
         logger.debug(f"YAML file not found: {file_path}")
@@ -191,11 +191,10 @@ def ensure_path_exists(
                 path.parent.mkdir(parents=True, exist_ok=True)
             # Don't create the file itself, just ensure parent exists
             return path.parent.exists()
-        else:
-            if create_parents:
-                path.mkdir(parents=True, exist_ok=True)
-                return True
-            return False
+        if create_parents:
+            path.mkdir(parents=True, exist_ok=True)
+            return True
+        return False
     except Exception as e:
         logger.error(f"Error ensuring path exists {path}: {e}")
         return False
@@ -304,8 +303,7 @@ def find_files(
 
         if recursive:
             return list(directory.rglob(pattern))
-        else:
-            return list(directory.glob(pattern))
+        return list(directory.glob(pattern))
     except Exception as e:
         logger.error(f"Error finding files in {directory}: {e}")
         return []
@@ -357,7 +355,7 @@ def run_command_safe(
             env=env,
         )
         return result
-    except subprocess.TimeoutExpired as e:
+    except subprocess.TimeoutExpired:
         logger.error(f"Command timed out: {command}")
         raise
     except subprocess.CalledProcessError as e:
@@ -436,7 +434,9 @@ def get_env_int(key: str, default: int = 0) -> int:
         return default
 
 
-def get_env_list(key: str, separator: str = ",", default: Optional[List[str]] = None) -> List[str]:
+def get_env_list(
+    key: str, separator: str = ",", default: Optional[List[str]] = None
+) -> List[str]:
     """
     Get list from environment variable.
 
@@ -476,6 +476,7 @@ def safe_import(module_name: str, fallback: Any = None) -> Any:
     """
     try:
         import importlib
+
         return importlib.import_module(module_name)
     except ImportError as e:
         logger.debug(f"Could not import {module_name}: {e}")
@@ -524,6 +525,7 @@ def deprecated(replacement: str = None):
     Returns:
         Decorated function
     """
+
     def decorator(func):
         def wrapper(*args, **kwargs):
             import warnings
@@ -539,4 +541,4 @@ def deprecated(replacement: str = None):
         wrapper.__doc__ = func.__doc__
         return wrapper
 
-    return decorator
+    return decorator

claude_mpm/utils/database_connector.py

@@ -10,7 +10,6 @@ DESIGN DECISION: We prioritize pure Python implementations over native ones
 for better cross-platform compatibility, even if they might be slightly slower.
 """
 
-import logging
 from typing import Any, Dict, Optional, Tuple
 
 from ..core.logger import get_logger
@@ -92,12 +91,14 @@ class DatabaseConnector:
         """
         try:
             import importlib
+
             return importlib.import_module(module_name)
         except ImportError:
             return None
 
-    def get_mysql_connection_string(self, host: str, database: str, user: str,
-                                   password: str, port: int = 3306) -> Optional[str]:
+    def get_mysql_connection_string(
+        self, host: str, database: str, user: str, password: str, port: int = 3306
+    ) -> Optional[str]:
         """
         Get a SQLAlchemy connection string for MySQL with automatic driver selection.
 
@@ -125,15 +126,16 @@ class DatabaseConnector:
         # Format connection string based on driver
         if package_name == "pymysql":
             return f"mysql+pymysql://{user}:{password}@{host}:{port}/{database}"
-        elif package_name == "mysqlclient":
+        if package_name == "mysqlclient":
             return f"mysql+mysqldb://{user}:{password}@{host}:{port}/{database}"
-        elif package_name == "mysql-connector-python":
+        if package_name == "mysql-connector-python":
             return f"mysql+mysqlconnector://{user}:{password}@{host}:{port}/{database}"
 
         return None
 
-    def get_postgresql_connection_string(self, host: str, database: str, user: str,
-                                        password: str, port: int = 5432) -> Optional[str]:
+    def get_postgresql_connection_string(
+        self, host: str, database: str, user: str, password: str, port: int = 5432
+    ) -> Optional[str]:
         """
         Get a SQLAlchemy connection string for PostgreSQL with automatic driver selection.
 
@@ -158,13 +160,14 @@ class DatabaseConnector:
         # Format connection string based on driver
         if package_name in ["psycopg2-binary", "psycopg2"]:
             return f"postgresql+psycopg2://{user}:{password}@{host}:{port}/{database}"
-        elif package_name == "pg8000":
+        if package_name == "pg8000":
             return f"postgresql+pg8000://{user}:{password}@{host}:{port}/{database}"
 
         return None
 
-    def get_oracle_connection_string(self, host: str, database: str, user: str,
-                                    password: str, port: int = 1521) -> Optional[str]:
+    def get_oracle_connection_string(
+        self, host: str, database: str, user: str, password: str, port: int = 1521
+    ) -> Optional[str]:
         """
         Get a SQLAlchemy connection string for Oracle with automatic driver selection.
 
@@ -189,7 +192,7 @@ class DatabaseConnector:
         # Format connection string based on driver
         if package_name == "cx_Oracle":
             return f"oracle+cx_oracle://{user}:{password}@{host}:{port}/{database}"
-        elif package_name == "oracledb":
+        if package_name == "oracledb":
             return f"oracle+oracledb://{user}:{password}@{host}:{port}/{database}"
 
         return None
@@ -295,4 +298,4 @@ def test_database_connectivity() -> None:
 
 if __name__ == "__main__":
     # Run connectivity test when module is executed directly
-    test_database_connectivity()
+    test_database_connectivity()

claude_mpm/utils/error_handler.py

@@ -11,6 +11,7 @@ from functools import wraps
 from typing import Any, Callable, Dict, List, Optional, Type
 
 from claude_mpm.core.logging_utils import get_logger
+
 logger = get_logger(__name__)
 
 

claude_mpm/utils/log_cleanup.py

@@ -13,6 +13,7 @@ from pathlib import Path
 from typing import Dict, Optional, Tuple
 
 from claude_mpm.core.logging_utils import get_logger
+
 logger = get_logger(__name__)
 
 

claude_mpm/utils/path_operations.py

@@ -11,6 +11,7 @@ from pathlib import Path
 from typing import Callable, List, Optional, Union
 
 from claude_mpm.core.logging_utils import get_logger
+
 logger = get_logger(__name__)
 
 

claude_mpm/utils/session_logging.py

@@ -9,9 +9,9 @@ Convenience functions for session-based response logging.
 import os
 from typing import Any, Dict, Optional
 
+from claude_mpm.core.logging_utils import get_logger
 from claude_mpm.services.claude_session_logger import get_session_logger
 
-from claude_mpm.core.logging_utils import get_logger
 logger = get_logger(__name__)
 
 

claude_mpm/utils/subprocess_utils.py

@@ -16,6 +16,7 @@ from typing import Any, Dict, List, Optional
 import psutil
 
 from claude_mpm.core.logging_utils import get_logger
+
 logger = get_logger(__name__)
 
 

claude_mpm/validation/agent_validator.py

@@ -34,8 +34,8 @@ from claude_mpm.core.constants import (
     SystemLimits,
     TimeoutConfig,
 )
-
 from claude_mpm.core.logging_utils import get_logger
+
 logger = get_logger(__name__)
 
 

{claude_mpm-4.4.3.dist-info → claude_mpm-4.4.5.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: claude-mpm
-Version: 4.4.3
+Version: 4.4.5
 Summary: Claude Multi-Agent Project Manager - Orchestrate Claude with agent delegation and ticket tracking
 Author-email: Bob Matsuoka <bob@matsuoka.com>
 Maintainer: Claude MPM Team
@@ -41,11 +41,6 @@ Requires-Dist: python-frontmatter>=1.0.0
 Requires-Dist: mistune>=3.0.0
 Requires-Dist: tree-sitter>=0.21.0
 Requires-Dist: ijson>=3.2.0
-Requires-Dist: kuzu-memory>=1.1.1
-Requires-Dist: mcp>=0.1.0
-Requires-Dist: mcp-vector-search>=0.1.0
-Requires-Dist: mcp-browser>=0.1.0
-Requires-Dist: mcp-ticketer>=0.1.0
 Requires-Dist: toml>=0.10.2
 Requires-Dist: packaging>=21.0
 Requires-Dist: pydantic>=2.0.0
@@ -54,6 +49,12 @@ Requires-Dist: rich>=13.0.0
 Requires-Dist: pyee>=13.0.0
 Requires-Dist: importlib-resources>=5.0; python_version < "3.9"
 Requires-Dist: pathspec>=0.11.0
+Provides-Extra: mcp
+Requires-Dist: mcp>=0.1.0; extra == "mcp"
+Requires-Dist: mcp-vector-search>=0.1.0; extra == "mcp"
+Requires-Dist: mcp-browser>=0.1.0; extra == "mcp"
+Requires-Dist: mcp-ticketer>=0.1.0; extra == "mcp"
+Requires-Dist: kuzu-memory>=1.1.1; extra == "mcp"
 Provides-Extra: dev
 Requires-Dist: pytest>=7.0; extra == "dev"
 Requires-Dist: pytest-asyncio; extra == "dev"
@@ -157,7 +158,7 @@ A powerful orchestration framework for Claude Code that enables multi-agent work
 - 🧠 **Persistent Knowledge System**: Project-specific kuzu-memory integration for intelligent context retention
 - 🔄 **Session Management**: Resume previous sessions with `--resume`
 - 📊 **Real-Time Monitoring**: Live dashboard with `--monitor` flag
-- 🔌 **Auto-Installing MCP Services**: mcp-vector-search and kuzu-memory automatically installed via pipx
+- 🔌 **Optional MCP Services**: mcp-vector-search and kuzu-memory with automatic fallback installation
 - 📁 **Multi-Project Support**: Per-session working directories with persistent knowledge graphs
 - 🔍 **Git Integration**: View diffs and track changes across projects
 - 🎯 **Smart Task Orchestration**: PM agent intelligently routes work to specialists
@@ -167,22 +168,32 @@ A powerful orchestration framework for Claude Code that enables multi-agent work
 ## Quick Installation
 
 ```bash
+# Basic installation
 pip install claude-mpm
+
+# Install with optional MCP services (recommended)
+pip install "claude-mpm[mcp]"
 ```
 
 Or with pipx (recommended for isolated installation):
 ```bash
-# Install with monitor support (recommended)
-pipx install "claude-mpm[monitor]"
-
-# Basic installation without monitor
+# Basic installation
 pipx install claude-mpm
 
+# Install with optional MCP services (recommended)
+pipx install "claude-mpm[mcp]"
+
+# Install with all features
+pipx install "claude-mpm[mcp,monitor]"
+
 # Configure MCP for pipx users:
 claude-mpm mcp-pipx-config
 ```
 
-**💡 Pipx Tip**: Use `"claude-mpm[monitor]"` to get full monitoring dashboard functionality! The `[monitor]` optional dependency includes Socket.IO and async web server components needed for real-time agent monitoring.
+**💡 Optional Dependencies**:
+- `[mcp]` - Include MCP services (mcp-vector-search, kuzu-memory) as dependencies
+- `[monitor]` - Full monitoring dashboard with Socket.IO and async web server components
+- Without optional dependencies, services auto-install on first use via pipx
 
 **🎉 Pipx Support Now Fully Functional!** Recent improvements ensure complete compatibility:
 - ✅ Socket.IO daemon script path resolution (fixed)
@@ -204,9 +215,18 @@ claude-mpm run --monitor
 # Use MCP Gateway for external tool integration
 claude-mpm mcp
 
-# Run health diagnostics  
+# Run comprehensive health diagnostics
 claude-mpm doctor
 
+# Generate detailed diagnostic report with MCP service analysis
+claude-mpm doctor --verbose --output-file doctor-report.md
+
+# Run specific diagnostic checks including MCP services
+claude-mpm doctor --checks installation configuration agents mcp
+
+# Check MCP service status specifically
+claude-mpm doctor --checks mcp --verbose
+
 # Manage memory for large conversation histories
 claude-mpm cleanup-memory
 ```
@@ -219,7 +239,7 @@ See [QUICKSTART.md](QUICKSTART.md) for complete usage examples.
 Following Phase 3 architectural simplification in v4.4.1, Claude MPM features:
 
 - **Streamlined Rich Interface**: Removed complex TUI system (~2,500 lines) for cleaner user experience
-- **Auto-Installing MCP Services**: mcp-vector-search and kuzu-memory install automatically via pipx
+- **Optional MCP Services**: mcp-vector-search and kuzu-memory with automatic fallback installation
 - **Persistent Knowledge System**: Project-specific kuzu-memory databases with intelligent prompt enrichment
 - **Service-Oriented Architecture**: Simplified five specialized service domains
 - **Interface-Based Contracts**: All services implement explicit interfaces
@@ -316,7 +336,7 @@ See [docs/MEMORY.md](docs/MEMORY.md) and [docs/developer/11-dashboard/README.md]
 #### 🚀 For Operations
 - **[🚀 Deployment](docs/DEPLOYMENT.md)** - Release management & versioning
 - **[📊 Monitoring](docs/MONITOR.md)** - Real-time dashboard & metrics
-- **[🐛 Troubleshooting](docs/TROUBLESHOOTING.md)** - Diagnostic & problem resolution
+- **[🐛 Troubleshooting](docs/TROUBLESHOOTING.md)** - Enhanced `doctor` command with detailed reports and auto-fix capabilities
 
 ### 🎯 Documentation Features
 - **Single Entry Point**: [docs/README.md](docs/README.md) is your navigation hub