claude-mpm 4.1.6__py3-none-any.whl → 4.24.0__py3-none-any.whl

claude_mpm/services/version_control/git_operations.py

@@ -15,7 +15,7 @@ import logging
 import os
 import subprocess
 from dataclasses import dataclass, field
-from datetime import datetime
+from datetime import datetime, timezone
 from typing import Any, Dict, List, Optional
 
 
@@ -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."""
@@ -333,7 +333,7 @@ class GitOperationsManager:
         """
         try:
             # Convert to relative path if absolute
-            if os.path.isabs(file_path):
+            if Path(file_path).is_absolute():
                 try:
                     file_path = os.path.relpath(file_path, self.project_root)
                 except ValueError:
@@ -366,7 +366,7 @@ class GitOperationsManager:
         Returns:
             GitOperationResult with operation details
         """
-        start_time = datetime.now()
+        start_time = datetime.now(timezone.utc)
         current_branch = self.get_current_branch()
 
         # Generate full branch name with prefix
@@ -398,7 +398,7 @@ class GitOperationsManager:
                     # Remote push failed, continue without remote tracking
                     pass
 
-            execution_time = (datetime.now() - start_time).total_seconds()
+            execution_time = (datetime.now(timezone.utc) - start_time).total_seconds()
 
             return GitOperationResult(
                 success=True,
@@ -411,7 +411,7 @@ class GitOperationsManager:
             )
 
         except GitOperationError as e:
-            execution_time = (datetime.now() - start_time).total_seconds()
+            execution_time = (datetime.now(timezone.utc) - start_time).total_seconds()
 
             return GitOperationResult(
                 success=False,
@@ -433,7 +433,7 @@ class GitOperationsManager:
         Returns:
             GitOperationResult with operation details
         """
-        start_time = datetime.now()
+        start_time = datetime.now(timezone.utc)
         current_branch = self.get_current_branch()
 
         try:
@@ -448,13 +448,15 @@ class GitOperationsManager:
                     branch_before=current_branch,
                     branch_after=current_branch,
                     files_changed=modified_files,
-                    execution_time=(datetime.now() - start_time).total_seconds(),
+                    execution_time=(
+                        datetime.now(timezone.utc) - start_time
+                    ).total_seconds(),
                 )
 
             # Switch to the branch
             result = self._run_git_command(["checkout", branch_name])
 
-            execution_time = (datetime.now() - start_time).total_seconds()
+            execution_time = (datetime.now(timezone.utc) - start_time).total_seconds()
 
             return GitOperationResult(
                 success=True,
@@ -467,7 +469,7 @@ class GitOperationsManager:
             )
 
         except GitOperationError as e:
