ripperdoc 0.2.10__py3-none-any.whl → 0.3.1__py3-none-any.whl

ripperdoc/cli/ui/spinner.py

@@ -1,4 +1,6 @@
 from contextlib import contextmanager
+import shutil
+import sys
 from typing import Any, Generator, Literal, Optional
 
 from rich.console import Console
@@ -6,31 +8,78 @@ from rich.live import Live
 from rich.text import Text
 from rich.spinner import Spinner as RichSpinner
 
+from ripperdoc.core.theme import theme_color
+
+# ANSI escape sequences for terminal control
+_CLEAR_LINE = "\r\033[K"  # Move to start of line and clear to end
+
 
 class Spinner:
     """Lightweight spinner wrapper that plays nicely with other console output."""
 
+    # Reserve space for spinner animation (e.g., "⠧ ") and safety margin
+    _SPINNER_MARGIN = 6
+
     def __init__(self, console: Console, text: str = "Thinking...", spinner: str = "dots"):
         self.console = console
         self.text = text
         self.spinner = spinner
-        self._style = "cyan"
+        self._style = theme_color("spinner")
         self._live: Optional[Live] = None
-        # Blue spinner for clearer visual separation in the terminal (icon + text)
+        # Spinner color from theme for visual separation in the terminal
         self._renderable: RichSpinner = RichSpinner(
-            spinner, text=Text(self.text, style=self._style), style=self._style
+            spinner,
+            text=Text(self._fit_to_terminal(self.text), style=self._style),
+            style=self._style,
         )
 
+    def _get_terminal_width(self) -> int:
+        """Get current terminal width, with fallback."""
+        try:
+            return shutil.get_terminal_size().columns
+        except Exception:
+            return 80  # Reasonable default
+
+    def _fit_to_terminal(self, text: str) -> str:
+        """Truncate text to fit within terminal width, preventing line wrap issues.
+
+        This ensures spinner text never causes terminal wrapping, which would
+        leave artifacts when the spinner refreshes or stops.
+        """
+        max_width = self._get_terminal_width() - self._SPINNER_MARGIN
+        if max_width < 20:
+            max_width = 20  # Minimum usable width
+
+        if len(text) <= max_width:
+            return text
+
+        # Smart truncation: keep the structure intact
+        # Find the last complete parenthetical group if possible
+        truncated = text[: max_width - 1] + "…"
+        return truncated
+
+    def _clear_line(self) -> None:
+        """Clear the current terminal line to prevent artifacts."""
+        if self.console.is_terminal:
+            try:
+                sys.stdout.write(_CLEAR_LINE)
+                sys.stdout.flush()
+            except Exception:
+                pass  # Ignore errors in non-TTY environments
+
     def start(self) -> None:
         """Start the spinner if not already running."""
         if self._live is not None:
             return
-        self._renderable.text = Text(self.text, style=self._style)
+        # Clear any residual content on current line before starting
+        self._clear_line()
+        self._renderable.text = Text(self._fit_to_terminal(self.text), style=self._style)
         self._live = Live(
             self._renderable,
             console=self.console,
             transient=True,  # Remove spinner line when stopped to avoid layout glitches
             refresh_per_second=12,
+            vertical_overflow="ellipsis",  # Prevent multi-line overflow issues
         )
         self._live.start()
 
@@ -40,7 +89,7 @@ class Spinner:
             return
         if text is not None:
             self.text = text
-        self._renderable.text = Text(self.text, style=self._style)
+        self._renderable.text = Text(self._fit_to_terminal(self.text), style=self._style)
         # Live.refresh() redraws the current renderable
         self._live.refresh()
 
@@ -50,6 +99,8 @@ class Spinner:
             return
         try:
             self._live.stop()
+            # Clear line to ensure no artifacts remain from long spinner text
+            self._clear_line()
         finally:
             self._live = None
 
@@ -82,4 +133,16 @@ class Spinner:
             yield
         finally:
             if was_running:
+                # Ensure all output is flushed and cursor is on a clean line
+                # before restarting the spinner
+                try:
+                    # Flush console buffer
+                    self.console.file.flush()
+                    # Clear any partial line content to prevent spinner
+                    # from appearing on the same line as previous output
+                    if self.console.is_terminal:
+                        sys.stdout.write(_CLEAR_LINE)
+                        sys.stdout.flush()
+                except Exception:
+                    pass
                 self.start()

