claude-mpm 3.4.7__py3-none-any.whl → 3.4.10__py3-none-any.whl

claude_mpm/cli/commands/run.py

@@ -467,6 +467,7 @@ def _check_socketio_server_running(port, logger):
     
     DESIGN DECISION: We try multiple endpoints and connection methods to ensure
     robust detection. Some servers may be starting up and only partially ready.
+    Added retry logic to handle race conditions during server initialization.
     
     Args:
         port: Port number to check
@@ -483,34 +484,50 @@ def _check_socketio_server_running(port, logger):
         # First, do a basic TCP connection check
         try:
             with socket.socket(socket.AF_INET, socket.SOCK_STREAM) as s:
-                s.settimeout(1.0)
+                s.settimeout(2.0)  # Increased from 1.0s for slower connections
                 result = s.connect_ex(('127.0.0.1', port))
                 if result != 0:
-                    logger.debug(f"TCP connection to port {port} failed (not listening)")
+                    logger.debug(f"TCP connection to port {port} failed (server not started yet)")
                     return False
         except Exception as e:
             logger.debug(f"TCP socket check failed for port {port}: {e}")
             return False
         
-        # If TCP connection succeeds, try HTTP health check
-        try:
-            response = urllib.request.urlopen(f'http://localhost:{port}/status', timeout=5)
-            
-            if response.getcode() == 200:
-                content = response.read().decode()
-                logger.debug(f"✅ Socket.IO server health check passed on port {port}")
-                logger.debug(f"📄 Server response: {content[:100]}...")
-                return True
-            else:
-                logger.debug(f"⚠️ HTTP response code {response.getcode()} from port {port}")
-                return False
+        # If TCP connection succeeds, try HTTP health check with retries
+        # WHY: Even when TCP is accepting connections, the HTTP handler may not be ready
+        max_retries = 3
+        for retry in range(max_retries):
+            try:
+                response = urllib.request.urlopen(f'http://localhost:{port}/status', timeout=10)  # Increased from 5s to 10s
                 
-        except urllib.error.HTTPError as e:
-            logger.debug(f"⚠️ HTTP error {e.code} from server on port {port}")
-            return False
-        except urllib.error.URLError as e:
-            logger.debug(f"⚠️ URL error connecting to port {port}: {e.reason}")
-            return False
+                if response.getcode() == 200:
+                    content = response.read().decode()
+                    logger.debug(f"✅ Socket.IO server health check passed on port {port} (attempt {retry + 1})")
+                    logger.debug(f"📄 Server response: {content[:100]}...")
+                    return True
+                else:
+                    logger.debug(f"⚠️ HTTP response code {response.getcode()} from port {port} (attempt {retry + 1})")
+                    if retry < max_retries - 1:
+                        time.sleep(0.5)  # Brief pause before retry
+                    
+            except urllib.error.HTTPError as e:
+                logger.debug(f"⚠️ HTTP error {e.code} from server on port {port} (attempt {retry + 1})")
+                if retry < max_retries - 1 and e.code in [404, 503]:  # Server starting but not ready
+                    logger.debug("Server appears to be starting, retrying...")
+                    time.sleep(0.5)
+                    continue
+                return False
+            except urllib.error.URLError as e:
+                logger.debug(f"⚠️ URL error connecting to port {port} (attempt {retry + 1}): {e.reason}")
+                if retry < max_retries - 1:
+                    logger.debug("Connection refused - server may still be initializing, retrying...")
+                    time.sleep(0.5)
+                    continue
+                return False
+        
+        # All retries exhausted
+        logger.debug(f"Health check failed after {max_retries} attempts - server not fully ready")
+        return False
             
     except (ConnectionError, OSError) as e:
         logger.debug(f"🔌 Connection error checking port {port}: {e}")
@@ -569,13 +586,17 @@ def _start_standalone_socketio_server(port, logger):
         # 2. Event loop setup (~1s) 
         # 3. aiohttp server binding (~2-5s)
         # 4. Socket.IO service initialization (~1-3s)
-        # Total: up to 10 seconds for full readiness
-        max_attempts = 20  # Increased from 10
-        initial_delay = 0.5  # seconds
-        max_delay = 2.0  # seconds
+        # Total: up to 15+ seconds for full readiness (especially on Python 3.13)
+        max_attempts = 30  # Increased from 20 to handle Python 3.13 slower initialization
+        initial_delay = 1.0  # Increased from 0.5s to give daemon more time to fork
+        max_delay = 3.0  # Increased from 2.0s for slower systems
         
         logger.info(f"Waiting up to {max_attempts * max_delay} seconds for server to be fully ready...")
         
+        # Give the daemon initial time to fork and start before checking
+        logger.debug("Allowing initial daemon startup time...")
+        time.sleep(0.5)
+        
         for attempt in range(max_attempts):
             # Progressive delay - start fast, then slow down for socket binding
             if attempt < 5:
@@ -596,9 +617,14 @@ def _start_standalone_socketio_server(port, logger):
             else:
                 logger.debug(f"Server not yet accepting connections on attempt {attempt + 1}")
         
-        logger.error(f"❌ Socket.IO server failed to start properly on port {port} after {max_attempts} attempts")
-        logger.error(f"💡 This may indicate a port conflict or dependency issue")
-        logger.error(f"🔧 Try a different port with --websocket-port or check for conflicts")
+        logger.error(f"❌ Socket.IO server health check failed after {max_attempts} attempts ({max_attempts * max_delay:.1f}s)")
+        logger.warning(f"⏱️  Server may still be starting - initialization can take 15+ seconds on some systems")
+        logger.warning(f"💡 The daemon process might be running but not yet accepting HTTP connections")
+        logger.error(f"🔧 Troubleshooting steps:")
+        logger.error(f"   - Wait a few more seconds and try again")
+        logger.error(f"   - Check for port conflicts: lsof -i :{port}")
+        logger.error(f"   - Try a different port with --websocket-port")
+        logger.error(f"   - Verify dependencies: pip install python-socketio aiohttp")
         return False
             
     except Exception as e:

claude_mpm/utils/dependency_manager.py

@@ -107,12 +107,14 @@ def ensure_socketio_dependencies(logger=None) -> Tuple[bool, str]:
     """
     Ensure Socket.IO dependencies are installed for monitoring features.
     
-    WHY: The --monitor flag requires python-socketio and aiohttp, but we want
-    the base package to work without these heavy dependencies. This function
-    installs them on-demand.
+    WHY: Socket.IO dependencies (python-socketio, aiohttp, python-engineio) are now
+    core dependencies and should be installed automatically with claude-mpm.
+    This function verifies they are available and provides helpful error messages
+    if something went wrong during installation.
     
-    DESIGN DECISION: We check each dependency individually and only install
-    what's missing, reducing installation time and avoiding unnecessary work.
+    DESIGN DECISION: We still check each dependency individually to provide
+    specific error messages if any are missing, which helps with troubleshooting
+    installation issues.
     
     Args:
         logger: Optional logger for output
@@ -142,8 +144,9 @@ def ensure_socketio_dependencies(logger=None) -> Tuple[bool, str]:
         logger.debug("All Socket.IO dependencies are already installed")
         return True, ""
     
-    # Install missing packages
-    logger.info(f"Installing missing Socket.IO dependencies: {missing_packages}")
+    # Install missing packages (should be rare since they're now core dependencies)
+    logger.warning(f"Socket.IO dependencies are missing despite being core dependencies: {missing_packages}")
+    logger.info(f"Attempting to install missing dependencies: {missing_packages}")
     success, error_msg = install_packages(missing_packages, logger)
     
     if success:

{claude_mpm-3.4.7.dist-info → claude_mpm-3.4.10.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: claude-mpm
-Version: 3.4.7
+Version: 3.4.10
 Summary: Claude Multi-agent Project Manager - Clean orchestration with ticket management
 Home-page: https://github.com/bobmatnyc/claude-mpm
 Author: Claude MPM Team
@@ -32,6 +32,9 @@ Requires-Dist: flask-cors>=4.0.0
 Requires-Dist: watchdog>=3.0.0
 Requires-Dist: tree-sitter>=0.21.0
 Requires-Dist: tree-sitter-language-pack>=0.8.0
+Requires-Dist: python-socketio>=5.11.0
+Requires-Dist: aiohttp>=3.9.0
+Requires-Dist: python-engineio>=4.8.0
 Provides-Extra: dev
 Requires-Dist: pytest>=7.0; extra == "dev"
 Requires-Dist: pytest-asyncio; extra == "dev"
@@ -40,9 +43,6 @@ Requires-Dist: black; extra == "dev"
 Requires-Dist: flake8; extra == "dev"
 Requires-Dist: mypy; extra == "dev"
 Provides-Extra: monitor
-Requires-Dist: python-socketio>=5.11.0; extra == "monitor"
-Requires-Dist: aiohttp>=3.9.0; extra == "monitor"
-Requires-Dist: python-engineio>=4.8.0; extra == "monitor"
 Dynamic: author-email
 Dynamic: home-page
 Dynamic: license-file

{claude_mpm-3.4.7.dist-info → claude_mpm-3.4.10.dist-info}/RECORD

@@ -43,7 +43,7 @@ claude_mpm/cli/commands/agents.py,sha256=FqqEQcfAfCxjz_E7fGQUtLznloJLz8fWQtnjQhk
 claude_mpm/cli/commands/info.py,sha256=ETL6jC08OTQVTPjs219Y0m3FzfKOUlI0-yI81AI8FXY,2990
 claude_mpm/cli/commands/memory.py,sha256=6jYD1bgfnWA0DvBpLJnZCPYMRoGAPBopAED8Qr-iIos,37357
 claude_mpm/cli/commands/monitor.py,sha256=80_tmSdfn_2cYpzxxPu9GnvFW0eixlSJ4wCqbn8VSCM,12407
-claude_mpm/cli/commands/run.py,sha256=j8kFTdx4fwCR400NH-I0ORMc76J62IxUeVDaaq_DG7E,28312
+claude_mpm/cli/commands/run.py,sha256=Rv6RT2S9WN8KcqX5e9rtmALP8yiOSw8ojl_Qp7gugqg,30234
 claude_mpm/cli/commands/tickets.py,sha256=SXyGtHSyGJwTeJwDAHf7kRbdiG1DlZkXkod5UoNy7Ik,2150
 claude_mpm/cli/commands/ui.py,sha256=FhBQiOKW61cNduyryRu0UhC366d6o1eEkBgbPd7Au1w,1900
 claude_mpm/cli_module/__init__.py,sha256=CkMp4gzWKoZZF_qKyBDi2sQaZw_GLWZYLtKouv-4f8s,390
@@ -184,7 +184,7 @@ claude_mpm/ui/rich_terminal_ui.py,sha256=gx_9TD0kR4Exq867pHjVVs_b-gw1q267t5oA2ZD
 claude_mpm/ui/terminal_ui.py,sha256=E_M-L-6EuGp6L8pRpaWEhTqf-ddDXhZp5W85D0XNRPw,11928
 claude_mpm/utils/__init__.py,sha256=E8Hvv6ykL6rnnc8-YmfoGNpRCZbcIirxcFuNz7YvDIg,346
 claude_mpm/utils/config_manager.py,sha256=TlekZYIWOz_ouWHQU4Gc-zckhoFK9AqA25b6A_XZdDc,16412
-claude_mpm/utils/dependency_manager.py,sha256=LcMFoXyGcqEHVvmNKpU-AyyYclgI6a722Mt9YVejXTM,7243
+claude_mpm/utils/dependency_manager.py,sha256=2Gs881HkzwMEm7X2z7Lx8l2Lws4PvS-rPBuH737-rkg,7543
 claude_mpm/utils/error_handler.py,sha256=W_Zc0FrKudpXvxT66Oxx0v8WkR8HA2KlynzzGy3eGcU,8168
 claude_mpm/utils/framework_detection.py,sha256=nzs1qRZK9K-zT0382z1FpGDvgzUNrUg8rBL-O_WLq-Q,1217
 claude_mpm/utils/import_migration_example.py,sha256=W4a4XH3FY_VBB00BB8Lae2aRPM021PxLHzdUfEs0B5w,2463
@@ -193,9 +193,9 @@ claude_mpm/utils/path_operations.py,sha256=6pLMnAWBVzHkgp6JyQHmHbGD-dWn-nX21yV4E
 claude_mpm/utils/paths.py,sha256=Xv0SZWdZRkRjN9e6clBcA165ya00GNQxt7SwMz51tfA,10153
 claude_mpm/validation/__init__.py,sha256=bJ19g9lnk7yIjtxzN8XPegp87HTFBzCrGQOpFgRTf3g,155
 claude_mpm/validation/agent_validator.py,sha256=GCA2b2rKhKDeaNyUqWxTiWIs3sDdWjD9cgOFRp9K6ic,18227
-claude_mpm-3.4.7.dist-info/licenses/LICENSE,sha256=cSdDfXjoTVhstrERrqme4zgxAu4GubU22zVEHsiXGxs,1071
-claude_mpm-3.4.7.dist-info/METADATA,sha256=tT9hiWd9pr90NS5y_UbVARexJZv7rFW09EdGSnwJttQ,6523
-claude_mpm-3.4.7.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
-claude_mpm-3.4.7.dist-info/entry_points.txt,sha256=3_d7wLrg9sRmQ1SfrFGWoTNL8Wrd6lQb2XVSYbTwRIg,324
-claude_mpm-3.4.7.dist-info/top_level.txt,sha256=1nUg3FEaBySgm8t-s54jK5zoPnu3_eY6EP6IOlekyHA,11
-claude_mpm-3.4.7.dist-info/RECORD,,
+claude_mpm-3.4.10.dist-info/licenses/LICENSE,sha256=cSdDfXjoTVhstrERrqme4zgxAu4GubU22zVEHsiXGxs,1071
+claude_mpm-3.4.10.dist-info/METADATA,sha256=IEX7hJ0U89JTyyFHps19mQ7qVFBENPk0z-nEGVIVOXU,6464
+claude_mpm-3.4.10.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
+claude_mpm-3.4.10.dist-info/entry_points.txt,sha256=3_d7wLrg9sRmQ1SfrFGWoTNL8Wrd6lQb2XVSYbTwRIg,324
+claude_mpm-3.4.10.dist-info/top_level.txt,sha256=1nUg3FEaBySgm8t-s54jK5zoPnu3_eY6EP6IOlekyHA,11
+claude_mpm-3.4.10.dist-info/RECORD,,