ripperdoc 0.2.8__py3-none-any.whl → 0.2.9__py3-none-any.whl

ripperdoc/core/agents.py

@@ -278,10 +278,15 @@ def _split_frontmatter(raw_text: str) -> Tuple[Dict[str, Any], str]:
                 body = "\n".join(lines[idx + 1 :])
                 try:
                     frontmatter = yaml.safe_load(frontmatter_text) or {}
-                except (yaml.YAMLError, ValueError, TypeError) as exc:  # pragma: no cover - defensive
+                except (
+                    yaml.YAMLError,
+                    ValueError,
+                    TypeError,
+                ) as exc:  # pragma: no cover - defensive
                     logger.warning(
                         "Invalid frontmatter in agent file: %s: %s",
-                        type(exc).__name__, exc,
+                        type(exc).__name__,
+                        exc,
                         extra={"error": str(exc)},
                     )
                     return {"__error__": f"Invalid frontmatter: {exc}"}, body
@@ -312,7 +317,8 @@ def _parse_agent_file(
     except (OSError, IOError, UnicodeDecodeError) as exc:
         logger.warning(
             "Failed to read agent file: %s: %s",
-            type(exc).__name__, exc,
+            type(exc).__name__,
+            exc,
             extra={"error": str(exc), "path": str(path)},
         )
         return None, f"Failed to read agent file {path}: {exc}"

ripperdoc/core/config.py

@@ -7,8 +7,8 @@ including API keys, model settings, and user preferences.
 import json
 import os
 from pathlib import Path
-from typing import Dict, Optional, Literal
-from pydantic import BaseModel, Field
+from typing import Any, Dict, Optional, Literal
+from pydantic import BaseModel, Field, field_validator, model_validator
 from enum import Enum
 
 from ripperdoc.utils.log import get_logger
@@ -111,7 +111,7 @@ class ModelProfile(BaseModel):
     # interactions into plain text to support providers that reject tool roles.
     openai_tool_mode: Literal["native", "text"] = "native"
     # Optional override for thinking protocol handling (e.g., "deepseek", "openrouter",
-    # "qwen", "gemini_openai", "openai_reasoning"). When unset, provider heuristics are used.
+    # "qwen", "gemini_openai", "openai"). When unset, provider heuristics are used.
     thinking_mode: Optional[str] = None
     # Pricing (USD per 1M tokens). Leave as 0 to skip cost calculation.
     input_cost_per_million_tokens: float = 0.0
@@ -130,7 +130,7 @@ class ModelPointers(BaseModel):
 class GlobalConfig(BaseModel):
     """Global configuration stored in ~/.ripperdoc.json"""
 
-    model_config = {"protected_namespaces": ()}
+    model_config = {"protected_namespaces": (), "populate_by_name": True}
 
     # Model configuration
     model_profiles: Dict[str, ModelProfile] = Field(default_factory=dict)
@@ -139,7 +139,8 @@ class GlobalConfig(BaseModel):
     # User preferences
     theme: str = "dark"
     verbose: bool = False
-    safe_mode: bool = True
+    yolo_mode: bool = Field(default=False)
+    show_full_thinking: bool = Field(default=False)
     auto_compact_enabled: bool = True
     context_token_limit: Optional[int] = None
 
@@ -154,6 +155,18 @@ class GlobalConfig(BaseModel):
     # Statistics
     num_startups: int = 0
 
+    @model_validator(mode="before")
+    @classmethod
+    def _migrate_safe_mode(cls, data: Any) -> Any:
+        """Translate legacy safe_mode to the new yolo_mode flag."""
+        if isinstance(data, dict) and "safe_mode" in data and "yolo_mode" not in data:
+            data = dict(data)
+            try:
+                data["yolo_mode"] = not bool(data.pop("safe_mode"))
+            except Exception:
+                data["yolo_mode"] = False
+        return data
+
 
 class ProjectConfig(BaseModel):
     """Project-specific configuration stored in .ripperdoc/config.json"""
@@ -167,7 +180,7 @@ class ProjectConfig(BaseModel):
     # Path ignore patterns (gitignore-style)
     ignore_patterns: list[str] = Field(
         default_factory=list,
-        description="Gitignore-style patterns for paths to ignore in file operations"
+        description="Gitignore-style patterns for paths to ignore in file operations",
     )
 
     # Context
@@ -222,10 +235,18 @@ class ConfigManager:
                             "profile_count": len(self._global_config.model_profiles),
                         },
                     )
