claude-mpm 4.3.20__py3-none-any.whl → 4.3.22__py3-none-any.whl

claude_mpm/utils/database_connector.py

@@ -0,0 +1,298 @@
+"""
+Database Connector with Fallback Support
+=========================================
+
+WHY: Database connectivity packages often have compilation requirements that
+can fail on certain systems. This module provides intelligent fallback to
+pure Python alternatives when native packages are unavailable.
+
+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
+
+logger = get_logger(__name__)
+
+
+class DatabaseConnector:
+    """
+    Manages database connections with automatic fallback to alternative drivers.
+
+    WHY: Provides a unified interface for database connections that automatically
+    handles missing or failed driver installations by falling back to alternatives.
+    """
+
+    # Database drivers in order of preference (first is preferred)
+    MYSQL_DRIVERS = [
+        ("pymysql", "pymysql"),  # Pure Python, no compilation required
+        ("mysqlclient", "MySQLdb"),  # Faster but requires MySQL dev headers
+        ("mysql-connector-python", "mysql.connector"),  # Oracle's pure Python driver
+    ]
+
+    POSTGRESQL_DRIVERS = [
+        ("psycopg2-binary", "psycopg2"),  # Binary wheel, no compilation
+        ("psycopg2", "psycopg2"),  # Requires PostgreSQL dev headers
+        ("pg8000", "pg8000"),  # Pure Python alternative
+    ]
+
+    ORACLE_DRIVERS = [
+        ("cx_Oracle", "cx_Oracle"),  # Requires Oracle client libraries
+        ("oracledb", "oracledb"),  # Pure Python Oracle driver (newer)
+    ]
+
+    def __init__(self):
+        """Initialize the database connector."""
+        self.available_drivers: Dict[str, Tuple[str, Any]] = {}
+        self._scan_available_drivers()
+
+    def _scan_available_drivers(self) -> None:
+        """
+        Scan for available database drivers and cache them.
+
+        WHY: Pre-scanning allows us to know what's available without
+        repeatedly trying imports, improving performance.
+        """
+        # Check MySQL drivers
+        for package_name, import_name in self.MYSQL_DRIVERS:
+            driver = self._try_import(import_name)
+            if driver:
+                if "mysql" not in self.available_drivers:
+                    self.available_drivers["mysql"] = (package_name, driver)
+                    logger.info(f"MySQL driver available: {package_name}")
+
+        # Check PostgreSQL drivers
+        for package_name, import_name in self.POSTGRESQL_DRIVERS:
+            driver = self._try_import(import_name)
+            if driver:
+                if "postgresql" not in self.available_drivers:
+                    self.available_drivers["postgresql"] = (package_name, driver)
+                    logger.info(f"PostgreSQL driver available: {package_name}")
+
+        # Check Oracle drivers
+        for package_name, import_name in self.ORACLE_DRIVERS:
+            driver = self._try_import(import_name)
+            if driver:
+                if "oracle" not in self.available_drivers:
+                    self.available_drivers["oracle"] = (package_name, driver)
+                    logger.info(f"Oracle driver available: {package_name}")
+
+    def _try_import(self, module_name: str) -> Optional[Any]:
+        """
+        Try to import a module and return it if successful.
+
+        Args:
+            module_name: Name of the module to import
+
+        Returns:
+            The imported module or None if import fails
+        """
+        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]:
+        """
+        Get a SQLAlchemy connection string for MySQL with automatic driver selection.
+
+        WHY: SQLAlchemy needs different connection string formats depending on the driver.
+        This method automatically selects the best available driver.
+
+        Args:
+            host: Database host
+            database: Database name
+            user: Username
+            password: Password
+            port: Port number (default 3306)
+
+        Returns:
+            SQLAlchemy connection string or None if no driver available
+        """
+        if "mysql" not in self.available_drivers:
+            logger.error(
+                "No MySQL driver available. Install one of: pymysql, mysqlclient, or mysql-connector-python"
+            )
+            return None
+
+        package_name, _ = self.available_drivers["mysql"]
+
+        # Format connection string based on driver
+        if package_name == "pymysql":
+            return f"mysql+pymysql://{user}:{password}@{host}:{port}/{database}"
+        elif package_name == "mysqlclient":
+            return f"mysql+mysqldb://{user}:{password}@{host}:{port}/{database}"
+        elif 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]:
+        """
+        Get a SQLAlchemy connection string for PostgreSQL with automatic driver selection.
+
+        Args:
+            host: Database host
+            database: Database name
+            user: Username
+            password: Password
+            port: Port number (default 5432)
+
+        Returns:
+            SQLAlchemy connection string or None if no driver available
+        """
+        if "postgresql" not in self.available_drivers:
+            logger.error(
+                "No PostgreSQL driver available. Install one of: psycopg2-binary, psycopg2, or pg8000"
+            )
+            return None
+
+        package_name, _ = self.available_drivers["postgresql"]
+
+        # 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":
+            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]:
+        """
+        Get a SQLAlchemy connection string for Oracle with automatic driver selection.
+
+        Args:
+            host: Database host
+            database: Database name/SID
+            user: Username
+            password: Password
+            port: Port number (default 1521)
+
+        Returns:
+            SQLAlchemy connection string or None if no driver available
+        """
+        if "oracle" not in self.available_drivers:
+            logger.error(
+                "No Oracle driver available. Install one of: cx_Oracle or oracledb"
+            )
+            return None
+
+        package_name, _ = self.available_drivers["oracle"]
+
+        # 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":
+            return f"oracle+oracledb://{user}:{password}@{host}:{port}/{database}"
+
+        return None
+
+    def get_available_drivers(self) -> Dict[str, str]:
+        """
+        Get a summary of available database drivers.
+
+        Returns:
+            Dictionary mapping database type to driver package name
+        """
+        return {
+            db_type: driver_info[0]
+            for db_type, driver_info in self.available_drivers.items()
+        }
+
+    def suggest_missing_drivers(self) -> Dict[str, str]:
+        """
+        Suggest drivers to install for databases without drivers.
+
+        Returns:
+            Dictionary mapping database type to recommended package
+        """
+        suggestions = {}
+
+        if "mysql" not in self.available_drivers:
+            suggestions["mysql"] = "pymysql"  # Pure Python, always works
+
+        if "postgresql" not in self.available_drivers:
+            suggestions["postgresql"] = "psycopg2-binary"  # No compilation needed
+
+        if "oracle" not in self.available_drivers:
+            suggestions["oracle"] = "oracledb"  # Pure Python, newer
+
+        return suggestions
+
+    @staticmethod
+    def get_installation_help() -> str:
+        """
+        Get helpful installation instructions for database drivers.
+
+        Returns:
+            Formatted help text
+        """
+        return """
+Database Driver Installation Help
+=================================
+
+For MySQL:
+  Recommended: pip install pymysql (pure Python, no compilation)
+  Alternative: pip install mysql-connector-python (Oracle's pure Python)
+  Fastest (requires MySQL dev headers):
+    macOS: brew install mysql && pip install mysqlclient
+    Ubuntu: sudo apt-get install libmysqlclient-dev && pip install mysqlclient
+
+For PostgreSQL:
+  Recommended: pip install psycopg2-binary (pre-compiled, no headers needed)
+  Alternative: pip install pg8000 (pure Python, slightly slower)
+  Fastest (requires PostgreSQL dev headers):
+    macOS: brew install postgresql && pip install psycopg2
+    Ubuntu: sudo apt-get install libpq-dev && pip install psycopg2
+
+For Oracle:
+  Recommended: pip install oracledb (pure Python, no Oracle client needed)
+  Alternative (requires Oracle Instant Client):
+    1. Download Oracle Instant Client from oracle.com
+    2. Set environment variables (LD_LIBRARY_PATH, etc.)
+    3. pip install cx_Oracle
+
+Note: Pure Python drivers are slightly slower but much easier to install
+and maintain. They're recommended unless you have specific performance needs.
+"""
+
+
+def test_database_connectivity() -> None:
+    """
+    Test database connectivity and report available drivers.
+
+    WHY: Helps users understand what database drivers are available and
+    what they might need to install.
+    """
+    connector = DatabaseConnector()
+
+    print("Database Driver Status")
+    print("=" * 50)
+
+    available = connector.get_available_drivers()
+    if available:
+        print("\nAvailable Drivers:")
+        for db_type, driver in available.items():
+            print(f"  {db_type:12} -> {driver}")
+    else:
+        print("\nNo database drivers found!")
+
+    suggestions = connector.suggest_missing_drivers()
+    if suggestions:
+        print("\nRecommended installations for missing drivers:")
+        for db_type, package in suggestions.items():
+            print(f"  {db_type:12} -> pip install {package}")
+
+    print("\n" + connector.get_installation_help())
+
+
+if __name__ == "__main__":
+    # Run connectivity test when module is executed directly
+    test_database_connectivity()

claude_mpm/utils/error_handler.py

@@ -10,7 +10,8 @@ from datetime import datetime, timezone
 from functools import wraps
 from typing import Any, Callable, Dict, List, Optional, Type
 
-logger = logging.getLogger(__name__)
+from claude_mpm.core.logging_utils import get_logger
+logger = get_logger(__name__)
 
 
 class MPMError(Exception):

claude_mpm/utils/log_cleanup.py

@@ -6,14 +6,14 @@ including session directory cleanup, archived log removal, and rotation manageme
 """
 
 import gzip
-import logging
 import os
 import shutil
 from datetime import datetime, timedelta, timezone
 from pathlib import Path
 from typing import Dict, Optional, Tuple
 
-logger = logging.getLogger(__name__)
+from claude_mpm.core.logging_utils import get_logger
+logger = get_logger(__name__)
 
 
 class LogCleanupConfig:

claude_mpm/utils/path_operations.py

@@ -4,14 +4,14 @@ This module provides a centralized PathOperations class for common path validati
 and file operations, reducing code duplication across the codebase.
 """
 
-import logging
 import os
 import shutil
 import tempfile
 from pathlib import Path
 from typing import Callable, List, Optional, Union
 
-logger = logging.getLogger(__name__)
+from claude_mpm.core.logging_utils import get_logger
+logger = get_logger(__name__)
 
 
 class PathOperations:

claude_mpm/utils/robust_installer.py

@@ -340,6 +340,12 @@ class RobustPackageInstaller:
             "tree-sitter-java",
             "tree-sitter-cpp",
             "tree-sitter-c",
+            # Database packages that require compilation
+            "mysqlclient",  # Requires MySQL development headers
+            "psycopg2",  # Requires PostgreSQL development headers
+            "cx_oracle",  # Requires Oracle client libraries
+            "pycairo",  # Requires Cairo development headers
+            "lxml",  # Requires libxml2 development headers
         }
 
         return package_name.lower() in special_packages
@@ -363,6 +369,35 @@ class RobustPackageInstaller:
                 InstallStrategy.PIP_NO_DEPS,
             ]
 
+        # Database packages that require compilation
+        compilation_packages = {
+            "mysqlclient": ["pymysql"],  # Pure Python alternative
+            "psycopg2": ["psycopg2-binary"],  # Binary wheel alternative
+            "cx_oracle": [],  # No good alternative
+            "pycairo": [],  # No good alternative
+            "lxml": [],  # Usually works with binary wheels
+        }
+
+        package_lower = package_name.lower()
+        if package_lower in compilation_packages:
+            # Try normal install first, but with limited retries
+            strategies = [InstallStrategy.PIP]
+
+            # If there are alternatives, log suggestion
+            alternatives = compilation_packages[package_lower]
+            if alternatives:
+                logger.info(
+                    f"Package {package_name} requires compilation. "
+                    f"Consider using alternative: {', '.join(alternatives)}"
+                )
+            else:
+                logger.warning(
+                    f"Package {package_name} requires compilation and may fail on systems "
+                    f"without development headers installed."
+                )
+
+            return strategies
+
         return [InstallStrategy.PIP, InstallStrategy.PIP_UPGRADE]
 
     def _verify_installation(self, package_spec: str) -> bool:
@@ -459,6 +494,27 @@ class RobustPackageInstaller:
         if not stderr:
             return "Unknown error"
 
+        # Check for specific compilation errors
+        stderr_lower = stderr.lower()
+
+        if "mysql_config" in stderr_lower or "mysql.h" in stderr_lower:
+            return (
+                "mysqlclient compilation failed (missing MySQL development headers). "
+                "Use 'pip install pymysql' for a pure Python alternative that doesn't require compilation."
+            )
+
+        if "pg_config" in stderr_lower or "libpq-fe.h" in stderr_lower:
+            return (
+                "psycopg2 compilation failed (missing PostgreSQL development headers). "
+                "Use 'pip install psycopg2-binary' for a pre-compiled version."
+            )
+
+        if "oracle" in stderr_lower and "client" in stderr_lower:
+            return (
+                "cx_Oracle compilation failed (missing Oracle client libraries). "
+                "Use 'pip install oracledb' for a pure Python alternative."
+            )
+
         # Look for ERROR: lines
         error_lines = []
         for line in stderr.splitlines():

claude_mpm/utils/session_logging.py

@@ -6,13 +6,13 @@ Session Logging Utilities
 Convenience functions for session-based response logging.
 """
 
-import logging
 import os
 from typing import Any, Dict, Optional
 
 from claude_mpm.services.claude_session_logger import get_session_logger
 
-logger = logging.getLogger(__name__)
+from claude_mpm.core.logging_utils import get_logger
+logger = get_logger(__name__)
 
 
 def is_session_logging_enabled() -> bool:

claude_mpm/utils/subprocess_utils.py

@@ -8,7 +8,6 @@ with proper error handling, timeouts, and process cleanup.
 
 import asyncio
 import contextlib
-import logging
 import shlex
 import subprocess
 import time
@@ -16,7 +15,8 @@ from typing import Any, Dict, List, Optional
 
 import psutil
 
-logger = logging.getLogger(__name__)
+from claude_mpm.core.logging_utils import get_logger
+logger = get_logger(__name__)
 
 
 class SubprocessError(Exception):

claude_mpm/validation/agent_validator.py

@@ -20,7 +20,6 @@ Security Considerations:
 """
 
 import json
-import logging
 from dataclasses import dataclass, field
 from datetime import datetime, timezone
 from pathlib import Path
@@ -36,7 +35,8 @@ from claude_mpm.core.constants import (
     TimeoutConfig,
 )
 
-logger = logging.getLogger(__name__)
+from claude_mpm.core.logging_utils import get_logger
+logger = get_logger(__name__)
 
 
 @dataclass

{claude_mpm-4.3.20.dist-info → claude_mpm-4.3.22.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: claude-mpm
-Version: 4.3.20
+Version: 4.3.22
 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