claude-mpm 5.4.55__py3-none-any.whl → 5.4.85__py3-none-any.whl

claude_mpm/constants.py

@@ -169,6 +169,7 @@ class SkillsCommands(str, Enum):
     INFO = "info"
     CONFIG = "config"
     CONFIGURE = "configure"  # Interactive skills selection (like agents configure)
+    SELECT = "select"  # Interactive topic-grouped skill selector
     # GitHub deployment commands
     DEPLOY_FROM_GITHUB = "deploy-github"
     LIST_AVAILABLE = "list-available"

claude_mpm/core/claude_runner.py

@@ -730,7 +730,7 @@ Use these agents to delegate specialized work via the Task tool.
                     import json
 
                     settings = json.loads(settings_file.read_text())
-                    if settings.get("activeOutputStyle") == "claude-mpm":
+                    if settings.get("activeOutputStyle") == "Claude MPM":
                         # Already active, check if file exists
                         output_style_file = (
                             Path.home() / ".claude" / "output-styles" / "claude-mpm.md"
@@ -740,7 +740,7 @@ Use these agents to delegate specialized work via the Task tool.
                                 "Output style 'Claude MPM' already deployed and active"
                             )
                             return
-                except Exception:
+                except Exception:  # nosec B110
                     pass  # Continue with deployment if we can't read settings
 
             # Read the OUTPUT_STYLE.md content if it exists

claude_mpm/core/interactive_session.py

@@ -12,7 +12,7 @@ of InteractiveSession without circular dependency issues.
 
 import contextlib
 import os
-import subprocess
+import subprocess  # nosec B404
 import uuid
 from pathlib import Path
 from typing import TYPE_CHECKING, Any, Dict, Optional, Tuple
@@ -359,19 +359,19 @@ class InteractiveSession:
                 osm = self.runner.framework_loader.output_style_manager
                 if osm:
                     if osm.claude_version and osm.supports_output_styles():
-                        # Check if claude-mpm style is active
+                        # Check if Claude MPM style is active
                         settings_file = osm.settings_file
                         if settings_file.exists():
                             import json
 
                             settings = json.loads(settings_file.read_text())
                             active_style = settings.get("activeOutputStyle")
-                            if active_style == "claude-mpm":
-                                return "Output Style: claude-mpm ✅"
+                            if active_style in ("Claude MPM", "Claude MPM Teacher"):
+                                return f"Output Style: {active_style} ✅"
                             return f"Output Style: {active_style or 'none'}"
                         return "Output Style: Available"
                     return "Output Style: Injected (legacy)"
-        except Exception:
+        except Exception:  # nosec B110
             pass
         return None
 
@@ -582,7 +582,7 @@ class InteractiveSession:
             )
 
         # This will not return if successful
-        os.execvpe(cmd[0], cmd, env)
+        os.execvpe(cmd[0], cmd, env)  # nosec B606
         return False  # Only reached on failure
 
     def _launch_subprocess_mode(self, cmd: list, env: dict) -> bool:
@@ -641,7 +641,7 @@ class InteractiveSession:
             cmd = environment["command"]
             env = environment["environment"]
 
