@agentunion/kite 1.3.1 → 1.4.0

package/extensions/services/model_service/entry.py

@@ -23,7 +23,113 @@ import websockets
 
 
 # ── Module configuration ──
-MODULE_NAME = "model_service"
+
+def _load_module_config() -> dict:
+    """Load module configuration from module.md frontmatter.
+
+    Returns:
+        Dict with keys: name, preferred_port, advertise_ip
+
+    Raises:
+        SystemExit: If module.md is invalid or name is non-compliant
+    """
+    _this_dir = os.path.dirname(os.path.abspath(__file__))
+    module_md = os.path.join(_this_dir, "module.md")
+
+    # Calculate relative path for error messages
+    project_root = os.environ.get("KITE_PROJECT", "")
+    if project_root and _this_dir.startswith(project_root):
+        rel_path = os.path.relpath(_this_dir, project_root)
+    else:
+        rel_path = _this_dir
+
+    # Default values (will be overridden if valid config exists)
+    result = {
+        "name": "",
+        "preferred_port": 0,
+        "advertise_ip": "0.0.0.0"
+    }
+
+    # Check if module.md exists
+    if not os.path.exists(module_md):
+        print(f"[{rel_path}] ERROR: Invalid module configuration in module.md")
+        print(f"  Path: {rel_path}/module.md")
+        print(f"  Reason: File not found")
+        sys.exit(1)
+
+    try:
+        with open(module_md, encoding="utf-8") as f:
+            text = f.read()
+
+        # Extract YAML frontmatter (between --- markers)
+        import re
+        m = re.match(r'^---\s*\n(.*?)\n---', text, re.DOTALL)
+        if not m:
+            print(f"[{rel_path}] ERROR: Invalid module configuration in module.md")
+            print(f"  Path: {rel_path}/module.md")
+            print(f"  Reason: Missing YAML frontmatter")
+            sys.exit(1)
+
+        # Parse YAML frontmatter
+        try:
+            import yaml
+            fm = yaml.safe_load(m.group(1)) or {}
+        except ImportError:
+            print(f"[{rel_path}] ERROR: PyYAML not installed, cannot parse module.md")
+            sys.exit(1)
+        except Exception as e:
+            print(f"[{rel_path}] ERROR: Invalid module configuration in module.md")
+            print(f"  Path: {rel_path}/module.md")
+            print(f"  Reason: YAML parse error: {e}")
+            sys.exit(1)
+
+        # Validate 'name' field (required)
+        if "name" not in fm:
+            print(f"[{rel_path}] ERROR: Invalid module configuration in module.md")
+            print(f"  Path: {rel_path}/module.md")
+            print(f"  Reason: Missing 'name' field")
+            sys.exit(1)
+
+        raw_name = str(fm["name"]).strip()
+
+        if not raw_name:
+            print(f"[{rel_path}] ERROR: Invalid module configuration in module.md")
+            print(f"  Path: {rel_path}/module.md")
+            print(f"  Reason: Empty module name")
+            sys.exit(1)
+
+        # Validate name characters
+        sanitized = re.sub(r'[^a-zA-Z0-9_\-]', '', raw_name)
+
+        if sanitized != raw_name:
+            invalid_chars = ''.join(sorted(set(c for c in raw_name if c not in sanitized)))
+            print(f"[{rel_path}] ERROR: Invalid module configuration in module.md")
+            print(f"  Path: {rel_path}/module.md")
+            print(f"  Reason: Invalid characters in name '{raw_name}': {repr(invalid_chars)}")
+            sys.exit(1)
+
+        result["name"] = sanitized
+
+        # Extract optional fields
+        if "preferred_port" in fm:
+            try:
+                result["preferred_port"] = int(fm["preferred_port"])
+            except (ValueError, TypeError):
+                pass
+
+        if "advertise_ip" in fm:
+            result["advertise_ip"] = str(fm["advertise_ip"])
+
+    except SystemExit:
+        raise  # Re-raise exit to prevent catching by outer except
+    except Exception as e:
+        print(f"[{rel_path}] ERROR: Failed to read module.md: {e}")
+        sys.exit(1)
+
+    return result
+
+_module_config = _load_module_config()
+MODULE_NAME = _module_config["name"]
 
 
 class _SafeWriter:
@@ -263,10 +369,20 @@ def _read_stdin_kite_message(expected_type: str, timeout: float = 10) -> dict |
 
 # Global WS reference for publish_event callback
 _ws_global = None
+_shutting_down = False
+_exit_code = 0  # Exit code for main() to use
+
+
+def _is_auth_failure(e: Exception) -> bool:
+    """Check if a WebSocket exception indicates authentication failure."""
+    if hasattr(e, 'rcvd') and e.rcvd is not None:
+        code = e.rcvd.code if hasattr(e.rcvd, 'code') else 0
+        return code in (4001, 4003)
+    return False
 
 
 async def main():
-    global _ws_global
+    global _ws_global, _shutting_down
     # Initialize log file paths
     global _log_dir, _log_latest_path, _crash_log_path
     module_data = os.environ.get("KITE_MODULE_DATA")
@@ -318,41 +434,90 @@ async def main():
 
     print(f"[model_service] Token received ({len(token)} chars), kernel port: {kernel_port} ({_fmt_elapsed(_t0)})")
 
-    # Connect to Kernel WebSocket
+    # Start reconnect loop
+    await _ws_loop(token, kernel_port, _t0)
+
+
+async def _ws_loop(token: str, kernel_port: int, _t0: float):
+    """Connect to Kernel with exponential backoff reconnection."""
+    global _shutting_down, _exit_code
+    retry_delay = 0.3
+    max_delay = 5.0
+    max_retries = 10
+    attempt = 0
+    while not _shutting_down:
+        try:
+            await _ws_connect(token, kernel_port, _t0)
+            retry_delay = 0.3
+            attempt = 0
+        except asyncio.CancelledError:
+            return
+        except Exception as e:
+            attempt += 1
+            if _is_auth_failure(e):
+                print(f"[model_service] Kernel 认证失败，退出")
+                _exit_code = 1
+                _shutting_down = True
+                return
+            if attempt >= max_retries:
+                print(f"[model_service] 重连失败 {max_retries} 次，退出")
+                _exit_code = 1
+                _shutting_down = True
+                return
+            _write_crash(type(e), e, e.__traceback__, severity="error", handled=True)
+            print(f"[model_service] 连接错误: {e}, {retry_delay:.1f}s 后重试 ({attempt}/{max_retries})")
+        _ws_global_clear()
+        if _shutting_down:
+            return
+        await asyncio.sleep(retry_delay)
+        retry_delay = min(retry_delay * 2, max_delay)
+
+
+def _ws_global_clear():
+    global _ws_global
+    _ws_global = None
+
+
+async def _ws_connect(token: str, kernel_port: int, _t0: float):
+    """Single WebSocket session: connect → subscribe → register → ready → receive loop."""
+    global _ws_global
+
     ws_url = f"ws://127.0.0.1:{kernel_port}/ws?token={token}&id=model_service"
     print(f"[model_service] Connecting to Kernel: {ws_url}")
 