ripperdoc/cli/ui/tool_renderers.py

@@ -43,7 +43,7 @@ class TodoResultRenderer(ToolResultRenderer):
         if lines:
             self.console.print(f"  ⎿  [dim]{escape(lines[0])}[/]")
             for line in lines[1:]:
-                self.console.print(f"      {line}", markup=False)
+                self.console.print(f"    {line}", markup=False)
         else:
             self.console.print("  ⎿  [dim]Todo update[/]")
 
@@ -107,7 +107,7 @@ class GlobResultRenderer(ToolResultRenderer):
         if self.verbose:
             for line in files[:30]:
                 if line.strip():
-                    self.console.print(f"      {line}", markup=False)
+                    self.console.print(f"    {line}", markup=False)
             if file_count > 30:
                 self.console.print(f"[dim]... ({file_count - 30} more)[/]")
 
@@ -125,7 +125,7 @@ class GrepResultRenderer(ToolResultRenderer):
         if self.verbose:
             for line in matches[:30]:
                 if line.strip():
-                    self.console.print(f"      {line}", markup=False)
+                    self.console.print(f"    {line}", markup=False)
             if match_count > 30:
                 self.console.print(f"[dim]... ({match_count - 30} more)[/]")
 
@@ -142,7 +142,7 @@ class LSResultRenderer(ToolResultRenderer):
         if self.verbose:
             preview = tree_lines[:40]
             for line in preview:
-                self.console.print(f"      {line}", markup=False)
+                self.console.print(f"    {line}", markup=False)
             if len(tree_lines) > len(preview):
                 self.console.print(f"[dim]... ({len(tree_lines) - len(preview)} more)[/]")
 
@@ -193,7 +193,8 @@ class BashResultRenderer(ToolResultRenderer):
             preview = stdout_lines if self.verbose else stdout_lines[:5]
             self.console.print(f"  ⎿  {preview[0]}", markup=False)
             for line in preview[1:]:
-                self.console.print(f"      {line}", markup=False)
+                # Use consistent 4-space indent to match the ⎿ prefix width
+                self.console.print(f"    {line}", markup=False)
             if not self.verbose and len(stdout_lines) > len(preview):
                 self.console.print(f"[dim]... ({len(stdout_lines) - len(preview)} more lines)[/]")
         else:
@@ -229,28 +230,28 @@ class BashResultRenderer(ToolResultRenderer):
             preview = stdout_lines if self.verbose else stdout_lines[:5]
             self.console.print("[dim]stdout:[/]")
             for line in preview:
-                self.console.print(f"      {line}", markup=False)
+                self.console.print(f"    {line}", markup=False)
             if not self.verbose and len(stdout_lines) > len(preview):
                 self.console.print(
                     f"[dim]... ({len(stdout_lines) - len(preview)} more stdout lines)[/]"
                 )
         else:
             self.console.print("[dim]stdout:[/]")
-            self.console.print("      [dim](no stdout)[/]")
+            self.console.print("    [dim](no stdout)[/]")
 
         # Render stderr
         if stderr_lines:
             preview = stderr_lines if self.verbose else stderr_lines[:5]
             self.console.print("[dim]stderr:[/]")
             for line in preview:
-                self.console.print(f"      {line}", markup=False)
+                self.console.print(f"    {line}", markup=False)
             if not self.verbose and len(stderr_lines) > len(preview):
                 self.console.print(
                     f"[dim]... ({len(stderr_lines) - len(preview)} more stderr lines)[/]"
                 )
         else:
             self.console.print("[dim]stderr:[/]")
-            self.console.print("      [dim](no stderr)[/]")
+            self.console.print("    [dim](no stderr)[/]")
 
 
 class ToolResultRendererRegistry:

ripperdoc/cli/ui/wizard.py

