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

claude_mpm/cli/commands/mpm_init/core.py

@@ -24,6 +24,7 @@ from claude_mpm.utils.display_helper import DisplayHelper
 
 # Import from sibling modules in the mpm_init package
 from . import display, git_activity, modes, prompts
+from .knowledge_extractor import ProjectKnowledgeExtractor
 
 logger = get_logger(__name__)
 
@@ -34,6 +35,7 @@ class MPMInitCommand:
     def __init__(self, project_path: Optional[Path] = None):
         """Initialize the MPM-Init command."""
         self.project_path = project_path or Path.cwd()
+        self.claude_mpm_dir = self.project_path / ".claude-mpm"
         self.claude_mpm_script = self._find_claude_mpm_script()
         self.console = Console()
 
@@ -155,6 +157,9 @@ class MPMInitCommand:
                 if pre_check_result.get("status") == OperationResult.ERROR:
                     return pre_check_result
 
+            # Note: .gitignore recommendations are now shown in 'mpm config' command
+            # instead of automatic updates during initialization
+
             # Build the delegation prompt
             if update_mode:
                 prompt = self._build_update_prompt(
@@ -198,6 +203,10 @@ class MPMInitCommand:
             if update_mode and result.get("status") == OperationResult.SUCCESS:
                 self._handle_update_post_processing()
 
+            # Deploy PM skills after successful initialization
+            if result.get("status") == OperationResult.SUCCESS:
+                self._deploy_pm_skills()
+
             return result
 
         except Exception as e:
@@ -317,7 +326,9 @@ class MPMInitCommand:
             error_msg = (
                 result.stderr
                 if result.stderr
-                else result.stdout if result.stdout else "Unknown error occurred"
+                else result.stdout
+                if result.stdout
+                else "Unknown error occurred"
             )
 
             logger.error(f"claude-mpm run failed: {error_msg}")
@@ -404,6 +415,10 @@ class MPMInitCommand:
             self.project_path, project_type, framework, ast_analysis
         )
 
+    def _is_initialized(self) -> bool:
+        """Check if project is already initialized with .claude-mpm directory."""
+        return self.claude_mpm_dir.exists()
+
     def _build_update_prompt(
         self,
         project_type: Optional[str],
@@ -411,10 +426,65 @@ class MPMInitCommand:
         ast_analysis: bool,
         preserve_custom: bool,
     ) -> str:
-        """Build prompt for update mode."""
+        """Build prompt for update mode with optional knowledge extraction."""
         # Get existing content analysis
         doc_analysis = self.doc_manager.analyze_existing_content()
 
+        # Check if project is initialized (.claude-mpm exists)
+        # If so, use enhanced update mode with knowledge extraction
+        if self._is_initialized():
+            self.console.print(
+                "[cyan]✓ Detected initialized project - activating enhanced update mode[/cyan]"
+            )
+
+            # Extract knowledge from all sources
+            extractor = ProjectKnowledgeExtractor(self.project_path)
+
+            self.console.print("[cyan]✓ Analyzing git history (last 90 days)...[/cyan]")
+            git_insights = extractor.extract_from_git(days=90)
+
+            if git_insights.get("available"):
+                arch_count = len(git_insights.get("architectural_decisions", []))
+                workflow_count = len(git_insights.get("workflow_patterns", []))
+                self.console.print(f"  - Found {arch_count} architectural decisions")
+                self.console.print(f"  - Detected {workflow_count} workflow patterns")
+
+            self.console.print("[cyan]✓ Analyzing session logs...[/cyan]")
+            log_insights = extractor.extract_from_logs()
+
+            if log_insights.get("available"):
+                learning_count = len(log_insights.get("learnings", []))
+                self.console.print(f"  - Found {learning_count} learning entries")
+
+            self.console.print("[cyan]✓ Analyzing memory files...[/cyan]")
+            memory_insights = extractor.extract_from_memory()
+
+            if memory_insights.get("available"):
+                total_insights = (
+                    len(memory_insights.get("architectural_knowledge", []))
+                    + len(memory_insights.get("implementation_guidelines", []))
+                    + len(memory_insights.get("common_mistakes", []))
+                    + len(memory_insights.get("technical_context", []))
+                )
+                self.console.print(f"  - Found {total_insights} accumulated insights")
+
+            self.console.print(
+                "[green]✓ Knowledge extraction complete - building enhanced prompt[/green]\n"
+            )
+
+            # Build enhanced prompt with extracted knowledge
+            return prompts.build_enhanced_update_prompt(
+                self.project_path,
+                doc_analysis,
+                git_insights,
+                log_insights,
+                memory_insights,
+                project_type,
+                framework,
+                ast_analysis,
+                preserve_custom,
+            )
+        # Standard update mode (no .claude-mpm directory)
         return prompts.build_update_prompt(
             self.project_path,
             doc_analysis,
@@ -500,6 +570,11 @@ class MPMInitCommand:
             self.archive_manager,
         )
 