-    try:
-        async with websockets.connect(ws_url, open_timeout=5, ping_interval=None, ping_timeout=None, close_timeout=10) as ws:
-            _ws_global = ws
-            print(f"[model_service] Connected to Kernel ({_fmt_elapsed(_t0)})")
-
-            # Subscribe to events
-            await _rpc_call(ws, "event.subscribe", {
-                "events": [
-                    "module.started",
-                    "module.stopped",
-                    "module.shutdown",
-                ],
-            })
-            print(f"[model_service] Subscribed to events ({_fmt_elapsed(_t0)})")
-
-            # Register to Kernel Registry via RPC
-            await _rpc_call(ws, "registry.register", {
-                "module_id": "model_service",
-                "module_type": "service",
-                "events_publish": {
-                    "model_service.test": {"description": "Test event from model_service module"},
-                },
-                "events_subscribe": [
-                    "module.started",
-                    "module.stopped",
-                    "module.shutdown",
-                ],
-            })
-            print(f"[model_service] Registered to Kernel ({_fmt_elapsed(_t0)})")
+    async with websockets.connect(ws_url, open_timeout=5, ping_interval=20, ping_timeout=20, close_timeout=10) as ws:
+        _ws_global = ws
+        print(f"[model_service] Connected to Kernel ({_fmt_elapsed(_t0)})")
+
+        # Subscribe to events
+        await _rpc_call(ws, "event.subscribe", {
+            "events": [
+                "module.started",
+                "module.stopped",
+                "module.shutdown",
+            ],
+        })
+        print(f"[model_service] Subscribed to events ({_fmt_elapsed(_t0)})")
+
+        # Register to Kernel Registry via RPC
+        await _rpc_call(ws, "registry.register", {
+            "module_id": "model_service",
+            "module_type": "service",
+            "events_publish": {
+                "model_service": {
+                    "test": {"description": "Test event from model_service module"},
+                }
+            },
+            "events_subscribe": [
+                "module.started",
+                "module.stopped",
+                "module.shutdown",
+            ],
+        })
+        print(f"[model_service] Registered to Kernel ({_fmt_elapsed(_t0)})")
 
-            # Publish module.ready
+        # Publish module.ready (every reconnect)
+        if not _shutting_down:
             await _rpc_call(ws, "event.publish", {
                 "event_id": str(uuid.uuid4()),
                 "event": "module.ready",
@@ -363,34 +528,34 @@ async def main():
             })
             print(f"[model_service] module.ready published ({_fmt_elapsed(_t0)})")
 
-            # Start test event loop in background
-            test_task = asyncio.create_task(_test_event_loop(ws))
-
-            # Message loop: handle incoming RPC + events
-            async for raw in ws:
-                try:
-                    msg = json.loads(raw)
-                except (json.JSONDecodeError, TypeError):
-                    continue
-
-                try:
-                    has_method = "method" in msg
-                    has_id = "id" in msg
-
-                    if has_method and not has_id:
-                        # Event Notification
-                        await _handle_event_notification(msg)
-                    elif has_method and has_id:
-                        # Incoming RPC request
-                        await _handle_rpc_request(ws, msg)
-                    # Ignore RPC responses (we don't await them in this simple impl)
-                except Exception as e:
-                    print(f"[model_service] 消息处理异常（已忽略）: {e}")
+        # Start test event loop in background
+        test_task = asyncio.create_task(_test_event_loop(ws))
 
-    except Exception as e:
-        _write_crash(type(e), e, e.__traceback__, severity="critical", handled=True)
-        _print_crash_summary(type(e), e.__traceback__)
-        sys.exit(1)
+        # Message loop: handle incoming RPC + events
+        # CRITICAL: RPC 死锁防范
+        # - 入站 RPC 请求必须用 create_task() 异步执行，不可 await
+        # - 原因：如果 handler 内部调用 rpc_call() 发出站请求，出站响应需要本接收循环来分发
+        # - 如果接收循环被 await handler 阻塞，出站响应永远收不到 → 超时死锁
+        # - 事件通知和 RPC 响应可以同步处理（它们不会反向调用 rpc_call）
+        async for raw in ws:
+            try:
+                msg = json.loads(raw)
+            except (json.JSONDecodeError, TypeError):
+                continue
+
+            try:
+                has_method = "method" in msg
+                has_id = "id" in msg
+
+                if has_method and not has_id:
+                    # Event Notification
+                    await _handle_event_notification(msg)
+                elif has_method and has_id:
+                    # Incoming RPC request — run in background to prevent deadlock
+                    asyncio.create_task(_handle_rpc_request(ws, msg))
+                # Ignore RPC responses (we don't await them in this simple impl)
+            except Exception as e:
+                print(f"[model_service] 消息处理异常（已忽略）: {e}")
 
 
 async def _rpc_call(ws, method: str, params: dict = None):
