claude-mpm 4.15.0__tar.gz → 4.15.2__tar.gz

claude_mpm-4.15.2/BUILD_NUMBER

@@ -0,0 +1 @@
+473

{claude_mpm-4.15.0 → claude_mpm-4.15.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: claude-mpm
-Version: 4.15.0
+Version: 4.15.2
 Summary: Claude Multi-Agent Project Manager - Orchestrate Claude with agent delegation and ticket tracking
 Author-email: Bob Matsuoka <bob@matsuoka.com>
 Maintainer: Claude MPM Team

claude_mpm-4.15.2/VERSION

@@ -0,0 +1 @@
+4.15.2

{claude_mpm-4.15.0 → claude_mpm-4.15.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "claude-mpm"
-version = "4.15.0"
+version = "4.15.2"
 description = "Claude Multi-Agent Project Manager - Orchestrate Claude with agent delegation and ticket tracking"
 readme = "README.md"
 requires-python = ">=3.8"
@@ -84,7 +84,7 @@ ignore-imports = false
 
 [tool.commitizen]
 name = "cz_conventional_commits"
-version = "4.15.0"
+version = "4.15.2"
 version_files = [ "VERSION", "src/claude_mpm/VERSION", "package.json:version",]
 tag_format = "v$version"
 update_changelog_on_bump = true

claude_mpm-4.15.2/src/claude_mpm/VERSION

@@ -0,0 +1 @@
+4.15.2

{claude_mpm-4.15.0 → claude_mpm-4.15.2}/src/claude_mpm/agents/OUTPUT_STYLE.md

@@ -76,7 +76,50 @@ You are a PROJECT MANAGER whose SOLE PURPOSE is to delegate work to specialized
 - **No simplification** without explicit user request
 - **No mocks** outside test environments
 - **Complete implementations** only - no placeholders
-- **FORBIDDEN**: "Excellent!", "Perfect!", "Amazing!", "You're absolutely right!" (and similar unwarrented phrasing)
+- **ABSOLUTELY FORBIDDEN - CRITICAL VIOLATION**:
+  - ❌ "Excellent!", "Perfect!", "Amazing!", "Fantastic!", "Wonderful!"
+  - ❌ "You're absolutely right!", "You're exactly right!", "Exactly as requested!"
+  - ❌ "Great job!", "Well done!", "Outstanding!", "Superb!", "Brilliant!"
+  - ❌ "I appreciate", "Thank you for"
+  - ❌ ANY unwarranted enthusiasm or excessive affirmations
+
+  **RULE**: Only use enthusiasm when objectively warranted (e.g., actual exceptional performance, breakthrough results)
+  **DEFAULT**: Maintain neutral, professional acknowledgment tone
+
+## 🚨 Enthusiasm Enforcement Protocol
+
+**Before ANY affirmative response, evaluate**:
+
+1. **Is this objectively exceptional?** (>99th percentile performance)
+   - ✅ Yes → Enthusiasm acceptable
+   - ❌ No → Use neutral acknowledgment
+
+2. **Does this warrant enthusiasm?**
+   - ✅ Breakthrough after significant challenge → "Outstanding result"
+   - ❌ Routine task completion → "Task completed"
+
+3. **Is this factual acknowledgment?**
+   - ✅ "Changes applied successfully" (factual)
+   - ❌ "Perfect! Changes applied!" (unwarranted)
+
+### Violation Examples with Corrections
+
+| Situation | ❌ FORBIDDEN Response | ✅ CORRECT Response |
+|-----------|---------------------|-------------------|
+| User provides simple request | "Perfect! I'll handle that..." | "Confirmed. Delegating to [Agent]..." |
+| User corrects PM error | "You're absolutely right!" | "Confirmed. Correcting approach..." |
+| Agent completes routine task | "Excellent work!" | "Task completed. Ready for QA." |
+| Routine delegation | "Amazing! Engineer will fix..." | "Delegating to Engineer for implementation." |
+| User confirms instruction | "Wonderful! Proceeding..." | "Understood. Proceeding with..." |
+
+### Acceptable Enthusiasm (Rare Cases <1%)
+
+Use enthusiasm ONLY when:
+- ✅ User achieves breakthrough after multiple failures
+- ✅ System achieves significant measurable improvement (>2x)
+- ✅ User provides critical insight that unblocks major issue
+
+**Estimated Usage**: <1% of all responses
 
 ## Error Handling Protocol
 
@@ -100,11 +143,13 @@ You are a PROJECT MANAGER whose SOLE PURPOSE is to delegate work to specialized
 ## Professional Communication
 
 - Maintain neutral, professional tone as default
-- Avoid overeager enthusiasm, NEVER SAY "You're exactly right!" (or similar)
-- Use appropriate acknowledgments
+- Avoid overeager enthusiasm - refer to Enthusiasm Enforcement Protocol above
+- NEVER use unwarranted affirmations like "You're exactly right!", "Perfect!", or similar
+- Use appropriate acknowledgments: "Understood", "Confirmed", "Noted"
 - Never fallback to simpler solutions without explicit user instruction
 - Never use mock implementations outside test environments
 - Provide clear, actionable feedback on delegation results
+- Follow the <1% enthusiasm threshold strictly
 
 ## Critical Operating Principles
 

{claude_mpm-4.15.0 → claude_mpm-4.15.2}/src/claude_mpm/services/diagnostics/__init__.py

@@ -13,6 +13,6 @@ DESIGN DECISIONS:
 
 from .diagnostic_runner import DiagnosticRunner
 from .doctor_reporter import DoctorReporter
-from .models import DiagnosticResult, DiagnosticStatus
+from .models import DiagnosticResult
 
-__all__ = ["DiagnosticResult", "DiagnosticRunner", "DiagnosticStatus", "DoctorReporter"]
+__all__ = ["DiagnosticResult", "DiagnosticRunner", "DoctorReporter"]

{claude_mpm-4.15.0 → claude_mpm-4.15.2}/src/claude_mpm/services/diagnostics/checks/agent_check.py

@@ -6,7 +6,8 @@ WHY: Verify that agents are properly deployed, up-to-date, and functioning corre
 
 from pathlib import Path
 
-from ..models import DiagnosticResult, DiagnosticStatus
+from ....core.enums import OperationResult, ValidationSeverity
+from ..models import DiagnosticResult
 from .base_check import BaseDiagnosticCheck
 
 
@@ -64,29 +65,29 @@ class AgentCheck(BaseDiagnosticCheck):
             available_count = details["available_count"]
 
             if deployed_count == 0:
-                status = DiagnosticStatus.ERROR
+                status = ValidationSeverity.ERROR
                 message = f"No agents deployed (0/{available_count} available)"
                 fix_command = "claude-mpm agents deploy"
                 fix_description = "Deploy all available agents"
             elif deployed_count < available_count:
-                status = DiagnosticStatus.WARNING
+                status = ValidationSeverity.WARNING
                 message = f"{deployed_count}/{available_count} agents deployed"
                 fix_command = "claude-mpm agents deploy"
                 fix_description = (
                     f"Deploy remaining {available_count - deployed_count} agents"
                 )
-            elif any(r.status == DiagnosticStatus.ERROR for r in sub_results):
-                status = DiagnosticStatus.ERROR
+            elif any(r.status == ValidationSeverity.ERROR for r in sub_results):
+                status = ValidationSeverity.ERROR
                 message = "Agents have critical issues"
                 fix_command = None
                 fix_description = None
-            elif any(r.status == DiagnosticStatus.WARNING for r in sub_results):
-                status = DiagnosticStatus.WARNING
+            elif any(r.status == ValidationSeverity.WARNING for r in sub_results):
+                status = ValidationSeverity.WARNING
                 message = "Agents have minor issues"
                 fix_command = None
                 fix_description = None
             else:
-                status = DiagnosticStatus.OK
+                status = OperationResult.SUCCESS
                 message = f"All {deployed_count} agents properly deployed"
                 fix_command = None
                 fix_description = None
@@ -104,7 +105,7 @@ class AgentCheck(BaseDiagnosticCheck):
         except Exception as e:
             return DiagnosticResult(
                 category=self.category,
-                status=DiagnosticStatus.ERROR,
+                status=ValidationSeverity.ERROR,
                 message=f"Agent check failed: {e!s}",
                 details={"error": str(e)},
             )
@@ -127,7 +128,7 @@ class AgentCheck(BaseDiagnosticCheck):
             # Neither exists, default to user directory for error message
             return DiagnosticResult(
                 category="Deployed Agents",
-                status=DiagnosticStatus.ERROR,
+                status=ValidationSeverity.ERROR,
                 message="No agents directory found (checked project and user)",
                 details={
                     "project_path": str(project_agents_dir),
@@ -144,7 +145,7 @@ class AgentCheck(BaseDiagnosticCheck):
         if not agent_files:
             return DiagnosticResult(
                 category="Deployed Agents",
-                status=DiagnosticStatus.ERROR,
+                status=ValidationSeverity.ERROR,
                 message=f"No agents deployed in {location} directory",
                 details={"path": str(agents_dir), "location": location, "count": 0},
                 fix_command="claude-mpm agents deploy",
@@ -159,7 +160,7 @@ class AgentCheck(BaseDiagnosticCheck):
         if missing_core:
             return DiagnosticResult(
                 category="Deployed Agents",
-                status=DiagnosticStatus.WARNING,
+                status=ValidationSeverity.WARNING,
                 message=f"Missing core agents in {location}: {', '.join(missing_core)}",
                 details={
                     "path": str(agents_dir),
@@ -174,7 +175,7 @@ class AgentCheck(BaseDiagnosticCheck):
 
         return DiagnosticResult(
             category="Deployed Agents",
-            status=DiagnosticStatus.OK,
+            status=OperationResult.SUCCESS,
             message=f"{len(agent_files)} agents deployed ({location} level)",
             details={
                 "path": str(agents_dir),
@@ -205,7 +206,7 @@ class AgentCheck(BaseDiagnosticCheck):
             else:
                 return DiagnosticResult(
                     category="Agent Versions",
-                    status=DiagnosticStatus.SKIPPED,
+                    status=OperationResult.SKIPPED,
                     message="No agents to check",
                     details={},
                 )
@@ -224,7 +225,7 @@ class AgentCheck(BaseDiagnosticCheck):
             if outdated:
                 return DiagnosticResult(
                     category="Agent Versions",
-                    status=DiagnosticStatus.WARNING,
+                    status=ValidationSeverity.WARNING,
                     message=f"{len(outdated)} agent(s) outdated",
                     details={"outdated": outdated, "checked": checked},
                     fix_command="claude-mpm agents update",
@@ -234,14 +235,14 @@ class AgentCheck(BaseDiagnosticCheck):
             if checked == 0:
                 return DiagnosticResult(
                     category="Agent Versions",
-                    status=DiagnosticStatus.WARNING,
+                    status=ValidationSeverity.WARNING,
                     message="No agents to check",
                     details={"checked": 0},
                 )
 
             return DiagnosticResult(
                 category="Agent Versions",
-                status=DiagnosticStatus.OK,
+                status=OperationResult.SUCCESS,
                 message=f"All {checked} agents up-to-date",
                 details={"checked": checked},
             )
@@ -249,7 +250,7 @@ class AgentCheck(BaseDiagnosticCheck):
         except Exception as e:
             return DiagnosticResult(
                 category="Agent Versions",
-                status=DiagnosticStatus.WARNING,
+                status=ValidationSeverity.WARNING,
                 message=f"Could not check versions: {e!s}",
                 details={"error": str(e)},
             )
@@ -273,7 +274,7 @@ class AgentCheck(BaseDiagnosticCheck):
             else:
                 return DiagnosticResult(
                     category="Agent Validation",
-                    status=DiagnosticStatus.SKIPPED,
+                    status=OperationResult.SKIPPED,
                     message="No agents to validate",
                     details={},
                 )
@@ -301,14 +302,14 @@ class AgentCheck(BaseDiagnosticCheck):
             if invalid:
                 return DiagnosticResult(
                     category="Agent Validation",
-                    status=DiagnosticStatus.WARNING,
+                    status=ValidationSeverity.WARNING,
                     message=f"{len(invalid)} validation issue(s)",
                     details={"issues": invalid, "validated": validated},
                 )
 
             return DiagnosticResult(
                 category="Agent Validation",
-                status=DiagnosticStatus.OK,
+                status=OperationResult.SUCCESS,
                 message=f"All {validated} agents valid",
                 details={"validated": validated},
             )
@@ -316,7 +317,7 @@ class AgentCheck(BaseDiagnosticCheck):
         except Exception as e:
             return DiagnosticResult(
                 category="Agent Validation",
-                status=DiagnosticStatus.WARNING,
+                status=ValidationSeverity.WARNING,
                 message=f"Validation failed: {e!s}",
                 details={"error": str(e)},
             )
@@ -358,14 +359,14 @@ class AgentCheck(BaseDiagnosticCheck):
         if issues:
             return DiagnosticResult(
                 category="Common Issues",
-                status=DiagnosticStatus.WARNING,
+                status=ValidationSeverity.WARNING,
                 message=f"{len(issues)} issue(s) found",
                 details={"issues": issues},
             )
 
         return DiagnosticResult(
             category="Common Issues",
-            status=DiagnosticStatus.OK,
+            status=OperationResult.SUCCESS,
             message="No common issues detected",
             details={},
         )

{claude_mpm-4.15.0 → claude_mpm-4.15.2}/src/claude_mpm/services/diagnostics/checks/claude_code_check.py

@@ -9,7 +9,8 @@ import json
 import subprocess
 from pathlib import Path
 
-from ..models import DiagnosticResult, DiagnosticStatus
+from ....core.enums import OperationResult, ValidationSeverity
+from ..models import DiagnosticResult
 from .base_check import BaseDiagnosticCheck
 
 
@@ -33,9 +34,9 @@ class ClaudeCodeCheck(BaseDiagnosticCheck):
             # Check if Claude Code CLI is installed
             install_result = self._check_installation()
             sub_results.append(install_result)
-            details["installed"] = install_result.status == DiagnosticStatus.OK
+            details["installed"] = install_result.status == OperationResult.SUCCESS
 
-            if install_result.status == DiagnosticStatus.OK:
+            if install_result.status == OperationResult.SUCCESS:
                 # Check version compatibility
                 version_result = self._check_version()
                 sub_results.append(version_result)
@@ -49,17 +50,17 @@ class ClaudeCodeCheck(BaseDiagnosticCheck):
                 # Check MCP integration
                 mcp_result = self._check_mcp_integration()
                 sub_results.append(mcp_result)
-                details["mcp_configured"] = mcp_result.status == DiagnosticStatus.OK
+                details["mcp_configured"] = mcp_result.status == OperationResult.SUCCESS
 
             # Determine overall status
-            if any(r.status == DiagnosticStatus.ERROR for r in sub_results):
-                status = DiagnosticStatus.ERROR
+            if any(r.status == ValidationSeverity.ERROR for r in sub_results):
+                status = ValidationSeverity.ERROR
                 message = "Claude Code CLI has critical issues"
-            elif any(r.status == DiagnosticStatus.WARNING for r in sub_results):
-                status = DiagnosticStatus.WARNING
+            elif any(r.status == ValidationSeverity.WARNING for r in sub_results):
+                status = ValidationSeverity.WARNING
                 message = "Claude Code CLI needs configuration"
             else:
-                status = DiagnosticStatus.OK
+                status = OperationResult.SUCCESS
                 message = "Claude Code CLI properly configured"
 
             return DiagnosticResult(
@@ -73,7 +74,7 @@ class ClaudeCodeCheck(BaseDiagnosticCheck):
         except Exception as e:
             return DiagnosticResult(
                 category=self.category,
-                status=DiagnosticStatus.ERROR,
+                status=ValidationSeverity.ERROR,
                 message=f"Claude Code CLI check failed: {e!s}",
                 details={"error": str(e)},
             )
@@ -92,7 +93,7 @@ class ClaudeCodeCheck(BaseDiagnosticCheck):
             if result.returncode == 0:
                 return DiagnosticResult(
                     category="Claude Code CLI Installation",
-                    status=DiagnosticStatus.OK,
+                    status=OperationResult.SUCCESS,
                     message="Claude Code CLI is installed and accessible",
                     details={
                         "installed": True,
@@ -115,14 +116,14 @@ class ClaudeCodeCheck(BaseDiagnosticCheck):
             if path.exists() and path.is_file():
                 return DiagnosticResult(
                     category="Claude Code CLI Installation",
-                    status=DiagnosticStatus.OK,
+                    status=OperationResult.SUCCESS,
                     message=f"Claude Code CLI found at {path}",
                     details={"installed": True, "path": str(path)},
                 )
 
         return DiagnosticResult(
             category="Claude Code CLI Installation",
-            status=DiagnosticStatus.ERROR,
+            status=ValidationSeverity.ERROR,
             message="Claude Code CLI not found",
             details={"installed": False},
             fix_description="Install Claude Code CLI from https://claude.ai/code",
@@ -150,7 +151,7 @@ class ClaudeCodeCheck(BaseDiagnosticCheck):
                         break
 
             # Check minimum version requirement (1.0.60+)
-            status = DiagnosticStatus.OK
+            status = OperationResult.SUCCESS
             message = f"Version: {version}"
 
             return DiagnosticResult(
@@ -166,7 +167,7 @@ class ClaudeCodeCheck(BaseDiagnosticCheck):
         except subprocess.SubprocessError as e:
             return DiagnosticResult(
                 category="Claude Code CLI Version",
-                status=DiagnosticStatus.WARNING,
+                status=ValidationSeverity.WARNING,
                 message=f"Could not determine version: {e}",
                 details={"version": "unknown", "error": str(e)},
             )
@@ -178,7 +179,7 @@ class ClaudeCodeCheck(BaseDiagnosticCheck):
         if not style_path.exists():
             return DiagnosticResult(
                 category="Output Style",
-                status=DiagnosticStatus.WARNING,
+                status=ValidationSeverity.WARNING,
                 message="Output style not deployed",
                 details={"deployed": False, "path": str(style_path)},
                 fix_command="claude-mpm deploy-style",
@@ -192,13 +193,13 @@ class ClaudeCodeCheck(BaseDiagnosticCheck):
                 if "Claude MPM Output Style" in content:
                     return DiagnosticResult(
                         category="Output Style",
-                        status=DiagnosticStatus.OK,
+                        status=OperationResult.SUCCESS,
                         message="Output style deployed",
                         details={"deployed": True, "path": str(style_path)},
                     )
                 return DiagnosticResult(
                     category="Output Style",
-                    status=DiagnosticStatus.WARNING,
+                    status=ValidationSeverity.WARNING,
                     message="Output style outdated",
                     details={
                         "deployed": True,
@@ -211,7 +212,7 @@ class ClaudeCodeCheck(BaseDiagnosticCheck):
         except Exception as e:
             return DiagnosticResult(
                 category="Output Style",
-                status=DiagnosticStatus.WARNING,
+                status=ValidationSeverity.WARNING,
                 message=f"Could not check output style: {e!s}",
                 details={"error": str(e)},
             )
@@ -224,7 +225,7 @@ class ClaudeCodeCheck(BaseDiagnosticCheck):
         if not config_path.exists():
             return DiagnosticResult(
                 category="MCP Integration",
-                status=DiagnosticStatus.WARNING,
+                status=ValidationSeverity.WARNING,
                 message="Claude Code CLI config not found",
                 details={"configured": False, "config_path": str(config_path)},
                 fix_command="claude-mpm mcp install",
@@ -239,7 +240,7 @@ class ClaudeCodeCheck(BaseDiagnosticCheck):
                 if "claude-mpm-gateway" in mcp_servers or "claude-mpm" in mcp_servers:
                     return DiagnosticResult(
                         category="MCP Integration",
-                        status=DiagnosticStatus.OK,
+                        status=OperationResult.SUCCESS,
                         message="MCP server configured",
                         details={
                             "configured": True,
@@ -249,7 +250,7 @@ class ClaudeCodeCheck(BaseDiagnosticCheck):
                     )
                 return DiagnosticResult(
                     category="MCP Integration",
-                    status=DiagnosticStatus.WARNING,
+                    status=ValidationSeverity.WARNING,
                     message="MCP server not configured",
                     details={
                         "configured": False,
@@ -263,7 +264,7 @@ class ClaudeCodeCheck(BaseDiagnosticCheck):
         except Exception as e:
             return DiagnosticResult(
                 category="MCP Integration",
-                status=DiagnosticStatus.WARNING,
+                status=ValidationSeverity.WARNING,
                 message=f"Could not check MCP configuration: {e!s}",
                 details={"error": str(e)},
             )

{claude_mpm-4.15.0 → claude_mpm-4.15.2}/src/claude_mpm/services/diagnostics/checks/common_issues_check.py

@@ -8,7 +8,8 @@ providing specific fixes for each issue.
 import json
 from pathlib import Path
 
-from ..models import DiagnosticResult, DiagnosticStatus
+from ....core.enums import OperationResult, ValidationSeverity
+from ..models import DiagnosticResult
 from .base_check import BaseDiagnosticCheck
 
 
@@ -33,7 +34,7 @@ class CommonIssuesCheck(BaseDiagnosticCheck):
             # Check for large .claude.json file
             claude_json_result = self._check_claude_json_size()
             if claude_json_result.has_issues:
-                if claude_json_result.status == DiagnosticStatus.ERROR:
+                if claude_json_result.status == ValidationSeverity.ERROR:
                     issues_found.append(claude_json_result)
                 else:
                     warnings_found.append(claude_json_result)
@@ -62,13 +63,13 @@ class CommonIssuesCheck(BaseDiagnosticCheck):
             total_issues = len(issues_found) + len(warnings_found)
 
             if issues_found:
-                status = DiagnosticStatus.ERROR
+                status = ValidationSeverity.ERROR
                 message = f"{len(issues_found)} critical issue(s), {len(warnings_found)} warning(s)"
             elif warnings_found:
-                status = DiagnosticStatus.WARNING
+                status = ValidationSeverity.WARNING
                 message = f"{len(warnings_found)} known issue(s) detected"
             else:
-                status = DiagnosticStatus.OK
+                status = OperationResult.SUCCESS
                 message = "No known issues detected"
 
             details = {
@@ -90,7 +91,7 @@ class CommonIssuesCheck(BaseDiagnosticCheck):
         except Exception as e:
             return DiagnosticResult(
                 category=self.category,
-                status=DiagnosticStatus.ERROR,
+                status=ValidationSeverity.ERROR,
                 message=f"Common issues check failed: {e!s}",
                 details={"error": str(e)},
             )
@@ -102,7 +103,7 @@ class CommonIssuesCheck(BaseDiagnosticCheck):
         if not claude_json_path.exists():
             return DiagnosticResult(
                 category="Large .claude.json",
-                status=DiagnosticStatus.OK,
+                status=OperationResult.SUCCESS,
                 message="No .claude.json file",
                 details={"exists": False},
             )
@@ -131,7 +132,7 @@ class CommonIssuesCheck(BaseDiagnosticCheck):
             if size_mb > 10:  # Critical: >10MB
                 return DiagnosticResult(
                     category="Large .claude.json",
-                    status=DiagnosticStatus.ERROR,
+                    status=ValidationSeverity.ERROR,
                     message=f"Critical: .claude.json is {size_mb:.1f}MB (causes memory issues)",
                     details=details,
                     fix_command="claude-mpm cleanup-memory",
@@ -140,7 +141,7 @@ class CommonIssuesCheck(BaseDiagnosticCheck):
             if size_mb > 1:  # Warning: >1MB
                 return DiagnosticResult(
                     category="Large .claude.json",
-                    status=DiagnosticStatus.WARNING,
+                    status=ValidationSeverity.WARNING,
                     message=f".claude.json is {size_mb:.1f}MB (may cause memory issues)",
                     details=details,
                     fix_command="claude-mpm cleanup-memory",
@@ -149,7 +150,7 @@ class CommonIssuesCheck(BaseDiagnosticCheck):
 
             return DiagnosticResult(
                 category="Large .claude.json",
-                status=DiagnosticStatus.OK,
+                status=OperationResult.SUCCESS,
                 message=f".claude.json size acceptable ({size_mb:.2f}MB)",
                 details=details,
             )
@@ -157,7 +158,7 @@ class CommonIssuesCheck(BaseDiagnosticCheck):
         except Exception as e:
             return DiagnosticResult(
                 category="Large .claude.json",
-                status=DiagnosticStatus.WARNING,
+                status=ValidationSeverity.WARNING,
                 message=f"Could not check .claude.json: {e!s}",
                 details={"error": str(e)},
             )
@@ -170,7 +171,7 @@ class CommonIssuesCheck(BaseDiagnosticCheck):
         if not memory_dir.exists():
             return DiagnosticResult(
                 category="Memory Usage",
-                status=DiagnosticStatus.OK,
+                status=OperationResult.SUCCESS,
                 message="No project memory directory",
                 details={"exists": False, "path": str(memory_dir)},
             )
@@ -200,7 +201,7 @@ class CommonIssuesCheck(BaseDiagnosticCheck):
             if size_mb > 100:  # >100MB of memory files
                 return DiagnosticResult(
                     category="Memory Usage",
-                    status=DiagnosticStatus.WARNING,
+                    status=ValidationSeverity.WARNING,
                     message=f"High memory usage: {size_mb:.1f}MB in {len(memory_files)} files",
                     details=details,
                     fix_command="claude-mpm memory clean --days 30",
@@ -209,7 +210,7 @@ class CommonIssuesCheck(BaseDiagnosticCheck):
             if old_files:
                 return DiagnosticResult(
                     category="Memory Usage",
-                    status=DiagnosticStatus.WARNING,
+                    status=ValidationSeverity.WARNING,
                     message=f"{len(old_files)} memory file(s) older than 30 days",
                     details=details,
                     fix_command="claude-mpm memory clean --days 30",
@@ -218,7 +219,7 @@ class CommonIssuesCheck(BaseDiagnosticCheck):
 
             return DiagnosticResult(
                 category="Memory Usage",
-                status=DiagnosticStatus.OK,
+                status=OperationResult.SUCCESS,
                 message=f"Memory usage normal ({size_mb:.1f}MB)",
                 details=details,
             )
@@ -226,7 +227,7 @@ class CommonIssuesCheck(BaseDiagnosticCheck):
         except Exception as e:
             return DiagnosticResult(
                 category="Memory Usage",
-                status=DiagnosticStatus.WARNING,
+                status=ValidationSeverity.WARNING,
                 message=f"Could not check memory: {e!s}",
                 details={"error": str(e)},
             )
@@ -257,7 +258,7 @@ class CommonIssuesCheck(BaseDiagnosticCheck):
         if stale_locks:
             return DiagnosticResult(
                 category="Lock Files",
-                status=DiagnosticStatus.WARNING,
+                status=ValidationSeverity.WARNING,
                 message=f"{len(stale_locks)} stale lock file(s) found",
                 details={"stale_locks": stale_locks},
                 fix_command=f"rm {' '.join(stale_locks)}",
@@ -266,7 +267,7 @@ class CommonIssuesCheck(BaseDiagnosticCheck):
 
         return DiagnosticResult(
             category="Lock Files",
-            status=DiagnosticStatus.OK,
+            status=OperationResult.SUCCESS,
             message="No stale lock files",
             details={"stale_locks": []},
         )
@@ -302,7 +303,7 @@ class CommonIssuesCheck(BaseDiagnosticCheck):
         if conflicts:
             return DiagnosticResult(
                 category="Config Conflicts",
-                status=DiagnosticStatus.WARNING,
+                status=ValidationSeverity.WARNING,
                 message=f"{len(conflicts)} configuration conflict(s)",
                 details={"conflicts": conflicts},
                 fix_description="Resolve conflicting configurations",
@@ -310,7 +311,7 @@ class CommonIssuesCheck(BaseDiagnosticCheck):
 
         return DiagnosticResult(
             category="Config Conflicts",
-            status=DiagnosticStatus.OK,
+            status=OperationResult.SUCCESS,
             message="No configuration conflicts",
             details={"conflicts": []},
         )
@@ -322,7 +323,7 @@ class CommonIssuesCheck(BaseDiagnosticCheck):
         if not cache_dir.exists():
             return DiagnosticResult(
                 category="Cache",
-                status=DiagnosticStatus.OK,
+                status=OperationResult.SUCCESS,
                 message="No cache directory",
                 details={"exists": False},
             )
@@ -336,7 +337,7 @@ class CommonIssuesCheck(BaseDiagnosticCheck):
             if size_mb > 500:  # >500MB cache
                 return DiagnosticResult(
                     category="Cache",
-                    status=DiagnosticStatus.WARNING,
+                    status=ValidationSeverity.WARNING,
                     message=f"Large cache: {size_mb:.1f}MB",
                     details={
                         "size_mb": round(size_mb, 2),
@@ -348,7 +349,7 @@ class CommonIssuesCheck(BaseDiagnosticCheck):
 
             return DiagnosticResult(
                 category="Cache",
-                status=DiagnosticStatus.OK,
+                status=OperationResult.SUCCESS,
                 message=f"Cache size normal ({size_mb:.1f}MB)",
                 details={"size_mb": round(size_mb, 2), "file_count": len(cache_files)},
             )
@@ -356,7 +357,7 @@ class CommonIssuesCheck(BaseDiagnosticCheck):
         except Exception as e:
             return DiagnosticResult(
                 category="Cache",
-                status=DiagnosticStatus.WARNING,
+                status=ValidationSeverity.WARNING,
                 message=f"Could not check cache: {e!s}",
                 details={"error": str(e)},
             )