claude-mpm 4.5.11__py3-none-any.whl → 4.5.12__py3-none-any.whl

claude_mpm/services/unified/config_strategies/unified_config_service.py

@@ -12,7 +12,7 @@ import threading
 from abc import ABC, abstractmethod
 from collections import defaultdict
 from dataclasses import dataclass, field
-from datetime import datetime, timedelta
+from datetime import datetime, timedelta, timezone
 from enum import Enum
 from pathlib import Path
 from typing import Any, Callable, Dict, List, Optional, TypeVar, Union
@@ -216,7 +216,7 @@ class UnifiedConfigService:
                     source=str(source),
                     format=format,
                     context=context,
-                    loaded_at=datetime.now(),
+                    loaded_at=datetime.now(timezone.utc),
                     checksum=self._calculate_checksum(config),
                     hot_reload=hot_reload,
                     ttl=ttl,
@@ -439,7 +439,7 @@ class UnifiedConfigService:
 
         path = Path(source)
         if path.exists():
-            with open(path) as f:
+            with path.open() as f:
                 return json.load(f)
 
         # Try to parse as JSON string
@@ -452,7 +452,7 @@ class UnifiedConfigService:
 
         path = Path(source)
         if path.exists():
-            with open(path) as f:
+            with path.open() as f:
                 return yaml.safe_load(f)
 
         # Try to parse as YAML string
@@ -476,7 +476,7 @@ class UnifiedConfigService:
             # Try to parse value
             try:
                 config[clean_key] = json.loads(value)
-            except:
+            except (json.JSONDecodeError, ValueError):
                 config[clean_key] = value
 
         return config
@@ -508,7 +508,7 @@ class UnifiedConfigService:
 
         path = Path(source)
         if path.exists():
-            with open(path) as f:
+            with path.open() as f:
                 return toml.load(f)
 
         return toml.loads(str(source))
@@ -706,7 +706,7 @@ class UnifiedConfigService:
         """Check if cached configuration is still valid"""
         if metadata.ttl:
             expiry = metadata.loaded_at + metadata.ttl
-            if datetime.now() > expiry:
+            if datetime.now(timezone.utc) > expiry:
                 return False
         return True
 
@@ -742,7 +742,7 @@ class UnifiedConfigService:
                 result = handler(error, source, context)
                 if result is not None:
                     return result
-            except:
+            except Exception:
                 continue
 
         self.logger.error(f"Failed to load config from {source}: {error}")

claude_mpm/services/unified/config_strategies/validation_strategy.py

@@ -402,7 +402,7 @@ class FormatValidator(BaseValidator):
         try:
             result = urllib.parse.urlparse(value)
             return all([result.scheme, result.netloc])
-        except:
+        except (ValueError, AttributeError, TypeError):
             return False
 
     @staticmethod
@@ -410,7 +410,7 @@ class FormatValidator(BaseValidator):
         try:
             result = urllib.parse.urlparse(value)
             return bool(result.scheme)
-        except:
+        except (ValueError, AttributeError, TypeError):
             return False
 
     @staticmethod
@@ -420,7 +420,7 @@ class FormatValidator(BaseValidator):
         try:
             uuid.UUID(value)
             return True
-        except:
+        except (ValueError, AttributeError, TypeError):
             return False
 
     @staticmethod
@@ -428,7 +428,7 @@ class FormatValidator(BaseValidator):
         try:
             ipaddress.IPv4Address(value)
             return True
-        except:
+        except (ValueError, ipaddress.AddressValueError):
             return False
 
     @staticmethod
@@ -436,7 +436,7 @@ class FormatValidator(BaseValidator):
         try:
             ipaddress.IPv6Address(value)
             return True
-        except:
+        except (ValueError, ipaddress.AddressValueError):
             return False
 
     @staticmethod
@@ -444,7 +444,7 @@ class FormatValidator(BaseValidator):
         try:
             ipaddress.ip_address(value)
             return True
-        except:
+        except (ValueError, ipaddress.AddressValueError):
             return False
 
     @staticmethod
@@ -461,7 +461,7 @@ class FormatValidator(BaseValidator):
         try:
             datetime.strptime(value, "%Y-%m-%d")
             return True
-        except:
+        except (ValueError, TypeError):
             return False
 
     @staticmethod
