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

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
@@ -309,9 +309,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}"
-        )
+        return 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]:
         """Create backup of target before deployment."""
@@ -324,7 +322,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 +341,9 @@ 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,9 @@ 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 +541,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 +549,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 +618,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 +658,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
@@ -471,6 +471,4 @@ 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}"
-        )
+        return 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."""

claude_mpm/services/version_control/semantic_versioning.py

@@ -100,7 +100,7 @@ class SemanticVersion:
             version += f"+{self.build}"
         return version
 
-    def __lt__(self, other: "SemanticVersion") -> bool:  # noqa: PLR0911
+    def __lt__(self, other: "SemanticVersion") -> bool:
         """Compare versions for sorting according to semver precedence.
 
         Comparison Rules:
@@ -416,7 +416,7 @@ class SemanticVersionManager:
                 return self._parse_toml_version_regex(file_path)
 
         try:
-            with open(file_path, "rb") as f:
+            with file_path.open("rb") as f:
                 data = tomllib.load(f)
 
             # Try different locations for version
@@ -437,7 +437,7 @@ class SemanticVersionManager:
     def _parse_toml_version_regex(self, file_path: Path) -> Optional[str]:
         """Parse version from TOML file using regex."""
         try:
-            with open(file_path) as f:
+            with file_path.open() as f:
                 content = f.read()
 
             # Look for version = "x.y.z" pattern
@@ -463,7 +463,7 @@ class SemanticVersionManager:
     def _parse_version_file(self, file_path: Path) -> Optional[str]:
         """Parse version from simple version file."""
         try:
-            with open(file_path) as f:
+            with file_path.open() as f:
                 return f.read().strip()
         except Exception:
             return None
@@ -471,7 +471,7 @@ class SemanticVersionManager:
     def _parse_pom_xml_version(self, file_path: Path) -> Optional[str]:
         """Parse version from Maven pom.xml."""
         try:
-            with open(file_path) as f:
+            with file_path.open() as f:
                 content = f.read()
 
             # Simple regex to find version in pom.xml
@@ -673,7 +673,7 @@ class SemanticVersionManager:
     def _update_toml_version(self, file_path: Path, new_version: str) -> bool:
         """Update version in TOML file."""
         try:
-            with open(file_path) as f:
+            with file_path.open() as f:
                 content = f.read()
 
             # Replace version field
@@ -691,7 +691,7 @@ class SemanticVersionManager:
                     break
 
             if updated:
-                with open(file_path, "w") as f:
+                with file_path.open("w") as f:
                     f.write(content)
                 return True
 
@@ -703,7 +703,7 @@ class SemanticVersionManager:
     def _update_simple_version_file(self, file_path: Path, new_version: str) -> bool:
         """Update version in simple version file."""
         try:
-            with open(file_path, "w") as f:
+            with file_path.open("w") as f:
                 f.write(new_version + "\n")
             return True
         except Exception:
@@ -712,7 +712,7 @@ class SemanticVersionManager:
     def _update_pom_xml_version(self, file_path: Path, new_version: str) -> bool:
         """Update version in Maven pom.xml."""
         try:
-            with open(file_path) as f:
+            with file_path.open() as f:
                 content = f.read()
 
             # Replace first version tag (project version)
@@ -722,7 +722,7 @@ class SemanticVersionManager:
             new_content = re.sub(pattern, replacement, content, count=1)
 
             if new_content != content:
-                with open(file_path, "w") as f:
+                with file_path.open("w") as f:
                     f.write(new_content)
                 return True
 
@@ -841,7 +841,7 @@ class SemanticVersionManager:
 
             # Read existing changelog or create new one
             if changelog_path.exists():
-                with open(changelog_path) as f:
+                with changelog_path.open() as f:
                     existing_content = f.read()
 
                 # Insert new entry after title
@@ -864,7 +864,7 @@ class SemanticVersionManager:
                 content = f"# Changelog\n\n{new_entry}\n"
 
             # Write updated changelog
-            with open(changelog_path, "w") as f:
+            with changelog_path.open("w") as f:
                 f.write(content)
 
             self.logger.info(f"Updated {changelog_file} with version {version}")
@@ -945,7 +945,7 @@ class SemanticVersionManager:
         versions = []
 
         try:
-            with open(changelog_path) as f:
+            with changelog_path.open() as f:
                 content = f.read()
 
             # Find version entries

claude_mpm/services/version_control/version_parser.py

@@ -365,7 +365,7 @@ class EnhancedVersionParser:
         package_file = self.project_root / "package.json"
         if package_file.exists():
             try:
-                with open(package_file) as f:
+                with package_file.open() as f:
                     data = json.load(f)
                     version = data.get("version")
                     if version and self._version_pattern.match(version):