claude-mpm 4.2.43__py3-none-any.whl → 4.2.51__py3-none-any.whl

claude_mpm/dashboard/templates/index.html

@@ -150,6 +150,23 @@
             <div class="header-row">
                 <div class="header-title">
                     <h1>🚀 Claude MPM Monitor</h1>
+                    <a href="/static/monitors-index.html" style="
+                        display: inline-block;
+                        margin-left: 20px;
+                        padding: 8px 16px;
+                        background: linear-gradient(135deg, #667eea 0%, #764ba2 100%);
+                        color: white;
+                        text-decoration: none;
+                        border-radius: 8px;
+                        font-size: 14px;
+                        font-weight: 500;
+                        transition: transform 0.2s, box-shadow 0.2s;
+                        vertical-align: middle;
+                    "
+                    onmouseover="this.style.transform='translateY(-2px)'; this.style.boxShadow='0 5px 15px rgba(102, 126, 234, 0.4)';"
+                    onmouseout="this.style.transform=''; this.style.boxShadow='';">
+                        Try New Dashboard →
+                    </a>
                     <div id="connection-status" class="status-badge status-disconnected">
                         <span>●</span> Disconnected
                     </div>
@@ -179,6 +196,15 @@
                         <div class="metric-mini-value" id="error-count">0</div>
                         <div class="metric-mini-label">Errors</div>
                     </div>
+                    <!-- Debug metrics for troubleshooting -->
+                    <div class="metric-mini" style="background: rgba(255,255,255,0.05);">
+                        <div class="metric-mini-value" id="debug-events-received">0</div>
+                        <div class="metric-mini-label">Received</div>
+                    </div>
+                    <div class="metric-mini" style="background: rgba(255,255,255,0.05);">
+                        <div class="metric-mini-value" id="debug-events-rendered">0</div>
+                        <div class="metric-mini-label">Rendered</div>
+                    </div>
                 </div>
             </div>
 
@@ -249,8 +275,7 @@
                         <a href="#tools" class="tab-button" data-tab="tools">🔧 Tools</a>
                         <a href="#files" class="tab-button" data-tab="files">📁 Files</a>
                         <a href="#activity" class="tab-button" data-tab="activity">🌳 Activity</a>
-                        <a href="#file_tree" class="tab-button active" data-tab="claude-tree">📝 File Tree</a>
-                        <a href="#browser_logs" class="tab-button" data-tab="browser-logs">🌐 Browser Logs</a>
+                        <a href="#file_tree" class="tab-button" data-tab="claude-tree">📝 File Tree</a>
                     </div>
 
                     <!-- Events Tab -->
@@ -390,19 +415,13 @@
                     </div>
 
                     <!-- File Tree Tab -->
-                    <div class="tab-content active" id="claude-tree-tab">
-                        <div id="claude-tree-container" style="width: 100%; height: 100%;">
+                    <div class="tab-content" id="claude-tree-tab">
+                        <div id="claude-tree-container" style="width: 100%; height: 100%; position: relative;">
                             <!-- File activity tree will be rendered here by code-viewer.js -->
+                            <!-- This container is ISOLATED from other tabs -->
                         </div>
                     </div>
 
-                    <!-- Browser Logs Tab -->
-                    <div class="tab-content" id="browser-logs-tab">
-                        <div id="browser-logs-container" style="width: 100%; height: 100%;" data-component="browser-logs">
-                            <!-- Browser logs will be rendered here by browser-log-viewer.js -->
-                            <!-- This container is EXCLUSIVELY for browser console logs -->
-                        </div>
-                    </div>
 
                 </div>
             </div>
@@ -498,45 +517,68 @@
             .then(() => loadModule('/static/js/components/code-tree/tree-search.js'))
             .then(() => loadModule('/static/js/components/code-tree/tree-breadcrumb.js'))
             .then(() => {
-                // Load UI State Manager v2 from source with nuclear browser log handling
-                return loadModule('/static/js/components/ui-state-manager.js?v=2.0-NUCLEAR');
+                // CRITICAL: Load socket-client.js FIRST (dependency of socket-manager)
+                return loadModule('/static/js/socket-client.js');
             })
             .then(() => {
-                // Now load main components in parallel
+                // CRITICAL: Load socket-manager BEFORE dashboard.js
+                // The source version must be loaded to make SocketManager globally available
+                return loadModule('/static/js/components/socket-manager.js');
+            })
+            .then(() => {
+                // Load all required dashboard dependencies BEFORE dashboard.js
+                // These must be available globally for the dist/dashboard.js to work
                 return Promise.all([
-                    loadModule('/static/dist/dashboard.js'),
+                    loadModule('/static/js/components/event-viewer.js'),
+                    loadModule('/static/js/components/module-viewer.js'),
+                    loadModule('/static/js/components/session-manager.js'),
+                    loadModule('/static/js/components/agent-inference.js'),
+                    loadModule('/static/js/components/agent-hierarchy.js'),
+                    loadModule('/static/js/components/ui-state-manager.js'),
+                    loadModule('/static/js/components/event-processor.js'),
+                    loadModule('/static/js/components/export-manager.js'),
+                    loadModule('/static/js/components/working-directory.js'),
+                    loadModule('/static/js/components/file-tool-tracker.js'),
+                    loadModule('/static/js/components/build-tracker.js')
+                ]);
+            })
+            .then(() => {
+                // Now load main components including dashboard.js which depends on the above
+                return Promise.all([
+                    loadModule('/static/dist/dashboard.js'),  // Use dist version that requires above components
                     loadModule('/static/dist/components/activity-tree.js'),
                     loadModule('/static/js/components/code-tree.js'),  // TEMPORARY: Direct source for debugging
-                    loadModule('/static/js/components/code-viewer.js'),  // Code viewer now includes file change tracking
-                    loadModule('/static/dist/components/file-viewer.js'),  // File viewer for viewing file contents
-                    // TEMPORARILY DISABLED for simplest implementation
-                    // loadModule('/static/js/components/browser-log-viewer.js?v=2.0-NUCLEAR')  // Browser console log viewer v2.0 NUCLEAR
-                    // TEMPORARILY DISABLED - BrowserLogViewer initialization
-                    //     .then(() => {
-                    //         // Initialize BrowserLogViewer v2.0 immediately after loading
-                    //         if (typeof BrowserLogViewer !== 'undefined') {
-                    //             const container = document.getElementById('browser-logs-container');
-                    //             if (container && !window.browserLogViewer) {
-                    //                 console.error('[Main] Initializing BrowserLogViewer v2.0 NUCLEAR...');
-                    //                 window.browserLogViewer = new BrowserLogViewer(container);
-                    //                 console.error('[Main] BrowserLogViewer v2.0 NUCLEAR initialized successfully');
-                    //             } else {
-                    //                 console.error('[Main] BrowserLogViewer v2.0 already initialized or container not found');
-                    //             }
-                    //         } else {
-                    //             console.error('[Main] BrowserLogViewer class not found - script may not have loaded');
-                    //         }
-                    //     })
+                    loadModule('/static/js/components/code-viewer.js').catch(err => {
+                        console.error('[CRITICAL] Failed to load code-viewer.js:', err);
+                        throw err;
+                    }),  // Code viewer now includes file change tracking
+                    loadModule('/static/dist/components/file-viewer.js')  // File viewer for viewing file contents
                 ]);
             })
             .then(() => {
             console.log('All dashboard modules loaded successfully');
             
+            // Debug: Check if CodeViewer loaded
+            if (window.CodeViewer) {
+                console.log('[DEBUG] CodeViewer is available on window object');
+            } else {
+                console.error('[ERROR] CodeViewer NOT FOUND on window object!');
+            }
+            
             // CodeViewer will auto-initialize and handle tab switching internally
             
             // Browser Log Viewer initialization is now handled by UIStateManager
             // This prevents duplicate event handlers and tab selection conflicts
             
+            // Load bulletproof tab isolation fix
+            loadModule('/static/js/tab-isolation-fix.js')
+                .then(() => {
+                    console.log('✅ Tab isolation fix loaded successfully');
+                })
+                .catch(err => {
+                    console.error('❌ Failed to load tab isolation fix:', err);
+                });
+
             // Hash navigation will handle default tab based on URL
             // If no hash, default will be 'events' as per hashToTab mapping
             // To start with File Tree, we can set hash if not present
@@ -548,6 +590,11 @@
                     console.log('Hash present:', window.location.hash);
                 }
             }, 500);
