claude-mpm 4.2.44__py3-none-any.whl → 4.3.0__py3-none-any.whl

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,11 @@ 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,8 +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 +437,7 @@ class HookHandler:
         Returns:
             Processed event data
         """
-        processed = {
+        return {
             "type": data.get("type"),
             "timestamp": data.get("timestamp"),
             "session_id": data.get("session_id"),
@@ -448,8 +446,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.
 

claude_mpm/services/monitor/server.py

@@ -15,6 +15,7 @@ DESIGN DECISIONS:
 """
 
 import asyncio
+import contextlib
 import os
 import threading
 import time
@@ -348,7 +349,7 @@ class UnifiedMonitorServer:
                     data = await request.json()
 
                     # Extract event data
-                    namespace = data.get("namespace", "hook")
+                    data.get("namespace", "hook")
                     event = data.get("event", "claude_event")
                     event_data = data.get("data", {})
 
@@ -508,6 +509,19 @@ class UnifiedMonitorServer:
                     {"working_directory": os.getcwd(), "success": True}
                 )
 
+            # Monitor page routes
+            async def monitor_page_handler(request):
+                """Serve monitor HTML pages."""
+                page_name = request.match_info.get("page", "agents")
+                static_dir = dashboard_dir / "static"
+                file_path = static_dir / f"{page_name}.html"
+
+                if file_path.exists() and file_path.is_file():
+                    with open(file_path, encoding="utf-8") as f:
+                        content = f.read()
+                    return web.Response(text=content, content_type="text/html")
+                return web.Response(text="Page not found", status=404)
+
             # Register routes
             self.app.router.add_get("/", dashboard_index)
             self.app.router.add_get("/health", health_check)
@@ -518,6 +532,17 @@ class UnifiedMonitorServer:
             self.app.router.add_post("/api/events", api_events_handler)
             self.app.router.add_post("/api/file", api_file_handler)
 
+            # Monitor page routes
+            self.app.router.add_get("/monitor", lambda r: monitor_page_handler(r))
+            self.app.router.add_get(
+                "/monitor/agents", lambda r: monitor_page_handler(r)
+            )
+            self.app.router.add_get("/monitor/tools", lambda r: monitor_page_handler(r))
+            self.app.router.add_get("/monitor/files", lambda r: monitor_page_handler(r))
+            self.app.router.add_get(
+                "/monitor/events", lambda r: monitor_page_handler(r)
+            )
+
             # Static files with cache busting headers for development
             static_dir = dashboard_dir / "static"
             if static_dir.exists():
@@ -666,10 +691,8 @@ class UnifiedMonitorServer:
             # Cancel heartbeat task if running
             if self.heartbeat_task and not self.heartbeat_task.done():
                 self.heartbeat_task.cancel()
-                try:
+                with contextlib.suppress(asyncio.CancelledError):
                     await self.heartbeat_task
-                except asyncio.CancelledError:
-                    pass
                 self.logger.debug("Heartbeat task cancelled")
 
             # Close the Socket.IO server first to stop accepting new connections

claude_mpm/tools/code_tree_analyzer.py

@@ -1739,7 +1739,7 @@ class CodeTreeAnalyzer:
 
         # Filter only very specific internal patterns
         # Be more conservative - only filter obvious internal handlers
-        if name_lower.startswith("handle_") or name_lower.startswith("on_"):
+        if name_lower.startswith(("handle_", "on_")):
             return True
 
         # Filter Python magic methods except important ones
@@ -1756,9 +1756,7 @@ class CodeTreeAnalyzer:
             return node.name not in important_magic
 
         # Filter very generic getters/setters only if they're trivial
-        if (name_lower.startswith("get_") or name_lower.startswith("set_")) and len(
-            node.name
-        ) <= 8:
+        if (name_lower.startswith(("get_", "set_"))) and len(node.name) <= 8:
             return True
 
         # Don't filter single underscore functions - they're often important