-                except (json.JSONDecodeError, OSError, IOError, UnicodeDecodeError, ValueError, TypeError) as e:
+                except (
+                    json.JSONDecodeError,
+                    OSError,
+                    IOError,
+                    UnicodeDecodeError,
+                    ValueError,
+                    TypeError,
+                ) as e:
                     logger.warning(
                         "Error loading global config: %s: %s",
-                        type(e).__name__, e,
+                        type(e).__name__,
+                        e,
                         extra={"error": str(e)},
                     )
                     self._global_config = GlobalConfig()
@@ -276,10 +297,18 @@ class ConfigManager:
                             "allowed_tools": len(self._project_config.allowed_tools),
                         },
                     )
-                except (json.JSONDecodeError, OSError, IOError, UnicodeDecodeError, ValueError, TypeError) as e:
+                except (
+                    json.JSONDecodeError,
+                    OSError,
+                    IOError,
+                    UnicodeDecodeError,
+                    ValueError,
+                    TypeError,
+                ) as e:
                     logger.warning(
                         "Error loading project config: %s: %s",
-                        type(e).__name__, e,
+                        type(e).__name__,
+                        e,
                         extra={"error": str(e), "path": str(config_path)},
                     )
                     self._project_config = ProjectConfig()
@@ -344,10 +373,18 @@ class ConfigManager:
                             "project_path": str(self.current_project_path),
                         },
                     )
-                except (json.JSONDecodeError, OSError, IOError, UnicodeDecodeError, ValueError, TypeError) as e:
+                except (
+                    json.JSONDecodeError,
+                    OSError,
+                    IOError,
+                    UnicodeDecodeError,
+                    ValueError,
+                    TypeError,
+                ) as e:
                     logger.warning(
                         "Error loading project-local config: %s: %s",
-                        type(e).__name__, e,
+                        type(e).__name__,
+                        e,
                         extra={"error": str(e), "path": str(config_path)},
                     )
                     self._project_local_config = ProjectLocalConfig()

ripperdoc/core/custom_commands.py

@@ -14,7 +14,6 @@ Features:
 
 from __future__ import annotations
 
-import asyncio
 import re
 import subprocess
 from dataclasses import dataclass, field
@@ -24,7 +23,6 @@ from typing import Any, Dict, Iterable, List, Optional, Sequence, Tuple
 
 import yaml
 
-from ripperdoc.utils.coerce import parse_boolish, parse_optional_int
 from ripperdoc.utils.log import get_logger
 
 logger = get_logger()
@@ -96,13 +94,14 @@ def _split_frontmatter(raw_text: str) -> Tuple[Dict[str, Any], str]:
         for idx in range(1, len(lines)):
             if lines[idx].strip() == "---":
                 frontmatter_text = "\n".join(lines[1:idx])
-                body = "\n".join(lines[idx + 1:])
+                body = "\n".join(lines[idx + 1 :])
                 try:
                     frontmatter = yaml.safe_load(frontmatter_text) or {}
                 except (yaml.YAMLError, ValueError, TypeError) as exc:
                     logger.warning(
                         "[custom_commands] Invalid frontmatter: %s: %s",
-                        type(exc).__name__, exc,
+                        type(exc).__name__,
+                        exc,
                     )
                     return {"__error__": f"Invalid frontmatter: {exc}"}, body
                 return frontmatter, body
