claude-mpm 4.4.0__py3-none-any.whl → 4.4.4__py3-none-any.whl

claude_mpm/services/project/archive_manager.py

@@ -25,12 +25,13 @@ import shutil
 import subprocess
 from datetime import datetime, timedelta
 from pathlib import Path
-from typing import Any, Dict, List, Optional, Set, Tuple
+from typing import Dict, List, Optional, Tuple
 
 from rich.console import Console
 from rich.table import Table
 
 from claude_mpm.core.logging_utils import get_logger
+
 logger = get_logger(__name__)
 console = Console()
 
@@ -125,13 +126,15 @@ Generated by Claude MPM Archive Manager
             # Create metadata file if provided
             if metadata or reason:
                 meta_data = metadata or {}
-                meta_data.update({
-                    "original_path": str(file_path),
-                    "archived_at": datetime.now().isoformat(),
-                    "reason": reason or "Manual archive",
-                    "file_size": file_path.stat().st_size,
-                    "file_hash": self._calculate_file_hash(file_path),
-                })
+                meta_data.update(
+                    {
+                        "original_path": str(file_path),
+                        "archived_at": datetime.now().isoformat(),
+                        "reason": reason or "Manual archive",
+                        "file_size": file_path.stat().st_size,
+                        "file_hash": self._calculate_file_hash(file_path),
+                    }
+                )
 
                 meta_path = self.archive_path / f"{archive_name}.meta.json"
                 meta_path.write_text(json.dumps(meta_data, indent=2))
@@ -252,7 +255,9 @@ Generated by Claude MPM Archive Manager
                 "name": archive_file.name,
                 "path": str(archive_file),
                 "size": archive_file.stat().st_size,
-                "modified": datetime.fromtimestamp(archive_file.stat().st_mtime).isoformat(),
+                "modified": datetime.fromtimestamp(
+                    archive_file.stat().st_mtime
+                ).isoformat(),
                 "compressed": archive_file.suffix == ".gz",
             }
 
@@ -308,7 +313,9 @@ Generated by Claude MPM Archive Manager
             # Backup current file if it exists
             if target_path.exists():
                 self.archive_file(
-                    target_path, reason="Backup before restoration", metadata={"restoration_from": archive_name}
+                    target_path,
+                    reason="Backup before restoration",
+                    metadata={"restoration_from": archive_name},
                 )
 
             # Restore file
@@ -325,7 +332,7 @@ Generated by Claude MPM Archive Manager
 
         except Exception as e:
             logger.error(f"Failed to restore {archive_name}: {e}")
-            return False, f"Restoration failed: {str(e)}"
+            return False, f"Restoration failed: {e!s}"
 
     def compare_with_archive(self, current_file: Path, archive_name: str) -> Dict:
         """Compare current file with an archived version."""
@@ -366,7 +373,7 @@ Generated by Claude MPM Archive Manager
             }
 
         except Exception as e:
-            return {"error": f"Comparison failed: {str(e)}"}
+            return {"error": f"Comparison failed: {e!s}"}
 
     def create_archive_report(self) -> Dict:
         """Generate a report of all archives."""
@@ -462,13 +469,15 @@ Generated by Claude MPM Archive Manager
             return []
 
         relative_path = file_path.relative_to(self.project_path)
-        output = self._run_git_command([
-            "log",
-            f"-{limit}",
-            "--pretty=format:%H|%an|%at|%s",
-            "--follow",
-            str(relative_path),
-        ])
+        output = self._run_git_command(
+            [
+                "log",
+                f"-{limit}",
+                "--pretty=format:%H|%an|%at|%s",
+                "--follow",
+                str(relative_path),
+            ]
+        )
 
         if not output:
             return []
@@ -477,12 +486,14 @@ Generated by Claude MPM Archive Manager
         for line in output.splitlines():
             parts = line.split("|", 3)
             if len(parts) == 4:
-                commits.append({
-                    "hash": parts[0][:8],
-                    "author": parts[1],
-                    "date": datetime.fromtimestamp(int(parts[2])).isoformat(),
-                    "message": parts[3],
-                })
+                commits.append(
+                    {
+                        "hash": parts[0][:8],
+                        "author": parts[1],
+                        "date": datetime.fromtimestamp(int(parts[2])).isoformat(),
+                        "message": parts[3],
+                    }
+                )
         return commits
 
     def get_file_last_modified(self, file_path: Path) -> Optional[datetime]:
@@ -491,12 +502,14 @@ Generated by Claude MPM Archive Manager
             return None
 
         relative_path = file_path.relative_to(self.project_path)