+        # Optimize CLAUDE.md with prompt-engineer if it exists
+        claude_md = self.project_path / "CLAUDE.md"
+        if claude_md.exists():
+            self._optimize_claude_md_with_prompt_engineer()
+
     def _display_results(self, result: Dict, verbose: bool):
         """Display initialization results."""
         display.display_results(self.display, self.console, result, verbose)
@@ -521,5 +596,157 @@ class MPMInitCommand:
         """Build structured Research agent delegation prompt from git analysis."""
         return prompts.build_research_context_prompt(git_analysis, days)
 
+    def _optimize_claude_md_with_prompt_engineer(self) -> None:
+        """Optimize CLAUDE.md with prompt-engineer for conciseness and clarity."""
+        claude_md_path = self.project_path / "CLAUDE.md"
+        if not claude_md_path.exists():
+            return
+
+        try:
+            # Read current content
+            original_content = claude_md_path.read_text()
+            original_tokens = self._estimate_tokens(original_content)
+
+            # Create backup
+            backup_path = self.archive_manager.auto_archive_before_update(
+                claude_md_path, update_reason="Before prompt-engineer optimization"
+            )
+
+            if not backup_path:
+                logger.warning("Could not create backup before optimization")
+                self.console.print(
+                    "[yellow]⚠️  Skipping optimization - backup failed[/yellow]"
+                )
+                return
+
+            self.console.print(
+                "\n[cyan]✓ Optimizing CLAUDE.md with prompt-engineer...[/cyan]"
+            )
+            self.console.print(f"  - Original: {original_tokens:,} tokens (estimated)")
+            self.console.print(f"  - Backup created: {backup_path}")
+
+            # Build optimization prompt
+            prompt = prompts.build_prompt_engineer_optimization_prompt(
+                original_content, original_tokens
+            )
+
+            # Run optimization through subprocess
+            with Progress(
+                SpinnerColumn(),
+                TextColumn("[progress.description]{task.description}"),
+                console=self.console,
+            ) as progress:
+                task = progress.add_task(
+                    "[cyan]Running prompt-engineer optimization...", total=None
+                )
+
+                result = self._run_initialization(
+                    prompt, verbose=False, update_mode=True
+                )
+
+                progress.update(task, description="[green]✓ Optimization complete")
+
+            # Check if optimization succeeded
+            if result.get("status") == OperationResult.SUCCESS:
+                # Read optimized content
+                optimized_content = claude_md_path.read_text()
+                optimized_tokens = self._estimate_tokens(optimized_content)
+
+                # Calculate reduction
+                token_reduction = original_tokens - optimized_tokens
+                reduction_percent = (
+                    (token_reduction / original_tokens * 100)
+                    if original_tokens > 0
+                    else 0
+                )
+
+                self.console.print(
+                    f"  - Optimized: {optimized_tokens:,} tokens ({reduction_percent:.1f}% reduction)"
+                )
+                self.console.print("[green]✓ CLAUDE.md optimization complete[/green]\n")
+            # Restore from backup on failure
+            elif backup_path and backup_path.exists():
+                import shutil
+
+                shutil.copy2(backup_path, claude_md_path)
+                self.console.print(
+                    "[yellow]⚠️  Optimization failed - restored from backup[/yellow]\n"
+                )
+
+        except Exception as e:
+            logger.error(f"Failed to optimize CLAUDE.md: {e}")
+            self.console.print(
+                f"[yellow]⚠️  Could not optimize CLAUDE.md: {e}[/yellow]\n"
+            )
+
+    def _estimate_tokens(self, text: str) -> int:
+        """Estimate token count for text (rough approximation).
+
+        Uses a simple heuristic: ~4 characters per token for English text.
+        This is a rough estimate but sufficient for displaying progress.
+        """
+        return len(text) // 4
+
+    def _deploy_pm_skills(self) -> None:
+        """Deploy PM skills templates to project .claude-mpm directory.
+
+        Copies PM skills from bundled templates to .claude-mpm/skills/pm/
+        with version tracking and checksum validation.
+        """
+        try:
+            from rich.progress import BarColumn, Progress, TaskProgressColumn, TextColumn
+
+            from claude_mpm.services.pm_skills_deployer import PMSkillsDeployerService
+
+            deployer = PMSkillsDeployerService()
+
+            # Use progress bar for deployment
+            with Progress(
+                TextColumn("[cyan]Deploying PM skills[/cyan]"),
+                BarColumn(bar_width=30),
+                TaskProgressColumn(),
+                TextColumn("[dim]{task.description}[/dim]"),
+                console=self.console,
+            ) as progress:
+                task = progress.add_task("", total=None)  # Unknown total initially
+
+                def update_progress(skill_name: str, current: int, total: int) -> None:
+                    progress.update(task, total=total, completed=current, description=skill_name)
+
+                result = deployer.deploy_pm_skills(
+                    self.project_path, progress_callback=update_progress
+                )
+
+            if result.success:
+                if result.deployed:
+                    self.console.print(
+                        f"[green]✓ Deployed {len(result.deployed)} PM skills[/green]"
+                    )
+
+                if result.skipped:
+                    self.console.print(
+                        f"[dim]  Skipped {len(result.skipped)} (already deployed)[/dim]"
+                    )
+
+                if result.errors:
+                    self.console.print(f"[yellow]⚠️  {len(result.errors)} errors:[/yellow]")
+                    for error in result.errors[:3]:  # Show first 3 errors
+                        self.console.print(f"  • {error['skill']}: {error['error']}")
+            else:
+                self.console.print(
+                    f"[yellow]⚠️  PM skills deployment had errors: {result.message}[/yellow]"
+                )
+
+        except ImportError as e:
+            logger.warning(f"PM skills deployer not available: {e}")
+            self.console.print(
+                "[yellow]⚠️  PM skills deployment skipped (service not available)[/yellow]"
+            )
+        except Exception as e:
+            logger.error(f"Failed to deploy PM skills: {e}")
+            self.console.print(
+                f"[yellow]⚠️  PM skills deployment failed: {e}[/yellow]"
+            )
+
 
 __all__ = ["MPMInitCommand"]