@@ -163,7 +162,8 @@ def _load_command_file(
     except (OSError, IOError, UnicodeDecodeError) as exc:
         logger.warning(
             "[custom_commands] Failed to read command file: %s: %s",
-            type(exc).__name__, exc,
+            type(exc).__name__,
+            exc,
             extra={"path": str(path)},
         )
         return None, CustomCommandLoadError(path=path, reason=f"Failed to read file: {exc}")
@@ -239,7 +239,8 @@ def _load_commands_from_dir(
     except OSError as exc:
         logger.warning(
             "[custom_commands] Failed to scan command directory: %s: %s",
-            type(exc).__name__, exc,
+            type(exc).__name__,
+            exc,
             extra={"path": str(commands_dir)},
         )
 

ripperdoc/core/default_tools.py

@@ -68,11 +68,20 @@ def get_default_tools() -> List[Tool[Any, Any]]:
             if isinstance(tool, Tool):
                 base_tools.append(tool)
                 dynamic_tools.append(tool)
-    except (ImportError, ModuleNotFoundError, OSError, RuntimeError, ConnectionError, ValueError, TypeError) as exc:
+    except (
+        ImportError,
+        ModuleNotFoundError,
+        OSError,
+        RuntimeError,
+        ConnectionError,
+        ValueError,
+        TypeError,
+    ) as exc:
         # If MCP runtime is not available, continue with base tools only.
         logger.warning(
             "[default_tools] Failed to load dynamic MCP tools: %s: %s",
-            type(exc).__name__, exc,
+            type(exc).__name__,
+            exc,
         )
 
     task_tool = TaskTool(lambda: base_tools)

ripperdoc/core/hooks/config.py

@@ -234,9 +234,7 @@ def _parse_hooks_file(data: Dict[str, Any]) -> HooksConfig:
                 hook_definitions.append(hook_def)
 
             if hook_definitions:
-                parsed_matchers.append(
-                    HookMatcher(matcher=matcher_pattern, hooks=hook_definitions)
-                )
+                parsed_matchers.append(HookMatcher(matcher=matcher_pattern, hooks=hook_definitions))
 
         if parsed_matchers:
             parsed_hooks[event_name] = parsed_matchers

ripperdoc/core/hooks/events.py

@@ -5,10 +5,9 @@ as well as the input/output data structures for each event type.
 """
 
 import json
-import os
 from enum import Enum
-from typing import Any, Dict, List, Literal, Optional, Union
-from pydantic import BaseModel, Field
+from typing import Any, Dict, Literal, Optional, Union
+from pydantic import BaseModel, ConfigDict, Field
 
 
 class HookEvent(str, Enum):
@@ -73,8 +72,7 @@ class HookInput(BaseModel):
     permission_mode: str = "default"  # "default", "plan", "acceptEdits", "bypassPermissions"
     hook_event_name: str = ""
 
-    class Config:
-        populate_by_name = True
+    model_config = ConfigDict(populate_by_name=True)
 
 
 class PreToolUseInput(HookInput):
@@ -150,6 +148,8 @@ class StopInput(HookInput):
 
     hook_event_name: str = "Stop"
     stop_hook_active: bool = False  # True if already continuing from a stop hook
+    reason: Optional[str] = None
+    stop_sequence: Optional[str] = None
 
 
 class SubagentStopInput(HookInput):
@@ -210,6 +210,8 @@ class SessionEndInput(HookInput):
 
     hook_event_name: str = "SessionEnd"
     reason: str = ""  # "clear", "logout", "prompt_input_exit", "other"
+    duration_seconds: Optional[float] = None
+    message_count: Optional[int] = None
 
 
 # ─────────────────────────────────────────────────────────────────────────────
@@ -232,8 +234,7 @@ class PreToolUseHookOutput(BaseModel):
     )  # Modified tool input
     additional_context: Optional[str] = Field(default=None, alias="additionalContext")
 
-    class Config:
-        populate_by_name = True
+    model_config = ConfigDict(populate_by_name=True)
 
 
 class PermissionRequestDecision(BaseModel):
@@ -244,8 +245,7 @@ class PermissionRequestDecision(BaseModel):
     message: Optional[str] = None
     interrupt: bool = False
 
-    class Config:
-        populate_by_name = True
+    model_config = ConfigDict(populate_by_name=True)
 
 
 class PermissionRequestHookOutput(BaseModel):
@@ -254,8 +254,7 @@ class PermissionRequestHookOutput(BaseModel):
     hook_event_name: Literal["PermissionRequest"] = "PermissionRequest"
     decision: Optional[PermissionRequestDecision] = None
 
-    class Config:
-        populate_by_name = True
+    model_config = ConfigDict(populate_by_name=True)
 
 
 class PostToolUseHookOutput(BaseModel):
@@ -264,8 +263,7 @@ class PostToolUseHookOutput(BaseModel):
     hook_event_name: Literal["PostToolUse"] = "PostToolUse"
     additional_context: Optional[str] = Field(default=None, alias="additionalContext")
 
-    class Config:
-        populate_by_name = True
+    model_config = ConfigDict(populate_by_name=True)
 
 
 class UserPromptSubmitHookOutput(BaseModel):
@@ -274,8 +272,7 @@ class UserPromptSubmitHookOutput(BaseModel):
     hook_event_name: Literal["UserPromptSubmit"] = "UserPromptSubmit"
     additional_context: Optional[str] = Field(default=None, alias="additionalContext")
 
-    class Config:
-        populate_by_name = True
+    model_config = ConfigDict(populate_by_name=True)
 
 
 class SessionStartHookOutput(BaseModel):
@@ -284,8 +281,7 @@ class SessionStartHookOutput(BaseModel):
     hook_event_name: Literal["SessionStart"] = "SessionStart"
     additional_context: Optional[str] = Field(default=None, alias="additionalContext")
 
-    class Config:
-        populate_by_name = True
+    model_config = ConfigDict(populate_by_name=True)
 
 
 HookSpecificOutput = Union[
@@ -325,7 +321,7 @@ class HookOutput(BaseModel):
     )
 
     # Additional context to inject
-    additional_context: Optional[str] = None
+    additional_context: Optional[str] = Field(default=None, alias="additionalContext")
 
     # Raw output (for non-JSON responses)
     raw_output: Optional[str] = None
@@ -336,8 +332,7 @@ class HookOutput(BaseModel):
     exit_code: int = 0
     timed_out: bool = False
 
-    class Config:
-        populate_by_name = True
+    model_config = ConfigDict(populate_by_name=True)
 
     @classmethod
     def from_raw(
@@ -420,10 +415,10 @@ class HookOutput(BaseModel):
                 # Handle PreToolUse specific fields
                 if event_name == "PreToolUse":
                     output.hook_specific_output = PreToolUseHookOutput(
-                        permission_decision=hso.get("permissionDecision"),
-                        permission_decision_reason=hso.get("permissionDecisionReason"),
-                        updated_input=hso.get("updatedInput"),
-                        additional_context=hso.get("additionalContext"),
+                        permissionDecision=hso.get("permissionDecision"),
+                        permissionDecisionReason=hso.get("permissionDecisionReason"),
+                        updatedInput=hso.get("updatedInput"),
+                        additionalContext=hso.get("additionalContext"),
                     )
                     # Map permissionDecision to decision
                     perm_decision = hso.get("permissionDecision")
@@ -444,7 +439,7 @@ class HookOutput(BaseModel):
                     if isinstance(decision_obj, dict):
                         decision_data = PermissionRequestDecision(
                             behavior=decision_obj.get("behavior", ""),
-                            updated_input=decision_obj.get("updatedInput"),
+                            updatedInput=decision_obj.get("updatedInput"),
                             message=decision_obj.get("message"),
                             interrupt=decision_obj.get("interrupt", False),
                         )
@@ -462,7 +457,7 @@ class HookOutput(BaseModel):
                 # Handle PostToolUse specific fields
                 elif event_name == "PostToolUse":
                     output.hook_specific_output = PostToolUseHookOutput(
-                        additional_context=hso.get("additionalContext"),
+                        additionalContext=hso.get("additionalContext"),
                     )
                     if hso.get("additionalContext"):
                         output.additional_context = hso["additionalContext"]
@@ -470,7 +465,7 @@ class HookOutput(BaseModel):
                 # Handle UserPromptSubmit specific fields
                 elif event_name == "UserPromptSubmit":
                     output.hook_specific_output = UserPromptSubmitHookOutput(
-                        additional_context=hso.get("additionalContext"),
+                        additionalContext=hso.get("additionalContext"),
                     )
                     if hso.get("additionalContext"):
                         output.additional_context = hso["additionalContext"]
@@ -478,7 +473,7 @@ class HookOutput(BaseModel):
                 # Handle SessionStart specific fields
                 elif event_name == "SessionStart":
                     output.hook_specific_output = SessionStartHookOutput(
-                        additional_context=hso.get("additionalContext"),
+                        additionalContext=hso.get("additionalContext"),
                     )
                     if hso.get("additionalContext"):
                         output.additional_context = hso["additionalContext"]

ripperdoc/core/hooks/executor.py

@@ -14,7 +14,7 @@ import os
 import subprocess
 import tempfile
 from pathlib import Path
-from typing import Any, Callable, Dict, Optional, Awaitable
+from typing import Callable, Dict, Optional, Awaitable
 
 from ripperdoc.core.hooks.config import HookDefinition
 from ripperdoc.core.hooks.events import AnyHookInput, HookOutput, HookDecision, SessionStartInput
@@ -191,7 +191,7 @@ class HookExecutor:
             pass
 
         # Not JSON, treat as additional context
-        return HookOutput(raw_output=response, additional_context=response)
+        return HookOutput(raw_output=response, additionalContext=response)
 
     async def execute_prompt_async(
         self,
@@ -224,7 +224,7 @@ class HookExecutor:
         prompt = self._expand_prompt(hook.prompt, input_data)
 
         logger.debug(
-            f"Executing prompt hook",
+            "Executing prompt hook",
             extra={
                 "event": input_data.hook_event_name,
                 "timeout": hook.timeout,
@@ -277,9 +277,7 @@ class HookExecutor:
         """
         # Prompt hooks require async - skip in sync mode
         if hook.is_prompt_hook():
-            logger.warning(
-                "Prompt hook skipped in sync mode. Use execute_async for prompt hooks."
-            )
+            logger.warning("Prompt hook skipped in sync mode. Use execute_async for prompt hooks.")
             return HookOutput()
 
         return self._execute_command_sync(hook, input_data)

ripperdoc/core/hooks/integration.py

@@ -4,11 +4,9 @@ This module provides convenient integration points for running hooks
 as part of tool execution flows.
 """
 
-from pathlib import Path
 from typing import Any, Callable, Dict, Optional, Tuple, TypeVar, Union
 
-from ripperdoc.core.hooks.events import HookDecision
-from ripperdoc.core.hooks.manager import HookManager, HookResult, hook_manager
+from ripperdoc.core.hooks.manager import HookManager, hook_manager
 from ripperdoc.utils.log import get_logger
 
 logger = get_logger()
@@ -98,9 +96,7 @@ class HookInterceptor:
         Returns:
             Tuple of (should_continue, block_reason, additional_context)
         """
-        result = self.manager.run_post_tool_use(
-            tool_name, tool_input, tool_output, tool_error
-        )
+        result = self.manager.run_post_tool_use(tool_name, tool_input, tool_output, tool_error)
 
         if result.should_block:
             return False, result.block_reason, result.additional_context
@@ -129,7 +125,7 @@ class HookInterceptor:
         tool_name: str,
         tool_input: Dict[str, Any],
         execute_fn: Callable[[], T],
-    ) -> Tuple[bool, Union[T, str], Optional[str]]:
+    ) -> Tuple[bool, Union[T, str, None], Optional[str]]:
         """Wrap synchronous tool execution with pre/post hooks.
 
         Args:
@@ -141,9 +137,7 @@ class HookInterceptor:
             Tuple of (success, result_or_error, additional_context)
         """
         # Run pre-tool hooks