@@ -469,11 +469,11 @@ class FormatValidator(BaseValidator):
         try:
             datetime.strptime(value, "%H:%M:%S")
             return True
-        except:
+        except (ValueError, TypeError):
             try:
                 datetime.strptime(value, "%H:%M")
                 return True
-            except:
+            except (ValueError, TypeError):
                 return False
 
     @staticmethod
@@ -488,7 +488,7 @@ class FormatValidator(BaseValidator):
             try:
                 datetime.strptime(value, fmt)
                 return True
-            except:
+            except (ValueError, TypeError):
                 continue
         return False
 
@@ -499,7 +499,7 @@ class FormatValidator(BaseValidator):
         try:
             json.loads(value)
             return True
-        except:
+        except (json.JSONDecodeError, ValueError, TypeError):
             return False
 
     @staticmethod
@@ -509,7 +509,7 @@ class FormatValidator(BaseValidator):
         try:
             base64.b64decode(value, validate=True)
             return True
-        except:
+        except (ValueError, base64.binascii.Error):
             return False
 
     @staticmethod
@@ -517,7 +517,7 @@ class FormatValidator(BaseValidator):
         try:
             Path(value)
             return True
-        except:
+        except (ValueError, TypeError, OSError):
             return False
 
     @staticmethod
@@ -709,7 +709,7 @@ class ConditionalValidator(BaseValidator):
 
         try:
             return comparator(actual, expected)