@@ -416,10 +581,14 @@ async def _handle_event_notification(msg: dict):
     event_type = params.get("event", "")
     data = params.get("data", {})
 
-    # Special handling for module.shutdown targeting model_service
-    if event_type == "module.shutdown" and data.get("module_id") == "model_service":
-        await _handle_shutdown()
-        return
+    # Special handling for module.shutdown
+    if event_type == "module.shutdown":
+        target = data.get("module_id", "")
+        reason = data.get("reason", "")
+        # Handle both targeted shutdown (module_id == "model_service") and broadcast shutdown (no module_id or launcher_lost)
+        if target == "model_service" or not target or reason == "launcher_lost":
+            await _handle_shutdown()
+            return
 
     # Log other events
     print(f"[model_service] Event received: {event_type}")
@@ -472,22 +641,37 @@ async def _rpc_status() -> dict:
 
 
 async def _handle_shutdown():
-    """Handle module.shutdown event — ack, cleanup, ready, exit."""
+    """Handle module.shutdown event — ack → exiting → cleanup → ready → exit."""
+    global _shutting_down
     print("[model_service] Received shutdown request")
-    # Step 1: Send ack
+    _shutting_down = True
+    # Step 1: Send ack (立即确认收到)
     await _publish_event(_ws_global, {
         "event": "module.shutdown.ack",
-        "data": {"module_id": "model_service", "estimated_cleanup": 2},
+        "data": {"module_id": "model_service"},
+    })
+    # Step 2: Send module.exiting (开始清理)
+    await _publish_event(_ws_global, {
+        "event": "module.exiting",
+        "data": {
+            "module_id": "model_service",
+            "type": "passive",
+            "reason": "shutdown_requested",
+            "restart": "auto",
+            "action": "none",
+            "timeout": 2.0,
+            "restart_delay": 0.0,
+        },
     })
-    # Step 2: Cleanup (nothing to clean up for model_service)
-    # Step 3: Send ready
+    # Step 3: Cleanup (nothing to clean up for model_service)
+    # Step 4: Send ready (清理完成)
     await _publish_event(_ws_global, {
         "event": "module.shutdown.ready",
         "data": {"module_id": "model_service"},
     })
     print("[model_service] Shutdown ready, exiting")
-    # Step 4: Exit
-    sys.exit(0)
+    # Step 5: Exit
+    sys.exit(_exit_code)
 
 
 async def _test_event_loop(ws):

package/extensions/services/model_service/module.md

@@ -1,22 +1,21 @@
----
-name: model_service
-display_name: Model Service
-version: "1.0"
-type: service
-state: enabled
-runtime: python
-entry: entry.py
-events:
-  - model_service.test
-subscriptions:
-  - module.started
-  - module.stopped
-  - module.shutdown
----
-
-# Model Service（大模型服务）
-
-大模型服务模块，提供统一的 LLM 调用接口。
-
-- 模型调用 — 封装多种大模型 API 的统一调用接口
-- 事件通知 — 通过 Event Hub 发布模型服务状态事件
+---
+name: model_service
+display_name: Model Service
+version: '1.0'
+type: service
+state: manual
+runtime: python
+entry: entry.py
+events:
+- model_service.test
+subscriptions:
+- module.started
+- module.stopped
+- module.shutdown
+---
+# Model Service（大模型服务）
+
+大模型服务模块，提供统一的 LLM 调用接口。
+
+- 模型调用 — 封装多种大模型 API 的统一调用接口
+- 事件通知 — 通过 Event Hub 发布模型服务状态事件