-        should_proceed, block_reason, pre_context = self.check_pre_tool_use(
-            tool_name, tool_input
-        )
+        should_proceed, block_reason, pre_context = self.check_pre_tool_use(tool_name, tool_input)
 
         if not should_proceed:
             return False, block_reason or "Blocked by hook", pre_context
@@ -157,9 +151,7 @@ class HookInterceptor:
             tool_error = str(e)
 
         # Run post-tool hooks
-        _, _, post_context = self.run_post_tool_use(
-            tool_name, tool_input, result, tool_error
-        )
+        _, _, post_context = self.run_post_tool_use(tool_name, tool_input, result, tool_error)
 
         # Combine contexts
         combined_context = None
@@ -168,7 +160,7 @@ class HookInterceptor:
             combined_context = "\n".join(parts) if parts else None
 
         if tool_error:
-            return False, tool_error, combined_context
+            return False, tool_error or "", combined_context
 
         return True, result, combined_context
 
@@ -177,7 +169,7 @@ class HookInterceptor:
         tool_name: str,
         tool_input: Dict[str, Any],
         execute_fn: Callable[[], T],
-    ) -> Tuple[bool, Union[T, str], Optional[str]]:
+    ) -> Tuple[bool, Union[T, str, None], Optional[str]]:
         """Wrap async tool execution with pre/post hooks."""
         # Run pre-tool hooks
         should_proceed, block_reason, pre_context = await self.check_pre_tool_use_async(
@@ -190,6 +182,7 @@ class HookInterceptor:
         # Execute the tool
         try:
             import asyncio
+
             if asyncio.iscoroutinefunction(execute_fn):
                 result = await execute_fn()
             else:
@@ -211,7 +204,7 @@ class HookInterceptor:
             combined_context = "\n".join(parts) if parts else None
 
         if tool_error:
-            return False, tool_error, combined_context
+            return False, tool_error or "", combined_context
 
         return True, result, combined_context
 
@@ -241,9 +234,7 @@ def run_post_tool_use(
     tool_error: Optional[str] = None,
 ) -> Tuple[bool, Optional[str], Optional[str]]:
     """Convenience function to run post-tool hooks using global interceptor."""
-    return hook_interceptor.run_post_tool_use(
-        tool_name, tool_input, tool_output, tool_error
-    )
+    return hook_interceptor.run_post_tool_use(tool_name, tool_input, tool_output, tool_error)
 
 
 async def run_post_tool_use_async(
@@ -333,7 +324,7 @@ def check_stop(
         - should_stop: True if agent should stop
         - continue_reason: Reason to continue if blocked
     """
-    result = hook_manager.run_stop(reason, stop_sequence)
+    result = hook_manager.run_stop(False, reason, stop_sequence)
 
     if result.should_block:
         return False, result.block_reason
@@ -345,7 +336,7 @@ async def check_stop_async(
     reason: Optional[str] = None, stop_sequence: Optional[str] = None
 ) -> Tuple[bool, Optional[str]]:
     """Async version of check_stop."""
-    result = await hook_manager.run_stop_async(reason, stop_sequence)
+    result = await hook_manager.run_stop_async(False, reason, stop_sequence)
 
     if result.should_block:
         return False, result.block_reason