-            result = subprocess.run(
+            result = subprocess.run(  # nosec B603
                 cmd, stdin=None, stdout=None, stderr=None, env=env, check=False
             )
 

claude_mpm/core/output_style_manager.py

@@ -13,7 +13,7 @@ Users can change it if they want, and the system will respect their choice.
 
 import json
 import re
-import subprocess
+import subprocess  # nosec B404
 from pathlib import Path
 from typing import Any, Dict, Literal, Optional, TypedDict, cast
 
@@ -27,7 +27,7 @@ _CACHED_CLAUDE_VERSION: Optional[str] = None
 _VERSION_DETECTED: bool = False
 
 # Output style types
-OutputStyleType = Literal["professional", "teaching"]
+OutputStyleType = Literal["professional", "teaching", "founders"]
 
 
 class StyleConfig(TypedDict):
@@ -41,9 +41,10 @@ class StyleConfig(TypedDict):
 class OutputStyleManager:
     """Manages output style deployment and version-based handling.
 
-    Supports two output styles:
+    Supports three output styles:
     - professional: Default Claude MPM style (claude-mpm.md)
-    - teaching: Adaptive teaching mode (claude-mpm-teach.md)
+    - teaching: Adaptive teaching mode (claude-mpm-teacher.md)
+    - founders: Non-technical mode for startup founders (claude-mpm-founders.md)
     """
 
     def __init__(self) -> None:
@@ -51,8 +52,8 @@ class OutputStyleManager:
         self.logger = get_logger("output_style_manager")  # type: ignore[misc]
         self.claude_version = self._detect_claude_version()
 
-        # Deploy to ~/.claude/styles/ directory (NOT output-styles/)
-        self.output_style_dir = Path.home() / ".claude" / "styles"
+        # Deploy to ~/.claude/output-styles/ directory (official Claude Code location)
+        self.output_style_dir = Path.home() / ".claude" / "output-styles"
         self.settings_file = Path.home() / ".claude" / "settings.json"
 
         # Style definitions
@@ -62,14 +63,21 @@ class OutputStyleManager:
                 / "agents"
                 / "CLAUDE_MPM_OUTPUT_STYLE.md",
                 target=self.output_style_dir / "claude-mpm.md",
-                name="claude-mpm",
+                name="Claude MPM",
             ),
             "teaching": StyleConfig(
                 source=Path(__file__).parent.parent
                 / "agents"
                 / "CLAUDE_MPM_TEACHER_OUTPUT_STYLE.md",
-                target=self.output_style_dir / "claude-mpm-teach.md",
-                name="claude-mpm-teach",
+                target=self.output_style_dir / "claude-mpm-teacher.md",
+                name="Claude MPM Teacher",
+            ),
+            "founders": StyleConfig(
+                source=Path(__file__).parent.parent
+                / "agents"
+                / "CLAUDE_MPM_FOUNDERS_OUTPUT_STYLE.md",
+                target=self.output_style_dir / "claude-mpm-founders.md",
+                name="Claude MPM Founders",
             ),
         }
 
@@ -93,7 +101,7 @@ class OutputStyleManager:
 
         try:
             # Run claude --version command