-        except:
+        except (TypeError, ValueError, AttributeError, KeyError):
             return False
 
     def _apply_rule(
@@ -910,7 +910,7 @@ class CrossFieldValidator(BaseValidator):
         if comparator:
             try:
                 return comparator(val1, val2)
-            except:
+            except (TypeError, ValueError, AttributeError, KeyError):
                 return False
 
         return False

claude_mpm/services/unified/deployment_strategies/base.py

@@ -15,7 +15,7 @@ This module provides:
 
 from abc import abstractmethod
 from dataclasses import dataclass, field
-from datetime import datetime
+from datetime import datetime, timezone
 from enum import Enum
 from pathlib import Path
 from typing import Any, Dict, List, Optional, Union
@@ -255,7 +255,7 @@ class DeploymentStrategy(BaseDeploymentStrategy):
         result = DeploymentResult(
             success=False,
             status=DeploymentStatus.PENDING,
-            started_at=datetime.now(),
+            started_at=datetime.now(timezone.utc),
         )
 
         try:
@@ -334,7 +334,7 @@ class DeploymentStrategy(BaseDeploymentStrategy):
                     result.errors.append(f"Rollback failed: {rollback_error!s}")
 
         finally:
-            result.completed_at = datetime.now()
+            result.completed_at = datetime.now(timezone.utc)
             if result.started_at:
                 result.duration_seconds = (
                     result.completed_at - result.started_at
@@ -532,7 +532,7 @@ class DeploymentStrategy(BaseDeploymentStrategy):
         return {
             "deployment_id": deployment_info.get("deployment_id"),
             "deployed_path": str(deployment_info.get("deployed_path")),
-            "timestamp": datetime.now().isoformat(),
+            "timestamp": datetime.now(timezone.utc).isoformat(),
             "artifacts": deployment_info.get("artifacts", []),
         }
 

claude_mpm/services/unified/deployment_strategies/cloud_strategies.py

@@ -8,7 +8,7 @@ Reduces duplication by sharing common cloud deployment patterns.
 
 import json
 import subprocess
-from datetime import datetime
+from datetime import datetime, timezone
 from pathlib import Path
 from typing import Any, Dict, List
 
@@ -48,7 +48,7 @@ class RailwayDeploymentStrategy(DeploymentStrategy):
         # Check Railway CLI
         try:
             subprocess.run(["railway", "--version"], capture_output=True, check=True)
-        except:
+        except (subprocess.CalledProcessError, FileNotFoundError, OSError):
             errors.append(
                 "Railway CLI not installed. Install with: npm i -g @railway/cli"
             )
@@ -56,7 +56,7 @@ class RailwayDeploymentStrategy(DeploymentStrategy):
         # Check authentication
         try:
             subprocess.run(["railway", "whoami"], capture_output=True, check=True)
-        except:
+        except (subprocess.CalledProcessError, FileNotFoundError, OSError):
             errors.append("Not authenticated with Railway. Run: railway login")
 
         return errors
@@ -97,13 +97,13 @@ class RailwayDeploymentStrategy(DeploymentStrategy):
                         break
 
             return {
-                "deployment_id": f"railway_{datetime.now().timestamp()}",
+                "deployment_id": f"railway_{datetime.now(timezone.utc).timestamp()}",
                 "deployment_url": deployment_url,
                 "deployed_path": deploy_dir,
                 "stdout": result.stdout,
             }
         except subprocess.CalledProcessError as e:
-            raise Exception(f"Railway deployment failed: {e.stderr}")
+            raise Exception(f"Railway deployment failed: {e.stderr}") from e
 
     def verify(
         self, context: DeploymentContext, deployment_info: Dict[str, Any]
@@ -150,7 +150,7 @@ class AWSDeploymentStrategy(DeploymentStrategy):
         # Check AWS CLI
         try:
             subprocess.run(["aws", "--version"], capture_output=True, check=True)
-        except:
+        except (subprocess.CalledProcessError, FileNotFoundError, OSError):
             errors.append("AWS CLI not installed")
 
         # Check credentials
@@ -158,7 +158,7 @@ class AWSDeploymentStrategy(DeploymentStrategy):
             subprocess.run(
                 ["aws", "sts", "get-caller-identity"], capture_output=True, check=True
             )
-        except:
+        except (subprocess.CalledProcessError, FileNotFoundError, OSError):
             errors.append("AWS credentials not configured")
 
         # Validate service type
@@ -218,7 +218,7 @@ class AWSDeploymentStrategy(DeploymentStrategy):
                 "--zip-file",
                 f"fileb://{artifact}",
             ]
-        except:
+        except (subprocess.CalledProcessError, FileNotFoundError, OSError):
             # Create new function
             cmd = [
                 "aws",
@@ -277,7 +277,7 @@ class AWSDeploymentStrategy(DeploymentStrategy):
                 ]
                 subprocess.run(cmd, capture_output=True, check=True)
                 return True
-            except:
+            except (subprocess.CalledProcessError, FileNotFoundError, OSError):
                 return False
 
         return True
@@ -301,7 +301,7 @@ class AWSDeploymentStrategy(DeploymentStrategy):
             try:
                 subprocess.run(cmd, check=True)
                 return True
-            except:
+            except (subprocess.CalledProcessError, FileNotFoundError, OSError):
                 pass
         return False
 
@@ -334,7 +334,7 @@ class DockerDeploymentStrategy(DeploymentStrategy):
         # Check Docker
         try:
             subprocess.run(["docker", "--version"], capture_output=True, check=True)
-        except:
+        except (subprocess.CalledProcessError, FileNotFoundError, OSError):
             errors.append("Docker not installed or not running")
 
         # Check Dockerfile exists
@@ -356,7 +356,7 @@ class DockerDeploymentStrategy(DeploymentStrategy):
         """Execute Docker deployment."""
         source_dir = artifacts[0] if artifacts[0].is_dir() else artifacts[0].parent
         image_name = context.config.get(
-            "image_name", f"app_{datetime.now().timestamp()}"
+            "image_name", f"app_{datetime.now(timezone.utc).timestamp()}"
         )
         container_name = context.config.get("container_name", image_name)
 
@@ -444,7 +444,7 @@ class GitDeploymentStrategy(DeploymentStrategy):
         # Check Git
         try:
             subprocess.run(["git", "--version"], capture_output=True, check=True)
-        except:
+        except (subprocess.CalledProcessError, FileNotFoundError, OSError):
             errors.append("Git not installed")
 
         # Check remote URL
@@ -529,7 +529,7 @@ class GitDeploymentStrategy(DeploymentStrategy):
                 check=True,
             )
             return True
-        except:
+        except (subprocess.CalledProcessError, FileNotFoundError, OSError):
             return False
 
     def rollback(self, context: DeploymentContext, result: DeploymentResult) -> bool:
@@ -553,7 +553,7 @@ class GitDeploymentStrategy(DeploymentStrategy):
                     check=True,
                 )
                 return True
