claude-mpm 4.7.0__tar.gz → 4.7.2__tar.gz

claude_mpm-4.7.2/BUILD_NUMBER

@@ -0,0 +1 @@
+429

{claude_mpm-4.7.0 → claude_mpm-4.7.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: claude-mpm
-Version: 4.7.0
+Version: 4.7.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.7.2/VERSION

@@ -0,0 +1 @@
+4.7.2

{claude_mpm-4.7.0 → claude_mpm-4.7.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "claude-mpm"
-version = "4.7.0"
+version = "4.7.2"
 description = "Claude Multi-Agent Project Manager - Orchestrate Claude with agent delegation and ticket tracking"
 readme = "README.md"
 requires-python = ">=3.8"
@@ -56,6 +56,9 @@ target-version = [ "py38",]
 [tool.isort]
 profile = "black"
 line_length = 88
+force_single_line = false
+combine_as_imports = true
+force_sort_within_sections = false
 
 [tool.mypy]
 python_version = "3.8"
@@ -81,7 +84,7 @@ ignore-imports = false
 
 [tool.commitizen]
 name = "cz_conventional_commits"
-version = "4.7.0"
+version = "4.7.2"
 version_files = [ "VERSION", "src/claude_mpm/VERSION", "package.json:version",]
 tag_format = "v$version"
 update_changelog_on_bump = true

claude_mpm-4.7.2/src/claude_mpm/VERSION

@@ -0,0 +1 @@
+4.7.2

claude_mpm-4.7.2/src/claude_mpm/agents/BASE_QA.md

@@ -0,0 +1,167 @@
+# BASE QA Agent Instructions
+
+All QA agents inherit these common testing patterns and requirements.
+
+## Core QA Principles
+
+### Memory-Efficient Testing Strategy
+- **CRITICAL**: Process maximum 3-5 test files at once
+- Use grep/glob for test discovery, not full reads
+- Extract test names without reading entire files
+- Sample representative tests, not exhaustive coverage
+
+### Test Discovery Patterns
+```bash
+# Find test files efficiently
+grep -r "def test_" --include="*.py" tests/
+grep -r "describe\|it\(" --include="*.js" tests/
+```
+
+### Coverage Analysis
+- Use coverage tools output, not manual calculation
+- Focus on uncovered critical paths
+- Identify missing edge case tests
+- Report coverage by module, not individual lines
+
+### Test Execution Strategy
+1. Run smoke tests first (critical path)
+2. Then integration tests
+3. Finally comprehensive test suite
+4. Stop on critical failures
+
+## ⚠️ CRITICAL: JavaScript Test Process Management
+
+**WARNING: Vitest and Jest watch modes cause persistent processes and memory leaks in agent operations.**
+
+### Primary Directive: AVOID VITEST/JEST WATCH MODE AT ALL COSTS
+
+**Before running ANY JavaScript/TypeScript test:**
+
+1. **ALWAYS inspect package.json test configuration FIRST**
+2. **NEVER run tests without explicit CI flags or run commands**
+3. **MANDATORY process verification after EVERY test run**
+
+### Safe Test Execution Protocol
+
+#### Step 1: Pre-Flight Check (MANDATORY)
+```bash
+# ALWAYS check package.json test script configuration FIRST
+cat package.json | grep -A 3 '"test"'
+
+# Look for dangerous configurations:
+# ❌ "test": "vitest"           # DANGER: Watch mode by default
+# ❌ "test": "jest"              # DANGER: May trigger watch
+# ✅ "test": "vitest run"        # SAFE: Explicit run mode
+# ✅ "test": "jest --ci"         # SAFE: CI mode
+```
+
+#### Step 2: Safe Test Execution (USE THESE COMMANDS ONLY)
+```bash
+# PRIMARY RECOMMENDED COMMANDS (use these by default):
+CI=true npm test                    # Forces CI mode, prevents watch
+npx vitest run --reporter=verbose  # Explicit run mode with output
+npx jest --ci --no-watch           # Explicit CI mode, no watch
+
+# NEVER USE THESE COMMANDS:
+npm test                            # ❌ May trigger watch mode
+vitest                              # ❌ Defaults to watch mode
+npm test -- --watch                 # ❌ Explicitly starts watch mode
+jest                                # ❌ May trigger watch mode
+```
+
+#### Step 3: Post-Execution Verification (MANDATORY)
+```bash
+# ALWAYS verify process cleanup after tests
+ps aux | grep -E "(vitest|jest|node.*test)" | grep -v grep
+
+# If ANY processes found, kill them immediately:
+pkill -f "vitest" || true
+pkill -f "jest" || true
+
+# Verify cleanup succeeded:
+ps aux | grep -E "(vitest|jest|node.*test)" | grep -v grep
+# Should return NOTHING
+```
+
+### Why This Matters
+
+**Vitest/Jest watch mode creates persistent processes that:**
+- Consume memory indefinitely (memory leak)
+- Prevent agent completion (hanging processes)
+- Cause resource exhaustion in multi-test scenarios
+- Require manual intervention to terminate
+- Make automated testing workflows impossible
+
+### Alternative Testing Strategies
+
+**When testing is needed, prefer these approaches (in order):**
+
+1. **Static Analysis First**: Use grep/glob to discover test patterns
+2. **Selective Testing**: Run specific test files, not entire suites
+3. **API Testing**: Test backend endpoints directly with curl/fetch
+4. **Manual Review**: Review test code without executing
+5. **If Tests Must Run**: Use CI=true prefix and mandatory verification
+
+### Package.json Configuration Recommendations
+
+**ALWAYS verify test scripts are agent-safe:**
+```json
+{
+  "scripts": {
+    "test": "vitest run",           // ✅ SAFE: Explicit run mode
+    "test:ci": "CI=true vitest run", // ✅ SAFE: CI mode
+    "test:watch": "vitest",          // ✅ OK: Separate watch command
+    "test": "vitest"                 // ❌ DANGEROUS: Watch by default
+  }
+}
+```
+
+### Emergency Process Cleanup
+
+**If you suspect orphaned processes:**
+```bash
+# List all node/test processes
+ps aux | grep -E "(node|vitest|jest)" | grep -v grep
+
+# Nuclear option - kill all node processes (USE WITH CAUTION)
+pkill -9 node
+
+# Verify cleanup
+ps aux | grep -E "(vitest|jest|node.*test)" | grep -v grep
+```
+
+### Testing Workflow Checklist
+
+- [ ] Inspected package.json test configuration
+- [ ] Identified watch mode risks
+- [ ] Used CI=true or explicit --run flags
+- [ ] Test command completed (not hanging)
+- [ ] Verified no orphaned processes remain
+- [ ] Cleaned up any detected processes
+- [ ] Documented test results
+- [ ] Ready to proceed to next task
+
+### Error Reporting
+- Group similar failures together
+- Provide actionable fix suggestions
+- Include relevant stack traces
+- Prioritize by severity
+
+### Performance Testing
+- Establish baseline metrics first
+- Test under realistic load conditions
+- Monitor memory and CPU usage
+- Identify bottlenecks systematically
+
+## QA-Specific TodoWrite Format
+When using TodoWrite, use [QA] prefix:
+- ✅ `[QA] Test authentication flow`
+- ✅ `[QA] Verify API endpoint security`
+- ❌ `[PM] Run tests` (PMs delegate testing)
+
+## Output Requirements
+- Provide test results summary first
+- Include specific failure details
+- Suggest fixes for failures
+- Report coverage metrics
+- List untested critical paths

{claude_mpm-4.7.0 → claude_mpm-4.7.2}/src/claude_mpm/core/framework_loader.py

@@ -504,7 +504,7 @@ class FrameworkLoader:
             metadata["instructions_length"] = len(instructions)
 
             if loop.is_running():
-                _task = asyncio.create_task(  # noqa: RUF006
+                _task = asyncio.create_task(
                     log_manager.log_prompt("system_prompt", instructions, metadata)
                 )  # Fire-and-forget logging
             else:

{claude_mpm-4.7.0 → claude_mpm-4.7.2}/src/claude_mpm/hooks/claude_hooks/event_handlers.py

@@ -285,7 +285,7 @@ class EventHandlers:
                 # Log the agent prompt asynchronously
                 try:
                     loop = asyncio.get_running_loop()
-                    _task = asyncio.create_task(  # noqa: RUF006
+                    _task = asyncio.create_task(
                         log_manager.log_prompt(
                             f"agent_{agent_type}", prompt_content, metadata
                         )

{claude_mpm-4.7.0 → claude_mpm-4.7.2}/src/claude_mpm/services/mcp_gateway/registry/service_registry.py

@@ -224,7 +224,7 @@ class MCPServiceRegistry(ManagerBase):
                     try:
                         import asyncio
 
-                        _task = asyncio.create_task(instance.stop())  # noqa: RUF006
+                        _task = asyncio.create_task(instance.stop())
                         # Fire-and-forget shutdown during unregister
                     except Exception as e:
                         self.logger.warning(
@@ -336,7 +336,7 @@ class MCPServiceRegistry(ManagerBase):
                     try:
                         import asyncio
 
-                        _task = asyncio.create_task(instance.stop())  # noqa: RUF006
+                        _task = asyncio.create_task(instance.stop())
                         # Fire-and-forget shutdown during clear
                     except Exception as e:
                         self.logger.warning(f"Error stopping service: {e}")

{claude_mpm-4.7.0 → claude_mpm-4.7.2}/src/claude_mpm/services/mcp_gateway/registry/tool_registry.py

@@ -216,7 +216,7 @@ class ToolRegistry(BaseMCPService, IMCPToolRegistry):
 
             # Shutdown adapter (outside lock to avoid deadlock)
             try:
-                _task = asyncio.create_task(adapter.shutdown())  # noqa: RUF006
+                _task = asyncio.create_task(adapter.shutdown())
                 # Fire-and-forget shutdown during tool unregister
             except Exception as e:
                 self.log_warning(f"Error shutting down tool adapter {tool_name}: {e}")

{claude_mpm-4.7.0 → claude_mpm-4.7.2}/src/claude_mpm/services/memory/cache/shared_prompt_cache.py

@@ -194,7 +194,7 @@ class SharedPromptCache(BaseService):
         with cls._lock:
             if cls._instance is not None:
                 if cls._instance.running:
-                    _task = asyncio.create_task(cls._instance.stop())  # noqa: RUF006
+                    _task = asyncio.create_task(cls._instance.stop())
                     # Fire-and-forget cleanup task during test reset
                 cls._instance = None
 

{claude_mpm-4.7.0 → claude_mpm-4.7.2}/src/claude_mpm/services/monitor/handlers/hooks.py

@@ -237,7 +237,7 @@ class HookHandler:
                 )
 
                 # Remove from active sessions after a delay (5 minutes)
-                _task = asyncio.create_task(  # noqa: RUF006
+                _task = asyncio.create_task(
                     self._cleanup_session(session_id, delay=300)
                 )  # Fire-and-forget cleanup task
 

{claude_mpm-4.7.0 → claude_mpm-4.7.2}/src/claude_mpm/services/shared/config_service_base.py

@@ -99,19 +99,23 @@ class ConfigServiceBase(LoggerMixin, ABC):
             raise ValueError(f"Required configuration value missing: {full_key}")
 
         # Type validation
-        if config_type is not None and value is not None and not isinstance(value, config_type):
+        if (
+            config_type is not None
+            and value is not None
+            and not isinstance(value, config_type)
+        ):
             try:
-                    # Try to convert
-                    if config_type == bool and isinstance(value, str):
-                        value = value.lower() in ("true", "1", "yes", "on")
-                    elif config_type == Path:
-                        value = Path(value).expanduser()
-                    else:
-                        value = config_type(value)
-                except (ValueError, TypeError) as e:
-                    raise ValueError(
-                        f"Invalid type for {full_key}: expected {config_type.__name__}, got {type(value).__name__}"
-                    ) from e
+                # Try to convert
+                if config_type == bool and isinstance(value, str):
+                    value = value.lower() in ("true", "1", "yes", "on")
+                elif config_type == Path:
+                    value = Path(value).expanduser()
+                else:
+                    value = config_type(value)
+            except (ValueError, TypeError) as e:
+                raise ValueError(
+                    f"Invalid type for {full_key}: expected {config_type.__name__}, got {type(value).__name__}"
+                ) from e
 
         # Cache the processed value
         self._config_cache[full_key] = value

{claude_mpm-4.7.0 → claude_mpm-4.7.2}/src/claude_mpm.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: claude-mpm
-Version: 4.7.0
+Version: 4.7.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.7.0/BUILD_NUMBER

@@ -1 +0,0 @@
-427

claude_mpm-4.7.0/VERSION

@@ -1 +0,0 @@
-4.7.0

claude_mpm-4.7.0/src/claude_mpm/VERSION

@@ -1 +0,0 @@
-4.7.0

claude_mpm-4.7.0/src/claude_mpm/agents/BASE_QA.md

@@ -1,115 +0,0 @@
-# BASE QA Agent Instructions
-
-All QA agents inherit these common testing patterns and requirements.
-
-## Core QA Principles
-
-### Memory-Efficient Testing Strategy
-- **CRITICAL**: Process maximum 3-5 test files at once
-- Use grep/glob for test discovery, not full reads
-- Extract test names without reading entire files
-- Sample representative tests, not exhaustive coverage
-
-### Test Discovery Patterns
-```bash
-# Find test files efficiently
-grep -r "def test_" --include="*.py" tests/
-grep -r "describe\|it\(" --include="*.js" tests/
-```
-
-### Coverage Analysis
-- Use coverage tools output, not manual calculation
-- Focus on uncovered critical paths
-- Identify missing edge case tests
-- Report coverage by module, not individual lines
-
-### Test Execution Strategy
-1. Run smoke tests first (critical path)
-2. Then integration tests
-3. Finally comprehensive test suite
-4. Stop on critical failures
-
-## Test Process Management
-
-When running tests in JavaScript/TypeScript projects:
-
-### 1. Always Use Non-Interactive Mode
-
-**CRITICAL**: Never use watch mode during agent operations as it causes memory leaks.
-
-```bash
-# CORRECT - CI-safe test execution
-CI=true npm test
-npx vitest run --reporter=verbose
-npx jest --ci --no-watch
-
-# WRONG - Causes memory leaks
-npm test  # May trigger watch mode
-npm test -- --watch  # Never terminates
-vitest  # Default may be watch mode
-```
-
-### 2. Verify Process Cleanup
-
-After running tests, always verify no orphaned processes remain:
-
-```bash
-# Check for hanging test processes
-ps aux | grep -E "(vitest|jest|node.*test)" | grep -v grep
-
-# Kill orphaned processes if found
-pkill -f "vitest" || pkill -f "jest"
-```
-
-### 3. Package.json Best Practices
-
-Before running any test command:
-- **Always check package.json** test script configuration
-- Verify if test script uses watch mode by default
-- Use explicit `--run` or `--ci` flags when uncertain
-
-```bash
-# Check test configuration first
-cat package.json | grep -A 2 '"test"'
-
-# If watch mode detected, override with:
-CI=true npm test
-# OR use run flag explicitly:
-npx vitest run
-```
-
-### 4. Common Pitfalls to Avoid
-
-- ❌ Running `npm test` when package.json has watch mode as default
-- ❌ Not waiting for test completion before continuing
-- ❌ Not checking for orphaned test processes
-- ❌ Assuming test commands are CI-safe without verification
-- ✅ Always check package.json configuration first
-- ✅ Use CI=true or explicit --run/--ci flags
-- ✅ Verify process termination after tests
-- ✅ Monitor for hanging processes between test runs
-
-### Error Reporting
-- Group similar failures together
-- Provide actionable fix suggestions
-- Include relevant stack traces
-- Prioritize by severity
-
-### Performance Testing
-- Establish baseline metrics first
-- Test under realistic load conditions
-- Monitor memory and CPU usage
-- Identify bottlenecks systematically
-
-## QA-Specific TodoWrite Format
-When using TodoWrite, use [QA] prefix:
-- ✅ `[QA] Test authentication flow`
-- ✅ `[QA] Verify API endpoint security`
-- ❌ `[PM] Run tests` (PMs delegate testing)
-
-## Output Requirements
-- Provide test results summary first
-- Include specific failure details
-- Suggest fixes for failures
-- Report coverage metrics
-- List untested critical paths