-            result = subprocess.run(
+            result = subprocess.run(  # nosec B603 B607
                 ["claude", "--version"],
                 capture_output=True,
                 text=True,
@@ -300,12 +308,12 @@ class OutputStyleManager:
             self.logger.error(f"Failed to deploy {style} style: {e}")
             return False
 
-    def _activate_output_style(self, style_name: str = "claude-mpm") -> bool:
+    def _activate_output_style(self, style_name: str = "Claude MPM") -> bool:
         """
         Update Claude Code settings to activate a specific output style.
 
         Args:
-            style_name: Name of the style to activate (e.g., "claude-mpm", "claude-mpm-teach")
+            style_name: Name of the style to activate (e.g., "Claude MPM", "Claude MPM Teacher")
 
         Returns:
             True if activated successfully, False otherwise
@@ -443,7 +451,7 @@ class OutputStyleManager:
 
         # Activate the default style if requested
         if activate_default and results.get("professional", False):
-            self._activate_output_style("claude-mpm")
+            self._activate_output_style("Claude MPM")
 
         return results
 

claude_mpm/core/unified_config.py

@@ -16,7 +16,7 @@ Design Principles:
 from pathlib import Path
 from typing import Any, Dict, List, Optional, Union
 
-from pydantic import BaseModel, Field, validator
+from pydantic import BaseModel, Field, field_validator
 from pydantic_settings import BaseSettings
 
 from .exceptions import ConfigurationError
@@ -54,8 +54,9 @@ class LoggingConfig(BaseModel):
         default=True, description="Enable console logging"
     )
 
-    @validator("level")
-    def validate_log_level(self, v):
+    @field_validator("level")
+    @classmethod
+    def validate_log_level(cls, v):
         valid_levels = ["DEBUG", "INFO", "WARNING", "ERROR", "CRITICAL"]
         if v.upper() not in valid_levels:
             raise ValueError(f"Invalid log level. Must be one of: {valid_levels}")
@@ -65,8 +66,32 @@ class LoggingConfig(BaseModel):
 class AgentConfig(BaseModel):
     """Agent system configuration."""
 
+    # Explicit deployment lists (simplified model)
+    enabled: List[str] = Field(
+        default_factory=list,
+        description="Explicit list of agent IDs to deploy (empty = use auto_discover)",
+    )
+
+    # Required agents that are always deployed
+    required: List[str] = Field(
+        default_factory=lambda: [
+            "research",
+            "mpm-skills-manager",
+            "mpm-agent-manager",
+            "memory-manager",
+            "ticketing",
+        ],
+        description="Agents that are always deployed (core system agents)",
+    )
+
+    include_universal: bool = Field(
+        default=True,
+        description="Auto-include all agents with 'universal' toolchain/category",
+    )
+
     auto_discover: bool = Field(
-        default=True, description="Enable automatic agent discovery"
+        default=False,
+        description="Enable automatic agent discovery (deprecated, use enabled list)",
     )
     precedence: List[str] = Field(
         default=["project", "user", "system"], description="Agent precedence order"
@@ -239,6 +264,21 @@ class DocumentationConfig(BaseModel):
     )
 
 
+class SkillConfig(BaseModel):
+    """Skill system configuration."""
+
+    # Explicit deployment lists (simplified model)
+    enabled: List[str] = Field(
+        default_factory=list,
+        description="Explicit list of skill IDs to deploy (includes agent dependencies)",
+    )
+
+    auto_detect_dependencies: bool = Field(
+        default=True,
+        description="Automatically include skills required by enabled agents",
+    )
+
+
 class UnifiedConfig(BaseSettings):
     """
     Unified configuration model for Claude MPM.
@@ -258,6 +298,7 @@ class UnifiedConfig(BaseSettings):
     network: NetworkConfig = Field(default_factory=NetworkConfig)
     logging: LoggingConfig = Field(default_factory=LoggingConfig)
     agents: AgentConfig = Field(default_factory=AgentConfig)
+    skills: SkillConfig = Field(default_factory=SkillConfig)
     memory: MemoryConfig = Field(default_factory=MemoryConfig)
     security: SecurityConfig = Field(default_factory=SecurityConfig)
     performance: PerformanceConfig = Field(default_factory=PerformanceConfig)
@@ -287,8 +328,9 @@ class UnifiedConfig(BaseSettings):
         validate_assignment = True
         extra = "allow"  # Allow extra fields for backward compatibility
 
-    @validator("environment")
-    def validate_environment(self, v):
+    @field_validator("environment")
+    @classmethod
+    def validate_environment(cls, v):
         valid_envs = ["development", "testing", "production"]
         if v not in valid_envs:
             raise ValueError(f"Invalid environment. Must be one of: {valid_envs}")
@@ -554,12 +596,12 @@ class ConfigurationService:
                 import yaml
 
                 with file_path.open("w") as f:
-                    yaml.dump(self._config.dict(), f, default_flow_style=False)
+                    yaml.dump(self._config.model_dump(), f, default_flow_style=False)
             elif format.lower() == "json":
                 import json
 
                 with file_path.open("w") as f:
-                    json.dump(self._config.dict(), f, indent=2)
+                    json.dump(self._config.model_dump(), f, indent=2)
             else:
                 raise ConfigurationError(f"Unsupported export format: {format}")
 

claude_mpm/core/unified_paths.py

@@ -1,4 +1,3 @@
-#!/usr/bin/env python3
 """
 Unified Path Management System for Claude MPM
 ==============================================
@@ -36,6 +35,24 @@ from claude_mpm.core.logging_utils import get_logger
 logger = get_logger(__name__)
 
 
+def _safe_cwd() -> Path:
+    """Safely get the current working directory.
+
+    If the current directory no longer exists (deleted/moved), fall back to home directory.
+    This prevents FileNotFoundError when Path.cwd() is called from a deleted directory.
+
+    Returns:
+        Path: Current working directory, or home directory if cwd doesn't exist
+    """
+    try:
+        return Path.cwd()
+    except (FileNotFoundError, OSError) as e:
+        logger.debug(
+            f"Current directory doesn't exist ({e}), falling back to home directory"
+        )
+        return Path.home()
+
+
 class PathType(Enum):
     """Enumeration of different path types for categorization."""
 
@@ -95,7 +112,7 @@ class PathContext:
 
             # Additional check: If we're running from within a claude-mpm development directory
             # This handles the case where pipx claude-mpm is invoked from within the dev directory
-            cwd = Path.cwd()
+            cwd = _safe_cwd()
             current = cwd
             for _ in range(5):  # Check up to 5 levels up from current directory
                 if (current / "pyproject.toml").exists() and (
@@ -114,7 +131,7 @@ class PathContext:
                             # Verify this is a development setup by checking for key files
                             if (current / "scripts" / "claude-mpm").exists():
                                 return True
-                    except Exception:
+                    except Exception:  # nosec B110
                         pass
                 if current == current.parent:
                     break
@@ -140,7 +157,7 @@ class PathContext:
                                         f"Found editable install via .pth file: {pth_file}"
                                     )
                                     return True
-                            except Exception:
+                            except Exception:  # nosec B112
                                 continue
 
                         # Check for egg-link files
@@ -156,7 +173,7 @@ class PathContext:
                                             f"Found editable install via egg-link: {egg_link}"
                                         )
                                         return True
-                                except Exception:
+                                except Exception:  # nosec B112
                                     continue
             except ImportError:
                 pass
@@ -186,7 +203,7 @@ class PathContext:
 
         # Check if current working directory is a claude-mpm development project
         # This handles the case where pipx claude-mpm is run from within the dev directory
-        cwd = Path.cwd()
+        cwd = _safe_cwd()
         current = cwd
         for _ in range(5):  # Check up to 5 levels up from current directory
             if (current / "pyproject.toml").exists() and (
@@ -206,7 +223,7 @@ class PathContext:
                             "Using development mode for local source preference"
                         )
                         return DeploymentContext.DEVELOPMENT
-                except Exception:
+                except Exception:  # nosec B110
                     pass
             if current == current.parent:
                 break
@@ -228,7 +245,7 @@ class PathContext:
                 if "pipx" in str(module_path):
                     # Running via pipx but from within a development directory
                     # Use development mode to prefer local source over pipx installation
-                    cwd = Path.cwd()
+                    cwd = _safe_cwd()
                     current = cwd
                     for _ in range(5):
                         if (current / "src" / "claude_mpm").exists() and (
@@ -346,7 +363,7 @@ class UnifiedPathManager:
             ):
                 # For development mode, first check if we're running from within a dev directory
                 # This handles the case where pipx is invoked from a development directory
-                cwd = Path.cwd()
+                cwd = _safe_cwd()
                 current = cwd
                 for _ in range(5):
                     if (current / "src" / "claude_mpm").exists() and (
@@ -360,7 +377,7 @@ class UnifiedPathManager:
                                     f"Found framework root via cwd at {current}"
                                 )
                                 return current
-                        except Exception:
+                        except Exception:  # nosec B110
                             pass
                     if current == current.parent:
                         break
@@ -403,7 +420,7 @@ class UnifiedPathManager:
     @lru_cache(maxsize=1)
     def project_root(self) -> Path:
         """Get the current project root directory."""
-        current = Path.cwd()
+        current = _safe_cwd()
         while current != current.parent:
             for marker in self._project_markers:
                 if (current / marker).exists():
@@ -413,7 +430,7 @@ class UnifiedPathManager:
 
         # Fallback to current directory
         logger.warning("Could not find project root, using current directory")
-        return Path.cwd()
+        return _safe_cwd()
 
     @property
     def package_root(self) -> Path:
@@ -562,7 +579,7 @@ class UnifiedPathManager:
         self, filename: str, start_path: Optional[Path] = None
     ) -> Optional[Path]:
         """Search for a file by traversing up the directory tree."""
-        current = start_path or Path.cwd()
+        current = start_path or _safe_cwd()
 
         while current != current.parent:
             candidate = current / filename

claude_mpm/services/agents/deployment/agent_template_builder.py

@@ -586,6 +586,14 @@ class AgentTemplateBuilder:
             tags_str = ",".join(metadata["tags"])
             frontmatter_lines.append(f"tags: {tags_str}")
 
+        # CRITICAL: Preserve skills field from template for selective skill deployment
+        # Skills are used by startup.py to determine which skills to deploy
+        skills = template_data.get("skills", [])
+        if skills and isinstance(skills, list):
+            frontmatter_lines.append("skills:")
+            for skill in skills:
+                frontmatter_lines.append(f"- {skill}")
+
         frontmatter_lines.extend(
             [
                 "---",