claude_mpm/cli/commands/mpm_init/git_activity.py

@@ -218,14 +218,14 @@ def export_activity_report(project_path: Path, report: Dict, export_path: str) -
     content = f"""# Activity Report
 
 **Generated**: {timestamp}
-**Analysis Period**: {report.get('period', 'Last 30 days')}
+**Analysis Period**: {report.get("period", "Last 30 days")}
 
 ## Summary
 
-- **Total Commits**: {summary.get('total_commits', 0)}
-- **Active Contributors**: {summary.get('total_authors', 0)}
-- **Files Modified**: {summary.get('files_changed', 0)}
-- **Current Branch**: {summary.get('current_branch', 'unknown')}
+- **Total Commits**: {summary.get("total_commits", 0)}
+- **Active Contributors**: {summary.get("total_authors", 0)}
+- **Files Modified**: {summary.get("files_changed", 0)}
+- **Current Branch**: {summary.get("current_branch", "unknown")}
 """
 
     if summary.get("has_uncommitted"):
@@ -301,13 +301,13 @@ def append_activity_notes(claude_md_path: Path, report: Dict) -> None:
 ## 📈 Recent Activity Summary
 
 **Last Updated**: {timestamp}
-**Analysis Period**: {report.get('period', 'Last 30 days')}
+**Analysis Period**: {report.get("period", "Last 30 days")}
 
 ### Activity Metrics
-- **Commits**: {summary.get('total_commits', 0)}
-- **Contributors**: {summary.get('total_authors', 0)}
-- **Files Changed**: {summary.get('files_changed', 0)}
-- **Current Branch**: {summary.get('current_branch', 'unknown')}
+- **Commits**: {summary.get("total_commits", 0)}
+- **Contributors**: {summary.get("total_authors", 0)}
+- **Files Changed**: {summary.get("files_changed", 0)}
+- **Current Branch**: {summary.get("current_branch", "unknown")}
 """
 
     if summary.get("has_uncommitted"):