-        output = self._run_git_command([
-            "log",
-            "-1",
-            "--format=%at",
-            str(relative_path),
-        ])
+        output = self._run_git_command(
+            [
+                "log",
+                "-1",
+                "--format=%at",
+                str(relative_path),
+            ]
+        )
 
         if output:
             return datetime.fromtimestamp(int(output))
@@ -523,10 +536,12 @@ Generated by Claude MPM Archive Manager
 
                 # Check for outdated content
                 if file_report.get("outdated_indicators"):
-                    report["outdated_sections"].append({
-                        "file": doc_file,
-                        "indicators": file_report["outdated_indicators"],
-                    })
+                    report["outdated_sections"].append(
+                        {
+                            "file": doc_file,
+                            "indicators": file_report["outdated_indicators"],
+                        }
+                    )
 
         # Check synchronization between docs
         sync_issues = self._check_documentation_sync()
@@ -561,10 +576,12 @@ Generated by Claude MPM Archive Manager
         for pattern_name, pattern in self.version_patterns.items():
             matches = pattern.findall(content)
             if matches:
-                report["version_references"].append({
-                    "type": pattern_name,
-                    "versions": matches[:5],  # First 5 matches
-                })
+                report["version_references"].append(
+                    {
+                        "type": pattern_name,
+                        "versions": matches[:5],  # First 5 matches
+                    }
+                )
 
         # Detect outdated indicators
         outdated_indicators = self._detect_outdated_content(content, file_path.name)
@@ -591,11 +608,13 @@ Generated by Claude MPM Archive Manager
             regex = re.compile(pattern, re.IGNORECASE)
             for i, line in enumerate(lines, 1):
                 if regex.search(line):
-                    indicators.append({
-                        "line": i,
-                        "type": description,
-                        "content": line.strip()[:100],  # First 100 chars
-                    })
+                    indicators.append(
+                        {
+                            "line": i,
+                            "type": description,
+                            "content": line.strip()[:100],  # First 100 chars
+                        }
+                    )
 
         # Check for old version numbers if VERSION file exists
         version_file = self.project_path / "VERSION"
@@ -605,13 +624,17 @@ Generated by Claude MPM Archive Manager
 
             for match in old_version_pattern.finditer(content):
                 found_version = match.group(0)
-                if found_version != current_version and self._is_older_version(found_version, current_version):
-                    pos = content[:match.start()].count('\n') + 1
-                    indicators.append({
-                        "line": pos,
-                        "type": "Old version reference",
-                        "content": f"Found {found_version} (current: {current_version})",
-                    })
+                if found_version != current_version and self._is_older_version(
+                    found_version, current_version
+                ):
+                    pos = content[: match.start()].count("\n") + 1
+                    indicators.append(
+                        {
+                            "line": pos,
+                            "type": "Old version reference",
+                            "content": f"Found {found_version} (current: {current_version})",
+                        }
+                    )
 
         return indicators[:20]  # Limit to 20 most relevant
 
@@ -648,22 +671,28 @@ Generated by Claude MPM Archive Manager
 
             if claude_versions and readme_versions:
                 if claude_versions[0] != readme_versions[0]:
-                    issues.append({
-                        "type": "Version mismatch",
-                        "files": ["CLAUDE.md", "README.md"],
-                        "details": f"CLAUDE.md: {claude_versions[0]}, README.md: {readme_versions[0]}",
-                    })
+                    issues.append(
+                        {
+                            "type": "Version mismatch",
+                            "files": ["CLAUDE.md", "README.md"],
+                            "details": f"CLAUDE.md: {claude_versions[0]}, README.md: {readme_versions[0]}",
+                        }
+                    )
 
             # Check for project name consistency
-            project_names = re.findall(r"Claude MPM|claude-mpm", readme_content, re.IGNORECASE)
+            project_names = re.findall(
+                r"Claude MPM|claude-mpm", readme_content, re.IGNORECASE
+            )
             if project_names:
                 unique_names = set(project_names)
                 if len(unique_names) > 1:
-                    issues.append({
-                        "type": "Inconsistent project naming",
-                        "files": ["README.md"],
-                        "details": f"Found variations: {', '.join(unique_names)}",
-                    })
+                    issues.append(
+                        {
+                            "type": "Inconsistent project naming",
+                            "files": ["README.md"],
+                            "details": f"Found variations: {', '.join(unique_names)}",
+                        }
+                    )
 
         # Check CHANGELOG.md exists and is recent
         changelog_path = self.project_path / "CHANGELOG.md"
@@ -672,17 +701,21 @@ Generated by Claude MPM Archive Manager
             if last_modified:
                 days_old = (datetime.now() - last_modified).days
                 if days_old > 30:
-                    issues.append({
-                        "type": "Stale changelog",
-                        "files": ["CHANGELOG.md"],
-                        "details": f"Last updated {days_old} days ago",
-                    })
+                    issues.append(
+                        {
+                            "type": "Stale changelog",
+                            "files": ["CHANGELOG.md"],
+                            "details": f"Last updated {days_old} days ago",
+                        }
+                    )
         else:
-            issues.append({
-                "type": "Missing file",
-                "files": ["CHANGELOG.md"],
-                "details": "CHANGELOG.md does not exist",
-            })
+            issues.append(
+                {
+                    "type": "Missing file",
+                    "files": ["CHANGELOG.md"],
+                    "details": "CHANGELOG.md does not exist",
+                }
+            )
 
         return issues
 
@@ -700,7 +733,9 @@ Generated by Claude MPM Archive Manager
         if report["synchronization_issues"]:
             for issue in report["synchronization_issues"]:
                 if issue["type"] == "Version mismatch":
-                    recommendations.append("🔄 Synchronize version numbers across documentation files")
+                    recommendations.append(
+                        "🔄 Synchronize version numbers across documentation files"
+                    )
                 elif issue["type"] == "Stale changelog":
                     recommendations.append("📅 Update CHANGELOG.md with recent changes")
                 elif issue["type"] == "Missing file":
@@ -708,17 +743,29 @@ Generated by Claude MPM Archive Manager
 
         # Check for TODO items
         total_todos = sum(
-            len([i for i in file_report.get("outdated_indicators", [])
-                 if i["type"] == "Unresolved TODOs"])
+            len(
+                [
+                    i
+                    for i in file_report.get("outdated_indicators", [])
+                    if i["type"] == "Unresolved TODOs"
+                ]
+            )
             for file_report in report["files_reviewed"].values()
         )
         if total_todos > 0:
-            recommendations.append(f"✅ Resolve {total_todos} TODO items in documentation")
+            recommendations.append(
+                f"✅ Resolve {total_todos} TODO items in documentation"
+            )
 
         # Check for deprecated references
         deprecated_count = sum(
-            len([i for i in file_report.get("outdated_indicators", [])
-                 if "deprecated" in i["type"].lower()])
+            len(
+                [
+                    i
+                    for i in file_report.get("outdated_indicators", [])
+                    if "deprecated" in i["type"].lower()
+                ]
+            )
             for file_report in report["files_reviewed"].values()
         )
         if deprecated_count > 0:
@@ -768,20 +815,24 @@ Generated by Claude MPM Archive Manager
                         "auto_detection": True,
                         "indicators": file_report.get("outdated_indicators", [])[:5],
                         "review_timestamp": review["timestamp"],
-                    }
+                    },
                 )
                 if archive_result:
-                    result["archived_files"].append({
+                    result["archived_files"].append(
+                        {
+                            "file": filename,
+                            "reason": archive_reason,
+                            "archive_path": str(archive_result),
+                        }
+                    )
+            elif should_archive:
+                result["skipped_files"].append(
+                    {
                         "file": filename,
                         "reason": archive_reason,
-                        "archive_path": str(archive_result),
-                    })
-            elif should_archive:
-                result["skipped_files"].append({
-                    "file": filename,
-                    "reason": archive_reason,
-                    "action": "Would archive (dry run)",
-                })
+                        "action": "Would archive (dry run)",
+                    }
+                )
 
         return result
 
@@ -854,7 +905,9 @@ Generated by Claude MPM Archive Manager
                     # Archive before update
                     self.archive_file(readme_path, reason="Before version sync")
                     readme_path.write_text(updated_readme)
-                    result["changes"].append(f"Updated README.md to version {current_version}")
+                    result["changes"].append(
+                        f"Updated README.md to version {current_version}"
+                    )
 
             # Update CHANGELOG.md header if exists
             if changelog_path.exists() and current_version:
@@ -877,14 +930,18 @@ Generated by Claude MPM Archive Manager
                         updated_changelog = "\n".join(lines)
 
                         # Archive before update
-                        self.archive_file(changelog_path, reason="Before adding new version")
+                        self.archive_file(
+                            changelog_path, reason="Before adding new version"
+                        )
                         changelog_path.write_text(updated_changelog)
-                        result["changes"].append(f"Added {current_version} section to CHANGELOG.md")
+                        result["changes"].append(
+                            f"Added {current_version} section to CHANGELOG.md"
+                        )
 
             result["synced"] = len(result["changes"]) > 0
 
         except Exception as e:
-            result["errors"].append(f"Sync failed: {str(e)}")
+            result["errors"].append(f"Sync failed: {e!s}")
             logger.error(f"Documentation sync failed: {e}")
 
         return result
@@ -919,7 +976,9 @@ Generated by Claude MPM Archive Manager
         table.add_column("Last Updated", style="magenta")
 
         for filename, report in review["files_reviewed"].items():
-            status = "✅ OK" if not report.get("outdated_indicators") else "⚠️ Needs Review"
+            status = (
+                "✅ OK" if not report.get("outdated_indicators") else "⚠️ Needs Review"
+            )
             issues = len(report.get("outdated_indicators", []))
 
             last_updated = "Unknown"
@@ -964,13 +1023,17 @@ Generated by Claude MPM Archive Manager
 
         # Generate diff if current file exists and review requested
         if target_path.exists() and review_changes:
-            diff_report = self.generate_documentation_diff_report(target_path, archive_file)
+            diff_report = self.generate_documentation_diff_report(
+                target_path, archive_file
+            )
 
             console.print("\n[bold cyan]📝 Changes to be applied:[/bold cyan]")
             console.print(diff_report)
 
             # Ask for confirmation
-            console.print("\n[bold yellow]Proceed with restoration? (y/n): [/bold yellow]", end="")
+            console.print(
+                "\n[bold yellow]Proceed with restoration? (y/n): [/bold yellow]", end=""
+            )
             # In automated context, assume yes
             confirm = True
 
@@ -978,4 +1041,4 @@ Generated by Claude MPM Archive Manager
                 return False, "Restoration cancelled by user"
 
         # Proceed with restoration
-        return self.restore_archive(archive_name, target_path)
+        return self.restore_archive(archive_name, target_path)

claude_mpm/services/project/documentation_manager.py

@@ -18,15 +18,15 @@ Created: 2025-01-26
 
 import difflib
 import hashlib
-import json
 import re
 from datetime import datetime
 from pathlib import Path
-from typing import Dict, List, Optional, Set, Tuple
+from typing import Dict, List, Tuple
 
 from rich.console import Console
 
 from claude_mpm.core.logging_utils import get_logger
+
 logger = get_logger(__name__)
 console = Console()
 
@@ -70,7 +70,9 @@ class DocumentationManager:
         if self.claude_md_path.exists():
             self.existing_content = self.claude_md_path.read_text(encoding="utf-8")
             self.content_hash = hashlib.md5(self.existing_content.encode()).hexdigest()
-            logger.info(f"Loaded existing CLAUDE.md ({len(self.existing_content)} chars)")
+            logger.info(
+                f"Loaded existing CLAUDE.md ({len(self.existing_content)} chars)"
+            )
 
     def has_existing_documentation(self) -> bool:
         """Check if project has existing CLAUDE.md."""
@@ -118,15 +120,17 @@ class DocumentationManager:
             if line.startswith("#"):
                 # Save previous section if exists
                 if current_section:
-                    sections.append({
-                        "title": current_section,
-                        "level": current_level,
-                        "start_line": section_start,
-                        "end_line": i - 1,
-                        "content_preview": self._get_content_preview(
-                            lines[section_start:i]
-                        ),
-                    })
+                    sections.append(
+                        {
+                            "title": current_section,
+                            "level": current_level,
+                            "start_line": section_start,
+                            "end_line": i - 1,
+                            "content_preview": self._get_content_preview(
+                                lines[section_start:i]
+                            ),
+                        }
+                    )
 
                 # Parse new section
                 level = len(line.split()[0])
@@ -137,13 +141,15 @@ class DocumentationManager:
 
         # Add last section
         if current_section:
-            sections.append({
-                "title": current_section,
-                "level": current_level,
-                "start_line": section_start,
-                "end_line": len(lines) - 1,
-                "content_preview": self._get_content_preview(lines[section_start:]),
-            })
+            sections.append(
+                {
+                    "title": current_section,
+                    "level": current_level,
+                    "start_line": section_start,
+                    "end_line": len(lines) - 1,
+                    "content_preview": self._get_content_preview(lines[section_start:]),
+                }
+            )
 
         return sections
 
@@ -160,13 +166,18 @@ class DocumentationManager:
 
         if self.existing_content:
             # Check for old patterns
-            if "## Installation" in self.existing_content and "pip install" not in self.existing_content:
+            if (
+                "## Installation" in self.existing_content
+                and "pip install" not in self.existing_content
+            ):
                 patterns.append("Missing installation instructions")
 
             if "TODO" in self.existing_content or "FIXME" in self.existing_content:
                 patterns.append("Contains TODO/FIXME items")
 