+            })
+            .catch(error => {
+                console.error('[CRITICAL] Error loading dashboard modules:', error);
+                console.error('Stack trace:', error.stack);
+            });
             
             // Give modules time to execute and initialize dashboard
             setTimeout(() => {
@@ -609,9 +656,6 @@
                     console.error('Connection toggle button not found in DOM');
                 }
             }, 1000);
-        }).catch(error => {
-            console.error('Failed to load dashboard modules:', error);
-        });
     </script>
 </body>
 </html>

claude_mpm/hooks/claude_hooks/services/connection_manager_http.py

@@ -151,7 +151,7 @@ class ConnectionManagerService:
 
             if loop:
                 # We're in an async context, create a task
-                task = loop.create_task(self._async_emit(namespace, event, data))
+                loop.create_task(self._async_emit(namespace, event, data))
                 # Don't wait for completion to maintain low latency
                 if DEBUG:
                     print(f"✅ Async emit scheduled: {event}", file=sys.stderr)

claude_mpm/services/agents/deployment/agent_discovery_service.py

@@ -219,6 +219,9 @@ class AgentDiscoveryService:
             agent_info = {
                 "name": metadata.get("name", template_file.stem),
                 "description": metadata.get("description", "No description available"),
+                "type": template_data.get(
+                    "agent_type", metadata.get("category", "agent")
+                ),  # Extract agent type
                 "version": template_data.get(
                     "agent_version",
                     template_data.get("version", metadata.get("version", "1.0.0")),

claude_mpm/services/agents/deployment/agent_template_builder.py

@@ -267,17 +267,21 @@ class AgentTemplateBuilder:
         # Include tools field only if agent is clearly restricted (missing core tools or very few tools)
         not has_core_tools or len(agent_tools) < 6
 
-        # Build YAML frontmatter using Claude Code's compatible format
-        # ONLY include fields that Claude Code recognizes
+        # Build YAML frontmatter with all relevant metadata from JSON template
+        # Include all fields that are useful for agent management and functionality
         #
-        # CLAUDE CODE COMPATIBLE FORMAT:
+        # COMPREHENSIVE AGENT FRONTMATTER FORMAT:
         # - name: kebab-case agent name (required)
         # - description: when/why to use this agent with examples (required, multiline)
         # - model: mapped model name (required)
+        # - type: agent type for categorization and functionality (optional but important)
+        # - category: organizational category (optional)
         # - color: visual identifier (optional)
         # - version: agent version for update tracking (optional)
         # - author: creator information (optional)
-        # NOTE: tags field REMOVED - not supported by Claude Code
+        # - created_at: creation timestamp (optional)
+        # - updated_at: last update timestamp (optional)
+        # - tags: list of tags for search and categorization (optional)
         frontmatter_lines = [
             "---",
             f"name: {claude_code_name}",
@@ -291,15 +295,28 @@ class AgentTemplateBuilder:
         # Add model field (required for Claude Code)
         frontmatter_lines.append(f"model: {model}")
 
-        # Add optional metadata (excluding tags field completely)
+        # Add type field (important for agent categorization)
+        if agent_type and agent_type != "general":
+            frontmatter_lines.append(f"type: {agent_type}")
+
+        # Add optional metadata fields
         if metadata.get("color"):
             frontmatter_lines.append(f"color: {metadata['color']}")
-        if (
-            agent_version and agent_version != "1.0.0"
-        ):  # Only include non-default versions
+        if metadata.get("category"):
+            frontmatter_lines.append(f"category: {metadata['category']}")
+        # Always include version field to prevent deployment comparison issues
+        if agent_version:
             frontmatter_lines.append(f'version: "{agent_version}"')
         if metadata.get("author"):
             frontmatter_lines.append(f'author: "{metadata["author"]}"')
+        if metadata.get("created_at"):
+            frontmatter_lines.append(f"created_at: {metadata['created_at']}")
+        if metadata.get("updated_at"):
+            frontmatter_lines.append(f"updated_at: {metadata['updated_at']}")
+        # Add tags as comma-separated string if they exist (consistent with tools format)
+        if metadata.get("tags") and isinstance(metadata["tags"], list):
+            tags_str = ",".join(metadata["tags"])
+            frontmatter_lines.append(f"tags: {tags_str}")
 
         frontmatter_lines.extend(
             [

claude_mpm/services/agents/deployment/agent_validator.py

@@ -326,6 +326,7 @@ class AgentValidator:
             "path": str(agent_file),
             "description": "No description",
             "version": "unknown",
+            "type": "agent",  # Default type
         }
 
         # Extract from YAML frontmatter
@@ -342,6 +343,8 @@ class AgentValidator:
                 agent_info["version"] = (
                     stripped_line.split(":", 1)[1].strip().strip("\"'")
                 )
+            elif stripped_line.startswith("type:"):
+                agent_info["type"] = stripped_line.split(":", 1)[1].strip().strip("\"'")
 
         return agent_info
 

claude_mpm/services/agents/deployment/validation/template_validator.py

@@ -190,10 +190,19 @@ class TemplateValidator:
 
         if "tags" in metadata:
             tags = metadata["tags"]
-            if not isinstance(tags, list):
-                result.add_error("Tags should be a list", field_name="metadata.tags")
-            elif len(tags) == 0:
-                result.add_warning("No tags specified", field_name="metadata.tags")
+            if isinstance(tags, str):
+                # Convert comma-separated string to list for validation
+                tag_list = [tag.strip() for tag in tags.split(",") if tag.strip()]
+                if len(tag_list) == 0:
+                    result.add_warning("No tags specified", field_name="metadata.tags")
+            elif isinstance(tags, list):
+                if len(tags) == 0:
+                    result.add_warning("No tags specified", field_name="metadata.tags")
+            else:
+                result.add_error(
+                    "Tags should be a list or comma-separated string",
+                    field_name="metadata.tags",
+                )
 
     def _validate_capabilities(
         self, capabilities: Dict[str, Any], result: ValidationResult

claude_mpm/services/agents/local_template_manager.py

@@ -250,13 +250,10 @@ class LocalAgentTemplateManager:
             Created LocalAgentTemplate object
         """
         # Determine author based on tier
-        if tier == "project":
-            author = self.get_project_name()
-        else:
-            author = Path.home().name
+        author = self.get_project_name() if tier == "project" else Path.home().name
 
         # Create template
-        template = LocalAgentTemplate(
+        return LocalAgentTemplate(
             agent_id=agent_id,
             agent_version="1.0.0",
             author=author,
@@ -283,7 +280,6 @@ class LocalAgentTemplateManager:
             parent_agent=parent_agent,
         )
 
-        return template
 
     def save_local_template(
         self, template: LocalAgentTemplate, tier: Optional[str] = None

claude_mpm/services/monitor/daemon.py

@@ -163,8 +163,7 @@ class UnifiedMonitorDaemon:
         # environment variable to prevent infinite recursion
         if self.daemon_manager.use_subprocess_daemon():
             # Start using subprocess - this returns immediately in parent
-            success = self.daemon_manager.start_daemon_subprocess()
-            return success
+            return self.daemon_manager.start_daemon_subprocess()
 
         # Check if we're in subprocess mode (environment variable set)
         if os.environ.get("CLAUDE_MPM_SUBPROCESS_DAEMON") == "1":

claude_mpm/services/monitor/daemon_manager.py

@@ -183,10 +183,7 @@ class DaemonManager:
                 return True
 
             # Try to identify claude-mpm processes
-            if self._kill_claude_mpm_processes():
-                return True
-
-            return False
+            return bool(self._kill_claude_mpm_processes())
 
         except Exception as e:
             self.logger.error(f"Error killing processes on port: {e}")
@@ -579,7 +576,7 @@ class DaemonManager:
                     stderr=subprocess.STDOUT if self.log_file else subprocess.DEVNULL,
                     start_new_session=True,  # Create new process group
                     close_fds=(
-                        False if self.log_file else True
+                        not self.log_file
                     ),  # Keep log file open if redirecting
                     env=env,  # Pass modified environment
                 )

claude_mpm/services/monitor/event_emitter.py

@@ -115,7 +115,7 @@ class AsyncEventEmitter:
         event: str,
         data: Dict[str, Any],
         force_http: bool = False,
-        endpoint: str = None,
+        endpoint: Optional[str] = None,
     ) -> bool:
         """
         Emit event with optimal routing (direct calls vs HTTP).
@@ -186,7 +186,7 @@ class AsyncEventEmitter:
         return False
 
     async def _emit_http(
-        self, namespace: str, event: str, data: Dict[str, Any], endpoint: str = None
+        self, namespace: str, event: str, data: Dict[str, Any], endpoint: Optional[str] = None
     ) -> bool:
         """Emit event via HTTP with connection pooling."""
         if not self._http_session:

claude_mpm/services/monitor/handlers/code_analysis.py

@@ -248,14 +248,14 @@ class CodeAnalysisHandler:
                 self.logger.info("All analysis cache cleared")
             elif cache_type == "file":
                 keys_to_remove = [
-                    k for k in self.analysis_cache.keys() if k.startswith("file:")
+                    k for k in self.analysis_cache if k.startswith("file:")
                 ]
                 for key in keys_to_remove:
                     del self.analysis_cache[key]
                 self.logger.info("File analysis cache cleared")
             elif cache_type == "directory":
                 keys_to_remove = [
-                    k for k in self.analysis_cache.keys() if k.startswith("dir:")
+                    k for k in self.analysis_cache if k.startswith("dir:")
                 ]
                 for key in keys_to_remove:
                     del self.analysis_cache[key]
@@ -281,10 +281,9 @@ class CodeAnalysisHandler:
         try:
             # Run analysis in thread pool to avoid blocking
             loop = asyncio.get_event_loop()
-            result = await loop.run_in_executor(
+            return await loop.run_in_executor(
                 None, self.analyzer.analyze_file, file_path
             )
-            return result
 
         except Exception as e:
             self.logger.error(f"Error in async file analysis: {e}")
@@ -305,10 +304,9 @@ class CodeAnalysisHandler:
         try:
             # Run tree building in thread pool
             loop = asyncio.get_event_loop()
-            result = await loop.run_in_executor(
+            return await loop.run_in_executor(
                 None, self.builder.build_tree, dir_path, max_depth
             )
-            return result
 
         except Exception as e:
             self.logger.error(f"Error in async directory tree building: {e}")

claude_mpm/services/monitor/handlers/hooks.py

@@ -416,7 +416,7 @@ class HookHandler:
         Returns:
             Processed event data
         """
-        processed = {
+        return {
             "type": data.get("type", "hook"),
             "subtype": data.get("subtype", "unknown"),
             "timestamp": data.get("timestamp", asyncio.get_event_loop().time()),
@@ -428,7 +428,6 @@ class HookHandler:
             "original_event": data,  # Keep original for debugging
         }
 
-        return processed
 
     def _process_hook_event(self, data: Dict) -> Dict:
         """Process and normalize hook event data.
@@ -439,7 +438,7 @@ class HookHandler:
         Returns:
             Processed event data
         """
-        processed = {
+        return {
             "type": data.get("type"),
             "timestamp": data.get("timestamp"),
             "session_id": data.get("session_id"),
@@ -448,7 +447,6 @@ class HookHandler:
             "processed_at": asyncio.get_event_loop().time(),
         }
 
-        return processed
 
     def _update_session_tracking(self, session_id: str, event: Dict):
         """Update session tracking with new event.