-            execution_time = (datetime.now() - start_time).total_seconds()
+            execution_time = (datetime.now(timezone.utc) - start_time).total_seconds()
 
             return GitOperationResult(
                 success=False,
@@ -498,7 +500,7 @@ class GitOperationsManager:
         Returns:
             GitOperationResult with operation details
         """
-        start_time = datetime.now()
+        start_time = datetime.now(timezone.utc)
         current_branch = self.get_current_branch()
 
         try:
@@ -542,7 +544,7 @@ class GitOperationsManager:
                     # Branch deletion failed, but merge was successful
                     pass
 
-            execution_time = (datetime.now() - start_time).total_seconds()
+            execution_time = (datetime.now(timezone.utc) - start_time).total_seconds()
 
             return GitOperationResult(
                 success=True,
@@ -555,7 +557,7 @@ class GitOperationsManager:
             )
 
         except GitOperationError as e:
-            execution_time = (datetime.now() - start_time).total_seconds()
+            execution_time = (datetime.now(timezone.utc) - start_time).total_seconds()
 
             return GitOperationResult(
                 success=False,
@@ -651,7 +653,7 @@ class GitOperationsManager:
         Returns:
             GitOperationResult with operation details
         """
-        start_time = datetime.now()
+        start_time = datetime.now(timezone.utc)
         current_branch = self.get_current_branch()
 
         if not branch_name:
@@ -668,7 +670,7 @@ class GitOperationsManager:
 
             result = self._run_git_command(push_args)
 
-            execution_time = (datetime.now() - start_time).total_seconds()
+            execution_time = (datetime.now(timezone.utc) - start_time).total_seconds()
 
             return GitOperationResult(
                 success=True,
@@ -681,7 +683,7 @@ class GitOperationsManager:
             )
 
         except GitOperationError as e:
-            execution_time = (datetime.now() - start_time).total_seconds()
+            execution_time = (datetime.now(timezone.utc) - start_time).total_seconds()
 
             return GitOperationResult(
                 success=False,
@@ -706,7 +708,7 @@ class GitOperationsManager:
         Returns:
             GitOperationResult with operation details
         """
-        start_time = datetime.now()
+        start_time = datetime.now(timezone.utc)
         current_branch = self.get_current_branch()
 
         if not branch_name:
@@ -725,7 +727,7 @@ class GitOperationsManager:
                 result = self._run_git_command(["pull", remote, branch_name])
                 self._run_git_command(["checkout", current_branch])
 
-            execution_time = (datetime.now() - start_time).total_seconds()
+            execution_time = (datetime.now(timezone.utc) - start_time).total_seconds()
 
             return GitOperationResult(
                 success=True,
@@ -738,7 +740,7 @@ class GitOperationsManager:
             )
 
         except GitOperationError as e:
-            execution_time = (datetime.now() - start_time).total_seconds()
+            execution_time = (datetime.now(timezone.utc) - start_time).total_seconds()
 
             return GitOperationResult(
                 success=False,
@@ -762,7 +764,7 @@ class GitOperationsManager:
         Returns:
             GitOperationResult with cleanup details
         """
-        start_time = datetime.now()
+        start_time = datetime.now(timezone.utc)
         current_branch = self.get_current_branch()
 
         try:
@@ -789,7 +791,7 @@ class GitOperationsManager:
             # Clean up remote tracking branches
             self._run_git_command(["remote", "prune", "origin"], check=False)
 
-            execution_time = (datetime.now() - start_time).total_seconds()
+            execution_time = (datetime.now(timezone.utc) - start_time).total_seconds()
 
             return GitOperationResult(
                 success=True,
@@ -802,7 +804,7 @@ class GitOperationsManager:
             )
 
         except GitOperationError as e:
-            execution_time = (datetime.now() - start_time).total_seconds()
+            execution_time = (datetime.now(timezone.utc) - start_time).total_seconds()
 
             return GitOperationResult(
                 success=False,

claude_mpm/services/version_control/semantic_versioning.py

@@ -42,7 +42,7 @@ Change Analysis:
 import logging
 import re
 from dataclasses import dataclass, field
-from datetime import datetime
+from datetime import datetime, timezone
 from enum import Enum
 from typing import Dict, List, Optional, Tuple
 
@@ -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
 
@@ -748,7 +748,7 @@ class SemanticVersionManager:
         Returns:
             Formatted changelog entry
         """
-        date_str = datetime.now().strftime("%Y-%m-%d")
+        date_str = datetime.now(timezone.utc).strftime("%Y-%m-%d")
         if metadata and metadata.release_date:
             date_str = metadata.release_date.strftime("%Y-%m-%d")
 
@@ -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

@@ -1,5 +1,3 @@
-from pathlib import Path
-
 """
 Enhanced version parsing system with multiple source support and fallback mechanisms.
 
@@ -17,13 +15,17 @@ The system includes caching for performance and validation for data integrity.
 
 import contextlib
 import json
-import logging
 import re
 import subprocess
-from datetime import datetime, timedelta
+from datetime import datetime, timedelta, timezone
 from functools import lru_cache
+from pathlib import Path
 from typing import Dict, List, Optional, Tuple
 
+from claude_mpm.core.logging_utils import get_logger
+
+logger = get_logger(__name__)
+
 
 class VersionSource:
     """Enumeration of version sources with priority ordering."""
@@ -61,7 +63,7 @@ class VersionMetadata:
     ):
         self.version = version
         self.source = source
-        self.release_date = release_date or datetime.now()
+        self.release_date = release_date or datetime.now(timezone.utc)
         self.commit_hash = commit_hash
         self.author = author
         self.message = message
@@ -104,7 +106,6 @@ class EnhancedVersionParser:
         """
         self.project_root = project_root or Path.cwd()
         self.cache_ttl = cache_ttl
-        self.logger = logging.getLogger(__name__)
         self._cache: Dict[str, Tuple[datetime, any]] = {}
 
         # Compile regex patterns once for efficiency
@@ -112,21 +113,23 @@ class EnhancedVersionParser:
             r"(\d+)\.(\d+)\.(\d+)(?:-([a-zA-Z0-9\-\.]+))?(?:\+([a-zA-Z0-9\-\.]+))?"
         )
         self._changelog_version_pattern = re.compile(
-            r"##\s*\[?([0-9]+\.[0-9]+\.[0-9]+[^\]]*)\]?\s*[-–]\s*(\d{4}-\d{2}-\d{2})?"
+            r"##\s*\[?([0-9]+\.[0-9]+\.[0-9]+[^\]]*)\]?\s*[--]\s*(\d{4}-\d{2}-\d{2})?"
         )
 
     def _get_cached(self, key: str) -> Optional[any]:
         """Get cached value if still valid."""
         if key in self._cache:
             timestamp, value = self._cache[key]
-            if datetime.now() - timestamp < timedelta(seconds=self.cache_ttl):
+            if datetime.now(timezone.utc) - timestamp < timedelta(
+                seconds=self.cache_ttl
+            ):
                 return value
             del self._cache[key]
         return None
 
     def _set_cached(self, key: str, value: any) -> any:
         """Set cached value with timestamp."""
-        self._cache[key] = (datetime.now(), value)
+        self._cache[key] = (datetime.now(timezone.utc), value)
         return value
 
     def get_current_version(
@@ -332,7 +335,7 @@ class EnhancedVersionParser:
             )
             if result.returncode == 0:
                 commit_hash = result.stdout.strip()[:7]
-        except:
+        except Exception:
             pass
 
         return VersionMetadata(
@@ -362,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):

claude_mpm/services/version_service.py

@@ -9,7 +9,7 @@ Extracted from ClaudeRunner to follow Single Responsibility Principle.
 """
 
 from pathlib import Path
-from typing import Any, Dict, Optional
+from typing import Any, Dict, List, Optional
 
 from claude_mpm.config.paths import paths
 from claude_mpm.core.base_service import BaseService
@@ -274,3 +274,106 @@ class VersionService(BaseService, VersionServiceInterface):
             "message": "Update checking not implemented",
             "checked_at": None,
         }
+
+    def get_agents_versions(self) -> Dict[str, List[Dict[str, str]]]:
+        """Get all agents grouped by tier with versions.
+
+        Returns:
+            Dict with keys: system, user, project
+            Each value is list of agent dicts with: name, version, id
+        """
+        from claude_mpm.core.unified_agent_registry import get_agent_registry
+
+        agents_by_tier = {"system": [], "user": [], "project": []}
+
+        try:
+            registry = get_agent_registry()
+            all_agents = registry.list_agents()
+
+            for agent in all_agents:
+                agent_info = {
+                    "name": agent.name,
+                    "version": agent.version,
+                    "id": agent.name,  # Use name as ID since agent_id doesn't exist
+                }
+                tier = (
+                    agent.tier.value
+                    if hasattr(agent.tier, "value")
+                    else str(agent.tier)
+                )
+                if tier in agents_by_tier:
+                    agents_by_tier[tier].append(agent_info)
+                else:
+                    agents_by_tier["system"].append(agent_info)
+
+            # Sort each tier alphabetically by name
+            for tier, agents in agents_by_tier.items():
+                agents.sort(key=lambda x: x["name"])
+
+        except Exception as e:
+            self.logger.error(f"Failed to get agent versions: {e}")
+
+        return agents_by_tier
+
+    def get_skills_versions(self) -> Dict[str, List[Dict[str, str]]]:
+        """Get all skills grouped by source with versions.
+
+        Returns:
+            Dict with keys: bundled, user, project
+            Each value is list of skill dicts with: name, version, description
+        """
+        from claude_mpm.skills.registry import get_registry
+
+        skills_by_source = {"bundled": [], "user": [], "project": []}
+
+        try:
+            registry = get_registry()
+
+            for skill in registry.list_skills():
+                skill_info = {
+                    "name": skill.name,
+                    "version": skill.version,
+                    "description": (
+                        skill.description[:60] + "..."
+                        if len(skill.description) > 60
+                        else skill.description
+                    ),
+                }
+                source = skill.source if skill.source in skills_by_source else "bundled"
+                skills_by_source[source].append(skill_info)
+
+            # Sort each source alphabetically by name
+            for source, skills in skills_by_source.items():
+                skills.sort(key=lambda x: x["name"])
+
+        except Exception as e:
+            self.logger.error(f"Failed to get skill versions: {e}")
+
+        return skills_by_source
+
+    def get_version_summary(self) -> Dict:
+        """Get complete version summary.
+
+        Returns:
+            Dict with project_version, build, agents, skills, and counts
+        """
+        agents = self.get_agents_versions()
+        skills = self.get_skills_versions()
+        build = self.get_build_number()
+
+        return {
+            "project_version": self.get_base_version(),
+            "build": build,
+            "agents": agents,
+            "skills": skills,
+            "counts": {
+                "agents_total": sum(len(v) for v in agents.values()),
+                "agents_system": len(agents.get("system", [])),
+                "agents_user": len(agents.get("user", [])),
+                "agents_project": len(agents.get("project", [])),
+                "skills_total": sum(len(v) for v in skills.values()),
+                "skills_bundled": len(skills.get("bundled", [])),
+                "skills_user": len(skills.get("user", [])),
+                "skills_project": len(skills.get("project", [])),
+            },
+        }

claude_mpm/services/visualization/__init__.py

@@ -0,0 +1,19 @@
+"""
+Visualization Services for Claude MPM
+=====================================
+
+This module provides visualization services for code analysis,
+including Mermaid diagram generation for various code structures.
+"""
+
+from .mermaid_generator import (
+    DiagramConfig,
+    DiagramType,
+    MermaidGeneratorService,
+)
+
+__all__ = [
+    "DiagramConfig",
+    "DiagramType",
+    "MermaidGeneratorService",
+]