-            if not re.search(r"Last Updated:|Last Modified:", self.existing_content, re.IGNORECASE):
+            if not re.search(
+                r"Last Updated:|Last Modified:", self.existing_content, re.IGNORECASE
+            ):
                 patterns.append("Missing update timestamp")
 
             if "```" not in self.existing_content:
@@ -193,12 +204,16 @@ class DocumentationManager:
         custom = []
         for section in sections:
             title_lower = section["title"].lower()
-            if not any(re.search(pattern, title_lower) for pattern in standard_patterns):
+            if not any(
+                re.search(pattern, title_lower) for pattern in standard_patterns
+            ):
                 custom.append(section["title"])
 
         return custom
 
-    def merge_with_template(self, new_content: str, preserve_custom: bool = True) -> str:
+    def merge_with_template(
+        self, new_content: str, preserve_custom: bool = True
+    ) -> str:
         """Merge existing content with new template content."""
         if not self.existing_content:
             return new_content
@@ -279,35 +294,37 @@ class DocumentationManager:
         # Map to known section types
         if "priority" in title and "index" in title:
             return "priority_index"
-        elif "critical" in title and "security" in title:
+        if "critical" in title and "security" in title:
             return "critical_security"
-        elif "critical" in title and "business" in title:
+        if "critical" in title and "business" in title:
             return "critical_business"
-        elif "important" in title and "architecture" in title:
+        if "important" in title and "architecture" in title:
             return "important_architecture"
-        elif "important" in title and "workflow" in title:
+        if "important" in title and "workflow" in title:
             return "important_workflow"
-        elif "project" in title and "overview" in title:
+        if "project" in title and "overview" in title:
             return "project_overview"
-        elif "standard" in title and "coding" in title:
+        if "standard" in title and "coding" in title:
             return "standard_coding"
-        elif "standard" in title and "tasks" in title:
+        if "standard" in title and "tasks" in title:
             return "standard_tasks"
-        elif "documentation" in title:
+        if "documentation" in title:
             return "documentation_links"
-        elif "optional" in title or "future" in title:
+        if "optional" in title or "future" in title:
             return "optional_future"
-        elif "meta" in title or "maintain" in title:
+        if "meta" in title or "maintain" in title:
             return "meta_maintenance"
-        else:
-            return "unknown"
+        return "unknown"
 
     def _merge_section_content(
         self, existing: str, new: str, section_header: str
     ) -> str:
         """Merge content from existing and new sections."""
         # For critical sections, prefer new content but append unique existing items
-        if "critical" in section_header.lower() or "important" in section_header.lower():
+        if (
+            "critical" in section_header.lower()
+            or "important" in section_header.lower()
+        ):
             # Extract bullet points from both
             existing_items = self._extract_bullet_points(existing)
             new_items = self._extract_bullet_points(new)
@@ -321,13 +338,11 @@ class DocumentationManager:
             # Reconstruct section
             if all_items:
                 return "\n".join([""] + all_items + [""])
-            else:
-                return new
-        else:
-            # For other sections, use new as base and append existing
-            if existing.strip() and existing.strip() != new.strip():
-                return f"{new}\n\n<!-- Preserved from previous version -->\n{existing}"
             return new
+        # For other sections, use new as base and append existing
+        if existing.strip() and existing.strip() != new.strip():
+            return f"{new}\n\n<!-- Preserved from previous version -->\n{existing}"
+        return new
 
     def _extract_bullet_points(self, content: str) -> List[str]:
         """Extract bullet points from content."""
@@ -343,7 +358,9 @@ class DocumentationManager:
         for existing in items:
             existing_clean = re.sub(r"[^a-zA-Z0-9\s]", "", existing.lower())
             # Use fuzzy matching for similarity
-            similarity = difflib.SequenceMatcher(None, item_clean, existing_clean).ratio()
+            similarity = difflib.SequenceMatcher(
+                None, item_clean, existing_clean
+            ).ratio()
             if similarity > 0.8:  # 80% similarity threshold
                 return True
         return False
@@ -464,7 +481,9 @@ class DocumentationManager:
                 issues.append(f"Missing required section: {section}")
 
         # Check for priority markers
-        has_markers = any(marker in content for marker in self.PRIORITY_MARKERS.values())
+        has_markers = any(
+            marker in content for marker in self.PRIORITY_MARKERS.values()
+        )
         if not has_markers:
             issues.append("No priority markers found (🔴🟡🟢⚪)")
 
@@ -533,4 +552,4 @@ class DocumentationManager:
 - **Last Updated**: {datetime.now().isoformat()}
 - **Created By**: Claude MPM /mpm-init
 - **Update Frequency**: As needed when requirements change
-"""
+"""