@@ -2,6 +2,7 @@
 Interactive onboarding wizard for Ripperdoc.
 """
 
+import os
 from typing import List, Optional, Tuple
 
 import click
@@ -46,6 +47,17 @@ def check_onboarding() -> bool:
     if config.has_completed_onboarding:
         return True
 
+    # 检查是否有有效的 RIPPERDOC_* 环境变量配置
+    # 如果设置了 RIPPERDOC_BASE_URL，可以跳过 onboarding
+    # 不写入配置文件，只在内存中处理
+    if os.getenv("RIPPERDOC_BASE_URL"):
+        # 在内存中标记已完成 onboarding，但不保存到配置文件
+        # 这样下次启动时如果环境变量存在仍然可以工作
+        config.has_completed_onboarding = True
+        config.last_onboarding_version = get_version()
+        save_global_config(config)
+        return True
+
     console.print("[bold cyan]Welcome to Ripperdoc![/bold cyan]\n")
     console.print("Let's set up your AI model configuration.\n")
 
@@ -135,7 +147,7 @@ def get_model_name_with_suggestions(
     api_base_override: Optional[str],
 ) -> Tuple[str, Optional[str]]:
     """Get model name with provider-specific suggestions and default API base.
-    
+
     Returns:
         Tuple of (model_name, api_base)
     """
@@ -152,7 +164,7 @@ def get_model_name_with_suggestions(
     if suggestions:
         console.print("\n[dim]Available models for this provider:[/dim]")
         for i, model_name in enumerate(suggestions[:5]):  # Show top 5
-            console.print(f"  [dim]{i+1}. {model_name}[/dim]")
+            console.print(f"  [dim]{i + 1}. {model_name}[/dim]")
         console.print("")
 
     # Prompt for model name
@@ -162,16 +174,12 @@ def get_model_name_with_suggestions(
         model = click.prompt("Model name", default=default_model)
         # Prompt for API base if still not set
         if api_base is None:
-            api_base_input = click.prompt(
-                "API base URL (optional)", default="", show_default=False
-            )
+            api_base_input = click.prompt("API base URL (optional)", default="", show_default=False)
             api_base = api_base_input or None
     elif provider.protocol == ProviderType.GEMINI:
         model = click.prompt("Model name", default=default_model)
         if api_base is None:
-            api_base_input = click.prompt(
-                "API base URL (optional)", default="", show_default=False
-            )
+            api_base_input = click.prompt("API base URL (optional)", default="", show_default=False)
             api_base = api_base_input or None
     else:
         model = click.prompt("Model name", default=default_model)
@@ -191,9 +199,7 @@ def get_context_window() -> Optional[int]:
         try:
             context_window = int(context_window_input.strip())
         except ValueError:
-            console.print(
-                "[yellow]Invalid context window, using auto-detected defaults.[/yellow]"
-            )
+            console.print("[yellow]Invalid context window, using auto-detected defaults.[/yellow]")
     return context_window
 
 
@@ -201,6 +207,7 @@ def get_version() -> str:
     """Get current version of Ripperdoc."""
     try:
         from ripperdoc import __version__
+
         return __version__
     except ImportError:
         return "unknown"

ripperdoc/core/agents.py

@@ -24,8 +24,10 @@ from ripperdoc.tools.glob_tool import GlobTool
 from ripperdoc.tools.grep_tool import GrepTool
 from ripperdoc.tools.kill_bash_tool import KillBashTool
 from ripperdoc.tools.ls_tool import LSTool
+from ripperdoc.tools.lsp_tool import LspTool
 from ripperdoc.tools.multi_edit_tool import MultiEditTool
 from ripperdoc.tools.notebook_edit_tool import NotebookEditTool
+from ripperdoc.tools.skill_tool import SkillTool
 from ripperdoc.tools.todo_tool import TodoReadTool, TodoWriteTool
 from ripperdoc.tools.tool_search_tool import ToolSearchTool
 from ripperdoc.tools.mcp_tools import (
@@ -66,6 +68,8 @@ TOOL_SEARCH_TOOL_NAME = _safe_tool_name(ToolSearchTool, "ToolSearch")
 MCP_LIST_SERVERS_TOOL_NAME = _safe_tool_name(ListMcpServersTool, "ListMcpServers")
 MCP_LIST_RESOURCES_TOOL_NAME = _safe_tool_name(ListMcpResourcesTool, "ListMcpResources")
 MCP_READ_RESOURCE_TOOL_NAME = _safe_tool_name(ReadMcpResourceTool, "ReadMcpResource")
+LSP_TOOL_NAME = _safe_tool_name(LspTool, "LSP")
+SKILL_TOOL_NAME = _safe_tool_name(SkillTool, "Skill")
 TASK_TOOL_NAME = "Task"
 
 

ripperdoc/core/config.py

@@ -94,6 +94,69 @@ def api_base_env_candidates(provider: ProviderType) -> list[str]:
     ]
 
 
+# Known vision-enabled model patterns for auto-detection
+VISION_ENABLED_MODELS = {
+    # Anthropic Claude models
+    "claude-haiku-4-5-20251001",
+    "claude-sonnet-4-5-20250929",
+    "claude-opus-4-5-20251101",
+    "claude-haiku-4-5",
+    "claude-sonnet-4-5",
+    "claude-opus-4-5",
+    "claude-3-5-sonnet",
+    "claude-3-5-sonnet-20241022",
+    "claude-3-5-sonnet-20240620",
+    "claude-3-5-haiku",
+    "claude-3-5-haiku-20241022",
+    "claude-3-opus",
+    "claude-3-opus-20240229",
+    "claude-3-sonnet",
+    "claude-3-sonnet-20240229",
+    "claude-3-haiku",
+    "claude-3-haiku-20240307",
+    # OpenAI models
+    "gpt-4o",
+    "gpt-4o-2024-08-06",
+    "gpt-4o-mini",
+    "gpt-4o-mini-2024-07-18",
+    "gpt-4-turbo",
+    "gpt-4-turbo-2024-04-09",
+    "gpt-4",
+    "gpt-4-0314",
+    "gpt-4-vision-preview",
+    "chatgpt-4o-latest",
+    # Google Gemini models
+    "gemini-3-pro-preview",
+    "gemini-3-flash-preview",
+    "gemini-2.5-pro",
+    "gemini-2.5-flash-lite",
+    "gemini-2.5-flash",
+    "gemini-2.0-flash-exp",
+    "gemini-2.0-flash-thinking-exp",
+    "gemini-exp-1206",
+    "gemini-pro-vision",
+    "gemini-1.5-pro",
+    "gemini-1.5-pro-001",
+    "gemini-1.5-flash",
+    "gemini-1.5-flash-001",
+    # Alibaba Qwen models (vision)
+    "qwen-vl-max",
+    "qwen-vl-plus",
+    "qwen-vl-plus-latest",
+    "qwen2-vl-72b-instruct",
+    "qwen-vl-chat",
+    "qwen-vl-7b-chat",
+    # DeepSeek models (some support vision)
+    "deepseek-vl",
+    "deepseek-vl-chat",
+    # Other vision models
+    "glm-4v",
+    "glm-4v-plus",
+    "minivision-3b",
+    "internvl2",
+}
+
+
 class ModelProfile(BaseModel):
     """Configuration for a specific AI model."""
 
@@ -113,11 +176,31 @@ class ModelProfile(BaseModel):
     # Optional override for thinking protocol handling (e.g., "deepseek", "openrouter",
     # "qwen", "gemini_openai", "openai"). When unset, provider heuristics are used.
     thinking_mode: Optional[str] = None
+    # Vision support flag. None = auto-detect based on model name, True/False = override.
+    supports_vision: Optional[bool] = None
     # Pricing (USD per 1M tokens). Leave as 0 to skip cost calculation.
     input_cost_per_million_tokens: float = 0.0
     output_cost_per_million_tokens: float = 0.0
 
 
+def model_supports_vision(model_profile: ModelProfile) -> bool:
+    """Detect whether a model supports vision/image input.
+
+    Args:
+        model_profile: The model profile to check
+
+    Returns:
+        True if the model supports vision capabilities, False otherwise
+    """
+    # If explicitly configured, use the config value
+    if model_profile.supports_vision is not None:
+        return model_profile.supports_vision
+
+    # Auto-detect based on model name
+    model_name = model_profile.model.lower()
+    return any(pattern in model_name for pattern in VISION_ENABLED_MODELS)
+
+
 class ModelPointers(BaseModel):
     """Pointers to different model profiles for different purposes."""
 
@@ -141,6 +224,8 @@ class GlobalConfig(BaseModel):
     show_full_thinking: bool = Field(default=False)
     auto_compact_enabled: bool = True
     context_token_limit: Optional[int] = None
+    # Default thinking tokens budget when thinking mode is enabled (0 = disabled by default)
+    default_thinking_tokens: int = Field(default=10240)
 
     # User-level permission rules (applied globally)
     user_allow_rules: list[str] = Field(default_factory=list)
@@ -591,3 +676,153 @@ def save_project_local_config(
 ) -> None:
     """Save project-local configuration."""
     config_manager.save_project_local_config(config, project_path)
+
+
+# ==============================================================================
+# RIPPERDOC_* 全局环境变量支持
+# ==============================================================================
+
+# 环境变量名称常量
+RIPPERDOC_BASE_URL = "RIPPERDOC_BASE_URL"
+RIPPERDOC_AUTH_TOKEN = "RIPPERDOC_AUTH_TOKEN"
+RIPPERDOC_MODEL = "RIPPERDOC_MODEL"
+RIPPERDOC_SMALL_FAST_MODEL = "RIPPERDOC_SMALL_FAST_MODEL"
+RIPPERDOC_API_KEY = "RIPPERDOC_API_KEY"
+RIPPERDOC_PROTOCOL = "RIPPERDOC_PROTOCOL"
+
+
+def _infer_protocol_from_url_and_model(base_url: str, model_name: str = "") -> ProviderType:
+    """根据 BASE_URL 和模型名称推断协议类型.
+
+    Args:
+        base_url: API基础URL
+        model_name: 模型名称
+
+    Returns:
+        推断的 ProviderType
+    """
+    base_lower = base_url.lower()
+    model_lower = model_name.lower()
+
+    # 显式域名检测
+    if "anthropic.com" in base_lower:
+        return ProviderType.ANTHROPIC
+    if "generativelanguage.googleapis.com" in base_lower or "gemini" in model_lower:
+        return ProviderType.GEMINI
+
+    # URL 路径检测 - 检查路径中是否包含协议标识
+    if "/anthropic" in base_lower or base_lower.endswith("/anthropic"):
+        return ProviderType.ANTHROPIC
+    if "/v1/" in base_lower or "/v1" in base_lower:
+        # 大多数 /v1/ 路径是 OpenAI 兼容格式
+        return ProviderType.OPENAI_COMPATIBLE
+
+    # 模型名称前缀检测
+    if model_lower.startswith("claude-"):
+        return ProviderType.ANTHROPIC
+    if model_lower.startswith("gemini-"):
+        return ProviderType.GEMINI
+
+    # 默认使用 OpenAI 兼容协议
+    return ProviderType.OPENAI_COMPATIBLE
+
+
+def _get_ripperdoc_env_overrides() -> Dict[str, Any]:
+    """获取所有 RIPPERDOC_* 环境变量的值.
+
+    Returns:
+        包含所有已设置环境变量的字典
+    """
+    overrides: Dict[str, Any] = {}
+    if base_url := os.getenv(RIPPERDOC_BASE_URL):
+        overrides["base_url"] = base_url
+    if api_key := os.getenv(RIPPERDOC_API_KEY):
+        overrides["api_key"] = api_key
+    if auth_token := os.getenv(RIPPERDOC_AUTH_TOKEN):
+        overrides["auth_token"] = auth_token
+    if model := os.getenv(RIPPERDOC_MODEL):
+        overrides["model"] = model
+    if small_fast_model := os.getenv(RIPPERDOC_SMALL_FAST_MODEL):
+        overrides["small_fast_model"] = small_fast_model
+    if protocol_str := os.getenv(RIPPERDOC_PROTOCOL):
+        try:
+            overrides["protocol"] = ProviderType(protocol_str.lower())
+        except ValueError:
+            logger.warning(
+                "[config] Invalid RIPPERDOC_PROTOCOL value: %s (must be anthropic, openai_compatible, or gemini)",
+                protocol_str,
+            )
+    return overrides
+
+
+def has_ripperdoc_env_overrides() -> bool:
+    """检查是否设置了任何 RIPPERDOC_* 环境变量."""
+    return bool(_get_ripperdoc_env_overrides())
+
+
+def get_effective_model_profile(pointer: str = "main") -> Optional[ModelProfile]:
+    """获取模型配置，应用 RIPPERDOC_* 环境变量覆盖.
+
+    当设置了 RIPPERDOC_BASE_URL 环境变量时，完全在内存中创建 ModelProfile，
+    不依赖也不写入配置文件。这是获取模型配置的新入口点，替代 get_current_model_profile()。
+
+    Args:
+        pointer: 模型指针名称 ("main" 或 "quick")
+
+    Returns:
+        应用环境变量覆盖后的 ModelProfile，如果没有则返回 None
+    """
+    env_overrides = _get_ripperdoc_env_overrides()
+    base_url = env_overrides.get("base_url")
+
+    # 如果设置了 RIPPERDOC_BASE_URL，完全在内存中创建 profile
+    if base_url:
+        # 确定模型名称
+        if pointer == "quick":
+            model_name = env_overrides.get("small_fast_model") or env_overrides.get("model")
+        else:
+            model_name = env_overrides.get("model")
+
+        if not model_name:
+            model_name = "claude-sonnet-4-5-20250929"
+
+        # 确定协议类型
+        protocol = env_overrides.get("protocol")
+        if not protocol:
+            protocol = _infer_protocol_from_url_and_model(base_url, model_name)
+
+        # 在内存中创建新的 profile，不写入配置文件
+        return ModelProfile(
+            provider=protocol,
+            model=model_name,
+            api_base=base_url,
+            api_key=env_overrides.get("api_key"),
+            auth_token=env_overrides.get("auth_token"),
+        )
+
+    # 没有设置 RIPPERDOC_BASE_URL，返回配置文件中的 profile
+    return get_current_model_profile(pointer)
+
+
+def get_ripperdoc_env_status() -> Dict[str, str]:
+    """获取 RIPPERDOC_* 环境变量状态信息用于诊断显示.
+
+    Returns:
+        字典，键为环境变量名称，值为格式化的显示字符串
+    """
+    status: Dict[str, str] = {}
+    if base_url := os.getenv(RIPPERDOC_BASE_URL):
+        status["BASE_URL"] = base_url
+    if protocol := os.getenv(RIPPERDOC_PROTOCOL):
+        status["PROTOCOL"] = protocol
+    if model := os.getenv(RIPPERDOC_MODEL):
+        status["MODEL"] = model
+    if small_fast_model := os.getenv(RIPPERDOC_SMALL_FAST_MODEL):
+        status["SMALL_FAST_MODEL"] = small_fast_model
+    if api_key := os.getenv(RIPPERDOC_API_KEY):
+        masked = api_key[:4] + "…" if len(api_key) > 4 else "set"
+        status["API_KEY"] = f"{masked} (${RIPPERDOC_API_KEY})"
+    if auth_token := os.getenv(RIPPERDOC_AUTH_TOKEN):
+        masked = auth_token[:4] + "…" if len(auth_token) > 4 else "set"
+        status["AUTH_TOKEN"] = f"{masked} (${RIPPERDOC_AUTH_TOKEN})"
+    return status

ripperdoc/core/default_tools.py

@@ -93,6 +93,7 @@ def filter_tools_by_names(
 
     # If Task is requested, recreate it with the filtered base tools
     if has_task:
+
         def _filtered_base_provider() -> List[Tool[Any, Any]]:
             return [t for t in filtered if getattr(t, "name", None) != "Task"]
 

ripperdoc/core/hooks/llm_callback.py

@@ -56,4 +56,3 @@ def build_hook_llm_callback(
             return f"Prompt hook evaluation failed: {exc}"
 
     return _callback
-

ripperdoc/core/hooks/manager.py

@@ -510,10 +510,14 @@ class HookManager:
         stop_sequence: Optional[str] = None,
     ) -> HookResult:
         """Run Stop hooks asynchronously."""
+        logger.debug("[hook_manager] run_stop_async ENTER")
         hooks = self._get_hooks(HookEvent.STOP)
+        logger.debug(f"[hook_manager] run_stop_async: got {len(hooks)} hooks")
         if not hooks:
+            logger.debug("[hook_manager] run_stop_async: no hooks, returning empty HookResult")
             return HookResult([])
 
+        logger.debug("[hook_manager] run_stop_async: creating StopInput")
         input_data = StopInput(
             stop_hook_active=stop_hook_active,
             reason=reason,
@@ -524,7 +528,9 @@ class HookManager:
             permission_mode=self.permission_mode,
         )
 
+        logger.debug("[hook_manager] run_stop_async: calling executor.execute_hooks_async")
         outputs = await self.executor.execute_hooks_async(hooks, input_data)
+        logger.debug("[hook_manager] run_stop_async: execute_hooks_async returned")
         return HookResult(outputs)
 
     # --- Subagent Stop ---