-            except:
+            except (subprocess.CalledProcessError, FileNotFoundError, OSError):
                 pass
         return False
 

claude_mpm/services/unified/deployment_strategies/local.py

@@ -13,7 +13,7 @@ Consolidates functionality from:
 import json
 import shutil
 import tempfile
-from datetime import datetime
+from datetime import datetime, timezone
 from pathlib import Path
 from typing import Any, Dict, List, Optional
 
@@ -183,7 +183,7 @@ class LocalDeploymentStrategy(DeploymentStrategy):
             "deployed_path": target_path,
             "deployed_files": deployed_files,
             "artifacts": [str(a) for a in artifacts],
-            "timestamp": datetime.now().isoformat(),
+            "timestamp": datetime.now(timezone.utc).isoformat(),
         }
 
     def verify(
@@ -300,7 +300,7 @@ class LocalDeploymentStrategy(DeploymentStrategy):
     def _check_write_permission(self, path: Path) -> bool:
         """Check if we have write permission to path."""
         try:
-            test_file = path / f".write_test_{datetime.now().timestamp()}"
+            test_file = path / f".write_test_{datetime.now(timezone.utc).timestamp()}"
             test_file.touch()
             test_file.unlink()
             return True
@@ -310,7 +310,7 @@ class LocalDeploymentStrategy(DeploymentStrategy):
     def _generate_deployment_id(self) -> str:
         """Generate unique deployment ID."""
         return (
-            f"local_{datetime.now().strftime('%Y%m%d_%H%M%S')}_{id(self) % 10000:04d}"
+            f"local_{datetime.now(timezone.utc).strftime('%Y%m%d_%H%M%S')}_{id(self) % 10000:04d}"
         )
 
     def _create_backup(self, context: DeploymentContext) -> Optional[Path]:
@@ -324,7 +324,7 @@ class LocalDeploymentStrategy(DeploymentStrategy):
             backup_dir = Path(tempfile.gettempdir()) / "claude_mpm_backups"
             backup_dir.mkdir(parents=True, exist_ok=True)
 
-            timestamp = datetime.now().strftime("%Y%m%d_%H%M%S")
+            timestamp = datetime.now(timezone.utc).strftime("%Y%m%d_%H%M%S")
             backup_name = f"{target_path.name}.backup_{timestamp}"
             backup_path = backup_dir / backup_name
 
@@ -343,7 +343,7 @@ class LocalDeploymentStrategy(DeploymentStrategy):
     def _write_version_file(self, target_path: Path, version: str) -> None:
         """Write version file to deployment."""
         version_file = target_path / ".version"
-        version_file.write_text(f"{version}\n{datetime.now().isoformat()}\n")
+        version_file.write_text(f"{version}\n{datetime.now(timezone.utc).isoformat()}\n")
 
     # Agent deployment methods (consolidating agent_deployment.py patterns)
 
@@ -409,7 +409,7 @@ class LocalDeploymentStrategy(DeploymentStrategy):
 
         for yaml_file in yaml_files:
             try:
-                with open(yaml_file) as f:
+                with yaml_file.open() as f:
                     data = yaml.safe_load(f)
                     # Basic agent structure validation
                     if not isinstance(data, dict):
@@ -425,11 +425,11 @@ class LocalDeploymentStrategy(DeploymentStrategy):
 
     def _convert_json_to_yaml(self, json_path: Path) -> Path:
         """Convert JSON agent to YAML format."""
-        with open(json_path) as f:
+        with json_path.open() as f:
             data = json.load(f)
 
         yaml_path = Path(tempfile.gettempdir()) / f"{json_path.stem}.yaml"
-        with open(yaml_path, "w") as f:
+        with yaml_path.open("w") as f:
             yaml.dump(data, f, default_flow_style=False, sort_keys=False)
 
         return yaml_path

claude_mpm/services/unified/deployment_strategies/utils.py

@@ -19,7 +19,7 @@ import json
 import shutil
 import subprocess
 import tempfile
-from datetime import datetime
+from datetime import datetime, timezone
 from pathlib import Path
 from typing import Any, Dict, List, Optional, Tuple, Union
 
@@ -162,7 +162,7 @@ def prepare_deployment_artifact(
     metadata = {
         "source": str(source_path),
         "type": artifact_type,
-        "created_at": datetime.now().isoformat(),
+        "created_at": datetime.now(timezone.utc).isoformat(),
     }
 
     # Auto-detect type
@@ -257,7 +257,7 @@ def verify_deployment_health(
     """
     health = {
         "status": "unknown",
-        "timestamp": datetime.now().isoformat(),
+        "timestamp": datetime.now(timezone.utc).isoformat(),
         "checks": {},
         "errors": [],
     }
@@ -464,7 +464,7 @@ def get_version_info(path: Union[str, Path]) -> Dict[str, Any]:
 
         if file_path.exists():
             if version_file == "package.json":
-                with open(file_path) as f:
+                with file_path.open() as f:
                     data = json.load(f)
                     version_info["version"] = data.get("version")
                     version_info["source"] = "package.json"
@@ -512,7 +512,7 @@ def update_version(
             shutil.copy2(version_file, backup_file)
 
         # Write new version
-        version_file.write_text(f"{new_version}\n{datetime.now().isoformat()}\n")
+        version_file.write_text(f"{new_version}\n{datetime.now(timezone.utc).isoformat()}\n")
         return True
 
     except Exception as e:
@@ -539,7 +539,7 @@ def calculate_checksum(path: Union[str, Path], algorithm: str = "sha256") -> str
     hasher = hashlib.new(algorithm)
 
     if path.is_file():
-        with open(path, "rb") as f:
+        with path.open("rb") as f:
             for chunk in iter(lambda: f.read(4096), b""):
                 hasher.update(chunk)
     elif path.is_dir():
@@ -547,7 +547,7 @@ def calculate_checksum(path: Union[str, Path], algorithm: str = "sha256") -> str
         for file_path in sorted(path.rglob("*")):
             if file_path.is_file():
                 hasher.update(str(file_path.relative_to(path)).encode())
-                with open(file_path, "rb") as f:
+                with file_path.open("rb") as f:
                     for chunk in iter(lambda: f.read(4096), b""):
                         hasher.update(chunk)
 
@@ -616,7 +616,7 @@ def load_env_file(env_file: Union[str, Path]) -> Dict[str, str]:
     env_path = Path(env_file)
 
     if env_path.exists():
-        with open(env_path) as f:
+        with env_path.open() as f:
             for line in f:
                 line = line.strip()
                 if line and not line.startswith("#") and "=" in line:
@@ -656,7 +656,7 @@ def export_env_to_file(env_vars: Dict[str, str], output_file: Union[str, Path])
     output_path = Path(output_file)
     output_path.parent.mkdir(parents=True, exist_ok=True)
 
-    with open(output_path, "w") as f:
+    with output_path.open("w") as f:
         for key, value in env_vars.items():
             # Escape special characters in value
             if " " in value or '"' in value:

claude_mpm/services/unified/deployment_strategies/vercel.py

@@ -9,7 +9,7 @@ Consolidates Vercel deployment patterns from multiple services.
 import json
 import subprocess
 import tempfile
-from datetime import datetime
+from datetime import datetime, timezone
 from pathlib import Path
 from typing import Any, Dict, List, Optional
 
@@ -203,13 +203,13 @@ class VercelDeploymentStrategy(DeploymentStrategy):
                     "deployed_path": deploy_dir,
                     "production": context.config.get("production", False),
                     "stdout": result.stdout,
-                    "timestamp": datetime.now().isoformat(),
+                    "timestamp": datetime.now(timezone.utc).isoformat(),
                 }
             raise Exception("Could not parse deployment URL from Vercel output")
 
         except subprocess.CalledProcessError as e:
             self._logger.error(f"Vercel deployment failed: {e.stderr}")
-            raise Exception(f"Deployment failed: {e.stderr}")
+            raise Exception(f"Deployment failed: {e.stderr}") from e
 
     def verify(
         self, context: DeploymentContext, deployment_info: Dict[str, Any]
@@ -336,7 +336,7 @@ class VercelDeploymentStrategy(DeploymentStrategy):
                             health["checks"][f"function_{func_name}"] = (
                                 response.status < 500
                             )
-                    except:
+                    except (urllib.error.URLError, OSError, TimeoutError):
                         health["checks"][f"function_{func_name}"] = False
 
             # Determine overall status
@@ -416,7 +416,7 @@ class VercelDeploymentStrategy(DeploymentStrategy):
 
         if vercel_config:
             config_path = deploy_dir / "vercel.json"
-            with open(config_path, "w") as f:
+            with config_path.open("w") as f:
                 json.dump(vercel_config, f, indent=2)
             return config_path
 
@@ -430,7 +430,7 @@ class VercelDeploymentStrategy(DeploymentStrategy):
 
         if env_vars:
             env_file = deploy_dir / ".env"
-            with open(env_file, "w") as f:
+            with env_file.open("w") as f:
                 for key, value in env_vars.items():
                     f.write(f"{key}={value}\n")
             return env_file
@@ -472,5 +472,5 @@ class VercelDeploymentStrategy(DeploymentStrategy):
     def _generate_deployment_id(self) -> str:
         """Generate unique deployment ID."""
         return (
-            f"vercel_{datetime.now().strftime('%Y%m%d_%H%M%S')}_{id(self) % 10000:04d}"
+            f"vercel_{datetime.now(timezone.utc).strftime('%Y%m%d_%H%M%S')}_{id(self) % 10000:04d}"
         )

claude_mpm/services/unified/unified_config.py

@@ -335,14 +335,14 @@ class UnifiedConfigManager(IConfigurationService, IUnifiedService):
 
             # Save based on file extension
             if target_path.suffix == ".json":
-                with open(target_path, "w") as f:
+                with target_path.open("w") as f:
                     json.dump(transformed_config, f, indent=2)
             elif target_path.suffix in [".yaml", ".yml"]:
                 # Would use yaml library here
-                with open(target_path, "w") as f:
+                with target_path.open("w") as f:
                     f.write(str(transformed_config))
             else:
-                with open(target_path, "w") as f:
+                with target_path.open("w") as f:
                     f.write(str(transformed_config))
 
             return ConfigurationResult(
@@ -685,6 +685,6 @@ class UnifiedConfigManager(IConfigurationService, IUnifiedService):
 
     def _get_timestamp(self) -> str:
         """Get current timestamp."""
-        from datetime import datetime
+        from datetime import datetime, timezone
 
-        return datetime.now().isoformat()
+        return datetime.now(timezone.utc).isoformat()

claude_mpm/services/unified/unified_deployment.py

@@ -460,6 +460,6 @@ class UnifiedDeploymentService(IDeploymentService, IUnifiedService):
 
     def _get_timestamp(self) -> str:
         """Get current timestamp."""
-        from datetime import datetime
+        from datetime import datetime, timezone
 
-        return datetime.now().isoformat()
+        return datetime.now(timezone.utc).isoformat()

claude_mpm/services/utility_service.py

@@ -110,7 +110,7 @@ class UtilityService(BaseService, UtilityServiceInterface):
                 **event_data,
             }
 
-            with open(log_file, "a") as f:
+            with log_file.open("a") as f:
                 f.write(json.dumps(log_entry) + "\n")
 
         except OSError as e:

claude_mpm/services/version_control/conflict_resolution.py

@@ -321,7 +321,7 @@ class ConflictResolutionManager:
 
         # Check content (simple heuristic)
         try:
-            with open(file_path, "rb") as f:
+            with file_path.open("rb") as f:
                 chunk = f.read(1024)
                 if b"\0" in chunk:
                     return True
@@ -596,7 +596,7 @@ class ConflictResolutionManager:
             )
 
             # Write resolved content
-            with open(file_path, "w", encoding="utf-8") as f:
+            with file_path.open("w", encoding="utf-8") as f:
                 f.write(resolved_content)
 
             return ConflictResolution(

claude_mpm/services/version_control/git_operations.py

@@ -147,10 +147,10 @@ class GitOperationsManager:
 
             return result
 
-        except FileNotFoundError:
-            raise GitOperationError("Git is not installed or not in PATH")
+        except FileNotFoundError as e:
+            raise GitOperationError("Git is not installed or not in PATH") from e
         except Exception as e:
-            raise GitOperationError(f"Error running Git command: {e}")
+            raise GitOperationError(f"Error running Git command: {e}") from e
 
     def get_current_branch(self) -> str:
         """Get the current Git branch name."""